Temporary Lists: Modifying code to re-show temporary list links
that were disabled due to bug 1639236.

Signed-off-by: Terran McCanna <tmccanna@georgialibraries.org>

LP#1639236: fix display of temporary list items

Peer to the fix for bug 1638921.

To test
-------
[1] Apply patch.
[2] In public catalog, add one or more bibs to the temporary
    list ('My List').
[3] Click the 'View List' button, then verify that the titles
    are displayed.

Signed-off-by: Galen Charlton <gmc@equinoxinitiative.org>
Signed-off-by: Chris Sharp <csharp@georgialibraries.org>

PINES Customization: Hide queue order from My Account Holds Screen
in OPAC.

Signed-off-by: Terran McCanna <tmccanna@georgialibraries.org>

LP#1661346 - trim quotes from setting value

Signed-off-by: Chris Sharp <csharp@georgialibraries.org>

LP#1661346 - Add User Setting Type View reporter source

With our newly-implemented action_trigger.event_definitions
running, it is now desirable to have easier access to the SMS
settings selected by users. This is best accomplished with a
new view that will be linked from the actor.usr object.

Signed-off-by: Chris Sharp <csharp@georgialibraries.org>

LP#1596595 Targeter leverages batch AOUS lookups

Use the batch-by-org AOUS lookup function to fetch settings with one
cstore call across a wide set of org units.  This reduces the number of
cstore calls required, significantly in some cases, for single-use hold
targeter instances (like placement time targeting, checkin retargeting).

Specifically, in cases where a hold has targetable copies at multiple
circ libs, only one cstore call is needed for each of the
circ.holds.org_unit_target_weight and
circ.holds.target_when_closed settings, as opposed to one cstore call
for each per circ lib.

Signed-off-by: Bill Erickson <berickxx@gmail.com>

LP#1596595 AOUS lookup batch by org id

Org unit setting value lookup for batches of org units, instead of the
traditional batches by setting name.

Perl live test included.

Signed-off-by: Bill Erickson <berickxx@gmail.com>

Revert "LP#1596595 AOUS lookup batch by org id"

This reverts commit 9b2126e90d75de8e30e381c65c78af7fca46dcb0.

Revert "LP#1596595 Targeter leverages batch AOUS lookups"

This reverts commit 63f4975386126b54dfb6d8165c9d58a571b6dc68.

LP#1596595 Targeter leverages batch AOUS lookups

Use the batch-by-org AOUS lookup function to fetch settings with one
cstore call across a wide set of org units.  This reduces the number of
cstore calls required, significantly in some cases, for single-use hold
targeter instances (like placement time targeting, checkin retargeting).

Specifically, in cases where a hold has targetable copies at multiple
circ libs, only one cstore call is needed for each of the
circ.holds.org_unit_target_weight and
circ.holds.target_when_closed settings, as opposed to one cstore call
for each per circ lib.

Signed-off-by: Bill Erickson <berickxx@gmail.com>

LP#1596595 AOUS lookup batch by org id

Org unit setting value lookup for batches of org units, instead of the
traditional batches by setting name.

Perl live test included.

Signed-off-by: Bill Erickson <berickxx@gmail.com>

Revert "LP#1596595 AOUS lookup batch by org id"

This reverts commit 483d641d4292f73e0473dd9be9bfc29b7d294958.

Revert "LP#1596595 Targeter leverages batch AOUS lookups"

This reverts commit 410f75107d8689cd2174bf6f029da5738dfa82b9.

LP#1596595 Targeter leverages batch AOUS lookups

Use the batch-by-org AOUS lookup function to fetch settings with one
cstore call across a wide set of org units.  This reduces the number of
cstore calls required, significantly in some cases, for single-use hold
targeter instances (like placement time targeting, checkin retargeting).

Specifically, in cases where a hold has targetable copies at multiple
circ libs, only one cstore call is needed for each of the
circ.holds.org_unit_target_weight and
circ.holds.target_when_closed settings, as opposed to one cstore call
for each per circ lib.

Signed-off-by: Bill Erickson <berickxx@gmail.com>

LP#1596595 AOUS lookup batch by org id

Org unit setting value lookup for batches of org units, instead of the
traditional batches by setting name.

Perl live test included.

