KPAC files to promote Summer Reading Program for 2016.

LP#1526547 Re-broaden backdate note setting

The previous commit missed a case where we still want the backdate
message to appear.  Let's recentralize it, but at a different level.

Signed-off-by: Dan Wells <dbw2@calvin.edu>
Signed-off-by: Kathy Lussier <klussier@masslnc.org>

LP#1526547 Improve note setting for backdates

The backdate note was the default note for voiding and adjusting, but
this proved to be an unexpected default.

Let's set the note only where we need it, and also genericize the
wording to cover both voiding and adjusting.

Signed-off-by: Dan Wells <dbw2@calvin.edu>
Signed-off-by: Kathy Lussier <klussier@masslnc.org>

LP#1526547 Prevent bogus notes when adjusting lost/lod overdues

If you void/adjust an overdue, but don't supply a reason, the code
automatically adds "VOIDED FOR BACKDATE" to the billing note. This is
obviously wrong for overdues adjusted or voided due to lost (or long
overdue) processing.

This commit simply adds a more appropriate note.

Signed-off-by: Dan Wells <dbw2@calvin.edu>
Signed-off-by: Kathy Lussier <klussier@masslnc.org>

Removing testing piece.

Actually, that never needed to be in parens.

Testing for now, so no commit;

Forgot a closing parens.

LP#1208613 Reset lineitem cache when resetting table

Stale cache entries in this table were causing the "expand all"
functionality to break.  This might be a nuclear option, but as far as
I could see, this "cache" gets repopulated on every pagination anyway,
so I don't think we're losing anything.

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

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

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

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

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

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

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

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

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

Adding PG 9.4 installation targets

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

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

adding a/t environment to new event def

Tweak to place holds screen to show just title instead of extended title.
This should be a usable fix for the group holds bug:
https://bugs.launchpad.net/evergreen/+bug/1454884

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

More changes to post-upgrade data file.

LP1532236 Evergreen should allow for HTML formatted action trigger emails

Simple fix for allowing the AT template to drive the email headers instead
of replacing the headers blindly. Kudos to tsbere for the unless idea.

Testing: setup an action trigger for SendEmail. Make sure your AT template includes
an email header (EG: Content-Type: text/html;charset=utf-8)
Setup a hold/circulation that meets the criteria of your action trigger. Check the
resuting email's headers to match what you had put into the template.

Signed-off-by: blake <blake@mobiusconsortium.org>

Differentiate patron interface URLs for decks

This prevents the deck code from thinking they are the same page, despite both
using the browser wrapper as their base URL.

Signed-off-by: Thomas Berezansky <tsbere@mvlc.org>
Signed-off-by: Chris Sharp <csharp@georgialibraries.org>

LP#1463166 Removed multiple perms

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

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

LP#1463166 Fix Batch update of Funds in Selection Lists

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

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

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

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

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

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

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

adding login redirection page

Adding post-upgrade datawork script

Adding PINES-ified notification wordings.

Adding the PNGs generated from mrpeters' SVG files and adjusting the file paths/names.

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

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

Create Invoice and Purchase Order Search Icon Vectors

Created SVG vector art derived from existing icon art to accomodate missing
search icons for purchase order and invoices in Acquisitions.

Signed-off-by: Michael Peters <mpeters@emeralddata.net>

Adding additional toolbar options for acq.

"Invoices" and "Purchase Orders"

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

These changes undo a couple of small things I tried to tweak on the web
client that didn't work the way I hoped they would.

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

Changes to splash screen and receipt template defaults for web client.

(Note: Could not test on terran-testbox since it is not https-enabled,
so I could not log in to the web client.)

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

Self-Check changes in preparation for upgrade:
Changed wording on some of the on-screen directions.

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

OPAC changes in preparation for upgrade:

1. Added instructions to circ history page in My Account for turning
on your circulation history.

2. Modified behavior or NoveList tab on item details page so that it
is closed by default in the staff client, and it will close when a
different tab is opened to reduce page scrolling.

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

KPAC changes in preparation for upgrade:

1. Biggest change was to move the library selector box to the main
search panel instead of on the side - had to modify quite a few files
in order to make this work and pass the values correctly.

2. Also changed the feature icons to use links instead of search terms
to allow more control over search limits and sorting and to see if
caching the searches would improve response speed.

