LP#1646166 Hatch availability display improvements

Hide printer settings in the print config UI when Hatch is not available
and provide better inline alerts.  Avoid attempts by the printer config
UI from talking to Hatch when Hatch is known not to be available.

Signed-off-by: Bill Erickson <berickxx@gmail.com>
Signed-off-by: Kathy Lussier <klussier@masslnc.org>

LP#1646166 À la carte Hatch, on-call settings, strict access.

1. Hatch now supports a al carte features instead of requiring all-or-none
functionality.  Supported features currently include printing, settings,
and offline.  (Note: offline handling pending merge of offline UI code).

2. Adds support for on-call setting keys.  On-Call keys are those that
can be set/get/remove'd from localStorage when Hatch is not avaialable,
even though Hatch is configured as the primary storage location for the
key in question.

The initital target use case for on-call keys are those that allow the
user to login and perform basic admin tasks (like disabling Hatch) even
when Hatch is down.  AKA Browser Staff Run Level 3.

3. egHatch no longer attempts requests at Hatch, falling through to
local requests when Hatch fails.  With the exception of on-call keys
(above), either Hatch is used or local requests are used, depending on
the configuration.  The goal is to prevent any unintended and
confusing blending of local and remote data.  In other words, if Hatch
is broken, it needs to be fixed or disabled for regular work flow to
continue.

4. Hatch now has a dedicated UI under workstation administration.

5. Workstation admin splash page rearranged to take advantage of more
horizontal space and avoid pushing so many options down the page.

Signed-off-by: Bill Erickson <berickxx@gmail.com>
Signed-off-by: Kathy Lussier <klussier@masslnc.org>

LP#1646166 Hatch print requires no print CSS

Avoid inserting the print CSS into the print content when printing
remotely.

Commit includes some refactoring of the print code to create more
obvious separation between remote vs. in-browser printing.

Signed-off-by: Bill Erickson <berickxx@gmail.com>
Signed-off-by: Kathy Lussier <klussier@masslnc.org>

LP#1646166 Hatch extension connect via DOM, remote cache

Determine Hatch availability by checking for the presence of a
well-known Hatch attribute in the DOM, inserted into the page
by the extension at load time.

Cache get/set values when talking to the Hatch extension (on a per-page
basis) to reduce unnecessary Hatch I/O.

Signed-off-by: Bill Erickson <berickxx@gmail.com>
Signed-off-by: Kathy Lussier <klussier@masslnc.org>

LP#1646166 Hatch print configiguration interface

1. Query printers for available options via Hatch.
2. Allow the user to apply print options on a per-context / per-printer
   basis.
3. As before, store printer conifugration options via Hatch.

Signed-off-by: Bill Erickson <berickxx@gmail.com>
Signed-off-by: Kathy Lussier <klussier@masslnc.org>

LP#1640255 Hatch native messaging extension

Replaces Hatch Websockets communication layer with browser extension-
based communication.

Hatch API remains the same with 2 notable exceptions:

1. appendItem() API call has been removed.  It did not work as designed
   and (thus far) has served no purpose.  It was originally intended for
   offline data storage, but that will probably require something a
   little smarter.

2. The printer configuration API is no more.  This will be replaced with
   an in-app configuration page.  Note, this does not prevent use of the
   printer dialog, it only means settings are not collected from the
   printer dialog.

Signed-off-by: Bill Erickson <berickxx@gmail.com>
Signed-off-by: Kathy Lussier <klussier@masslnc.org>

Apply sort for ident_type dropdown

We may want YAOUS for "do we want to sort ident_type by name"...

Signed-off-by: Thomas Berezansky <tsbere@mvlc.org>
Signed-off-by: Remington Steed <rjs7@calvin.edu>
Signed-off-by: Mike Rylander <mrylander@gmail.com>

Stamping upgrade script for separating fingerprint components

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

LP#1528901: avoid accidental bib fingerprint collisions

This patch fixes a problem where the bib fingerprint algorithm
could end up putting completely different works in the same
metarecord. For example,

