X-Git-Url: http://wagnertech.de/git?a=blobdiff_plain;f=SL%2FController%2FTopQuickSearch.pm;h=03d6504438a8534d761d835a603b460de8e5021a;hb=d4c5ff04f33e7fe29e079c4489c4126a7ea2493c;hp=8c6661e833e1963af5b3db380ab84c641c6625bd;hpb=c8c1042a337f37b85c1dc3d550e3cbd7bc57e9d8;p=kivitendo-erp.git diff --git a/SL/Controller/TopQuickSearch.pm b/SL/Controller/TopQuickSearch.pm index 8c6661e83..03d650443 100644 --- a/SL/Controller/TopQuickSearch.pm +++ b/SL/Controller/TopQuickSearch.pm @@ -6,6 +6,7 @@ use parent qw(SL::Controller::Base); use SL::ClientJS; use SL::JSON; use SL::Locale::String qw(t8); +use SL::Helper::UserPreferences; use Rose::Object::MakeMethods::Generic ( 'scalar --get_set_init' => [ qw(module js) ], @@ -16,8 +17,17 @@ my @available_modules = ( 'SL::Controller::TopQuickSearch::Part', 'SL::Controller::TopQuickSearch::Service', 'SL::Controller::TopQuickSearch::Assembly', + 'SL::Controller::TopQuickSearch::Assortment', 'SL::Controller::TopQuickSearch::Contact', + 'SL::Controller::TopQuickSearch::SalesQuotation', + 'SL::Controller::TopQuickSearch::SalesOrder', + 'SL::Controller::TopQuickSearch::SalesDeliveryOrder', + 'SL::Controller::TopQuickSearch::RequestForQuotation', + 'SL::Controller::TopQuickSearch::PurchaseOrder', + 'SL::Controller::TopQuickSearch::PurchaseDeliveryOrder', 'SL::Controller::TopQuickSearch::GLTransaction', + 'SL::Controller::TopQuickSearch::Customer', + 'SL::Controller::TopQuickSearch::Vendor', ); my %modules_by_name; @@ -57,12 +67,31 @@ sub available_modules { map { $_->new } @available_modules; } -sub active_modules { +sub enabled_modules { + my $user_prefs = SL::Helper::UserPreferences->new( + namespace => 'TopQuickSearch', + ); + + my @quick_search_modules; + if (my $prefs_val = $user_prefs->get('quick_search_modules')) { + @quick_search_modules = split ',', $prefs_val; + } else { + @quick_search_modules = @{ $::instance_conf->get_quick_search_modules }; + } + + my %enabled_names = map { $_ => 1 } @quick_search_modules; + grep { - $::auth->assert($_->auth, 1) + $enabled_names{$_->name} } $_[0]->available_modules } +sub active_modules { + grep { + !$_->auth || $::auth->assert($_->auth, 1) + } $_[0]->enabled_modules +} + sub init_module { my ($self) = @_; @@ -72,7 +101,7 @@ sub init_module { die 'Unknown module ' . $::form->{module} unless my $class = $modules_by_name{$::form->{module}}; - $::auth->assert($class->auth); + $::auth->assert($class->auth) if $class->auth; return $class->new; } @@ -105,21 +134,22 @@ SL::Controller::TopQuickSearch - Framework for pluggable quicksearch fields in t =head1 SYNOPSIS -use SL::Controller::TopQuickSearch; -my $search = SL::Controller::TopQuickSearch->new; + use SL::Controller::TopQuickSearch; + my $search = SL::Controller::TopQuickSearch->new; + $::request->layout->add_javascripts('kivi.QuickSearch.js'); -# in layout -[%- FOREACH module = search.available_modules %] - -[%- END %] + # in template + [%- FOREACH module = search.enabled_modules %] + + [%- END %] =head1 DESCRIPTION This controller provides abstraction for different search plugins, and ensures that all follow a common useability scheme. -Modules should be configurable, but currently are not. Diabling modules can be -done by removing them from available_modules. +Modules should be configurable per user, but currently are not. Disabling +modules can be done by removing them from available_modules or in client_config. =head1 BEHAVIOUR REQUIREMENTS @@ -157,26 +187,19 @@ redirect should occur. Each search must check rights and must not present a backdoor into data that the user should not see. -=back +=item * -=head1 INTERFACE +By design the search must not try to guess C. -Plugins need to provide: +=back - - name - - localized description for config - - localized description for textfield - - autocomplete callback - - redirect callback +=head1 INTERFACE -the frontend will only generate urls of the forms: - action=TopQuickSearch/autocomplete&module=&term= - action=TopQuickSearch/search&module=&term= +The full interface is described in L =head1 TODO - - filter available searches with auth - - toggling with cofiguration doesn't work yet + * user configuration =head1 BUGS