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>

LP#1287510 Add missing column to upgrade scripts

If you come via upgrade and not a fresh 2.5 DB, you are probably
missing the action.aged_hold_request.behind_desk column.  Add it in a
harmless way.

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

Bumping version numbers, adding Upgrade Script and Changelog

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>

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>

Stamping 0868 - 'behind_desk' fixes

Signed-off-by: Dan Wells <dbw2@calvin.edu>
Conflicts:
Open-ILS/src/sql/Pg/002.schema.config.sql

LP#1281744 Aged hold purging column repair

Add missing non-NULLable "behind_desk" column to
action.all_hold_request, which is used during holds deletion and
archiving.

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

Stamping 0862 - changes for Encode.pm 2.54+

Also, remove extra XXXX file which someone had preserved by mistake.

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

Upgrade file for Encode.pm 2.54+ changes

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

Add pgTAP test for normalized MARC records

Specifically, we're concerned about ensuring that any changes related to
the fix for Encode (requiring the removal of the effectively useless but
now erroring decode_utf8()) do not inadvertently change the content of
the MARC records on ingest. Because we wouldn't want to have to reingest
everything to normalize search, etc.

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

Test Perl Unicode normalization process

Add some basic tests of our Unicode normalization (and entityization)
process.

Also, 'use utf8;' tells the script that it contains UTF8 characters,
not binary bytes, and resolves a test problem.

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

Encode.pm change to the UTF8 flag

In the past, decode_utf8() silently ignored attempts to decode data
with the UTF8 flag on. However, Encode.pm changed around 2.53 and
resulted in wide-character errors preventing the likes of
--load-all-sample from working.

Since our data should already be UTF8 when it hits these functions, we
should not be trying to decode it in the first place.

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

LP1125471 - Use label instead of id in recording in-house use prompt

When recording multiple in-house use with the library settings for threshold
prompts, the prompt would bring up the ID as the variable instead of the name
of the non-cataloged material. This caused confusion among staff members who
might see a message like: "Are you sure you want to mark 9 as having been used
21 times?" (9 being the id, not the name of the non-cataloged material)

This commit leverages existing code which had been commented out and under
the ID to use the label name for the non-cataloged material instead. Also,
minor bugfix with a missing semi-colon at the end of the function being used.

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

Vandelay: increase queued item creation timeout

Allow up to 10 minutes to create each queued vandelay record to avoid
timeouts in item creation caused by pathologicaly long-running record
match set queries.

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

Keep standard INNER joins above possible OUTERs

We always join metabib.record_attr, and we always use an INNER join
to the core table, so keep that join first.  Likewise with the join
to biblio.record_entry.  Dynamic joins may be OUTER, though, and while
join reordering in Postgres is smart, let's not give it the chance to
fail.

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

Assume that future checkins are junk input

This can happen with SIP2 clients with badly set clocks.

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

LP1046026 - Fix duplicate rows in holds pull list

If there are multiple CAPTURE-blocking penalties we could see dupes on the
pull list.  This avoids that.

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

Prevent odd TPAC base URL redirect

As it stands, in a default install, if you browse to

http://yourserver.com/eg/opac

you are redirected to a login page, and if you login, you then get
to the 404 page. This behavior is caused by the fact that *any*
URLs which do not match a few select cases are assumed to be
"secure" resources, when in fact they might not even exist.

This commit catches any attempt to browse this 'base' and simply
redirects to the "home" page instead.

This addresses the most obvious and likely 404 of this type, but
we might consider catching some other cases (e.g. eg/opac/record/).
In addition (or as an alternative), we could also create a function
call which simply checks the validity of a URL before we ask the
user to login to see it.

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

Fix some javascript errors, trying to access nonexistent widgets

Signed-off-by: Pasi Kallinen <pasi.kallinen@pttk.fi>
Signed-off-by: Melissa Ceraso <mceraso@biblio.org>
Signed-off-by: Ben Shum <bshum@biblio.org>

Include calculated 0-prox locations in the stalling interval for op capture

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

Use copy owning library as the context for finding best hold sort order setting

In the olden days before best hold sort order and proximity adjustment, there was
no need to consider any context org other than "here", because the choices were,
essentially, FIFO or Traditional applied within the hard-boundary range.  However,
now we need to decided whose rule to follow, and the most correct arbiter of that
is the owning library of the call number of the copy.  We avoid copy circ lib
because that can change with floating copies.

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

LP1155769: Items repeated in tpac display, others not visible

This was an issue with the query and ordering by a column that was
sometimes null (copy_number).  Added barcode to be a sort rather than
replacing copy_number, per bshum's suggestion.  Not replacing allows
libraries that use copy_number to continue to use them as well.

