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

svn at svn.open-ils.org svn at svn.open-ils.org
Thu Aug 12 13:50:07 EDT 2010


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

Modified:
   branches/rel_1_6/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/Open-ILS/web/js/ui/default/actor/user/register.js
===================================================================
--- branches/rel_1_6/Open-ILS/web/js/ui/default/actor/user/register.js	2010-08-12 16:07:55 UTC (rev 17189)
+++ branches/rel_1_6/Open-ILS/web/js/ui/default/actor/user/register.js	2010-08-12 17:50:03 UTC (rev 17190)
@@ -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) };
     );
 }
 



More information about the open-ils-commits mailing list