]> git.evergreen-ils.org Git - Evergreen.git/log
Evergreen.git
7 years agoLP#1497335 Aged/All circulation API access
Bill Erickson [Wed, 3 Aug 2016 17:34:29 +0000 (13:34 -0400)]
LP#1497335 Aged/All circulation API access

Various SQL, IDL, and API changes for accessing aged circulations,
primarily via all_circulation objects, for imporoved staff client
integration.

*. Support open-ils.pcrud access to the action.all_circulation DB view /
   'combcirc' class.

*. Add missing parent_circ, checkin_scan_time, checkin_workstation to
   combcirc class.

*. Add 'usr' field to combcirc.  The action.all_circulation VIEW will
   return NULL as the 'usr' column value when returning data for an
   aged_circulation.

*. Add virtual 'active_circ' and 'aged_circ' fields to combcirc for
   fleshing the related action.circulation or action.aged_circulation
   object.

* Adds SQL functions for action.all_circ_chain and
  action.summarize_all_circ_chain, which pull data from
  aged_circulation.

*. API calls
open-ils.circ.[prev_]renewal_chain.retrieve_by_circ[.summary] now return
data for active and aged circulations by using action.all_circ_chain and
action.summarize_all_circ_chain.  When using these APIs, a null value
in the 'usr' column is the indication that a given circulation or circ
chain summary represents an aged circulation.

* API open-ils.circ.copy_details.retrieve will now optionally return
  aged circ data within the copy circ history.

Signed-off-by: Bill Erickson <berickxx@gmail.com>
7 years agoStamping upgrade script for Canceled Transit status branch
Mike Rylander [Wed, 24 Aug 2016 22:07:13 +0000 (18:07 -0400)]
Stamping upgrade script for Canceled Transit status branch

Signed-off-by: Mike Rylander <mrylander@gmail.com>
7 years agoLP#1613374 - Check stored copy status before clobbering it.
Chris Sharp [Wed, 24 Aug 2016 18:23:09 +0000 (14:23 -0400)]
LP#1613374 - Check stored copy status before clobbering it.

Some transit.copy_statuses would result in some other behavior
upon checkin, so we check all that would go into 'Reshelving'
status, otherwise, restore the stored status.

Signed-off-by: Chris Sharp <csharp@georgialibraries.org>
Signed-off-by: Michele Morgan <mmorgan@noblenet.org>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
7 years agoLP#1613374 - Add 'Canceled Transit' to non-'BAD' copy statuses.
Chris Sharp [Tue, 23 Aug 2016 13:32:59 +0000 (09:32 -0400)]
LP#1613374 - Add 'Canceled Transit' to non-'BAD' copy statuses.

During checkin, Circulate.pm checks the copy status and if it
isn't listed as an okay status, the event COPY_BAD_STATUS is
returned, which requires an override.  This change makes copies
in 'Canceled Transit' status check in without an alert.

Signed-off-by: Chris Sharp <csharp@georgialibraries.org>
Signed-off-by: Michele Morgan <mmorgan@noblenet.org>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
7 years agoLP#1613374 - Add 'Canceled Transit' to "magical statuses"
Chris Sharp [Tue, 16 Aug 2016 11:11:24 +0000 (07:11 -0400)]
LP#1613374 - Add 'Canceled Transit' to "magical statuses"

Signed-off-by: Chris Sharp <csharp@georgialibraries.org>
Signed-off-by: Michele Morgan <mmorgan@noblenet.org>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
7 years agoLP#1613374 - Make 'Canceled Transit' status holdable and OPAC-visible.
Chris Sharp [Mon, 22 Aug 2016 12:58:24 +0000 (08:58 -0400)]
LP#1613374 - Make 'Canceled Transit' status holdable and OPAC-visible.

Signed-off-by: Chris Sharp <csharp@georgialibraries.org>
Signed-off-by: Michele Morgan <mmorgan@noblenet.org>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
7 years agoLP#1613374 - Set item status to 'Canceled Transit' when transit is canceled.
Chris Sharp [Wed, 24 Aug 2016 18:12:04 +0000 (14:12 -0400)]
LP#1613374 - Set item status to 'Canceled Transit' when transit is canceled.

I opened a separate bug report on this issue (bug 1613374) before realizing
bug 1306666 was reported.  This commit creates a new 'Canceled Transit' status
that signals to staff what happened to an item when a transit is aborted/canceled.

Signed-off-by: Chris Sharp <csharp@georgialibraries.org>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
Conflicts:
Open-ILS/src/perlmods/lib/OpenILS/Application/Circ/Transit.pm

Signed-off-by: Michele Morgan <mmorgan@noblenet.org>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
7 years agoStamping upgrade for email receipts
Mike Rylander [Wed, 24 Aug 2016 22:01:33 +0000 (18:01 -0400)]
Stamping upgrade for email receipts

Signed-off-by: Mike Rylander <mrylander@gmail.com>
7 years agoLP#1356477: release notes for email checkout receipts
Galen Charlton [Mon, 22 Aug 2016 22:08:37 +0000 (18:08 -0400)]
LP#1356477: release notes for email checkout receipts

A portion of the text was written by Angela Kilsdonk.

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
7 years agoLP#1356477: update selfcheck interface
Mike Rylander [Thu, 4 Aug 2016 14:38:48 +0000 (10:38 -0400)]
LP#1356477: update selfcheck interface

This patch replaces the Logout and Logout with Receipt buttons in
the selfcheck interface with a single Logout button and a set of radio
buttons (whose labels are clickable) that allow the user to specify
whether they want an email receipt, a print receipt, or no receipt upon
logging out.

If the user has no email address, the option to select email receipts
will not be presented to them.

