(sitka) [rt15292] Hold policy display showing non-relevant policies
authorLiam Whalen <liam.whalen@bc.libraries.coop>
Fri, 25 Oct 2013 16:43:47 +0000 (09:43 -0700)
committerLiam Whalen <liam.whalen@bc.libraries.coop>
Thu, 31 Oct 2013 05:18:40 +0000 (22:18 -0700)
Currently, when a user loads the Hold policy display via Admin->Local
Administration->Hold Policies, the hold polices from 5 OUs were always
being loaded into the grid regardless of which OU the workstation was
assigned.  Once loaded, if the user selected a different OU then
returned to their OU, the gird would load properly.

This error was due to the fact that the inital load of the hold polices
was loading all policies without considering the workstation's OU.
However, this initial load was looking for policies where the
circ_modifier was NOT NULL.  In our current database, there are only 5
OUs with polices that are not null.  Those OUs are  455, 630, 96, 50,
and 97.  These were the policies showing up in the inital load that were
not relevant to the workstation's OU.

This fix adds a limit to the inital load if there is an OU defined for
the User's worksation.  I imagine this is true at all times in the staff
client, but I retained the previous code in an else just in case there
might be a situation where an OU is not defined that I am not aware of.

Open-ILS/src/templates/conify/global/config/hold_matrix_matchpoint.tt2

index cd6e9e3..5120a8e 100644 (file)
     dojo.require('dijit.layout.ContentPane');
     dojo.require('dijit.form.Button');
     dojo.require('openils.widget.AutoGrid');
-    openils.Util.addOnLoad(
-        function(){
-            hmGrid.loadAll({order_by:{chmm:'circ_modifier'}});
-        }
-    );
-</script>
-[% END %]
+    var org_id = openils.User.user.ws_ou();
+    var list = fieldmapper.aou.findOrgUnit(org_id).orgNodeTrail().map(
+        function (i) {return i.id() } );
+
+    if (list.size()) {
+        openils.Util.addOnLoad(
+            function(){
+                hmGrid.loadAll({order_by:{chmm:'circ_modifier'}}, {item_owning_ou:list});
+            }
+        );
+    } else {
+        openils.Util.addOnLoad(
+            function(){
+                hmGrid.loadAll({order_by:{chmm:'circ_modifier'}});
+            }
+        );
+    }
+</script> [% END %]