Signed-off-by: Kyle Tomita <tomitakyle@gmail.com>
Signed-off-by: Ben Shum <bshum@biblio.org>

LP#1164720 Prevent empty names in TPAC lists

Form submission for list creation with an empty name value now prevents
list creation and redirects the user to a new error page explaining why
list creation failed.

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

Skip duplicate username check when username unchanged

Changing a username, then changing back to the user's current
username results in a duplicate username detection -- a false
positive due to the API call finding a user with that username --
the current user. This prevents you from saving any other changes
you may have made to the user.

This can be prevented by skipping the duplicate username check if
the user is not new and the username in the field matches the
username stored in the in-memory Javascript representation of the
user.

There's still a potential corner case which could be avoided by
using a new/modified API call to either accept an "except this
au.id" argument or to return the au.id of the found user, but I
don't see a need for that at this point.

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

LP1271218 - clarify how opt-in user notifications are shown in my account

As reported in LP127128, end users may not need to be shown the message that
"No notification preferences are configured" when in fact this message is only
displayed when the user opt-in notification settings are not configured.

So, alter this to only show the table and div when there are settings to be
configured and otherwise show nothing.

Signed-off-by: Ben Shum <bshum@biblio.org>
Signed-off-by: Chris Sharp <csharp@georgialibraries.org>
Signed-off-by: Ben Shum <bshum@biblio.org>

Revert "Eliminate an annoying and useless warning in the JavaScript Console."

This reverts commit 88cd3947a1b8394bd0307cbd2f6eedb8e98e3668.

My fault for picking this in without more testing. This broke things like noted
in LP1275074.

Include RFID docs with full path

Ben Shum noticed that the RFID docs weren't showing up in the documentation; we
need to include the full relative path to the RFID docs.

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

Forward port 2.5.2 upgrade script

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

LP#842991 Reports needs to error when deleting fails

When attempting to delete a report template, the reports interface
incorrectly shows "Action Succeeded" even when it partly fails. If the
user performing the delete does not own all the dependent reports and
outputs, then the perl function returns 1 to indicate that some were
deleted, but not all. However, the javascript that calls that function
does not check for a return value of 1. This commit adds that check, and
also improves the formatting of the related error messages.

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

Serials: improve routing slips per Dan Wells' suggestions on LP #1229349

  - Don't repeat the same address for the same user
  - Label mailing vs. billing
  - Border to separate mailing and billing addresses
  - [Also] move some inline styles to a CSS file

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

Fixed title/author display at checkout for non-pre-cat items.

Fixed a display problem on checkout where items that were formely pre-cats
were still displaying the pre-cat data. Now it will make sure the item is
still a pre-cat before using the dummy data.

Signed-off-by: Steven Callender <stevecallender@esilibrary.com>
Signed-off-by: Ben Shum <bshum@biblio.org>

TPAC: Display authors using inline-block

Rather than letting author credits wrap willy-nilly, use inline-block mode so
that each line will begin with a new author. Devices with constrained screens
in particular benefit from this, as the list of authors is much easier to scan,
and wider screens can still fit multiple authors on a single line.

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

TPAC: Use indexed subfields only in author search links

Addresses LP# 1267231 in which we found that the titles of works in the added
author field (such as subfield t) were showing up in the link ahead of the
author's birth and death date (if applicable). Now we reserve the link for
only the indexed author subfields (depends on whether the name is personal,
corporate, or conference, but generally subfields a/b/c/d/e/n/q), then the
extra subfields go after the name + dates, then we finally put the relationship
into the parentheses after everything else.

We also simplify the markup so that each name is contained in a single <span
class="rdetail_author_div"> element to make it easier to control the layout.

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

Documentation: Update upgrade instructions to 2.5.2

Update instructions to point ot latest release in 2.5 series.
Fix download location which was pointing to preview directory and not the stable release download directory.

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

Serials: Batch Receive interface Copy Location widget was inert

Choosing a value from the Copy Location widget wasn't working.  You'd
get a default copy location (or the one supplied by the Receive Unit
Copy Template) no matter what you set that control too.

This is fixed.  One does not read from XUL menulist element by using its
'value' property.

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

LP#882586: In Serial Control View, show copy templates owned by ancestors

When creating/editing distributions, dropdowns for Receive Unit Template
and Bind Unit Template now show copy templates owned not only at the
distribution's holding lib, but at any of the holding lib's ancestors.

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

Pull forward previous upgrade scripts

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

Create 008 - take only the first field/subfield combo