100 $a Steel, Danielle
245 $a Blue

and

*no 1XX
245 $a Blue steel

previously (with stock config.biblio_fingerprint settings) got
a fingerprint of "bluesteel". With this patch, their fingerprints
are now:

"Title:blue Author:steel"

and

"Title:bluesteel Author:"

The upgrade script supplied with this patch remaps the metarecords
after updating the fingerprints. While existing metarecord holds
may get moved, note that there is no known way of ensuring that a
metarecord hold placed on a collided metarecord will end up attach
to whatever work the patron intended to request.

To test:

[1] Add records for "Blue" and "Blue steel".
[2] Note that they end up on the same metarecord.
[3] Apply the patch and perform the update.
[4] The two bibs should now be on separate metarecords.

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Signed-off-by: Rogan Hamby <rogan.hamby@gmail.com>
Signed-off-by: Kathy Lussier <klussier@masslnc.org>
Signed-off-by: Mike Rylander <mrylander@gmail.com>

Stamping upgrade script for including parts in bib fingerprints

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

LP#1553287: Add part information to biblio.fingerprint

Evergreen metarecord searching will sometimes group together different works
that are part of the same series because biblio.fingerprint doesn't incorporate
subfield n or p from the title. For example, bib records for the Mockinjay
movies list the Hunger Games in the 245a with Mockinjay in subfield p.
Without the part information in the fingerprint, Evergreen will group these
movies together with versions of the first Hunger Games book.

This branch adds parts subfields to biblio.fingerprint to allow us to
distinguish among different parts in a series.

Signed-off-by: Kathy Lussier <klussier@masslnc.org>
Signed-off-by: Mike Rylander <mrylander@gmail.com>

LP#1663374: Release notes for metarecord search by default

Signed-off-by: Mike Rylander <mrylander@gmail.com>
Signed-off-by: Kathy Lussier <klussier@masslnc.org>

LP#1663374: Allow metarecord searching by default

Before the TPAC, a site could configure the OPAC to use metarecord searching
by default, via a configuration file.  Here we bring that back.

A new setting is present in templates/opac/parts/config.tt2 to enable this
feature.  By setting this to a true value (normally 1) the TPAC will silently
include the #metabib search modifier in the search form on any interfaces that
do not have a UI component that allows the user to control the setting.

Signed-off-by: Mike Rylander <mrylander@gmail.com>
Signed-off-by: Kathy Lussier <klussier@masslnc.org>

LP#1573734: Stamping upgrade script for transcendent bibs in unapi-mmr

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

LP#1573734: Minor tweak to heading

A minor tweak to change the heading to sentence case to match other similar
headings in the summary.tt2 file.

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

LP1573734_Link_to_sibling_metarecord_bibs_in_record_detail_page

This feature will display sibling records when viewing a bib record in the OPAC.
Altered Record.pm to gather up additional metarecord information to pass to
misc_util.tt2 for parsing. Displayed on summary.tt2 styled in responsive in
style.css.tt2. Also fixed the URL in mmr search results for single bib results in
table.tt2. Transcendent bibs needed to be included in the unapi.mmr_mma function.

This code requires the code from LP1629108.
http://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;
h=refs/heads/user/blake/LP1629108-metarecord-constituent-result-reroute

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

Forward-port 2.11.2 Upgrade Script

Signed-off-by: Dan Wells <dbw2@calvin.edu>

Forward-port 2.11.1 Upgrade Script

Signed-off-by: Dan Wells <dbw2@calvin.edu>

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>
Signed-off-by: Jason Etheridge <jason@esilibrary.com>

LP 1473054: No-op Checkin Should Not Always Float a Floating Copy.

Prior to this patch, a no-op checkin would not check if a copy should
float using the evergreen.can_float() function.  It would simply float
the copy to the org_unit where the checkin happened if the copy had a
float value set.

This commit changes the behavior to use evergreen.can_float() to
determine if the copy should float to the org_unit where the checkin
happened.  This change makes the behavior consistent with a regular
checkin.