Signed-off-by: Bill Erickson <berickxx@gmail.com>

LP#1596595 Targeter use child editor for settings

Use the CStoreEditor linked to the ::Single (child) targeter object when
possible to fetch org unit setting values.

In cases where settings for many org units have to be retrieved at once,
the settings lookups can take long enough that the in-transaction editor
on the child targeter can timeout.  Using the child's editor directly
for the lookups will prevent this timeout and make the lookups a little
bit faster, since a new connect will not be required for each lookup.

This timeout scenario can occur with settings like
circ.holds.max_org_unit_target_loops and
circ.holds.target_when_closed, when there is wide variety of targetable
copies, because each have to be fetched once per target-able copy circ lib.

A secondary optimization would be a batch org setting lookup that
batches on org unit instead of setting name.

Signed-off-by: Bill Erickson <berickxx@gmail.com>

LP#1596595 Targeter accepts a list of hold ID's

Allow the caller to pass a list of hold ID's (consistent with the
current hold targeter).  This resolves the issue where the checkin
process attempts to retarget a set of holds via a single targeter call.

Signed-off-by: Bill Erickson <berickxx@gmail.com>

LP#1660059: Protect against null value in group field

If a nullable event grouping field is configured, and a null value is indeed
encountered when pulling together events, the Action/Trigger code will exit
unceremoniously.  To prevent this, we will now collect events with either
a null grouping object or grouping field, and use a new batch invalidation
API call to get rid of them as quickly as possible after group sorting is
complete.

Signed-off-by: Mike Rylander <mrylander@gmail.com>

Revert "LP#1660059 - Allow for undef value for A/T Processing Group Context Field"

This reverts commit 9636eab1103b69b52d276ea6884bed3b8d549dee.

Revert "Adding Bill Erickson's fix for endless open cstore transactions"

This reverts commit c585bfdedac4f2e35bcb0be2daae4a892f413f39.

Adding Bill Erickson's fix for endless open cstore transactions

After applying the fix to LP#1660059, the server running A/T notices
generated multiple cstore processes without closing them.  Changing
xact_commit to commit closes the cstore process after each event is
evaluated.

LP#1660059 - Allow for undef value for A/T Processing Group Context Field

For an action_trigger.event_definition that uses action.hold_request.sms_notify
as the Processing Group Context Field/group_field, the event processing
shows multiple Perl warnings and the open-ils.trigger process dies.

Tracing this to the relevant code in Trigger.pm shows that there is a
check to make sure that the fieldmapper object actually exists ($node),
but then assumes that the $ident_value (the actual field) is defined/not null.

Adding an additional check fixes the problem.

Signed-off-by: Chris Sharp <csharp@georgialibraries.org>

LP#1528647 Self-check only accepts user name value if regex
for barcode not set up

This fix allows a patron to log into the self-check interface with either
a barcode or a user name.

Written by: Bob Wicksall <bwicksall@pls-net.org>

Signed-off-by: Terran McCanna <tmccanna@georgialibraries.org>
Signed-off-by: Chris Sharp <csharp@georgialibraries.org>

LP#1596595 Hold copy permit test thinko repair

Pickup and requesting org unit IDs were passed in the wrong order to
the copy permit test.  This resulted in some items, particularly age
protected copies, appearing on the holds pull list even though they were
not (yet) permitted for hold capture at a remoted library.

Signed-off-by: Bill Erickson <berickxx@gmail.com>

New Book Lists: Final correction to title links

This was already applied to production during the upgrade process -
adding to GIT here for future reference.

Signed-off-by: Terran McCanna <tmccanna@georgialibraries.org>
Conflicts:
Open-ILS/xsl/ATOM2XHTML.xsl

Adding create date and active date to holdings maintenance interface

Revert "New Book Lists - Title Links"

This reverts commit bd0b61a767a69e42cfa8f69615926878802a3b58.

New Book Lists - Title Links

Improved title link to go to standard record page that the current
OPAC uses.

Signed-off-by: Terran McCanna <tmccanna@georgialibraries.org>

Adding usrname index to upgrade script

Adding post-upgrade file.

Modify upgrade scripts since we're doing the reingest separately.

OPAC Mobile View Display customizations
Mainly to Lists page and Holds page in My Accounts

