working/Evergreen.git
4 years agoLP#1074096: Remove Bib Call Number from query type selector
Jeff Davis [Thu, 16 Apr 2015 15:58:17 +0000 (08:58 -0700)]
LP#1074096: Remove Bib Call Number from query type selector

Signed-off-by: Jeff Davis <jdavis@sitka.bclibraries.ca>
Signed-off-by: Michael Peters <mpeters@emeralddata.net>
Signed-off-by: Ben Shum <bshum@biblio.org>
4 years agoLP#1452883: prevent staff client warnings w/ Google Analytics
Jason Etheridge [Thu, 7 May 2015 20:02:00 +0000 (16:02 -0400)]
LP#1452883: prevent staff client warnings w/ Google Analytics

In particular, the Security Warning that goes "You have requested an encrypted
page that contains some unencrypted information..."

Signed-off-by: Jason Etheridge <jason@esilibrary.com>
Signed-off-by: Josh Stompro <stomproj@larl.org>
Signed-off-by: Ben Shum <bshum@biblio.org>
4 years agoLP#1403966: Remove publication-specific information from metarecord results page
Kathy Lussier [Thu, 28 May 2015 02:11:49 +0000 (22:11 -0400)]
LP#1403966: Remove publication-specific information from metarecord results page

The records that make up a matarecord search result typically contain different
publication information. Therefore, we should remove the master record's
specific publication information from that search results page.

Test plan
Prior to loading the patch, conduct a search on the Concerto dataset
for "brahms piano concerto no. 1" with "Group Formats and Editions" turned on.
In the brief view, you will see a call number and, in the more details view, you
will see publisher and physical description information. Run the same search
after loading the patch, and you will no longer see that information.

Signed-off-by: Kathy Lussier <klussier@masslnc.org>
Signed-off-by: Andrea Neiman <aneiman@kent.lib.md.us>
Signed-off-by: Ben Shum <bshum@biblio.org>
4 years agoLP#1194860: You have permission to override some of the failed holds. appearing when...
Blake Henderson [Mon, 8 Dec 2014 21:52:15 +0000 (15:52 -0600)]
LP#1194860: You have permission to override some of the failed holds. appearing when it should not for patrons in the OPAC Edit

A quick value check before assigning the override variable.
This should eliminate the OPAC showing the override option
for those who do not have the permission.

Signed-off-by: Blake Henderson <blake@mobiusconsortium.org>
Signed-off-by: Liam Whalen <liam.whalen@bc.libraries.coop>
Signed-off-by: Ben Shum <bshum@biblio.org>
4 years agoLP#1456021 Enable apache modcgi during install for Debian Jessie
Josh Stompro [Sun, 31 May 2015 22:49:34 +0000 (17:49 -0500)]
LP#1456021 Enable apache modcgi during install for Debian Jessie

Apache mod cgi isn't enabled when installing apache2.4 on Debian Jessie,
this causes the /updates/manualupdate.html to not be processed and
to just return the raw code of that script since it uses mod cgi.

This adds 'cgi' to the list of apache modules to enable during the
dependency install.

Signed-off-by: Josh Stompro <stomproj@larl.org>
Signed-off-by: Ben Shum <bshum@biblio.org>
4 years agoLP#1457094 Switch to package libtemplate-plugin-posix-perl for Debian Jessie
Josh Stompro [Sun, 31 May 2015 22:19:58 +0000 (17:19 -0500)]
LP#1457094 Switch to package libtemplate-plugin-posix-perl for Debian Jessie

Debian Jessie includes a package for Template::Plugin::Posix that was
being installed via CPAN.  This patch removes the CPAN install and adds
the Debian package.

Signed-off-by: Josh Stompro <stomproj@larl.org>
Signed-off-by: Ben Shum <bshum@biblio.org>
4 years agoLP#757533: adds scrollbars (when necessary) to the item status alternate view tab...
Bradley Bonner [Wed, 10 Jun 2015 21:26:57 +0000 (14:26 -0700)]
LP#757533: adds scrollbars (when necessary) to the item status alternate view tab box.

Signed-off-by: Bradley Bonner <bbonner@kcls.org>
Signed-off-by: Ben Shum <bshum@biblio.org>
4 years agoLP#1289486 Stamping upgrade for dropping acq audit fkey
Dan Wells [Wed, 10 Jun 2015 19:05:13 +0000 (15:05 -0400)]
LP#1289486 Stamping upgrade for dropping acq audit fkey

Signed-off-by: Dan Wells <dbw2@calvin.edu>
4 years agoLP#1289486: Remove blocking constraint from audit table
Mike Rylander [Mon, 8 Jun 2015 21:30:52 +0000 (17:30 -0400)]
LP#1289486: Remove blocking constraint from audit table

The ACQ lineitem audit table was augmented with an fkey pointing
to the queued record that was related to the line item.  That's
bad because there are also triggers which attempt to manage that
table.  Hilarity ensues.  We should not have an fkey on an audit
table, so, we now remove that.

In order to test this, first attempt to delete an ACQ queue and
have that fail.  Apply the DB change, attempt again, and have it
succeed.

Signed-off-by: Mike Rylander <mrylander@gmail.com>
Signed-off-by: Dan Wells <dbw2@calvin.edu>
4 years agoLP#1447168 Stamping upgrade script for index on ash record
Dan Wells [Wed, 10 Jun 2015 18:29:27 +0000 (14:29 -0400)]
LP#1447168 Stamping upgrade script for index on ash record

Signed-off-by: Dan Wells <dbw2@calvin.edu>
4 years agoLP#1447168: add an index on authority.simple_heading.record
Galen Charlton [Wed, 22 Apr 2015 13:43:19 +0000 (13:43 +0000)]
LP#1447168: add an index on authority.simple_heading.record

This index is needed so that full table scans on ash aren't
needed during authority record reingest.

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Signed-off-by: Dan Wells <dbw2@calvin.edu>
4 years agoLP#1449709: Always get a real hostname for the cache key
Mike Rylander [Fri, 22 May 2015 17:11:18 +0000 (13:11 -0400)]
LP#1449709: Always get a real hostname for the cache key

EGCatLoader overwrites $ctx->{hostname} when serving the staff client
version of TT-generated content.  This is bad for us because we use
that as part of the TT Processor cache key.  Instead, we'll just
always ask Apache, via $r.

