LP1810802 org_top() SQL upgrade repairs

1. Drop the org_top function before recreating it since the return type
changes.

2. Fis syntax error in "COMMIT:"

Signed-off-by: Bill Erickson <berickxx@gmail.com>
Signed-off-by: Jason Stephenson <jason@sigio.com>

LP1765179 - fix issue with pending/staged user reg

After completing a staged user registration, the controller
in regctl.js was refreshing to the same route, and
trying to fetch the same staged user, but never finding it
as it was no longer staged/pending. This addresses that bug
and adds a bChannel refresh to the pending grid, so once
the user is registered in the child tab, it closes the tab and
the pending user grid is updated.

Signed-off by: Cesar Velez <cesar.velez@equinoxinitiative.org>

Signed-off-by: Jason Etheridge <jason@EquinoxInitiative.org>

docs: LP1258628: screenshots & updates to actiontriggers.adoc

Added a handful of screenshots; added language for Message Center definitions, and pulled in language about testing action triggers from this document: http://docs.evergreen-ils.org/dev/_creating_action_triggers.html

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

LP1782652: Make template dropdown larger in Volume/Copy Editor

To test:
1) Open an item in the volume/copy editor.
2) Go to the Copy Templates tab. Note the spacing of the template bar.
3) Create a few templates.  Give them names of 30 characters or more.
4) Go back to the Edit tab and select one of your new templates.  Note
that after you have selected it, the name of your template does not fit
in the dropdown box, and is cut off at the end.
5) Apply this commit.
6) Go to the Copy Templates tab again.  Note that the spacing of the
template bar has not changed.
7) Go to the Edit tab.  Note that the template dropdown has changed
to be much larger.

Signed-off-by: Jane Sandberg <sandbej@linnbenton.edu>
Signed-off-by: Janet Schrader <jschrader@cwmars.org>
Signed-off-by: Jason Stephenson <jason@sigio.com>

LP1806968 Stamping SQL upgrade: Vand. session tracker fixes

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

LP#1806968 Teach Vandelay to pass correct auth tracker type

Pass the correct value ('authority') when creating a Vandelay session
tracker entry for authority records.

Signed-off-by: Bill Erickson <berickxx@gmail.com>
Signed-off-by: Jason Boyer <jboyer@library.in.gov>

LP#1806968 Vand ses. tracker upgrade SQL additions

Force the default value to show in the DB as 'bib'::text instead of
'bib'::vandelay.bib_queue_queue_type, which might be confusing.

Removed some stray SQL from the script.

Signed-off-by: Bill Erickson <berickxx@gmail.com>
Signed-off-by: Jason Boyer <jboyer@library.in.gov>

LP#1806968 Vandelay record_type sql fix

- Change record_type to TEXT rather than an enum, constrained
with the option of "authority" or "bib"

Signed-off-by: Kyle Huckins <khuckins@catalyte.io>
 Changes to be committed:
modified:   Open-ILS/src/sql/Pg/012.schema.vandelay.sql
new file:   Open-ILS/src/sql/Pg/upgrade/XXXX.schema.vandelay-record-type-fix.sql

Signed-off-by: Bill Erickson <berickxx@gmail.com>
Signed-off-by: Jason Boyer <jboyer@library.in.gov>

LP1807523: Adding some randomness to DOM ID generation

Ensures ID values across multiple FmEditor instances in a page will not
collide.

Signed-off-by: Jane Sandberg <sandbej@linnbenton.edu>
Signed-off-by: Bill Erickson <berickxx@gmail.com>

LP1807523: Associating labels and inputs in angular fmeditor

Signed-off-by: Jane Sandberg <sandbej@linnbenton.edu>
Signed-off-by: Bill Erickson <berickxx@gmail.com>

Translation updates - newpot

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

LP#1730726: Stamping upgrade script

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

LP 1730726: Add Release Notes for PostgreSQL 10 Support.

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

Lp 1730726: Fix lp1501781-unaccent_and_squash.pg for PostgreSQL 9.6