Signed-off-by: Jason Stephenson <jason@sigio.com>
Signed-off-by: Josh Stompro <stomproj@larl.org>
Signed-off-by: Galen Charlton <gmc@equinoxinitiative.org>

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

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

LP#1077947 - Clarifying meaning of ITEM_ON_HOLDS_SHELF event

This updates the "This requested item is currently on the holds shelf"
message to clarify that it's on the hold shelf for a different patron.

Signed-off-by: Chris Sharp <csharp@georgialibraries.org>
Signed-off-by: Kathy Lussier <klussier@masslnc.org>

LP#1629108: Stamping upgrade script for reroute metarecord constituent results

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

LP1629108 setup "Do_cache" bit on the function call for production use

Flipped the do_cache bit from false to true.

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

LP#1629108: Release notes for metarecord search improvements

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

LP1629108 Finalizing Search.pm removing if block

For ease of diffing, I left the old if block in place because removing it would
cause all of the lines to show changes when in fact it was just spacing. This commit
simply removes the if block and the spacing is the only change.

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

LP1629108 metarecord_constituent_result_reroute

This patch will route the metasearch logic through the "standard" search logic
in order to leverage the heavy use of filters and other features.

A column is introduced to unapi.mmr_mr to include the constituent bibs in the
return. A tweak was required in the template toolkit code to take advantage of the
new payload. This enables TT to decide which icons should be displayed when search
results are filtered.

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

LP#1621412: Fix typo in acq li_attr.tt2

As noticed by Eva Cerninakova, there is a mispelling for "identifier"
in this file. Fixed.

Signed-off-by: Ben Shum <ben@evergreener.net>
Signed-off-by: Galen Charlton <gmc@equinoxinitiative.org>

LP#1659588 Patron summary displays alert message

Display the contents of the patron alert message in the web staff patron
summary side-bar.  Alert displays below blocking penalties, above the
name, etc. with the same style as a blocking penalty.

Signed-off-by: Bill Erickson <berickxx@gmail.com>
Signed-off-by: Galen Charlton <gmc@equinoxinitiative.org>

LP#1537215 webstaff: require confirmation to delete patron note

Call egConfirmDialog to confirm deletion of
patron note.

Create strings for Note Deletion.

Signed-off-by: Kyle Huckins <khuckins@catalystdevworks.com>
Conflicts:
Open-ILS/web/js/ui/default/staff/circ/patron/app.js

Signed-off-by: Galen Charlton <gmc@equinoxinitiative.org>

LP#1657589: add release notes entry

Signed-off-by: Galen Charlton <gmc@equinoxinitiative.org>

LP#1657589: Add a couple of SMS Carriers

Add entries for Google Fi and Republic Wireless.

No upgrade script because systems may have added them manually already.

Signed-off-by: Thomas Berezansky <tsbere@mvlc.org>
Signed-off-by: Galen Charlton <gmc@equinoxinitiative.org>

LP#1664021: Add more to .gitignore

The following three support scripts can be added to the .gitignore

Open-ILS/src/support-scripts/action_trigger_runner.pl
Open-ILS/src/support-scripts/authority_authority_linker.pl
Open-ILS/src/support-scripts/offline-blocked-list.pl

Also, ignore the created test-driver file.

Signed-off-by: Ben Shum <ben@evergreener.net>
Signed-off-by: Galen Charlton <gmc@equinoxinitiative.org>

LP#1527770 webstaff: remove code flash

Add ng-cloak to base.tt2
Add style to enable ng-cloak to base.tt2

Signed-off-by: Billy Horn <bhorn@catalystdevworks.com>
Signed-off-by: Galen Charlton <gmc@equinoxinitiative.org>

LP1664376: Correct Closing Tag for Badges on record summary page

Signed-off-by: Jason Boyer <jboyer@library.in.gov>
Signed-off-by: Galen Charlton <gmc@equinoxinitiative.org>

LP#1643932: stamp schema update