Signed-off-by: Mike Rylander <mrylander@gmail.com>
Signed-off-by: Galen Charlton <gmc@esilibrary.com>
7 years agoLP#1356477: move email receipt checkbox on patron editor
Galen Charlton [Thu, 4 Aug 2016 16:07:14 +0000 (12:07 -0400)]
LP#1356477: move email receipt checkbox on patron editor

This patch updates the patron editor so that the checkbox
for setting whether the user wishes to get email checkout
receipts by default is moved next to the email address
input.

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
7 years agoLP#1356477: add quick receipt button
Mike Rylander [Wed, 3 Aug 2016 20:51:54 +0000 (16:51 -0400)]
LP#1356477: add quick receipt button

This patch adds a new button to the webstaff checkout page
called Quick Receipt. If the button itself is pushed, a
receipt containing the current checkouts is either printed
or emailed, depending on the user's preference. If the
drop-down portion of the button is used, staff members can
override the user's default preference to print or email
the receipt.

The Quick Receipt button is enabled only if at least one
checkout has been made during the current session.

Note that email receipts is an option only when the patron
has an email address supplied.

An icon next to the Quick Receipt button will be either a
printer or an envelope depending on the user's preferred
receipt setting.

This patch also modifies the "Done" button. If pressed, the session
is ended and the receipt is generated according to the user's
preferences, but the drop-down portion can be used to end
the session while letting the staff member choose how the
receipt is emitted.

If a receipt is emailed, a toast is displayed saying so.

Signed-off-by: Mike Rylander <miker@esilibrary.com>
Signed-off-by: Galen Charlton <gmc@esilibrary.com>
7 years agoLP#1356477: teach OPAC patron registration form about staged user (opt-in) settings
Galen Charlton [Wed, 3 Aug 2016 19:38:49 +0000 (15:38 -0400)]
LP#1356477: teach OPAC patron registration form about staged user (opt-in) settings

This patch causes a checkbox to be added for each user setting
that controls opt-in for active Action/Trigger event definitions. In
the specific context of email checkout receipts, this means that
a patron who is self-registering can elect to get email checkout
receipts by default.

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
7 years agoLP#1356477: teach webstaff patron editor about staged user settings
Galen Charlton [Wed, 3 Aug 2016 19:36:18 +0000 (15:36 -0400)]
LP#1356477: teach webstaff patron editor about staged user settings

This patch also fixes a bug that preferred the webstaff patron editor
from successfully removing a staged patron upon completing the
registration of the patron.

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
7 years agoLP#1356477: create new staging table for user settings
Galen Charlton [Wed, 3 Aug 2016 19:37:39 +0000 (15:37 -0400)]
LP#1356477: create new staging table for user settings

This patch creates a new table called staging.setting_stage
that allows for storing user settings when a patron uses
(for example) the self-registration page.

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
7 years agoLP#1356477: add ability to unconditionally send email
Galen Charlton [Mon, 8 Aug 2016 20:50:08 +0000 (16:50 -0400)]
LP#1356477: add ability to unconditionally send email

This patch defines and uses an
open-ils.trigger.event.autocreate.ignore_opt_in method that does what
it says on the tin; the caller is reponsible for not abusing this.

This supports a use case in the webstaff checkout interface where staff
can choose to have the receipt be emailed, provided that the patron
record has an email address defined, even if the user setting to
email receipts by default is not set. (It is assumed that the staff
member would have gotten permission from the patron on the spot.)

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
7 years agoLP#1356477: business logic for email checking receipts
Mike Rylander [Wed, 3 Aug 2016 16:45:10 +0000 (12:45 -0400)]
LP#1356477: business logic for email checking receipts

This patch adds two methods, open-ils.circ.checkout.batch_notify
and open-ils.circ.checkout.batch_notify.session, that when passed
a set of circulation IDs, creates A/T events for the
checkout notification hooks. In the specific case of the email
checkout receipts feature, the seed data contains a stock
event that would send out emails, but it is conceivable
that other forms of notification could be done by defining
A/T events appropriately.

Signed-off-by: Mike Rylander <miker@esilibrary.com>
Signed-off-by: Galen Charlton <gmc@esilibrary.com>
7 years agoLP#1356477: seed data for email checkout receipts
Galen Charlton [Wed, 3 Aug 2016 16:18:19 +0000 (12:18 -0400)]
LP#1356477: seed data for email checkout receipts

This patch adds seed data for email checkout receipts,
including:

- The action/trigger hooks circ.checkout.batch_notify
  and circ.checkout.batch_notify.session.
- The default event definition
- The circ.send_email_checkout_receipts (Email checkout receipts by
  default?) user setting.

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
Conflicts:
Open-ILS/src/sql/Pg/950.data.seed-values.sql

7 years agoLP#1603708: Remove support for Ubuntu 12.04 Precise
Ben Shum [Tue, 5 Jul 2016 17:03:57 +0000 (13:03 -0400)]
LP#1603708: Remove support for Ubuntu 12.04 Precise

With support for Ubuntu 16.04 Xenial in place, remove the oldest LTS, which is
Ubuntu 12.04 Precise.

Signed-off-by: Ben Shum <ben@evergreener.net>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
7 years agoStamping upgrade script for badge seed data
Mike Rylander [Wed, 24 Aug 2016 18:55:29 +0000 (14:55 -0400)]
Stamping upgrade script for badge seed data

Signed-off-by: Mike Rylander <mrylander@gmail.com>
7 years agoLP#1614237: Popularity badge seed data
Kathy Lussier [Tue, 23 Aug 2016 20:01:35 +0000 (16:01 -0400)]
LP#1614237: Popularity badge seed data

Adding popularity badge seed data for a badge based on Holds Requested over the
past 5 years.

Signed-off-by: Kathy Lussier <klussier@masslnc.org>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
7 years agoLP#1614290: Add badge_score_generator to example crontab
Kathy Lussier [Wed, 17 Aug 2016 22:29:09 +0000 (18:29 -0400)]
LP#1614290: Add badge_score_generator to example crontab

