Merge commit '2.11+2.111' into HEAD

[maypole.git] / lib / Maypole / Model / Base.pm

diff --git a/lib/Maypole/Model/Base.pm b/lib/Maypole/Model/Base.pm
index 338f0e8a85449808cc5e9954a581cd5dceaa9c29..d5d325c6dfaaf374414ac728ec9f894e9b2de1a3 100644 (file)
--- a/lib/Maypole/Model/Base.pm
+++ b/lib/Maypole/Model/Base.pm
@@ -133,6 +133,9 @@ Empty Action.
 
 Empty Action.
 
+=item index
+
+Empty Action, calls list if provided with a table.
 
 =back
 
@@ -148,6 +151,14 @@ sub view : Exported {
 sub edit : Exported {
 }
 
+sub index : Exported {
+    my ( $self, $r ) = @_;
+    if ($r->table) {
+       $r->template("list");
+       return $self->list($r);
+    } 
+}
+
 =pod
 
 Also, see the exported commands in C<Maypole::Model::CDBI>.
@@ -189,19 +200,22 @@ Defaults to checking if the sub has the C<:Exported> attribute.
 
 =cut
 
-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;
+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
@@ -209,16 +223,15 @@ defines the C<Exported> attribute.
 
 =cut
 
-sub method_attrs
-{
-    my ($class, $method) = @_;
+sub method_attrs {
+    my ($class, $method, $cv) = @_;
     
-    my $cv = $class->can($method);
+    $cv ||= $class->can($method);
     
     return unless $cv;
     
     my @attrs = attributes::get($cv);
-    
+
     return @attrs;
 }