package Maypole::Model::Base;
+use strict;
use Maypole::Constants;
use attributes ();
+# don't know why this is a global - drb
our %remember;
-sub MODIFY_CODE_ATTRIBUTES { $remember{ $_[1] } = $_[2]; () }
+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 FETCH_CODE_ATTRIBUTES { @{ $remember{$_[1]} || [] } }
sub process {
my ( $class, $r ) = @_;
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} } );
}
=head1 DESCRIPTION
This is the base class for Maypole data models. This is an abstract class
-meant to define the interface, and can't be used directly.
+that defines the interface, and can't be used directly.
=head2 process
-This is the engine of this module. It populates all the relevant variables
-and calls the requested action.
+This is the engine of this module. Given the request object, it populates
+all the relevant variables and calls the requested action.
Anyone subclassing this for a different database abstraction mechanism
needs to provide the following methods:
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 C<adopt>ed. It should also populate C<< $config->{tables} >> and
-C<< $config->{classes} >> with the names of the classes and tables
+then be C<adopt>ed. 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<beer> should be mapped to the class
C<BeerDB::Beer>.
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.
+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
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 Commands
+=head2 Actions
=over
sub do_edit { die "This is an abstract method" }
-
=item list
-The C<list> method should fill C<< $r-> objects >> with all of the
+The C<list> method should fill C<$r-E<gt>objects> with all of the
objects in the class. You may want to page this using C<Data::Page> or
similar.
=item edit
-Empty Action
+Empty Action.
=item view
=cut
-
sub list : Exported {
die "This is an abstract method";
}
=head2 display_columns
-Returns a list of columns to display in the model. by default returns
+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.
} $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) = @_;
+
+ my %attrs = map {$_ => 1} $self->method_attrs($action);
+
+ return 1 if $attrs{Exported};
+
+ warn "'$action' not exported";
+
+ return 0;
+}
-=head2 is_public
+=head2 method_attrs
-should return true if a certain action is supported, or false otherwise.
-Defaults to checking if the sub has the :Exported attribute.
+Returns the list of attributes defined for a method. Maypole itself only
+defines the C<Exported> attribute.
=cut
-sub is_public {
- my ( $self, $action ) = @_;
- my $cv = $self->can($action);
- return 0 unless $cv;
- my $attrs = join " ", attributes::get($cv);
- do {
- warn "$action not exported" if Maypole->debug;
- return 0;
- } unless $attrs =~ /\bExported\b/i;
- return 1;
+sub method_attrs
+{
+ my ($class, $method) = @_;
+
+ my $cv = $class->can($method);
+
+ return unless $cv;
+
+ my @attrs = attributes::get($cv);
+
+ return @attrs;
}
=head2 related
}
1;
+
+
projects / maypole.git / blobdiff