Merge remote-tracking branch 'conifer/feature/tpac_boreal_2_7' into rel_2_7_mergery_tpac

Slight tweaks to Boreal skin for 2.7

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

Merge branch 'tpac_algoma_2_7' into rel_2_7_mergery_tpac

Make use of the brave new css overrides

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

Merge branch 'tpac_conifer_2_7' into rel_2_7_mergery_tpac

Add style_overrides.css to the OPAC base

This will enable sites to minimize the differences in their styles
and the core styles, at the cost of an extra HTTP request.

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

Merge remote-tracking branch 'conifer/feature/tpac_algoma_2_7' into rel_2_7_mergery_tpac

Minimize rel_2_7 differences with Algoma skin

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

Merge branch 'feature/circ_scripts_2_7' into rel_2_7_mergery

MEDIACEN doesn't charge anything on AV-EQUIP items

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

Add 3-day loan rule for AV-EQUIP for media centre

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

Prevent renew if item already is targeted for a hold

From http://biblio.laurentian.ca/tickets/conifer/wiki/devHolds
apparently I missed this in moving our customizations from SVN (or it
was only ever local?)

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

OSBO: Accidental copying of the "DVD VIDEO" rule

Meant to remove the "DVD" from the "DVD VIDEO" part of the circ modifier
the second time around.

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

OSBO really did want $0.25/hour for all reserve periods

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

Add Reserve circ rules for Boreal

Still need some clarification on how these rules are to be applied, but
better to get the durations working right now and fix up the fines
later.

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

Add LEDL MISC to the list of allowed circ modifiers for Leddy

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

Add ILL-BOOK to the list of allowed circ modifiers for Leddy

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

Add LEDL MISC 1 year 0 renew rule

This had been working in production, but apparently never got
checked into git?

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

Add Boreal circulation rules

Still need to add their limits based on user profile & item type

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

Add ILL-BOOK circ rule for OWA

Per ticket #683:

Loan period of 3 weeks
NO RENEWALS

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

Track Conifer JavaScript circulation rules

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

Merge branch 'rel_2_7_mergery' of git.evergreen-ils.org:contrib/Conifer into rel_2_7_mergery_tpac

Merge remote-tracking branch 'conifer/feature/ldap_osul_2_7' into rel_2_7_mergery

Merge remote-tracking branch 'conifer/feature/tpac_osul_2_7' into rel_2_7_mergery_tpac

Merge branch 'feature/tpac_osul_2_7' of git.evergreen-ils.org:contrib/Conifer into feature/tpac_osul_2_7

There is no password hint for Laurentian

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

libyaz-dev needs to be specifically 4 or 5

If you're using yaz from Indexdata to avoid the Z39.50
server errors that you get with the packaging from Ubuntu,
then you need to specify either libyaz4-dev or libyaz5-dev.
We were working with libyaz5-dev, so roll with that.

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

Merge remote-tracking branch 'remotes/conifer/feature/tpac_hide_awards_2_7' into mergery_2_7

Merge remote-tracking branch 'remotes/conifer/feature/move_to_storage_2_7' into mergery_2_7

Merge remote-tracking branch 'remotes/conifer/feature/marctemplates_2_7' into mergery_2_7

Merge remote-tracking branch 'remotes/conifer/feature/barcode_generator_2_7' into mergery_2_7

Merge remote-tracking branch 'remotes/conifer/feature/tpac_eresources_highlander_2_7' into mergery_2_7

Merge remote-tracking branch 'remotes/conifer/feature/patron_privacy_2_7' into mergery_2_7

Merge remote-tracking branch 'remotes/conifer/feature/fts-polish-l_2_7' into mergery_2_7

Merge remote-tracking branch 'remotes/conifer/feature/tpac_show_details_config_2_7' into mergery_2_7

Merge remote-tracking branch 'remotes/conifer/features/2.4-to-2.7' into mergery_2_7

Prefer a metabib reingest that is not a single transaction

At least this way we can keep the doors open for business while the reingest
goes on.

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

Minor fix to web client section headers in root.txt

I accidentally added the circulation header twice when adding the new
sections for web client docs. This commit removes one of the headers.

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

Web client section headings

Adding section headings for web client documentation, which will be coming
shortly.

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

Docs: Update to template receipt docs

Thanks to Lynn Floyd. When adding the new content I ended up splitting new and old content into three files.

