Micro update: 4.16.3

Merge pull request #478 from rych/syspref_bug

[#53548549] Duplicate holds followup.

[#53548549] Duplicate holds followup.

Allows multiple holds to be placed on a single title.
Requires multiple 'ok' popup clicks, but the javascript
needs to be rewritten to prevent that.

[#53481867] (fixup) Use proper syspref API in GetDelimiterChoices()

Micro update: 4.16.2

[#52749573] Catch still more solrsync exceptions

Merge pull request #477 from wagnerj/PT53577373

[#53577373] Add authority link search to XSLT 830

[#53577373] Add authority link search to XSLT 830

The 830 link in both staff and OPAC was doing a series title
search by default, not searching against the authority RCN.

Merge pull request #476 from rych/syspref_bug

[#53548549] nudge holds block logic on already-reserved titles.

[#53548549] nudge holds block logic on already-reserved titles.

Logic is fragile and needs to be refactored,
but hopefully this change will give desired
behavior without side-effects.

Merge pull request #475 from rych/syspref_bug

[#53481867] Casing problem in Syspref causes holds shelf action crash.

[#53481867] Casing problem in Syspref causes holds shelf action crash.

Micro update: 4.16.1

[#42577819] (fixup) Include standard-id in Solr schema.xml

[#52749573] Remove uncaught exception possibility from solrsync

The "next unless $doc" evaluates $doc as a string, and that
prompts the Koha::Solr::Document overloading to call the
underlying to_xml() method. This method can fail if there
are render errors, and doing so in this code section means
that the exception does not get caught.

Minor update: 4.16.0

Micro update: 4.15.8

Merge pull request #473 from dlbptfs/PT44086709

[#44086709] Reading history retention - preference in patron record not ...

Merge pull request #471 from wagnerj/PT51649105

[#51649105] Rearrange system preferences; move unused ones

Merge pull request #469 from wagnerj/PT51483347

[#51483347] add 866$z to XSLT display

Merge pull request #468 from dlbptfs/PT40301087

[#40301087] Differing hold expiration date between DB table and notice

Merge pull request #467 from dlbptfs/PT50984189

[#50984189] Expired holds appear on OPAC side but NOT displaying title f...

Merge pull request #466 from dlbptfs/PT31400651

[#31400651] Error adding total for Defaults for this Library

Merge pull request #465 from wagnerj/PT49666551

[#49666551] Add printed music formats to advanced search subtypes

Merge pull request #461 from wagnerj/PT49526067

[#49526067] revise 8xx XSLT display to include subfield t

[#44086709] Reading history retention - preference in patron record not being followed

The main issue was the anonymise_previous_borrowers.pl cron script not
being run when the KeepPreviousBorrower syspref was turned ON.

There was, however, an issue with the placement of the call to the
AnonymiseIssueHistory subroutine.  It was occurring before the latest
return was _MarkIssueReturned.  As a result, it was still in the issues
table and not immediately anonymized.

[#52077905] Add further subfield exceptions to authority matching

Also remove bundled version of NACO module, adding dependency on
CPAN-installed version.

[#51649105] Rearrange system preferences; move unused ones

Moved unused or irrelevant system preferences to a new Ignore
tab.  Hid the following ones entirely from display:

BibliosCataloging
insecure
marc
marcflavour

[#50884419] Actually hide hidden sysprefs

[#38767751] Add batch upload resiliency to solrsync

The solrsync tool uploads renders documents in batches (of 100
by default). If one of those turns out to be rejected by the Solr
server for a schema violation or some such, the entire batch gets
rejected. This patch modifies the failure handling so that
solrsync splits the batch and recursively retries the upload with
smaller and smaller portions of the batch, failing only on the
documents that are specifically objected to by the Solr schema.

Secondarily, it renames the "--batch_size" command line parameter
to "--batch-size" to conform with convention.

[#51483347] add 866$z to XSLT display

Add subfield z (public note) to 866 display in staff and
OPAC XSLT details.

[#51486797] Apply correct pref name for borrowerRelationship

[#50884419] fixups to installer and pref setter

[#40301087] Differing hold expiration date between DB table and notice

The hold expiration date was getting updated in the message_queue table
as a result of off days set via the calender tool, but was not showing
up in the actual email/print notice.  The problem was the expiration date
was being recalculated in the Letters::parseletter subroutine.

[#50984189] Expired holds appear on OPAC side but NOT displaying title for staff

There was an issue on the staff side where the code was always anticipating
an item number.  Since an "any record" hold is not originally attached to a
specific item, this is not always the case.  Added additional logic to account
for this particular situation.

Micro update: 4.15.7

[#50884419] Lots of fixes to syspref handling and normalization

Modifies systempreferences DB schema to store only
variable/value pairs. All other fields are derived from
the values in the defaults file.

Adds a "tags" value to each variable. This is a list
of tabs on which a pref should be included in the prefs
editor. Currently only the first value is supported.

Prefs are no longer case insensitive. The update script
makes an effort to normalize syspref names to the
canonical version.

Removes abililty for the editor to add new prefs, delete
prefs, or edit their type, explanation, options, etc.
There was never a legit use case for this functionality,
and it has the potential to cause considerable trouble.

Attempts to retrieve bogus prefs now result in croaks
rather than warnings.

It's now possible to change a syspref's description
without running incrementing the db schema version.

[#50376721] Use NACO normalization in authority matching

See http://www.loc.gov/aba/pcc/naco/normrule-2.html

Apply normalization rules to authorities and bib headings
to account for typical defects in heading values. This
also removes Solr searching from the match process, instead
storing the normalized value in the database. This obviates
the recently created auth_cache table, so that is dropped.

[#51023687] Add SearchByRcn syspref

Turning on this sypref (the default) will cause Koha to link
heading searches of authors and subjects with the linked_rcn
field. Disabling the pref employs a value search.

Note that this does not address the existing bug where value
searches are constructed using only the $a rather than the
whole field.

[#31400651] Error adding total for Defaults for this Library

SQL statements were expecting an extra parameter to be pushed to them
in a couple of cases.

[#49666551] Add printed music formats to advanced search subtypes

Added Printed Music (leader 6 c or d) to subtype format search
options for staff and OPAC advanced search.

Merge pull request #464 from dlbptfs/PT50774187

[#50774187] SIP_RenewOnIssue conflict with max issue circ rule

[#50774187] SIP_RenewOnIssue conflict with max issue circ rule

When the SIP_RenewOnIssue syspref is ON, there was insufficient logic to
handle the case where the max issue circ rule for a given item type was
reached.  This resulted in the renewal not occurring.

[#46930265] Include entire 245 for Solr title sorting

[#50306713] Properly update and delete cached authority heading values

[#50003917] Patron barcode matches display incomplete data

I hate to repeat myself, but sometimes expedience rules the day.

DB rev increment

Merge pull request #462 from frogomatic/PT5557764A

[delivers PT5557764] Port of Standing Orders code to LK

[#50015971] Further raise limit of linked bibs

[#50007653] Move edit/generate buttons for serials collection

[#50016503] Display $0 in MARCdetail.pl views

[#45854197] Workaround for EBSCO's broken EDS availability checker

Activation of this feature is done by creating a config file option:

<option name="explicit_availability">STRING</option>

The value of "STRING" can be whatever value or even a zero-length string.
The value will be set as the value of the $q subfield of any 952 tags
that have *no* $q already. If the tag has an existing $q subfield, this
option will make no change.

c.f. LAK #41832679

Add module reference

[#25948787] Initialize javascript variable

The "dotransfer" form input wasn't being added in some
cases, causing a JS runtime error when the "ignore"
button was pushed for a pending hold.

[delivers PT5557764] Port of Standing Orders code to LK

Fix record counter for auth import

[#49526067] revise 8xx XSLT display to include subfield t

Revise earlier 800/810/811 display change so that it now
shows both subfields a and t

Merge pull request #460 from ctfliblime/PT49605843-dupe-issues

[#49605843] Eliminate duplicated item issues

Merge pull request #459 from wagnerj/PT49599431

[#49599431] Hide password change boxes

[#49605843] Eliminate duplicated item issues

[#49599431] Hide password change boxes

If the staff user does not have permission to change a
staff password, hide the change boxes. Previously, an
error message appeared but the boxes still displayed.

[#24766391 fixup] Generalize checks for item availability

Adds two functions to C4::Circulation, IsItemAvailable()
and BiblioIsAvailableAt(), aimed at consolidating the logic
to determine whether or not a given item is sitting on
the shelf available for checkout.

[#48583869/#49228801] Fix more authority pager quirks

[#39137907] Include extended attributes in patron searches

Using the 'SearchExtendedPatronAttributes' syspref, libraries can
opt to include extended attribute values when conducting patron
searches via C4::SearchMember(). This includes the main page
search, nav bar search, circ search, and possibly other search
interfaces.

Note that this option potentially has a very high cost associated
with it in that the borrower_attributes.attributes values are not
indexed. A full table search is required for these searches. The
time required to complete a search query will increase linearly
with the number of staff_searchable rows in the table.

Micro update: 4.15.6

Revert "[#44806147] Force AND-ing of grouped search terms"

This reverts commit e7475c9ee46a19f7e9aa694c42e0f1ef766c29ef.

Merge pull request #458 from wagnerj/PT49148271

[#49148271] Limit breadcrumb bib edit link

[#49148271] Limit breadcrumb bib edit link

Limit edit item breadcrumb link to bib record to
users with superlibrarian or edit_bibliographic
permissions.

Merge pull request #456 from wagnerj/PT45526759-rev3

[#45526759] fix installer syntax

Merge pull request #457 from wagnerj/PT49145115

[#49145115] Fix display problem with Firefox and RSS

[#49145115] Fix display problem with Firefox and RSS

Fixes problem with creating/displaying RSS feed in Firefox

Small cleanups in solrsync and C4::Items

[#39138291] fixup: correctly call GetCircControlBranch

[#45526759] fix installer syntax

Fix .sql syntax in a couple of installer files

Merge pull request #455 from wagnerj/PT49063335

[#49063335] Minor fixes to 505 display

Merge pull request #454 from wagnerj/PT46622533

[#46622533] Add 264abc to staff and OPAC display

[#49063335] Minor fixes to 505 display

Fixing typo, spacing issues.

[#46622533] Add 264abc to staff and OPAC display

Add the RDA field 264 subfields a, b, c to staff and OPAC
results and display XSLT, with labels based on second
indicator.

Micro update: 4.15.5

[#21296233] Show all subscriptions in OPAC detail

Clear up some warns

Fix edge case that could crash solrsync

Testing the conditional "unless $doc" causes $doc
to be tested as a string value, which in turn
causes the stringify operator to unroll the doc data
into and XML string. If there's an error in rendering
the XML (char encoding, etc.), XML::Easy will die.

Even in the successful case, turning the doc to XML
is expensive. Testing for definedness rather than truth
is cheap and won't die.

[#48704245] Fix case of multiple subfields included in more_subfields_xml

Merge remote-tracking branch 'origin/ptfs-master' into ptfs-master

Merge remote-tracking branch 'sjw/PT31401085-rev' into ptfs-master

Conflicts:
koha-tmpl/opac-tmpl/prog/en/modules/opac-advsearch.tmpl

Merge pull request #412 from wagnerj/PT33078631

[#33078631] Remove obsolete syspref emailLibrarianWhenHoldIsPlaced

Micro update: 5.15.4

[#24766391] Improve accuracy of checking title's availability

[#30168727] Display session checkouts on circ screen

The list of "session checkouts" is derived from looking at the
list of all checkouts made to the same patron within the past
10 minutes. This allows the librarian to briefly interrupt the
checkout flow (for a search, question about fines, whatever)
and return to the circ screen still seeing the current issue
stack.

However if a patron, for example, were to checkout out a stack
of books with a self-check station and then go to the counter
to check out a special resrve book, the librarian's screen will
still show the entire stack of books even though that librarian
did not personally check them out.

[#39138291] Use branch email for DUE/PREDUE from address

[#21296233] Restore "Serials" tab in OPAC title detail view

[#44086709] Include public and private items notes into Solr keyword index

[#48704245] Include more_subfields_xml with GetMarcWithItems()

[#30482891/#45622583] Alphabetize patron attribute values

[#48583869] Force correct entries_per_page for authorities

[#48599865] Raise max of linked bibs

[#48584845] Ignore punctuation in authority sorting

[#46720347] Clear unused subfield values when assigning new authority

[#44806147] Force AND-ing of grouped search terms

[#48612413] Simplify C4::Matcher::_normalize()