Signed-off-by: Terran McCanna <tmccanna@georgialibraries.org>

LP#1596595 Find parallel holds in main query

Identify holds to process by which metarecord the hold is (ultimately)
linked to within the main holds query instead of via a secondary filter.
This avoids the overhead of starting a new batch of hold targeters,
where each process has to fetch all possible holds, then filter down to
those targetable within the current parallel slot.  In thise case, each
process only retrieves the holds it plans to process.

Signed-off-by: Bill Erickson <berickxx@gmail.com>

LP#1596595 Hold targeter perl live tests

1. Batch of tests for concerto hold 1 / title hold
2. Batch of tests for concerto hold 265 / metarecord hold with
   holdable_formats restriction.
3. --skip-viable test

Signed-off-by: Bill Erickson <berickxx@gmail.com>

LP#1596595 Adds sample metarecord hold to concerto

Adds a holdable_formats option to concerto's populate_hold() function.
Inserts one metarecord hold for testing.

Signed-off-by: Bill Erickson <berickxx@gmail.com>

LP#1596595 Hold targeter refactoring and optimization.

* New open-ils.hold-targeter service

* Ports hold targeter code to a Perl utility function, communicating w/
  the DB via cstore instead of storage.

* Adds a new global flag 'circ.holds.retarget_interval' for configuring
  the hold retarget interval in the database.

* Adds a new DB function to regenerating hold copy maps to make map
  deletion and creation more efficient.

* Adds an option for targeting holds in newest to oldest order.

* Caches all org unit settings per targeter run.

* Adds support for "skip_viable" option.  This tells the hold targeter
  to avoid modifying any holds that target viable copies.  AKA "fix
  broken" mode.

  For example, you might run in skip_viable mode with a retarget
  interval of 24hr once a day to repair non-viable holds, then also run
  the targeter in regular mode once a day with a retarget interval of 48
  hours to give staff 2 days to process viable holds.

* Hold target loops logic changes:

 ** Org units with fewer target attempts are prioritized during loop
    processing.  So, instead of segregating org units into 2 categetories,
    those attempted in the current loop and those not attempted, sort those
    not attempted by the number number of times they have been attempted.
    Within each grouping, prioritize by target weight/proximity as before.

 ** All org units that have been attempted less than the max configured
    amount are on the table for targeting, not just those that have been
    targeted less than the current loop max.  If no orgs with
    less-than-current-max attempts are found, try orgs that match the
    current max (but are still less than the configured max).

 ** When activated, target looping treats the pickup lib like any
    other org unit.  If a targeted copy at the pickup lib remains
    un-captured, at re-target time, a copy at a different branch is
    chosen (if one is available) even if other copies at the pickup
    lib are targetable.

* Parallel targeting support baked into service.

  Teach the targeter to process a subset of holds based on the number of
  parallel targeters at play and the parallel targeting slot each targeter
  instance occupies.

  As with the existing hold targeter, group holds by their metarecord to
  avoid multiple targeter processes targeting the same sets of potential
  copies.

* Logging / code refactoring and clean up.

* New hold_targeter_v2.pl script for batch hold targeting.  Existing
  targeter remains for backwards-compat.

hold_targeter_v2.pl options:

--verbose
    Print process counts

--parallel <parallel-process-count>
    Number of parallel hold processors to run.  This overrides any
    value found in opensrf.xml

--target-all
    Target all active holds, regardless of when they were last targeted.

--skip-viable
    Avoid modifying holds that currently target viable copies.
    In other words, only (re)target holds in a non-viable state.

--retarget-interval
    Override the 'circ.holds.retarget_interval' global_flag value.

--parallel-init-sleep
    Time to wait between starting each parallel instance.  Useful for
    avoiding dog-piling the DB.

Signed-off-by: Bill Erickson <berickxx@gmail.com>

New Book Lists: More fixes
Changed title and added 'On Order' message for new records that aren't
available for holds or checkout yet.

Signed-off-by: Terran McCanna <tmccanna@georgialibraries.org>

Changing hard-coded string to allow locale to work.

New Book Lists: Additional tweaks for author and image display

Signed-off-by: Terran McCanna <tmccanna@georgialibraries.org>

New Book Lists
Adding functionality to PINES home page so that patrons can select
a library and see a list of new materials.

