This works either way. list.append( row_params ) will return a modified row_params...

git-svn-id: svn://svn.open-ils.org/ILS/trunk@13775 dcc99617-32d9-48b4-a31d-7c20da2025e4

diff --git a/Open-ILS/xul/staff_client/server/patron/standing_penalties.js b/Open-ILS/xul/staff_client/server/patron/standing_penalties.js
index 622deb2..1c54e89 100644 (file)
--- a/Open-ILS/xul/staff_client/server/patron/standing_penalties.js
+++ b/Open-ILS/xul/staff_client/server/patron/standing_penalties.js
@@ -93,8 +93,7 @@ function populate_list() {
                     }
                 }
             };
-            rows[ xulG.patron.standing_penalties()[i].id() ] = function(p){ return p; }(row_params); // careful with vars in loops
-            list.append( row_params );
+            rows[ xulG.patron.standing_penalties()[i].id() ] = list.append( row_params );
         };
 
     } catch(E) {
@@ -160,8 +159,7 @@ function generate_request_handler_for_penalty_apply(penalty,id) {
                         }
                     }
                 };
-                rows[ penalty.id() ] = row_params;
-                list.append( row_params );
+                rows[ penalty.id() ] = list.append( row_params );
             }
             /*
             if (xulG && typeof xulG.refresh == 'function') {