In the event that there were multiple $a subfields in, say, 044, the create 008
function could spew raw XML into the 008 field. Avoid that by taking the primary
instance.

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

Allow generating fixed data in control field 008 quickly

Language and country codes come from 041a and 044a,
Date1 comes from 260c, and the record date is set to current day.

Signed-off-by: Pasi Kallinen <pasi.kallinen@pttk.fi>
Signed-off-by: Dan Scott <dscott@laurentian.ca>
Conflicts:
Open-ILS/xul/staff_client/server/cat/marcedit.js

LP#1235474: fix failure of authority browse context menu to render

This patch fixes a situation where if an authority record that would
be brought up by the authority browse context menu in the MARC editor
lacks an 003 field, the context menu would not be rendered all, meaning
that the default context menu (e.g., cut/copy/paste/etc.) would be
displayed instead.

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

Move hard-coded style out of the copy status layout.

Signed-off-by: Pasi Kallinen <pasi.kallinen@pttk.fi>
Signed-off-by: Dan Scott <dscott@laurentian.ca>

LP104785: Selfcheck needs to be run with HTTPS

Redirect attempts to connect to the self-check via HTTP to the more secure
HTTPS instead. Addresses this bug reported in launchpad:
https://bugs.launchpad.net/evergreen/+bug/1047485

Based on work for Apache 2.2 by Michael Peters <mrpeters@library.in.gov>,
updated and applied to 2.4 by Elliot Voris.

Signed-off-by: Elliot Voris <evoris@slcconline.edu>
Signed-off-by: Dan Scott <dscott@laurentian.ca>

More explicit database configuration instructions

These README tweaks should help new folk install Evergreen with
a few less hurdles.

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

Fedora: Net::SSLeay 1.55 is too new for Net::HTTPS::Any

Although Net::HTTPS:Any wants Net:SSLeay 1.30 or later, 1.55 is apparently too
late, or something. The alternative is Crypt::SSLeay, which seems to work, so
let's go with that.

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

LP#1269884 repair MVR id / doc_id thinko

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

Replace deprecated javascript escape() with encodeURIComponent()

Signed-off-by: Pasi Kallinen <pasi.kallinen@pttk.fi>
Signed-off-by: Dan Scott <dscott@laurentian.ca>
Conflicts:
Open-ILS/xul/staff_client/chrome/content/auth/controller.js
Open-ILS/xul/staff_client/chrome/content/util/error.js
Open-ILS/xul/staff_client/chrome/content/util/print.js
Open-ILS/xul/staff_client/server/admin/offline_manage_xacts.js
Open-ILS/xul/staff_client/server/circ/print_list_template_editor.js

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

Renumbering the upgrade script

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

Upgrade script

Signed-off-by: Mike Rylander <mrylander@gmail.com>
Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>
Signed-off-by: Mike Rylander <mrylander@gmail.com>

Fix proximity adjustment calculator

The canonical use case for proximity adjustment is to use it broadly
to effect the local proximity of items within a particular area, in
a like manner, across many parts of the org tree.  However, in cases
where it is lightly used, or used with non-overlapping criteria columns,
there are cases where the adjustment will spread beyond its intended
range of influence.

The changes here create a much stricter and correct test for rule
matching, which will end in the correct choice of rules in all cases.

Signed-off-by: Mike Rylander <mrylander@gmail.com>
Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>
Signed-off-by: Mike Rylander <mrylander@gmail.com>

make_release cleanup and options

* Avoid packaging the XULRunner packages within the Evergree tarball

* Skip the inline SQL upgrade script edit step by default; continue
  supporting option inline editing with a new -r option.

* New -i option forces the script to bypass the lengthy i18n process,
  which is primarily useful for quickly testing changes to this script.

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

In prereq installer, don't try to chown extracted files to original UID/GID

As this Makefile is designed to be used by a make process running as
root, this will avoid problems on some systems when tar extracts
files and tries to give them the same UID/GID as they have in their
archives, but those accounts don't exist on the local system.

e.g, libdbi has files set 1000/1000. On most systems, there's a user
and a group with these IDs, but there can be systems where nothing
uses those IDs, and where tar fails with e.g. 'tar: libdbi-0.8.3:
Cannot change ownership to uid 1000, gid 1000: Invalid argument'

Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>
Signed-off-by: Mike Rylander <mrylander@gmail.com>

LP#1234201: fix menu item to display patron requests (if summary is horizontal)

This patch adds a missing command element to the horizontal
patron summary display so that the 'Acquisition Requests' menu
item will work.

To test:

[1] Set the Patron circulation summary is horizontal library setting
    to true.
