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

[jifty-commit at lists.jifty.org]

Author: sterling
Date: Fri Aug 24 21:53:26 2007
New Revision: 3976

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

Log:
 r8955 at riddle:  andrew | 2007-08-24 20:52:50 -0500
 Cleaning up and fixing problems with late column trigger registration.


Modified: jifty/branches/virtual-models/lib/Jifty/Model/ModelClassColumn.pm
==============================================================================
--- jifty/branches/virtual-models/lib/Jifty/Model/ModelClassColumn.pm	(original)
+++ jifty/branches/virtual-models/lib/Jifty/Model/ModelClassColumn.pm	Fri Aug 24 21:53:26 2007
@@ -195,11 +195,13 @@
     $self->load_by_cols(id => $$idref);
     $self->model_class->add_column($self);
     unless ($self->virtual) {
-        my $ret = Jifty->handle->simple_query( $self->model_class->qualified_class->add_column_sql( $self->name ) );
-        for my $mixin (@{ $self->RECORD_MIXINS || [] }) {
+        my $class = $self->model_class->qualified_class;
+        my $ret = Jifty->handle->simple_query( $class->add_column_sql( $self->name ) );
+        my $mixins = $class->RECORD_MIXINS || [];
+        for my $mixin (@$mixins) {
             if (my $triggers_for_column 
-                    = $self->can('register_triggers_for_column')) {
-                $triggers_for_column->($self, $self->name);
+                    = $mixin->can('register_triggers_for_column')) {
+                $triggers_for_column->($class, $self->name);
             }
         }
         $ret || $self->log->fatal( "error updating a table: " . $ret->error_message);