Signed-off-by: Galen Charlton <gmc@equinoxinitiative.org>

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>
Signed-off-by: Galen Charlton <gmc@equinoxinitiative.org>

LP#1663663: Stamping upgrade script for overlay merge profile fix

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

LP#1663663: seed data upgrade script

This patch attempts to repair the Full Overlay merge profile
in 2.11 and 2.12-masterish databases.

Signed-off-by: Galen Charlton <gmc@equinoxinitiative.org>
Signed-off-by: Kathy Lussier <klussier@masslnc.org>

LP#1663663: unbreak Full Overlay merge profile

This patch fixes a regression introduced in 2.11 that had
the effect of making the stock Full Overlay merge profile
act like a match-only merge that also updates the bib source.

To test
-------
[1] After applying the patches, verify that the Full Overlay
    merge profile now fullly updates the target bib record.

Signed-off-by: Galen Charlton <gmc@equinoxinitiative.org>
Signed-off-by: Kathy Lussier <klussier@masslnc.org>

LP#1507807: Stamping upgrade script for in-house copy alerts

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

LP 1507807: Release Notes for In-House-Use Copy Alerts

Add a release notes document explaining how the settings work.

If you want to test the functionality, you should set the settings to
true.  Next, find a copy with an alert message and another copy with
the check in alert set to true on its location.  Recording an
in-house-use for either of these copies should cause an alert to
display the respective message.  If a copy has both, then two alerts
will appear one after the other.

Resetting the settings to false or deleting the settings should stop
the alerts from displaying.

Signed-off-by: Jason Stephenson <jason@sigio.com>
Signed-off-by: Kathy Lussier <klussier@masslnc.org>

LP 1507807: Show Alert dialogs in XUL staff client.

If the circ.in_house_use.copy_alert org setting is true and the copy
has an alert_message, then we display an alert dialog with the copy
alert message when doing an in-house-use in the XUL staff client.

If the circ.in_house_use.checkin_alert org setting is true and the
copy location checkin_alert field is also true, then we display an
alert dialog with the "item needs to be routed to..." alert message
when doing an in-house-use in the XUL staff client.

Signed-off-by: Jason Stephenson <jason@sigio.com>
Signed-off-by: Kathy Lussier <klussier@masslnc.org>

LP 1507807: Show Alert dialogs in browser staff client.

If the circ.in_house_use.copy_alert org setting is true and the copy
has an alert_message, then we display an alert dialog with the copy
alert message when doing an in-house-use in the browser staff client.

If the circ.in_house_use.checkin_alert org setting is true and the
copy location checkin_alert field is also true, then we display an
alert dialog with the "item needs to be routed to..." alert message
when doing an in-house-use in the browser staff client.

Signed-off-by: Jason Stephenson <jason@sigio.com>
Signed-off-by: Kathy Lussier <klussier@masslnc.org>

LP 1507807: Add org unit settings for in-house-use copy alerts.

Add circ.in_house_use.copy_alert and circ.in_house_use.checkin_alert
org. unit setting types to seed data and upgrade script.

No default values are provided for the new settings.  They are not
defined (i.e. off) by default.

Add a PgTap test to verify that the org. unit setting types exist in
the database.

See the release notes for what these settings do.

Signed-off-by: Jason Stephenson <jason@sigio.com>
Signed-off-by: Kathy Lussier <klussier@masslnc.org>

LP#1664386: fix certain subrequests that control DB transactions

This patch removes all cases where the current OpenSRF client
object is passed to

$self->method_lookup('open-ils.storage.transaction.{begin/rollback}')->run()

This idiom is no longer needed, as session information required to
generate a transaction ID is reliably passed to subrequests. Without
this patch, current Evergreen master and OpenSRF master will cause
the following methods to return not only their results, but also
"1" from each use of this idiom:

open-ils.storage.actor.user.checked_out
open-ils.storage.booking.reservation.resource_targeter
open-ils.storage.action.hold_request.copy_targeter

To test
-------
[1] Be running current Evergreen master and OpenSRF master
    (or the 2.5 alpha)
