Merge branch 'user/dbs/tpac_propagate_search_loc' into dbs/master_servant

Merge remote-tracking branch 'working/user/dyrcona/config.bib_source.can_have_copies' into dbs/master_servant

Merge remote-tracking branch 'working/user/dbs/lp896405_fix_2.2_upgrade-db' into dbs/master_servant

Conflicts:
Open-ILS/src/sql/Pg/002.schema.config.sql

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

Merge remote-tracking branch 'working/user/dbs/marc_export_fix_location' into dbs/master_servant

Merge branch 'user/dbs/avoid_2.1_upgrade_pain' into dbs/master_servant

Conflicts:
Open-ILS/src/sql/Pg/version-upgrade/2.0-2.1-upgrade-db.sql

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

Merge remote-tracking branch 'working/user/dbs/array_accum_2_array_agg' into dbs/master_servant

Merge branch 'master' of git.evergreen-ils.org:Evergreen into dbs/array_accum_2_array_agg

Conflicts:
Open-ILS/src/sql/Pg/990.schema.unapi.sql

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

Merge remote-tracking branch 'working/user/dbs/lp893315_filter_deleted_acns_unapi' into dbs/master_servant

Merge remote-tracking branch 'working/user/dbs/avoid_901c_death' into dbs/master_servant

Merge remote-tracking branch 'working/user/dbs/fix-nonfiling-titles' into dbs/master_servant

LP881774: Missing schema on function in 2.0-2.1 upgrade

Although the search path is altered to include the evergreen schema at
the start of the upgrade, one of the functions in the evergreen schema
does not resolve when invoked without the explicit schema later in the
upgrade script. We address that superficial (but failure provoking / time
wasting) problem by qualifying the function with the evergreen schema in
this commit, but we should also find out why the altered search path
does not seem to be functioning in the upgrade script.

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

LP896405: Address SQL upgrade issues getting to 2.2-alpha1

Martha Driscoll tried out the 2.2-alpha1 upgrade script and found a few
problems noted in this thread:
http://list.georgialibraries.org/pipermail/open-ils-dev/2011-November/007747.html

"upgrade_log" does not exist
> LINE 1: INSERT INTO config.upgrade_log (version, applied_to) VALUES .

This fails because the "applied_to" column does not exist at this point
in the schema upgrade. Just remove the "applied_to" column from the
INSERT statement.

This will probably fail because most sites won't have applied 0526 to
their production server, so move the statements outside of the
transaction.

Also, in my own testing I ran into:

public / evergreen function namespace issues we tried to address back in
the 2.0(?) timeframe. New sites will have the is_json() function in the
evergreen namespace, while old sites will have it in public, so we do
_not_ qualify the function name and just let the searchpath handle the
resolution.

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

Start collecting version-upgrade scripts in one spot

Rename the upgrade scripts missing "-upgrade-db" in their name

Add the 2.0.* upgrade scripts to version-upgrade

Discard the really old upgrade scripts that don't necessarily match what
was actually packaged and released.

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

Propagate search scope through advanced search interfaces

If a user had a chosen search scope (via the CGI "loc" parameter), it
would be lost when switching to the advanced search interfaces. Now we
progagate the search scope through the advanced search interfaces.

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

pin a version number for the patron card editor upgrade script

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

When editing cards, lock tab on apply changes

Hopefully to prevent people from moving away before saving.

Signed-off-by: Thomas Berezansky <tsbere@mvlc.org>
Signed-off-by: Jason Etheridge <jason@esilibrary.com>

Seed values/upgrade script for card edit perms

Not assigned to anyone by default.

Signed-off-by: Thomas Berezansky <tsbere@mvlc.org>
Signed-off-by: Jason Etheridge <jason@esilibrary.com>

Check Profile Editing Perms for Card Editor

Don't allow Active/Primary card changes unless you have the ability to edit
the patron's primary profile group.

Signed-off-by: Thomas Berezansky <tsbere@mvlc.org>
Signed-off-by: Jason Etheridge <jason@esilibrary.com>

Active/Primary Editing for Patron Cards

Changes "See All" into an editing interface, allowing for changing of the
active and primary state of each card.