Signed-off-by: Yamil Suarez <yamil@yamil.com>

Avoid ambiguous function references in upgrade script

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

Funkiness around 0887 which appears to be missing from 2.6-2.7.0 upgrade

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

Apply fix for "Could not find trigger" (LP 1261355)

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

Overcome metabib.record_attr_vector_list missing

Perhaps the transaction is too long? Anyway, commit and move on.

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

Sync up OSUL skin with base templates

Looks like, in many cases, the base skin now has what we were
carrying along as customizations. Sweet!

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

First rough 2.4 - 2.7 database upgrade for Conifer

Nothing special, you know.

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

LP#1207529: Add /openils assumption note

Signed-off-by: alzr <aleksey.lazar@gmail.com>
Signed-off-by: Galen Charlton <gmc@esilibrary.com>

LP#1207529: Make sure $PATH includes /openils/bin when configuring

https://bugs.launchpad.net/evergreen/+bug/1207529

Signed-off-by: alzr <aleksey.lazar@gmail.com>
Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Conflicts:
docs/installation/server_upgrade.txt

LP#1384932: document the zips.txt ZIP code database feature

Patron Address City/State/County Pre-Populate by ZIP Code.

Documented the zips.txt feature that auto fills patron address
(City, State, County) information.

Signed-off-by: Josh Stompro <stomproj@larl.org>
Signed-off-by: Galen Charlton <gmc@esilibrary.com>

LP#1246371: Allow BibCommon::title_is_empty to accept a bre id or bre object.

I was trying to delete copies with the record fleshed in the call number
object and got the following error:

Can't use an undefined value as an ARRAY reference at
/usr/local/share/perl/5.14.2/OpenILS/Application/Cat/BibCommon.pm line 399

Having title_is_empty check for a bre object and use its id solved this
problem for me.

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

LP#1083639: use "cp -r" instead of "cp -R"

This change is purely for the sake of consistency; there's
no functional difference.

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

LP#1083639 - Added command to copy fonts into the KPAC2 / Alternate monster skin dir

Signed-off-by: Josh Stompro <stomproj@larl.org>
Signed-off-by: Galen Charlton <gmc@esilibrary.com>

LP#1116387 - adding kpac setup notes.

Revised instructions based on my testing.  Swapped around the Monster
and Default skin discriptions since the listed Alternate Skin is the default skin.

Signed-off-by: Josh Stompro <stomproj@larl.org>
Signed-off-by: Galen Charlton <gmc@esilibrary.com>

Add KPAC configuration info to the community docs

Adds the KPAC TechRef docs to root.txt so that this information can be
available via the community docs. Also adds a section to the docs on
configuring the alternate skin and removes a reference to configuration
from the end-user KPAC docs.

Signed-off-by: Kathy Lussier <klussier@masslnc.org>
Signed-off-by: Galen Charlton <gmc@esilibrary.com>

LP#1246839: marc_stream_importer.pl no longer crashes with vs 0.23 of File::Temp

updated marc_stream_importer.pl to use the 0.23 API

Signed-off-by: Fredric T Parks <fparks@catalystitservices.com>
Signed-off-by: Bill Erickson <berickxx@gmail.com>
Signed-off-by: Galen Charlton <gmc@esilibrary.com>

Basic TPAC Boreal branding: colours and logo and links

Link from logo to College Boreal itself
Remove "Tembec" from name, hide old OPAC link
Change links to union catalogue for Boreal tpac skin

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

Basic Laurentian University branding: colours and logos

Required some wrestling with the default stylesheet but otherwise we
appear to be good.

Phone notifications for holds do not happen. Hide any mention of phone
numbers or widgets for changing phone numbers.

We also do not want to enable users to update their username, password, or
email address.

Override style.css in a separate stylesheet

One more HTTP request, but much simpler tracking of HEAD stylesheet
changes.

Customize login form for Laurentian LDAP login

Set ou_email_host via a hidden form variable.

Provide more rational inline help for OSUL login form.

Link to the OSUL password reset form on login failure

Avoid the "place holds on available" lists hole by ripping out the option
entirely, naturally.

Turn on Google Book Previews

Hide the display of 659 fields in Laurentian records. They appear to
have been used for some sort of identifiers, not genres, and are
meaningless to users.

Remove the "Stay logged in" checkbox until we uncover the actual problem
with logging in via LDAP and persistence.

