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

[jifty-commit at lists.jifty.org]

Author: sterling
Date: Mon Jul  2 11:25:40 2007
New Revision: 3600

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

Log:
 r7925 at dynpc145:  andrew | 2007-07-02 10:20:41 -0500
 Added a needed true return in the ModelClassColumn::after_create handler.


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	Mon Jul  2 11:25:40 2007
@@ -161,6 +161,7 @@
     $self->model_class->add_column($self);
     my $ret = Jifty->handle->simple_query( $self->model_class->qualified_class->add_column_sql( $self->name ) );
     $ret || $self->log->fatal( "error updating a table: " . $ret->error_message);
+    return 1;
 }
 
 =head2 table

Modified: jifty/branches/virtual-models/lib/Jifty/Record.pm
==============================================================================
--- jifty/branches/virtual-models/lib/Jifty/Record.pm	(original)
+++ jifty/branches/virtual-models/lib/Jifty/Record.pm	Mon Jul  2 11:25:40 2007
@@ -96,7 +96,7 @@
                 $attribs{$column_name} = $record;
             }
 
-            # XXX sterling: this should log something and quit
+            # log something and quit
             else {
                 $self->log->error("The UUID for $column_name was not found in the database.");
                 if ($class) {