Comment 7 for bug 1259196

Revision history for this message
Jason Boyer (jboyer) wrote :

I've added support to SIPServer to send the Location (CP) field when signing in (LP 1579144) and here's a patch to make use of this in Evergreen: http://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/jboyer/lp1259196_sip_workstation

To test you'll need a SIP client that allows you to specify the Location (CP) field on login. (This is distinct from the Institution ID and is unique to the Login (93) message) You'll also want to log at INFO or lower to verify the results.

Testing:
Pre-patch:
Sign in with the CP field empty.
Perform one or more circulations.
No workstation will be set when looking at the logs or the circulations.
Set the CP field to the name of an existing workstation and repeat the above steps. The results will be the same.

Post-patch:
Sign in with the CP field empty.
Perform one or more circulations.
No workstation will be set when looking at the logs or the circulations.
Set the CP field to the name of an existing workstation and repeat the above steps. In the logs the workstation will be shown when the client logged in, and the workstation id will be present in the action.circulation rows.

NOTE: Currently, if you supply an invalid workstation name the connection will just fail. This is the same thing that happens if the account is allowed to go inactive. If using a workstation isn't desired, configure your selfcheck to leave the Location field blank or not send it at all.