symlink display_horiz.xul to display.xul

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

LP#1410369: hook up patron message center to XUL staff client

port to alt patron summary ui -- phasefx

This adds a new "Message Center" item in the "Other" menu
on the patron display.

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Signed-off-by: Kathy Lussier <klussier@masslnc.org>
LP#1410369: typo fix

(feel free to squash when merging)

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

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

linebreak after "Confirm the email address"

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

style Alert Message red

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

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

push CSS hook for PATRON_EXPIRED in alt patron summary

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

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

This fixes the invocation of Bill History

when used with this new patron display/summary

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

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

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

bump up upgrade instructions for 2.10.7

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

small typo fix in 2.10 release notes

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

Forward port 2.9.7 to 2.9.8 db upgrade script.

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

schema update script: 2.10.6 -> 2.10.7

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

Docs: Adding 2.10.7 point release notes

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

Docs: Adding 2.9.8 point release notes

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

LP#1623541: Booking will fail if its router isn't called "router"

The circ code checks to see if the booking app is running, and if not, it
skips some booking-related logic. This is a useful performance enhancement
where booking is not in use. It does this by asking the OpenSRF router if
the "open-ils.booking" app has registered with it. Most of the time the
OpenSRF router will be called, unimaginatively, "router" but when it's not,
booking will not be considered by the circ module even if it's running.

Instead of assuming "router" we should simply look up the router's name.

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

Docs: improving documentation for catalog searching widgets

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

LP#1624846 Add charset declaration to TPAC print output

Non-ASCII characters were printed incorrectly in some browsers, such as
Firefox, that still default to ISO-8859-1 encodings in the absence of a
specifically declared encoding.

Our output encoding will always be UTF-8, so declare it.

Signed-off-by: Dan Scott <dscott@laurentian.ca>
Signed-off-by: Mike Rylander <mrylander@gmail.com>

LP#1620803 Add missing workstation passthru to AuthProxy

The AuthProxy.pm login function was not passing through a possible
'workstation' parameter to auth_internal's 'session.create' method.
Without this, all login's end up with a workstation set to the user's
home_ou, which can affect things including opt-in and transit
behaviors.

While not needed for the initial 'user.validate', it seems cleaner
and simpler to just bring it over when constructing the initial
$args hash.

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

LP#1413352 Brief record price sets lineitem price

Propagate lineitem MARC 'price' attribute value to the lineitem
estimated_unit_price field.  This allows users to apply a lineitem
price directly from the ACQ Brief Record interface.

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

forward-port 2.10.5-2.10.6 schema update script

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

bump up upgrade instructions for 2.10.6

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

forward-port 2.9.6-2.9.7 schema upgrade script

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

Docs: Add Additional 2.9.7 Acknowledgments for Testing/Signoff

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

Docs: Adding 2.9.7 Release Notes

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

Docs: Adding 2.9.6 Release Notes

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

let the 2.10.6 release notes be free of errorr

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

first pass at 2.10.6 release notes

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

Translation updates - po files, part 2

Missed the *new* po files in the previous commit.  Number of new
translations for Spanish now stands at 9000+ (wow!).

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

Translation updates - po files

Translations have been significantly increased.  In particular, Spanish
has received a huge update with over 8,000 new translations, Czech has
received a sizeable update of over 800 translations, and additional
smaller updates have been added for Arabic, French (Canada), and
Armenian.

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

LP#1617017: make another TPAC string translatable

"Export List" in hold_history.tt2

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

LP1496556 Claimed never checked out count not increased when applied to LOST circulation

Bug within the Lost copy status checkin block. Needed to set the stop_fines

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

LP#1613709: Make DOB validation alert failure translatable.

The alert message about an invalid DOB being entered in the OPAC
patron registration is not translatable.  The solution, suggested by
Eva Cerninakova, is to wrap the string in the l() template function to
make it so.  That is what this branch does.

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

LP#1616882: mark another string as translateable

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

LP#1616882: mark string as translateable

