[Jifty-commit] r7449 - jifty/trunk/lib/Jifty/Action/Record

Jifty commits jifty-commit at lists.jifty.org
Tue Aug 25 21:16:29 EDT 2009


Author: sartak
Date: Tue Aug 25 21:16:29 2009
New Revision: 7449

Modified:
   jifty/trunk/lib/Jifty/Action/Record/Update.pm

Log:
Render PKs as read-only in update actions

    This may affect your application! If you do not specify which
    parameters to render for update actions, this will begin rendering
    id as a read-only field.

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	Tue Aug 25 21:16:29 2009
@@ -46,11 +46,9 @@
     for my $pk (@{ $self->record->_primary_keys }) {
         $arguments->{$pk}{'constructor'} = 1;
         $arguments->{$pk}{'mandatory'} = 1;
-        # XXX TODO IS THERE A BETTER WAY TO NOT RENDER AN ITEM IN arguments
-        $arguments->{$pk}{'render_as'} = 'Unrendered'; 
-        # primary key fields should always be hidden fields
+        $arguments->{$pk}{'render_mode'} = 'read';
     }
-    
+
     if ( $self->can('PARAMS') ) {
         use Jifty::Param::Schema;
         return Jifty::Param::Schema::merge_params(


More information about the Jifty-commit mailing list