PostgreSQL 9.6 apparently exhibits the same behavior as Postgresql 10
when it comes the OE ligature in the unaccent function.  We modify the
test case to take that into account.

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

Lp 1730726: Fix lp1501781-unaccent_and_squash.pg for PostgreSQL 10

PostgreSQL 10 has improvements in the unaccent function used by the
evergreen.unaccent_and_squash function.  The test needs a small
modification to take this into account.  Actual code change from Mike
Rylander with a little typo fix.

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

Lp 1730726: Fix a number of PgTap tests for PostgreSQL 10.

A number of tests disabled output via "\set ECHO".  In PostgreSQL 10,
the lack of a value in \set is an error.  We change these lines to
"\set ECHO none".

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

Lp 1730726: Basic Patch to build the database with PostgreSQL 10.

* Modify the evergreen.org_top() function, with code from Galen
  Charlton, to be non-set returning.

* Modify Open-ILS/src/support-scripts/eg_db_config.in to not strip the
  period from PostgreSQL version numbers and to use the first two
  components when checking for supported PostgreSQL versions.

* Add link from Open-ILS/src/sql/Pg/000.english.pg94.fts-config.sql
  to Open-ILS/src/sql/Pg/000.english.pg10.fts-config.sql.

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

LP#1793585: Add Release Notes

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

LP#1793585: Fix PostgreSQL apt repo source setup

For standalone DB server install, ensure that we are applying the proper
PostgreSQL community apt repo for packages.

DO NOT backport this branch.  There are presently known issues with
Ubuntu 18.04 and email generation from action/trigger and other
interfaces.  We plan to address these before the 3.3 release, but do
not necessarily expect to backport those fixes.

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

LP#1793585: Change to use MARC::File::XML

Use MARC::File::XML instead of MARC::XML (this is ancient and not used)

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

LP#1793585: PGSQL dependency update

The MARC::XML cpan dependency needs to be included as part of the standalone
Postgres server install for Ubuntu 18.04.

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

LP#1793585: Fix Unescaped left brace in regex Perl error.

Running make check results in the following error on Perl 5.26:

Unescaped left brace in regex is illegal here in regex; marked by <-- HERE in m/\${ <-- HERE EMAIL_SENDER}/ at Open-ILS/src/perlmods/lib/OpenILS/Application/Circ/HoldNotify.pm line 358.

Prior to Ubuntu 18.04 and Perl 5.26, this was a deprecation warning.
This commit fixes the regular expressions in HoldNotify.pm to escape
the left braces to make this error/warning go away.

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

LP#1793585: Update README to include Bionic references

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

LP#1793585: Support Ubuntu 18.04 Bionic Beaver

Changes to Makefile.install to support this version of Ubuntu.

Note: We are using PG9.6 from the PostgreSQL apt source instead of the
standard PG10 version that ships with Ubuntu 18.04. Further testing required
before we switch PG versions so relying on a known working version for now.

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

Add Release Notes for Lp 1662535 & Lp 1743783

Signed-off-by: Jason Stephenson <jason@sigio.com>

LP#1743783 Web Client Bill Full Detail Display Issues

Fleshes out retrieved data to be able to display the billing
location for both circulation and grocery bills on the full
details page as well as the owning library for circulation bills.

Signed-off-by: Terran McCanna <tmccanna@georgialibraries.org>
Signed-off-by: John Amundson <jamundson@cwmars.org>
Signed-off-by: Jason Stephenson <jason@sigio.com>

LP#1662535 Patron Bill Grid Columns

1) Adding missing columns, including:

- Call Number
- Owning Library
- Shelving Location
- Last Billing Note
- Last Billing Timestamp
- Last Payment Timestamp
- Last payment Note
- Last Payment Type

2) Corrected path for Type

3) Changed column name for Location to 'Billing Location'
and the former Billing Location to 'Grocery Billing Location' for clarity

Signed-off-by: Terran McCanna <tmccanna@georgialibraries.org>
Signed-off-by: John Amundson <jamundson@cwmars.org>
Signed-off-by: Jason Stephenson <jason@sigio.com>

