working/Evergreen.git
3 years agoLP1183964: Payment Limit Options
Jason Boyer [Fri, 13 May 2016 16:51:07 +0000 (12:51 -0400)]
LP1183964: Payment Limit Options

Add support for 2 new OU settings, a payment warning
amount (try to pay more than this and you're required
to confirm it) and a hard limit where no payment larger
will be accepted. Support is also added to both the XUL
client and webstaff client.

Signed-off-by: Jason Boyer <jboyer@library.in.gov>
Signed-off-by: Jeanette Lundgren <jlundgren@cwmars.org>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
3 years agoLP 1481441: Improve Hold Failure Messages
Jason Boyer [Fri, 14 Aug 2015 15:26:54 +0000 (11:26 -0400)]
LP 1481441: Improve Hold Failure Messages

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

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

3 years agoStamping upgrade for reingest speedup
Mike Rylander [Tue, 9 Aug 2016 16:04:35 +0000 (12:04 -0400)]
Stamping upgrade for reingest speedup

Signed-off-by: Mike Rylander <mrylander@gmail.com>
3 years agoLP#1588543: schema update script
Galen Charlton [Fri, 29 Jul 2016 15:40:46 +0000 (11:40 -0400)]
LP#1588543: schema update script

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
3 years agoLP#1588543: tests for verifying correct generated of record attributes
Galen Charlton [Fri, 29 Jul 2016 15:34:06 +0000 (11:34 -0400)]
LP#1588543: tests for verifying correct generated of record attributes

This patch adds tests to verify that the expected
set of record attributes are generated when a bib record
is ingested.  The tests serve a couple purposes:

- verifying that changes to the record attribute ingest
  stored procedure don't have unexpected side-effects
- validating record attribute definitions in the seed data

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
3 years agoLP#1588543: further speed up record attribute ingest
Galen Charlton [Fri, 29 Jul 2016 15:24:07 +0000 (11:24 -0400)]
LP#1588543: further speed up record attribute ingest

Following a suggestion from Dan Wells, this patch adds
an index on config.coded_value_map(ctype). My testing shows
that the index by itself can halve the time required to
fully ingest a new record; in conjunction with the
change in the previous patch, full reingest time is
reduced by about 60%.

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
3 years agoLP#1588543: Speed up record attribute ingest
Dan Wells [Wed, 1 Jun 2016 18:36:29 +0000 (14:36 -0400)]
LP#1588543: Speed up record attribute ingest

A huge number of rows (700+) have been added to
config.record_attr_definition in recent versions of Evergreen. However,
these rows do not actually define any indexing at all, but are used for
other purposes.  The reingest process currently churns over all of these
rows to ultimately do nothing.

This commit limits the group of attributes considered at ingest to only
those which might index something.  Testing indicates a substantial
speed boost to typical use of metabib.reingest_record_attributes()
(over 2x speedup).

Signed-off-by: Dan Wells <dbw2@calvin.edu>
Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
4 years agoLP#1608711: Update schema.org property name from "seller" to "offeredBy"
Dan Scott [Mon, 1 Aug 2016 22:09:04 +0000 (18:09 -0400)]
LP#1608711: Update schema.org property name from "seller" to "offeredBy"

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

Signed-off-by: Dan Scott <dscott@laurentian.ca>
Signed-off-by: Ben Shum <ben@evergreener.net>
4 years agoLP#1549505: Stamping upgrade scripts 0983-84 for stat pop ratings
Kathy Lussier [Fri, 29 Jul 2016 21:51:14 +0000 (17:51 -0400)]
LP#1549505: Stamping upgrade scripts 0983-84 for stat pop ratings

Signed-off-by: Kathy Lussier <klussier@masslnc.org>
4 years agoLP#1549505: update $modal to $uibModal
Galen Charlton [Thu, 28 Jul 2016 16:19:15 +0000 (12:19 -0400)]
LP#1549505: update $modal to $uibModal

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Signed-off-by: Kathy Lussier <klussier@masslnc.org>
4 years agoLP#1549505: Decrease value of Max popularity importance multiplier
Kathy Lussier [Thu, 21 Jul 2016 16:02:38 +0000 (12:02 -0400)]
LP#1549505: Decrease value of Max popularity importance multiplier

My testing found that a setting between 1.1 and 1.2 resulted in fairly good
results when using the popularity-adjusted relevance sort. Adjusting the value
in the seed data to 1.1 since it will be a better starting point for libraries.

Signed-off-by: Kathy Lussier <klussier@masslnc.org>
Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Signed-off-by: Kathy Lussier <klussier@masslnc.org>
4 years agoLP#1549505: Remove stray semicolon from PgTap test
Kathy Lussier [Thu, 21 Jul 2016 15:52:24 +0000 (11:52 -0400)]
LP#1549505: Remove stray semicolon from PgTap test

The semicolon was causing failures in the test.

Signed-off-by: Kathy Lussier <klussier@masslnc.org>
Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Signed-off-by: Kathy Lussier <klussier@masslnc.org>
4 years agoLP#1549505: Query literal interpolation casts incorrectly
Mike Rylander [Thu, 26 May 2016 21:51:49 +0000 (17:51 -0400)]
LP#1549505: Query literal interpolation casts incorrectly

We need to be sure that all query values are of the same type in the same
columns because PLPGSQL functions are compiled and cached.  In this case,
the core query of the in-db search cannot have its shape change.  This
commit assures that browse and search uses of the SP provide core queries
that match on their SELECT lists.  Of particular importance is the type
of the "rel" output column, which was variously float8 or numeric, depending
on whether the search contained any terms (a "search") or not (a browse
link).