Signed-off-by: Kathy Lussier <klussier@masslnc.org>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
7 years agoMoving function creation to later in the schema def, where its deps exist.
Mike Rylander [Wed, 24 Aug 2016 18:51:32 +0000 (14:51 -0400)]
Moving function creation to later in the schema def, where its deps exist.

Signed-off-by: Mike Rylander <mrylander@gmail.com>
7 years agoStamping upgrade script for authority edit changes and propagation improvement
Mike Rylander [Wed, 24 Aug 2016 16:56:28 +0000 (12:56 -0400)]
Stamping upgrade script for authority edit changes and propagation improvement

Signed-off-by: Mike Rylander <mrylander@gmail.com>
7 years agoLP#1588948: Only attempt a bib update if the heading changes
Mike Rylander [Wed, 24 Aug 2016 15:38:49 +0000 (11:38 -0400)]
LP#1588948: Only attempt a bib update if the heading changes

This should significantly reduce the churn on bibs and the time to save
an authority where the heading that would propagate to bibs has not changed.

Signed-off-by: Mike Rylander <mrylander@gmail.com>
Signed-off-by: Bill Erickson <berickxx@gmail.com>
7 years agoLP#1588948 Auth propagate bib meta on change only
Bill Erickson [Mon, 13 Jun 2016 17:57:30 +0000 (13:57 -0400)]
LP#1588948 Auth propagate bib meta on change only

Only update bib record editor and edit_date if an authority record
change propagation resulted in a modified bib record.

Signed-off-by: Bill Erickson <berickxx@gmail.com>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
7 years agoLP#1588948 Release notes (auth prop. bib edit[or|_date])
Bill Erickson [Mon, 6 Jun 2016 14:19:30 +0000 (10:19 -0400)]
LP#1588948 Release notes (auth prop. bib edit[or|_date])

Signed-off-by: Bill Erickson <berickxx@gmail.com>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
7 years agoLP#1588948 Authority propagation PGTAP test
Bill Erickson [Mon, 6 Jun 2016 15:14:47 +0000 (11:14 -0400)]
LP#1588948 Authority propagation PGTAP test

Adds a new general purpose authority->bib propagation pgtap test,
covering basic propagation and editor / edit_date propagation.

Signed-off-by: Bill Erickson <berickxx@gmail.com>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
7 years agoLP#1588948 Propagate authority->bib edit[or|_date]
Bill Erickson [Fri, 3 Jun 2016 19:16:58 +0000 (15:16 -0400)]
LP#1588948 Propagate authority->bib edit[or|_date]

Adds a new global flag 'ingest.disable_authority_auto_update_bib_meta',
which is disabled by default.

When disabled, the 'editor' and 'edit_date' columns on bib records
updated via authority field propagation are updated to the value of the
editor on the authority record and NOW(), respectively.

When enabled, editor and edit_date are not modified.

Signed-off-by: Bill Erickson <berickxx@gmail.com>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
7 years agoLP#1205492: fix crash when attempting to renew deposit & rental loans in OPAC
Galen Charlton [Wed, 24 Aug 2016 15:41:57 +0000 (11:41 -0400)]
LP#1205492: fix crash when attempting to renew deposit & rental loans in OPAC

This patch fixes a bug where attempting to renew a loan on an item
that requires a deposit or a rental fee can cause an Apache
internal server error.

In particular, this patch supplies descriptions for the following
events:

  ITEM_DEPOSIT_REQUIRED
  ITEM_RENTAL_FEE_REQUIRED
  ITEM_DEPOSIT_PAID

It also normalizes how 'fail_part' is set in the renewal response,
as the payload of an event returned when a renewal fails can
be either a hash or an acp Fieldmapper object depending on the
type of event.  In the former case, attempting to access an
nonexistent ->fail_part method is what causes the crash.

To test
-------
[1] Create an item that requires a deposit or a rental fee
    and check it out.
[2] Attempt to renew the loan in the public catalog. Note
    that an internal server error is returned.
[3] Apply the patch and attept step 2 again. This time, the
    public catalog should display a notification to the patron
    that the renewal cannot take place, rather than crashing.
[4] Set up other situations where a renewal should fail for
    reasons other than a deposit/rental fee being required. Verify
    that appropriate error messages are displayed in the public
    catalog.

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
7 years agoStamping upgrade script for transient usr_activity
Mike Rylander [Wed, 24 Aug 2016 16:38:58 +0000 (12:38 -0400)]
Stamping upgrade script for transient usr_activity

Signed-off-by: Mike Rylander <mrylander@gmail.com>
7 years agoLP#1570909 User activity purge release notes
Bill Erickson [Fri, 15 Apr 2016 21:07:02 +0000 (17:07 -0400)]
LP#1570909 User activity purge release notes

Signed-off-by: Bill Erickson <berickxx@gmail.com>
Signed-off-by: Chris Sharp <csharp@georgialibraries.org>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
7 years agoLP#1570909 User activity purge pgtap test
Bill Erickson [Fri, 15 Apr 2016 20:54:48 +0000 (16:54 -0400)]
LP#1570909 User activity purge pgtap test

Signed-off-by: Bill Erickson <berickxx@gmail.com>
Signed-off-by: Chris Sharp <csharp@georgialibraries.org>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
7 years agoLP#1570909 User activity purge function
Bill Erickson [Fri, 15 Apr 2016 20:26:57 +0000 (16:26 -0400)]
LP#1570909 User activity purge function

Utility function for removing all activity entries by activity type,
except for the most recent entry per user.  This is primarily useful
when cleaning up rows prior to setting the transient flag on an
activity type to true.  It allows for immediate cleanup of data (e.g.
for patron privacy) and lets admins control when the data is deleted,
which could be useful for huge activity tables.