Docs: Fix screenshot file name

The asciidoc is looking for "My_Lists_dd.png", but the file has a
capitalized extension ".PNG". This commit renames the file to match the
code, since lowercase is our common naming convention.

Signed-off-by: Remington Steed <rjs7@calvin.edu>

Docs: LP#1731048: Update json_query documentation for new join syntax

This commit simply adds the text from the related commit message (see
LP#1527731) to the original DocBook file. NOTE: This documentation is
also available on the wiki, and has been updated there as well:

https://wiki.evergreen-ils.org/doku.php?id=documentation:tutorials:json_query

Signed-off-by: Remington Steed <rjs7@calvin.edu>

LP#1807398 marc_stream/oils_header auth repairs

Fixed a few code thinkos in marc stream workstation login support.

Signed-off-by: Bill Erickson <berickxx@gmail.com>
Signed-off-by: Jason Boyer <JBoyer@library.in.gov>

LP1807398: Add workstation to marc_stream_importer

In order to work properly with vandelay session
tracking in 3.2+ a workstation is required at login.
This branch adds a workstation option to the importer
and alerts the user if it's missing.

Signed-off-by: Jason Boyer <JBoyer@library.in.gov>
Signed-off-by: Bill Erickson <berickxx@gmail.com>

Docs: Followup commit to fix screenshot for My_Lists3.jpg

The previous commit added some new screenshots for My_Lists, but this one
was saved with .JPG extension instead of .jpg so the original file was
retained.

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

docs: update my_lists.adoc for 3.2, with new and updated screenshots

Removed references to Temporary Lists (now subsumed by Baskets) and updated screenshots.

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

Docs: virtual index definitions

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

Docs: Infrastructure changes to authority browse

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

Docs: bibliographic record merging and overlay

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

Docs: Batch actions from search

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

Docs: LP1783387: update print templates to reflect new name

launchpad 1783387 - update receipt template editor documentation to reflect language change to "print templates" in web client
put both terms in index
updated title to Print (Receipt) Templates

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

Docs: LP1488497: Update Org_Unit_Proximity_Adjustments.adoc

Added link to 'best hold selection sort order', per LP1488497

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

Docs: Update purchase_requests_patron_view.adoc

Added some language to match the work done here https://bugs.launchpad.net/evergreen/+bug/1774277 & to match documentation submitted on PR 62

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

Docs: Update purchase_requests_management.adoc

Updated to reflect the work EOLI did here https://bugs.launchpad.net/evergreen/+bug/1774277

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

Docs: Update basic_holds.adoc

Removing reference to XUL

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

Docs: Update marc_tag_table.adoc

Striking reference to "sprint 2 preview".  Adding location information & screenshots.

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

Docs: Binding Template added to G-binding.adoc

Added a section on how to apply a binding template to a distribution as this is necessary to ensure the binding does not fail. Also updated an instance of "Action" to "Actions".

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

LP#1739606: Allow saving Billing History xacts grid

For a web staff eg-grid to save and retrieve its settings, it needs a
unique persist-key. This commit adds one for the Transactions tab grid
under Billing History (on patron record Bills tab, click the History
grid button).

Signed-off-by: Remington Steed <rjs7@calvin.edu>
Signed-off-by: Terran McCanna <tmccanna@georgialibraries.org>
Signed-off-by: Jason Stephenson <jason@sigio.com>

LP#1746769 - add item tag owner OU to label in volcopy editor

Adding the owner to differentiate item tags with same label
that are owned by different OU.

Signed-off by: Cesar Velez <cesar.velez@equinoxinitiative.org>

Signed-off-by: Jason Etheridge <jason@EquinoxInitiative.org>

Forward-port 3.2.2 upgrade script

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

Forward-port 3.1.8 upgrade script

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

LP#1714070 Stamping upgrade script

Signed-off-by: Jason Etheridge <jason@EquinoxInitiative.org>

LP#1714070 Parent/guardian field release notes

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

LP#1714070 Patron parent/guardian field

Adds a new dedicated patron parent/guardian field.  This field is
editable in the patron edit interface, displays in the patron summary
side bar on the browser client, and is search-able from the patron
search interface in the browser client.

Adds patron editor "show" and "suggest" library settings.

Also adds a new library setting
'ui.patron.edit.guardian_required_for_juv' ("GUI: Juvenile account
requires parent/guardian").  When this setting is applied, a value
will be required in the patron editor when the juvenile flag is active.

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

Docs: Copy alerts documentation

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

Docs: removing broken link from circ manual

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

Docs: create autorenewals documentation

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

Docs: correcting syntax error in copy location documentation

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

Release notes entry for LP#1804038

Also, add a missing contributor.

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

LP#1804038 Fix-ups for older 'clense' functions

Way back when, we switched from a misspelled 'clense_ISO8601' to a
corrected 'cleanse_ISO8601' in OpenSRF.  This function has again moved,
and is now part of OpenILS::Utils::DateTime, and is named
'clean_ISO8601'.  This was done as part of LP bug #1552778.

However, a few stragglers of the misspelling remained, so these were
missed when the big changeover occurred.  This hopefully catches them
all!

Signed-off-by: Dan Wells <dbw2@calvin.edu>
Signed-off-by: Jason Boyer <jboyer@library.in.gov>

Docs: Multilingual search documentation

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

Release notes entry for LP#1793154

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

LP1793154: Cancel Holds from OPAC Record

The name of the grid handler was misspelled
in the holds grid for records in the opac.
Swapping the location of the 'wide' part of
the function name will allow it to match up
with the functions available in the holds
grid module.

Signed-off-by: Jason Boyer <jboyer@library.in.gov>
Signed-off-by: Michele Morgan <mmorgan@noblenet.org>
Signed-off-by: Dan Wells <dbw2@calvin.edu>

Docs: documenting multiple emails in patron editor (LP1755625)

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

Docs: release notes for 3.2.2

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

Docs: adding release notes for 3.1.8

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

LP#1643709: Stamping upgrade scripts

Include placeholder 1137 for 3.1 version of the SQL function changed
vs. 1138 which contains the one for 3.2 and master and beyond.

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

LP#1643709 User merge + purge pgtap test

Signed-off-by: Rogan Hamby <rogan.hamby@gmail.com>
Signed-off-by: Bill Erickson <berickxx@gmail.com>
Signed-off-by: Ben Shum <ben@evergreener.net>

LP#1643709 purge users on merge instead of flag deleted

Purge merged user accounts.

Update the upgraded version of the usr_merge() db function to match
recnet changes in master.

Signed-off-by: Rogan Hamby <rogan.hamby@gmail.com>
Signed-off-by: Bill Erickson <berickxx@gmail.com>
Signed-off-by: Ben Shum <ben@evergreener.net>

LP1801156: Add missing assets to 3.2 Offline mode

t_org_select was missing from the UpUp asset list,
causing offline mode to be non-functional when unable
to connect to the server.

Signed-off-by: Jason Boyer <JBoyer@library.in.gov>
Signed-off-by: Galen Charlton <gmc@equinoxinitiative.org>
Signed-off-by: Bill Erickson <berickxx@gmail.com>

LP#1761242 Z39.50 Marc View Usability with Mobile Repsonsiveness

This patch changes the size of the image in the Z39.50 Marc View from large to
medium.  It also incorporates bootstrap's media classes to present the Marc
record and the cover image side by side.  If there is no image the marc record
display floats to the left of the modal. From a suggestion by Jane Sandbert, it
also hides the book cover image on small devices.

To test.
1. Perform a search in z39.50.  An LOC title search for "Cat in the Hat"
   returms results of titles with and without images.
2. View some of the titles in Marc View, noting the large image.
3. Apply the patch.
4. Perform the same search and view the results in Marc View.

Signed-off-by: Garry Collum <gcollum@gmail.com>
Signed-off-by: Janet Schrader <jschrader@cwmars.org>
Signed-off-by: Galen Charlton <gmc@equinoxinitiative.org>

Lp 1753813: Wrap all oils_persist:source_definitions in CDATA blocks.

For safety's sake, we add CDATA sections in all fm_ILD.xml
oils_persist:source_definition elements as show in the example at the
top of the IDL.

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

LP#1764542: Fixing upgrade script stamping

Last commit deleted it, not changed it.  Repairing issue.

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

LP#1764542: Stamping upgrade script

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

LP#1764542 - Update older sites to MODS 3.3

MODS 3.3 support was introduced in March 2009, but this
change was never included in an upgrade script.  Older
sites on 3.1+ will need to apply this update to avoid scary
segmentation faults, and it should be harmless for newer
sites and desirable for those on 3.0 or older.

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

LP#1792621: Fix think-o in Hold Shelf Delay YAOUS test

Signed-off-by: Mike Rylander <mrylander@gmail.com>
Signed-off-by: Rogan Hamby <rogan.hamby@gmail.com>
Signed-off-by: Ben Shum <ben@evergreener.net>

LP#1792621: Ignore deleted items on hold shelf

The hold shelf rewrite brought back an old behavioral bug where items deleted
while on the hold shelf continued to show on the hold shelf list.  The most
recent behavior was to ignore those items, which this commit does.  It also
addresses an adjacent issue where holds may show with "Hold Shelf Delay"
instead of "Ready For Pickup" in some cases.

Signed-off-by: Mike Rylander <mrylander@gmail.com>
Signed-off-by: Rogan Hamby <rogan.hamby@gmail.com>
Signed-off-by: Ben Shum <ben@evergreener.net>

LP1755625: Multiple emails in patron editor

Even though the HTML5 "email" input type allows multiple
addresses to be separated by commas there is nothing that
can be done to signal to Angular.js that this is ok; it
will only ever accept a single address. This patch forces
the type of the input tag to plain text so Angular will
rely on any supplied regex to verify the entry.

The ui.patron.edit.au.email.regex (Regex for email field on patron
registration) library setting can be used to control whether
one or multiple email addresses will be permitted on the
patron registration form.

To allow multiple addresses, that setting can be:

^(?:(?:\b[^@,\s]+@[^@,\s]+\.[^@.,\s]+\b)(?:,\s?(?!$)|$))*$

To allow only a single address, use:

^(?:\b[^@,\s]+@[^@,\s]+\.[^@.,\s]+\b)$

To test
-------
[1] In the web staff client, try to register a new patron, entering
    multiple email addresseses in the email field separated by
    commas.
[2] Note that the email field displays as invalid and the
    record cannot be saved.
[3] Apply the patch and repeat step #1, This time, the email
    field should accept multiple addresses.
[4] Set the Regex for email field on patron registration library
    setting to the multiple-email form listed above. Reload
    the patron registration form and verify that it will
    continue to accept multiple email addresses but will not
    accept entry that doesn't look like an email address.
[5] Set the library setting to the single-address value. Reload
    the patron registration form; this time it should accept
    a valid email address but not multiple ones.

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

Docs: LP#1578719 Update DIG Attributions page

Signed-off-by: Jeanette Lundgren <jlundgren@cwmars.org>
Signed-off-by: Ben Shum <ben@evergreener.net>

LP1490616: Adjust "Penalties & Messages" label (webby)

The pane in the web-based staff client to display penalties and messages
currently has a label of "Staff-Generated Penalties/Messages", which is
misleading as the list includes system-generated penalties and messages.

Modify this to "Penalties and Messages" to avoid the "/" construction
(which is considered harder to translate) and avoid the impression that
it contains only staff-generated penalties and messages.

Signed-off-by: Dan Scott <dscott@laurentian.ca>
Signed-off-by: Ben Shum <ben@evergreener.net>

LP#1091885: Stamping upgrade script

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

LP#1091885: make sure bibs get reingested on undelete

Signed-off-by: Rogan Hamby <rogan@equinoxinitiative.org>
Signed-off-by: Cesar Velez <cesar.velez@equinoxinitiative.org>
Signed-off-by: Ben Shum <ben@evergreener.net>

LP#1757455: followup to fix installing i18n

Actually installing a language also runs updatepo and subsequently attempts
to copy the various dtd and property files to their correct places.

Remove from the Makefile target for install, but retain the dtd web opac
folder which is apparently where we put fieldmapper and some other files
later.

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

LP#1797007 Grid cell tooltips apply to all cells (Ang)

Avoid excessive browser page re-flows, caused by checking cell
widths for rendering tooltips, by rendering tooltips for all cells
regardless of the size of the content.

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

LP#1797007 Grid cell tooltips apply to all cells

Avoid excessive browser page re-flows, caused by checking cell widths
for rendering tooltips, by rendering tooltips for all cells regardless
of the size of the content.

This reverts commit 1e9c0ab388a0697a1f099970240f2da21bfb5e1f.

Signed-off-by: Bill Erickson <berickxx@gmail.com>
Signed-off-by: Jason Boyer <JBoyer@library.in.gov>
Signed-off-by: Galen Charlton <gmc@equinoxinitiative.org>

LP#1757455: Deprecate i18n building for XUL client

Remove older dtds and properties files from the Makefile for i18n build
process.

To test
-------
[1] Apply patch.
[2] Make a change to a XUL properties or DTD file
[3] Run 'make newpot' in build/i18n.
[4] Verify that your change is not reflected in the updated POT files.

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

LP#1783440: add release notes entry

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

LP#1783440: adding UPC to numeric search types in advanced search

To test
-------
[1] Go the Advanced Search page in the public catalog, then to the
    Numeric Search tab.
[2] Search on a UPC value (e.g., in Concerto, "883929159529").
[3] Verify that a hit is retrieved.

Signed-off-by: Rogan Hamby <rhamby@equinoxinitiative.org>
Signed-off-by: Jennifer Pringle <jennifer.pringle@bc.libraries.coop>
Signed-off-by: Galen Charlton <gmc@equinoxinitiative.org>

LP#1684202 - fix holdings grid refresh... bre_id array always be integers

For some strange reason the volcopy bchannel event data.records
would be an array of string integers or a array of ints depending
on whether you're adding or editing holdings. This addresses that
weirdness.

Signed-off by: Cesar Velez <cesar.velez@equinoxinitiative.org>

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

LP#1684202 - fix bug the prevented automatic refresh

Turns out that Array.includes() does not do implicit type convertions
Typical javascript...

Signed-off by: Cesar Velez <cesar.velez@equinoxinitiative.org>

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

LP#1684202: Protect against missing features; Make feature more general

I've wrapped the BroadcastChannel code in a test derived from the auth version
of the same concept.  Also broadcasting the full set of record IDs and the
pre-update call number IDs for other interfaces that might be interested. The
channel name is also made more generic and made to follow the precedent from
the auth channel name.

Signed-off-by: Mike Rylander <mrylander@gmail.com>
Signed-off-by: Chris Sharp <csharp@georgialibraries.org>

Stamping upgrade script

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

LP#1773479 Show Located URIs in browse

Located URIs depend upon bib-level visibility, as there are no copies
to work with.  The browse code, however, was joining in the copy
visibility table as if it would always have at least one row per bib,
but in the case of located URIs, it does not.

Let's change it to a LEFT JOIN to allow the bib row to show up, at
which point the existing bib visibility check can do its job.

Signed-off-by: Dan Wells <dbw2@calvin.edu>
Signed-off-by: Mike Rylander <mrylander@gmail.com>

LP#1724348 Honor default tab from catalog search

The default tab selection was not being honored from catalog searches,
as the search had already set $scope.record_tab, and we were honoring
that value in all cases.

Instead, let's honor that value in cases where the OPAC load doesn't
change our current record, which should only happen if we load the
record directly to a non-opac tab, then go to the OPAC view.

To test:
1) Set any tab other than "OPAC View" as your default view in record
details.
2) Do a catalog search.
3) Select a record, and notice your default view is not set.
4) Apply patch, do the same steps, and notice the default view is now
selected.

