working/Evergreen.git
5 years agoLP#1339118: Ignore permission checks for "jump" context
Mike Rylander [Tue, 8 Jul 2014 15:11:54 +0000 (11:11 -0400)]
LP#1339118: Ignore permission checks for "jump" context

We don't worry about checking permissions on foreign objects when we're
simply using them to get a context OU for the local object permission
testing.  The "jump" field just gives us more layers of indirection, and
shouldn't cause a perm check, just like normal foreign context OUs.

Signed-off-by: Mike Rylander <mrylander@gmail.com>
Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Ben Shum <bshum@biblio.org>
5 years agoLP#1334693 make_release optional -j <osrf_js_path>
Bill Erickson [Mon, 21 Jul 2014 13:39:41 +0000 (09:39 -0400)]
LP#1334693 make_release optional -j <osrf_js_path>

make_release supports a "-j" option for passing in the OpenSRF JS path:

make_release [options] -j /openils/lib/javascript

This is useful when running make_release, specifically the staff client
building portion, on a machine where the OpenSRF libs are available
(say, from a git checkout) but osrf_config is not available because
OpenSRF is not actually installed.

The script will attempt to find osrf_config and the JS path
automatically, so under normal circumstances, -j is not required.

Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Ben Shum <bshum@biblio.org>
5 years agoLP#827442 Z39.50 will split the total cap between locations when multiple locations...
Dan Pearl [Wed, 20 Nov 2013 17:17:51 +0000 (12:17 -0500)]
LP#827442 Z39.50 will split the total cap between locations when multiple locations selected

This code does a bit of oversampling, and buffers the results so that a
full batch of results are returned.

In addition, the values in the hover text under "Displaying xxx of yyy records"
has been repaired, as it was easily confused.

Signed-off-by: Dan Pearl <dpearl@cwmars.org>
Signed-off-by: Jason Stephenson <jstephenson@mvlc.org>
5 years agoLP#1341703 Thinko in Batch Edit (hidden by older OpenSRFs)
Mike Rylander [Mon, 14 Jul 2014 18:57:38 +0000 (14:57 -0400)]
LP#1341703 Thinko in Batch Edit (hidden by older OpenSRFs)

There's a bug on line 272 of O::A::Cat.pm where we called a non-existent
method named respond_complete on the invocant object. Instead, we need to
call that on the client object (spelled $conn in this method). Batch bib
edit won't work on new OpenSRF's until this is fixed.

Signed-off-by: Mike Rylander <mrylander@gmail.com>
Signed-off-by: Bill Erickson <berick@esilibrary.com>
5 years agoLP#1341013: Load apache mod CGI
Ben Shum [Mon, 14 Jul 2014 15:44:07 +0000 (11:44 -0400)]
LP#1341013: Load apache mod CGI

As of Apache 2.4, mod CGI needs to be enabled directly and is not built
with apache.  So let's add it to the Makefile for Ubuntu Trusty 14.04.

Also, add handling for disabling unwanted apache conf like serve-cgi-bin,
which can break offline cgi-bin script alias.

Signed-off-by: Ben Shum <bshum@biblio.org>
Signed-off-by: Jason Stephenson <jstephenson@mvlc.org>
5 years agoLP#1341013: Remove NameVirtualHost from eg.conf.in for Apache 2.4.
Jason Stephenson [Fri, 11 Jul 2014 14:57:59 +0000 (10:57 -0400)]
LP#1341013: Remove NameVirtualHost from eg.conf.in for Apache 2.4.

NameVirtualHost does nothing in Apache 2.4 and will be removed in
the next release.  We might as well get rid of it now and silence
a warning when starting Apache.

Signed-off-by: Jason Stephenson <jason@sigio.com>
Signed-off-by: Ben Shum <bshum@biblio.org>
5 years agoForward-port 2.6.2 upgrade script
Dan Wells [Mon, 14 Jul 2014 21:21:48 +0000 (17:21 -0400)]
Forward-port 2.6.2 upgrade script

Signed-off-by: Dan Wells <dbw2@calvin.edu>
5 years agoForward-port 2.5.6 upgrade script
Dan Wells [Mon, 14 Jul 2014 21:18:20 +0000 (17:18 -0400)]
Forward-port 2.5.6 upgrade script

Signed-off-by: Dan Wells <dbw2@calvin.edu>
5 years agoFix 950.data.seed-values.sql.
Jason Stephenson [Mon, 14 Jul 2014 20:54:24 +0000 (16:54 -0400)]
Fix 950.data.seed-values.sql.

Code from the lost and paid copy status was missing an argument
on the insert.

Signed-off-by: Jason Stephenson <jstephenson@mvlc.org>
5 years agoRelease notes entry for lost and paid status
Kathy Lussier [Fri, 6 Jun 2014 02:19:32 +0000 (22:19 -0400)]
Release notes entry for lost and paid status

Signed-off-by: Kathy Lussier <klussier@masslnc.org>
Signed-off-by: Jason Stephenson <jason@sigio.com>
5 years agoLP#1198475: Wrapping upgrade script for lost and paid copy status.
Jason Stephenson [Mon, 14 Jul 2014 12:37:30 +0000 (08:37 -0400)]
LP#1198475: Wrapping upgrade script for lost and paid copy status.

Signed-off-by: Jason Stephenson <jason@sigio.com>
5 years agoLP1198475: Add a new, optional, Lost and Paid copy status.
Jason Stephenson [Fri, 6 Jun 2014 00:49:41 +0000 (20:49 -0400)]
LP1198475: Add a new, optional, Lost and Paid copy status.

Add the Lost and Paid status to config.copy_status.

Add org_unit_settig to control if the copy status is used.

Add checks to O::A::Circ::Money::make_payments to check for LOST
status and org_unit setting and change the copy status to
Lost and Paid as appropriate.

Also add checks for Lost and Paid copy status wherever LOST copy
status is used.

Add the COPY_STATUS_LOST_AND_PAID event.

Add the event to ils_events.xml with id 7026.

Check for the event in circ/util.js in the staff client.

Add pgTAP tests for the existence of the Lost and Paid copy status
and the new org_unit setting to control if it isused.

Signed-off-by: Jason Stephenson <jason@sigio.com>
Signed-off-by: Kathy Lussier <klussier@masslnc.org>
Signed-off-by: Rogan Hamby <rogan.hamby@gmail.com>
5 years agoLP#1234845: Stamping upgrade script for improved evergreen.ranked_volumes()
Ben Shum [Thu, 10 Jul 2014 20:37:00 +0000 (16:37 -0400)]
LP#1234845: Stamping upgrade script for improved evergreen.ranked_volumes()

