[Jifty-commit] r2171 - in jifty/trunk: .

[jifty-commit at lists.jifty.org]

Author: jesse
Date: Thu Nov 16 15:37:29 2006
New Revision: 2171

Modified:
   jifty/trunk/   (props changed)
   jifty/trunk/lib/Jifty/Param/Schema.pm

Log:
 r45080 at 109:  jesse | 2006-11-16 14:36:19 -0500
 * Jifty::Param::Schema merge algorithm rescued from the obscurity of commit logs.


Modified: jifty/trunk/lib/Jifty/Param/Schema.pm
==============================================================================
--- jifty/trunk/lib/Jifty/Param/Schema.pm	(original)
+++ jifty/trunk/lib/Jifty/Param/Schema.pm	Thu Nov 16 15:37:29 2006
@@ -153,8 +153,14 @@
 
 Takes two hashrefs. Merges them together and returns the merged hashref.
 
+    - Empty fields in subclasses don't override nonempty fields in superclass anymore.
+    - Arrays don't merge; e.g. if parent class's valid_values is [1,2,3,4], and
+      subclass's valid_values() is [1,2], they don't somehow become [1,2,3,4,1,2].
+
 BUG: This should either be a private routine or factored out into Jifty::Util
 
+
+
 =cut
 
 sub merge_params {