Signed-off-by: Terran McCanna <tmccanna@georgialibraries.org>

Add Invoice Summary Reporting View.

Before now, there have been no easy ways to gather acq
receiving totals from reports.  Here is an attempt to remedy
that situation.

Signed-off-by: Chris Sharp <csharp@georgialibraries.org>

better idea with Check Number

unset default Check Number, disable widget if not Check Payment,
and disable Apply Payment button if Check Payment with invalid Check Number

Signed-off-by: Jason Etheridge <jason@esilibrary.com>

webstaff: default Check Number to 1

instead of 0, so that it doesn't speedbump payments (of any type)

Signed-off-by: Jason Etheridge <jason@esilibrary.com>

Driver's license guidelines
Added on-screen tip to remind staff how driver's licenses should be
entered in PINES.

Signed-off-by: Terran McCanna <tmccanna@georgialibraries.org>

Hide Email Checkout Receipts Option

This doesn't work in the staff client, so hiding option from the patron
self-registration form and from the patron's My Account settings until
we have moved to the web client. (It still appears in the staff client,
so can be set there, but it will be virtually non-functional if set.)

Signed-off-by: Terran McCanna <tmccanna@georgialibraries.org>

Shelf Browser display - bug fix

Signed-off-by: Terran McCanna <tmccanna@georgialibraries.org>

Temporary Lists: Hide temporary list functionality until bug is resolved.

Signed-off-by: Terran McCanna <tmccanna@georgialibraries.org>

Hiding sort option for Popularity Adjusted Relevance.

Signed-off-by: Terran McCanna <tmccanna@georgialibraries.org>

LP#1360347 - Acq custom copy status at receiving

Current acq behavior is that received items get a
copy status of "In Process".  This status is typically
holdable and opac_visible, and some libraries have a need
for an intermediate status between "On Order" and "In Process".

Signed-off-by: Chris Sharp <csharp@georgialibraries.org>

Fix SIP timeouts due to invalid sessions

Signed-off-by: Dan Scott <dscott@laurentian.ca>

LP#1643932 - Fix search slowness when patron searching by username

Bug 1501781 introduced a feature to improve patron name searching,
but which was causing very slow queries when searching by username.
Adding an index to username solves the slowness problem.

Suggested by Mike Rylander.

Signed-off-by: Chris Sharp <csharp@georgialibraries.org>

LP#1586509 Bug fix to LP#1352542 caused extraneous blank line to appear in
spine label.  This affected LC call numbers that had only one cutter number
plus additional text following.

Signed-off-by: Dan Pearl <dpearl@cwmars.org>
Signed-off-by: Kathy Lussier <klussier@masslnc.org>

LP#1594937 Fix off-by-one display of closed dates

The switch to toISOString() to format dates introduced an off-by-one error in
the closed dates display, showing one extra day of closure due to the timezone
being ignored in toISOString().

toLocaleDateString() is the future of locale-sensitive date formats. In XUL,
because it is an old version of Firefox, it lacks locale sensitivity, but for
the purposes of the web staff client it's a good base to build on as even
Internet Explorer supports the locale and options arguments as of IE 11.

And for the immediate purposes of showing the right dates in the closed dates
editor, it works.

Signed-off-by: Dan Scott <dscott@laurentian.ca>
Signed-off-by: Chris Sharp <csharp@georgialibraries.org>

LP#1432753 Restore "All day" verbiage to Closed Dates editor

Commit ede7e78925 replaced the JSAN calls to util.date.formatted_date() with
inline date/time handling, in the process returning times with granularity to
the minute instead of to the second. This resulted in the test for "all day"
closings always failing.

Signed-off-by: Dan Scott <dscott@laurentian.ca>
Signed-off-by: Chris Sharp <csharp@georgialibraries.org>

LP#1528916 Patron Holds Ready/Total

Switch order or patron_stats().holds.ready and
patron_stats().holds.total in t_summary and patron
index.

Signed-off-by: Kyle Huckins <khuckins@catalystdevworks.com>
 Changes to be committed:
modified:   Open-ILS/src/templates/staff/circ/patron/index.tt2
modified:   Open-ILS/src/templates/staff/circ/patron/t_summary.tt2

Signed-off-by: Kathy Lussier <klussier@masslnc.org>

