add lost with overdue count in alt patron summary

and fix the total out value.

This ignores the org unit setting circ.tally_lost and hardcodes the behavior.

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

Remove "Confirm email..."

This is better handled by the Invalidate functionality in the patron editor and
the display of penalties already in the summary

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

linebreak after "Confirm the email address"

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

style Alert Message red

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

tweaks to Patron Summary

This commit re-uses the same template for the Patron Summary in both display and
search contexts.  The original one for the display context was the one kept and
is the base for these additional changes:

* Card: has been repositioned
* DOB: and EXP: have been added
* only one phone number is displayed, the order of precedence being Day,
  Evening, and Other.  In other words, if the Day phone is not set, then the
  Evening phone is displayed instead.  And if Evening phone is not set, then
  Other phone is displayed
* Bills: is styled in red and bold, if the amount is greater than $0.00
* If an email address is not set, then the text "Confirm the email address" is
  displayed in the alerts/messages column
* Card: is hyperlinked and copies the barcode to the clipboard if clicked

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

symlink display_horiz.xul to display.xul

disable checkout interface

if an inactive card was used to retrieve the patron via the "F1" interface.

This is the stock behavior outside of this branch, but was obscurred due to
https://bugs.launchpad.net/evergreen/+bug/1154235

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

push CSS hook for PATRON_EXPIRED in alt patron summary

Add Card and OPAC back to Patron Search Summary

Not ideal placement, since it's not in the same place as in the regular Patron
Summary, but I was more concerned about not adding more height to the pane.

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

This fixes the invocation of Bill History

when used with this new patron display/summary

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

more tweaks based on feedback from CW/MARS

Thanks! This restores the original address scheme to the patron display summary,
where the mailing address is displayed under the patron name, if set and valid,
otherwise the billing address is displayed, if set and valid.  No address is
displayed if there is no valid address set for mailing or billing.

This also adds a potential 'Account has EXPIRED' message to the alerts column.

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

enhancements based on feedback from CW/MARS

Thanks!  Major changes includes different patron summary panes based on context,
removal of the new-tab behavior when selecting different patron interfaces, and
having the patron summary pane for searches replace the search form instead of
the list results (I wonder if catalogers would want similar behavior in the
z39.50 interface...)

For the search interface, the summary pane is mostly used to help identify
patrons, and thus focuses on displaying identifying information like addresses
and phone numbers.

For the patron interface, the summary pane focuses mostly on statuses, alerts,
and notifications.

I've backed away a little bit from the "only show information if it exists"
philosophy.  Another school of thought is that information should always be
in the same place to take advantage of spatial memory.  I was hedging my bets
before by hoping column position would be enough for spatial memory.  It might
still be true, but here's to experimentation. :)

For switching between patron interfaces, the different components are still
disintegrated, and get completely reloaded on context switch, including the
summary pane.  I fear this is going to be very noticeable, but it'll take a
major overhaul to have persistently loaded sub-interfaces that stay in sync.

Still, the load times for retrieving a patron are much faster this way, plus
this wrapper for the sub-interfaces is likely easier to maintain, and likely
less of a source of memory leaks than before.

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

Patron Display/Summary Revamp

This is a proof of concept for speeding up the Patron Display and Summary. It
does this by

1) producing the summary entirely server-side via Template Toolkit and perl
2) disintegrating all of the sub-interfaces like Check Out, Items Out, etc.
   These open in new tabs (with their own summary panes), which should help
   break the expectation that these interfaces should be dynamically
   synchronized, but on the downside will annoy those who are used to
   dedicating specific tab positions to specific interfaces.

The summary is less information dense than before, and completely
non-interactive.  The idea is that identifying information is found in the first
column, and the second column is reserved just for showing information like
holds available, alert message, barred status, and standing penalties--i.e.
things that should prompt the staff to dig deeper.  The third and fourth columns
have some other useful information, and though there's no unifying concept for
those at the moment, there is some of the only-show-what's-useful mindset. For
example, if the user has no DOB on record, then we just don't show the DOB at
all, not even a label and a placeholder.