Signed-off-by: Mike Rylander <mrylander@gmail.com>
Signed-off-by: Bill Erickson <berickxx@gmail.com>
4 years agoLP#1452366: allow EGWeb context loaders to have child_init actions
Galen Charlton [Tue, 5 May 2015 22:15:39 +0000 (22:15 +0000)]
LP#1452366: allow EGWeb context loaders to have child_init actions

This patch adds the ability for EGWeb to call
an initialization function for context loaders
during the child_init phase of Apache backend
startup.

In particular, portions of the R/O object cache
for EGCatLoader are now initialized when a backend
starts up; testing indicates that this can shave
a couple seconds off the time it takes for a
backend to render a bib details page the first time.

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
Signed-off-by: Bill Erickson <berickxx@gmail.com>
4 years agoLP#1452352: use fresh CStore editors when populating R/O cache
Galen Charlton [Wed, 6 May 2015 18:23:49 +0000 (18:23 +0000)]
LP#1452352: use fresh CStore editors when populating R/O cache

This change ensures that there's no longer a cached
editor with its original session locale grabbing
the wrong labels when a template invokes ctx.search_foo.

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
Signed-off-by: Bill Erickson <berickxx@gmail.com>
4 years agoLP#1452352: don't leak $ctx when initializing TPAC R/O object cache
Mike Rylander [Wed, 6 May 2015 15:10:44 +0000 (11:10 -0400)]
LP#1452352: don't leak $ctx when initializing TPAC R/O object cache

Create a closure over a simple scalar instead of the
whole $ctx object when all we need is the locale.

Signed-off-by: Mike Rylander <mrylander@gmail.com>
Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Signed-off-by: Bill Erickson <berickxx@gmail.com>
4 years agoLP#1452352: fix generation of locale-specific RO object fetchers
Galen Charlton [Wed, 6 May 2015 13:50:32 +0000 (13:50 +0000)]
LP#1452352: fix generation of locale-specific RO object fetchers

The TPAC R/O object cache is now keyed by locale.  This
fixes an issue where an OPAC session that starts in one
locale and gets switched to another could see (e.g.)
format labels for the original locale.

The issue fixed by this patch can be most readily reproduced
as follows:

[1] Enable both the en-US and fr-CA locales. Make sure
    that there are some translations for record format
    coded values in place as well.
[2] Run Apache in single-process mode
    (e.g., ". /etc/apache2/envvars && apache2 -X")
[3] Visit a record details page in the en-US locale. Note
    the label associated with the record format.
[4] Change the locale to fr-CA. Note that the label is
    still the English version.

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Signed-off-by: Bill Erickson <berickxx@gmail.com>
4 years agoLP#1449709: add release notes
Galen Charlton [Tue, 5 May 2015 16:43:18 +0000 (16:43 +0000)]
LP#1449709: add release notes

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Signed-off-by: Bill Erickson <berickxx@gmail.com>
4 years agoLP#1449709: caching compiled Template Toolkit templates
Galen Charlton [Tue, 28 Apr 2015 19:06:48 +0000 (19:06 +0000)]
LP#1449709: caching compiled Template Toolkit templates

This patch enables caching of compiled Template Toolkit
files on Evergreen web servers, which can provide a modest
improvement in the time it takes (say) TPAC to render
a page, particularly by a fresh Apache backend.

This is controlled by a new Apache virtualhost variable,
OILSWebCompiledTemplateCache, which can be set to a
directory on the webserver for storing compiled templates. This
is enabled by default for new installations.

This patch also adds OILSWebTemplateStatTTL, which can be
use to tweak the STAT_TTL Template Toolkit setting. Note this
bit works because of the caching of TT handlers added by
the previous patch.

Finally, this patch also fixes a bug where attempting to
disable OILSWebDebugTemplate would result in internal server errors;
it also sets the default value of this setting to false.

Note: the caching added by this patch and the previous one
mean that if a change to (say) TPAC templates is made on
the file system, it is now required to reload Apache to have
a guarantee that the change is visible, although one can also
wait the OILSWebTemplateStatTTL interval.

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Signed-off-by: Bill Erickson <berickxx@gmail.com>
4 years agoLP#1449709: Persist template processor for speed
Mike Rylander [Mon, 4 May 2015 21:06:54 +0000 (17:06 -0400)]
LP#1449709: Persist template processor for speed

By persisting the Template Toolkit processor object per
vhost/locale/context-loader/xml-strictness instead
of creating a new one for every request, we allow TT
to maintain an in-memory cache of all the templates it
has compiled.  This increases speed by as much as 80%.

In addition, we reduce the number of times the TPAC checks
for the existance of a template by remembering the state
of the first readability probe.  This is most important
on systems where the templates are stored on NFS, where
stat(2) system calls can be particularly expensive.

We now pass the Apache object ($r) to the template processor
instead of relying on tied-STDOUT behaviour, because each
request gets a new tied file handle.  After the first one,
it's the wrong one.

Signed-off-by: Mike Rylander <mrylander@gmail.com>
Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Signed-off-by: Bill Erickson <berickxx@gmail.com>
4 years agoDocs: Replace broken link, go to new LSE section
Remington Steed [Wed, 10 Jun 2015 16:05:04 +0000 (12:05 -0400)]
Docs: Replace broken link, go to new LSE section

This commit fixes a newly broken link, caused by removing the old
Library Settings Editor docs section. The new link goes to the new LSE
section.

Signed-off-by: Remington Steed <rjs7@calvin.edu>
4 years agoDocs: Fix spelling errors in Library Settings docs
Remington Steed [Tue, 9 Jun 2015 20:12:36 +0000 (16:12 -0400)]
Docs: Fix spelling errors in Library Settings docs

This commit fixes some spelling errors only in the docs, not in the text
of the actual settings.

Signed-off-by: Remington Steed <rjs7@calvin.edu>
4 years agoDocs: New chapter for Library Settings Editor
Lynn Floyd [Tue, 9 Jun 2015 15:44:16 +0000 (11:44 -0400)]
Docs: New chapter for Library Settings Editor

This commit removes the old piecemeal documentation and adds a more
complete chapter on using the Library Settings Editor, including
categorized tables of the various library settings.

Thanks to Lynn Floyd for the documentation and screenshots.

