LP#1499537 Fixed kpac.xml caching problem KPAC config files must be cached per config path in order to support cases where more than one KPAC interface has been configured in different vhosts or locations. Signed-off-by: Liam Whalen <liam.whalen@bc.libraries.coop> Signed-off-by: Mike Rylander <mrylander@gmail.com>
LP#1548869: tidy up whitespace Signed-off-by: Galen Charlton <gmc@esilibrary.com>
LP#1548869 KPAC hold results screen not displaying complete info The getit_results.tt2 page was not displaying item title after hold placement or after adding to a list. It was also not pulling in the item id (which was needed for navigational links back to the record) and was not pulling in the list name after adding an item to a named list. This patch adds some logic to prevent subroutine conflicts, adds missing parameters to a function call, and corrects a variable name. To test: Log in to the KPAC and perform the following actions: 1. Place a successful hold (results screen should show item title and pickup library, and provide links back to record page). 2. Place an unsuccessful hold (try placing a hold on an item you already have a hold on - the results screen should show the item title). 3. Add item to a temporary list (results screen should show item title). 4. Add item to a named list (results screen should show item title and title of list it was added to). Signed-off-by: Terran McCanna <tmccanna@georgialibraries.org> Signed-off-by: Jennifer Pringle <jennifer.pringle@bc.libraries.coop> Signed-off-by: Galen Charlton <gmc@esilibrary.com>
LP#1415898 Fix Potential Cstore Starvation in KPAC The getit_results function in EGKPacLoader.pm does xact_begin, which opens a connected session to a cstore drone. It later does xact_rollback, which does not disconnect the session. This leaves a drone hanging for several seconds unable to responde to new requests. This commit switches to a rollback call that does disconnect the session. Signed-off-by: Jason Stephenson <jason@sigio.com> Signed-off-by: Bill Erickson <berickxx@gmail.com>
LP#1282783: Use patron hold notification defaults for KPAC Since the KPAC interface doesn't display a patron's email/phone notification preferences, we had initially assumed it was pulling in the patron's default notification info, but we've found that it does not. Emails are not sent to patrons who have their notification preferences set to email, and hold slips print out without phone numbers or email addresses. This causes extra work for Circ staff in processing incoming holds as they need to look up each account number to get the patron's contact information. This patch pulls in the patron's email & phone notification preferences and related values in hidden fields and allows the patron to choose their pickup library at time of hold placement (defaults to their preferred pickup location). Signed-off-by: Terran McCanna <tmccanna@georgialibraries.org> Signed-off-by: Ben Shum <bshum@biblio.org>
Begin enabling KPAC-branded library info page This needs work and is going to get squashed like a bug when we're through. Yeesh. Signed-off-by: Dan Scott <dscott@laurentian.ca> Signed-off-by: Ben Shum <bshum@biblio.org>
Stage 1: Perl Attempt to ensure that we always write to oils://remote when needed. Signed-off-by: Thomas Berezansky <tsbere@mvlc.org> Signed-off-by: Dan Scott <dscott@laurentian.ca>
kpac : global filter config option Allows for configuration of an always-on global search filter. All searches, regardless of origin, in the kpac will be appended by this filter when configured. Signed-off-by: Bill Erickson <berick@esilibrary.com> Signed-off-by: Mike Rylander <mrylander@gmail.com>
kpac : repair hold retrieval code Signed-off-by: Bill Erickson <berick@esilibrary.com> Signed-off-by: Mike Rylander <mrylander@gmail.com>
kpac : ensure the single-hit redirect goes to kpac ... instead of to the tpac record detail page. Signed-off-by: Bill Erickson <berick@esilibrary.com> Signed-off-by: Mike Rylander <mrylander@gmail.com>
kpac : hold placement; more auth refresh repairs Signed-off-by: Bill Erickson <berick@esilibrary.com> Signed-off-by: Mike Rylander <mrylander@gmail.com>
kpac : wire up home search; auth timeout redirect fixes Signed-off-by: Bill Erickson <berick@esilibrary.com> Signed-off-by: Mike Rylander <mrylander@gmail.com>
kpac : initial login + place hold combined action Signed-off-by: Bill Erickson <berick@esilibrary.com> Signed-off-by: Mike Rylander <mrylander@gmail.com>
kpac : getit save to existing list Signed-off-by: Bill Erickson <berick@esilibrary.com> Signed-off-by: Mike Rylander <mrylander@gmail.com>
tpac: minor code formatting Signed-off-by: Bill Erickson <berick@esilibrary.com> Signed-off-by: Mike Rylander <mrylander@gmail.com>
kpac: save to anon-list Signed-off-by: Bill Erickson <berick@esilibrary.com> Signed-off-by: Mike Rylander <mrylander@gmail.com>
kpac : config and template work Signed-off-by: Bill Erickson <berick@esilibrary.com> Signed-off-by: Mike Rylander <mrylander@gmail.com>
kpac : rename files/paths for tpac consistency Signed-off-by: Bill Erickson <berick@esilibrary.com> Signed-off-by: Mike Rylander <mrylander@gmail.com>
kpac : initial holds; misc Signed-off-by: Bill Erickson <berick@esilibrary.com> Signed-off-by: Mike Rylander <mrylander@gmail.com>
kpac : copy table; record details Signed-off-by: Bill Erickson <berick@esilibrary.com> Signed-off-by: Mike Rylander <mrylander@gmail.com>