[2] Run (say) open-ils.storage.actor.user.checked_out in srfsh:

srfsh# request open-ils.storage open-ils.storage.actor.user.checked_out 13

Received Data: 1

Received Data: 1

Received Data: {
  "out":[

  ],
  "claims_returned":[

  ],
  "long_overdue":[

  ],
  "overdue":[
    "55",
    "56",
    "57",
    "58",
    "59",
    "60"
  ],
  "lost":[

  ]
}

[3] Note the extraneous 'Received Data: 1' sent prior to the substantive
    response.
[4] Apply the patch and rerun step 2; note that this time
    only the desired response is sent.

Signed-off-by: Galen Charlton <gmc@equinoxinitiative.org>
Signed-off-by: Bill Erickson <berickxx@gmail.com>

LP#1528919: Alphabetise non-catalogued item types

Use an orderBy expression on the lists of noncat types
on the patron checkout page and inhouse-use page.

Signed-off-by: Kyle Huckins <khuckins@catalystdevworks.com>
Signed-off-by: Galen Charlton <gmc@equinoxinitiative.org>

LP#1528911 Billing Type Alphabetical

Add an orderBy expression to the options in the
billing type selection box.

Signed-off-by: Kyle Huckins <khuckins@catalystdevworks.com>
Signed-off-by: Galen Charlton <gmc@equinoxinitiative.org>

docs: LP1655400 Phonelist.pm broken formatting

Signed-off-by: Jeanette Lundgren <jlundgren@cwmars.org>
Signed-off-by: Galen Charlton <gmc@equinoxinitiative.org>

LP#1662319: zindex Tweak in Patron Account

The Other menu in a patron's account was dropping down behind the specify due
date input box. A tweak to the zindex for the Patron Account tabs was needed
for the menu to display in front of the input box.

Test plan:
Retrieve a patron in the web client. To the right of the Patron Account tabs,
click the Other dropdown menu. The menu will appear behind due date input.
Depending on the browser / screen resolution, it may also hide one of the
options in the dropdown menu. After the patch is loaded, the menu should display in front of the input as expected.

Signed-off-by: Kathy Lussier <klussier@masslnc.org>
Signed-off-by: Galen Charlton <gmc@equinoxinitiative.org>

LP#1455606: Add release note about removal of unused selfcheck password library setting

Just in case, added a quick release note about the removal of this setting.

Signed-off-by: Ben Shum <ben@evergreener.net>

LP#1455606: Stamping upgrade script for removing unused selfcheck password setting

Signed-off-by: Ben Shum <ben@evergreener.net>

LP#1455606 - Also delete from config.org_unit_setting_type log.

As Ben Shum pointed out, the upgrade script will fail if entries
still exist for the removed setting, so we delete them too.

Signed-off-by: Chris Sharp <csharp@georgialibraries.org>
Signed-off-by: Ben Shum <ben@evergreener.net>

LP#1455606 remove unused self checkout setting

There is an unused (and apparently never used) org unit setting
related to self checkout:

circ.selfcheck.require_patron_password

This commit removes the redundant unused setting, related
permissions, and related strings.

Thanks to Lynn Floyd with the Anderson County Library for pointing
this out.

Signed-off-by: Jeff Godin <jgodin@tadl.org>
Signed-off-by: Chris Sharp <csharp@georgialibraries.org>
Signed-off-by: Ben Shum <ben@evergreener.net>
Conflicts:
build/i18n/po/db.seed/db.seed.pot

Signed-off-by: Ben Shum <ben@evergreener.net>

LP 1656549: Remove Makefile, Makefile.in, Makefile.am

This commit adds a line to Open-ILS/web/Makefile.am to delete the
Makefile, Makefile.in, and Makefile.am that are copied to the
destination WEBDIR.

Signed-off-by: Jason Stephenson <jason@sigio.com>
Signed-off-by: Ben Shum <ben@evergreener.net>

LP#697926: change ar-AR to ar-JO