Signed-off-by: Bill Erickson <berickxx@gmail.com>
Signed-off-by: Chris Sharp <csharp@georgialibraries.org>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
7 years agoLP#1570909 User activity transient default
Bill Erickson [Fri, 15 Apr 2016 15:23:33 +0000 (11:23 -0400)]
LP#1570909 User activity transient default

Only retain the most recent user activity entry (per activity type) by
default.

Signed-off-by: Bill Erickson <berickxx@gmail.com>
Signed-off-by: Chris Sharp <csharp@georgialibraries.org>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
7 years agoStamping upgrade of is_available branch
Mike Rylander [Wed, 24 Aug 2016 16:17:11 +0000 (12:17 -0400)]
Stamping upgrade of is_available branch

Signed-off-by: Mike Rylander <mrylander@gmail.com>
7 years agoLP#1464709 Copy status is_available release notes
Bill Erickson [Wed, 24 Aug 2016 14:26:28 +0000 (10:26 -0400)]
LP#1464709 Copy status is_available release notes

Signed-off-by: Bill Erickson <berickxx@gmail.com>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
7 years agoLP#1464709: Disentangle on-holds-shelf from is_available
Mike Rylander [Tue, 23 Aug 2016 20:59:33 +0000 (16:59 -0400)]
LP#1464709: Disentangle on-holds-shelf from is_available

Avoid treating On-Holds-Shelf items as is_available so they do not
appear in the catalog as #available.

Signed-off-by: Mike Rylander <mrylander@gmail.com>
Signed-off-by: Bill Erickson <berickxx@gmail.com>
7 years agoLP#1464709: teach QP about is_available statuses
Mike Rylander [Thu, 18 Aug 2016 13:52:20 +0000 (09:52 -0400)]
LP#1464709: teach QP about is_available statuses

We look them up, cache them, and use them for the #available modifier.

Signed-off-by: Mike Rylander <mrylander@gmail.com>
Signed-off-by: Bill Erickson <berickxx@gmail.com>
7 years agoLP#1464709 Rename checkout_ok to is_available
Bill Erickson [Thu, 27 Aug 2015 21:24:38 +0000 (17:24 -0400)]
LP#1464709 Rename checkout_ok to is_available

And update pgptap to check for only 2 is_available statuses (not
on-holds shelf).

Signed-off-by: Bill Erickson <berickxx@gmail.com>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
7 years agoLP#1464709 Non-standard copy status pgtap tests
Bill Erickson [Mon, 15 Jun 2015 22:09:01 +0000 (18:09 -0400)]
LP#1464709 Non-standard copy status pgtap tests

Includes unit test plus a set of live tests.

Signed-off-by: Bill Erickson <berickxx@gmail.com>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
7 years agoLP#1464709 Seamless non-standard copy status checkout
Bill Erickson [Wed, 3 Jun 2015 23:30:40 +0000 (19:30 -0400)]
LP#1464709 Seamless non-standard copy status checkout

Add a new boolean configuration option to copy statuses which allow such
copies to be checked out without requiring an alert or override.

New column is config.copy_status.checkout_ok.  Any status with this
boolean enabled will be treated the same as Available, Reshelving, and
Holds Shelf status copies in that the status will be considered
checkout-able.

Signed-off-by: Bill Erickson <berickxx@gmail.com>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
7 years agoStamping upgrade script for ranked-volumes update
Mike Rylander [Wed, 24 Aug 2016 15:58:56 +0000 (11:58 -0400)]
Stamping upgrade script for ranked-volumes update

Signed-off-by: Mike Rylander <mrylander@gmail.com>
7 years agoLP#1315552 Don't use circ_lib name in ranked_volumes
Dan Wells [Tue, 10 May 2016 19:41:54 +0000 (15:41 -0400)]
LP#1315552 Don't use circ_lib name in ranked_volumes

ranked_volumes() is supposed to return only volume info, but the name
returned was from copy.circ_lib, not acn.owning_lib, so it could result
in duplicate rows if circ_lib's varied across a single acn.

Let's just use the owning_lib name instead.  Note: the circ_lib join
is left undisturbed to keep the current sorting behavior
(necessary?).

Signed-off-by: Dan Wells <dbw2@calvin.edu>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
7 years agoLP1259196: Use Location as Workstation on Login
Jason Boyer [Fri, 6 May 2016 18:08:48 +0000 (14:08 -0400)]
LP1259196: Use Location as Workstation on Login

When using a SIPServer with LP1579144 applied, use the
Location (CP) field of the Login (93) message as a
workstation name. This allows you to determine which
selfcheck performed which circulation.

Signed-off-by: Jason Boyer <jboyer@library.in.gov>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
7 years agoCorrect upgrade script
Mike Rylander [Wed, 24 Aug 2016 14:49:41 +0000 (10:49 -0400)]
Correct upgrade script

Signed-off-by: Mike Rylander <mrylander@gmail.com>
7 years agoLP#1612274: Release notes for improved holds interfaces
Kathy Lussier [Fri, 19 Aug 2016 01:10:43 +0000 (21:10 -0400)]
LP#1612274: Release notes for improved holds interfaces

Signed-off-by: Kathy Lussier <klussier@masslnc.org>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
7 years agoLP#1612274: Add distinct classes for hold statuses
Kathy Lussier [Thu, 11 Aug 2016 17:24:00 +0000 (13:24 -0400)]
LP#1612274: Add distinct classes for hold statuses

With more data in the My Account hold status field, libraries may want to
style some statuses or dates differently so that they pop out to the user. Each
hold status and date now has its own class for more flexible styling.

The available status also now has a green, bold font and the suspended status
has a red, bold font.

