[open-ils-commits] r17191 - branches/rel_1_6_1/Open-ILS/web/js/ui/default/actor/user (dbs)

[svn at svn.open-ils.org]

Author: dbs
Date: 2010-08-12 13:50:32 -0400 (Thu, 12 Aug 2010)
New Revision: 17191

Modified:
   branches/rel_1_6_1/Open-ILS/web/js/ui/default/actor/user/register.js
Log:
Enable "Delete address" button to work in rel_1_6

In some ways the inverse of r17023, the else clause that enabled
the delete button to be enabled was never being reached because
the preceding clauses caught all cases.

Moving it outside of the foreach() loop entirely seems to resolve the problem.


Modified: branches/rel_1_6_1/Open-ILS/web/js/ui/default/actor/user/register.js
===================================================================
--- branches/rel_1_6_1/Open-ILS/web/js/ui/default/actor/user/register.js	2010-08-12 17:50:03 UTC (rev 17190)
+++ branches/rel_1_6_1/Open-ILS/web/js/ui/default/actor/user/register.js	2010-08-12 17:50:32 UTC (rev 17191)
@@ -721,11 +721,10 @@
                 if(mkLinks || (patron.mailing_address() && patron.mailing_address().id() == id))
                     ma.checked = true;
 
-            } else {
-                var btn = dojo.query('[name=delete-button]', row)[0];
-                if(btn) btn.onclick = function(){ uEditDeleteAddr(id) };
-            }
         }
+
+	var btn = dojo.query('[name=delete-button]', row)[0];
+	if(btn) btn.onclick = function(){ uEditDeleteAddr(id) };
     );
 }