Some caveats/notes:

There are a lot of hard-coded strings.  I will go back and I18N-ize them. :-)

I wanted to have two separate XUL files for search and display, but for now I
have them combined into a single file so that others can test using stock staff
clients. For the same reason, I've also symlinked (yay, git) display_horiz.xul
to the new display.xul.  As a consequence of this, sticky settings are unique
for each filename.

I put PatronSummary.pm under EGCatLoader for expediency, but it should really
live somewhere else, and in particular, make use of oils auth proxy, and not
the TPAC login page when its session cookie expires. :-)

disable Bill Patron button during use

Even though the resulting window is modal, I've seen errors when
trying to spam the button while waiting for that window.

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

Bumping version numbers, adding Upgrade Script and Changelog

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

Bump OpenILS.pm version

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

Changes to smooth out make_release

1) Make the fake 'HeadURL' into a URL which works with our web repo.
Note that this has 'tags/' hardcoded in, so it only works with true
release branches (e.g. rel_x_y_z) not "working" branches
(e.g. rel_x_y).

2) Change range selector when building log to better exclude commits
from previous releases.

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

Forward port 2.5.5 and 2.6.1 upgrade scripts

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

Documentation: Add the authentication proxy chapter

- Some content taken from the 2.2 release notes.
- Added a few steps on how to set up authentication proxy in Evergreen.

Signed-off-by: Robert Soulliere <robert.soulliere@mohawkcollege.ca>

LP#968514: Revive bib/auth importing docs from 2.1, add intro

This commit revives the following sections from the 2.1 version of the docs
and updates them for 2.6:

  - marc_export: Exporting Bibliographic Records into MARC files
    - includes new options for 2.6
  - Importing Authority Records from Command Line
  - Importing Authority Records from the Staff Client

The first two are included in a new section called "Support Scripts"
within the "Developer Resources" section, with a new intro and summary
of other commonly used scripts provided with Evergreen. The third section
is moved inside the Cataloging chapter "Batch Importing MARC Records",
with slight modifications to the intro to that chapter.

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

Docs: Minor wording change

Signed-off-by: Erica Rohlfs <erohlfs@esilibrary.com>
Signed-off-by: Remington Steed <rjs7@calvin.edu>

Documentation: Update upgrade instruction for 2.6.1

- Update location references for 2.6.1.
- Update database updates to include 2.6.1 update.
- Add a note regarding reingesting bib records after updates.

Signed-off-by: Robert Soulliere <robert.soulliere@mohawkcollege.ca>

Documentation: added "search URL" text inside of "using OPAC" section

Replaces temporary text that was accidentally added to the "using OPAC" section.

Signed-off-by: Yamil Suarez <yamil@yamil.com>

Documentation: Added 'Using the OPAC' documentation & OPAC WCAG support mention

Older DocBook content that covered how to do searches on the OPAC was updated
to use Asciidoc. Also a simple mention of the 2.6 OPAC WCAG support was added.

Signed-off-by: Yamil Suarez <yamil@yamil.com>

Documentation for Located URI Visibility

Documentation for the new visibility options available for Located URI's
in 2.6.

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

Fix syntax in filter_group_selector.tt2

Attempt to fix build breakage introduced by 7e4e9d669

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

LP#1326149 Use a TPAC-settable TIME_FORMAT for local time formats

We had tried to use the format.time library setting, but that is meant
for Dojo which uses the Unicode formats, that are incompatible with
the POSIX strftime() formats required by Template::Plugin::Date.

At the same time, we need to expose the raw machine-readable time
for schema.org consumers, so express that in a @content attribute.

And we might as well use a more meaningful <time> element instead of
the meaningless <span>, just for good measure.

Signed-off-by: Dan Scott <dscott@laurentian.ca>
Signed-off-by: Ben Shum <bshum@biblio.org>

LP#1301599 Remove duplicate title attributes