Signed-off-by: Kathy Lussier <klussier@masslnc.org>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
7 years agoLP#1612274: Improve display of holds status in My Account
Kathy Lussier [Thu, 11 Aug 2016 03:41:51 +0000 (23:41 -0400)]
LP#1612274: Improve display of holds status in My Account

Collapse the data contained in the Activate, Active, and Date Fulfilled columns
into the status filed on the My Account holds and holds history interfaces.
This change cleans up the display and removes extraneous information that the
user doesn't need to see.

Signed-off-by: Kathy Lussier <klussier@masslnc.org>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
7 years agoLP#1612364: Add transparency to some format icons
Kathy Lussier [Sat, 20 Aug 2016 05:39:09 +0000 (01:39 -0400)]
LP#1612364: Add transparency to some format icons

Some format icons had a white background instead of a transparent one. The
transparent ones will look better in the My Account screens, where the icons
site on a gray background.

Signed-off-by: Kathy Lussier <klussier@masslnc.org>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
7 years agoLP#1614807: Make holds history more responsive
Kathy Lussier [Mon, 22 Aug 2016 10:10:14 +0000 (06:10 -0400)]
LP#1614807: Make holds history more responsive

When retrieving the holds history page on a mobile device, re-orient the table
as we do with other My Account screens.

Test plan
Pre-patch, retrieve the holds history screen for a patron that has enabled
holds history. Resize the screen to a width of 600 pixels or smaller. No
adjustments will be made to the table, and horizontal scrolling is required to
see some data.

Post-patch, after resizing this screen, the table should reorient itself with
the headers displaying in the left column, as is done on the other My Account
screens.

Signed-off-by: Kathy Lussier <klussier@masslnc.org>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
7 years agoLP#1614807: Fix spacing issues in responsive design for My Account screens
Kathy Lussier [Sat, 20 Aug 2016 04:54:40 +0000 (00:54 -0400)]
LP#1614807: Fix spacing issues in responsive design for My Account screens

This commit fixes the following spacing issues that are found in the My Account
screens when loaded on a small-screen device:

* When a field has no value, the text in the table header, which displays along
 the left, is cut off at the bottom. The Notes field on the Holds page is one
place where this issue is commonly seen. With the fix, the headers are slightly
out of alignment with the field data, but this is still an improvement over
cutting off text on the screen.

* If the only title(s) listed on the screen are short, requiring little width
in a table cell, the text in the table header runs into field data. A good
example in the Concerto data is if the user only has a check out for record ID
2 (Le concerto) on their account.

* The checkboxes in the Check Out History and Items on Hold screen were pushed
to the right. We fix it by giving the cells with a checkbox a "checkbox_column"
class, which was already in use by the checkbox on the Items Out screen.

Signed-off-by: Kathy Lussier <klussier@masslnc.org>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
7 years agoLP#1614807: Holds history should look like other My Account screens
Kathy Lussier [Fri, 19 Aug 2016 17:41:14 +0000 (13:41 -0400)]
LP#1614807: Holds history should look like other My Account screens

The Holds History My Account interface looks entirely different than the Items
Out, Check Out History, and Items On Holds interfaces. This commit adds a new
acct_holds_hist_header CSS class that shares the same styles as the CSS classes
used for the other three interfaces. Inline styles are removed from this table.

Test plan
Pre-patch: Go to the My Account Items Out, Check Out History, Items on Hold,
and Holds History interfaces. While the first three interfaces use a table with
a white background for the headers and a gray background for the body, the Holds
History interface is the opposite.

Post patch: All four interfaces should show a similar style to each other.

Signed-off-by: Kathy Lussier <klussier@masslnc.org>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
7 years agoLP#1614807: Fix Circ History table header display on small screens
Kathy Lussier [Fri, 19 Aug 2016 15:59:27 +0000 (11:59 -0400)]
LP#1614807: Fix Circ History table header display on small screens

Because the table in circs.tt2 and circ_history.tt2 shared the same CSS class,
circ_history.tt2 was displaying circs.tt2's column headers on mobile devices
and other small screens. The two tables now have distinct classes, and
styles.css.tt2 has been updated to provide the correct table headers for the
circ history page.

Test plan
Pre-patch, pull up a user account that has a circ history in My Account, go to
the Check Out History tab, then resize the screen to a width of 600 pixels or
smaller. The table headers will now display along the side, but the headers
will be the ones that typically display in the Items Out screen. The Barcode
will display in the Date Returned row and the Call number will display in the
Barcode row.

Post-patch, after resizing the screen, the headers that display along the side
should match the ones from the Checkout History interface. All data should
display in its appropriate row.

Signed-off-by: Kathy Lussier <klussier@masslnc.org>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
7 years agoLP1593834: Add Date Header to A/T Email Examples
Jason Boyer [Mon, 20 Jun 2016 16:38:00 +0000 (12:38 -0400)]
LP1593834: Add Date Header to A/T Email Examples

RFCs 822 and 2822 specify that a Date: header has to be sent
indicating when an email was sent. Without this, clients that
default to displaying the Sent Date may end up using a "NULL"
date, or 1970-01-01 00:00:00 UTC, which is not ideal. This
adds the header to the example A/T templates for email and
SMS messages, and includes instructions on correcting
existing tempalates.

Signed-off-by: Jason Boyer <jboyer@library.in.gov>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
7 years agoLP#1306666: Rename test file
Mike Rylander [Wed, 24 Aug 2016 12:11:11 +0000 (08:11 -0400)]
LP#1306666: Rename test file

Signed-off-by: Mike Rylander <mrylander@gmail.com>
7 years agoLP 1306666: Add Perl tests for new behavior.
Jason Stephenson [Mon, 28 Mar 2016 19:34:32 +0000 (15:34 -0400)]
LP 1306666: Add Perl tests for new behavior.

These tests check that the copy status is correct after aborting
transits dependent upon the status of the copy.  More tests could be
added to cover more situations, but these tests cover the basics.