Signed-off-by: Ben Shum <bshum@biblio.org>
5 years agoLP#1234845: Performance improvement to evergreen.ranked_volumes() database function.
hubert depesz lubaczewski [Thu, 3 Oct 2013 18:26:33 +0000 (14:26 -0400)]
LP#1234845: Performance improvement to evergreen.ranked_volumes() database function.

For the client I analyzed logs from production Pg instance for ~ 10
days.

During this time, the single most time consuming query (summarized time
for all instances of the query, with different parameters) was:

SELECT * FROM unapi.bre ( ... ) AS "unapi.bre";

I profiled this function, and found that in my test case most of the
time (2.04s out of 2.06s, so ~ 99%) was spent in call to
unapi.holdings_xml() function.

When I profiled this function, I found that most of the time (sorry,
don't have the number now with me) was spent in call to
evergreen.ranked_volumes() function.

At this moment in my research something changed on the server I was
testing on, and all subsequent times were ~ 4-5 times lower, but the
ratios were more or less the same.

Anyway - call to evergreen.ranked_volumes() showed repeatable time (with
full caches/buffers) of ~ 380ms.

I modified the function by:
1. inlining actor.org_unit_descendants(?, ?)
2. inlining evergreen.rank_ou(?, ?, ?)
3. extracting depth calculation to separate call
4. switched to plpgsql (which gives me ability to use variables)
5. removed evergreen.rank_ou() and evergreen.rank_cp_status() from
   select clause - these are still in WINDOW definition, but they
   weren't used in the SELECT, so it's better to remove from there.
6. in passing renamed arguments to avoid name clash (argument depth vs.
   field depth)
7. in passing changed usage of $* to access parameters to using named
   parameters, for readability.

New function did the same work in ~ 18ms.

EDIT: Convert to SQL, keeping all of the improvements from depesz
EDIT2: Added Signed-off-by line for depesz, see http://markmail.org/message/rv4vaarwixeswqgu

Signed-off-by: Hubert depesz Lubaczewski <depesz@depesz.com>
Signed-off-by: Jason Stephenson <jstephenson@mvlc.org>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
Signed-off-by: Kathy Lussier <klussier@masslnc.org>
Signed-off-by: Ben Shum <bshum@biblio.org>
5 years agoForce the installation of Business::Stripe on Ubuntu Trusty.
Jason Stephenson [Thu, 10 Jul 2014 20:10:36 +0000 (16:10 -0400)]
Force the installation of Business::Stripe on Ubuntu Trusty.

Apparently the required force was missed in the Makefile.

Signed-off-by: Jason Stephenson <jstephenson@mvlc.org>
5 years agoLP#1254918 limit ACQ batch update fund retrieval by perms
Bill Erickson [Fri, 9 May 2014 18:32:14 +0000 (14:32 -0400)]
LP#1254918 limit ACQ batch update fund retrieval by perms

In the ACQ batch update bar along the top of the PO page, limit the
number of funds which require permission checks on the server by
limiting the fund search to those at org units where the requesting user
has the CREATE_PURCHSE_ORDER or MANAGE_FUND permissions.

In other words, if a user only has create-po perms at Branch 1, we can
specifically request funds at that branch (or below) for display so that
the server (PCRUD) does not have to inspect unrelated funds for
permissibility.  This speeds up fund retrieval.

Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Ben Shum <bshum@biblio.org>
5 years agoLP#1254918: Allow skiping of user-object perms
Mike Rylander [Fri, 2 May 2014 21:00:43 +0000 (17:00 -0400)]
LP#1254918: Allow skiping of user-object perms

Previous to this commit, permissions in Evergreen check a cominbation
of:
 * user-object permissions (does the user have a direct permission
   mapping to the object in question)
 * user-context permissions (does the user have the permission at the
   object's context location, whose field is defined in the IDL)
 * user-global permission (lacking a context location, does the user
   have the permission globally (at the top of the org tree) and therefore
   can apply the action to all objects of this typ)

In practice, there are almost no user-object permissions.  When retrieving
just on object from the database, the cost of this check is negligable to
the point that we can completely ignore it.  However, when retrieving a
large set of objects, such as the list of all funds in a large, consortial
environment, the cost to check the user-object permission adds up to a
noticable amount of time.

To address this, we add a new construct to the IDL instructing the PCRUD
infrastructure to skip user-object permission checking in those cases where
the design and use of the system makes user-specific object permissions
needless or superfluous.  This is embodied in a new XML attribute on the
<pcrud> element: ignore_object_perms.  When set to "true", pcrud will skip
all user-object permission checks, resulting in faster time-to-first-result.

Additionally, we add a new "owning_user" attribute on the <action> element
of the <pcrud> section. This new attribute specifies the field containing
the actor.usr.id of the user that "owns" the object.  This allows PCRUD to
test ownership of an object directly, and if the requesting user and owning
user are the same, the action is allowed.

Finaly, when "global_required" is "true" for the permission check, and there
is no "owning_user" attribute defined for the class in the IDL, we skip the
above-mentioned user-object permission check.  When "global_required" is
"false" or there is an "owning_user" attribute, we check for user permissions.

In all cases, the "ignore_object_perms" attribute is honored, and in its
presence we skip non-owner user-object permissions.

The net result is an immediate increase in speed for retrieval of objects
in the presence of the "global_required" attribute, and a mechanism to
increase the speed of specific cases of context-aware retrival by the use
of "ignore_object_perms".

We use this new mechanism to speed the retrieval of fund objects in the
ACQ interfaces that draw available-fund dropdowns.

Signed-off-by: Mike Rylander <mrylander@gmail.com>
Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Ben Shum <bshum@biblio.org>
5 years agoLP#1334693 ./configure avoid osrf_config without core
Bill Erickson [Thu, 26 Jun 2014 14:35:38 +0000 (10:35 -0400)]
LP#1334693 ./configure avoid osrf_config without core

Avoid the check for and use of osrf_config during Evergreen ./configure
when run with --disable-core.  This allows the staff client to be built
and, in general, for make_release to be run on a machine that does not
have opensrf installed (or has multiple versions).

Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Ben Shum <bshum@biblio.org>
5 years agoLP# 1337550 remove schema.org itemOffered, add price property
Dan Scott [Thu, 3 Jul 2014 20:20:08 +0000 (16:20 -0400)]
LP# 1337550 remove schema.org itemOffered, add price property

We use the "offers" property to link the Product to the Offer, so there
is no reason to use a reciprocal "itemOffered" property to link back to
the Product from the Offer. The circular reference confuses some tools
(http://rdfa.info/play) and may complicate general processing, so let's
simply remove it.

Signed-off-by: Dan Scott <dscott@laurentian.ca>
Signed-off-by: Ben Shum <bshum@biblio.org>
5 years agoLP# 1337550 Remove AggregateOffer schema.org instance
Dan Scott [Thu, 3 Jul 2014 19:53:25 +0000 (15:53 -0400)]
LP# 1337550 Remove AggregateOffer schema.org instance

As our record details page lists each copy as a separate Offer (well, in
many cases anyway, unless paging gets invoked), the AggregateOffer
entity is not really necessary--and using it on the same page as
separate Offer entities is not recommended by at least one search engine
(per Google https://support.google.com/webmasters/answer/146750?hl=en
says a given page should include "_one_ of the following: Offer or
Offer-aggregate").

Signed-off-by: Dan Scott <dscott@laurentian.ca>
Signed-off-by: Ben Shum <bshum@biblio.org>
5 years agoLP# 1330784: Release notes for sitemap builder
Dan Scott [Mon, 7 Jul 2014 13:58:01 +0000 (09:58 -0400)]
LP# 1330784: Release notes for sitemap builder

More like documentation than release notes, but more is probably better
than less.

Signed-off-by: Dan Scott <dscott@laurentian.ca>
Signed-off-by: Ben Shum <bshum@biblio.org>
5 years agoLP#1330784 Add a sitemap generator for Evergreen
Dan Scott [Thu, 19 Jun 2014 19:52:42 +0000 (15:52 -0400)]
LP#1330784 Add a sitemap generator for Evergreen

Following the requirements at sitemaps.org, generate a
set of sitemaps that reflect the bib record's last edit
date, with 50,000 records per sitemap file.

Users can run this script targeting different libraries
and generating different output filenames using the
documented options in the script.

Signed-off-by: Dan Scott <dscott@laurentian.ca>
Signed-off-by: Ben Shum <bshum@biblio.org>
5 years agoLP#1308239 support holds fulfillment on precat copies
Bill Erickson [Tue, 15 Apr 2014 20:30:19 +0000 (16:30 -0400)]
LP#1308239 support holds fulfillment on precat copies

During checkout, allow holds on precat copies to get fulfilled.

Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Ben Shum <bshum@biblio.org>
5 years agoLP#1308239 Support pre-cat copy hold targeting
Bill Erickson [Tue, 25 Jun 2013 13:32:48 +0000 (09:32 -0400)]
LP#1308239 Support pre-cat copy hold targeting

Remove an unused line of code in the hold targeter which was preventing
targeting of pre-cat copies.

In OpenILS::Utils::PermitHold::permit_copy_hold, neither in-db nor
script based hold viability checks inspect the value of the
"title_descriptor" field in the provided params.  Remove this field and
pre-cats just work.

Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Ben Shum <bshum@biblio.org>
5 years agoRelease notes entry for holds shelf expire change
Kathy Lussier [Fri, 30 May 2014 19:45:18 +0000 (15:45 -0400)]
Release notes entry for holds shelf expire change

Signed-off-by: Kathy Lussier <klussier@masslnc.org>
Signed-off-by: Ben Shum <bshum@biblio.org>
5 years agoLP#1306753: Only look at holds that expire before 'today'
Mike Rylander [Fri, 11 Apr 2014 18:09:35 +0000 (14:09 -0400)]
LP#1306753: Only look at holds that expire before 'today'

We should not count today when generating the hold expiration list.
So, we'll look for holds that expire before 'today' instead of those
that expire before 'now'.

Signed-off-by: Mike Rylander <mrylander@gmail.com>
Signed-off-by: Kathy Lussier <klussier@masslnc.org>
Signed-off-by: Ben Shum <bshum@biblio.org>
5 years agoLP#1316814: Change potentially incorrect information display in hold_status.tt2
Victoria Lewis [Wed, 7 May 2014 20:10:19 +0000 (13:10 -0700)]
LP#1316814: Change potentially incorrect information display in hold_status.tt2

Removed word "circulating" from patron hold status message regarding the number of copies of the material.
Some copies may be ON ORDER and therefore not circulating.

Signed-off-by: Victoria Lewis <vcamklewis@gmail.com>
Signed-off-by: Ben Shum <bshum@biblio.org>
5 years agoDocumentation: added release notes for LP#1327284 Display "Imported As" column
Yamil Suarez [Tue, 8 Jul 2014 18:21:58 +0000 (14:21 -0400)]
Documentation: added release notes for LP#1327284 Display "Imported As" column

LP#1327284 Display "Imported As" in Vandelay queue

Signed-off-by: Yamil Suarez <yamil@yamil.com>
Signed-off-by: Ben Shum <bshum@biblio.org>
5 years agoLP#1327284 Display Imported As column in vandelay queue
Bill Erickson [Fri, 6 Jun 2014 16:35:59 +0000 (12:35 -0400)]
LP#1327284 Display Imported As column in vandelay queue

For Bib and Authority records which have been imported / merged /
overlayed, etc. into the system via Vandelay, show the record ID of the
catalog bib/auth record to which the record was imported.

Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Yamil Suarez <yamil@yamil.com>
Signed-off-by: Ben Shum <bshum@biblio.org>
5 years agoLP#1210161: Stamping upgrade script to rename bookbag to list
Ben Shum [Thu, 10 Jul 2014 18:51:57 +0000 (14:51 -0400)]
LP#1210161: Stamping upgrade script to rename bookbag to list

Signed-off-by: Ben Shum <bshum@biblio.org>
5 years agoLP#1210161: Upgrade script to rename bookbag to list
Remington Steed [Wed, 12 Feb 2014 15:34:51 +0000 (10:34 -0500)]
LP#1210161: Upgrade script to rename bookbag to list

Signed-off-by: Remington Steed <rjs7@calvin.edu>
Signed-off-by: Ben Shum <bshum@biblio.org>
5 years agoLP#1210161: Finish renaming bookbag to list
Remington Steed [Wed, 27 Nov 2013 20:39:11 +0000 (15:39 -0500)]
LP#1210161: Finish renaming bookbag to list

The JSPAC bookbag feature was renamed to "Lists" in TPAC, but several
references still exist to "bookbag", "book bag", "Book Bag", etc. in the
public and staff interfaces. This commit attempts to provide simple and
clear replacements for the old term. In some cases "list" seems best,
but where the context is uncertain "book list" is used to err on the
side of clarity. Some of these changes may be in old code that is no
longer used.

This commit also includes two minor wording changes - one to fix grammar,
and the second to update instructions for sharing a list.

Signed-off-by: Remington Steed <rjs7@calvin.edu>
Signed-off-by: Ben Shum <bshum@biblio.org>
5 years agoLP#1315531 Remove remaining references to Lucid from README and Makefile
Ben Shum [Thu, 10 Jul 2014 18:25:18 +0000 (14:25 -0400)]
LP#1315531 Remove remaining references to Lucid from README and Makefile

There were still a few references to Ubuntu Lucid in README and makefile
comments area.  Also, the Makefile itself for Lucid was still there.

Remove these last traces forever!

Signed-off-by: Ben Shum <bshum@biblio.org>
5 years agoLP#1315531 Fix README asciidoc syntax
Ben Shum [Thu, 10 Jul 2014 17:51:50 +0000 (13:51 -0400)]
LP#1315531 Fix README asciidoc syntax

Noticed when generating the asciidoc output to HTML there was an error in the
README file. This adds the proper syntax around the text blocks.

Signed-off-by: Ben Shum <bshum@biblio.org>
5 years agoLP#1315531 Quick clarification about which Ubuntu we mean.
Chris Sharp [Wed, 18 Jun 2014 20:10:18 +0000 (16:10 -0400)]
LP#1315531 Quick clarification about which Ubuntu we mean.

Signed-off-by: Chris Sharp <csharp@georgialibraries.org>
Signed-off-by: Ben Shum <bshum@biblio.org>
5 years agoLP#1315531 Correcting typo: "eg_host.conf" -> "eg_vhost.conf".
Chris Sharp [Tue, 3 Jun 2014 18:13:47 +0000 (14:13 -0400)]
LP#1315531 Correcting typo: "eg_host.conf" -> "eg_vhost.conf".

Signed-off-by: Chris Sharp <csharp@georgialibraries.org>
Signed-off-by: Ben Shum <bshum@biblio.org>
5 years agoLP#1315531 Adding further changes to the server install docs for Ubuntu 14.04:
Chris Sharp [Sat, 31 May 2014 20:25:29 +0000 (16:25 -0400)]
LP#1315531 Adding further changes to the server install docs for Ubuntu 14.04:

- adding apache configuration instructions that adjust for the new
mpm_prefork module in Apache 2.4 on 14.04
- adding new standalone database dependency installation instructions
that rely on OS packages rather than CPAN where possible

Signed-off-by: Chris Sharp <csharp@georgialibraries.org>
Signed-off-by: Ben Shum <bshum@biblio.org>
5 years agoLP#1315531 Ben Shum pointed out an error, which I have now corrected.
Chris Sharp [Fri, 30 May 2014 02:17:10 +0000 (22:17 -0400)]
LP#1315531 Ben Shum pointed out an error, which I have now corrected.

Signed-off-by: Chris Sharp <csharp@georgialibraries.org>
Signed-off-by: Ben Shum <bshum@biblio.org>
5 years agoLP#1315531 Change default location of libdbi libraries.
Chris Sharp [Thu, 29 May 2014 19:49:38 +0000 (15:49 -0400)]
LP#1315531 Change default location of libdbi libraries.

In libdbi 0.8.4, the default installed location has changed to
/usr/lib/x86_64-linux-gnu/dbd/.  On the advice of Thomas Berezansky,
I made the new location the default and ensured that the previous
locations are in the fallback.

Signed-off-by: Chris Sharp <csharp@georgialibraries.org>
Signed-off-by: Ben Shum <bshum@biblio.org>
5 years agoLP#1315531 Missed libtest-output-perl in the deb list.
Chris Sharp [Thu, 3 Apr 2014 18:22:33 +0000 (14:22 -0400)]
LP#1315531 Missed libtest-output-perl in the deb list.

Signed-off-by: Chris Sharp <csharp@georgialibraries.org>
Signed-off-by: Ben Shum <bshum@biblio.org>
5 years agoLP#1315531 Adding Business::Stripe; Moving CPAN stuff to debs where I can
Chris Sharp [Thu, 3 Apr 2014 18:11:11 +0000 (14:11 -0400)]
LP#1315531 Adding Business::Stripe; Moving CPAN stuff to debs where I can

Signed-off-by: Chris Sharp <csharp@georgialibraries.org>
Signed-off-by: Ben Shum <bshum@biblio.org>
5 years agoLP#1315531 Continuing to add trusty support and remove lucid.
Chris Sharp [Thu, 3 Apr 2014 16:54:45 +0000 (12:54 -0400)]
LP#1315531 Continuing to add trusty support and remove lucid.

Adding PostgreSQL 9.3 as the default database for 14.04.

Signed-off-by: Chris Sharp <csharp@georgialibraries.org>
Signed-off-by: Ben Shum <bshum@biblio.org>
5 years agoLP#1315531 adding trusty Makefile.install, removing lucid support
Chris Sharp [Fri, 7 Feb 2014 11:14:52 +0000 (06:14 -0500)]
LP#1315531 adding trusty Makefile.install, removing lucid support

Signed-off-by: Chris Sharp <csharp@georgialibraries.org>
Signed-off-by: Ben Shum <bshum@biblio.org>
5 years agoDocumentation: Upgrade instructions examples
Remington Steed [Wed, 9 Jul 2014 19:40:00 +0000 (15:40 -0400)]
Documentation: Upgrade instructions examples

This commit adds a note (with examples) about upgrading the database schema
across major versions. It also reformats a long line into multiple lines,
changes a NOTE to a TIP, makes a small correction, and replaces a repetitive
command with a short explanation.

Signed-off-by: Remington Steed <rjs7@calvin.edu>
5 years agoDocumentation: Update upgrade instruction for 2.6.1
Robert Soulliere [Wed, 2 Jul 2014 18:57:16 +0000 (14:57 -0400)]
Documentation: Update upgrade instruction for 2.6.1

- Update location references for 2.6.1.
- Update database updates to include 2.6.1 update.
- Add a note regarding reingesting bib records after updates.

Signed-off-by: Robert Soulliere <robert.soulliere@mohawkcollege.ca>
5 years agoDocumentation: updated some SIP content
Yamil Suarez [Wed, 9 Jul 2014 21:23:49 +0000 (17:23 -0400)]
Documentation: updated some SIP content

Signed-off-by: Yamil Suarez <yamil@yamil.com>
5 years agoDocumentation: added AsciiDoc version of old SIP docs from EG 1.6/2.0 docs
Yamil Suarez [Wed, 9 Jul 2014 01:13:07 +0000 (21:13 -0400)]
Documentation: added AsciiDoc version of old SIP docs from EG 1.6/2.0 docs

Signed-off-by: Yamil Suarez <yamil@yamil.com>
5 years agoLP#1339767: follow-up: Thumbs.db, not Thumbs.DB
Galen Charlton [Wed, 9 Jul 2014 16:07:32 +0000 (09:07 -0700)]
LP#1339767: follow-up: Thumbs.db, not Thumbs.DB

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Signed-off-by: Ben Shum <bshum@biblio.org>
5 years agoLP#1339767: add Thumbs.DB to .gitignore
Galen Charlton [Wed, 9 Jul 2014 15:47:09 +0000 (08:47 -0700)]
LP#1339767: add Thumbs.DB to .gitignore

Removes a minor annoyance for documenters who use recent
versions of Windows

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Signed-off-by: Ben Shum <bshum@biblio.org>
5 years agoDocumentation: Add the authentication proxy chapter
Robert Soulliere [Thu, 3 Jul 2014 15:18:15 +0000 (11:18 -0400)]
Documentation: Add the authentication proxy chapter

- Some content taken from the 2.2 release notes.
- Added a few steps on how to set up authentication proxy in Evergreen.

Signed-off-by: Robert Soulliere <robert.soulliere@mohawkcollege.ca>
5 years agoDocs: Minor wording change
Erica Rohlfs [Fri, 30 May 2014 19:12:29 +0000 (15:12 -0400)]
Docs: Minor wording change

Signed-off-by: Erica Rohlfs <erohlfs@esilibrary.com>
Signed-off-by: Remington Steed <rjs7@calvin.edu>
5 years agoLP#968514: Revive bib/auth importing docs from 2.1, add intro
Remington Steed [Fri, 6 Jun 2014 15:59:58 +0000 (11:59 -0400)]
LP#968514: Revive bib/auth importing docs from 2.1, add intro

This commit revives the following sections from the 2.1 version of the docs
and updates them for 2.6:

  - marc_export: Exporting Bibliographic Records into MARC files
    - includes new options for 2.6
  - Importing Authority Records from Command Line
  - Importing Authority Records from the Staff Client

The first two are included in a new section called "Support Scripts"
within the "Developer Resources" section, with a new intro and summary
of other commonly used scripts provided with Evergreen. The third section
is moved inside the Cataloging chapter "Batch Importing MARC Records",
with slight modifications to the intro to that chapter.

Signed-off-by: Remington Steed <rjs7@calvin.edu>
5 years agoDocumentation: added "search URL" text inside of "using OPAC" section
Yamil Suarez [Tue, 1 Jul 2014 18:58:33 +0000 (14:58 -0400)]
Documentation: added "search URL" text inside of "using OPAC" section

Replaces temporary text that was accidentally added to the "using OPAC" section.

Signed-off-by: Yamil Suarez <yamil@yamil.com>
5 years agoDocumentation: Added 'Using the OPAC' documentation & OPAC WCAG support mention
Yamil Suarez [Mon, 30 Jun 2014 21:55:46 +0000 (17:55 -0400)]
Documentation: Added 'Using the OPAC' documentation & OPAC WCAG support mention

Older DocBook content that covered how to do searches on the OPAC was updated
to use Asciidoc. Also a simple mention of the 2.6 OPAC WCAG support was added.

Signed-off-by: Yamil Suarez <yamil@yamil.com>
5 years agoDocumentation for Located URI Visibility
Kathy Lussier [Fri, 27 Jun 2014 19:31:56 +0000 (15:31 -0400)]
Documentation for Located URI Visibility

Documentation for the new visibility options available for Located URI's
in 2.6.

Signed-off-by: Kathy Lussier <klussier@masslnc.org>
5 years agoFix syntax in filter_group_selector.tt2
Dan Wells [Thu, 26 Jun 2014 21:46:30 +0000 (17:46 -0400)]
Fix syntax in filter_group_selector.tt2

Attempt to fix build breakage introduced by 7e4e9d669

Signed-off-by: Dan Wells <dbw2@calvin.edu>
5 years agoLP#1326149 Use a TPAC-settable TIME_FORMAT for local time formats
Dan Scott [Wed, 4 Jun 2014 17:09:48 +0000 (13:09 -0400)]
LP#1326149 Use a TPAC-settable TIME_FORMAT for local time formats

We had tried to use the format.time library setting, but that is meant
for Dojo which uses the Unicode formats, that are incompatible with
the POSIX strftime() formats required by Template::Plugin::Date.

At the same time, we need to expose the raw machine-readable time
for schema.org consumers, so express that in a @content attribute.

And we might as well use a more meaningful <time> element instead of
the meaningless <span>, just for good measure.

Signed-off-by: Dan Scott <dscott@laurentian.ca>
Signed-off-by: Ben Shum <bshum@biblio.org>
5 years agoLP#1301599 improved TPAC jacket alt text
Bill Erickson [Thu, 10 Apr 2014 21:11:21 +0000 (17:11 -0400)]
LP#1301599 improved TPAC jacket alt text

Change "Image of Item" to "Book Cover" as it's slightly more
descriptive/specific.

Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Ben Shum <bshum@biblio.org>
5 years agoLP#1301599 Remove duplicate title attributes
Bill Erickson [Fri, 30 May 2014 16:55:47 +0000 (12:55 -0400)]
LP#1301599 Remove duplicate title attributes

Thanks to Ben Shum for spotting these!

Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Ben Shum <bshum@biblio.org>
5 years agoLP#1301599 Additional TPAC facets structure markup
Bill Erickson [Thu, 10 Apr 2014 20:59:14 +0000 (16:59 -0400)]
LP#1301599 Additional TPAC facets structure markup

Labeling facets with <h4> tags provides additional page structure for
screen readers in the TPAC results pages.

Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Ben Shum <bshum@biblio.org>
5 years agoLP#1301599 TPAC replace aria-label with title
Bill Erickson [Thu, 10 Apr 2014 20:49:36 +0000 (16:49 -0400)]
LP#1301599 TPAC replace aria-label with title

As appropriate, replace aria-label attributes with traditional HTML
title attributes.

For ref, from the LP entry:

http://www.w3.org/TR/2013/WD-aria-in-html-20131003/#first-rule-of-aria-use

"If you can use a native HTML element [HTML5] or attribute with the
semantics and behaviour you require already built in, instead of
re-purposing an element and adding an ARIA role, state or property to
make it accessible, then do so."

Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Ben Shum <bshum@biblio.org>
5 years agoLP#1301599 TPAC add missing metabib filter label
Bill Erickson [Thu, 3 Apr 2014 18:53:36 +0000 (14:53 -0400)]
LP#1301599 TPAC add missing metabib filter label

In the quick search bar along the top.

Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Ben Shum <bshum@biblio.org>
5 years agoLP#1301599 TPAC advanced search from labels
Bill Erickson [Thu, 3 Apr 2014 17:31:29 +0000 (13:31 -0400)]
LP#1301599 TPAC advanced search from labels

Instead of using titles / aria-labels directly on advanced search
selectors, provide selector IDs so the existing text labels can be
turned into <labels> and linked (by id) to their form control.

This means we only need one label per form control, which improves
usability from an accessibility perspective.

Signed-off-by: Ben Shum <bshum@biblio.org>
5 years agoLP#1301599 TPAC advanced search additional labels
Bill Erickson [Wed, 2 Apr 2014 20:46:25 +0000 (16:46 -0400)]
LP#1301599 TPAC advanced search additional labels

Provides a number of labels, primarily for select widgets, in the TPAC
advanced search form for improved accessibility.

Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Ben Shum <bshum@biblio.org>
5 years agoLP#1301599 TPAC result table accessibility improvements
Bill Erickson [Wed, 2 Apr 2014 20:43:49 +0000 (16:43 -0400)]
LP#1301599 TPAC result table accessibility improvements

* Give the main result table a screenreader-only header to describe the
  table columns.
* Set the role=presentation in one of the sub-tables used only for
  layout.

Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Ben Shum <bshum@biblio.org>
5 years agoLP#925776 Add placeholder script for backport to 2.5 for function staff uri visibility
Ben Shum [Thu, 26 Jun 2014 19:33:30 +0000 (15:33 -0400)]
LP#925776 Add placeholder script for backport to 2.5 for function staff uri visibility

Signed-off-by: Ben Shum <bshum@biblio.org>
5 years agoLP#925776 Stamping upgrade script for staff uri visibility
Ben Shum [Thu, 26 Jun 2014 19:30:26 +0000 (15:30 -0400)]
LP#925776 Stamping upgrade script for staff uri visibility

Signed-off-by: Ben Shum <bshum@biblio.org>
5 years agoLP#925776 Adding upgrade script
Mike Rylander [Fri, 2 May 2014 13:42:28 +0000 (09:42 -0400)]
LP#925776 Adding upgrade script

Signed-off-by: Mike Rylander <mrylander@gmail.com>
Signed-off-by: Ben Shum <bshum@biblio.org>
5 years agoLP#925776: Recheck located uri visibility
Mike Rylander [Fri, 2 May 2014 13:10:09 +0000 (09:10 -0400)]
LP#925776: Recheck located uri visibility

Previous to this commit, once we have determined that there are no
local Located URIs or foreign copies, we include the record in the
result set in the staff client.  However, the desire of library staff
is that foreign Located URIs be valid as a visibility limiter.  So
we now recheck at the end for that situation.  This honors the new
"Located URIs act like copies" global flag.

Signed-off-by: Mike Rylander <mrylander@gmail.com>
Signed-off-by: Ben Shum <bshum@biblio.org>
5 years agoForward port 2.5.5 and 2.6.1 upgrade scripts
Dan Wells [Fri, 6 Jun 2014 13:19:58 +0000 (09:19 -0400)]
Forward port 2.5.5 and 2.6.1 upgrade scripts

Signed-off-by: Dan Wells <dbw2@calvin.edu>
5 years agoLP#1303544 Trim junk from the ISBN in record summary
Dan Scott [Fri, 6 Jun 2014 00:27:39 +0000 (20:27 -0400)]
LP#1303544 Trim junk from the ISBN in record summary

When asserting that we are publishing an ISBN, move the "(pbk.)"
and similar such matter outside of the schema.org <span property="isbn">
element.

See "The walking dead" in the sample dataset for an example.

Signed-off-by: Dan Scott <dscott@laurentian.ca>
Signed-off-by: Ben Shum <bshum@biblio.org>
5 years agoLP#1326983: excluded fulfilled holds when adding hold_request.shelf_expires_soon...
Galen Charlton [Thu, 5 Jun 2014 22:16:23 +0000 (15:16 -0700)]
LP#1326983: excluded fulfilled holds when adding hold_request.shelf_expires_soon events

This patch adjusts the example A/T filter for the
hold_request.shelf_expires_soon hook to exclude hold requests that are
already marked as fulfilled.  This saves time creating events that
would immediately be marked as invalid per the HoldIsAvailable
validator.

It also addresses an issue for large databases that have lots of
fulfilled holds wherein hold_request.shelf_expires_soon events can
fail to be added at all due to a cstore timeout.

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Signed-off-by: Ben Shum <bshum@biblio.org>
5 years agoLP#1326806 Minor repairs to EDI admin documentation
Bill Erickson [Thu, 5 Jun 2014 13:47:36 +0000 (09:47 -0400)]
LP#1326806 Minor repairs to EDI admin documentation

Updates to EDI account Path/Incoming/Account docs.

Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Ben Shum <bshum@biblio.org>
5 years agoLP#1284832: Add missing seed data: enable combined search for only the subject class...
Jeff Davis [Tue, 25 Feb 2014 20:41:56 +0000 (12:41 -0800)]
LP#1284832: Add missing seed data: enable combined search for only the subject class by default

Signed-off-by: Jeff Davis <jdavis@sitka.bclibraries.ca>
Signed-off-by: Galen Charlton <gmc@esilibrary.com>
5 years agoLP#1321363: Suppress Hold Transit use elicits client error
Dan Pearl [Tue, 20 May 2014 17:12:47 +0000 (13:12 -0400)]
LP#1321363: Suppress Hold Transit use elicits client error

This change restores the behavior as designed.

Signed-off-by: Dan Pearl <dpearl@cwmars.org>
Signed-off-by: Galen Charlton <gmc@esilibrary.com>
5 years agoForward port 2.5.4 upgrade script
Dan Wells [Thu, 29 May 2014 13:15:01 +0000 (09:15 -0400)]
Forward port 2.5.4 upgrade script

Signed-off-by: Dan Wells <dbw2@calvin.edu>
5 years agoLP#1296937: move the $force_bc parameter of ->charged_items() to an implementation...
Galen Charlton [Tue, 20 May 2014 17:09:08 +0000 (10:09 -0700)]
LP#1296937: move the $force_bc parameter of ->charged_items() to an implementation method

Argubably, the $force_bc parameter of OpenILS::SIP::Patron->charged_items()
method exists solely as an implementation convenience for the renew-all
code.  Consequently, this patch removes the $force_bc parameter from the
public interface for charged_items() and spins it off into a new
implementation method.

This patch should be applied at the same time that a reversion of
SIPServer commit c97d64412bc is applied.  Note that a user who upgrades
SIPServer without upgrading Evergreen to a version that contains this
patch will end up in a situation where the code that counts active
loans for the patron information response will fetch the loans using
$force_bc set to true; this works only by coincidence.

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Signed-off-by: Jeff Godin <jgodin@tadl.org>
5 years agoLP#1321017: Order constituent records by quality
Mike Rylander [Tue, 20 May 2014 14:37:30 +0000 (10:37 -0400)]
LP#1321017: Order constituent records by quality

Previously, in some cases (often needlessly) we ordered constituent
records within a metarecord by a combination of type/form/blvl.  This
is not only of little use, but also expensive.  Instead, order them
by the bib's calculated quality, which takes type/form/blvl into account
already.

Also, use the new metabib.record_sorter to find the title tie-breaker
instead of using the view-of-a-view-of-a-view mrd compatability shim.

The net result is several orders of magnitude speed increase for
constituent record retrieval.  In particular, this resolves a problem
where doing a SIP2 patron information lookup and requesting a list
of holds could cause a timeout if the patron has one or more
metarecord holds.

This patch also causes the format filter parameter of the following
methods to be ignored:

open-ils.search.biblio.metarecord_to_records*
open-ils.search.biblio.metarecord.mods_slim.batch.retrieve*
open-ils.search.biblio.metarecord.mods_slim.retrieve*

Since the only use of the format filter was in JSPac code, and since
the record attributes mechanism has changed considerably, this removal
should be considered intentional.

Note that this patch includes a squash of a minor follow-up by Galen
Charlton to remove variables made unused by the original patch.

Signed-off-by: Mike Rylander <mrylander@gmail.com>
Signed-off-by: Galen Charlton <gmc@esilibrary.com>
(for minor follow-up by Galen)
Signed-off-by: Dan Wells <dbw2@calvin.edu>
5 years agoLP#1310751: Reorder query-building wrapper for QueryParser
Mike Rylander [Wed, 14 May 2014 15:54:47 +0000 (11:54 -0400)]
LP#1310751: Reorder query-building wrapper for QueryParser

In order to make sure that the transparent boolean pushdown does
not cause filters and modifiers to end up deeply embedded in the
query tree, we put them at the front.

This is a short-term fix.  Long-term, we need to make use of the
floating subquery syntax ( {{ ... }} ) available in QueryParser,
which is designed to address the semantic separation between what
the user types into the search box, and the various UI widgets that
can add query clauses.

Signed-off-by: Jason Etheridge <jason@esilibrary.com>
Signed-off-by: Dan Wells <dbw2@calvin.edu>
5 years agoLP#1322303 cleanse backdate for checkin overdue voiding
Bill Erickson [Thu, 22 May 2014 18:41:17 +0000 (14:41 -0400)]
LP#1322303 cleanse backdate for checkin overdue voiding

Passing raw backdates, particularly from offline transactions, which
have a space between the date and time, to
DateTime::Format::ISO8601->parse_datetime results in "Invalid date
format:" errors.  Pass the date through the cleanse routine first.

Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Ben Shum <bshum@biblio.org>
5 years agoFix 2.6 Release Notes typos and formatting
Remington Steed [Thu, 24 Apr 2014 14:09:08 +0000 (10:09 -0400)]
Fix 2.6 Release Notes typos and formatting

This commit fixes an AsciiDoc URL processing bug by marking two fake URLs
as lines of code. It also fixes a few small typos and misspellings.

Signed-off-by: Remington Steed <rjs7@calvin.edu>
5 years agoUpdate docs to point to 2.6 release notes
Remington Steed [Wed, 23 Apr 2014 13:52:25 +0000 (09:52 -0400)]
Update docs to point to 2.6 release notes

Signed-off-by: Remington Steed <rjs7@calvin.edu>
5 years ago(documentation) fixed typo.
Erica Rohlfs [Fri, 23 May 2014 18:00:16 +0000 (14:00 -0400)]
(documentation) fixed typo.

Signed-off-by: Erica Rohlfs <erohlfs@esilibrary.com>
Signed-off-by: Galen Charlton <gmc@esilibrary.com>
5 years ago(doc) 'debian-wheezy' was left out of the OS options
Josh Stompro [Wed, 21 May 2014 13:37:58 +0000 (08:37 -0500)]
(doc) 'debian-wheezy' was left out of the OS options

Just added 'debian-wheezy' to the OS options, it was left out by mistake.

Signed-off-by: Josh Stompro <Stomproj@larl.org>
Signed-off-by: Galen Charlton <gmc@esilibrary.com>
5 years ago(doc) add overdues to summary of the phone list feature
Josh Stompro [Mon, 19 May 2014 20:17:15 +0000 (15:17 -0500)]
(doc) add overdues to summary of the phone list feature

Added "or overdues" to the summary so it is clear that phonelist
handles both holds and overdues.

Signed-off-by: Josh Stompro <Stomproj@larl.org>
Signed-off-by: Galen Charlton <gmc@esilibrary.com>
5 years agoLP#1314920 Stamping upgrade script
Dan Wells [Thu, 22 May 2014 15:24:41 +0000 (11:24 -0400)]
LP#1314920 Stamping upgrade script

Signed-off-by: Dan Wells <dbw2@calvin.edu>
5 years agoLP#1314920 change the desc for the org hiding setting
Jason Etheridge [Thu, 1 May 2014 08:15:55 +0000 (04:15 -0400)]
LP#1314920 change the desc for the org hiding setting

so that it references physical_loc instead of ol

Signed-off-by: Jason Etheridge <jason@esilibrary.com>
Signed-off-by: Dan Wells <dbw2@calvin.edu>
5 years agoRemove buggy, unused function get_leaders_from_usergroup
Remington Steed [Fri, 7 Mar 2014 18:56:25 +0000 (13:56 -0500)]
Remove buggy, unused function get_leaders_from_usergroup

Recent editing and testing of get_leaders_from_usergroup() uncovered an
unrelated bug which renders the function useless (it returns all group
members, not just the leader). It wasn't previously caught because
Evergreen has no apparent references to this function. This commit
removes the unused function.

Signed-off-by: Remington Steed <rjs7@calvin.edu>
Signed-off-by: Dan Wells <dbw2@calvin.edu>
5 years agoLP#1321411: restore ability to handle credit payments using a processor other than...
Galen Charlton [Tue, 20 May 2014 18:42:32 +0000 (11:42 -0700)]
LP#1321411: restore ability to handle credit payments using a processor other than Stripe

This patch ensures that the processor name is passed
to OpenILS::Application::Circ::CreditCard::process_payment, avoiding
an exception thrown by Business::OnlinePayment.

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Signed-off-by: Dan Wells <dbw2@calvin.edu>
5 years agoLP#1246843: Don't show contact invalidators for new users
Jeff Godin [Wed, 30 Oct 2013 18:37:10 +0000 (14:37 -0400)]
LP#1246843: Don't show contact invalidators for new users

Don't show contact invalidator buttons for new users. This affects
staged users and cloned users, where the contact fields have a value
when the editor initially loads.

Signed-off-by: Jeff Godin <jgodin@tadl.org>
Signed-off-by: Ben Shum <bshum@biblio.org>
5 years agoLP#1321429: Use server-local time as best-guess
Mike Rylander [Tue, 20 May 2014 18:30:20 +0000 (14:30 -0400)]
LP#1321429: Use server-local time as best-guess

When first calculating the shelf expire time, we assume "now" as
the starting point by calling DateTime->now().  However, that gives
us the time in UTC.  That's not good because closed dates are stored
timezone-aware.  Instead, ask for server-local time.

Signed-off-by: Mike Rylander <mrylander@gmail.com>
Signed-off-by: Ben Shum <bshum@biblio.org>
5 years agoLP#1314902: Restore header bar styles on results pages
Dan Wells [Thu, 1 May 2014 17:47:46 +0000 (13:47 -0400)]
LP#1314902: Restore header bar styles on results pages

Commit fc99f4f2 made an incomplete style change, changing
'results_header_bar' to be a class, but not changing the CSS.  The
simplest solution is to restore 'results_header_bar' to be an id, as
this fixes the main view, the responsive view, and also any custom CSS
which might affect this id.

Signed-off-by: Dan Wells <dbw2@calvin.edu>
Signed-off-by: Remington Steed <rjs7@calvin.edu>
Signed-off-by: Galen Charlton <gmc@esilibrary.com>
5 years agoLP#1312945: pin schema update
Galen Charlton [Tue, 20 May 2014 23:48:57 +0000 (16:48 -0700)]
LP#1312945: pin schema update

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
5 years agoLP#1312945: add regression test
Galen Charlton [Tue, 20 May 2014 23:46:02 +0000 (16:46 -0700)]
LP#1312945: add regression test

This adds a test for the change to the
authority.calculate_authority_linking stored function.

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
5 years agoLP#1312945: baseline schema update
Mike Rylander [Wed, 30 Apr 2014 22:38:29 +0000 (18:38 -0400)]
LP#1312945: baseline schema update

Signed-off-by: Mike Rylander <mrylander@gmail.com>
Signed-off-by: Srey Seng <sreyseng@gmail.com>
Signed-off-by: Galen Charlton <gmc@esilibrary.com>
5 years agoLP#1312945: authority.calculate_authority_linking and multiple linked tags
Srey Seng [Wed, 30 Apr 2014 22:20:17 +0000 (15:20 -0700)]
LP#1312945: authority.calculate_authority_linking and multiple linked tags

The function "authority.calculate_authority_linking" was not able to deal with
situations where there are multiple marc tags that are linkable. The function
only process the first tag that contains a linking_subfield and returns that
to be inserted into the authority_linking table, even when there are more than
one of the same tag with linking_subfield.

Added an additional loop to loop through each tag, for situations where there are
multiples of each tag.

Signed-off-by: Srey Seng <sreyseng@gmail.com>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
Signed-off-by: Galen Charlton <gmc@esilibrary.com>
5 years agoLP#1312945: auth-auth linking: cache less agressively and look for all links
Mike Rylander [Wed, 30 Apr 2014 16:58:11 +0000 (12:58 -0400)]
LP#1312945: auth-auth linking: cache less agressively and look for all links

There was a logic inversion that was causing miss-use of a per-
record cache mechanism.  I remove that entirely, as it's not helpful
in practice.

Also, after finding one use of a linked field we moved on to the
next, though it is possible for more than one link to exist for
different uses of a name, say.  Now we'll loop through all fields
to find all possible linkages.

Signed-off-by: Mike Rylander <mrylander@gmail.com>
Signed-off-by: Srey Seng <sreyseng@gmail.com>
Signed-off-by: Galen Charlton <gmc@esilibrary.com>
5 years agoLP#1296937: (follow-up) make $ids_only be the last parameter for ->charged_items()
Galen Charlton [Tue, 20 May 2014 15:46:47 +0000 (08:46 -0700)]
LP#1296937: (follow-up) make $ids_only be the last parameter for ->charged_items()

This change ensures that if a site fails to upgrade SIPServer at the
same time that they upgrade Evergreen, the renew all message won't
break on them.

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
5 years agoLP#1296937: (SIP2) Add ids_only parameter to _items functions
Thomas Berezansky [Thu, 23 Jan 2014 17:07:06 +0000 (12:07 -0500)]
LP#1296937: (SIP2) Add ids_only parameter to _items functions

This allows the caller to skip loading of barcodes and/or titles when
they are not needed, such as when the plan is to return counts, not
details.  Per bug 1321017, this is particular useful for patrons
that have metarecord holds.

Signed-off-by: Thomas Berezansky <tsbere@mvlc.org>
Signed-off-by: Galen Charlton <gmc@esilibrary.com>
5 years agoLP#1320048: remove the Date.W3CDTF() JavaScript module
Galen Charlton [Fri, 16 May 2014 17:40:49 +0000 (10:40 -0700)]
LP#1320048: remove the Date.W3CDTF() JavaScript module

With the closed dates editor now using util.date, and
no other active use of Date.W3CDTF() going on, it can go.

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