[KSS-checkins] r49441 - in kukit/kukit.js/trunk: doc kukit

[reebalazs at codespeak.net]

Author: reebalazs
Date: Thu Dec  6 11:50:55 2007
New Revision: 49441

Modified:
   kukit/kukit.js/trunk/doc/HISTORY.txt
   kukit/kukit.js/trunk/kukit/forms.js
Log:
 Fix multiple selection form fields marshalling on Safari
(fixes #22 in kssproject)

Modified: kukit/kukit.js/trunk/doc/HISTORY.txt
==============================================================================
--- kukit/kukit.js/trunk/doc/HISTORY.txt	(original)
+++ kukit/kukit.js/trunk/doc/HISTORY.txt	Thu Dec  6 11:50:55 2007
@@ -6,6 +6,11 @@
 
     - ...
 
+    - Fix multiple selection form fields
+      marshalling on Safari 
+      (fixes #22 in kssproject)
+      [ree]
+
     - Fix error fallback handling
       [ree]
 

Modified: kukit/kukit.js/trunk/kukit/forms.js
==============================================================================
--- kukit/kukit.js/trunk/kukit/forms.js	(original)
+++ kukit/kukit.js/trunk/kukit/forms.js	Thu Dec  6 11:50:55 2007
@@ -62,8 +62,11 @@
         var elem = new _FormQueryElem(name, value);
         this.l.push(elem);
     }
+    // value.length is for detection of an Array.
+    // In addition we also check that value.pop is a function
     else if (typeof(value) == 'object' && 
-        value.constructor.toString().indexOf('Array') != -1) {
+        typeof(value.length) == 'number' &&
+        typeof(value.pop) == 'function') {
         // Special marshalling of arrays
         for (var i=0; i < value.length; i++) {
             var elem = new _FormQueryElem(name, value[i]);