Signed-off-by: Jason Stephenson <jstephenson@mvlcstaff.org>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
7 years agoLP 1306666: Abort Transit Only Change Copy Status if In Transit
Jason Stephenson [Fri, 18 Mar 2016 20:38:33 +0000 (16:38 -0400)]
LP 1306666: Abort Transit Only Change Copy Status if In Transit

When aborting a transit, the copy status is presently changed, either
to the status stored in action.transit_copy or to Reshelving if the
transit was for a hold.  This causes problems with "hanging" transits
where something was not handled properly before.  We've had cases
where a transit sticks around on a copy that is later checked out, and
while that copy is checked out, the transit is aborted, thus setting
the copy status to Reshelving when it should be Checked Out.

The code change in this branch only changes the copy's status if it is
In Transit.  A copy in any other status has its status left untouched.

Signed-off-by: Jason Stephenson <jstephenson@mvlcstaff.org>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
7 years agoLP#1613730: Stamping upgrade script for copy count badge
Kathy Lussier [Mon, 22 Aug 2016 18:02:58 +0000 (14:02 -0400)]
LP#1613730: Stamping upgrade script for copy count badge

Signed-off-by: Kathy Lussier <klussier@masslnc.org>
7 years agoLP#1613730: Add a "Copy Count" rating function for badges
Mike Rylander [Thu, 18 Aug 2016 16:58:25 +0000 (12:58 -0400)]
LP#1613730: Add a "Copy Count" rating function for badges

Signed-off-by: Mike Rylander <mrylander@gmail.com>
Signed-off-by: Kathy Lussier <klussier@masslnc.org>
7 years agoLP#1586221 - Remove "no spaces" message from login form.
Chris Sharp [Sat, 20 Aug 2016 18:52:15 +0000 (14:52 -0400)]
LP#1586221 - Remove "no spaces" message from login form.

Since the fix to bug 1086064 removes any spaces entered into
the login form, it is now inaccurate to require the user to type
their cards with no spaces (since most library cards have spaces
in the number on the physical card).

Signed-off-by: Chris Sharp <csharp@georgialibraries.org>
Signed-off-by: Kathy Lussier <klussier@masslnc.org>
7 years agoLP#1615025 TPAC large circ history sort repair
Bill Erickson [Fri, 19 Aug 2016 15:41:14 +0000 (11:41 -0400)]
LP#1615025 TPAC large circ history sort repair

Fix TPAC circ history retrieval code thinko so that history rows are
retrieved one at a time instead of in a batch (.atomic) to avoid
overwhelming Ejabberd.

Signed-off-by: Bill Erickson <berickxx@gmail.com>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
7 years agoLP#1613381: combine two tables in patron notification preferences
Galen Charlton [Mon, 15 Aug 2016 18:07:21 +0000 (14:07 -0400)]
LP#1613381: combine two tables in patron notification preferences

This patch consolidates the two tables in the public catalog
notification preferences page into one table, as patrons have
no reason to care about the distinction between A/T opt-in
settings and the hold, phone, and SMS user settings.

This patch also adds a <label> element for opt-in setting
names.

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
7 years agoLP#1476049: disable serve-cgi-bin Apache config on Jessie and Xenial
Galen Charlton [Wed, 17 Aug 2016 14:53:19 +0000 (10:53 -0400)]
LP#1476049: disable serve-cgi-bin Apache config on Jessie and Xenial

This patch ensures that the stock Apache configuration
"serve-cgi-bin" is disabled when installing on Debian Jessie
and Ubuntu Xenial, as otherwise the default cgi-bin location
can override what Evergreen specifies for offline.pl.

To test:

[1] Run the Makefile.install step on Jessie or Xenial; verify
    that there is no /etc/apache2/conf-enabled/serve-cgi-bin.conf
    symlink.
[2] Proceed with the rest of the Evergreen installation; verify
    that the Apache error log doesn't contain a warning like this:

    "The Alias directive in /etc/apache2/sites-enabled/eg.conf ...
     will probably never match because it overlaps an earlier
     ScriptAlias."

[3] For extra credit, verify that one can access offline circulation
    sessions.

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
7 years agoLP#1583729 Item status screen column options do not include age protection
Michele Morgan [Tue, 16 Aug 2016 18:47:43 +0000 (14:47 -0400)]
LP#1583729 Item status screen column options do not include age protection

Adds the age_protect field to several missing copy interfaces:

XUL client:

- Item status list view column picker
- Item status alternate view
- Holdings Maintenance column picker

Web client:

- Item status alternate view
- Holdings view column picker

Also addresses an issue with displaying the circ modifier in some item
interfaces.

Signed-off-by: Michele Morgan <mmorgan@noblenet.org>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
7 years agoLP#1527694 Webstaff egHatch supports 'LoginSession' storage
Bill Erickson [Fri, 5 Aug 2016 16:39:54 +0000 (12:39 -0400)]
LP#1527694 Webstaff egHatch supports 'LoginSession' storage

Adds support for a class of cached value (AKA "LoginSession" items)
that are cleared when either the user logs out or the browser is closed.
Values are stored as cookies.

Authentication tokens and "retrieve last patron" data are now stored as
"LoginSession" items.

Signed-off-by: Bill Erickson <berickxx@gmail.com>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
7 years agoLP#1527694 Webstaff clear last patron
Bill Erickson [Thu, 4 Aug 2016 16:34:56 +0000 (12:34 -0400)]
LP#1527694 Webstaff clear last patron

Store most recent patron ID via cookie instead of localStorage so the
value can expire when then browser is closed.

Adds angular-cookies (ngCookies) dependency.

Signed-off-by: Bill Erickson <berickxx@gmail.com>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
Conflicts:
Open-ILS/web/js/ui/default/staff/bower.json

