[Jifty-commit] r4313 - in Jifty-DBI/trunk: lib/Jifty/DBI

[jifty-commit at lists.jifty.org]

Author: sartak
Date: Thu Oct 25 14:37:25 2007
New Revision: 4313

Modified:
   Jifty-DBI/trunk/   (props changed)
   Jifty-DBI/trunk/lib/Jifty/DBI/Record.pm

Log:
 r44182 at onn:  sartak | 2007-10-25 14:37:13 -0400
 Add a warning for mandatory+till until we fix it


Modified: Jifty-DBI/trunk/lib/Jifty/DBI/Record.pm
==============================================================================
--- Jifty-DBI/trunk/lib/Jifty/DBI/Record.pm	(original)
+++ Jifty-DBI/trunk/lib/Jifty/DBI/Record.pm	Thu Oct 25 14:37:25 2007
@@ -1225,11 +1225,13 @@
             $attribs{$column->name} = $column->default;
         }
 
-        next unless $column->active;
-
         if (not defined $attribs{$column->name} and $column->mandatory and $column->type ne "serial" ) {
             # Enforce "mandatory"
             Carp::carp "Did not supply value for mandatory column ".$column->name;
+            unless ($column->active) {
+                Carp::carp "The mandatory column ".$column->name." is no longer active. This is likely to cause problems!";
+            }
+
             return ( 0 );
         }
     }