Signed-off-by: Remington Steed <rjs7@calvin.edu>
4 years agoDocs: General improvements to EDI docs
Jane Sandberg [Tue, 19 May 2015 21:13:05 +0000 (17:13 -0400)]
Docs: General improvements to EDI docs

Signed-off-by: Remington Steed <rjs7@calvin.edu>
4 years agoDocs: Added hard due dates docs
Jane Sandberg [Sun, 17 May 2015 16:32:21 +0000 (12:32 -0400)]
Docs: Added hard due dates docs

Signed-off-by: Yamil Suarez <yamil@yamil.com>
4 years agoDocs: Added sub-section for permalink to 'using the catalog' section
Josh Stompro [Sun, 17 May 2015 01:20:50 +0000 (21:20 -0400)]
Docs: Added sub-section for permalink to 'using the catalog' section

Signed Off Josh Stompro (github@stompro.org)
Signed Off Amy Terlaga (terlaga@biblio.org)
Signed-off-by: Yamil Suarez <yamil@yamil.com>
4 years agoDocs 2.8: SMS Call Number new screenshots, improve wording
Christine Burns [Wed, 13 May 2015 23:36:34 +0000 (19:36 -0400)]
Docs 2.8: SMS Call Number new screenshots, improve wording

This commit includes updated screenshots and some minor wording edits to
the SMS Call Number / Copy Details feature.

New screenshots and edits provided by
* Christine Burns <christine.burns@bc.libraries.coop>
* Carol Yarrison

Signed-off-by: Remington Steed <rjs7@calvin.edu>
Signed-off-by: Yamil Suarez <yamil@yamil.com>
4 years agoLP#1125270: Add a DROP FUNCTION for actor.usr_purge_data
Ben Shum [Tue, 3 Mar 2015 02:49:55 +0000 (21:49 -0500)]
LP#1125270: Add a DROP FUNCTION for actor.usr_purge_data

Apparently this caused problems on upgrading from 2.0 to 2.1 back in the day,
and adding a DROP FUNCTION ahead of the CREATE dealt with the situation.

Signed-off-by: Ben Shum <bshum@biblio.org>
Signed-off-by: Galen Charlton <gmc@esilibrary.com>
4 years agoLP1406317: Stray Semicolons in OPAC Browse
Terran McCanna [Fri, 15 May 2015 17:02:17 +0000 (10:02 -0700)]
LP1406317: Stray Semicolons in OPAC Browse

Fixed display issue where semicolons were appearing at the end of
each result in browse title screen. Semicolon had been placed outside
of a bracket rather than inside a bracket. Moved semicolon to inside
bracket to correct.

URL of page showing error:
https://my-library/eg/opac/browse?blimit=10&qtype=title&bterm=m&locg=1

Signed-off-by: Terran McCanna <tmccanna@georgialibraries.org>
Signed-off-by: Galen Charlton <gmc@esilibrary.com>
4 years agoDocs: Active date display in OPAC
Lynn Floyd [Wed, 13 May 2015 23:01:05 +0000 (19:01 -0400)]
Docs: Active date display in OPAC

Additional thanks to Debbie Luchenbill <debbie@mobiusconsortium.org>

Signed-off-by: Yamil Suarez <yamil@yamil.com>
4 years agoDocs: Update mention of Schema.org use to include OPAC bib record views
Debbie Luchenbill [Wed, 13 May 2015 22:41:16 +0000 (18:41 -0400)]
Docs: Update mention of Schema.org use to include OPAC bib record views

Signed-off-by: Yamil Suarez <yamil@yamil.com>
4 years agoDocs: config.tt2 option for "Show more details"
Yamil Suarez [Wed, 13 May 2015 00:48:13 +0000 (20:48 -0400)]
Docs: config.tt2 option for "Show more details"

Signed-off-by: Yamil Suarez <github@yamil.com>
4 years agoDocs: Removal of Bib Call Number Search
Yamil Suarez [Wed, 13 May 2015 01:27:28 +0000 (21:27 -0400)]
Docs: Removal of Bib Call Number Search

Signed-off-by: Yamil Suarez <github@yamil.com>
4 years agoLP#800706: Stamping upgrade script
Jason Stephenson [Wed, 13 May 2015 17:51:35 +0000 (13:51 -0400)]
LP#800706: Stamping upgrade script

Signed-off-by: Jason Stephenson <jstephenson@mvlc.org>
4 years agoLP#800706: add regression test
Galen Charlton [Tue, 12 May 2015 20:02:09 +0000 (16:02 -0400)]
LP#800706: add regression test

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Signed-off-by: Jason Stephenson <jstephenson@mvlc.org>
4 years agoLP#800706: add index to serial.record_entry.record
Galen Charlton [Tue, 12 May 2015 19:27:07 +0000 (15:27 -0400)]
LP#800706: add index to serial.record_entry.record

This patch adds index on the record column of serial.record_entry,
which can improve performance when displaying record details pages
in databases that have a large number MFHDs.

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Signed-off-by: Jason Stephenson <jstephenson@mvlc.org>
4 years agoLP#1445150: Symlink for 000.english.pg94.fts-config.sql
Ben Shum [Sat, 9 May 2015 02:10:19 +0000 (22:10 -0400)]
LP#1445150: Symlink for 000.english.pg94.fts-config.sql

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

Signed-off-by: Ben Shum <bshum@biblio.org>
4 years agoLP#1445182 Changed Debian Jessie dependency install to use packages for dbi dbd-pgsql.
Josh Stompro [Thu, 30 Apr 2015 14:37:50 +0000 (09:37 -0500)]
LP#1445182 Changed Debian Jessie dependency install to use packages for dbi dbd-pgsql.

Signed-off-by: Josh Stompro <stomproj@larl.org>
Signed-off-by: Ben Shum <bshum@biblio.org>
Conflicts:
Open-ILS/src/extras/install/Makefile.debian-jessie

4 years agoLP#1362260 - Email::Sender/libemail-send-perl change in Jessie
Josh Stompro [Thu, 16 Apr 2015 16:21:13 +0000 (11:21 -0500)]
LP#1362260 - Email::Sender/libemail-send-perl change in Jessie

libemail-send-perl no longer exists in Debian Jessie.  Change
Makefile.debian-jessie to install cpan version instead of
debian package.

