[Jifty-commit] r1320 - in jifty/trunk: lib/Jifty/Action/Record

jifty-commit at lists.jifty.org jifty-commit at lists.jifty.org
Mon Jun 19 18:15:44 EDT 2006


Author: jesse
Date: Mon Jun 19 18:15:44 2006
New Revision: 1320

Modified:
   jifty/trunk/   (props changed)
   jifty/trunk/lib/Jifty/Action/Record/Create.pm

Log:
 r12761 at pinglin:  jesse | 2006-06-19 12:15:38 -0400
 * Backed out seanmil's "Skip nonexistent columns" change, as it breaks 
   model classes that have custom (non-database-backed) arguments.
 
 * All the validation of arguments/columns we need is taken care of by the loop 
   we're currently inside


Modified: jifty/trunk/lib/Jifty/Action/Record/Create.pm
==============================================================================
--- jifty/trunk/lib/Jifty/Action/Record/Create.pm	(original)
+++ jifty/trunk/lib/Jifty/Action/Record/Create.pm	Mon Jun 19 18:15:44 2006
@@ -58,9 +58,6 @@
 
     my %values;
     for (grep { defined $self->argument_value($_) } $self->argument_names) {
-        # Skip nonexistent fields
-        next unless $self->record->column($_);
-
         $values{$_} = $self->argument_value($_);
         if (ref $values{$_} eq "Fh") { # CGI.pm's "lightweight filehandle class"
             local $/;


More information about the Jifty-commit mailing list