X-Git-Url: https://git.decadent.org.uk/gitweb/?a=blobdiff_plain;f=lib%2FMaypole%2FModel%2FBase.pm;h=de33ac98c991137b24712b86d947553629d2f01e;hb=f19715f56244cc6d862169c2dd656b8a2f3845b5;hp=804f23519d0a808b1000d621660fe2b010d4f0fc;hpb=5ba93a09518403ae28e8a71b5299e2458a6dfa0b;p=maypole.git diff --git a/lib/Maypole/Model/Base.pm b/lib/Maypole/Model/Base.pm index 804f235..de33ac9 100644 --- a/lib/Maypole/Model/Base.pm +++ b/lib/Maypole/Model/Base.pm @@ -1,30 +1,42 @@ package Maypole::Model::Base; -our %remember; -sub MODIFY_CODE_ATTRIBUTES { $remember{ $_[1] } = $_[2]; () } -sub FETCH_CODE_ATTRIBUTES { $remember{ $_[1] } } +use strict; +use Maypole::Constants; +use attributes (); -sub view : Exported { -} +# don't know why this is a global - drb +our %remember; -sub edit : Exported { +sub MODIFY_CODE_ATTRIBUTES +{ + shift; # class name not used + my ($coderef, @attrs) = @_; + + $remember{$coderef} = \@attrs; + + # previous version took care to return an empty array, not sure why, + # but shall cargo cult it until know better + return; } +sub FETCH_CODE_ATTRIBUTES { @{ $remember{$_[1]} || [] } } + sub process { my ( $class, $r ) = @_; my $method = $r->action; return if $r->{template}; # Authentication has set this, we're done. $r->{template} = $method; - $r->objects( [] ); - my $obj = $class->retrieve( $r->{args}->[0] ); - if ($obj) { - $r->objects( [$obj] ); - shift @{ $r->{args} }; - } + my $obj = $class->fetch_objects($r); + $r->objects([$obj]) if $obj; + $class->$method( $r, $obj, @{ $r->{args} } ); } +sub list_columns { + shift->display_columns; +} + sub display_columns { sort shift->columns; } @@ -35,20 +47,16 @@ Maypole::Model::Base - Base class for model classes =head1 DESCRIPTION -Anyone subclassing this for a different database abstraction mechanism -needs to provide the following methods: +This is the base class for Maypole data models. This is an abstract class +that defines the interface, and can't be used directly. -=head2 do_edit +=head2 process -If there is an object in C<$r-Eobjects>, then it should be edited -with the parameters in C<$r-Eparams>; otherwise, a new object should -be created with those parameters, and put back into C<$r-Eobjects>. -The template should be changed to C, or C if there were any -errors. A hash of errors will be passed to the template. - -=cut +This is the engine of this module. Given the request object, it populates +all the relevant variables and calls the requested action. -sub do_edit { die "This is an abstract method" } +Anyone subclassing this for a different database abstraction mechanism +needs to provide the following methods: =head2 setup_database @@ -57,8 +65,8 @@ sub do_edit { die "This is an abstract method" } Uses the user-defined data in C<@data> to specify a database- for example, by passing in a DSN. The model class should open the database, and create a class for each table in the database. These classes will -then be Ced. It should also populate C<< $config->{tables} >> and -C<< $config->{classes} >> with the names of the classes and tables +then be Ced. It should also populate C<< $config->tables >> and +C<< $config->classes >> with the names of the classes and tables respectively. The classes should be placed under the specified namespace. For instance, C should be mapped to the class C. @@ -69,52 +77,77 @@ C. This maps between a table name and its associated class. -=head2 retrieve +=head2 fetch_objects -This turns an ID into an object of the appropriate class. +This class method is passed a request object and is expected to return an +object of the appropriate table class from information stored in the request +object. =head2 adopt -This is called on an model class representing a table and allows the -master model class to do any set-up required. - -=head2 related - -This can go either in the master model class or in the individual -classes, and returns a list of has-many accessors. A brewery has many -beers, so C needs to return C. +This class method is passed the name of a model class that represensts a table +and allows the master model class to do any set-up required. =head2 columns This is a list of all the columns in a table. You may also override -C, which is the list of columns you want to view, in -the right order. +see also C =head2 table This is the name of the table. -=head2 Commands +=cut + +sub class_of { die "This is an abstract method" } +sub setup_database { die "This is an abstract method" } +sub fetch_objects { die "This is an abstract method" } + +=head2 Actions =over +=item do_edit + +If there is an object in C<$r-Eobjects>, then it should be edited +with the parameters in C<$r-Eparams>; otherwise, a new object should +be created with those parameters, and put back into C<$r-Eobjects>. +The template should be changed to C, or C if there were any +errors. A hash of errors will be passed to the template. + +=cut + +sub do_edit { die "This is an abstract method" } + =item list -The C method should fill C<< $r-> objects >> with all of the +The C method should fill C<$r-Eobjects> with all of the objects in the class. You may want to page this using C or similar. +=item edit + +Empty Action. + +=item view + +Empty Action. + + =back =cut -sub class_of { die "This is an abstract method" } -sub setup_database { die "This is an abstract method" } - sub list : Exported { die "This is an abstract method"; } +sub view : Exported { +} + +sub edit : Exported { +} + =pod Also, see the exported commands in C. @@ -124,6 +157,16 @@ Also, see the exported commands in C. Additionally, individual derived model classes may want to override the following methods: +=head2 display_columns + +Returns a list of columns to display in the model. By default returns +all columns in alphabetical order. Override this in base classes to +change ordering, or elect not to show columns. + +=head2 list_columns + +Same as display_columns, only for listings. Defaults to display_columns + =head2 column_names Return a hash mapping column names with human-readable equivalents. @@ -139,13 +182,59 @@ sub column_names { } $class->columns; } -=head2 description +=head2 is_public -A description of the class to be passed to the template. +should return true if a certain action is supported, or false otherwise. +Defaults to checking if the sub has the C<:Exported> attribute. =cut -sub description { "A poorly defined class" } +sub is_public { + my ( $self, $action, $attrs ) = @_; + my $cv = $self->can($action); + warn "is_public failed . action is $action. self is $self" and return 0 unless $cv; + + my %attrs = (ref $attrs) ? %$attrs : map {$_ => 1} $self->method_attrs($action,$cv) ; + + do { + warn "is_public failed. $action not exported. attributes are : ", %attrs; + return 0; + } unless $attrs{Exported}; + return 1; +} + + + +=head2 method_attrs + +Returns the list of attributes defined for a method. Maypole itself only +defines the C attribute. + +=cut + +sub method_attrs { + my ($class, $method, $cv) = @_; + + $cv ||= $class->can($method); + + return unless $cv; + + my @attrs = attributes::get($cv); + + return @attrs; +} + +=head2 related + +This can go either in the master model class or in the individual +classes, and returns a list of has-many accessors. A brewery has many +beers, so C needs to return C. + +=cut + +sub related { +} 1; +