[KSS-checkins] r50398 - in kukit/kukit.js/branch/ree-1.4-markup-and-syntax-changes: . kukit

reebalazs at codespeak.net reebalazs at codespeak.net
Sun Jan 6 17:19:36 CET 2008


Author: reebalazs
Date: Sun Jan  6 17:19:36 2008
New Revision: 50398

Modified:
   kukit/kukit.js/branch/ree-1.4-markup-and-syntax-changes/   (props changed)
   kukit/kukit.js/branch/ree-1.4-markup-and-syntax-changes/kukit/kssparser.js
Log:
Fix handling of kss special parameters in production mode.
preprocessValues is now doing value.check(), which is
needed always, so preprocessValues also needs to be called
in production mode.

Modified: kukit/kukit.js/branch/ree-1.4-markup-and-syntax-changes/kukit/kssparser.js
==============================================================================
--- kukit/kukit.js/branch/ree-1.4-markup-and-syntax-changes/kukit/kssparser.js	(original)
+++ kukit/kukit.js/branch/ree-1.4-markup-and-syntax-changes/kukit/kssparser.js	Sun Jan  6 17:19:36 2008
@@ -328,9 +328,11 @@
 ;;;     }
 ;;;     // We ignore actual results here, and just check. 
 ;;;     kukit.E = 'kss action parameter [' + key + ']';
-;;;     // last parameter is true: means we do _not_ require the existence
-;;;     // of a string type.
-;;;     this.preprocessValues(values, allowedReturnTypes, kukit.E, true);
+        // Call preprocessValues in both production and development mode:
+        // it is always needed, since it calls check() on the value.
+        // Last parameter is true: means we do _not_ require the existence
+        // of a string type.
+        this.preprocessValues(values, allowedReturnTypes, kukit.E, true);
     } else {
         // Normal selectors: can have more values
         // check its return types


More information about the Kukit-checkins mailing list