[Checkins] SVN: zope.app.form/trunk/ Fixed invalid markup.

[Malthe Borch]

Log message for revision 99847:
  Fixed invalid markup.

Changed:
  U   zope.app.form/trunk/CHANGES.txt
  U   zope.app.form/trunk/src/zope/app/form/browser/orderedSelectionList.pt

-=-
Modified: zope.app.form/trunk/CHANGES.txt
===================================================================
--- zope.app.form/trunk/CHANGES.txt	2009-05-11 15:16:22 UTC (rev 99846)
+++ zope.app.form/trunk/CHANGES.txt	2009-05-11 16:36:39 UTC (rev 99847)
@@ -5,7 +5,7 @@
 3.7.3 (unreleased)
 ==================
 
-- ...
+- Fixed invalid markup.
 
 3.7.2 (2009-03-12)
 ==================

Modified: zope.app.form/trunk/src/zope/app/form/browser/orderedSelectionList.pt
===================================================================
--- zope.app.form/trunk/src/zope/app/form/browser/orderedSelectionList.pt	2009-05-11 15:16:22 UTC (rev 99846)
+++ zope.app.form/trunk/src/zope/app/form/browser/orderedSelectionList.pt	2009-05-11 16:36:39 UTC (rev 99847)
@@ -69,7 +69,7 @@
       selectionError();
   else if (toSel.options[0].selected)
       alert("Cannot move further up!");
-  else for (var i = 0; i < toSel.length; i++)
+  else for (var i = 0; toSel.length > i; i++)
     if (toSel.options[i].selected)
       {
       swapFields(toSel.options[i-1], toSel.options[i]);
@@ -110,7 +110,7 @@
 
   // create new hidden input fields - one for each selection item of
   // "to" selection
-  for (var i = 0; i < toSel.options.length; i++)
+  for (var i = 0; toSel.options.length > i; i++)
     {
     // create virtual node with suitable attributes
     var newNode = document.createElement("input");