[Jifty-commit] r4536 - jifty/trunk/lib/Jifty

jifty-commit at lists.jifty.org jifty-commit at lists.jifty.org
Sun Nov 25 17:39:42 EST 2007


Author: clkao
Date: Sun Nov 25 17:39:42 2007
New Revision: 4536

Modified:
   jifty/trunk/lib/Jifty/ClassLoader.pm

Log:
r4534 belongs to trunk.

 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/trunk/lib/Jifty/ClassLoader.pm
==============================================================================
--- jifty/trunk/lib/Jifty/ClassLoader.pm	(original)
+++ jifty/trunk/lib/Jifty/ClassLoader.pm	Sun Nov 25 17:39:42 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