Enable Google Analytics in Laurentian skin

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

First pass at Algoma U tpac skin for rel_2_7

This was done in a hurry and simply handles the esential links, logos and branding.

 *changed the main logo on the homepate
 *changed the header links to reflect new library website
 *added footer links
 *added config file based on lul's to enable OpenURL

In addition to adding the same graphical header so that the OPAC
more closely resembles the new Wishart website, this change
incorporates several smaller fixes:
    * replace the AlgomaU "small university/big education" logo
      with the small AlgomaU wordmark
    * center the logo via CSS, not <center>
    * fix vertical centering of the "your account log in" text --
      needs margins only left & right, not on all 4 sides
    * replace border on the user/pass input boxen so that they
      are visible once again.

Forbid holds on items that are currently available

Represents a merge of all of the 2.3 and 2.4 commits.

TPAC: Make "Show more details" in results optional

Academic sites typically handle a lot of electronic content and want to
bring most details about entries, such as availability, status, and
URLs, on a results screen to the front.

This adds a 'show_more_details.default' option in 'config.tt2' that can
have the values 'true', 'false' and 'hide'.  The 'true' and 'false'
values set the default for the button to more and less respectively, and
'hide' suppresses the button from showing at all.

Signed-off-by: Art Rhyno <art632000@yahoo.ca>
Signed-off-by: Dan Scott <dscott@laurentian.ca>
Conflicts:
Open-ILS/src/templates/opac/parts/searchbar.tt2
Open-ILS/src/templates/opac/results.tt2

TPAC: Hide Awards and Added Content always

Awards can only ever be displayed if you have Novelist or Chilifresh
patron reviews activated, but are currently always displayed. Which is
annoying. A better approach would be to display this only if you have
Novelist or Chilifresh activated, but as we at Conifer do not have these
across the board, always hide the tab.

Similarly, the added content tab appears to generate plenty of false
positives, so just hide it for now rather than irritate people.

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

TPAC: Avoid showing URIs from 856 if we have OpenURL results

There is already a lot of logic to deduplicate OpenURL results versus
URIs in 856 fields, but in practice URIs in 856s tend to be subject to
bitrot, while SFX generally should be trustworthy. Thus, don't generate
electronic resources for 856s if we have OpenURL results.

Another optimization might be to hide the 856 results in case SFX is
wrong, then give users the ability to show them, but that might be
something we only want to enable in the staff client...

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

Remove display of patron birthdate, internet access level, etc

These fields in the staff client are either not applicable or
potentially privacy-invasive or both. If the data somehow does
get into the system, we don't want it to show up on the staff
client anyway - and we don't want annoying warnings about unset
dates of birth, etc.

This is the hardcore way of doing things, we really should make
it yet another configuration setting. Oh well.

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

Add moveto templates for LAL Circulation

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

Quick delete UI

For mass deletion efforts, go to http://hostname/eg/cat/moveto/delete in
a web browser and start scanning barcodes. Each barcode will be deleted
immediately. Note that call numbers and bib records will not be deleted
if the copy was the last one, but the records will no longer show up in
search results for regular patrons... and one could always follow up
with a clean up SQL job to delete call numbers and records where
appropriate.

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

Add production "moveto" options

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

Move the moveto to the new moveto template place

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

Add "Move to storage" quick UI for bulk location moves

This simplistic interface accepts a barcode and immediately returns
some bibliographic information if the barcode was found - and the
item is moved immediately to the specified location. Goal was to
minimize clicking, this pretty much requires just scan scan scan.

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

Add Media Centre MARC template

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

Flag the UTF8 encoding for MARC templates

The UTF8 encoding is flagged by LDR09 being 'a'. If this is not set, it
greatly confuses Evergreen, which tries to convert what it thinks are
MARC8 characters to UTF8 before saving the record.

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

Add Conifer's custom MARC templates

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

"next" doesn't work if you're not in a for loop

This was causing grievous "Internal server error" problems
when people actually did enter their email address. More
robust, more better. Yay.

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

At Laurentian, all usernames will be lowercase

This will prevent mismatches like the user entering "dscott" and the database
having the value "DScott" (in that we're going to force all of the Laurentian
user names in the database to lower case).

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

Add an ou_host_name parameter for TPAC login forms

If set, and the incoming username does not already include an '@' symbol
(a very simple attempt to detect if we're already dealing with an email
address), then append '@' + the ou_host_name value to the username for
authentication purposes.