Thanks to Ben Shum for spotting these!

Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Ben Shum <bshum@biblio.org>

LP#1301599 Additional TPAC facets structure markup

Labeling facets with <h4> tags provides additional page structure for
screen readers in the TPAC results pages.

Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Ben Shum <bshum@biblio.org>

LP#1301599 TPAC replace aria-label with title

As appropriate, replace aria-label attributes with traditional HTML
title attributes.

For ref, from the LP entry:

http://www.w3.org/TR/2013/WD-aria-in-html-20131003/#first-rule-of-aria-use

"If you can use a native HTML element [HTML5] or attribute with the
semantics and behaviour you require already built in, instead of
re-purposing an element and adding an ARIA role, state or property to
make it accessible, then do so."

Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Ben Shum <bshum@biblio.org>

LP#1301599 TPAC add missing metabib filter label

In the quick search bar along the top.

Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Ben Shum <bshum@biblio.org>

LP#1301599 TPAC advanced search from labels

Instead of using titles / aria-labels directly on advanced search
selectors, provide selector IDs so the existing text labels can be
turned into <labels> and linked (by id) to their form control.

This means we only need one label per form control, which improves
usability from an accessibility perspective.

Signed-off-by: Ben Shum <bshum@biblio.org>

LP#1301599 TPAC advanced search additional labels

Provides a number of labels, primarily for select widgets, in the TPAC
advanced search form for improved accessibility.

Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Ben Shum <bshum@biblio.org>

LP#1301599 TPAC result table accessibility improvements

* Give the main result table a screenreader-only header to describe the
  table columns.
* Set the role=presentation in one of the sub-tables used only for
  layout.

Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Ben Shum <bshum@biblio.org>

LP#925776 Add placeholder script for backport to 2.5 for function staff uri visibility

Signed-off-by: Ben Shum <bshum@biblio.org>

LP#925776 Stamping upgrade script for staff uri visibility

Signed-off-by: Ben Shum <bshum@biblio.org>

LP#925776 Adding upgrade script

Signed-off-by: Mike Rylander <mrylander@gmail.com>
Signed-off-by: Ben Shum <bshum@biblio.org>

LP#925776: Recheck located uri visibility

Previous to this commit, once we have determined that there are no
local Located URIs or foreign copies, we include the record in the
result set in the staff client.  However, the desire of library staff
is that foreign Located URIs be valid as a visibility limiter.  So
we now recheck at the end for that situation.  This honors the new
"Located URIs act like copies" global flag.

Signed-off-by: Mike Rylander <mrylander@gmail.com>
Signed-off-by: Ben Shum <bshum@biblio.org>

LP#1303544 Trim junk from the ISBN in record summary

When asserting that we are publishing an ISBN, move the "(pbk.)"
and similar such matter outside of the schema.org <span property="isbn">
element.

See "The walking dead" in the sample dataset for an example.

Signed-off-by: Dan Scott <dscott@laurentian.ca>
Signed-off-by: Ben Shum <bshum@biblio.org>

LP#1326983: excluded fulfilled holds when adding hold_request.shelf_expires_soon events

This patch adjusts the example A/T filter for the
hold_request.shelf_expires_soon hook to exclude hold requests that are
already marked as fulfilled.  This saves time creating events that
would immediately be marked as invalid per the HoldIsAvailable
validator.

It also addresses an issue for large databases that have lots of
fulfilled holds wherein hold_request.shelf_expires_soon events can
fail to be added at all due to a cstore timeout.

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Signed-off-by: Ben Shum <bshum@biblio.org>

LP#1326806 Minor repairs to EDI admin documentation

Updates to EDI account Path/Incoming/Account docs.

Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Ben Shum <bshum@biblio.org>

LP#1284832: Add missing seed data: enable combined search for only the subject class by default

Signed-off-by: Jeff Davis <jdavis@sitka.bclibraries.ca>
Signed-off-by: Galen Charlton <gmc@esilibrary.com>

LP#1321363: Suppress Hold Transit use elicits client error

This change restores the behavior as designed.