[2] Retrieve a patron and try using the Acquisition Requests menu
    item.  It will do nothing.
[3] Apply the patch and clear cache in the staff client.
[4] Repeat step 2.  This time, the Acquisitions Request menu item
    should bring up the list of the patron's requests.

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

LP#1269042: prevent acq seach from building dropdown of every copy ID in the DB

This patch ensures that if a user sets a search field in the
acquisitions general search form to the line item details copy ID
(labeled "LID - Evergreen Copy ID"), the form presents a normal
input widget rather than trying to build a drop-down containing
every copy ID in the database.  Except for very small databases,
the previous behavior would mean that an open-ils.pcrud backend
would eventually consume a great deal of memory trying to service
a request for all copies in the system.

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

Treat empty username as invalid in user editor

There seems to be undesired interaction between the required
attribute and the isValid method on a dojo/dijit ValidationTextBox.
If both are set, isValid needs to check for an empty value,
otherwise the field is considered valid the moment it gains focus.

We don't want an empty username to be considered valid, because we
know that we will be unable to save the user.

We can work around this by teaching the isValid function to consider
an empty value as invalid.

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

LP#1267224: repair another fm_IDL.xml typo

vam.eg_record points to authority.record_entry, not
biblio.record_entry.

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

LP#1267224: Repair IDL typo for vandelay authority match / rec link

"vqbr" should be "vqar" for authority records.

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

LP1038240 - Patron editor field documentation does not display text

Changed the legend tag to have a closing tag.
Also modified the css id uedit-help-div, to add a top margin so
it does not overlap the save div.

Signed-off-by: Kyle Tomita <tomitakyle@gmail.com>
Signed-off-by: Bill Erickson <berick@esilibrary.com>

LP#1266937: Fix missing/incorrect regex for authority in Vandelay.pm

Add in missing "~" in function import_record_list_impl when
type is authority records. Also replace "auth" with "authority"
so that the generated values match up with functions in the
database.

Signed-off-by: Srey Seng <sreyseng@gmail.com>
Signed-off-by: Bill Erickson <berick@esilibrary.com>

fix pub date data type in reporter.classic_item_list

As noticed by Ben Shum in IRC, reporter.classic_item_list
had pub_date, a free text field, classified as "int". This
changes that to "text" to prevent errors.

Signed-off-by: Chris Sharp <csharp@georgialibraries.org>
Signed-off-by: Ben Shum <bshum@biblio.org>

Fix bug that caused deleted holdings to be visible in SRU/Z39.50 results

Signed-off-by: Jeff Davis <jdavis@sitka.bclibraries.ca>
Signed-off-by: Ben Shum <bshum@biblio.org>

Stamping 0850 - translate icon labels

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

Translate the icon labels in TPAC

Instead of just using English.

Also, in the event we have a search_label, use it.

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

Stamping 0849 - desk renewal description

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

Updated the label on the desk renewal global setting.

Updated the label on the desk renewal global flag circ.desk_renewal.use_original_circ_lib
to properly explain the it's the workstation library that will not be used rather than the
user home library. The OPAC renewal uses the users home library and desk renewals use
the workstation library.

Signed-off-by: Steven Callender <stevecallender@esilibrary.com>
Signed-off-by: Ben Shum <bshum@biblio.org>

Break out of focus() in cat/z3950.js if obj.active_services is undefined.

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

Eliminate an annoying and useless warning in the JavaScript Console.

Timestamp: 12/10/2013 02:52:51 PM
Warning: Empty string passed to getElementById().
Source File: oils://remote/xul/rel_3_0_20131206_120120/server/OpenILS/global_util.js
Line: 354

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

Asciidoc fix

Another asciidoc fix for 2.5 documentation:
-Disable patron credit.

Signed-off-by: Angela Kilsdonk <akilsdonk@esilibrary.com>

Documentation for 2.5

asciidoc fixes for:
-Disable Patron Credit
-Lost and Long Overdue Colors
-PV Supa Goodstuff Integration

Signed-off-by: Angela Kilsdonk <akilsdonk@esilibrary.com>

Documentation for 2.5

Includes:
-Patron Self Registration
-PV Supa Goodstuff Integration
-Disable Patron Credit in Billing Interface

Signed-off-by: Angela Kilsdonk <akilsdonk@esilibrary.com>

LP#1254816: prevent cases where a Google Book preview is not displayed

Due to quirks in how different parts of the Google Books API
processes ISBN searches, it is possible for a Google Books Preview
badge to get displayed, but when the user tries to display the preview,
the embedded preview does not get displayed.