The rationale is that in a large consortium, you might want to enable
users to log in with short usernames (like 'fred'), but you also want to
avoid conflicts between short usernames at different organizational
units. Thus, create the users with the email equivalent of their
usernames, like 'fred@br1.example.com' and 'fred@br4.example.com', and
let the templates for the TPAC in br1 and br4 contain a hidden input
field to append the appropriate email hostname.

Truth be told, this is probably most appropriate for a large consortium
containing two or more academic institutions that hope to use LDAP
authentication rather than native authentication, and therefore have
LDAP CNs that map to email addresses of CN@hostname that can then be
mapped to actor.usr.usrname (and actor.usr.email, of course).

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

Very rough LDAP authentication enablement for OSUL

Currently requires that the user enters their email address at the
username prompt; we then just use the local-part of the email address to
authenticate against the LDAP server.

In the VirtualHost sections of Apache, we can use SetEnv to force the
physical_loc to match the org unit ID(s) specified in opensrf.xml; for
example:

SetEnv physical_loc 103

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

Treat a Polish l as an ASCII l in full-text search

Is this code path even hit these days? Would have expected
naco_normalize to be poking around here if that was the case...

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

Generic patron barcode generation (OpenSRF and DB)

Laurentian University needed the ability to generate barcodes as part of
its LDAP integration work, and the first generation (so to speak) of the
was specific to LU - including hard-coded prefixes and database
functions that include the "lu" name.

This commit makes the functionality much more generic and thus more
likely to be able to be adopted by other institutions. The principle
components are:

Database functions:

evergreen.actor_generate_barcode([prefix TEXT]) - returns a 14-digit
  barcode from the evergreen.actor_barcode_seq sequence with a prefix of
  'AUTOBC' or the specific prefix of up to 6 characters. If the
  resulting barcode is all digits, then the 14th character will be a
  mod10 check digit; otherwise the 14th digit will be '0'.

evergreen.actor_update_barcode(usr_id INTEGER[, prefix TEXT]) -
  generates a new barcode for the specified user, with the optional
  barcode prefix.

evergreen.mod10(barcode TEXT) - given a barcode, generates a mod10
  check digit and returns the barcode with the appended check digit

OpenSRF method:

open-ils.actor.generate_patron_barcode([usr_id INT[, prefix TEXT]]) -
  generates a new barcode for the patron

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

Add and use a patron search that overrides opt-in invisibility

Something like this is required for API calls that need to operate
against a number of libraries in a given instance that are using opt-in;
otherwise, attempts to search for users will fail and you may end up
creating near-duplicates etc.

The implementation adds an
open-ils.actor.search.patron.advanced.opt_in_override method to
open-ils.actor, which, if invoked, checks to see if the caller has the
OPT_IN_OVERRIDE permission. If so, then the crazy_search ignores the
normal opt-in limits and searches all pertinent users in the database.

As a global permission, OPT_IN_OVERRIDE is a blunt instrument. Others
might want to put together a more refined version that uses OU depths to
define boundaries.

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

LP#1133158 - Fix typos in action_trigger_runner.pl

Added equals sign to --hooks and added --process-hooks to command so
--hooks=x is actually used.

Signed-off-by: Josh Stompro <stomproj@larl.org>
Signed-off-by: Ben Shum <bshum@biblio.org>

LP#1391290: Stamping upgrade script for authority reingest setting

Signed-off-by: Ben Shum <bshum@biblio.org>

LP#1391290: Respect setting to disable authority propagation on reingest

In commit a1612cfd6 we lost the ability to disable authority propagation on
record update.  Here, we put it back.

Signed-off-by: Chris Sharp <csharp@georgialibraries.org>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
Signed-off-by: Ben Shum <bshum@biblio.org>

LP#1203734 copy circ counts include anon. circs

Circ counts displayed on "Show Last Few Circulations" (XUL) includes
active, anonymized, and legacy circulations (via cstore)

Signed-off-by: Bill Erickson <berickxx@gmail.com>
Signed-off-by: Chris Sharp <csharp@georgialibraries.org>

LP#778989: Stamping upgrade script for copy location trigger

Signed-off-by: Ben Shum <bshum@biblio.org>

LP#778989: Adding DROP TRIGGER IF EXISTS to allow for smooth upgrades.

