use UNIVERSAL::require;
use strict;
use warnings;
-our $VERSION = "1.3";
+our $VERSION = "1.8";
__PACKAGE__->mk_classdata($_) for qw( config init_done view_object );
-__PACKAGE__->mk_accessors ( qw( ar params query objects model_class
-args action template ));
-__PACKAGE__->config({});
+__PACKAGE__->mk_accessors(
+ qw( ar params query objects model_class
+ args action template )
+);
+__PACKAGE__->config( {} );
__PACKAGE__->init_done(0);
-
-# Ape Apache::Constants interface
-use constant OK => 0;
-use constant DECLINED => -1;
+use Maypole::Constants;
sub debug { 0 }
my $calling_class = shift;
$calling_class = ref $calling_class if ref $calling_class;
{
- no strict 'refs';
- # Naughty.
- *{$calling_class."::handler"} = sub { Maypole::handler($calling_class, @_) };
+ no strict 'refs';
+
+ # Naughty.
+ *{ $calling_class . "::handler" } =
+ sub { Maypole::handler( $calling_class, @_ ) };
}
my $config = $calling_class->config;
$config->{model} ||= "Maypole::Model::CDBI";
$config->{model}->require;
die "Couldn't load the model class $config->{model}: $@" if $@;
- $config->{model}->setup_database($config, $calling_class, @_);
- for my $subclass (@{$config->{classes}}) {
+ $config->{model}->setup_database( $config, $calling_class, @_ );
+ for my $subclass ( @{ $config->{classes} } ) {
no strict 'refs';
- unshift @{$subclass."::ISA"}, $config->{model};
+ unshift @{ $subclass . "::ISA" }, $config->{model};
$config->{model}->adopt($subclass)
- if $config->{model}->can("adopt");
+ if $config->{model}->can("adopt");
}
}
sub init {
- my $class = shift;
+ my $class = shift;
my $config = $class->config;
- $config->{view} ||= "Maypole::View::TT";
+ $config->{view} ||= "Maypole::View::TT";
$config->{view}->require;
die "Couldn't load the view class $config->{view}: $@" if $@;
- $config->{display_tables} ||= [ @{$class->config->{tables}} ];
- $class->view_object($class->config->{view}->new);
+ $config->{display_tables} ||= [ @{ $class->config->{tables} } ];
+ $class->view_object( $class->config->{view}->new );
$class->init_done(1);
}
sub handler {
+
# See Maypole::Workflow before trying to understand this.
- my $class = shift;
+ my ( $class, $req ) = @_;
$class->init unless $class->init_done;
my $r = bless { config => $class->config }, $class;
- $r->get_request();
+ $r->get_request($req);
$r->parse_location();
+ my $status = $r->handler_guts();
+ return $status unless $status == OK;
+ $r->send_output;
+ return $status;
+}
- $r->model_class($r->config->{model}->class_of($r, $r->{table}));
- my $status = $r->is_applicable;
- if ($status == OK) {
- $status = $r->call_authenticate;
- if ($r->debug and $status != OK and $status != DECLINED) {
- $r->view_object->error($r,
- "Got unexpected status $status from calling authentication");
- }
- return $status unless $status == OK;
- $r->additional_data();
-
- $r->model_class->process($r);
- } else {
- # Otherwise, it's just a plain template.
- $r->call_authenticate; # No harm in it
+sub handler_guts {
+ my $r = shift;
+ $r->model_class( $r->config->{model}->class_of( $r, $r->{table} ) );
+ my $applicable = $r->is_applicable;
+ unless ( $applicable == OK ) {
+
+ # It's just a plain template
delete $r->{model_class};
- $r->{path} =~ s{/}{}; # De-absolutify
- $r->template($r->{path});
+ $r->{path} =~ s{/$}{}; # De-absolutify
+ $r->template( $r->{path} );
}
- $status = OK;
- if (!$r->{output}) { # You might want to do it yourself
- $status = $r->view_object->process($r);
+
+ # We authenticate every request, needed for proper session management
+ my $status;
+ eval { $status = $r->call_authenticate };
+ if ( my $error = $@ ) {
+ $status = $r->call_exception($error);
+ if ( $status != OK ) {
+ warn "caught model error: $error";
+ return $r->debug ? $r->view_object->error( $r, $error ) : ERROR;
+ }
}
- $r->send_output;
- return $status;
+ if ( $r->debug and $status != OK and $status != DECLINED ) {
+ $r->view_object->error( $r,
+ "Got unexpected status $status from calling authentication" );
+ }
+ return $status unless $status == OK;
+
+ # We run additional_data for every request
+ $r->additional_data;
+ if ( $applicable == OK ) {
+ eval { $r->model_class->process($r) };
+ if ( my $error = $@ ) {
+ $status = $r->call_exception($error);
+ if ( $status != OK ) {
+ warn "caught model error: $error";
+ return $r->debug ? $r->view_object->error( $r, $error ) : ERROR;
+ }
+ }
+ }
+ if ( !$r->{output} ) { # You might want to do it yourself
+ return $r->view_object->process($r);
+ }
+ else { return OK; }
}
sub is_applicable {
- my $self = shift;
+ my $self = shift;
my $config = $self->config;
- $config->{ok_tables} = {map {$_ => 1} @{$config->{display_tables}}};
+ $config->{ok_tables} ||= $config->{display_tables};
+ $config->{ok_tables} = { map { $_ => 1 } @{ $config->{ok_tables} } }
+ if ref $config->{ok_tables} eq "ARRAY";
warn "We don't have that table ($self->{table})"
- if $self->debug and not $config->{ok_tables}{$self->{table}};
- return DECLINED() unless exists $config->{ok_tables}{$self->{table}};
+ if $self->debug
+ and not $config->{ok_tables}{ $self->{table} };
+ return DECLINED() unless exists $config->{ok_tables}{ $self->{table} };
# Does the action method exist?
- my $cv = $self->model_class->can($self->{action});
- warn "We don't have that action ($self->{action})"
- if $self->debug and not $cv;
+ my $cv = $self->model_class->can( $self->{action} );
+ warn "We don't have that action ($self->{action})"
+ if $self->debug and not $cv;
return DECLINED() unless $cv;
# Is it exported?
$self->{method_attribs} = join " ", attributes::get($cv);
- do { warn "$self->{action} not exported" if $self->debug;
- return DECLINED()
- } unless $self->{method_attribs} =~ /\bExported\b/i;
+ do {
+ warn "$self->{action} not exported" if $self->debug;
+ return DECLINED();
+ } unless $self->{method_attribs} =~ /\bExported\b/i;
return OK();
}
sub call_authenticate {
my $self = shift;
- return $self->model_class->authenticate($self) if
- $self->model_class->can("authenticate");
- return $self->authenticate($self); # Interface consistency is a Good Thing
+
+ # Check if we have a model class
+ if ( $self->{model_class} ) {
+ return $self->model_class->authenticate($self)
+ if $self->model_class->can("authenticate");
+ }
+ return $self->authenticate($self); # Interface consistency is a Good Thing
}
-sub additional_data {}
+sub call_exception {
+ my $self = shift;
+ my ($error) = @_;
+
+ # Check if we have a model class
+ if ( $self->{model_class}
+ && $self->model_class->can('exception') )
+ {
+ my $status = $self->model_class->exception( $self, $error );
+ return $status if $status == OK;
+ }
+ return $self->exception($error);
+}
+
+sub additional_data { }
sub authenticate { return OK }
+sub exception { return ERROR }
+
sub parse_path {
my $self = shift;
$self->{path} ||= "frontpage";
my @pi = split /\//, $self->{path};
shift @pi while @pi and !$pi[0];
- $self->{table} = shift @pi;
+ $self->{table} = shift @pi;
$self->{action} = shift @pi;
- $self->{args} = \@pi;
+ $self->{args} = \@pi;
}
=head1 NAME
=cut
sub get_template_root { "." }
-sub get_request { }
-sub parse_location { die "Do not use Maypole directly; use Apache::MVC or similar" }
-sub send_output{ die "Do not use Maypole directly; use Apache::MVC or similar" }
+sub get_request { }
+
+sub parse_location {
+ die "Do not use Maypole directly; use Apache::MVC or similar";
+}
+
+sub send_output {
+ die "Do not use Maypole directly; use Apache::MVC or similar";
+}
=head1 SEE ALSO
L<Apache::MVC>, L<CGI::Maypole>.
+=head1 MAINTAINER
+
+Sebastian Riedel, c<sri@oook.de>
+
=head1 AUTHOR
Simon Cozens, C<simon@cpan.org>
+=head1 THANK YOU
+
+Jesse Scheidlower, Jody Belka, Markus Ramberg, Mickael Joanne, Simon Flack,
+Veljko Vidovic and all the others who've helped.
+
=head1 LICENSE
You may distribute this code under the same terms as Perl itself.
=cut
1;
-