Also change update script and .gitignore file references

Signed-off-by: Ben Shum <ben@evergreener.net>

LP#697926: change ar-AR to ar-JO

With an active translator from Jordan for the Arabic strings, we are opting
to change the locale from ar-AR to ar-JO.

Signed-off-by: Ben Shum <ben@evergreener.net>

Translation updates - po files

Signed-off-by: Ben Shum <ben@evergreener.net>

LP#1653742 Copy tree authoritative API share cstores

Avoid opening one cstore connection per requested org unit in the
open-ils.cat.asset.copy_tree.retrieve.authoritative API by creating a
single shared cstore connection at the top of the API shared by all
API actions.

To test:

1. In one terminal:

% tail osrfsys.log | grep transaction.begin

2. In another terminal:

srfsh% request open-ils.cat
open-ils.cat.asset.copy_tree.retrieve.authoritative "AUTOTOKEN", 217,
[1,2,3,4,5,6,7,8,9]

3. 9 cstore transactions are opened before patching.  After patching,
only 1 is created.

Signed-off-by: Bill Erickson <berickxx@gmail.com>
Signed-off-by: Galen Charlton <gmc@equinoxinitiative.org>

LP 1648918: Remove Workstation at Root of User's Permission Tree

If you register a workstation at the root of the user's permission
tree in the webstaff client, you are not able to later remove it.  The
Remove button is dimmed.  You can remove a workstation registered at
any other org_unit where the user has the REGISTER_WORKSTATION
permission.

To reproduce this bug, login as the admin user and register a
workstation at the CONS org_unit.  Then, go to Workstation
Administration and see that the Remove button is dimmed.

To test the patch, do the same, but see that the Remove button is not
dimmed.  When you click it, the workstation is also removed.

Signed-off-by: Jason Stephenson <jason@sigio.com>
Signed-off-by: Michele Morgan <mmorgan@noblenet.org>
Signed-off-by: Galen Charlton <gmc@equinoxinitiative.org>

LP#1360347: Stamping upgrade script for custom status at receiving

Signed-off-by: Kathy Lussier <klussier@masslnc.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>
Signed-off-by: Kathy Lussier <klussier@masslnc.org>

Database upgrade script fixes

Fixing up a couple of upgrade scripts that I merged this week.

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

LP#1654534: Prevent loop that occurs when staff us 'place another hold' link

A hidden field that tracked the page the user was on before placing a hold was
inadvertently removed in another bug fix. As a result, if staff used the link
to place another hold on the same title, they were stuck in a loop where they
couldn't return to the source page after hitting the 'continue' button. This
commit restores that hidden field.

Also provides a fix so that hold labels used in the place another hold link
can be translated.

Signed-off-by: Kathy Lussier <klussier@masslnc.org>
Signed-off-by: Ben Shum <ben@evergreener.net>

LP#1613341: Stamping upgrade script for punctuation-insensitive patron search

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

LP#1613341 - Patron name search should be punctuation-insensitive

O'Brien will be matched by Obrien, O'Brien, O Brien etc.
Grubbly-Plank will be matched by Grubbly Plank, Grubbly-Plank, Grubblyplank, etc.
This feature is controlled by the same org setting that controls ignoring
of diacritical marks.

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

LP#1541801: grab search field code correctly

When using orderObjectBy, the search field code is now
to be found in a property called 'key'.

Signed-off-by: Galen Charlton <gmc@equinoxinitiative.org>
Signed-off-by: Kathy Lussier <klussier@masslnc.org>

LP#1541801: order search fields in Z39.50 alphabetically

This patch causes search fields in the Z39.50 search interface
in the web staff client to be place in alphabetical order by
field label.

To test:

[1] Open the Z39.50 search page in the web staff client and select
    one or more search services.
[2] Verify that the list of fields is ordered alphabetically.

Signed-off-by: Galen Charlton <gmc@equinoxinitiative.org>
Signed-off-by: Kathy Lussier <klussier@masslnc.org>

LP#1541801: add angular-order-object-by dependency