Each requires a specific permission:

UPDATE_PATRON_ACTIVE_CARD
UPDATE_PATRON_PRIMARY_CARD

Without the appropriate permission the interface will show the green check
or red x instead of the editing elements. If you don't have either
permission then the Apply Changes button is hidden.

As the editing interface exists, also show the "See All" button when there
is only one card, but not when registering a new patron. This allows for
toggling the lone card's active flag, as well as replacing a card but
leaving the old one active.

To ensure that you can do the above, and to avoid some other errors, the
new card on a patron is saved to the editCard variable, rather than using
patron.card(). This both allows for "Replacing" a new non-primary card into
place as well as avoiding issues with editing the primary and active states
at the same time (the interface would try and update the new primary card
with the old primary card's barcode). Also, when checking for duplicate
barcodes, update the editCard with the barcode so the See All interface can
show the new card's barcode in the list properly.

Finally, included is localization for the All Cards interface headers and
buttons.

Signed-off-by: Thomas Berezansky <tsbere@mvlc.org>
Signed-off-by: Jason Etheridge <jason@esilibrary.com>

Tpac: ability to edit name, description of bookbags later

You could already name and describe when creating them, and now you can
edit that later.

Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>
Signed-off-by: Dan Scott <dscott@laurentian.ca>

Prevent null 852 $a in marc_export

852 $a is supposed to correspond to the institution holding the item
from which access is given, according to the MARC Format for Holdings
Data specification. As the org unit shortnames typically don't
correspond to one of the MARC Code List for Organization entries, we've
assumed that users will define the --location parameter; however, if
that parameter isn't defined, the empty string is used and an empty
subfield is generated. This, in turn, makes other MARC-parsing tools
unhappy. So, only generate an 852 $a if we have an actual value for the
--location parameter.

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

LP896405: Address SQL upgrade issues getting to 2.2-alpha1

Martha Driscoll tried out the 2.2-alpha1 upgrade script and found a few
problems noted in this thread:
http://list.georgialibraries.org/pipermail/open-ils-dev/2011-November/007747.html

"upgrade_log" does not exist
> LINE 1: INSERT INTO config.upgrade_log (version, applied_to) VALUES .

This fails because the "applied_to" column does not exist at this point
in the schema upgrade. Just remove the "applied_to" column from the
INSERT statement.

This will probably fail because most sites won't have applied 0526 to
their production server, so move the statements outside of the
transaction.

Also, in my own testing I ran into:

public / evergreen function namespace issues we tried to address back in
the 2.0(?) timeframe. New sites will have the is_json() function in the
evergreen namespace, while old sites will have it in public, so we do
_not_ qualify the function name and just let the searchpath handle the
resolution.

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

LP893315: Do not return delected call numbers in holdings_xml

Per LP893315, deleted call numbers were obscuring the actual active URIs
associated with a given bib record in a given scope at the default limit
of 5 results per record in unapi.holdings_xml.

Filtering out the deleted call numbers from the results seems like a
good idea, as we're generally interested in only showing the active
results (at least as in-db unapi is currently used). This also addresses
the problem of not showing active URIs when there are active URIs to
display.

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

TPAC: Protect against missing 901c in legacy MARCXML

Bad MARCXML to which Evergreen has been unable to add a 901c happens -
particularly with legacy MARCXML. Currently, TPAC crashes hard with an
ugly error if a 901c is not found when attempting to display the record
in search results. In comparison, the JSPAC displays the record without
a problem.

This commit logs a warning and enables the TPAC to continue somewhat
gracefully (the problem record is simply skipped in the search results).
A more robust solution may be to teach unapi.bre to add the record ID to
the results and to have the TPAC pull from that, rather than relying on
the MARCXML itself to contain the record ID.

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

Fix for LP894125 - reshelving interval

Update to reshelving code to utilize actor.org_unit_ancestor_setting function
and properly retrieve org settings for "circ.reshelving_complete.interval" for
org unit or ancestors.

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

Enable record summary to be scrollable at larger font sizes

The record summary at the top of the cataloguing interfaces appears to
determine its size based on the "normal" font size at load; then, when a
larger font size is applied to the page, the text gets hidden by the
fixed size of the groupbox.  Ideally the groupbox would adjust to
accomodate the larger font size, but after trying many different
approaches the best short-term approach seems to be to enable overflow
so that scrollbars can appear when necessary.

Signed-off-by: Dan Scott <dscott@laurentian.ca>
Signed-off-by: Jason Etheridge <jason@esilibrary.com>

Avoid updating deleted call numbers for 2.0 -> 2.1 upgrade

25% of the call numbers on our production system are deleted, so
filtering out the deleted call numbers from the update in the 2.0 -> 2.1
upgrade saves a significant amount of time.

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

Limit display to 856 with ind1 = 4 and ind2 = 0 or 1.

Minor modification to Open-ILS/src/templates/opac/parts/
misc_util.tt2 that limits the display of 856 urls to that have
indicator 1 equal to 4 and indicator 2 equal to 0 or 1. These should
be those that point directly to the resource.

Signed-off-by: Jason Stephenson <jstephenson@mvlc.org>
Signed-off-by: Dan Scott <dscott@laurentian.ca>

TPac: show non-asset.uri 856 URLs on detail page

Display "global" 856 URLs in record detail page.  A "global" URL in this
case is one that has no ownership and has not been extracted out as an
asset.uri.  As of right now, this is any 856 that has no $9, $w, or $n
subfield data.

Links are displayed directly after asset.uri links using the same style.

Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Dan Scott <dscott@laurentian.ca>

Avoid redundant install_cpan_force dependencies

The "install" target and each of the generic distro targets in
Makefile.install included install_cpan_force; make the "install" target
more useful by factoring out the install_libdbi dependency so that
"install" is truly common again (and we only add the install_cpan_force
dependency once).

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

Add install_yaz dependency to debian and ubuntu Makefile.install

To build SimpleServer 1.15 we need a version of yaz 4.2+, so add
install_yaz as a dependency for debian and ubuntu. This should have been
added in LP 872284 but was overlooked due to local testing and didn't
get any further testing before being committed.

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

bump xulrunner version

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

Switch to new Google Books API; make SSL friendly

As implied in the title, this commit does two things.  First, it
switches to the new Google Books API (which is both imminent and
also necessary to make SSL calls work).  Though the information is
scant, from what I have read and experienced, we do not need an
API key to do searches and previews.  I have also not hit any kind
of unauthenticated limit in several days of heavy testing, so I
would figure we are safe (at this point) for normal end-user OPAC
browsing.

Second, all Google Book requests are now done over https.  This
eliminates the majority of mixed content warnings when browsing
securely, though you still get a warning when you actually do
preview a book.

In addition to possibly implementing protocol detection (rather
than doing https all the time as a "lowest" common denominator),
there are a few minor points where we might consider future changes.
Those points are commented within the code.

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

Stamp upgrade scripts for both unapi update and vandelay-to-bucket

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

Default to org tree root for bre unapi funcs

If the org unit ("site") parameter to unapi.biblio_record_entry_feed()
or unapi.bre() is equal to "-" or NULL, default to the root of the org
unit tree.  In cases where the caller does not care to limit by site,
determing the shortname of the root org node is one less step the client
now has to take.

Original patch from Mike Rylander.

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

Vandelay; copy imported bib queue recs to bucket

Adds a "Copy to Bucket" option within the Vandelay queue interface.
User is given the option to name the bucket.  If the bucket already
exists, the records are added to the existing bucket.  Otherwise, a new
bucket is created and the records are added.

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

Vandelay; API for creating buckets from queues

* Adds a new biblio record entry bucket type of "vandelay_queue"
* Adds a new API call that allows callers to create new buckets (or add
to existing buckets), copying imported records from a vandelay queue
into the bucket.

open-ils.vandelay.bib_queue.to_bucket

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

Bumping version numbers, adding Upgrade Script and Changelog

Signed-off-by: Thomas Berezansky <tsbere@mvlc.org>

Repair errant explode_array in upgrade script.

The upgrade script 0562.schema.copy_active_date.sql included 2 calls to
explode_array after explode_array had been removed from the baseline
schema.  This upgrade just copies the contents of the
action.hold_request_permit_test function from the baseline schema to
repair the function for anyone following the upgrade scripts.

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

Stamping upgrade script for copy visibility vs peer-bibs fix

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

Distinct cache_copy_visibility copy/record

Because if you peer bib a copy to the bib it is on things don't like you.

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

Fix staff_client/server/cat/util.js.

Missed the = function(arg) when adding cat.util.get_cbs_from_bre_id.

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

Revert changes to opac.js from "previous" commit.

Turns out that opac.js has problems using files from the server via
JSAN.

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

Refactor retrieval of cbs object into cat.util.get_cbs_for_bre_id().

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

Teach OpenILS/Application/Storage/CDBI/config.pm about new field.

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

Minor adjustment in source_init method of copy_browser.js.

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

Add can have copies check to the Add MFHD Record link in rdetail.

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

Can have copies checks code improvement.

Add can have copies check to create_mfhd() in opac.js.

Rearrange the code a bit to be more concise and to eliminate the
can_have_copies variable.

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

Whitespace cleanup in copy_broswer.js.

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

Refactor can_have_copies check in copy_broswer.js.

Since we want to check in two places, move the can_have_copies flag
and source name to be variable members of the this object.

Initialize the members in the source_init function.

Check the members in the add_volumes and add_items commands.

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

Fix copy/paste-o in copy_browser.js.

Swap obj. for g. that was copied from opac.js.

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

Modify copy browser to check for bib_source.can_have_copies.

Add code to server/cat/copy_browser.js.
Add error string to server/local/en-US/cat.properties.

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

Fix thinko in opac.js and bib_brief.js.

Forgot to change the variable name from cbs to cbsObj when reporting that
the source cannot have copies.

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

Modify server/cat/bib_brief.js to check for source can have volumes.

Copy/paste code from chrome/content/cat/opac.js.

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

Add fmall.js to opac.xul so objects are mapped in opac.js.

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

Fix VIA/BY typo.

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

Check for bib_source.can_have_copies in cataloging.

Retrieve the BRE and CBS object in the add_volumes method of
opac.js.

Check if the CBS can have copies, and if not, tell the user such
and return.

Also required adding an api entry in constants.js and a new message
in offline.properties.

This is the beginning of the client code changes. There are very
likely to be more.

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

Add upgrade script for config.bib_source.can_have_copies.

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

Add can_have_copies field to config.bib_source.

Add the column definition to config.bib_source in
002.schema.config.sql.

Insert TRUE into the column for the default sources created in
950.data.seed-values.sql.

Add the field definition to class id cbs in fm_IDL.xml.

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

Obey renewal rules for SIP2 Checkout

If local policy OR the selfcheck are set to not do renewals than don't
attempt them.

Also, return whether a renewal was attempted (success or failure) so that
the self check on the other end has a better clue what went on.

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

Improvement for telephony: just-in-time event revalidation

One of the shortcomings with using the Action/Trigger based telephony in
Evergreen until now was that while you might have overdue notices
generated and sent to a system where Asterisk runs for later calling,
but if the notice was generated on a Saturday night, and you have Asterisk
set up not to place any calls again until Monday morning, Asterisk has
no way of revalidating that call at the last minute.  That is, the
system could not determine whether the items that were overdue on
Saturday night are still overdue on Monday morning, and whether the call
should still be made.

Now we have a workable solution to that.

The eg-pbx-allocator.pl script, which takes call files for Asterisk from
a "staging" directory and slowly drips them onto Asterisk's spool can
now consult an open-ils.justintime which in turn asks open-ils.trigger
whether given events, enumerated within the call files themselves, are
still valid.

open-ils.trigger is designed to run as a private service, so that's why
we need a public service that doesn't do anything too sensitive.

This open-ils.justintime service can potentially be extended to offer other
just-in-time information to the allocator right before a call goes onto
Asterisk's spool. For example, that might be a good time to check the time
of day and make a late decision on which phone number to use for a given
user (day_phone, evening_phone, other_phone).

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

Stamped upgrade for full circ count view fix

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

Use DISTINCT in counts to avoid dupes

Otherwise you get:
Legacy count + 2(normal count)(aged count)

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

Full Circ Count View Amplifies Legacy Circs

The current extend_reported.full_circ_count sums the legacy circ count
column, but this results in amplifying the count by the number of current
circs in the system.  This commit adds the legacy count to the GROUP BY
instead of summing it.

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

Skip shelved holds at checkout auto-fill

Already happened for volume/title hold, but target map checking wasn't
checking to see if the current copy was shelved.

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

Net::Z3950::SimpleServer 1.15 should not require sedding

Checking both versions of the packages listed on ftp.indexdata.dk, the
hardcoded path to yaz-config does not exist in Makefile.PL - thus, we
should be able to drop the "sed" instruction for 1.15.

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

Install yaz-4.2 and Net::Z3950::Simpleserver 1.15

Net::Z3950::Simpleserver 1.14+ requires yaz 4.2.x, as well as currently
suffering from a Makefile.PL problem that involves a hard-coded path to
yaz-config (https://rt.cpan.org/Public/Bug/Display.html?id=70444).
Provide a sed command to resolve the latter problem and upgrade our
"current version of yaz" to yaz 4.2.x.

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

Fix a problem in the clear holds shelf list printer

This is a "special" interface that you get to via Admin -> For
Developers -> Local Administration in the staff client.

The addition features to one "side" of this hackish interface of label affix
and parts had caused breakage on the other "side."  This fixes that.

Also improve document printing, making it less likely the progress
dialog itself gets printed.

One day this will be replaced with something better.

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

Stamped upgrade for asset.merge_record_assets fix

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

LP#849143 fix some gaps in bib record merging

Tweaks asset.merge_record_assets such that it considers acquisition line items and user requests, serial subscriptions, booking resource types, biblio monograph parts, and biblio peer bib copy maps. I'm not handling vandelay.queued_record.imported_as or any other vandelay tables, but we may wish to do that too. We're also leaving acq.acq_lineitem_history alone.

Signed-off-by: Jason Etheridge <jason@esilibrary.com>
Signed-off-by: Mike Rylander <mrylander@gmail.com>

Aligned the MARC editor subfields to 'start' fixes long subfield display

When viewing a record in the MARC editor that has extremely long
entries in a subfield, the output would align in the center of the
subfield making the output hard to read.  This patch adds an
align: 'start' option to both the subfield, and the subfield code
elements in the marcSubfield function to make the subfields align in a
 more readable manner.
Signed-off-by: Liam Whalen <lwhalen@evergreen-dev.catalogue.nrcan.gc.ca>
Signed-off-by: Mike Rylander <mrylander@gmail.com>

Merge remote-tracking branch 'eg-working/collab/phasefx/merged_bill_and_receipt_fixes'

Acq: (un)receive w/o assets should not attempt a copy update

When receiveing a lineitem for a PO that was activated without assets,
avoid any attempts to update said non-existent assets.

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

LP#848945 - Hide Empty Libs checkbox

The "Show Libraries With Items" button in Holdings Maintenance shows all the
libs with items regardless of the library/depth filters. One problem is that
when the list refreshes (whenever an item/volume is edited, deleted, or added),
staff using this feature have to re-press the button, which can take a while for
some bibs. This branch removes this button and replaces it with a "sticky"
checkbox labelled "Hide Empty Libs".

This allows you to do almost the same thing; you can set the context library to
Consortium and the depth filter to the lowest depth, which would normally show
all libs, but then use the checkbox to prune out the ones that have no children
and no volumes(*). One possible use case that we'd lose is the ability to show a
specific set of libs that don't already have volumes in addition to all the libs
that do.

* - Also, the test for whether to hide an org or not is very simplistic, but
easy to implement. A more aggressive test would hide a library and all its
descendants if none of the descendants have volumes.

Signed-off-by: Jason Etheridge <jason@esilibrary.com>
Signed-off-by: Mike Rylander <mrylander@gmail.com>

Patch from Jeff Davis (SITKA) fixing cardless patron updates

Quoth the bug report (https://bugs.launchpad.net/evergreen/+bug/851110):

  On our Evergreen 2.0.8 system, it sometimes arises that a patron's card gets deleted. If it was the patron's only card, it becomes impossible to add a new card to the account via the staff client.

  How to reproduce:
  1. Retrieve a patron record for a patron who has no associated card.
  2. Click the Edit button.
  3. Click Replace Barcode and enter a new barcode.
  4. Click Save. This does not work - the process hangs, and if you hit Reload, you will find that your changes were not saved.

  I believe this happens because some code in register.js assumes the patron has at least one card. I have a potential fix that this comment box is too small to contain ... uh, that is to say, I will post it momentarily.

Which he did, and I applied.

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

Throw out non-digit barcodes for strict check

Characters like "." are valid for numbers, but not the algorithm.

Thus, we need to throw out "16.00" despite the check for
"16.00" == Number("16.00")
returning true.

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

Don't check barcode without strict enabled

Item status was, unlike other interfaces, doing the strict check, THEN
looking at the checkbox and if it was checked return.

This caused issues when the barcode crashes out the strict checking.

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

Fix JSPac Hold Editor Org Selector

Don't wipe out all changes just because the OU was changed.

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

Stamped upgrade script for hold_notification update via A/T cleanup module

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

Hold notification creation A/T cleanup module

New CreateHoldNotification Action/Trigger cleanup module that, when
linked to a hold-based event_definition, adds an entry into
action.hold_notification for each hold processed.  The main user-facing
change from this will be that the hold notification time / count in the
staff client will now take A/T notices into effect in addition to
manually created hold notifications.

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

Further support for "circ.collections.exempt" setting

Added support for skipping users that have the
"circ.collections.exempt" setting in the API call:

open-ils.collections.users_of_interest.warning_penalty.retrieve

This call is an alternate entry point for finding users to consider
for collections processing

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

Reshelving update.

Update to the reshelving code to make it faster and utilize the new status change time field on asset.copy.

Signed-off-by: Steven Callender <stevecallender@esilibrary.com>
Signed-off-by: Mike Rylander <mrylander@gmail.com>

TPac: Fix glitch in displaying received holdings

Under "issues held" on the record detail page, holdings summaries for
received issues are nested in an additional layer of array as compared
to MFHD-based holdings, so compensate.

Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>
Signed-off-by: Bill Erickson <berick@esilibrary.com>

Clear shelf canceled holds notification fix

After cancelling a batch of holds via the clear-shelf process, before we
send the holds off to action/trigger for potential notification
generation, re-fetch the holds from the DB to pick up the calculated
cancel_time value (i.e. get a real value for "now").  Otherwise, A/T may
die attempting to create a DateTime object by parsing "now".

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

TPac; hold success confirmation; redirect repairs

1. After a successful hold placement, provide feedback to the user that
the hold placement succeeded.  After placement, the user is taken to a
page (same page for failed holds) where the title(s) are listed along
with the success/failure information for each.

2. In some cases, the user may be redirected to the my account page
instead of the point of origin for holds placement.  This occurs
generally when the user is not already logged in and is asked to log in
prior to holds placement.

3. As a side effect, this change replaces one of the ornery
history.go(-1) actions with a true href/URL.

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

TPac; reduce vertical space caused by chilifresh link

CSS loaded from chilifresh forces a fixed height on the
.chili_review class, which causes significant vertical space gobbling in
the results and details pages.  Solution is to change the chili_review
container from a <div> to a <span>, where the height has no effect.

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

Make idlval build, rather than be dead code

The tool "idlval" contained in Open-ILS/src/c-apps can be used to check
the correctness of fieldmapper definitions in files like
Open-ILS/examples/fm_IDL.xml - but currently does not have a build
target, nor does it compile cleanly due to the use of unspecified
OpenSRF header files. This commit adds a build target, fixes the compile
problems, and installs the tool in the bin directory.

Signed-off-by: Dan Scott <dscott@laurentian.ca>
Signed-off-by: Jason Stephenson <jstephenson@mvlc.org>
Signed-off-by: Mike Rylander <mrylander@gmail.com>

Add link to cbs from bre via bre.source -> cbs.id in fm_IDL.xml.

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

Show the patron account Create Date in both horizontal and vertical patron summary panes

Signed-off-by: Jason Etheridge <jason@esilibrary.com>
Signed-off-by: Mike Rylander <mrylander@gmail.com>

LP887822 mint_condition/Quality not set correctly

during item creation.

Specifically, the Javascript object for a new item (acp) does not explicitly set
the mint_condition field, leaving it null. The item attribute editor treats the
null as false (which it displays as "Mediocre" for that field). The database has
a not null constraint the corresponding column in asset.copy, and defaults to
true. So a new item in the attribute editor will show "Mediocre", but when
actually created, will become "Good".

This branch just sets mint_condition to true when an item is created.

Signed-off-by: Jason Etheridge <jason@esilibrary.com>
Signed-off-by: Mike Rylander <mrylander@gmail.com>

Tpac; repair login link after password reset

After a successful password reset, the UI offers a link to "Log In to My
Account".  Loging in this way would ultimately redirect the user back to
the password reset form.  This patch updates the Log In link to point to
myopac/main, which will direct the user to log in first,  instead of
pointing directly to /login, which, lacking any other information,
will return the user to the referer after logging in.

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

Merge remote-tracking branch 'eg-working/collab/berick/lp_888239_prevent_post-capture_hold_suspend'

TPac; propagate query params to adv search (results link)

Update the "Advanced Search" link that shows on search results pages
(next to "Another Search") to propagate the current query into the
advanced search page, just like the Advanced Search link just above the
search bar.

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

tpac; honor password reset matching email setting

If the "circ.password_reset_request_requires_matching_email" org unit
setting is enable, prompt the user to enter the email address associated
with the account and subsequently pass it on to the password_reset api
call.

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

Serials: in the holding code mini-wizard, offer a dropdown for seasons

This spares the user from having to remember the MFHD code for the
season, when dealing with periodicals issued by season (21 - 24 for
Spring - Winter, respectively).

This also applies sets the "scrollOnFocus" attribute to false for the
dijits within the mini wizard, as the jumpy behavior you get in
less-than-fullscreen staff client windows is otherwise maddening.

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

tpac; improve ssl and authentication redirection

Create a better separation between pages the require SSL and those
that also require authentication.  This change has a number of
beneficial side effects:

1. If a user is already logged in and is somehow redirected to the
non-SSL version of a page requiring SSL (via, for example, some external
link), they are simply taken to the SSL equivlent of the page first.  If
they are found to be logged in, we're done.  If not, they are directed to
the login page like usual.

2. password_reset page now requires SSL (but not auth, obviously)

3. Removed the unnecessary warning about logging out on a non-SSL
connection.  We now redirect the user to the SSL version of logout so,
if they are in fact logged in, they can be successfully logged out.

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

Reindex is not just for non-filing indicators anymore

Since we discovered the apostrophe parsing problem affects records with
text containing apostrophes at any position, don't limit ourselves to
non-filing indicator titles. In theory you might want to reindex any
record that contains an apostrophe, but in practice we found the most
confusion occurs with searching for titles - so target those in the
upgrade script.

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

Add the search_normalize() variant to in-database

And teach our normalizers to use search_normalize() instead of
naco_normalize() across the board.

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

Create a search_normalize() variant of naco_normalize()

In the quest to support searching text that contains leading articles
joined by an apostrophe - for example, "l'histoire" - such that a
searcher can enter either "l'histoire" or "histoire" and get results -
add a variant of naco_normalize() that does not strip the apostrophe
entirely, but rather replaces it with a space such that the root word
can be indexed appropriately.

This implementation refactors the OpenILS::Utils::Normalize code to make
the differences between search_normalize() and naco_normalize() as clear
as possible, but duplicates code significantly in the in-db version of
the code. Someday maybe the database can rely on
OpenILS::Utils::Normalize instead of inline functions :)

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

LP825039: Add reingest logic for non-filing titles

Rather than reindexing all titles that have a non-filing indicator, this
upgrade handles the known problematic cases (apostrophes with leading
articles). On our system that's 55K out of 2.2M records - not bad at
all.

We also ensure the format is set to MODS32 for title:proper, as sites
might have customized it on a MARC basis and we don't want to reindex
the records with NULL values due to a mix of format and XPath.

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