To test regression of bug #1708951:
1) In Firefox, load a record directly (e.g.
/eg/opac/staff/cat/catalog/record/123).
2) Note the default view loaded.
3) Click "OPAC View".
4) Note the view does not return to your default view, but stays on
the OPAC.

Signed-off-by: Dan Wells <dbw2@calvin.edu>
Signed-off-by: Jason Boyer <JBoyer@library.in.gov>

LP#1635737 Due date DST noncat thinko fix

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

LP#1635737 Due date DST-aware thinko fix

Minor code refactor to fix thinko and syntax issues.

Signed-off-by: Bill Erickson <berickxx@gmail.com>
Signed-off-by: Dan Wells <dbw2@calvin.edu>

LP#1635737 Apply DST-aware timezone to context dates

Do our best to enforce the rule required by OpenSRF's interval_to_seconds
that when a context date is in use, and you care about DST awareness, you
must set the timezone to a DST-aware value, e.g., 'America/New_York'. In
most situations, 'local' will suffice for this, as the server is typically
configured with a DST-aware timezone in its environment.  However, we will
look for an org unit setting called 'lib.timezone' and use that where we
can.  See LP#1705524 for info on that setting.

Signed-off-by: Mike Rylander <mrylander@gmail.com>
Conflicts:
Open-ILS/src/perlmods/lib/OpenILS/Application/Circ/Circulate.pm
Open-ILS/src/perlmods/lib/OpenILS/Application/Circ/NonCat.pm

