Author: jmorliaguet
Date: Sat Feb 11 16:33:05 2006
New Revision: 2354

Modified:
   cpsskins/branches/jmo-perspectives/ui/framework/ctal.js
   
cpsskins/branches/jmo-perspectives/ui/framework/tests/unit/cpsskins_ctal_test.html
Log:

- more robust code (e.g. Opera) - test updates



Modified: cpsskins/branches/jmo-perspectives/ui/framework/ctal.js
==============================================================================
--- cpsskins/branches/jmo-perspectives/ui/framework/ctal.js     (original)
+++ cpsskins/branches/jmo-perspectives/ui/framework/ctal.js     Sat Feb 11 
16:33:05 2006
@@ -170,11 +170,16 @@
   if (tmpl.nodeType == 1) {
     var parsers = ctal.parsers;
     for (ctal_attr in parsers) {
-      var attrnode = tmpl.getAttribute(ctal_attr);
-      if (attrnode) {
+      var attrs = tmpl.attributes;
+      for (i=0; i<attrs.length; i++) {
+        var attrnode = attrs[i];
+        var attrname = attrnode.nodeName;
+        if (attrname != ctal_attr) {
+          continue;
+        }
         tmpl.removeAttribute(ctal_attr);
         var parser = parsers[ctal_attr];
-        parser(tmpl, attrnode, data)
+        parser(tmpl, attrnode.value, data)
       }
     }
   }

Modified: 
cpsskins/branches/jmo-perspectives/ui/framework/tests/unit/cpsskins_ctal_test.html
==============================================================================
--- 
cpsskins/branches/jmo-perspectives/ui/framework/tests/unit/cpsskins_ctal_test.html
  (original)
+++ 
cpsskins/branches/jmo-perspectives/ui/framework/tests/unit/cpsskins_ctal_test.html
  Sat Feb 11 16:33:05 2006
@@ -9,6 +9,7 @@
   <meta http-equiv="content-type" content="text/html; charset=utf-8" />
   <script src="../../prototype.js" type="text/javascript"></script>
   <script src="../../ctal.js" type="text/javascript"></script>
+  <script src="../../cpsskins.js" type="text/javascript"></script>
   <script src="../unittest.js" type="text/javascript"></script>
   <link rel="stylesheet" href="../test.css" type="text/css" />
 </head>
@@ -252,37 +253,6 @@
      ctal:repeat="item f"
      ctal:content="item">...</div></div>
 
-<!-- ctal:attributes -->
-<div id="e30"><span
-     ctal:attributes="color string:red">...</span></div>
-
-<div id="e30b"><span
-     ctal:attributes="color string:red;">...</span></div>
-
-<div id="e30c"><span
-     ctal:attributes="color string:red ;">...</span></div>
-
-<div id="e30d"><span
-     ctal:attributes="color string:red; class string: large">...</span></div>
-
-<div id="e30e"><span
-     ctal:attributes="color string:red; class string: large;">...</span></div>
-
-<div id="e30f"><span
-     ctal:attributes=" color string:red; class string: large; 
">...</span></div>
-
-<div id="e30g"><span
-     ctal:attributes="color string: red; class string: large; 
">...</span></div>
-
-<!-- javascript: -->
-<div id="e31"><span ctal:content="javascript:1">...</span></div>
-
-<div id="e31b"><span ctal:content="javascript:1+1">...</span></div>
-
-<div id="e31c"><span
-                ctal:content="javascript:data['k']['l']['m']">...</span></div>
-
-<!-- ctal:attributes -->
 
 <!-- Tests follow -->
 <script type="text/javascript">
@@ -460,26 +430,6 @@
       assertEqual('<div>4</div><div>5</div><div>6</div>', $("e28").innerHTML);
 
       assertEqual('', $("e29").innerHTML);
-    }},
-
-    testTALAttributes: function() { with(this) {
-      assertEqual('<span color="red">...</span>', $("e30").innerHTML);
-      assertEqual('<span color="red">...</span>', $("e30b").innerHTML);
-      assertEqual('<span color="red">...</span>', $("e30c").innerHTML);
-      assertEqual('<span class="large" color="red">...</span>',
-                  $("e30d").innerHTML);
-      assertEqual('<span class="large" color="red">...</span>',
-                  $("e30e").innerHTML);
-      assertEqual('<span class="large" color="red">...</span>',
-                  $("e30f").innerHTML);
-      assertEqual('<span class="large" color=" red">...</span>',
-                  $("e30g").innerHTML);
-    }},
-
-    testJavascript: function() { with(this) {
-      assertEqual('<span>1</span>', $("e31").innerHTML);
-      assertEqual('<span>2</span>', $("e31b").innerHTML);
-      assertEqual('<span>10</span>', $("e31c").innerHTML);
     }}
 
   });
-- 
http://lists.nuxeo.com/mailman/listinfo/z3lab-checkins

Reply via email to