Signed-off-by: Josh Stompro <stomproj@larl.org>
Signed-off-by: Ben Shum <bshum@biblio.org>
4 years agoLP#1445187 - Force disable of deflate
Josh Stompro [Thu, 16 Apr 2015 19:55:25 +0000 (14:55 -0500)]
LP#1445187 - Force disable of deflate

Add the -f (force) flag to a2dismod to remove the chance of getting
a confirmation prompt when removing deflate.

Signed-off-by: Josh Stompro <stomproj@larl.org>
Signed-off-by: Ben Shum <bshum@biblio.org>
4 years agoLP#1447195 Updates for Debian Jessie in README
Josh Stompro [Mon, 20 Apr 2015 19:45:59 +0000 (14:45 -0500)]
LP#1447195 Updates for Debian Jessie in README

- Adding in a few missed references to Debian Jessie
- Noted instances where Jessie uses the same instructions as Trusty
- Debian Jessie includes Apache2.4 so needs to use those steps.
- Added Jessie to postgres server install

Signed-off-by: Josh Stompro <stomproj@larl.org>
Signed-off-by: Ben Shum <bshum@biblio.org>
4 years agoLP#1445150 Update Debian Jessie depend make to PG9.4
Josh Stompro [Thu, 16 Apr 2015 18:13:17 +0000 (13:13 -0500)]
LP#1445150 Update Debian Jessie depend make to PG9.4

Signed-off-by: Josh Stompro <stomproj@larl.org>
Signed-off-by: Ben Shum <bshum@biblio.org>
4 years agoDocs: Change references to release 2.8.1 for server upgrade
Ben Shum [Mon, 4 May 2015 16:58:22 +0000 (12:58 -0400)]
Docs: Change references to release 2.8.1 for server upgrade

Since 2.8.1 is released, change references in upgrade docs to this newest
release.

Signed-off-by: Ben Shum <bshum@biblio.org>
4 years agoLP#1449283: fix auth when running under Apache 2.4
Galen Charlton [Thu, 30 Apr 2015 18:07:14 +0000 (11:07 -0700)]
LP#1449283: fix auth when running under Apache 2.4

When running under Apache 2.4 using the stock configuration
derived from apache_24/eg_vhost.conf.in, protected
URLs such as https://eghost/reporter/ that are meant to
require valid EG staff credentials were not in fact
requiring authentication.

This patch does the following to fix this:

[1] Removes several uses of "Require all granted" that
    was causing authentication to be ignored.
[2] Changes OpenILS::WWW::Proxy::Authen so that it always
    sets the username in the Apache request object if
    authentication was successful; it appears that starting
    with Apache 2.4, authentication handlers must ensure
    that a user name is set for a "Require valid-user"
    directive to work.

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Signed-off-by: Jason Stephenson <jstephenson@mvlc.org>
Signed-off-by: Bill Erickson <berickxx@gmail.com>
4 years agoForward-port 2.8.0 to 2.8.1 SQL upgrade
Bill Erickson [Fri, 1 May 2015 20:06:14 +0000 (16:06 -0400)]
Forward-port 2.8.0 to 2.8.1 SQL upgrade

Signed-off-by: Bill Erickson <berickxx@gmail.com>
4 years agoAdd 2.8.1 release notes to the 2.8 release note file.
Kathy Lussier [Wed, 29 Apr 2015 03:09:57 +0000 (23:09 -0400)]
Add 2.8.1 release notes to the 2.8 release note file.

Signed-off-by: Kathy Lussier <klussier@masslnc.org>
4 years agoLP#1427309 Fix Crash in Collections User Balance Summary
Jason Boyer [Mon, 2 Mar 2015 20:06:33 +0000 (15:06 -0500)]
LP#1427309 Fix Crash in Collections User Balance Summary

Previously a patron in collections that paid off all
transactions would cause a crash and stop processing
any balance summary file that they are supposed to
appear in. Now user balance summaries can be created
in full even if some users have a 0 balance.

Signed-off-by: Jason Boyer <jboyer@library.in.gov>
Signed-off-by: Bill Erickson <berickxx@gmail.com>
4 years agoLP 1444130: Add max_chunk_size guards to Holds.pm.
Jason Stephenson [Tue, 14 Apr 2015 14:43:49 +0000 (10:43 -0400)]
LP 1444130: Add max_chunk_size guards to Holds.pm.

Web staff client fixes introduced a bug in using the Clear
Holds Shelf checkin modifier.  Commit 4ccbf980 added calls
to max_chunk_size on the client to use chunked responses for
better feedback during long operations.  Thise works just
fine when a regular OpenSRF AppRequest is used, but leads to
a network error when AppSubrequest is used, as apparently the
xul client does with the Clear Holds Shelf checkin modifier.

This commit adds code to check if the client object can do
the max_chunk_size call when the max_chunk_size call is
made.  This will prevent the network errors in cases where
the client is an AppSubrequest object and will still use
chunked responses when the client is capable of doing it.

Signed-off-by: Jason Stephenson <jstephenson@mvlc.org>
Signed-off-by: Bill Erickson <berickxx@gmail.com>
4 years agoLP#1443952 Lost fine handling refactor
Dan Wells [Fri, 20 Feb 2015 22:36:25 +0000 (17:36 -0500)]
LP#1443952 Lost fine handling refactor

This commit moves the lost fine logic out of the lost processing
portion of Circulate.pm and into the new handle_fines() method.
This is accomplished through setting a new flag and lost
options hash on the main Circulator object.

This change improves the overall process flow, and in doing so
addresses a bug where overdue fines could be doubled if both
restore-overdue-on-lost-return and generate-new-overdues-on-lost-return
are enabled.  Before this change, the new fines would generate before
the old fines were unvoided, which effectively circumvented the max
fines setting.

A secondary change made here is that stop_fines is also now being set
during the handle_fines() stage. This improves any case where overdues
are being set on checkin, particularly lost returns, as the stop_fines
will end up as the proper value of MAX_FINES or CHECKIN.

Signed-off-by: Dan Wells <dbw2@calvin.edu>
Signed-off-by: Ben Shum <bshum@biblio.org>
4 years agoLP#1443952 Move overdue restore above lost void/adjustment
Dan Wells [Fri, 20 Feb 2015 22:28:44 +0000 (17:28 -0500)]
LP#1443952 Move overdue restore above lost void/adjustment