Signed-off-by: Dan Pearl <dpearl@cwmars.org>
Signed-off-by: Galen Charlton <gmc@esilibrary.com>

Forward port 2.5.4 upgrade script

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

LP#1296937: move the $force_bc parameter of ->charged_items() to an implementation method

Argubably, the $force_bc parameter of OpenILS::SIP::Patron->charged_items()
method exists solely as an implementation convenience for the renew-all
code.  Consequently, this patch removes the $force_bc parameter from the
public interface for charged_items() and spins it off into a new
implementation method.

This patch should be applied at the same time that a reversion of
SIPServer commit c97d64412bc is applied.  Note that a user who upgrades
SIPServer without upgrading Evergreen to a version that contains this
patch will end up in a situation where the code that counts active
loans for the patron information response will fetch the loans using
$force_bc set to true; this works only by coincidence.

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Signed-off-by: Jeff Godin <jgodin@tadl.org>

LP#1321017: Order constituent records by quality

Previously, in some cases (often needlessly) we ordered constituent
records within a metarecord by a combination of type/form/blvl.  This
is not only of little use, but also expensive.  Instead, order them
by the bib's calculated quality, which takes type/form/blvl into account
already.

Also, use the new metabib.record_sorter to find the title tie-breaker
instead of using the view-of-a-view-of-a-view mrd compatability shim.

The net result is several orders of magnitude speed increase for
constituent record retrieval.  In particular, this resolves a problem
where doing a SIP2 patron information lookup and requesting a list
of holds could cause a timeout if the patron has one or more
metarecord holds.

This patch also causes the format filter parameter of the following
methods to be ignored:

open-ils.search.biblio.metarecord_to_records*
open-ils.search.biblio.metarecord.mods_slim.batch.retrieve*
open-ils.search.biblio.metarecord.mods_slim.retrieve*

Since the only use of the format filter was in JSPac code, and since
the record attributes mechanism has changed considerably, this removal
should be considered intentional.

Note that this patch includes a squash of a minor follow-up by Galen
Charlton to remove variables made unused by the original patch.

Signed-off-by: Mike Rylander <mrylander@gmail.com>
Signed-off-by: Galen Charlton <gmc@esilibrary.com>
(for minor follow-up by Galen)
Signed-off-by: Dan Wells <dbw2@calvin.edu>

LP#1310751: Reorder query-building wrapper for QueryParser

In order to make sure that the transparent boolean pushdown does
not cause filters and modifiers to end up deeply embedded in the
query tree, we put them at the front.

This is a short-term fix.  Long-term, we need to make use of the
floating subquery syntax ( {{ ... }} ) available in QueryParser,
which is designed to address the semantic separation between what
the user types into the search box, and the various UI widgets that
can add query clauses.

Signed-off-by: Jason Etheridge <jason@esilibrary.com>
Signed-off-by: Dan Wells <dbw2@calvin.edu>

LP#1322303 cleanse backdate for checkin overdue voiding

Passing raw backdates, particularly from offline transactions, which
have a space between the date and time, to
DateTime::Format::ISO8601->parse_datetime results in "Invalid date
format:" errors.  Pass the date through the cleanse routine first.

Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Ben Shum <bshum@biblio.org>

(documentation) fixed typo.

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

(doc) 'debian-wheezy' was left out of the OS options

Just added 'debian-wheezy' to the OS options, it was left out by mistake.

Signed-off-by: Josh Stompro <Stomproj@larl.org>
Signed-off-by: Galen Charlton <gmc@esilibrary.com>

(doc) add overdues to summary of the phone list feature

Added "or overdues" to the summary so it is clear that phonelist
handles both holds and overdues.

Signed-off-by: Josh Stompro <Stomproj@larl.org>
Signed-off-by: Galen Charlton <gmc@esilibrary.com>

LP#1314920 Stamping upgrade script

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

LP#1314920 change the desc for the org hiding setting

so that it references physical_loc instead of ol