This adds a orderObjectBy filter that permits the ordering
of results in ng-repeat when the source collection is an
object instead of any array, e.g.,

<div ng-repeat=(foo, bar) in object | orderObjectBy: 'sortfield' ...

The dependency is MIT-licensed and is copyright (c) 2015 Fabricio Quagliariello

Signed-off-by: Galen Charlton <gmc@equinoxinitiative.org>
Signed-off-by: Kathy Lussier <klussier@masslnc.org>

LP#1005040: Stamping upgrade script for realign search layers

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

Adjust comment about apostrophes in opensearch code.  This is a marker for future work.

Signed-off-by: Mike Rylander <mrylander@gmail.com>
Signed-off-by: Kathy Lussier <klussier@masslnc.org>

LP#1005040: Release notes entry for advanced search limiter improvements

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

LP#1005040: Styling cleanup for filter display

1) Use a unicode X instead of the string "Remove"
2) Provide a border around filters
3) Label filters with "Filtered by"
4) Only show "[X filters applied ]" when in mobile mode, and make it
   work like the "Refine these results" button.

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

LP1281280: Allow test script to run without a full installation

--no-connect causes it to do what it says on the tin.

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

LP1281280: Improve query tree compression

In addition to collapsing adjacent nodes sharing the same boolean operator,
we'll now also do the following two things: collapse filters, facets and
modifiers when there exists only a single subnode; and absorb single node
subplans.

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

LP1281280: Implement adjacent-node pull-up optimization

If one has a long list of boolean operations, such as can
be generated by third-party products looking for a book that
might have one of several different ISBNs, those are currently
searched separatelly, in a deep nested set of joined full-text
queries.  This behavior was introduced to address problems of
non-deterministic grouping of boolean operations, along with
them generally not working in complex situations.  We call
the mechanism "boolean push-down", because it explicitly
bifurcates the tree at each boolean operator.

This is suboptimal in the case of adjacent, like boolean ops.

This commit re-compresses the parse tree for adjacent nodes
that use the same boolean operator and are composed of atoms
only (that is, no modifiers, filters, or phrases).  It ignores
any explicit groupings by the user, the "magical" floating
subplan, any subplans that include filters or modifiers, and
any nodes that have a mix of boolean operators between their
atoms.

This is probably more conservative that is strictly necessary,
and phrases would likely be safe, but baby steps.

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

LP#1005040: Add more ignorable filters to the sidebar count calculation

We need to ignore site() and location_groups() when counting filters, as
these have widgets in the search bar.

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

LP#1005040: adjust test cases

* Now reflects change in signature of
  OpenILS::WWW::EGCatLoader::_prepare_biblio_search()
* add test case for change in date filter mapping

Signed-off-by: Galen Charlton <gmc@esilibrary.com>

LP#1005040: show number of filters applied

If at least advanced search filter is applied, the number
of them in force is displayed just below the search bar
next to the refine search link.

This patch also renames "opac-auto-102" CSS class to "refine_search"
and tweaks its left padding.

Signed-off-by: Galen Charlton <gmc@esilibrary.com>

LP#1005040: add filter control widget for publication year

This patch also changes the rewriting of an "is"
pubdate filter from between(value,value) to date1(value),
which should be slightly faster.

Signed-off-by: Galen Charlton <gmc@esilibrary.com>

LP#1005040: teach filter boxes about human-readable crad descriptions

This patch ensures that the human-readable record attribute
description, if available, will be used for the title of each
filter control widget.  It also gives descriptions to record
attributes commonly used for advanced search filters.

Signed-off-by: Galen Charlton <gmc@esilibrary.com>

LP#1005040: display search filter sidebar on lowhits page

With this, if a user over-filters their initial search, they
can more easily remove filters.

Signed-off-by: Galen Charlton <gmc@esilibrary.com>

LP#1005040: various improvements to the filter control widgets

* ensure that filter values are sorted
* fix the link for removing location filters
* don't display a filter box for search_format, as that
  already has a separate drop-down in the search bar