If a person loses something and has their overdues voided/zeroed,
then pays for part of it, then returns the item, the part they
actually paid should apply to the overdues.  In order for this to
happen, the overdues must exist *before* the lost fines are adjusted.

Signed-off-by: Dan Wells <dbw2@calvin.edu>
Signed-off-by: Ben Shum <bshum@biblio.org>
4 years agoLP#1442695: install purge_pending_users.srfsh to /openils/bin by default
Galen Charlton [Fri, 10 Apr 2015 15:47:29 +0000 (15:47 +0000)]
LP#1442695: install purge_pending_users.srfsh to /openils/bin by default

Since purge_pending_users.srfsh is in the example crontab, it should
be installed to the Evergreen binaries directory (typically /openils/bin)
by default.

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Signed-off-by: Ben Shum <bshum@biblio.org>
4 years agoLP#1442701: prune 'message_id' and 'single' CGI params from TPAC dashboard links
Galen Charlton [Fri, 10 Apr 2015 16:09:26 +0000 (16:09 +0000)]
LP#1442701: prune 'message_id' and 'single' CGI params from TPAC dashboard links

This patch fixes a bug where if one goes to the single patron
message view in TPAC, clicking on the 'Message' button in the
dashboard button bar does not return one to the list of messages, but
instead takes the user back to the single message the user was
viewing.  This patch also prunes the CGI parameter from other
links in the patron dashboard.

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Signed-off-by: Kathy Lussier <klussier@masslnc.org>
Signed-off-by: Ben Shum <bshum@biblio.org>
4 years agoLP#1380803 Update PO summary amounts
Bill Erickson [Mon, 6 Apr 2015 21:33:26 +0000 (17:33 -0400)]
LP#1380803 Update PO summary amounts

Refresh the PO summary amounts (spent, encumbered, estimated) each time
an amount-changing event occurs.  These include changing the lineitem
price, adding/removing a direct charge, and adding/removing copies via
the copy grid.

Note that adding/removing copies via the batch-updater alread causes a
page refresh, which updates the summary amounts.

Signed-off-by: Bill Erickson <berickxx@gmail.com>
Signed-off-by: Kathy Lussier <klussier@masslnc.org>
4 years agoLP#1380803 Include direct charges in PO esimated price
Bill Erickson [Mon, 6 Apr 2015 18:54:55 +0000 (14:54 -0400)]
LP#1380803 Include direct charges in PO esimated price

Move the estimated PO price calculation into the middle layer, along
with the total encumbered and spent calculation.  Add a new
PO.amount_estimated field for carrying the data.

Signed-off-by: Bill Erickson <berickxx@gmail.com>
Signed-off-by: Kathy Lussier <klussier@masslnc.org>
4 years agoDocs: Update 2.6 RELEASE NOTES to include new section on Bug Fixes
Ben Shum [Fri, 10 Apr 2015 14:53:59 +0000 (10:53 -0400)]
Docs: Update 2.6 RELEASE NOTES to include new section on Bug Fixes

Signed-off-by: Ben Shum <bshum@biblio.org>
4 years agoDocs: Update 2.8 RELEASE NOTES to include new Bug Fixes section
Ben Shum [Fri, 10 Apr 2015 14:48:43 +0000 (10:48 -0400)]
Docs: Update 2.8 RELEASE NOTES to include new Bug Fixes section

Signed-off-by: Ben Shum <bshum@biblio.org>
4 years agoDocs: Move security bug fix note into the Bug Fixes area of 2.7 RELEASE NOTES
Ben Shum [Fri, 10 Apr 2015 14:43:31 +0000 (10:43 -0400)]
Docs: Move security bug fix note into the Bug Fixes area of 2.7 RELEASE NOTES

Signed-off-by: Ben Shum <bshum@biblio.org>
4 years agoDocs: Start a new section for bug fixes for 2.7 series
Ben Shum [Fri, 10 Apr 2015 14:38:45 +0000 (10:38 -0400)]
Docs: Start a new section for bug fixes for 2.7 series

And pull in the reporter changes here.

Signed-off-by: Ben Shum <bshum@biblio.org>
4 years agoDOCS: Simplify release notes entry for AccessHandler
Remington Steed [Tue, 7 Apr 2015 13:35:19 +0000 (09:35 -0400)]
DOCS: Simplify release notes entry for AccessHandler

This commit trims down the AccessHandler docs to provide a simpler
introduction to the feature for use in the release notes. Also, a few
minor wording edits were made.

Signed-off-by: Remington Steed <rjs7@calvin.edu>
Signed-off-by: Kathy Lussier <klussier@masslnc.org>
4 years agoLP#1436906 clean up PO direct charges on delete/cancel
Bill Erickson [Mon, 6 Apr 2015 15:24:33 +0000 (11:24 -0400)]
LP#1436906 clean up PO direct charges on delete/cancel

1. Deleting a "direct charge" from a PO removes the linked fund_debit if
the fund_debit has not been paid (i.e. it's still encumbered).

If the debit has been paid, the charge cannot be removed, unless/until
the debit is re-encumbered by un-invoicing the charge.

2. When a PO is canceled, fund_debits linked to PO items are removed.
As above, if the debit has been paid (invoiced), the PO cannot be
canceled, unless/until direct charge debits are un-invoiced.

Signed-off-by: Bill Erickson <berickxx@gmail.com>
Signed-off-by: Kathy Lussier <klussier@masslnc.org>
Signed-off-by: Ben Shum <bshum@biblio.org>
4 years agoLP#1435494: add release notes entry
Galen Charlton [Thu, 9 Apr 2015 20:50:24 +0000 (20:50 +0000)]
LP#1435494: add release notes entry

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Signed-off-by: Ben Shum <bshum@biblio.org>
4 years agoLP#1435494: suggest 1048575 as a default resultset_limit
Galen Charlton [Thu, 9 Apr 2015 20:09:59 +0000 (20:09 +0000)]
LP#1435494: suggest 1048575 as a default resultset_limit

Per a suggestion by Thomas Berezansky; this magic number
represents the number of rows supported by XSLX, less one
for a header row.

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Signed-off-by: Ben Shum <bshum@biblio.org>
4 years agoLP#1435494: do not encourage <resultset_limit></resultset_limit>
Galen Charlton [Thu, 9 Apr 2015 20:06:25 +0000 (20:06 +0000)]
LP#1435494: do not encourage <resultset_limit></resultset_limit>

An empty node in opensrf.xml gets parsed as an empty hashref,
not an empty scalar, so we'll use <resultset_limit>0</resultlet_limit>
in the suggested opensrf.xml config.  This fixes an issue
noticed by Ben Shum during testing where reports could fail with
the following message:

  DBD::Pg::st execute failed: ERROR: syntax error at or near "0"
  LINE 43: ) limited_to_HASH(0x2a974f8)_hits LIMIT HASH(0x2a974f8)
                             ^ at /openils/bin/clark-kent.pl line 243.

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Signed-off-by: Ben Shum <bshum@biblio.org>
4 years agoLP#1435494: set limits on Clark Kent's resource usage
Galen Charlton [Fri, 20 Mar 2015 20:33:39 +0000 (20:33 +0000)]
LP#1435494: set limits on Clark Kent's resource usage