3. A number of smaller UI changes.

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

Selfcheck Cookie Path

Change base.xul/base.js to support a "cookie path" option to limit what paths
will use the login cookie, then tell the selfcheck interface to use it.

Signed-off-by: Thomas Berezansky <tsbere@mvlc.org>
Signed-off-by: Chris Sharp <csharp@georgialibraries.org>

Additional changes to the self-check interface.

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

Images for new "no image available" functionality.

Lines 94-104 in eg_vhost.conf need to be un-commented.

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

Fixes to OPAC holds & patron self-registration pages

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

Merge branch 'rel_2_9_0' into rel_2_9_1

Conflicts:
ChangeLog
Open-ILS/src/perlmods/lib/OpenILS.pm
Open-ILS/src/perlmods/lib/OpenILS/Application.pm
Open-ILS/src/sql/Pg/002.schema.config.sql
Open-ILS/xul/staff_client/chrome/content/main/about.html
Open-ILS/xul/staff_client/defaults/preferences/prefs.js
README
configure.ac
docs/RELEASE_NOTES_2_9.txt

These are modifications to the Evergreen self-check interface to:

1. Fix bugs with printing holds and fines.
2. Improve page layout overall, and with consideration for
   1024x768 resolution monitors that a lot of libraries use.
3. Hide hold status from view on holds table.
4. Hide buttons/fields from screen when user is not logged in.

Signed-off-by: Terran McCanna <tmccanna@georgialibraries.org>
Conflicts:
Open-ILS/src/templates/circ/selfcheck/banner.tt2

Further adjustments to staff client splash page so it will fit onto
older 1024x768 resolution monitors better.

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

Modifications to patron self-registration form to make it
display better and change library locator link.

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

Miscellaneous OPAC UI tweaks for spacing / layout.

Signed-off-by: Terran McCanna <tmccanna@georgialibraries.org>
Conflicts:
Open-ILS/src/templates/opac/parts/org_selector.tt2
Open-ILS/src/templates/opac/parts/record/copy_table.tt2
Open-ILS/src/templates/opac/parts/searchbar.tt2

Numerous local customizations to My Account pages for layout and
to add on-screen assistance to patrons.

Signed-off-by: Terran McCanna <tmccanna@georgialibraries.org>
Conflicts:
Open-ILS/src/templates/opac/myopac/circs.tt2
Open-ILS/src/templates/opac/myopac/holds.tt2
Open-ILS/src/templates/opac/myopac/prefs.tt2
Open-ILS/src/templates/opac/myopac/prefs_settings.tt2

Staff Client tweaks to splash page:
- Added link to PINES Documentation
- Changed logo at top to PINES logo
- Adjusted spacing
- Updated copyright date

Also, hid footer on OPAC pages while viewing from
within the staff client because it clutters the page
without adding any value.

Signed-off-by: Terran McCanna <tmccanna@georgialibraries.org>
Conflicts:
Open-ILS/web/opac/locale/en-US/lang.dtd
Open-ILS/xul/staff_client/server/index.xhtml

Improved list display in OPAC (title, description, annotations).

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

Removed added content from the KPAC for now since it doesn't
display properly. Possibly add back in later if we can get
the NoveList content to pull in right.

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

Improvements to KPAC list display, especially adding
display of list annotations that were not shown.

Signed-off-by: Terran McCanna <tmccanna@georgialibraries.org>
Conflicts:
Open-ILS/src/templates/kpac/parts/record_row.tt2
Open-ILS/src/templates/kpac/results.tt2

Updated kpac.xml configuration file to be ready to roll out after upgrade.

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

Adding corrected wording for self-registration agreement.

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

Customizations to patron self-registration form and footer link.

Conflicts:
Open-ILS/src/templates/opac/parts/footer.tt2
Open-ILS/src/templates/opac/register.tt2

PINES has already applied 0940

Docs: fix typo for version upgrade 2.8.4-2.9.0 in server upgrade

Oops, got a little ahead of myself, this is more accurate.

Thanks to 2.9 RM Jason Stephenson for noting my mistake.

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

Docs: Missed some acknowledgements in the 2.9.1 point release notes

Missed a few ackwnowledgements in the release notes. Apologies to those who
I missed. Also added bshum since he made a doc change after the release notes
were merged.

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