Signed-off-by: Jason Etheridge <jason@esilibrary.com>
Signed-off-by: Dan Wells <dbw2@calvin.edu>

Remove buggy, unused function get_leaders_from_usergroup

Recent editing and testing of get_leaders_from_usergroup() uncovered an
unrelated bug which renders the function useless (it returns all group
members, not just the leader). It wasn't previously caught because
Evergreen has no apparent references to this function. This commit
removes the unused function.

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

LP#1321411: restore ability to handle credit payments using a processor other than Stripe

This patch ensures that the processor name is passed
to OpenILS::Application::Circ::CreditCard::process_payment, avoiding
an exception thrown by Business::OnlinePayment.

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Signed-off-by: Dan Wells <dbw2@calvin.edu>

LP#1246843: Don't show contact invalidators for new users

Don't show contact invalidator buttons for new users. This affects
staged users and cloned users, where the contact fields have a value
when the editor initially loads.

Signed-off-by: Jeff Godin <jgodin@tadl.org>
Signed-off-by: Ben Shum <bshum@biblio.org>

LP#1321429: Use server-local time as best-guess

When first calculating the shelf expire time, we assume "now" as
the starting point by calling DateTime->now().  However, that gives
us the time in UTC.  That's not good because closed dates are stored
timezone-aware.  Instead, ask for server-local time.

Signed-off-by: Mike Rylander <mrylander@gmail.com>
Signed-off-by: Ben Shum <bshum@biblio.org>

LP#1314902: Restore header bar styles on results pages

Commit fc99f4f2 made an incomplete style change, changing
'results_header_bar' to be a class, but not changing the CSS.  The
simplest solution is to restore 'results_header_bar' to be an id, as
this fixes the main view, the responsive view, and also any custom CSS
which might affect this id.

Signed-off-by: Dan Wells <dbw2@calvin.edu>
Signed-off-by: Remington Steed <rjs7@calvin.edu>
Signed-off-by: Galen Charlton <gmc@esilibrary.com>

LP#1312945: pin schema update

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

LP#1312945: add regression test

This adds a test for the change to the
authority.calculate_authority_linking stored function.

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

LP#1312945: baseline schema update

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

LP#1312945: authority.calculate_authority_linking and multiple linked tags

The function "authority.calculate_authority_linking" was not able to deal with
situations where there are multiple marc tags that are linkable. The function
only process the first tag that contains a linking_subfield and returns that
to be inserted into the authority_linking table, even when there are more than
one of the same tag with linking_subfield.

Added an additional loop to loop through each tag, for situations where there are
multiples of each tag.

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

LP#1312945: auth-auth linking: cache less agressively and look for all links

There was a logic inversion that was causing miss-use of a per-
record cache mechanism.  I remove that entirely, as it's not helpful
in practice.

Also, after finding one use of a linked field we moved on to the
next, though it is possible for more than one link to exist for
different uses of a name, say.  Now we'll loop through all fields
to find all possible linkages.

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

LP#1296937: (follow-up) make $ids_only be the last parameter for ->charged_items()

This change ensures that if a site fails to upgrade SIPServer at the
same time that they upgrade Evergreen, the renew all message won't
break on them.

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

LP#1296937: (SIP2) Add ids_only parameter to _items functions

This allows the caller to skip loading of barcodes and/or titles when
they are not needed, such as when the plan is to return counts, not
details.  Per bug 1321017, this is particular useful for patrons
that have metarecord holds.

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

Documentation for adding OpenSearch to Firefox

With some sample screenshots!

Signed-off-by: Ben Shum <bshum@biblio.org>
Signed-off-by: Remington Steed <rjs7@calvin.edu>
Signed-off-by: Jason Stephenson <jstephenson@mvlc.org>

LP#1320048: remove the Date.W3CDTF() JavaScript module

With the closed dates editor now using util.date, and
no other active use of Date.W3CDTF() going on, it can go.

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Signed-off-by: Jason Stephenson <jstephenson@mvlc.org>

LP#1320048: remove last use of Date.W3CDTF() from closed dates editor