Signed-off-by: Dan Wells <dbw2@calvin.edu>
Signed-off-by: Bill Erickson <berickxx@gmail.com>

LP#1635737 Use new OpenSRF interval_to_seconds() context

Use the optional context for interval_to_seconds() to account for the
variable length of duration components.  For example, "1 day" may be
shorter or longer than 24 hours during a time change event, "1 month"
may be shorter or longer depending on which month it is currently, etc.

Also, remove some timestamp munging, as that happens within
interval_to_seconds() already.

Signed-off-by: Dan Wells <dbw2@calvin.edu>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
Conflicts:
Open-ILS/src/perlmods/lib/OpenILS/Application/Circ/Circulate.pm

Signed-off-by: Dan Wells <dbw2@calvin.edu>
Signed-off-by: Bill Erickson <berickxx@gmail.com>

LP#1635737: Unit tests for DST and date math

Signed-off-by: Mike Rylander <mrylander@gmail.com>
Signed-off-by: Dan Wells <dbw2@calvin.edu>
Signed-off-by: Bill Erickson <berickxx@gmail.com>

LP#1635737 Add optional context to interval_to_seconds

Any given interval (e.g. "1 month") can be a different amount of
seconds depending on the context (i.e. "1 month" after February 1 is
March 1, but "1 month" after March 1 is April 1, yet March is longer
than February).  This affects months all the time, but also can
affect days, hours, and even seconds once you consider DST and "leap"
times.

By giving an optional context to interval_to_seconds, you can find
the true number of seconds in, for example, "1 month", when starting
from "February 1" (the context).

Signed-off-by: Dan Wells <dbw2@calvin.edu>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
Signed-off-by: Bill Erickson <berickxx@gmail.com>

Add eg_startup back to .gitignore

Since we moved eg_startup to apache_24, we also need to ignore there.

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

LP#1783602: Remove copy counts from mr search results page

The copy counts on the mr search results page can be wrong if a limiter was
applied to the metarecord search. Rather than supply inaccurate information to
the user, let's remove the counts from that page until a long-term solution can
be developed.

Signed-off-by: Kathy Lussier <klussier@masslnc.org>
Signed-off-by: Remington Steed <rjs7@calvin.edu>
Signed-off-by: Galen Charlton <gmc@equinoxinitiative.org>

LP#1755543: Change placement of help popovers

Set the placement of help tip popovers so that they automatically position
to a visible place on the screen.

Signed-off-by: Kathy Lussier <klussier@masslnc.org>
Signed-off-by: Remington Steed <rjs7@calvin.edu>
Signed-off-by: Galen Charlton <gmc@equinoxinitiative.org>