[jifty-devel] Re: [Jifty-commit] r1144 - in jifty/trunk: lib/Jifty/Action/Record

[Jesse Vincent]

On Fri, Jun 02, 2006 at 02:26:32PM -0400, Sean E. Millichamp wrote:
> On Fri, 2006-06-02 at 14:13 -0400, Jesse Vincent wrote:
> > Do we want to be testing for truth or definedness? I'm betting ths will
> > stop you from creating a record with a value of 0.
> 
> > On Fri, Jun 02, 2006 at 01:48:34PM -0400, jifty-commit at lists.jifty.org wrote:
> 
> > >      for (grep { defined $self->argument_value($_) } $self->argument_names) {
> > > +        # Skip nonexistent fields
> > > +        next unless $self->record->column($_);
> > > +
> 
> It is my understanding that $self->record->column(FIELD_NAME) returns a
> Jifty::DBI::Column reference if the column named FIELD_NAME exists in
> the record and undef if the column does not exist.
> 
> I borrowed this code straight from Jifty::Action::Record::Update so that
> creating records with column type 'password' wouldn't choke on the
> auto-generated 'password_confirm' argument field.

I may well have been two quick on the draw. yeah. that looks sane.

> Sean
> 
> -- 
>  Sean E. Millichamp, Enertron LLC, 586-757-1200 x102
> 



> _______________________________________________
> jifty-devel mailing list
> jifty-devel at lists.jifty.org
> http://lists.jifty.org/cgi-bin/mailman/listinfo/jifty-devel


--