7 years agoLP#1522635 Webstaff lost (etc.) checkout completes
Bill Erickson [Thu, 4 Aug 2016 21:56:32 +0000 (17:56 -0400)]
LP#1522635 Webstaff lost (etc.) checkout completes

Resolves an issue in the browser client where checkout of LOST
(and other non status='checked out' copies) failed to show the
open-circ-exists dialog.  In short, when searching for an existing
open circulation, look for any open circulation linked to the copy,
regardless of the status of the copy.

Signed-off-by: Bill Erickson <berickxx@gmail.com>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
7 years agoLP#1464350 Webstaff home page catalog search
Bill Erickson [Thu, 4 Aug 2016 19:22:22 +0000 (15:22 -0400)]
LP#1464350 Webstaff home page catalog search

Search the catalog directly from the home/splash page in the browser
client.

A side effect of this commit is that it's now possible to jump directly
to a set of search results in the embedded catalog by going to:

/eg/staff/cat/catalog/results?<query-params>

E.g.

/eg/staff/cat/catalog/results?query=scores&qtype=subject

Signed-off-by: Bill Erickson <berickxx@gmail.com>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
7 years agostart adding form validation to receipt template editor
Galen Charlton [Wed, 10 Aug 2016 16:10:58 +0000 (12:10 -0400)]
start adding form validation to receipt template editor

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
7 years agoIgnore null fleshed objects in autofleshing grid columns
Mike Rylander [Mon, 8 Aug 2016 21:05:10 +0000 (17:05 -0400)]
Ignore null fleshed objects in autofleshing grid columns

Signed-off-by: Mike Rylander <mrylander@gmail.com>
7 years agoRepair output popup, and add Reports to the splash page
Mike Rylander [Mon, 8 Aug 2016 19:09:43 +0000 (15:09 -0400)]
Repair output popup, and add Reports to the splash page

Signed-off-by: Mike Rylander <mrylander@gmail.com>
7 years agoReport Templates!
Mike Rylander [Mon, 8 Aug 2016 18:15:09 +0000 (14:15 -0400)]
Report Templates!

Signed-off-by: Mike Rylander <mrylander@gmail.com>
7 years agoLet the legacy interface do the right thing when embedded in the web client
Mike Rylander [Mon, 8 Aug 2016 18:10:32 +0000 (14:10 -0400)]
Let the legacy interface do the right thing when embedded in the web client

Signed-off-by: Mike Rylander <mrylander@gmail.com>
7 years agotoward ou type dropdown
Jason Etheridge [Mon, 11 Jul 2016 19:57:48 +0000 (15:57 -0400)]
toward ou type dropdown

7 years agochild node creation
Jason Etheridge [Mon, 11 Jul 2016 18:52:53 +0000 (14:52 -0400)]
child node creation

Signed-off-by: Jason Etheridge <jason@esilibrary.com>
7 years agochange to delete to remove, since JS uses delete
Jason Etheridge [Mon, 11 Jul 2016 15:59:06 +0000 (11:59 -0400)]
change to delete to remove, since JS uses delete

Signed-off-by: Jason Etheridge <jason@esilibrary.com>
7 years agoUI tweak and org deletion
Jason Etheridge [Mon, 11 Jul 2016 15:25:00 +0000 (11:25 -0400)]
UI tweak and org deletion

Signed-off-by: Jason Etheridge <jason@esilibrary.com>
7 years agotoast, comments, logging
Jason Etheridge [Mon, 11 Jul 2016 14:20:43 +0000 (10:20 -0400)]
toast, comments, logging

Signed-off-by: Jason Etheridge <jason@esilibrary.com>
7 years agoprogrammatic selection of tree node still not working
Jason Etheridge [Fri, 10 Jun 2016 18:22:45 +0000 (14:22 -0400)]
programmatic selection of tree node still not working

Signed-off-by: Jason Etheridge <jason@esilibrary.com>
7 years agorefactor
Jason Etheridge [Fri, 10 Jun 2016 18:17:13 +0000 (14:17 -0400)]
refactor

Signed-off-by: Jason Etheridge <jason@esilibrary.com>
7 years agogetting changes to stick within a session
Jason Etheridge [Fri, 10 Jun 2016 18:16:22 +0000 (14:16 -0400)]
getting changes to stick within a session

Signed-off-by: Jason Etheridge <jason@esilibrary.com>
7 years agobetter stub out the other tabs, and try to select CONS by default
Jason Etheridge [Fri, 10 Jun 2016 16:37:01 +0000 (12:37 -0400)]
better stub out the other tabs, and try to select CONS by default

Signed-off-by: Jason Etheridge <jason@esilibrary.com>
7 years agoToward an Angular replacement for Org Units Conify
Jason Etheridge [Wed, 11 May 2016 19:17:47 +0000 (15:17 -0400)]
Toward an Angular replacement for Org Units Conify

Signed-off-by: Jason Etheridge <jason@esilibrary.com>
7 years agowebstaff: add admin interface for MARC tag tables
Galen Charlton [Fri, 10 Jun 2016 22:21:31 +0000 (18:21 -0400)]
webstaff: add admin interface for MARC tag tables

This is a simple interface using eg-edit-fm-record to start;
more work will be required to better deal with the semantics
for overriding tag definitions at various levels of the OU
hierarchy.

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
7 years agowebstaff: a couple strings for a generic record deletion dialog
Galen Charlton [Fri, 10 Jun 2016 22:19:17 +0000 (18:19 -0400)]
webstaff: a couple strings for a generic record deletion dialog

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
7 years agowebstaff: new directive: egEditFmRecord
Galen Charlton [Fri, 10 Jun 2016 22:14:20 +0000 (18:14 -0400)]
webstaff: new directive: egEditFmRecord

This implements a generic IDL record editor widget:

