From: miker <miker@dcc99617-32d9-48b4-a31d-7c20da2025e4>
Date: Mon, 29 Nov 2010 17:45:17 +0000 (+0000)
Subject: Patch from Jeff Godin, backported from trunkified version:
X-Git-Url: https://old-git.evergreen-ils.org/?a=commitdiff_plain;h=d851b1f0c78c19515c6918b41c7fe039fc6610af;p=working%2FEvergreen.git

Patch from Jeff Godin, backported from trunkified version:

In the staff client, the holds context menu option "Edit PickupLibrary" displays and allows selection of org units that it shouldn't.

Org units whose type has can_have_users = FALSE are not valid pickup locations.

Using the default types as an example, CONS and SYS1 are invalid pickup locations.

The OPAC enforces this when allowing patrons to edit a hold.

The staff client does not enforce this when staff edit a hold.

The can_have_users() method of the au object returns a string 't' or 'f' which is then compared to 0.

The attached patch corrects this issue. ( foo == 0 becomes !isTrue(foo) ).



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

diff --git a/Open-ILS/xul/staff_client/server/patron/holds.js b/Open-ILS/xul/staff_client/server/patron/holds.js
index 2aee909493..72e74f6c02 100644
--- a/Open-ILS/xul/staff_client/server/patron/holds.js
+++ b/Open-ILS/xul/staff_client/server/patron/holds.js
@@ -346,7 +346,7 @@ patron.holds.prototype = {
                                         return [
                                             o.name() ? sname + ' ' + o.name() : o.shortname(),
                                             o.id(),
-                                            ( obj.data.hash.aout[ o.ou_type() ].can_have_users() == 0),
+                                            ( !isTrue(obj.data.hash.aout[ o.ou_type() ].can_have_users()) ),
                                             ( obj.data.hash.aout[ o.ou_type() ].depth() * 2),
                                         ]; 
                                     }