To test, verify that the closed dates editor continues to
function normally, i.e.,

* single all-day closures can be added
* a multi-day closing can be added
* one can set a "detailed closing" with a custom start
  time and end time
* existing closures are display properly

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Signed-off-by: Jason Stephenson <jstephenson@mvlc.org>

LP#1320048: correctly display closures in the closed dates editor

This patch fixes a problem introduced by the patch for bug 1187035
that caused the closed dates editor to display all closures as if
they started and ended at the current time.

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Signed-off-by: Jason Stephenson <jstephenson@mvlc.org>

Move authority.record_entry trigger changes out of transaction

This should prevent 'cannot ALTER TABLE "record_entry" because it has
pending trigger events' errors.

This is the same approach used in the original 0875 script, but we
might also work around the problem with a well-placed
SET CONSTRAINTS ALL IMMEDIATE;

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

Preserve script version from 2.6.0

The forward-port had some fixes, so revert those (to be reapplied
in the next commit) to better show fix history for 2.6.1.

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

Forward-port 2.6.0 version upgrade script

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

Documentation: add a 'where' clause to bib record import stored procedure

The stored procedure for doing bib record import was lacking a 'where'
clause in one of the update statements.

Signed-off-by: Yamil Suarez <yamil@yamil.com>

Fix for docs on library information pages

Fixes an AsciiDoc URL processing error.

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

Docs 2.6: Lost Item Billing Min/Max Price Settings

Add a short description for the new 2.6 feature "Lost Item Billing: New
Min/Max Price Settings" to the only section where it seems to fit, and it
seems obscure and hard to find. It could be useful to create a new, more
prominent section for common Org Unit Settings, possibly organized by
category. Sending this via GitHub to test the new Docs workflow.

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

Documentation: added missing word to 'library settings editor' docs

Added "editor" to label of what menu should be used.

Signed-off-by: Yamil Suarez <yamil@yamil.com>

Documentation: Removed bare setting names from Stripe payment doc

Per jboyer-isl's suggestions

Signed-off-by: Yamil Suarez <yamil@yamil.com>

Documentation: Copied Stripe payment info into 'Library Settings Editor'

Only modified slightly from the documentation provided in the EG 2.6
release notes.

Signed-off-by: Yamil Suarez <yamil@yamil.com>

Library information pages

Updating the linked libraries documentation to include details on the new
library information pages introduced in 2.6.

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

Linked libraries docs

Documentation to the 2.5 feature that allows sites to add links in the
catalog from copy details area of the catalog.

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

Only create extension intarray if it does not already exist

Simple change to skip over adding intarray if it already exists in a database
for some reason.

Signed-off-by: Ben Shum <bshum@biblio.org>

Fix 2.6 Release Notes typos and formatting

This commit fixes an AsciiDoc URL processing bug by marking two fake URLs
as lines of code. It also fixes a few small typos and misspellings.

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

LP#1310283: Propogate .staff through Z3950 searches

So that local catalog Z3950 searches don't only show opac visible bibs.

Signed-off-by: Thomas Berezansky <tsbere@mvlc.org>
Signed-off-by: Ben Shum <bshum@biblio.org>

Update docs to point to 2.6 release notes

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

LP#1311467: fix heading validation in bib editor

This patch fixes a regression whereby clicking the validate button
in the bib editor would result in no headings getting checked.

In the process, it enables 'startwith' to be used as a JSON query
operator for all types of predicates, not just the results of
transformed values.

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

Compile and Tweak Release Notes for 2.6.0

Also, add '.txt' to template name, since our build process expects
and uses that extension.

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

Rejigger 2.4.3-2.5.0 upgrade bits

As reported in LP#1261355, Postgres fails in some cases when changing
constraints on existing data inside a transaction.  This may or may not
fix the actual problem, but it at least tidies up these changes, and
will make it much cleaner to move outside the transaction if needed.

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

LP#1308590: Fix schema.org type mapping