This patch makes the "Branch relationship" string in TPAC
available to be translated.

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

LP 1583608: The make_release script always makes a db upgrade script.

Alter make_release to always create a db upgrade script, even when no
actual db upgrades need to be applied.  The minimal script created
will at least set the upgrade version in config.upgrade_log.

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

LP#1205492: fix crash when attempting to renew deposit & rental loans in OPAC

This patch fixes a bug where attempting to renew a loan on an item
that requires a deposit or a rental fee can cause an Apache
internal server error.

In particular, this patch supplies descriptions for the following
events:

  ITEM_DEPOSIT_REQUIRED
  ITEM_RENTAL_FEE_REQUIRED
  ITEM_DEPOSIT_PAID

It also normalizes how 'fail_part' is set in the renewal response,
as the payload of an event returned when a renewal fails can
be either a hash or an acp Fieldmapper object depending on the
type of event.  In the former case, attempting to access an
nonexistent ->fail_part method is what causes the crash.

To test
-------
[1] Create an item that requires a deposit or a rental fee
    and check it out.
[2] Attempt to renew the loan in the public catalog. Note
    that an internal server error is returned.
[3] Apply the patch and attept step 2 again. This time, the
    public catalog should display a notification to the patron
    that the renewal cannot take place, rather than crashing.
[4] Set up other situations where a renewal should fail for
    reasons other than a deposit/rental fee being required. Verify
    that appropriate error messages are displayed in the public
    catalog.

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

LP#1612364: Add transparency to some format icons

Some format icons had a white background instead of a transparent one. The
transparent ones will look better in the My Account screens, where the icons
site on a gray background.

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

LP#1614807: Make holds history more responsive

When retrieving the holds history page on a mobile device, re-orient the table
as we do with other My Account screens.

Test plan
Pre-patch, retrieve the holds history screen for a patron that has enabled
holds history. Resize the screen to a width of 600 pixels or smaller. No
adjustments will be made to the table, and horizontal scrolling is required to
see some data.

Post-patch, after resizing this screen, the table should reorient itself with
the headers displaying in the left column, as is done on the other My Account
screens.

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

LP#1614807: Fix spacing issues in responsive design for My Account screens

This commit fixes the following spacing issues that are found in the My Account
screens when loaded on a small-screen device:

* When a field has no value, the text in the table header, which displays along
 the left, is cut off at the bottom. The Notes field on the Holds page is one
place where this issue is commonly seen. With the fix, the headers are slightly
out of alignment with the field data, but this is still an improvement over
cutting off text on the screen.

* If the only title(s) listed on the screen are short, requiring little width
in a table cell, the text in the table header runs into field data. A good
example in the Concerto data is if the user only has a check out for record ID
2 (Le concerto) on their account.

* The checkboxes in the Check Out History and Items on Hold screen were pushed
to the right. We fix it by giving the cells with a checkbox a "checkbox_column"
class, which was already in use by the checkbox on the Items Out screen.

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

LP#1614807: Holds history should look like other My Account screens

The Holds History My Account interface looks entirely different than the Items
Out, Check Out History, and Items On Holds interfaces. This commit adds a new
acct_holds_hist_header CSS class that shares the same styles as the CSS classes
used for the other three interfaces. Inline styles are removed from this table.

Test plan
Pre-patch: Go to the My Account Items Out, Check Out History, Items on Hold,
and Holds History interfaces. While the first three interfaces use a table with
a white background for the headers and a gray background for the body, the Holds
History interface is the opposite.

Post patch: All four interfaces should show a similar style to each other.

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

LP#1614807: Fix Circ History table header display on small screens

Because the table in circs.tt2 and circ_history.tt2 shared the same CSS class,
circ_history.tt2 was displaying circs.tt2's column headers on mobile devices
and other small screens. The two tables now have distinct classes, and
styles.css.tt2 has been updated to provide the correct table headers for the
circ history page.