<eg-edit-fm-record
  idl-class            = "xyz"
  mode                 = "update"
  record-id            = "223"
  hidden-fields        = "bar,baz"
  readonly-fields      = "quux"
  required-fields      = "foo"
  is-required-override = "bundle_of_custom_functions"
  on-save              = "on_save_handler"
  on-cancel            = "on_cancel"
></eg-edit-fm-record>

The mode can be either "create" or "update"; if it is "create",
then it is not necessary or desired to pass a record-id.

Currently eg-edit-fm-record expects to be invoked from
inside a uibModal.

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
7 years agoIDL improvements for classes releated to MARC tag tables
Galen Charlton [Fri, 10 Jun 2016 16:15:18 +0000 (12:15 -0400)]
IDL improvements for classes releated to MARC tag tables

In the course of building the admin interface for the
MARC tag tables, some deficiencies in the IDL were
run across. This patch fixes them.

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
7 years agowebstaff: fix typo in field name
Galen Charlton [Fri, 13 May 2016 15:51:42 +0000 (11:51 -0400)]
webstaff: fix typo in field name

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
7 years agoLP#1585369: Angular broke my copy editor!
Mike Rylander [Wed, 25 May 2016 17:14:44 +0000 (13:14 -0400)]
LP#1585369: Angular broke my copy editor!

It seems Angular 1.5 is either less flexible or less forgiving in its
ng-repeat implementation, and does not like an orderBy filter on object
iterators.  Removing that, and the track by clause, allows the holdings
editor to render properly.

Signed-off-by: Mike Rylander <mrylander@gmail.com>
7 years agowebstaff: circ audio alerts
Bill Erickson [Wed, 18 May 2016 02:09:34 +0000 (22:09 -0400)]
webstaff: circ audio alerts

Signed-off-by: Bill Erickson <berickxx@gmail.com>
7 years agowebstaff: audio disable and testing options
Bill Erickson [Tue, 17 May 2016 03:16:34 +0000 (23:16 -0400)]
webstaff: audio disable and testing options

For workstation admin UI.

Signed-off-by: Bill Erickson <berickxx@gmail.com>
7 years agowebstaff: egAudio HTML5 audio service
Bill Erickson [Sun, 15 May 2016 16:51:24 +0000 (12:51 -0400)]
webstaff: egAudio HTML5 audio service

egCore.audio.play('audio.event.dot.path');

Service to look up audio URL's by key name.  Supports fall-thru behavior
where 'foo.bar.baz' will fall-thru to 'foo.bar' and 'foo' depending on
whether an audio file is avaialable.

Signed-off-by: Bill Erickson <berickxx@gmail.com>
7 years agoLP#1581126: webstaff: make egDateInput respect format.date OUS
Galen Charlton [Thu, 12 May 2016 16:27:11 +0000 (12:27 -0400)]
LP#1581126: webstaff: make egDateInput respect format.date OUS

This patch makes the egDateInput directive fetch the
date format from the format.date library setting. The
directive also now accepts a dateFormat attribute for cases
where there is a reason to override the library setting.

If no format is set via library setting or in how the
directive is invoked, the format defaults to "mediumDate",
e.g., "May 2, 1999".

To test:

[1] Open the webstaff patron registration form. Verify that
    date widgets display the date in the format that
    corresponds to the value of the format.date library setting
    for the current work station, or (if the library setting
    is not set, "Month day, year".

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
7 years agoAdd images for angular-tree-control to bower
Jason Etheridge [Thu, 12 May 2016 15:51:09 +0000 (11:51 -0400)]
Add images for angular-tree-control to bower

Signed-off-by: Jason Etheridge <jason@esilibrary.com>
7 years agowebstaff: tweak legacy OU editor
Galen Charlton [Wed, 11 May 2016 20:31:33 +0000 (16:31 -0400)]
webstaff: tweak legacy OU editor

This adds a CSS hack so that the right-hand pane of
the legacy OU editor is displayed when embedded in
the web staff client.

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
7 years agowebstaff: tweak permission group editor
Galen Charlton [Wed, 11 May 2016 20:30:34 +0000 (16:30 -0400)]
webstaff: tweak permission group editor

This adds a couple hackish formating tweaks so that
the permission group editor works when embedded
in the web staff client.

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
7 years agowebstaff: add server administration page
Galen Charlton [Wed, 11 May 2016 17:17:55 +0000 (13:17 -0400)]
webstaff: add server administration page

This patch adds a landing page for Server Administration and
code to embed all of the admin pages available under Serer
Administration in the XUL staff client.

TODO:

* fix the permission groups editor so that it works
  when embedded in the web staff client
* refactor admin/local/app.js and admin/server/app.js to
  unify some of the copy and pasting
* replace the embedded legacy OU editor with a new
  angular one
* fixes some CSS issues that make text hard to read in
  a few places

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
7 years agowebstaff: work log: various improvements
Galen Charlton [Wed, 4 May 2016 19:04:23 +0000 (15:04 -0400)]
webstaff: work log: various improvements

* Don't use $location.path() to generate a URL for when opening
  a new window, as that can cause the parent window to reset
  itself.
* Fix the refresh button
* Add support for paging through worklog entries
* Don't prematurely resolve the promise that is
  feeding rows to the grid
* Ensure that entries are displayed in timestamp order

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
7 years agowebstaff: egWorkLog service and Work Log UI
Jason Etheridge [Fri, 4 Dec 2015 16:06:56 +0000 (11:06 -0500)]
webstaff: egWorkLog service and Work Log UI

under Administration -> Local Administration

The original XUL feature starts here: 29d1b357eef061bb3698e4ce0506eb93b63421be

Make sure egCore from the calling interface is pulling in these org unit
settings:

ui.admin.work_log.max_entries
ui.admin.patron_log.max_entries

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