Signed-off-by: Mike Rylander <mrylander@gmail.com>
Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Signed-off-by: Kathy Lussier <klussier@masslnc.org>
4 years agoLP#1549505: fix staged browse
Galen Charlton [Wed, 25 May 2016 16:53:24 +0000 (12:53 -0400)]
LP#1549505: fix staged browse

This patch ensures that the core query passed to
query_parser_fts by staged browse contains dummy
badges and popularity columns (and thus, doesn't break);
note that browse itself cares nothing about popularity.

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Signed-off-by: Kathy Lussier <klussier@masslnc.org>
4 years agoLP#1549505: update baseline database schema
Galen Charlton [Fri, 6 May 2016 15:29:39 +0000 (11:29 -0400)]
LP#1549505: update baseline database schema

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Signed-off-by: Kathy Lussier <klussier@masslnc.org>
4 years agoLP 1549505: Fix syntax error in OpenILS/WWW/EGCatLoader/Record.pm
Jason Stephenson [Fri, 15 Apr 2016 13:36:51 +0000 (09:36 -0400)]
LP 1549505: Fix syntax error in OpenILS/WWW/EGCatLoader/Record.pm

Typo/error apparently introduced on line 72 in commit eeee27c.

Signed-off-by: Jason Stephenson <jstephenson@mvlcstaff.org>
Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Signed-off-by: Kathy Lussier <klussier@masslnc.org>
4 years agoLP#1549505: Release notes for statistically generated record ratings (popularity)
Mike Rylander [Tue, 5 Apr 2016 20:26:41 +0000 (16:26 -0400)]
LP#1549505: Release notes for statistically generated record ratings (popularity)

Signed-off-by: Mike Rylander <mrylander@gmail.com>
Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Signed-off-by: Kathy Lussier <klussier@masslnc.org>
4 years agoLP#1549505: PGTap test to show the infrastructure is working
Mike Rylander [Tue, 5 Apr 2016 16:10:53 +0000 (12:10 -0400)]
LP#1549505: PGTap test to show the infrastructure is working

Signed-off-by: Mike Rylander <mrylander@gmail.com>
Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Signed-off-by: Kathy Lussier <klussier@masslnc.org>
4 years agoLP#1549505: add flag to tweak popularity-adjusted relevance
Galen Charlton [Fri, 11 Mar 2016 18:27:03 +0000 (13:27 -0500)]
LP#1549505: add flag to tweak popularity-adjusted relevance

This adds a new global_flag, search.max_popularity_importance_multiplier,
to control the factor by which popularity affects Popularity Adjusted
Relevance ranking.

The value should be a decimal number, typically between 1.0 and 2.0:

* 1.0 be would be equivalent to not adjusting relevance for popularity
  at all.
* 1.1 would mean that the multiplier would range from 1 (for zero
  popularity) to 1.1 (for maximum popularity), for a maximum boost of
  10% of the base relevance value of the record.

What's happening in the code:

Scale the 0-5 effect of popularity badges by providing a multiplier
for the badge average based on the overall maximum multiplier.  Two
examples, comparing the effect to the default $max_mult value of 2.0,
which causes a $adjusted_scale value of 0.2:

 * Given the default $max_mult of 2.0, the value of
   $adjusted_scale will be 0.2 [($max_mult - 1.0) / 5.0].
   For a record whose average badge score is the maximum
   of 5.0, that would make the relevance multiplier be
   2.0:
      1.0 + (5.0 [average score] * 0.2 [ $adjusted_scale ],
   This would have the effect of doubling the effective
   relevance of highly popular items.

 * Given a $max_mult of 1.1, the value of $adjusted_scale
   will be 0.02, meaning that the average badge value will be
   multiplied by 0.02 rather than 0.2, then added to 1.0 and
   used as a multiplier against the base relevance.  Thus a
   change of at most 10% to the base relevance for a record
   with a 5.0 average badge score. This will allow records
   that are naturally very relevant to avoid being pushed
   below badge-heavy records.

 * Given a $max_mult of 3.0, the value of $adjusted_scale
   will be 0.4, meaning that the average badge value will be
   multiplied by 0.4 rather than 0.2, then added to 1.0 and
   used as a multiplier against the base relevance. Thus a
   change of as much as 200% to (or three times the size of)
   the base relevance for a record with a 5.0 average badge
   score.  This in turn will cause badges to outweigh
   relevance to a very large degree.

The maximum badge multiplier can be set to a value less than
1.0; this would have the effect of making less popular items
show up higher in the results.  While this is not a likely
option for production use, it could be useful for identifying
interesting long-tail hits, particularly in a database
where enough badges are configured so that very few records
have an average badge score of zero.

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
Signed-off-by: Kathy Lussier <klussier@masslnc.org>
4 years agoLP#1549505: add admin interface to manage badges
Galen Charlton [Wed, 24 Feb 2016 22:19:14 +0000 (17:19 -0500)]
LP#1549505: add admin interface to manage badges

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
Signed-off-by: Kathy Lussier <klussier@masslnc.org>
4 years agoLP#1549505: Add ability to set default sorter via global flag
Mike Rylander [Thu, 4 Feb 2016 22:19:03 +0000 (17:19 -0500)]
LP#1549505: Add ability to set default sorter via global flag

In addition, support for sorting by ascending popularity remains
in the back end, however, since if one's configured badges such
that almost every record has a badge score, that sort order could
be useful for examining the long tail of the collection.

Signed-off-by: Mike Rylander <mrylander@gmail.com>
Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Signed-off-by: Kathy Lussier <klussier@masslnc.org>
4 years agoLP#1549505: Provide a cron-able script to perform badge recalculation
Mike Rylander [Wed, 3 Feb 2016 20:46:52 +0000 (15:46 -0500)]
LP#1549505: Provide a cron-able script to perform badge recalculation

Signed-off-by: Mike Rylander <mrylander@gmail.com>
Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Signed-off-by: Kathy Lussier <klussier@masslnc.org>
4 years agoLP#1549505: Teach QP and its caller stack how to use badges
Mike Rylander [Fri, 22 Jan 2016 22:13:25 +0000 (17:13 -0500)]
LP#1549505: Teach QP and its caller stack how to use badges

Signed-off-by: Mike Rylander <mrylander@gmail.com>
Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Signed-off-by: Kathy Lussier <klussier@masslnc.org>
4 years agoLP#1549505: schema and IDL for statistical poularity ratings
Mike Rylander [Fri, 8 Jan 2016 03:00:20 +0000 (22:00 -0500)]
LP#1549505: schema and IDL for statistical poularity ratings

Signed-off-by: Mike Rylander <mrylander@gmail.com>
Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Signed-off-by: Kathy Lussier <klussier@masslnc.org>
4 years agoLP#1581196: webstaff: fix dirty form detection in patron editor
Galen Charlton [Thu, 12 May 2016 19:58:49 +0000 (15:58 -0400)]
LP#1581196: webstaff: fix dirty form detection in patron editor

This patch fixes a problem wherein the patron editor
form would think that it had been modified by the
user immediately after initialization.

To test
-------
[1] Load the patron editor, then refresh the page. Note that
    you are warned that there is unsaved input.
[2] As above, but hit the save button. Note that after saving,
    you still get a warning that there is unsaved input.
[3] Apply the page, then repeat tests 1 and 2. This time, you
    shouldn't get inappropriate unsaved data warnings.

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Signed-off-by: Bill Erickson <berickxx@gmail.com>
4 years agoLP#1589586: use all fund debits (even encumbrances) when doing prorate calculations
Jeff Davis [Thu, 14 Jul 2016 21:10:13 +0000 (14:10 -0700)]
LP#1589586: use all fund debits (even encumbrances) when doing prorate calculations

Signed-off-by: Jeff Davis <jdavis@sitka.bclibraries.ca>
Signed-off-by: Kathy Lussier <klussier@masslnc.org>
4 years agoDocs: Fix syntax for code block within list
Remington Steed [Wed, 13 Jul 2016 13:37:11 +0000 (09:37 -0400)]
Docs: Fix syntax for code block within list

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

Signed-off-by: Remington Steed <rjs7@calvin.edu>
4 years agoDocs: Small syntax fix, formatting improvements
Remington Steed [Fri, 8 Jul 2016 14:32:05 +0000 (10:32 -0400)]
Docs: Small syntax fix, formatting improvements

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

Signed-off-by: Remington Steed <rjs7@calvin.edu>
4 years agoLP#1568046: Stamping upgrade script for eliminating connectby from functions
Ben Shum [Tue, 5 Jul 2016 05:14:38 +0000 (01:14 -0400)]
LP#1568046: Stamping upgrade script for eliminating connectby from functions

Signed-off-by: Ben Shum <ben@evergreener.net>
4 years agoLP#1568046: Replace last two uses of connectby with other function calls.
Jason Stephenson [Fri, 1 Jul 2016 17:13:16 +0000 (13:13 -0400)]
LP#1568046: Replace last two uses of connectby with other function calls.

Modify permission functions permission.usr_has_perm_at_nd and
permission.usr_has_perm_at_all_nd to no longer require the connectby
function from the tablefunc extension.  The connectby functionality is
replaced by existing database functions to retrieve org. unit
descendants.

This change removes the need to load the tablefunc extension, so it is
dropped from the create scripts.  The upgrade script does not remove
the extension as sites may be using connectby or other functions from
the extension.

Signed-off-by: Jason Stephenson <jason@sigio.com>
Signed-off-by: Ben Shum <ben@evergreener.net>
4 years agoDocs: Add tip for using "temp" as a login_types when using "Change Operator: New"
Yamil Suarez [Sun, 26 Jun 2016 16:52:43 +0000 (12:52 -0400)]
Docs: Add tip for using "temp" as a login_types when using "Change Operator: New"

Signed-off-by: Yamil Suarez <yamil@yamil.com>
4 years ago2.10.4-2.10.5 schema upgrade script
Galen Charlton [Thu, 16 Jun 2016 00:32:10 +0000 (20:32 -0400)]
2.10.4-2.10.5 schema upgrade script

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
4 years agoDocs: Adding 2.10 Release Notes
Kathy Lussier [Wed, 15 Jun 2016 16:45:31 +0000 (12:45 -0400)]
Docs: Adding 2.10 Release Notes

Signed-off-by: Kathy Lussier <klussier@masslnc.org>
Signed-off-by: Galen Charlton <gmc@esilibrary.com>
4 years agoLP#1592891: Fix SIP2 standing penalty failures
Jeff Godin [Wed, 15 Jun 2016 16:25:24 +0000 (12:25 -0400)]
LP#1592891: Fix SIP2 standing penalty failures

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

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

Signed-off-by: Jeff Godin <jgodin@tadl.org>
Signed-off-by: Galen Charlton <gmc@esilibrary.com>
4 years agoLP#1592891: tweak tests
Galen Charlton [Thu, 16 Jun 2016 00:16:52 +0000 (20:16 -0400)]
LP#1592891: tweak tests

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

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
4 years agoLP#1592891: Tests - SIP2 standing penalty failures
Jeff Godin [Wed, 15 Jun 2016 17:46:09 +0000 (13:46 -0400)]
LP#1592891: Tests - SIP2 standing penalty failures

Tests for SIP2 standing penalty failures in LP#1592891

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

Signed-off-by: Jeff Godin <jgodin@tadl.org>
Signed-off-by: Galen Charlton <gmc@esilibrary.com>
4 years agoForward port 2.9.5 to 2.9.6 upgrade script.
Jason Stephenson [Wed, 15 Jun 2016 19:08:53 +0000 (15:08 -0400)]
Forward port 2.9.5 to 2.9.6 upgrade script.

Signed-off-by: Jason Stephenson <jstephenson@mvlcstaff.org>
4 years agoLP#1592565 Log successful authtoken and workstation
Bill Erickson [Tue, 14 Jun 2016 20:16:08 +0000 (16:16 -0400)]
LP#1592565 Log successful authtoken and workstation

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

Signed-off-by: Bill Erickson <berickxx@gmail.com>
Signed-off-by: Jeff Godin <jgodin@tadl.org>
4 years agoLP#1554714: Set angular 1.5.5 as minimum, not exact version
Ben Shum [Wed, 8 Jun 2016 11:41:30 +0000 (07:41 -0400)]
LP#1554714: Set angular 1.5.5 as minimum, not exact version

By adding the ~ in front of the angular version specified, we say
make this the minimum version to look for.  This bypasses the issue
for when new releases of the 1.5.x angular series is released, such
as 1.5.6, which is the current new version as of this patch.

Signed-off-by: Ben Shum <ben@evergreener.net>
Signed-off-by: Bill Erickson <berickxx@gmail.com>
4 years agoLP#1526858: Stamping upgrade script for faster alhr query
Kathy Lussier [Fri, 10 Jun 2016 15:51:41 +0000 (11:51 -0400)]
LP#1526858: Stamping upgrade script for faster alhr query

Signed-off-by: Kathy Lussier <klussier@masslnc.org>
4 years agoLP#1526858: speed up queries for shelf-expired holds during checkin
Galen Charlton [Wed, 16 Dec 2015 16:03:42 +0000 (11:03 -0500)]
LP#1526858: speed up queries for shelf-expired holds during checkin

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

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

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

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Signed-off-by: Jeanette Lundgren <jlundgren@cwmars.org>
Signed-off-by: Kathy Lussier <klussier@masslnc.org>
4 years agoAdds two missing link fields (patron profile and patron home library) to
Codey Kolasinski [Mon, 16 May 2016 13:28:09 +0000 (09:28 -0400)]
Adds two missing link fields (patron profile and patron home library) to
the fm_idl.xml for the Combined Active and Aged Circulations (combcirc)
reporter source. The fix was originally posted in a comment to the
Launchpad bug by Codey Kolasinski.

Signed-off-by: Michele Morgan <mmorgan@noblenet.org>
Signed-off-by: Jeanette Lundgren <jlundgren@cwmars.org>
Signed-off-by: Kathy Lussier <klussier@masslnc.org>
4 years agoLP#1566016: remove SSN as a stock ident type
Galen Charlton [Mon, 4 Apr 2016 21:31:58 +0000 (17:31 -0400)]
LP#1566016: remove SSN as a stock ident type

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

To test:

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

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Signed-off-by: Kathy Lussier <klussier@masslnc.org>
4 years agoLP#1373578: fix translatability of hold placement errors
Galen Charlton [Mon, 23 May 2016 16:29:04 +0000 (12:29 -0400)]
LP#1373578: fix translatability of hold placement errors

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

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

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Signed-off-by: Ben Shum <ben@evergreener.net>
4 years agoLP 1548993: Remember the Show More/Fewer Details button selection.
Jason Stephenson [Thu, 26 May 2016 19:23:03 +0000 (15:23 -0400)]
LP 1548993: Remember the Show More/Fewer Details button selection.

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

Signed-off-by: Jason Stephenson <jstephenson@mvlcstaff.org>
Signed-off-by: Jeanette Lundgren <jlundgren@cwmars.org>
Signed-off-by: Kathy Lussier <klussier@masslnc.org>
4 years agoLP 1548993: Restore Show More/Show Fewer Details button functionality.
Jason Stephenson [Thu, 19 May 2016 19:53:48 +0000 (15:53 -0400)]
LP 1548993: Restore Show More/Show Fewer Details button functionality.

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

Signed-off-by: Jason Stephenson <jstephenson@mvlcstaff.org>
Signed-off-by: Jeanette Lundgren <jlundgren@cwmars.org>
Signed-off-by: Kathy Lussier <klussier@masslnc.org>
4 years ago2.10.3-2.10.4 schema upgrade script
Galen Charlton [Wed, 25 May 2016 21:07:52 +0000 (17:07 -0400)]
2.10.3-2.10.4 schema upgrade script

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
4 years agofix typos in 2.10.4 release notes
Galen Charlton [Wed, 25 May 2016 21:01:41 +0000 (17:01 -0400)]
fix typos in 2.10.4 release notes

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
4 years agoDocs: Adding 2.10.4 Point Release Notes
Kathy Lussier [Wed, 25 May 2016 15:29:22 +0000 (11:29 -0400)]
Docs: Adding 2.10.4 Point Release Notes

Signed-off-by: Kathy Lussier <klussier@masslnc.org>
4 years agoLP#1584801: fix tagging of previous circs in metarecord search results
Galen Charlton [Mon, 23 May 2016 14:29:09 +0000 (10:29 -0400)]
LP#1584801: fix tagging of previous circs in metarecord search results

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

To test:

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

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Signed-off-by: Kathy Lussier <klussier@masslnc.org>
4 years agoLP1567514 - Don't Output Null Byte for Spool Files
Jason Boyer [Thu, 7 Apr 2016 16:08:09 +0000 (12:08 -0400)]
LP1567514 - Don't Output Null Byte for Spool Files

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

To test:

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

Signed-off-by: Jason Boyer <jboyer@library.in.gov>
Signed-off-by: Galen Charlton <gmc@esilibrary.com>
4 years agoLP#1517556 - Exclude inactive event defs from find_event_def_by_hook.
Josh Stompro [Wed, 30 Mar 2016 13:47:57 +0000 (08:47 -0500)]
LP#1517556 - Exclude inactive event defs from find_event_def_by_hook.

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

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

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

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

Signed-off-by: Josh Stompro <stomproj@larl.org>
Signed-off-by: Galen Charlton <gmc@esilibrary.com>
4 years agoLP#1580676: fix error message
Galen Charlton [Tue, 24 May 2016 17:27:05 +0000 (13:27 -0400)]
LP#1580676: fix error message

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

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
4 years agoLP#1580676 MARC stream authority cleanup repair
Bill Erickson [Wed, 11 May 2016 16:19:12 +0000 (12:19 -0400)]
LP#1580676 MARC stream authority cleanup repair

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

Signed-off-by: Bill Erickson <berickxx@gmail.com>
Signed-off-by: Galen Charlton <gmc@esilibrary.com>
4 years agoLP#1447746: Fix lp957466_update_date_and_source regression test
Kathy Lussier [Mon, 23 May 2016 23:17:36 +0000 (19:17 -0400)]
LP#1447746: Fix lp957466_update_date_and_source regression test

The new code introduced through LP1447746 adding an update_bib_source flag to
Vandelay merge profiles broke a test that was verifying the bib source and
edit fields were updated in Vandelay uploads.

This patch fixes the test and also adds a new test to ensure the flag works as
expected when set to both true and false. It also renames the test since the
it's now testing the newer code rather than the previous expected behavior.

Signed-off-by: Kathy Lussier <klussier@masslnc.org>
Signed-off-by: Ben Shum <ben@evergreener.net>
4 years agoLP#1554714 Add explicit phantomjs-prebuilt dependency
Bill Erickson [Thu, 19 May 2016 18:28:11 +0000 (14:28 -0400)]
LP#1554714 Add explicit phantomjs-prebuilt dependency

Signed-off-by: Bill Erickson <berickxx@gmail.com>
Signed-off-by: Ben Shum <ben@evergreener.net>
Signed-off-by: Jason Stephenson <jstephenson@mvlcstaff.org>
4 years agoLP#1554714 Ang. 2.5 hotkeys plugin update
Bill Erickson [Thu, 19 May 2016 20:25:12 +0000 (16:25 -0400)]
LP#1554714 Ang. 2.5 hotkeys plugin update

As with other dependencies, the hotkeys package name changed since the
last update.  It's now cfp-angular-hotkeys

Signed-off-by: Bill Erickson <berickxx@gmail.com>
Signed-off-by: Ben Shum <ben@evergreener.net>
Signed-off-by: Jason Stephenson <jstephenson@mvlcstaff.org>
4 years agoLP 1499123: 0951.schema.config.standing_penalty.ignore_proximity.sql.
Jason Stephenson [Thu, 19 May 2016 14:33:18 +0000 (10:33 -0400)]
LP 1499123: 0951.schema.config.standing_penalty.ignore_proximity.sql.

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

Signed-off-by: Jason Stephenson <jstephenson@mvlcstaff.org>
Signed-off-by: Kathy Lussier <klussier@masslnc.org>
4 years agoForward port 2.9.4 to 2.9.5 db upgrade script.
Jason Stephenson [Wed, 18 May 2016 18:36:15 +0000 (14:36 -0400)]
Forward port 2.9.4 to 2.9.5 db upgrade script.

Signed-off-by: Jason Stephenson <jstephenson@mvlcstaff.org>
4 years agoDocs: Adding 2.9.5 Release Notes
Kathy Lussier [Wed, 18 May 2016 13:55:55 +0000 (09:55 -0400)]
Docs: Adding 2.9.5 Release Notes

Signed-off-by: Kathy Lussier <klussier@masslnc.org>
4 years agoLP1497374 fm_IDL.xml metabib::record_attr_flat contains an incorrect link
blake [Fri, 18 Sep 2015 17:24:28 +0000 (12:24 -0500)]
LP1497374 fm_IDL.xml metabib::record_attr_flat contains an incorrect link

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

Signed-off-by: blake <blake@mobiusconsortium.org>
Signed-off-by: Michele Morgan <mmorgan@noblenet.org>
Signed-off-by: Kathy Lussier <klussier@masslnc.org>
4 years agoLP#1447746 Stamping upgrade script for update-bib-source flag in Vandelay
Kathy Lussier [Thu, 12 May 2016 20:49:11 +0000 (16:49 -0400)]
LP#1447746 Stamping upgrade script for update-bib-source flag in Vandelay

Signed-off-by: Kathy Lussier <klussier@masslnc.org>
4 years agoLP#1447746: Do not update bib source on match-only merge
Dan Pearl [Thu, 4 Feb 2016 17:46:46 +0000 (12:46 -0500)]
LP#1447746: Do not update bib source on match-only merge

This adds a flag to control the updating of the bib source on a vandelay operation.

Signed-off-by: Dan Pearl <dpearl@cwmars.org>
Signed-off-by: Kathy Lussier <klussier@masslnc.org>
4 years agoLP#1578716 Fix Responsive Items Out
Dan Wells [Thu, 5 May 2016 15:58:52 +0000 (11:58 -0400)]
LP#1578716 Fix Responsive Items Out

With "Title" and "Author" as separate columns, here is a quick fix
to get the responsive labels to line up again.

Signed-off-by: Dan Wells <dbw2@calvin.edu>
Signed-off-by: Ben Shum <ben@evergreener.net>
4 years agorelease notes for 2.10.3
Galen Charlton [Mon, 9 May 2016 21:03:20 +0000 (17:03 -0400)]
release notes for 2.10.3

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
4 years ago2.10.2 to 2.10.3 schema upgrade script
Galen Charlton [Mon, 9 May 2016 21:08:40 +0000 (17:08 -0400)]
2.10.2 to 2.10.3 schema upgrade script

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
4 years agoLP#1579225: fix handling of passwords in patron registration
Galen Charlton [Sat, 7 May 2016 01:40:12 +0000 (21:40 -0400)]
LP#1579225: fix handling of passwords in patron registration

This patch improves how the new password hashing is invoked
by open-ils.actor.patron.update; in particular, it fixes
a problem whereby newly registered patrons could not
log in.  It also fixes other issues:

- actor.usr.passwd would be set to an MD5 of the password
  for new users, vitiating the strong hashes in actor.passwd
- certain types of updates via patron registration, such as
  adding or deleting addresses, could result in the patron's
  password getting doubly-hashed, thereby locking them out
  of their account.

To test
-------
[1] Register a new patron; verify that they can log in.
[2] Edit an existing patron and change their password; verify
    that they can log in.
[3] Edit an existing patron but do NOT change their password;
    verify that they can still log in.
[4] Inspect the actor.usr rows for these patrons and verify
    that actor.usr.passwd is set to the value MD5(''), not
    the MD5 of their password.

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Signed-off-by: Dan Wells <dbw2@calvin.edu>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
Signed-off-by: Kathy Lussier <klussier@masslnc.org>
Signed-off-by: Galen Charlton <gmc@esilibrary.com>
4 years agoLP#1579225: add live_t regression test
Galen Charlton [Mon, 9 May 2016 20:40:52 +0000 (16:40 -0400)]
LP#1579225: add live_t regression test

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
4 years agoLP#1551084: Add 16.04 version notes in server_upgrade doc
Ben Shum [Wed, 4 May 2016 17:22:28 +0000 (13:22 -0400)]
LP#1551084: Add 16.04 version notes in server_upgrade doc

Signed-off-by: Ben Shum <ben@evergreener.net>
Signed-off-by: Jason Stephenson <jason@sigio.com>
4 years agoLP#1551084: Fixup typo for ubuntu-xenial-developer
Ben Shum [Tue, 19 Apr 2016 14:36:39 +0000 (10:36 -0400)]
LP#1551084: Fixup typo for ubuntu-xenial-developer

Jason spotted a typo. Can probably squash this in later.

Signed-off-by: Ben Shum <ben@evergreener.net>
Signed-off-by: Jason Stephenson <jason@sigio.com>
4 years agoLP#1551084: Update README to include references for 'ubuntu-xenial'
Ben Shum [Mon, 29 Feb 2016 06:30:29 +0000 (01:30 -0500)]
LP#1551084: Update README to include references for 'ubuntu-xenial'

Signed-off-by: Ben Shum <ben@evergreener.net>
Signed-off-by: Jason Stephenson <jason@sigio.com>
4 years agoLP#1551084: Start new Makefile target for 'ubuntu-xenial'
Ben Shum [Mon, 29 Feb 2016 06:20:34 +0000 (01:20 -0500)]
LP#1551084: Start new Makefile target for 'ubuntu-xenial'

For Ubuntu 16.04, let's start a new Makefile target for 'ubuntu-xenial'

For major changes, the packaged version of PostgreSQL that ships with
Ubuntu 16.04 is PostgreSQL 9.5.

Also, during the configure step, the build process looks for libpcre3-dev,
so we add that as an additional prerequisite now.

Signed-off-by: Ben Shum <ben@evergreener.net>
Signed-off-by: Jason Stephenson <jason@sigio.com>
4 years agoLP#1551084: Symlink for 000.english.pg95.fts-config.sql
Ben Shum [Mon, 29 Feb 2016 06:14:38 +0000 (01:14 -0500)]
LP#1551084: Symlink for 000.english.pg95.fts-config.sql

We need this so that the eg_db_config script can make the Evergreen
database with PostgreSQL 9.5 systems.

Signed-off-by: Ben Shum <ben@evergreener.net>
Signed-off-by: Jason Stephenson <jason@sigio.com>
4 years agoLP#1568046: Stamping upgrade script for changes to permission.grp_ancestors
Ben Shum [Wed, 4 May 2016 16:59:01 +0000 (12:59 -0400)]
LP#1568046: Stamping upgrade script for changes to permission.grp_ancestors

Signed-off-by: Ben Shum <ben@evergreener.net>
4 years agoLP#1568046: Fix some mistakes in previous commit.
Jason Stephenson [Fri, 8 Apr 2016 19:30:32 +0000 (15:30 -0400)]
LP#1568046: Fix some mistakes in previous commit.

Looks like a few thinkos crept in from it being dashed off in a hurry.

Signed-off-by: Jason Stephenson <jstephenson@mvlcstaff.org>
Signed-off-by: Ben Shum <ben@evergreener.net>
4 years agoLP#1568046: Use CTE instead of connectby for group tree queries
Mike Rylander [Fri, 8 Apr 2016 18:49:10 +0000 (14:49 -0400)]
LP#1568046: Use CTE instead of connectby for group tree queries

Signed-off-by: Mike Rylander <mrylander@gmail.com>
Signed-off-by: Jason Stephenson <jstephenson@mvlcstaff.org>
Signed-off-by: Ben Shum <ben@evergreener.net>
4 years agoLP#1031335 No-op the escape_email_header helper
Dan Scott [Wed, 7 Jan 2015 23:22:22 +0000 (18:22 -0500)]
LP#1031335 No-op the escape_email_header helper

As suggested by Mike Rylander, keep the escape_email_header helper
around to avoid breaking templates that already adopted it, but make it
a no-op as the SendEmail reactor now encodes the email headers of
interest by default.

Signed-off-by: Dan Scott <dscott@laurentian.ca>
Signed-off-by: Dan Wells <dbw2@calvin.edu>
4 years agoFix LP#1031335 by escaping some email headers automatically
Pasi Kallinen [Wed, 24 Jul 2013 06:28:13 +0000 (09:28 +0300)]
Fix LP#1031335 by escaping some email headers automatically

Signed-off-by: Pasi Kallinen <pasi.kallinen@pttk.fi>
Signed-off-by: Dan Scott <dscott@laurentian.ca>
Signed-off-by: Dan Wells <dbw2@calvin.edu>
4 years agoAdd entries to .mailmap.
Jason Stephenson [Fri, 29 Apr 2016 11:48:44 +0000 (07:48 -0400)]
Add entries to .mailmap.

Signed-off-by: Jason Stephenson <jstephenson@mvlcstaff.org>
4 years agoForward port 2.9.3 to 2.9.4 db upgrade script.
Jason Stephenson [Thu, 28 Apr 2016 23:32:47 +0000 (19:32 -0400)]
Forward port 2.9.3 to 2.9.4 db upgrade script.

Signed-off-by: Jason Stephenson <jstephenson@mvlcstaff.org>
4 years agoAdding 2.8.7 -> 2.8.8 DB upgrade script
Bill Erickson [Thu, 28 Apr 2016 19:51:47 +0000 (15:51 -0400)]
Adding 2.8.7 -> 2.8.8 DB upgrade script

Signed-off-by: Bill Erickson <berickxx@gmail.com>
4 years ago2.10.1 to 2.10.2 schema upgrade script
Galen Charlton [Thu, 28 Apr 2016 17:15:42 +0000 (13:15 -0400)]
2.10.1 to 2.10.2 schema upgrade script

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
4 years agoupdates to the 2.10.2 release notes
Galen Charlton [Thu, 28 Apr 2016 16:26:19 +0000 (12:26 -0400)]
updates to the 2.10.2 release notes

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Signed-off-by: Bill Erickson <berickxx@gmail.com>
4 years agoAdding 2.10.2 point release notes
Kathy Lussier [Thu, 28 Apr 2016 13:27:16 +0000 (09:27 -0400)]
Adding 2.10.2 point release notes

Signed-off-by: Kathy Lussier <klussier@masslnc.org>
Signed-off-by: Galen Charlton <gmc@esilibrary.com>
4 years agoupdate release notes for 2.9.4
Galen Charlton [Thu, 28 Apr 2016 16:21:50 +0000 (12:21 -0400)]
update release notes for 2.9.4

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Signed-off-by: Bill Erickson <berickxx@gmail.com>
4 years agoAdding 2.9.4 release notes
Kathy Lussier [Thu, 28 Apr 2016 12:47:05 +0000 (08:47 -0400)]
Adding 2.9.4 release notes

Signed-off-by: Kathy Lussier <klussier@masslnc.org>
Signed-off-by: Galen Charlton <gmc@esilibrary.com>
4 years agoupdates to 2.8.8 release notes
Galen Charlton [Thu, 28 Apr 2016 16:20:45 +0000 (12:20 -0400)]
updates to 2.8.8 release notes

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Signed-off-by: Bill Erickson <berickxx@gmail.com>
4 years agoPoint release notes for the 2.8.8 release
Kathy Lussier [Thu, 28 Apr 2016 12:35:43 +0000 (08:35 -0400)]
Point release notes for the 2.8.8 release

Signed-off-by: Kathy Lussier <klussier@masslnc.org>
Signed-off-by: Galen Charlton <gmc@esilibrary.com>
4 years agoLP#1562153: fix case where changing sort order to relevance can fail
Galen Charlton [Fri, 25 Mar 2016 20:46:24 +0000 (16:46 -0400)]
LP#1562153: fix case where changing sort order to relevance can fail

This patch fixes a cause where sending a 'sort' CGI parameter
with the empty string as value (which specifies relevance sorting)
fails to override a previously-set sort order.

To test:

[1] Do a public catalog search starting from the advanced search form,
    e.g., keyword = "cats"
[2] Change the sort order to (say) Title A-Z from the results page.
[3] Note that the query string changes to "sort(titlesort) cats" and
    that the results are re-sorted
[4] Attempt to change the sort order to relevance.
[5] This time, the sort order does not change ... and it should have.
[6] Apply the patch and repeat steps 1-5. This time, the sort order
    should be successfully changed.

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
4 years agoLP#1562153: add regression test
Galen Charlton [Fri, 25 Mar 2016 20:44:15 +0000 (16:44 -0400)]
LP#1562153: add regression test

This patch adds a regression test for successfully
overrding the sort order in a query from a 'sort' CGI
parameter.

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
4 years agoLP#1563531: add libpcre3/libpcre3-dev deps for Jessie
Galen Charlton [Wed, 30 Mar 2016 17:29:10 +0000 (13:29 -0400)]
LP#1563531: add libpcre3/libpcre3-dev deps for Jessie

This patch adds libpcre3 and libpcre3-dev to the list
of packages installed when installing Evergreen on Debian
Jessie. Evergreen's direct dependency on libpcre3 is new in 2.10
(see bug 1468422); it happens that libpcre3 and libpcre3-dev
are installed on Wheezy by virtue of a different dependency
chain.

To test
-------
[1] On a fresh Debian Jessie netinstall, follow the OpenSRF
    and Evergreen installation instructions. Verify that with
    this patch in place, Evergreen's "configure" step succeeds.

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
4 years agoLP#1568195: Stamping upgrade script
Mike Rylander [Thu, 28 Apr 2016 15:54:28 +0000 (11:54 -0400)]
LP#1568195: Stamping upgrade script

Signed-off-by: Mike Rylander <mrylander@gmail.com>
4 years agoLP#1568195: fix retrieving big OUS batches
Galen Charlton [Sat, 9 Apr 2016 01:46:35 +0000 (21:46 -0400)]
LP#1568195: fix retrieving big OUS batches

This patch fixes a regression introduced in LP#1501471
where the Library Settings Editor could fail to retrieve
the values of org unit settings if more than 99 were
requested at a time.

To test
-------
[1] Open the XUL library settings editor and ensure that
    no search filters are in effect. Note that values
    are not displayed for any of the OU settings, and that
    the Pg log contains error messages like this:

    "ERROR: cannot pass more than 100 arguments to a function"

[2] Apply the patch.
[3] Repeat step one, and verify that values are now retrieved
    for all of the OU settings that have values set.
[4] Verify that the pgTAP tests in live_t/aous_batch.pg pass.

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
4 years agoLP#1569884: stamp schema update
Galen Charlton [Thu, 28 Apr 2016 15:31:20 +0000 (11:31 -0400)]
LP#1569884: stamp schema update

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
4 years agoLP#1569884: fix schema update script
Galen Charlton [Thu, 28 Apr 2016 15:27:11 +0000 (11:27 -0400)]
LP#1569884: fix schema update script

CREATE INDEX cannot run concurrently in a transaction
block period, at least as of 9.4.

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
4 years agoLP#1569884 Fix two problems in previous commit.
Jason Stephenson [Wed, 27 Apr 2016 19:43:31 +0000 (15:43 -0400)]
LP#1569884 Fix two problems in previous commit.

The CREATE TABLE biblio.monograph_part statement in
010.schema.biblio.sql ended up with a trailing comma.

The CREATE UNIQUE INDEX on biblio.monograph_part cannot be done
CONCURRENTLY at DB creation time, apparently.  At least not on
PostgreSQL 9.3.

Signed-off-by: Jason Stephenson <jstephenson@mvlcstaff.org>
Signed-off-by: Galen Charlton <gmc@esilibrary.com>
4 years agoLP#1569884 Updated base schema with monograph parts index.
Steven Callender [Wed, 27 Apr 2016 16:05:48 +0000 (12:05 -0400)]
LP#1569884 Updated base schema with monograph parts index.

Signed-off-by: Steven Callender <stevecallender@esilibrary.com>
Signed-off-by: Jason Stephenson <jstephenson@mvlcstaff.org>
Signed-off-by: Galen Charlton <gmc@esilibrary.com>
4 years agoLP#1569884 Allow deleted monograph parts to be recreated.
Steven Callender [Wed, 13 Apr 2016 15:15:27 +0000 (11:15 -0400)]
LP#1569884 Allow deleted monograph parts to be recreated.

Changed the constraint on the biblio.monograph_part table to allow for a
monograph part to be created if a duplicate deleted version is found.

To test:

[1] Add a monograph part to a bib.
[2] Delete that monograph part.
[3] Attempt to add a monograph part to the same bib using
    the same part label. Without the patches installed, this
    will fail; upon applying the database update, this operation
    should succeed.

Signed-off-by: Steven Callender <stevecallender@esilibrary.com>
Signed-off-by: Jason Stephenson <jstephenson@mvlcstaff.org>
Signed-off-by: Galen Charlton <gmc@esilibrary.com>
4 years agoLP#1569884: add regression test
Galen Charlton [Thu, 28 Apr 2016 15:20:29 +0000 (11:20 -0400)]
LP#1569884: add regression test

This pgTAP test verifies that if you logically delete a monograph part,
you can subsequently add a new one with the same label as that of the
deleted one.

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
4 years agoLP#1564079 Stamping upgrade script
Kathy Lussier [Wed, 27 Apr 2016 14:35:49 +0000 (10:35 -0400)]
LP#1564079 Stamping upgrade script

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