Test plan
Pre-patch, pull up a user account that has a circ history in My Account, go to
the Check Out History tab, then resize the screen to a width of 600 pixels or
smaller. The table headers will now display along the side, but the headers
will be the ones that typically display in the Items Out screen. The Barcode
will display in the Date Returned row and the Call number will display in the
Barcode row.

Post-patch, after resizing the screen, the headers that display along the side
should match the ones from the Checkout History interface. All data should
display in its appropriate row.

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

LP1593834: Add Date Header to A/T Email Examples

RFCs 822 and 2822 specify that a Date: header has to be sent
indicating when an email was sent. Without this, clients that
default to displaying the Sent Date may end up using a "NULL"
date, or 1970-01-01 00:00:00 UTC, which is not ideal. This
adds the header to the example A/T templates for email and
SMS messages, and includes instructions on correcting
existing tempalates.

Signed-off-by: Jason Boyer <jboyer@library.in.gov>
Signed-off-by: Mike Rylander <mrylander@gmail.com>

LP#1306666: Rename test file

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

LP 1306666: Add Perl tests for new behavior.

These tests check that the copy status is correct after aborting
transits dependent upon the status of the copy.  More tests could be
added to cover more situations, but these tests cover the basics.

Signed-off-by: Jason Stephenson <jstephenson@mvlcstaff.org>
Signed-off-by: Mike Rylander <mrylander@gmail.com>

LP 1306666: Abort Transit Only Change Copy Status if In Transit

When aborting a transit, the copy status is presently changed, either
to the status stored in action.transit_copy or to Reshelving if the
transit was for a hold.  This causes problems with "hanging" transits
where something was not handled properly before.  We've had cases
where a transit sticks around on a copy that is later checked out, and
while that copy is checked out, the transit is aborted, thus setting
the copy status to Reshelving when it should be Checked Out.

The code change in this branch only changes the copy's status if it is
In Transit.  A copy in any other status has its status left untouched.

Signed-off-by: Jason Stephenson <jstephenson@mvlcstaff.org>
Signed-off-by: Mike Rylander <mrylander@gmail.com>

LP#1615025 TPAC large circ history sort repair

Fix TPAC circ history retrieval code thinko so that history rows are
retrieved one at a time instead of in a batch (.atomic) to avoid
overwhelming Ejabberd.

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

LP#1476049: disable serve-cgi-bin Apache config on Jessie

This patch ensures that the stock Apache configuration
"serve-cgi-bin" is disabled when installing on Debian Jessie,
as otherwise the default cgi-bin location
can override what Evergreen specifies for offline.pl.

To test:

[1] Run the Makefile.install step on Jessie; verify
    that there is no /etc/apache2/conf-enabled/serve-cgi-bin.conf
    symlink.
[2] Proceed with the rest of the Evergreen installation; verify
    that the Apache error log doesn't contain a warning like this:

    "The Alias directive in /etc/apache2/sites-enabled/eg.conf ...
     will probably never match because it overlaps an earlier
     ScriptAlias."

[3] For extra credit, verify that one can access offline circulation
    sessions.

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

fixing missing newline

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

resolving merge conflict

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

LP954213 adding MARC Editor documentation somewhat based on 2.1

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

lp1592934 removed lines referencing edition fields 534 and 775. These fields refer to other, different editions of the work and not to the work actually being returned.

Signed-off-by: Jim Keenan <jkeenan@cwmars.org>
Signed-off-by: Kathy Lussier <klussier@masslnc.org>

LP#1612054: unbreak test case now that SSN is no longer an ID type

The patch for bug 1566016 broke the live_t/17-lp1579225_new_patron_passwords.t
test, as it used a patron ID type that is no longer in the seed
data. This patch fixes the test.

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

LP#1576502 Fulfilled holds also should not display as active

Once a hold has been fulfilled, the patron doesn't consider it active, so let's
change the active status for those holds too.

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

LP#1609556: (follow-up) restore previous 852$d output

The previous patch had a side-effect of changing the 852$d
value from the circ library's shortname to its long name. This
patch restores the previous behavior.

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

