[prev in list] [next in list] [prev in thread] [next in thread] 

List:       rt-commit
Subject:    [Rt-commit] r19385 - in rt/3.8/trunk:
From:       sartak () bestpractical ! com
Date:       2009-04-30 21:20:24
Message-ID: 20090430212024.E5BD34DD00F () diesel ! bestpractical ! com
[Download RAW message or body]

Author: sartak
Date: Thu Apr 30 17:20:24 2009
New Revision: 19385

Modified:
   rt/3.8/trunk/   (props changed)
   rt/3.8/trunk/share/html/REST/1.0/Forms/ticket/default

Log:
 r82601@onn:  sartak | 2009-04-30 17:20:16 -0400
 More informative comment


Modified: rt/3.8/trunk/share/html/REST/1.0/Forms/ticket/default
==============================================================================
--- rt/3.8/trunk/share/html/REST/1.0/Forms/ticket/default	(original)
+++ rt/3.8/trunk/share/html/REST/1.0/Forms/ticket/default	Thu Apr 30 17:20:24 2009
@@ -286,9 +286,11 @@
         elsif (exists $dates{$key}) {
             $key = $dates{$key};
 
-            # LastUpdated might be updated automatically based on
-            # nondeterministic ordering (by changing any other field), so
-            # always just skip updating it to avoid "Immutable field" warnings
+            # We try to detect whether it should update a field by checking
+            # whether its current value equals the entered value. Since the
+            # LastUpdated field is automatically updated as other columns are
+            # changed, it is not properly skipped. Users cannot update this
+            # field anyway.
             next if $key eq 'LastUpdated';
 
             $set = "Set$key";
_______________________________________________
Rt-commit mailing list
Rt-commit@lists.bestpractical.com
http://lists.bestpractical.com/cgi-bin/mailman/listinfo/rt-commit
[prev in list] [next in list] [prev in thread] [next in thread]