[Jifty-commit] r4201 - in jifty/trunk: lib/Jifty/Action/Record
jifty-commit at lists.jifty.org
jifty-commit at lists.jifty.org
Wed Oct 3 14:43:28 EDT 2007
Author: sterling
Date: Wed Oct 3 14:43:26 2007
New Revision: 4201
Modified:
jifty/trunk/ (props changed)
jifty/trunk/lib/Jifty/Action/Record/Update.pm
Log:
r12827 at riddle: andrew | 2007-10-03 13:41:53 -0500
Use blessed() instead of ref() to keep an if statement from tripping on ->isa().
Modified: jifty/trunk/lib/Jifty/Action/Record/Update.pm
==============================================================================
--- jifty/trunk/lib/Jifty/Action/Record/Update.pm (original)
+++ jifty/trunk/lib/Jifty/Action/Record/Update.pm Wed Oct 3 14:43:26 2007
@@ -18,6 +18,8 @@
use base qw/Jifty::Action::Record/;
+use Scalar::Util qw/ blessed /;
+
=head1 METHODS
=head2 arguments
@@ -125,7 +127,7 @@
# Skip fields that have not changed
my $old = $self->record->$field;
# XXX TODO: This ignore "by" on columns
- $old = $old->id if ref($old) and $old->isa( 'Jifty::Record' );
+ $old = $old->id if blessed($old) and $old->isa( 'Jifty::Record' );
# if both the new and old values are defined and equal, we don't want to change em
# XXX TODO "$old" is a cheap hack to scalarize datetime objects
More information about the Jifty-commit
mailing list