TPAC: Fix org_unit location detection for login collab/dbwells/lp1013672_tpac_auth_proxy
authorDan Wells <dbw2@calvin.edu>
Tue, 24 Jul 2012 16:39:49 +0000 (12:39 -0400)
committerDan Wells <dbw2@calvin.edu>
Tue, 24 Jul 2012 16:39:49 +0000 (12:39 -0400)
commitbfb36e2a8610ceee200ca6c8300892e2ff03eecd
treeac46b7269db46d6187ba317b602c59f0dd36daef
parent532d7350de0497375d3b3fa9452bf3dd1e127af6
TPAC: Fix org_unit location detection for login

The TPAC login function currently looks for a 'loc' param to pass
to the login service.  This isn't right because:

A. 'loc' is a search-related param, and shouldn't affect logins
B. The login form doesn't actually pass through the 'loc' param
C. The 'loc' param has been more or less replaced by 'locg'

No, this isn't multiple choice, it's all of the above.  Our best
option at this point is to use the 'physical_loc' value instead.
This matches what was done in JSPAC (though it was called 'ol' for
"original location" back then).

Finally, this commit is 95% the work of Dan Scott, including the
additions of logging info and a few whitespace fixes.

Signed-off-by: Dan Wells <dbw2@calvin.edu>
Open-ILS/src/perlmods/lib/OpenILS/WWW/EGCatLoader.pm