X-Git-Url: http://wagnertech.de/git?a=blobdiff_plain;f=SL%2FController%2FHelper%2FGetModels.pm;h=481f1ee2984feea7252a7f22396e9142eafe8f7a;hb=cb1341a6715ae92b278b49d8b14eee7d4018859e;hp=a4cd973b8977aa8404ac9de251006c342503d4b9;hpb=95f9f85a003a94c70a8daaaef7aef91522050fa6;p=kivitendo-erp.git diff --git a/SL/Controller/Helper/GetModels.pm b/SL/Controller/Helper/GetModels.pm index a4cd973b8..481f1ee29 100644 --- a/SL/Controller/Helper/GetModels.pm +++ b/SL/Controller/Helper/GetModels.pm @@ -63,11 +63,28 @@ sub get_paginate_args { $self->paginated->get_current_paginate_params(%params); } +sub get_sort_spec { + my ($self) = @_; + + $self->sorted->specs; +} + +sub get_current_sort_params { + my ($self) = @_; + + $self->sorted->read_params; +} + sub init { my ($self, %params) = @_; - # TODO: default model - $self->model(delete $params{model}); + my $model = delete $params{model}; + if (!$model && $params{controller} && ref $params{controller}) { + $model = ref $params{controller}; + $model =~ s/.*:://; + die 'Need a valid model' unless $model; + } + $self->model($model); my @plugins; for my $plugin (qw(filtered sorted paginated)) { @@ -107,11 +124,10 @@ sub register_handlers { map { push @{ $handlers->{$_} }, $additional_handlers{$_} if $additional_handlers{$_} } keys %$handlers; } -# TODO fix this sub get_models_url_params { - my ($class, $sub_name_or_code) = @_; + my ($self, $sub_name_or_code) = @_; - my $code = (ref($sub_name_or_code) || '') eq 'CODE' ? $sub_name_or_code : sub { shift->$sub_name_or_code(@_) }; + my $code = (ref($sub_name_or_code) || '') eq 'CODE' ? $sub_name_or_code : sub { shift->controller->$sub_name_or_code(@_) }; my $callback = sub { my ($self, %params) = @_; my @additional_params = $code->($self); @@ -121,7 +137,7 @@ sub get_models_url_params { ); }; - push @{ _registered_handlers($class)->{callback} }, $callback; + $self->registere_handlers('callback' => $callback); } sub get_callback { @@ -176,112 +192,221 @@ __END__ =head1 NAME -SL::Controller::Helper::GetModels - Base mixin for controller helpers -dealing with semi-automatic handling of sorting and paginating lists +SL::Controller::Helper::GetModels - Base class for a the GetModels system. =head1 SYNOPSIS -For a proper synopsis see L. +In controller: -=head1 OVERVIEW + use SL::Controller::Helper::GetModels; + + my $get_models = SL::Controller::Helper::GetModels->new( + controller => $self, + ); -For a generic overview see L. + my $models = $self->get_models->get; -This base module is the interface between a controller and specialized -helper modules that handle things like sorting and paginating. The -specialized helpers register themselves with this module via a call to -L during compilation time (e.g. in the -case of C this happens when the controller calls -L). +=head1 OVERVIEW + +Building a CRUD controller would be easy, were it not for those stupid +list actions. People unreasonable expect stuff like filtering, sorting, +paginating, exporting etc simply to work. Well, lets try to make it simply work +a little. -A controller will later usually call the L -function. Templates will call the L function. Both -functions run the registered handlers handing over control to the -specialized helpers so that they may inject their parameters into the -call chain. +This class is a proxy between a controller and specialized +helper modules that handle these things (sorting, paginating etc) and gives you +the means to retrieve the information when needed to display sort headers or +paginating footers. -The C helper hooks into the controller call to the action -via a C hook. This is done so that it can remember the -action called by the user. This is used for constructing the callback -in L. +Information about the requested data query can be stored into the object up to +a certain point, from which on the object becomes locked and can only be +accessed for information. (See C). -=head1 PACKAGE FUNCTIONS +=head1 INTERFACE METHODS =over 4 -=item C +=item new PARAMS + +Create a new GetModels object. Params must have at least an entry +C, other than that, see C for options. + +=item get + +Retrieve all models for the current configuration. Will finalize the object. + +=item get_models_url_params SUB -Register one of the controller's subs to be called whenever an URL has -to be generated (e.g. for sort and pagination links). This is a way -for the controller to add additional parameters to the URL (e.g. for -filter parameters). +Register a sub to be called whenever an URL has to be generated (e.g. for sort +and pagination links). This is a way for the controller to add additional +parameters to the URL (e.g. for filter parameters). -The C<$sub> parameter can be either a code reference or the name of +The parameter can be either a code reference or the name of one of the controller's functions. -The value returned by this C<$sub> must be either a single hash +The value returned by C must be either a single hash reference or a hash of key/value pairs to add to the URL. -=item C +=item get_callback + +Returns a URL suitable for use as a callback parameter. It maps to the +current controller and action. All registered handlers of type +'callback' (e.g. the ones by C and C) can inject +the parameters they need so that the same list view as is currently +visible can be re-rendered. + +Optional C<%params> passed to this function may override any parameter +set by the registered handlers. + +=item enable_plugin PLUGIN + +=item disable_plugin PLUGIN + +=item is_enabled_plugin PLUGIN + +Enable or disable the specified plugin. Useful to disable paginating for +exports for example. C can be used to check the current +state fo a plugin. + +Must not be finalized to use this. + +=item finalize + +Forces finalized state. Can be used on finalized objects without error. + +Note that most higher functions will call this themselves to force a finalized +state. If you do use it it must come before any other finalizing methods, and +will most likely function as a reminder or maintainers where your codes +switches from configuration to finalized state. -This function should only be called from other controller helpers like -C or C. It is not exported and must therefore be -called its full name. The first parameter C<$class> must be the actual -controller's class name. +=item source HASHREF -If C<%handlers> contains a key C then it is passed to the hook -registration in L. +The source for user supplied information. Defaults to $::form. Changing it +after C phase has no effect. -The C<%handlers> register callback functions in the specialized -controller helpers that are called during invocation of -L or L. Possible keys are C and -C. +=item controller CONTROLLER -Each handler (the value in the hash) can be either a code reference -(in which case it is called directly) or the name of an instance -function callable on a controller instance. In both cases the handler -receives a hash of parameters built during this very call to -L or L respectively. The handler's return -value must be the new hash to be used in calls to further handlers and -to the actual database model functions later on. +A weakened link to the controller that created the GetModels object. Needed for +certain plugin methods. =back -=head1 INSTANCE FUNCTIONS +=head1 DELEGATION METHODS + +All of these finalize. + +Methods delegating to C: =over 4 -=item C +=item * -Return an URL suitable for use as a callback parameter. It maps to the -current controller and action. All registered handlers of type -'callback' (e.g. the ones by C and C) can inject -the parameters they need so that the same list view as is currently -visible can be re-rendered. +set_report_generator_sort_options -Optional C<%params> passed to this function may override any parameter -set by the registered handlers. +=item * -=item C +get_sort_spec -Query the model manager via C and return its result. The -parameters to C are constructed by calling all registered -handlers of type 'models' (e.g. the ones by C and -C). +=item * -Optional C<%params> passed to this function may override any parameter -set by the registered handlers. +get_current_sort_params + +=back + +Methods delegating to C: + +=over 4 + +=item * + +get_paginate_args + +=back + +=head1 STATES + +A GetModels object is in one of 3 states at any given time. Their purpose is to +make a class of bugs impossible that orginated from changing the configuration +of a GetModels object halfway during the request. This was a huge problem in +the old implementation. + +=over 4 + +=item Base + +This is the state after creating a new object. + +=item Init + +In this state every information needed from the source ($::form) has been read +and subsequent changes to the source have no effect. In the current +implementation this will happen during creation, so that the return value of +C is already in state C. + +=item Finalized + +In this state no new configuration will be accepted so that information gotten +through the various methods is consistent. Every information retrieval method +will trigger finalize. + +=back + + +=head1 CONFIGURATION -The return value is the an array reference of C models. +Most of the configuration will be handed to GetModels on creation via C. +This is a list of accepted params. + +=over 4 + +=item controller SELF + +The creating controller. Currently this is mandatory. + +=item model MODEL + +The name of the model for this GetModels instance. If none is given, the model +is inferred from the name of the controller class. + +=item sorted PARAMS + +=item paginated PARAMS + +=item filtered PARAMS + +Configuration for plugins. If the option for any plugin is omitted, it defaults +to enabled and configured by default. Giving a falsish value as first argument +will disable the plugin. + +If the value is a hashref, it will be passed to the plugin's C method. + +=item query + +=item with_objects + +Additional static parts for Rose to include into the final query. + +=item source + +Source for plugins to pull their data from. Defaults to $::form. =back -=head1 BUGS +=head1 BUGS AND CAVEATS -Nothing here yet. +=over 4 -=head1 AUTHOR +=item * + +Delegation is not as clean as it should be. Most of the methods rely on action +at a distance and should be moved out. + +=back + +=head1 AUTHORS Moritz Bunkus Em.bunkus@linet-services.deE +Sven Schöling Es.schoeling@linet-services.deE + =cut