Revert "Returning all selfcheck templates to stock versions."

This reverts commit 2d1fbbf5de79758b9c8b36863135554ecbdf9d18.

Returning all selfcheck templates to stock versions.

Since we only have bits and pieces of Spanish translations for our
customized KPAC right now, removing the parts that would display in
Spanish if Spanish has been selected in the OPAC. In the future when
all parts of the KPAC have been translated and graphic buttons have
been switched to text buttons that can be translated, we will then
need to go back in and re-add the language markup to allow the
translations to occur.

Signed-off-by: Terran McCanna <tmccanna@georgialibraries.org>

Hiding parent link from branch info pages to prevent confusion when patrons
see system hours (and other info) and think that they are the same as
branch hours.

Signed-off-by: Terran McCanna <tmccanna@georgialibraries.org>

LP#1647819 - Missing closing angle bracket in anon_list.tt2

Signed-off-by: Chris Sharp <csharp@georgialibraries.org>
Conflicts:
Open-ILS/src/templates/opac/parts/anon_list.tt2

LP#1647819 - Missing closing angle bracket in anon_list.tt2

Signed-off-by: Chris Sharp <csharp@georgialibraries.org>

Language files - a few more corrections.

Signed-off-by: Terran McCanna <tmccanna@georgialibraries.org>

Language - here are three files I missed including in the prior
collection of translation-related files.

Signed-off-by: Terran McCanna <tmccanna@georgialibraries.org>

OPAC Updates related to page text and translations. Also required are the
en-PINES.po custom text file and the es-ES.po translation file.

Signed-off-by: Terran McCanna <tmccanna@georgialibraries.org>
Conflicts:
Open-ILS/src/templates/opac/parts/advanced/search.tt2
Open-ILS/src/templates/opac/parts/pref_lib_display.tt2

Restoring selfcheck.js to stock version

Lists page - re-added note and link to Lists instructions

Signed-off-by: Terran McCanna <tmccanna@georgialibraries.org>

UI tweaks to the basic search form so that it looks better on smaller
screens.

Signed-off-by: Terran McCanna <tmccanna@georgialibraries.org>

OPAC 'preferred library' - removed question mark link

since it doesn't go to a meaningful page and confuses patrons.

(Terran)

OPAC advanced search page UI tweaks. (Terran)

Rolling prefs_notify.tt2 back to stock 2.11.1.

Checking out stock version of lists.tt2 to re-customize

Restoring stock advanced search template while we troubleshoot.

LP#1639782 - Add Active Date to Item Status Column Picker

Circ and Cataloging staff need a quick way to view the Active
Date of an item in the "Item Status" (F5) interface.  This
makes that column available for selection.

Signed-off-by: Chris Sharp <csharp@georgialibraries.org>

PINES already has 0977 applied, so comment it out.

Creating PINES pre-upgrade script.  Drop public.demographic view

Addition of new item format icon for direct audio (ie, Playaways and
similar items that are pre-loaded with audiobooks and do not require
access to a computer to use).

Signed-off-by: Terran McCanna <tmccanna@georgialibraries.org>

LP#1599634 - Remove DROP VIEW statement, add .sql to name.

When running the upgrade script, the DROP VIEW statement resulted in an
error since the view did not already exist.  You could add IF EXISTS, but
CREATE OR REPLACE VIEW is sufficient.

Signed-off-by: Chris Sharp <csharp@georgialibraries.org>

LP1599634 Circulation report source to include in-house(non cat), and non cat circ

Added a simple view and supporting IDL. This will allow all the circulation
data to output in the same report.

Signed-off-by: blake <blake@mobiusconsortium.org>
Signed-off-by: Chris Sharp <csharp@georgialibraries.org>
Conflicts:
Open-ILS/src/sql/Pg/090.schema.action.sql

LP#1610246 Classic Circulation View excluding circulations

The reporter.classic_current_circ view, which is available via
Open-ILS/src/sql/Pg/example.reporter-extension.sql, was excluding
circulations because it was assuming all users have a billing address
This branch corrects that with a LEFT JOIN.

No upgrade script will be included since these views are not assumed
to be installed by all Evergreen instances.  Please re-run the
example.reporter-extension.sql file to apply this change for your
instance.