Docs: Change references to release 2.9.1

In the server upgrade docs, change references so that we are talking
about major series 2.9 now and note the latest version which is 2.9.1.

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

Bumping version numbers, adding Upgrade Script and Changelog

Signed-off-by: Jason Stephenson <jstephenson@mvlc.org>

bumping Perl version string for 2.9.1

Signed-off-by: Jason Stephenson <jstephenson@mvlc.org>

Docs: 2.8.5 point release notes

Adding bug fixes and acknowledgements for the 2.8.5 point release.

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

Adding bug fixes and acknowledgements to 2.9.1. Release Notes

The structure of these point release notes is a little different from what
we did with the 2.8 series. I restructured the headings a bit so that there
is just the one level one header. The two headers in the 2.8 release notes
led to a confusing outline.

Also, I'm adding acknowledgements for each point release rather than adding the
entire batch at the bottom of the file. There are two reasons for this change:

1. The release notes in the official documentation are broken up into different
sections, whereas the ones on the downloads page are available all on one page.
In the official docs, it appeared as if everyone being acknowledged had
contributed to the .0 release, when, in actuality, some had contributed patches
in other point releases. By adding acknowledgements to every point release
section, it is clear who contributed to which release.

2. Consistent contributors get acknolwedgement for every point release they
contribute to, which is a nice thing to do.

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

LP#1516707: Phrase search ignores CD modifiers

Because phrases are subqueries, we need to look at the top of the parse tree
when looking for modifiers for cover density.  This is because modifers are
normally only allowed at the top of the parse tree (or in the floating sub-
query).  We do that by asking the plan's QueryParse for it's reference to its
parse_tree member, which is always the top of the parse tree in QP.

Included in this commit is a small change to the QP testing script to
facilitate testing of this fix.

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

LP1402018_Acq_Copy_location_UI_scoped_to_registered_workstation

This little patch will open up the acq copy location dropdown UI
to include other library's copy locations outside of the workstation
registration. When the user permissions are scoped to more branches.

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

Fix LP1175711, OPAC can't renew item on booking resource list

A test sequence to show the behaviour before and after the fix:

1. Using staff client, check out an item to test account.
2. Go to Copy Details on the OPAC, click Book Now, and cancel the form;
the side effect is to put the item on the Resource list.
3. Using the OPAC on a web browser, log in with test account.
4. Go to Items Checked Out, select the item in the list and click Renew
Selected Items.

Before the fix, we saw the OPAC spin forever, and we saw the Javascript
console of the web browser log the error noted in previous comments.

With the fix, we see an error prompt saying, 'The system is unable to
renew the selected item at this time.  This usually means the item is
needed to fulfill a hold.  Please see a librarian for further help.'

Note: A patron with the normal group of permissions will not be able to
renew an item on resource list.

The fix is to check for an error object emitted by the function
Booking.pm/reservation_list_by_filters(), and to propagate it back to
the client, by returning a call to bail_on_events().

We also clarify the coding of the function's parameter list and how the
function checks for mandatory parameters.

Signed-off-by: Steven Chan <schan@sitka.bclibraries.ca>
Signed-off-by: Josh Stompro <stomproj@larl.org>
Signed-off-by: Kathy Lussier <klussier@masslnc.org>

LP 1516786: 0938 Upgrade Script ends in END and not COMMIT.

Change the END; on the last line of 0938.data.some_rda_attributes.sql
to COMMIT; for reasons of consistency and to avoid issues when wrapped
in larger upgrade scripts using the make release script.

Signed-off-by: Jason Stephenson <jstephenson@mvlc.org>
Signed-off-by: Ben Shum <bshum@biblio.org>

LP#1513554 - Stamping Upgrade Script

Signed-off-by: Jason Stephenson <jstephenson@mvlc.org>

LP#1513554 - Doc update for cancel reason selection

Adding an update to the docs to inform users that default cancel reasons
cannot be selected.

Signed-off-by: Kathy Lussier <klussier@masslnc.org>
Signed-off-by: Jason Stephenson <jstephenson@mvlc.org>

LP#1513554 - Prevent deletion of reserved rows.

It was previously possible to delete reserved rows in
the acq.cancel_reasons table via the UI.  We take a belt
and suspenders approach here:

1) Grey out the checkboxes beside cancel reasons with an
   ID lower than 2000.
2) Create a trigger on the acq.cancel_reason table that
   prevents deletion of rows with an ID lower than 2000.

The trigger executes a new generally available function
for use in similar situations in the future.

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

LP#937789: Stamping upgrade script for part deletion

Signed-off-by: Ben Shum <bshum@biblio.org>
Conflicts:
Open-ILS/src/sql/Pg/002.schema.config.sql

LP#937789: Hide 'deleted' from parts table & editor

This commit hides the new parts 'deleted' field from the Manage Parts
dojo table and popup editor.

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

LP#937789: various improvements to logical deletion of parts

This patch ensures that when a monograph part is marked as
deleted, that mappings between it and its copies are also
removed, similar to the previous behavior of the cascade
on delete.

This patch also adds some pgTAP tests and ensures that
unapi.bmp() and unapi.holdings_xml() get updated during upgrade.

To test the fix for this bug:

[1] Apply the patches.
[2] Create a monograph part on a record, then delete it. Verify that
    it no longer shows up in the parts maintenance interface, but that
    a row in the biblio.monograph_parts table still exists with the
    deleted flag set to true.
[3] Create another monograph part on the same record.
[4] Create a new volume/copy and assign the part created in step
    3 to it. Verify that the part shown in step 2 is *not*
    included in the XUL part selector.
[5] Place a part-level hold. Verify that the hold request
    is displayed correct in both the public catalog and the staff
    interface.
[6] Delete the part created in step 3.
[7] Verify that viewing the list of the patron's holds in the XUL staff
    client doesn't result in any 'network errors'.  Also verify that
    the hold request is still displayed in patron's view in the public
    catalog.
[8] Using SQL or some other means, set the expire_time and
    prev_check_time of the test hold request to a couple days in the
    past, then run the hold targeter.  Verify that the hold gets
    cancelled for lack of a suitable copy to fill it.

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Signed-off-by: Remington Steed <rjs7@calvin.edu>
Signed-off-by: Ben Shum <bshum@biblio.org>

LP#937789: fake-delete support for monograph parts

Unlike copies, call numbers, and bibs, parts can be directly removed from the
database.  This has a negative impact on holds, in that the holds become not
just orphaned, but broken entirely.  With this commit we bring part holds to
parity with other hold types such that they can be system-canceled when their
target goes away.  This will avoid spurious UI-level errors to staff.

Signed-off-by: Mike Rylander <mrylander@gmail.com>
Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Signed-off-by: Remington Steed <rjs7@calvin.edu>
Signed-off-by: Ben Shum <bshum@biblio.org>

Docs: add 'Place another hold link' docs

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

LP#1092018: Make the reports UI use UTF-8.

To test:

[1] Install the translation for a language that uses
    non-Latin-1 characters, such as Russian.
[2] Start the XUL staff client and open the reports
    interface.  Note that labels are mangled.
[3] Apply the Apache configuration changes in this patch,
    reload Apache, clear cache in the staff client, and
    reload the reports UI. The labels should now be
    displayed correctly.

Signed-off-by: Pasi Kallinen <pasi.kallinen@pttk.fi>
Signed-off-by: Galen Charlton <gmc@esilibrary.com>

LP#1511828: sort proximities numerically when targeting holds

The hold targeter had three instances of using lexical sorting
rather than numerical sorting.

Two of the instances affected hold targeting if proximity values were
over 9.  Proximity values were being sorted lexically. For instance,
1,2,5,11,15,100,120 was sorted as 1,100,11,120,15,2,5, causing
interesting hold targeting results. Normally proximity doesn't go
that high so it isn't a problem, but we were using proximity adjustments
to strictly order locations so each location had a different proximity,
driving the numbers above 9.

One instance wasn't causing any current issue but could cause problems in
the future if more best hold selection sort options are added to bring the
total number of sort options over 9.

Also included are some changes to reduce warnings in the logs
suggested by Bill Erickson.  The sorting problem was found by
Galen Charlton, thanks Galen.

Testing Notes - to trigger this issue you need to have adjusted
  proximities over 9.

1. Use proximity adjustment rules to add a +10 proximity adjustment to
  one item circ lib, Branch A and a +2 proximity adjustment to the
  item circ lib Branch B.