The switch to config.coded_value_map for format icon types and labels
inadvertently broke schema.org type mapping. Restore it and attempt
to take advantage of the finer-grained types that are now available
to us.

Signed-off-by: Dan Scott <dscott@laurentian.ca>
Signed-off-by: Galen Charlton <gmc@esilibrary.com>

Translation updates - po files

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

LP#1306176: force installation of Business::Stripe from CPAN

Business::Stripe can fail to pass its tests on platforms that
happen to have Pod::Simple 3.24 or later and Test::Pod.  In
particular, Debian Jessie and Ubuntu Trusty meet those conditions
(since the OpenSRF installation process installs libtest-pod-perl)
and Debian Wheezy with backports enabled does as well.

Since the failing test cases are only for POD, this patches moves
Business::Stripe to the CPAN_MODULES_FORCE list across the board.

This can be undone if/when Business::Stripe releases a fix for
the POD validation warnings.

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

LP#1304559 - stamping upgrade script for vandelay_record_attr_to_flat

Signed-off-by: Ben Shum <bshum@biblio.org>

LP#1304559 Fix slow Vandelay-based imports

Whether a planning limitation or otherwise, we have seen a few cases
where queries run very, very slowly (appear to hang) when using the
updated metabib.record_attr view in a WHERE clause.

The simplest code change is to switch Vandelay to use the intermediate
'_flat' view instead, and early reports indicate that this at least
gets us back to a functional state.

Signed-off-by: Dan Wells <dbw2@calvin.edu>
Signed-off-by: Ben Shum <bshum@biblio.org>

LP#1305958 Fix copy table header ID error

Currently there is a <td headers='copy_header_date_format'> entry in
copy_table.tt2 that does not link to an associated table header with an
ID of that value. The desired target ID is "copy_header_create_date".

Signed-off-by: Dan Scott <dscott@laurentian.ca>
Signed-off-by: Dan Wells <dbw2@calvin.edu>

LP#1306133: Avoid metabib.rec_descriptor where possible

metabib.rec_descriptor has been a view over other data structures
for some time now, but with the advent of the new attribute infrastructure
it is a view over a view over a view, variously transforming IDs to
a slim table to an HSTORE structure and finally to a wide table. Thus,
it is extremely expensive to query. So we won't unless we're asked to.
Which, in the TPAC world, is never.

Signed-off-by: Mike Rylander <mrylander@gmail.com>
Signed-off-by: Doug Kyle <dkyle@grpl.org>

LP#1302113: acknowledgments for 2.6.0

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Signed-off-by: Dan Wells <dbw2@calvin.edu>

LP#1302113: define a file for storing lists of folks to acknowledge in the release notes

If a file called _acknowlegments is present in the docs/RELEASE_NOTES_NEXT
directory, its contents will be appended to the end of the release
notes under an "Acknowledgments" heading when
create_release_notes.sh is run.

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Signed-off-by: Dan Wells <dbw2@calvin.edu>

LP#1301510 Align element name with Fixed Field type

We depend on the field name in the XUL to line up the fixed field
value maps in the database with the appropriate context menus. ELvl
was not capitalized the same in all required spots, causing the
context menu to render but be ineffective.

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

LP#1303987: Fully populate Ills from context menu

The Ills fixed field is 4 characters long (allowing for up to four
types of illustrations to be noted, per 008/006), but the fixed field
editor dropdown menus are not (a) multi-select capable nor (b) ready
to support multi-value-per-field, because, well Ills is the only one
like this, and that would be a pretty big complication to the code.
However, even though multi-select is not supported, we should still
apply the value selected and update the fixed field, otherwise the
value is not updated in the record itself.

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

LP#1281678 Fixed field context menus should trigger close confirmation dialog

Same for using the Physical Characteristics Wizard.

Thanks Dan Wells for the spot.

Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>
Signed-off-by: Remington Steed <rjs7@calvin.edu>
Signed-off-by: Dan Wells <dbw2@calvin.edu>

Reset the displayed mouse cursor

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