Signed-off-by: Galen Charlton <gmc@esilibrary.com>

LP#1005040: CSS styling of filter control boxes

This patch adds several CSS classes to support
distinguishing filter boxes from facet boxes:

filter_box_wrapper
filter_box_temp
filter_template
filter

It also sets a different background color for the header of filter
boxes. Padding between entries in a filter list is reduced as
compared to facets, both to save a bit of vertical space and to
subtly distinguish filters from facets.

Signed-off-by: Galen Charlton <gmc@esilibrary.com>

LP#1005040: add filter control widgets to TPAC

This patch also moves facet retrieval to after record retrieval, to
make sure facet data is available, and wait for it

Signed-off-by: Galen Charlton <gmc@esilibrary.com>

LP#1005040: implement business logic

This patch gut most of the top level Search/Biblio.pm wrapper,
inlines opensearch search params, uses the new dispach method,
for OpenSRF subrequests, and return the abstract query when
requested.

It also adds CDBI classes for asset.copy_location_group which
is needed for looking them up at search time.

Signed-off-by: Mike Rylander <mrylander@gmail.com>
Signed-off-by: Galen Charlton <gmc@esilibrary.com>

LP#1649180: Add release note for translator make target

Signed-off-by: Ben Shum <ben@evergreener.net>
Signed-off-by: Jason Stephenson <jason@sigio.com>

LP#1649180: Update README to reflect translator make target

Signed-off-by: Ben Shum <ben@evergreener.net>
Signed-off-by: Jason Stephenson <jason@sigio.com>

LP#1649180: Add translator make target

For the four major supported Linux distros:

  - Ubuntu Xenial
  - Ubuntu Trusty
  - Debian Jessie
  - Debian Wheezy

Signed-off-by: Ben Shum <ben@evergreener.net>
Signed-off-by: Jason Stephenson <jason@sigio.com>

LP#1639782: Release note entry for Active Date Column Picker Option

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

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>
Signed-off-by: Kathy Lussier <klussier@masslnc.org>

LP 1656547: Add XML directive to oils_sip.xml.example.

Signed-off-by: Jason Stephenson <jason@sigio.com>
Signed-off-by: Galen Charlton <gmc@equinoxinitiative.org>

LP#1308090: Stamping upgrade script for trim trailing punctuation normalizer

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

LP#1308090: Updating release notes to reflect both parts of this new feature

The code introduced two distinct features related to authors and RDA records,
but only one was highlighted in the release notes. Notes are updated to reflect
both changes.

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

LP#1308090: pgTAP fixes

Signed-off-by: Kathy Lussier <klussier@masslnc.org>
Signed-off-by: Dan Pearl <dpearl@cwmars.org>
LP#1308090 - Additional fix to pgtap test.

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

LP#1308090 Relator fields and facets need normalization.
This incorporates two changes: facets were generated with trailing
punctuation, which resulted in more than one entry for the same item,
differing only in punctuation.  In addition, relator codes were suppressed
in the record detail unnecessarily.

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

LP#1534787 Patron Message Center port

Port patron message center from XUL client to
web client.

Signed-off-by: Kyle Huckins <khuckins@catalystdevworks.com>
Signed-off-by: Christine Burns <christine.burns@bc.libraries.coop>
Signed-off-by: Kathy Lussier <klussier@masslnc.org>

LP#1537217 Precat Checkout Circ Modifier

Precat checkout in the browser client no longer
requires the use of a circ modifier(when circ
modifiers exist).  Similarly, no circ modifier
is chosen by default.

Signed-off-by: Kyle Huckins <khuckins@catalystdevworks.com>
Signed-off-by: Christine Burns <christine.burns@bc.libraries.coop>
Signed-off-by: Kathy Lussier <klussier@masslnc.org>

Docs: Updating to reflect Web staff client

Signed-off-by: Jane Sandberg <sandbej@linnbenton.edu>

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>
Signed-off-by: Chris Sharp <csharp@georgialibraries.org>