LP#1609556: use unapi functions to retrieve holdings for SRU/Z39.50

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

LP#1611495 Don't use .xml for MARC21 files in Vandelay Export

The new 'Export Non-imported Records' feature in Vandelay (see bug
LP#1548147) was tweaked before merge to export MARC21 files by default
(rather than MARC-XML). However, the file save dialog still defaults to
(and filters by) ".xml" for the file extension. This is confusing and
otherwise problematic.

Let's go ahead and save the export using a ".mrc" extension instead.
While not an official extension (as far as I know), it is widely used,
easy to understand, and far less likely to cause issues with other
software.

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

Delay Novelist loading slightly

So that if Novelist holds things up other JavaScript, like staff client calls,
are not delayed due to Novelist issues.

Signed-off-by: Thomas Berezansky <tsbere@mvlc.org>
Signed-off-by: Mike Rylander <mrylander@gmail.com>

LP1559281: Allow Baker and Taylor Fake ISBNs

For Content Cafe, jam two B&T entrys into the hash so that Business::ISBN
doesn't register them as invalid prefix ISBNs.

B&T uses internal ISBN's with country codes 630 and 631 for materials
ordered through them when no assigned ISBN exists.  The ContentCafe
added content service knows about these ISBNs, but they are not generally
used by other providers.

To Test:  This only works if you are using the ContentCafe added content provider.

 - Before Patch -
1. Add the isbn 9786315820243 as the only isbn on a record. Remove UPCs also..
2. Clear the added content cache by restarting memcached or using the staff link.
3. View record in catalog and you should see no cover art.

 - After Patch -
1. Clear the added content cache.
2. View the record in the catalog and you should now see cover art for the title
"Video games: the movie"

Signed-off-by: Thomas Berezansky <tsbere@mvlc.org>
Signed-off-by: Josh Stompro <stomproj@larl.org>
Signed-off-by: Mike Rylander <mrylander@gmail.com>

LP#1424656: Fix grammatical error in My Account holds screen

Changes the holds queue position string from 'x hold on y copies' to 'Hold #x on
y copies', which is gramatically correct.

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

LP 1481441: Improve Hold Failure Messages

Rearranging the order of the various sources of hold
failure messages makes it less likely to see spurious
"Item is too new to transit this far" messages. Now
errors due to excessive fines or number of existing
holds.

Signed-off-by: Jason Boyer <jboyer@library.in.gov>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
Conflicts:
Open-ILS/src/templates/opac/parts/place_hold_result.tt2

LP#1608711: Update schema.org property name from "seller" to "offeredBy"

Per the 2015-11-05 release of schema.org
(http://schema.org/docs/releases.html#v2.2), the "offeredBy" property is now
the preferred property to use for non-commercial entities (such as libraries)
making an offer of an item for borrowing, sale, etc.

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

LP#1589586: use all fund debits (even encumbrances) when doing prorate calculations

Signed-off-by: Jeff Davis <jdavis@sitka.bclibraries.ca>
Signed-off-by: Kathy Lussier <klussier@masslnc.org>

Docs: Fix syntax for code block within list

This commit fixes two small AsciiDoc syntax problems that prevented a
line of code from displaying properly. First, it changes the code block
lines from three dashes to four dashes. Second, it adds a plus sign on
the preceding line, to connect the code block to the previous list item.

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

Docs: Small syntax fix, formatting improvements

This commit fixes a small AsciiDoc syntax bug caused by a real '_' that
accidentally closed the '_' italics tag. This block of documentation
will now more closely match the identical content found in
'admin/template_toolkit.txt' (chapter labeled "TPac Configuration and
Customization").

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

Docs: Add tip for using "temp" as a login_types when using "Change Operator: New"

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

Forward port 2.9.5 to 2.9.6 upgrade script.

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

bump up upgrade instructions for 2.10.5

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

2.10.4-2.10.5 schema upgrade script

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

Docs: Adding 2.10 Release Notes

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

LP#1592891: Fix SIP2 standing penalty failures

Update too_many_overdue and excessive_fines in OpenILS::SIP::Patron
to account for the fact that the user object in this context no
longer contains an array of ausp objects, but just an array of ausp
IDs.

This fixes SIP2 failures with patron information messages when a
patron has one or more blocking penalties that are not otherwise
ignored.

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

LP#1592891: tweak tests

This patch wraps the standing penalty checks in evals so that if they
crash, the test script can carry on -- and make it to the point where
it cleans up the standing penalties it adds.

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

LP#1592891: Tests - SIP2 standing penalty failures

Tests for SIP2 standing penalty failures in LP#1592891

Much of the code in these tests was copied or adapted from Jason
Stephenson's tests in 14-lp1499123_csp_ignore_proximity.t

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

LP#1592565 Log successful authtoken and workstation

Recover a previously existing activity log entry that logged the
username, authtoken, and workstation (when available) for successful
logins.

Signed-off-by: Bill Erickson <berickxx@gmail.com>
Signed-off-by: Jeff Godin <jgodin@tadl.org>

LP#1526858: Stamping upgrade script for faster alhr query

Signed-off-by: Kathy Lussier <klussier@masslnc.org>
Conflicts:
Open-ILS/src/sql/Pg/002.schema.config.sql

LP#1526858: speed up queries for shelf-expired holds during checkin

This patch adds indexes on action.hold_request and tweaks
the definition of the alhr IDL view to more efficiently
identify holds that can be canceled if the "Clear Holds Shelf"
checkin modifier is in use.  This can prevent a problem
where such checkins can time out in very large databases.

The tweaks were created by Jason Etheridge and have been in
production at a large consortium for some time.

To test
-------
[1] With a suitably large database, set up an item that
    is on the holds shelf but has passed its shelf-expiration
    time, then check it in using the "Clear Holds Shelf" modifier.
[2] The checkin may fail with a "network failure" message due
    the query on the alhr timing out.
[3] Apply the patch and repeat step 1. This time, the checkin
    should succeed.

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Signed-off-by: Jeanette Lundgren <jlundgren@cwmars.org>
Signed-off-by: Kathy Lussier <klussier@masslnc.org>

Adds two missing link fields (patron profile and patron home library) to
the fm_idl.xml for the Combined Active and Aged Circulations (combcirc)
reporter source. The fix was originally posted in a comment to the
Launchpad bug by Codey Kolasinski.

Signed-off-by: Michele Morgan <mmorgan@noblenet.org>
Signed-off-by: Jeanette Lundgren <jlundgren@cwmars.org>
Signed-off-by: Kathy Lussier <klussier@masslnc.org>

LP#1566016: remove SSN as a stock ident type

This patch removes the SSN (Social Security Number) as a stock
patron identification type, as it is no longer recommended
practice for libraries to store SSNs.

To test:

[1] Verify that in a fresh installation of Evergreen that
    has this patch in place, that SSN is no longer listed
    as a patron identification type.

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

LP#1373578: fix translatability of hold placement errors

This patch fixes a circumstance where ->maketext() could be
passed an empty string, causing it to emit the PO file header text
rather than a desired string.

To test
-------
[1] Configure Evergreen to use a TPAC localization (e.g.,
    the Czech translation).
[2] Enable the "Has Local Copy Alert" library setting.
[3] In the OPAC, change locale and attempt to place a
    hold that would trigger the alert saying that there's
    already a local copy available.
[4] Note that the alert reads as "Project-Id-Version: evergreen
    Report-Msgid-Bugs-To: ..."
[5] Apply the patch.
[6] Try step 3; this time the alert will be translated correctly,
    e.g., "Ve vaší domovské knihovně je již dostupný exemplář."

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

LP 1548993: Remember the Show More/Fewer Details button selection.

The state of the Show More/Fewer Details button selection was
previously not remembered across searches or across changes to
advanced search.  This commit attempts to remedy that.

Signed-off-by: Jason Stephenson <jstephenson@mvlcstaff.org>
Signed-off-by: Jeanette Lundgren <jlundgren@cwmars.org>
Signed-off-by: Kathy Lussier <klussier@masslnc.org>

LP 1548993: Restore Show More/Show Fewer Details button functionality.

When using a show_more_details.default setting of true in config.tt2,
the Show More/Show Fewer Details button functionality is broken.  The
detail record view is always shown, regardless of the button's state.
This patch attempts to repair the behavior so that the user can toggle
the detail record view even though the default is set to true.

Signed-off-by: Jason Stephenson <jstephenson@mvlcstaff.org>
Signed-off-by: Jeanette Lundgren <jlundgren@cwmars.org>
Signed-off-by: Kathy Lussier <klussier@masslnc.org>

bump up upgrade instructions for 2.10.4

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

2.10.3-2.10.4 schema upgrade script

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

fix typos in 2.10.4 release notes

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

Docs: Adding 2.10.4 Point Release Notes

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

LP#1584801: fix tagging of previous circs in metarecord search results

This patch fixes a bug where previously-checked-out items were not
indicated as such in search results when doing a metarecord search.

To test:

[1] Ensure that the opac.search.tag_circulated_items library setting
    is enabled and that you are testing with a user that has opted-in
    to retaining circ history.
[2] Make a couple loans to that user.
[3] Log into the public catalog using that user.
[4] Perform a catalog search, choosing search terms such that
    the previously-loaned items appear in results.
[5] Verify that "I have checked this item out before" is displayed
    on the relevant hits.
[6] Click the "Group Formats and Editions" checkbox; note that
    after the page refreshes, previously-borrowed hits are /not/
    tagged.
[7] Apply the patch and restart the open-ils.search service.
[8] Repeat step 6; verify that this time previously-borrowed hits
    are tagged.

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

LP1567514 - Don't Output Null Byte for Spool Files

If a spool file is defined on the marc_stream_importer.pl
command line, no null byte is output after the record(s)
are processed.

To test:

[1] Use marc_stream_importer.pl to import a record from
    a file; verify that the output messages do not include
    a null character.

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

LP#1517556 - Exclude inactive event defs from find_event_def_by_hook.

Currently when find_event_def_by_hook is called to find an appropriate
event_def based on the hook, it doesn't exclude inactive event defs.  So
strange behavior can happen if inactive event defs are present.

I ran into this issue when I was trying out different versions of the self
check receipt event def.  I had one ative and one inactive while I was
troubleshooting, and when the system would grab the inactive event def by
chance, the self check UI would freeze.  I think it is a resonable expectation
that an event def marked inactive doesn't get used by the system.

To test this out, clone one of the self check event defs, such as
"Self-Checkout Items Out Receipt" and mark the copy as inactive.  Now
go and trigger this event in the self check interface by attempting to
print from the Items Out screen.  It should now fail 50% of the time, since
both the active and inactive defs are returned.

After the fix the inactive event def will always be excluded, so the action
should no longer fail.

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

LP#1580676: fix error message

This patch fixes a typo in the error message presented
if the queued record could not be cleaned up for some
reason; it also reports the correct type of the record.

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

LP#1580676 MARC stream authority cleanup repair

Delete the successfully imported authority record from the vandelay
authority queue instead of the bib record that happens to have the same
ID as the imported authority record.

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

LP 1499123: 0951.schema.config.standing_penalty.ignore_proximity.sql.

The script was not inserting the version into config.upgrade_log when
run standalone because the line to do so was still commented out.
This commit removes the comment characters.

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

Forward port 2.9.4 to 2.9.5 db upgrade script.

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

Docs: Adding 2.9.5 Release Notes

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

LP1497374 fm_IDL.xml metabib::record_attr_flat contains an incorrect link

Corrected the link in fm_IDL.xml. Also added a link to metabib::record_attr_flat
in biblio::record_entry

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