This patch fixes the problem by grabbing the preview link from
the GB volumes search results, which in turn can be fed into the
Embedded Viewer initialization.

An ISBN that can be used to reproduce the problem is 9781937994198.
Before the patch, the badge is displayed but not the preview; after
the patch, both badge and preview should be displayed.

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

Documentation for 2.5

asciidoc fix

Signed-off-by: Angela Kilsdonk <akilsdonk@esilibrary.com>

Documentation for Sorting Billing Columns

Docs for sorting money columns in billing interface.  In ersions 2.3-2.5.

Signed-off-by: Angela Kilsdonk <akilsdonk@esilibrary.com>

New documentation for 2.5

Documentation for:
-Z39.50 Search Enhancements
-Catalog Browse
-Alternative Hold Pick-up Location
-Marking Items Long Overdue
-Lost and Long Overdue Bill Colors
-Vandelay Item Load

Signed-off-by: Angela Kilsdonk <akilsdonk@esilibrary.com>

Fix LP1180140, View Holds not working for a serial with subscription and no issuances

A request is made to
open-ils.circ.holds.retrieve_all_from_title,
which is the function Circ/Holds.pm/all_rec_holds().

In the function, when checking for issuance holds, there is a missing @
to dereference the list of issuance IDs, so that the ahr table is search
with badly formed input, which results in returning an undefined list
instead of an empty list.  The error will occur for any record with a
serial.subscription entry but with no related entries in
serial.issuance.

We also recode the initialization of the response object in order to
clarify its expected structure, a hash of array references.

Signed-off-by: Steven Chan <schan@sitka.bclibraries.ca>
Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>

Always show Email Address label to staff

When placing holds staff commonly change out patrons, and pointing staff
towards the "My Account" area makes no sense to begin with.

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

Documentation typo in Authorities chapter subheading
Addresses this bugs reported in launchpad:
https://bugs.launchpad.net/evergreen/+bug/1248726

I also browsed through the rest of the docs/admin
directory, and made some (very non-consequential)
changes.

Signed-off-by: Elliot Voris <evoris@slcconline.edu>
Signed-off-by: Robert Soulliere <robert.soulliere@mohawkcollege.ca>

Pushing upgrade scripts forward

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

Forward port 2.3.11->2.3.12 DB upgrade

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

Pull in 2.5 version upgrade files

For 2.5.0 and 2.5.1

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

Fix a broken link in the holds docs

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

Docs: integrate holds docs from EG 2.1

The small amount of Holds docs present in master seemed to fit well at
the end of the older Holds documentation, so this commit appends them
to the new holds.txt file, deletes the previous holds_management.txt file,
and makes the appropriate change to root.txt.

This commit also includes several small formatting improvements, wording
changes, and bug fixes.

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

Fix boolean lists; Better atom regex; Caching

First, we didn't need to make $last_type local, and it broke explicit
grouping anyway.  That's removed, and we now reset that (and a few more
like it) at calls to the top level parse() method.  This introduces a
situation where a long list of booleans could cause query plan problems,
so we limit the plan depth to 40 (20 ||'d conditions).

Second, we are smarter about finding the boundary of atoms.  Previous
to this commit, and curly brace could send the parser into a tailspin
from which it would not recover.  Now we use alternation instead of
a character class, which is much safer with the default multi-character
float syntax specifier.

Third, as a catch-all, if we can't parse the remained of a query we
now simply say so (when in debug mode) and go away, instead of risking
an infinite loop.  We do this via a final, unqualified "else" clause
in decompose().

Finally, instead of building 10+ regexp objects on each query parse,
cache them per QP subclass and reuse them.

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

LP#1238015 hide patron credit summary when disabled

When the circ.disable_patron_credit / "Disable Patron Credit" org unit
setting is disabled, also hide the patron credit summary information in
the bills interface.

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

Re-apply the changes provided by  0802 for backport

Preserve record order of subfields for authority heading extraction

When extracting headings from authority records we currently read the
subfields of a tag in configuration order.  We should, instead, read
them in record order, to preserve the desired sorting properties that
the cataloger has encoded in the record.

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

Fix copy_table HTML markup errors

TD elements take a "headers" attribute, not a "header" attribute. Also,
we have a few typos in identifying the header attributes.

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

LP#1254146 gracefully handle custom trees sans entries

Avoid references to custom tree entries when no tree entries for a
custom tree exist.  Prevents:

egweb: template error: undef error - Can't call method "org_unit" on an
undefined value at
/usr/local/share/perl/5.10.1/OpenILS/WWW/EGCatLoader/Util.pm line 185.\n

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