[Jifty-commit] r4574 - in jifty/branches/virtual-models: .

jifty-commit at lists.jifty.org jifty-commit at lists.jifty.org
Thu Nov 29 16:29:45 EST 2007


Author: sterling
Date: Thu Nov 29 16:29:45 2007
New Revision: 4574

Modified:
   jifty/branches/virtual-models/   (props changed)
   jifty/branches/virtual-models/lib/Jifty/Model/ModelClass.pm

Log:
 r14287 at dynpc145:  andrew | 2007-11-29 15:29:21 -0600
 Adding an additional call to register_triggers_for_column() on mixins when columns are instantiated since that happens after the mixins are imported.


Modified: jifty/branches/virtual-models/lib/Jifty/Model/ModelClass.pm
==============================================================================
--- jifty/branches/virtual-models/lib/Jifty/Model/ModelClass.pm	(original)
+++ jifty/branches/virtual-models/lib/Jifty/Model/ModelClass.pm	Thu Nov 29 16:29:45 2007
@@ -179,6 +179,16 @@
     }
 
     $self->qualified_class->_init_methods_for_column($column);
+
+    # Since columns are added after the mixins have already been imported, we
+    # need to make sure to call register_triggers_for_column() ourselves.
+    my $mixins = $self->qualified_class->RECORD_MIXINS || [];
+    for my $mixin (@$mixins) {
+        if (my $triggers_for_column
+                = $mixin->can('register_triggers_for_column')) {
+            $triggers_for_column->($self->qualified_class, $column->name);
+        }
+    }
 }
 
 


More information about the Jifty-commit mailing list