Clark Kent can sometimes consume more RAM, swap space, or CPU
than is reasonable or productive. For example:

- a badly constructed query with multiple Cartesian joins may
  never terminate, potentially tying up a Clark child process,
  pegging a CPU on the database server, and/or causing significant
  scratch disk usage on the database server keeping a snapshot alive.
- a query that returns a very large number of rows can cause a Clark
  child to bloat, and in extreme cases cause a OOM on the server
  running Clark.
- a report that asks for a chart of an unreasonably large number of
  rows can peg a CPU on the Clark server as GD::Graph attempts to
  compute sub-pixel graph elements.

In each of these cases, a requested report may never finish.

This patch adds the ability set set some limits on Clark.  These
limits can be set either in opensrf.xml for the settings service
to distribute or via command-line switches to clark-kent.pl:

//reporter/setup/statement_timeout / --statement-timeout

  Number of minutes to allow a report's underlying SQL query
  to run before it gets cancelled.  Default value is
  60 minutes.  If a report's query gets cancelled, the
  error_text value will be set to a valid that indicates that
  the allowed time was exceeded.

//reporter/setup/max_rows_for_charts / --max-rows-for-charts

  Number of rows permitted in the query's output before
  Clark Kent refuses to attempt to draw a graph. Default
  value is 1,000 rows.

//reporter/setup/resultset_limit / --resultset-limit

  If set, truncates the report's output to the specified
  number of hits.  Note that it will not be apparent
  to a staff user if the report's output has been
  truncated.  Default value is unlimited.

