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

[Jifty commits]

Author: alexmv
Date: Tue Apr  1 14:33:47 2008
New Revision: 5252

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

Log:
 r28953 at kohr-ah:  chmrr | 2008-04-01 14:33:38 -0400
  * Skip ajax canonicalization on checkboxes
  * Perltidy


Modified: jifty/trunk/lib/Jifty/Action/Record.pm
==============================================================================
--- jifty/trunk/lib/Jifty/Action/Record.pm	(original)
+++ jifty/trunk/lib/Jifty/Action/Record.pm	Tue Apr  1 14:33:47 2008
@@ -357,28 +357,30 @@
 
 
 sub _argument_canonicalizer {
-    my $self = shift;
+    my $self   = shift;
     my $column = shift;
-    my $field = $column->name;
+    my $field  = $column->name;
     my $method;
     my $do_ajax = 0;
 
+    # Add a canonicalizer for the column if the record provides one
+    if ( $self->record->has_canonicalizer_for_column($field) ) {
+        $do_ajax = 1 unless lc($column->render_as) eq 'checkbox';
+        $method ||= sub {
+            my ( $self, $value ) = @_;
+            return $self->record->run_canonicalization_for_column(
+                column => $field,
+                value  => $value
+            );
+        };
+    }
 
-        # Add a canonicalizer for the column if the record provides one
-        if ( $self->record->has_canonicalizer_for_column($field) ) {
-            $do_ajax = 1;
-            $method ||= sub {
-                my ( $self, $value ) = @_;
-                return $self->record->run_canonicalization_for_column(column => $field, value => $value);
-            };
-        } 
-        
-        # Otherwise, if it's a date, we have a built-in canonicalizer for that
-        elsif ( lc($column->render_as) eq 'date') {
-            $do_ajax = 1;
-        }
-    return ($method, $do_ajax);
+    # Otherwise, if it's a date, we have a built-in canonicalizer for that
+    elsif ( lc( $column->render_as ) eq 'date' ) {
+        $do_ajax = 1;
     }
+    return ( $method, $do_ajax );
+}
 
 sub _argument_autocompleter {
     my $self = shift;