[Jifty-commit] r4534 - jifty/branches/prototype-1.6/lib/Jifty

jifty-commit at lists.jifty.org jifty-commit at lists.jifty.org
Sun Nov 25 09:47:43 EST 2007


Author: clkao
Date: Sun Nov 25 09:47:43 2007
New Revision: 4534

Modified:
   jifty/branches/prototype-1.6/lib/Jifty/ClassLoader.pm

Log:
In ClassLoader, if an action matches Create* that isn't about
creating a model, don't just return but continue to see if there's
any matching actions from plugins.

This solves CreateOpenIDUser in particular.


Modified: jifty/branches/prototype-1.6/lib/Jifty/ClassLoader.pm
==============================================================================
--- jifty/branches/prototype-1.6/lib/Jifty/ClassLoader.pm	(original)
+++ jifty/branches/prototype-1.6/lib/Jifty/ClassLoader.pm	Sun Nov 25 09:47:43 2007
@@ -215,14 +215,15 @@
         Jifty::Util->_require( module => $modelclass, quiet => 1);
 
         # Don't generate the action unless it really is a model
-        return undef unless eval { $modelclass->isa('Jifty::Record') };
+        if ( eval { $modelclass->isa('Jifty::Record') } ) {
 
-        return $self->return_class(
+            return $self->return_class(
                   "package $module;\n"
                 . "use base qw/$base\::Action::Record::$1/;\n"
                 . "sub record_class { '$modelclass' };\n"
                 . "sub _autogenerated { 1 };\n"
             );
+        }
 
     }
 


More information about the Jifty-commit mailing list