This patch also adds the ability for the concurrency
to be set via an opensrf.xml setting (//reporter/setup/parallel).

If both a command-line switch and an opensrf.xml setting
are supplied, the value set in the command line takes
precedence.

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Signed-off-by: Ben Shum <bshum@biblio.org>
4 years agoLP#1442246: (follow-up) use pre-wrap rather than pre
Galen Charlton [Thu, 9 Apr 2015 20:52:48 +0000 (20:52 +0000)]
LP#1442246: (follow-up) use pre-wrap rather than pre

This allows long messages to wrap; such messages can easily
be produced by manual creation of user notes.

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Signed-off-by: Ben Shum <bshum@biblio.org>
4 years agoLP#1442246: improve styling of patron messages
Galen Charlton [Tue, 31 Mar 2015 20:34:42 +0000 (20:34 +0000)]
LP#1442246: improve styling of patron messages

This patch adds new CSS classes for pieces of the
patron message display, uses a preformatted display
for the message body, and fixes an HTML nesting error.

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Signed-off-by: Ben Shum <bshum@biblio.org>
4 years agoLP#1442254: Stamping upgrade script for use-ws-lib-for-messages
Ben Shum [Fri, 10 Apr 2015 01:58:07 +0000 (21:58 -0400)]
LP#1442254: Stamping upgrade script for use-ws-lib-for-messages

Signed-off-by: Ben Shum <bshum@biblio.org>
4 years agoLP#1442254: adjust how sending library set for messages created from notes
Galen Charlton [Tue, 31 Mar 2015 21:23:22 +0000 (21:23 +0000)]
LP#1442254: adjust how sending library set for messages created from notes

When creating a new user message from a public note, the library
of the workstation of the staff member who created the original
note is used if available from get_audit_info().  If it's not
available (e.g., if notes are added using direct SQL, the
home library of the note creator is used instead).

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Signed-off-by: Kathy Lussier <klussier@masslnc.org>
Signed-off-by: Ben Shum <bshum@biblio.org>
4 years agoLP#1442262: move patron messages link to dashboard button bar
Galen Charlton [Wed, 8 Apr 2015 22:42:26 +0000 (22:42 +0000)]
LP#1442262: move patron messages link to dashboard button bar

Also implement a Facebook-style display of the unread
messages count.

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Signed-off-by: Kathy Lussier <klussier@masslnc.org>
Signed-off-by: Ben Shum <bshum@biblio.org>
4 years agoLP#1442260: include unread message count in page title
Galen Charlton [Tue, 31 Mar 2015 21:03:14 +0000 (21:03 +0000)]
LP#1442260: include unread message count in page title

If a patron is logged in and has at least one unread message,
the count of unread messages is prepended to the page title
for all TPAC pages.

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Signed-off-by: Kathy Lussier <klussier@masslnc.org>
Signed-off-by: Ben Shum <bshum@biblio.org>
4 years agoLP#1442250: fix thinko preventing message deletion acknowledgment from being displayed
Galen Charlton [Tue, 31 Mar 2015 20:38:16 +0000 (20:38 +0000)]
LP#1442250: fix thinko preventing message deletion acknowledgment from being displayed

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Signed-off-by: Kathy Lussier <klussier@masslnc.org>
Signed-off-by: Ben Shum <bshum@biblio.org>
4 years agoLP#1441634: add release notes for removal of biblios.net Z39.50 target
Galen Charlton [Wed, 8 Apr 2015 16:30:19 +0000 (16:30 +0000)]
LP#1441634: add release notes for removal of biblios.net Z39.50 target

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
4 years agoLP#1441634 Remove biblios.net from Z39.50 sources
Dan Scott [Wed, 8 Apr 2015 13:53:14 +0000 (09:53 -0400)]
LP#1441634 Remove biblios.net from Z39.50 sources

The biblios.net Z39.50 server has not been functional for years.
Remove it from the list of Z39.50 servers that we populate in a
default install of Evergreen.

Signed-off-by: Dan Scott <dscott@laurentian.ca>
Signed-off-by: Galen Charlton <gmc@esilibrary.com>
4 years agoDocs: Change all .PNG to .png
Ben Shum [Tue, 7 Apr 2015 21:22:10 +0000 (17:22 -0400)]
Docs: Change all .PNG to .png

There were files with uppercase .PNG and referneces to .PNG in the docs.

Change all to lowercase .png for consistency.

Signed-off-by: Ben Shum <bshum@biblio.org>
4 years agoDocs: Delete remnant .JPG files that had already been converted to .jpg
Ben Shum [Tue, 7 Apr 2015 20:04:47 +0000 (16:04 -0400)]
Docs: Delete remnant .JPG files that had already been converted to .jpg

In a previous commit, these files were already re-added as lowercase .jpg,
so let's go ahead and remove the .JPG versions of those files.

Signed-off-by: Ben Shum <bshum@biblio.org>
4 years agoDocs: Change all .JPG to .jpg
Ben Shum [Tue, 7 Apr 2015 20:03:39 +0000 (16:03 -0400)]
Docs: Change all .JPG to .jpg

There were files with uppercase .JPG and references to .JPG in the docs.

Change all to lowercase .jpg for consistency.

Signed-off-by: Ben Shum <bshum@biblio.org>
4 years agoDocs: Minor edits to 2.8 release notes
Ben Shum [Tue, 7 Apr 2015 15:39:22 +0000 (11:39 -0400)]
Docs: Minor edits to 2.8 release notes

Fix spelling mistake for "Berkless" to "Berklee" in the acknowledgements.

Also, change from "2_8" to "2.8" for consistency in reading.

Signed-off-by: Ben Shum <bshum@biblio.org>
4 years agoRelease Notes acknowledgements addition
Kathy Lussier [Thu, 2 Apr 2015 23:36:21 +0000 (19:36 -0400)]
Release Notes acknowledgements addition

Adding KCLS to the acknowledgements.

Signed-off-by: Kathy Lussier <klussier@masslnc.org>
4 years agoRelease Note Acknowledgements user/kmlussier/docs-2-8-acknowledgements
Kathy Lussier [Thu, 2 Apr 2015 20:11:19 +0000 (16:11 -0400)]
Release Note Acknowledgements

Added acknowledgements to the Release Notes. Also added a missing feature and
did some minor cleanup.

Signed-off-by: Kathy Lussier <klussier@masslnc.org>
4 years ago2.8 release notes creation and cleanup
Bill Erickson [Fri, 27 Feb 2015 17:14:32 +0000 (12:14 -0500)]
2.8 release notes creation and cleanup

* Create RELEASE_NOTES_2_8.txt
* Clean up RELEASE_NOTE_NEXT
* Bump doc header level for security announcement

Signed-off-by: Bill Erickson <berickxx@gmail.com>
4 years agoForward-porting 2.7.4-2.8.0 DB upgrade
Bill Erickson [Tue, 31 Mar 2015 14:48:43 +0000 (10:48 -0400)]
Forward-porting 2.7.4-2.8.0 DB upgrade

Signed-off-by: Bill Erickson <berickxx@gmail.com>
4 years agoLP 1435966: Clark Kent should write lock file after it forks.
Jason Stephenson [Tue, 24 Mar 2015 16:49:39 +0000 (12:49 -0400)]
LP 1435966: Clark Kent should write lock file after it forks.

Rearrange the code in clark-kent.pl so that the lock/pid file is
opened and written to after the reporter daemonizes itself.  This
way, the pid in the file can actually be used to stop the running
reporter daemon.

Signed-off-by: Jason Stephenson <jstephenson@mvlc.org>
Signed-off-by: Galen Charlton <gmc@esilibrary.com>
4 years agoLP#1234220 Stamping DB upgrade copy/ratio messages
Bill Erickson [Fri, 13 Mar 2015 21:23:56 +0000 (17:23 -0400)]
LP#1234220 Stamping DB upgrade copy/ratio messages

Signed-off-by: Bill Erickson <berickxx@gmail.com>
4 years agoLP#1234220 hold ratio renewal override perms
Bill Erickson [Fri, 6 Mar 2015 16:43:43 +0000 (11:43 -0500)]
LP#1234220 hold ratio renewal override perms

New permissions for overriding renewal failure events:

TOTAL_HOLD_COPY_RATIO_EXCEEDED.override
AVAIL_HOLD_COPY_RATIO_EXCEEDED.override

Signed-off-by: Bill Erickson <berickxx@gmail.com>
Signed-off-by: Kathy Lussier <klussier@masslnc.org>
4 years agoLP#1234220 Improve hold/copy ratio renewal messages
Bill Erickson [Fri, 6 Mar 2015 16:04:01 +0000 (11:04 -0500)]
LP#1234220 Improve hold/copy ratio renewal messages

Display friendlier messages in the TPAC and staff client client when a
hold/copy ratio rule prevents a renewal.

This creates top-level ILS events for total/avail hold copy ratios,
since that's what best works with the staff client.  The TPAC displays a
simple "item is needed for a hold" message, whereas the staff client
displays the raw event description, which is slightly more descriptive.

Signed-off-by: Bill Erickson <berickxx@gmail.com>
Signed-off-by: Kathy Lussier <klussier@masslnc.org>
4 years agoDocs: Link optional docs from Apache config
Remington Steed [Thu, 12 Mar 2015 13:15:37 +0000 (09:15 -0400)]
Docs: Link optional docs from Apache config

This commit links to the new optional Apache config docs from the Apache
config step in the installation instructions.

Signed-off-by: Remington Steed <rjs7@calvin.edu>
4 years agoLP#1413624 Docs: Move Access Handler docs to official docs
Remington Steed [Wed, 11 Mar 2015 21:23:57 +0000 (17:23 -0400)]
LP#1413624 Docs: Move Access Handler docs to official docs

This commit moves the new Access Handler doc file to the official docs
folder, adds the file reference to the root.txt, changes the main header
text for easier linking, adds the missing block delimiter lines around a
code block, and adds a blank line at the end of another doc file to
prevent the docs from running together.

Signed-off-by: Remington Steed <rjs7@calvin.edu>
4 years agoLP#1413621 Docs: Minor changes to Apache Rewrite Tricks
Remington Steed [Wed, 21 Jan 2015 19:14:00 +0000 (14:14 -0500)]
LP#1413621 Docs: Minor changes to Apache Rewrite Tricks

This commit moves this doc file from TechRef into the official docs
area, adds it to the root.txt file, and adds line breaks for easier
viewing and editing of the file.

Signed-off-by: Remington Steed <rjs7@calvin.edu>
4 years agoLP#1413621 Docs: Apache Rewrite Tricks
Thomas Berezansky [Wed, 21 Jan 2015 19:14:00 +0000 (14:14 -0500)]
LP#1413621 Docs: Apache Rewrite Tricks

Because multiple sites have found this kind of thing useful in the past.

Signed-off-by: Thomas Berezansky <tsbere@mvlc.org>
Signed-off-by: Remington Steed <rjs7@calvin.edu>
4 years agoDocs: Fix AsciiDoc processing errors
Remington Steed [Tue, 10 Mar 2015 13:00:06 +0000 (09:00 -0400)]
Docs: Fix AsciiDoc processing errors

This commit fixes an AsciiDoc link syntax error, and renames section ids
to prevent duplicate names in the preview version of the Web Client
docs.

Signed-off-by: Remington Steed <rjs7@calvin.edu>
4 years agoDocs: Eliminate accidental invalid level 5 AsciiDoc headings
Yamil Suarez [Fri, 6 Mar 2015 21:26:02 +0000 (16:26 -0500)]
Docs: Eliminate accidental invalid level 5 AsciiDoc headings

To properly integrate updated receipt template editor documentation
we origimally used invalid level 5 AsciiDoc headings. Asciidoc only
supports 4 levels of headings

Signed-off-by: Yamil Suarez <yamil@yamil.com>
4 years agoDocs: Add authority section and Authority subject heading thesaurus
Yamil Suarez [Fri, 6 Mar 2015 21:03:54 +0000 (16:03 -0500)]
Docs: Add authority section and Authority subject heading thesaurus

Signed-off-by: Yamil Suarez <yamil@yamil.com>
4 years agoDocs: 2.8 feature correction
Remington Steed [Fri, 6 Mar 2015 17:22:51 +0000 (12:22 -0500)]
Docs: 2.8 feature correction

This commit corrects a mistake in the new feature documentation for
deleting copy locations.

Signed-off-by: Remington Steed <rjs7@calvin.edu>
4 years agoDocs: 2.8 feature "Deleted flag for copy locations"
Remington Steed [Fri, 6 Mar 2015 15:46:50 +0000 (10:46 -0500)]
Docs: 2.8 feature "Deleted flag for copy locations"

This commit documents the new 2.8 feature, the ability to delete copy
locations. It also updates the screenshot, and adds lines to document
other common features of the interface.

Signed-off-by: Remington Steed <rjs7@calvin.edu>
4 years agoLP#1403967: show 'subject heading thesaurus' value release notes
Yamil Suarez [Thu, 5 Mar 2015 17:26:38 +0000 (12:26 -0500)]
LP#1403967: show 'subject heading thesaurus' value release notes

The 'subject heading thesaurus' basically defines the issuing authority
that created the authority record. For example, a value of "a" means that
it came from LoC. A value of "n" means that it is a locally created
authority. Note that within the authority MARC editor the 'Subject
Heading Thesaurus' is stored/displayed in the 'Subj' fixed field.

Signed-off-by: Yamil Suarez <yamil@yamil.com>
4 years agoForward-port 2.7.3-2.7.4 upgrade script
Ben Shum [Tue, 3 Mar 2015 23:01:13 +0000 (18:01 -0500)]
Forward-port 2.7.3-2.7.4 upgrade script

Signed-off-by: Ben Shum <bshum@biblio.org>
4 years agoLP#1424755: Add IMPORTANT SECURITY INFORMATION release note.
Jason Stephenson [Thu, 19 Feb 2015 16:17:08 +0000 (11:17 -0500)]
LP#1424755: Add IMPORTANT SECURITY INFORMATION release note.

Signed-off-by: Jason Stephenson <jstephenson@mvlc.org>
Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Signed-off-by: Ben Shum <bshum@biblio.org>
4 years agoLP#1424755: Org Unit Setting view permissions can be bypassed
Jason Stephenson [Wed, 18 Feb 2015 21:28:31 +0000 (16:28 -0500)]
LP#1424755: Org Unit Setting view permissions can be bypassed

Fix private org. unit setting leakage by forcing the $auth argument
to true if not passed in when open-ils.actor.ou_setting.ancestor_default
or open-ils.actor.ou_setting.ancestor_default.batch are called.

Other than a change to the desc of the public methods to reflect that
they now check permissions if permissions are required, there are no
required api changes to the back or the front ends.

Signed-off-by: Jason Stephenson <jstephenson@mvlc.org>
Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Signed-off-by: Ben Shum <bshum@biblio.org>
4 years agoLP#1206589: Temporary Fix for Org. Unit Settings History Bug
Jason Stephenson [Tue, 24 Feb 2015 19:05:16 +0000 (14:05 -0500)]
LP#1206589: Temporary Fix for Org. Unit Settings History Bug

This commit makes two changes to the coustl IDL entry.

1. It adds a retrieve permission of STAFF_LOGIN.  This at least
requires someone with staff permission to be able to view settings
history.

2. It fixed the primary key field from date_applied to id.  The
prior setting was just an outright error.

NOTE: This is intended as a short term fix for the Library Settings
Editor history feature.

Signed-off-by: Jason Stephenson <jstephenson@mvlc.org>
Signed-off-by: Ben Shum <bshum@biblio.org>
4 years agolp1427331: Remove relator information from author links
Kathy Lussier [Mon, 2 Mar 2015 20:06:03 +0000 (15:06 -0500)]
lp1427331: Remove relator information from author links

Removes the display and linking of relator information from the author field
on the search results page. The display of subfield e was limiting the number
or results retrieved when clicking these links.

Signed-off-by: Kathy Lussier <klussier@masslnc.org>
Signed-off-by: Dan Scott <dscott@laurentian.ca>
4 years agoRevert LP#1205072
Dan Wells [Tue, 3 Mar 2015 17:11:35 +0000 (12:11 -0500)]
Revert LP#1205072

It was decided that these changes caused too much potential for
confusion, and would be better served by a simultaneous change to
making 'granularity' case insensitive.  Pulling back for now.

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