2. Find a title with a copy at Branch A and B. Place a hold with a
  pickup location of Branch B.

3. The copy at Branch A should get targeted since it has a proximity of
  12-14 (10+normal proximity) and Branch B has a proximity of 2
  (2+normal proximity).  12 gets sorted before 2 lexically.

4. After the fix the copy at Branch B should get targeted since 2
  is sorted before 12.

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

LP#1400801 - Updated email validation regex for Org Unit admin gui

Grabbed the example of a simplified RFC 5322 email regex from
http://www.regular-expressions.info/email.html and lightly modified it
for case sensitivity and removed the "&" character from the localpart allowed
characters, which causes a 500 server error for some reason.

This matches many more of the valid localpart special characters.

Signed-off-by: Josh Stompro <stomproj@larl.org>
Signed-off-by: Ben Shum <bshum@biblio.org>

LP#1406387 Fix for Holds Placement Advanced Options

In the staff client, when placing a hold and clicking
Advanced Hold Options, the barcode input will populate
with the staff member's barcode if it was previously
empty, regardless of whether the radio input for the
hold was specified for a patron or the staff member.

This patch corrects that behavior, and also populates
the input radio on load after clicking Advanced Hold
Options.

Signed-off-by: Adam Bowling <abowling@emeralddata>
Signed-off-by: Michael Peters <mpeters@emeralddata.net>
Signed-off-by: Terran McCanna <tmccanna@georgialibraries.org>
Signed-off-by: Ben Shum <bshum@biblio.org>

LP#1340852: Retain search params for copy location groups