Signed-off-by: Chris Sharp <csharp@georgialibraries.org>
Signed-off-by: Ben Shum <bshum@biblio.org>

LP#778989: Add circ lib to location fixer

Because the circ lib is valid for picking locations.

Signed-off-by: Thomas Berezansky <tsbere@mvlc.org>
Signed-off-by: Chris Sharp <csharp@georgialibraries.org>
Signed-off-by: Ben Shum <bshum@biblio.org>

LP#778989: Attempt to find "Correct" copy location

This adds a trigger to asset.copy that on insert or change of the location
and/or call number tries to find an identically named copy location that
is scoped to the owning library of the copy.

If it does not find one it uses the location provided.

Signed-off-by: Thomas Berezansky <tsbere@mvlc.org>
Signed-off-by: Chris Sharp <csharp@georgialibraries.org>
Signed-off-by: Ben Shum <bshum@biblio.org>

Docs: remove extra '\r' in cataloging/MARC_batch_edit.txt

Signed-off-by: Yamil Suarez <yamil@yamil.com>

Docs: Documentation for MARC Batch Edit

Signed-off-by: Yamil Suarez <yamil@yamil.com>

Documentation: LP#1386854 - Locally Hosted Added Content.

Adds documentation that describes how to use the local content bypass
feature of the AddedContent modules.  This allows for local added content like
cover art, reviews, TOC, exerpts or annotations to be shown in bib records.

I also changed the headers in the External Added Content section to use title case.

Signed-off-by: Josh Stompro <stomproj@larl.org>
Signed-off-by: Yamil Suarez <yamil@yamil.com>

LP#1390225: Fail to care about errors from auth.session.delete

We're just tossing this call over the wall and moving on.  We don't
care if the session wasn't there to delete or even if we didn't pass
a session to the server.

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

LP#1390225: redirect to ctx.home_page instead of through ctx.logout_page

This way, we skip the auth.session.delete call that will error out because
we no longer have the authtoken cookie.

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

Forward port 2.7.0-2.7.1 upgrade script

Signed-off-by: Ben Shum <bshum@biblio.org>

LP#1248636: Clear session at OPAC logout

When the user explicitly logs out, clear the session on the server side.

Signed-off-by: Mike Rylander <mrylander@gmail.com>
Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Signed-off-by: Ben Shum <bshum@biblio.org>

Forward port 2.6-2.7 upgrade script

Signed-off-by: Ben Shum <bshum@biblio.org>

LP#1366964 Update libdbi connection test error parsing

Modern version of libdbi return a slightly different error message for
queries which fail mid-transaction, causing an C-based DB backends to
exit abruptly unnecessarily.  Update the error message test to include
the new format.

Includes Perl live test for confirming functionality.

Signed-off-by: Bill Erickson <berickxx@gmail.com>
Signed-off-by: Jason Stephenson <jstephenson@mvlc.org>

LP1383697: Long Overdue needs to be a "magical status" for the copy editor

Signed-off-by: Chris Sharp <csharp@georgialibraries.org>
Signed-off-by: Ben Shum <bshum@biblio.org>

lp1383763 - Display OpenURL embargo statement

The ResolverResolver service fetches the embargo statement (such as
"Last year not available") from OpenURL resolvers and stores it in a
target_embargo field. In JSPAC we used to display this along with the
coverage statement, because it is very important that people know that a
resource may be available from 1999 but the last year isn't available
online.

Restore the display of this information to the TPAC.

Signed-off-by: Dan Scott <dscott@laurentian.ca>
Signed-off-by: Galen Charlton <gmc@esilibrary.com>

Docs: Changed 'Importing materials' in the staff client section LP#1371615

Signed-off-by: Yamil Suarez <yamil@yamil.com>

Docs: add default icon definitions LP#1371598

Documents the default icon definitions that are included with a stock Evergreen install.

Signed-off-by: Yamil Suarez <yamil@yamil.com>

LP#1384868: limit fund drop-downs on the invoice page to only active funds

This patch fixes the fund selectors for adding extra copies
and charges to invoices to only active funds (in addition
to the existing restriction to funds visible to the user).

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Signed-off-by: Bill Erickson <berickxx@gmail.com>

LP#1261486 Action/trigger aggregator script repairs

* If both date filters are selected, apply both instead of replacing the
  first with the second

* Select distinct event output to prevent dupes

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