Signed-off-by: Chris Sharp <csharp@georgialibraries.org>

Updates to KPAC Features (addition of TumbleBooks)

Signed-off-by: Terran McCanna <tmccanna@georgialibraries.org>

LP#1528647 Self-check only accepts user name value if regex
for barcode not set up

This patch was written by Bob Wicksall and posted to launchpad. I've
tested on my own server and packaged it for review here.

The fix allows a patron to log into the self-check interface with either
a barcode or a user name.

Signed-off-by: Terran McCanna <tmccanna@georgialibraries.org>
Revert "LP#1528647 Self-check only accepts user name value if regex"

Terran reports this not working as expected.

This reverts commit 0eb3a73c2fba4f609c0dab015a843a550e03516e.

LP#1528647 Self-check only accepts user name value if regex
for barcode not set up

This fix allows a patron to log into the self-check interface with either
a barcode or a user name.

Written by: Bob Wicksall <bwicksall@pls-net.org>

Signed-off-by: Terran McCanna <tmccanna@georgialibraries.org>
LP#1528647 Self-check only accepts user name value if regex for barcode
not set up

The original fix only worked if password was not required for self-check.
This fix allows it to work when password is required as well.

Signed-off-by: Terran McCanna <tmccanna@georgialibraries.org>

LP#1528647 Self-check only accepts user name value if regex
for barcode not set up

This patch was written by Bob Wicksall and posted to launchpad. I've
tested on my own server and packaged it for review here.

The fix allows a patron to log into the self-check interface with either
a barcode or a user name.

Signed-off-by: Terran McCanna <tmccanna@georgialibraries.org>

KPAC SRP 2016 tweaks to pre-set search categories

KPAC files to promote Summer Reading Program for 2016.

Removing testing piece.

Actually, that never needed to be in parens.

Testing for now, so no commit;

Forgot a closing parens.

I've redone all of the kpac category links so they no longer use
a link shortener. I also rearranged some of the links so that there
are more options on the home page.

Signed-off-by: Terran McCanna <tmccanna@georgialibraries.org>
Conflicts:
Open-ILS/examples/kpac.xml

KPAC - Hold results not displaying item title and throwing log error

I believe the path comparison operator in the 'load' subroutine has
a flaw in logic caused by a path-naming problem. The path-matching
function recognizes the kpac/getit_results path as matching positively
for both the load_getit subroutine and the load_getit_results subroutine.

This patch reverses the order in which the matching occurs in an attempt
to resolve the conflict.

Signed-off-by: Terran McCanna <tmccanna@georgialibraries.org>

Fix for org unit order in OPAC holdings list (adapted from
http://git.evergreen-ils.org/?p=evergreen/pines.git;a=commit;h=adc566e152d10796b73c992a16ae46e69cc3ab77)

Signed-off-by: Terran McCanna <tmccanna@georgialibraries.org>

Adding PG 9.4 installation targets

Second piece to the bug fix for group holds...

Signed-off-by: Terran McCanna <tmccanna@georgialibraries.org>

adding a/t environment to new event def

More changes to post-upgrade data file.

LP#1463166 Removed multiple perms

The use of multiple perms in creating the widget was causing problems.
I changed the permlist back to a single perm.

Signed-off-by: Liam Whalen <liam.whalen@bc.libraries.coop>

LP#1463166 Fix Batch update of Funds in Selection Lists

Selection lists are currently unable to batch update funds.  The problem
stems from the fact that the Fund selection list is not populated in the
copy rows.  New code used to limit the number of funds, in the dropdown
list, to those funds available to the user was added for purchase
orders.  However, these changes were not added to Selection Lists.  This
code adds the new call to initBatchUpdater to the selection list code.

Signed-off-by: Liam Whalen <liam.whalen@bc.libraries.coop>

Temporary modifications to My Account > Holds History until more
permanent fixes are developed.

Hid paging because it is broken. See:
https://bugs.launchpad.net/evergreen/+bug/1422927

Showing "Canceled" instead of "Active" for canceled holds. See:
https://bugs.launchpad.net/evergreen/+bug/1275118

Signed-off-by: Terran McCanna <tmccanna@georgialibraries.org>

Adding customization to disable the "Clear Holds Shelf" function.

adding login redirection page