The order in which we determined what to 'selected' in the "Library"
dropdown put the resolved locg first (which allows for shortnames, e.g.
BR1).  That only gives us half the location group though, which is why
it was bumping up a level.  Now, if we have a location group selected,
it will use that first (since they don't have shortnames); the remaining
select order will be the same.

Signed-off-by: Jake Litrell <jake@masslnc.org>
Signed-off-by: Kathy Lussier <klussier@masslnc.org>
Signed-off-by: Ben Shum <bshum@biblio.org>

LP#1467634: Hide saved searches from screen readers for non-staff client

This commit will only allow screen readers to see the saved searches for
users that are logged through XUL client.
Assuming that a screen reader can be attached to the XUL client.

It should also work for a user logged in through the web staff
client that is using a screen reader.

To test
-------
[1] Apply the patch, then perform a search in the public catalog and
    view the HTML source.  Verify that there's no "sr-only" heading
    for "Saved searches".
[2] Set the opac.staff_saved_search.size library setting to a non-zero
    value.
[3] In the web staff client, perform a search using the embeded OPAC;
    verify that the recent searches list is displayed.
[4] Repeat step 4 in the XUL client.

Signed-off-by: Yamil Suarez <yamil@yamil.com>
Signed-off-by: Josh Stompro <stomproj@larl.org>
Signed-off-by: Galen Charlton <gmc@esilibrary.com>

LP#1503782: fix horizontal scrollbar in MARC import queue inspector

In the XUL client (and webstaff), attempting to view the contents
of a MARC record import queue displays a table that, due to
its width, will almost always have a horizontal scrollbar. However,
attempting to actually use the scrollbar often causes the
focus to jump to the top of the table instead.  This patch
fixes that.

To test:

[1] Create a bib queue that has at least a dozen records or so.
[2] Inspect the contents of the queue, scroll to the bottom of
    the table, and interact with the horizontal scrollbar.
[3] Observe that doing so typically results in jumping up
    to the top of the table.
[4] Apply the patch, clear cache and refresh, and try step #2
    again.
[5] This time, the horizontal scrollbar should act normally.

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

LP#1406786: fix remapping copies during parts merge

This patch fixes an issue with how copies are mapped
on the target part when using "Merge Selected" in the
Monograph Parts interface.

To test:

[1] Create some monograph parts on a record and assign
    some copies to use them.
[2] Use the merge parts function to collapse all of the
    parts created in step 1 onto a single part.
[3] Verify that the copies whose original parts were removed
    are now mapped onto the part that was the target of the
    merge.

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

LP#1509479: set a limit on loops in QP's decompose() method

Following a suggestion of Dan Scott's, add a pair of
suspenders to the belt and enforce an arbitrary limit
of 1000 passes through QP->decompse().

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

LP#1509479: fix infinite loop bug in QueryParser

An unclosed phrase search that has a modifier can cause QueryParser to
enter an infinite loop, tying up open-ils.storage backends.

Examples of such searches include:

  -"cats and dogs
  subject:+"physical chemistry

This patch fixes the bug by allowing the end of the query string
to terminate a phrase (in addition to a quotation mark).

To test:

[1] Verify that the t/21-QueryParser.t unit test passes
[2] To test in a *development* database, before applying the patch
    run one of the example queries.  Observe that no
    results are returned, and that one of the open-ils.storage
    drones is running at 100% CPU.
[3] Apply the patch and restart the open-ils.storage service.
[4] Try the query again; this time, it should return results
    immediately without causing an open-ils.storage drone to
    peg a CPU.

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

LP 1502156: Fix marc_export error when dumping authorities.

There is a chance that marc_export will attempt to call a nonexistent
field on MARC::Record if an error occurs while exporting authority
records.  The bug could only be triggered if a conversion error
occurred while exporting the authority as either MARCXML or USMARC.

The fix is to rename a couple of variables in the Marque::Authority->next
method to be more like those used in Marque::Biblio->next.  This will have
the side effect of making marc_export easier to maintain, since the
variables used in one method will now have the same meaning as those
in the other.

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

Docs 2.9: Add summary of Web Client changes

This commit adds the Web Client release notes summary to the appropriate
docs section. It also updates the wording and links regarding the
completed UI Style Guide internship.

Next we need to copy the Cataloging content into the web client section
and provide web client screenshots.

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

Doc build: use the implicit ID for lib settings editor

We no longer have an explicitly set ID for the library settings editor
section, so update references to use the implicit ID instead.

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

Doc build: link to the right media file

The screenshot for the list of call numbers is a PNG image, not a JPEG
image, so referring to it with the appropriate extension will make it
show up in the docs and enable the epub and PDF builds to successfully
complete.

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

Doc build: update the implicit ID for release notes

The implicit ID for a given heading changes each release as we update
the release notes, so bump 2.8 to 2.9 so that the reference to the ID
can be resolved.

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

Doc build: use XML-compliant IDs

Doc builds that depend on an intermediate XML transform (epub, PDF) also
need to use IDs that comply to the XML standard. In this case, numbers
are not valid NameStartChars:
http://www.w3.org/TR/REC-xml/#NT-NameStartChar

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

Avoid duplicate IDs in doc build

Defining the same ID twice in the same document results in broken builds
that depend on an intermediate XML transform (epub, PDF), so redefine
the second ID in this document.

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

Docs: Documentation for negative balance work

Documentation for negative balance settings in 2.9. The updates in this branch
include:

* Updates of the circulation, patron record, and library settings sections
to reflect the new features contained in the negative balance work.
* General updates, consolidation, and re-organization of some components
of the billing section.

A new understanding and confiuring billing section is still to come, but this
branch brings existing documentaiton up to date.

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

Release Notes: Move/copy relevant sections to Upgrade Notes

This commit moves two sections from "New Features -> Administration ->
Upgrade Notes" to the main level "Upgrade Notes" section, which had been
accidentally left empty and was causing AsciiDoc-to-ePub conversion
errors. This commit also copies two other sections to the main "Upgrade
Notes" section, since they contain information relevant to the upgrade
process.

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

correcting typo causing Chilifresh to continue to display

Correcting a typo that caused the copy_table.tt2 template to break

Enable Halloween holiday feature in KPAC.

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

Conflicts:
Open-ILS/examples/kpac.xml

Change links old library map to new interactive library locator map.

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

Updated link to old library map to new library locator map in KPAC.

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

Added PINES Quick Reports icon and link to Evergreen Staff Client
main page. The link will open in a browser window.

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

Enabled Google Analytics for OPAC usage tracking.

Adds B4 Book Log icon to KPAC search results page and
KPAC item details page.

(This is a PINES-only customization.)

Adds B4 Book Log icon and link to KPAC search results page and
KPAC item details page.

(This is a PINES-only customization.)

Conflicts:
Open-ILS/src/templates/kpac/parts/record_row.tt2
Open-ILS/src/templates/kpac/record.tt2

Adding GALILEO in-library link logic to topnav links.