Allowing unfillable hold placement in the staff client, not the OPAC

In PINES, we do not allow patrons to override hold policies, but staff
may in certain circumstances.  This change makes the option visible in
the staff client, but not the OPAC.

Small changes to Search Limit names and Category names.

Further tweaks to search limits and modifications to book bag links.

Testing modifications to navigation.

Modifications to quick search functions and stylesheet.

Conflicts:

Open-ILS/web/css/skin/default/kpac/style.css

Image handling has changed in the new version...

Removing PINES-customized code that referred to a now-missing image.

Correcting git-conflict-induced typo in form.tt2

PINES GALILEO TPAC link and auth redirector

* adds a GALILEO link in the footer template
* adds a new TPAC URL target for /eg/opac/galileo, which redirects the
  user to the GALILEO URL, ensuring the user is first logged in.

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

Safe auth token activity logging

Expand safe token generation to include user user ID in the cached data,
which can be retrieved later for activity logging.  Add activity logging
directly to the open-ils.actor.safe_token.home_lib.shortname API.

Typo in footer.tt2 causing a server error.

Removing "kpac-original" directory in order to keep a clean source directory.

PINES customization of KPAC - beta ready for testing

Updated dewey.tt2

Initial commit of PINES KPAC customizations.

Conflicts:

Open-ILS/src/templates/kpac/home.tt2
Open-ILS/src/templates/kpac/parts/copy_table.tt2
Open-ILS/src/templates/kpac/parts/header.tt2
Open-ILS/web/css/skin/default/kpac/style.css

Removing "You have permission to override some of the failed holds." OPAC message

PINES is not yet keeping a history of circ or holds

Conflicts:

Open-ILS/src/templates/opac/myopac/prefs_settings.tt2

Changing the default hold message for when an item is age protected.

PINES Survey Link - TPAC Edition

PINES does an online survey that runs for a week every Spring. These
changes create a red link to the survey above the search bar on the
OPAC home page.

disabling visibility of hold shelf expiration date

Remove "Claims Never Checked Out" in Items Out summary

Signed-off-by: Chris Sharp <csharp@georgialibraries.org>

adding in gmcharlt's fix for trailing TCN spaces

Conflicts:

Open-ILS/src/perlmods/lib/OpenILS/Application/Cat/BibCommon.pm

adding new macros to listed ones

Adding GPLS Helpdesk link to Staff Client portal page and correcting a small typo

Conflicts:

Open-ILS/web/opac/locale/en-US/lang.dtd

re-adding STAFF_MIDDLENAME, PATRON_MIDDLENAME, PATRON_EXPIRE_DATE, and PATRON_EXPIRE_DATE_YMD to rel_2_3_4

Conflicts:

Open-ILS/xul/staff_client/chrome/content/util/print.js

Revert "Use my_asv here.. surveys pertinent to the workstation library.. in case survey usage ever takes off.  Should probably move some of this to the middle layer"

This reverts commit 54d45c37d91c78cffa00ca3ff0f6298261d548b2.

adding file that returns circ mod view to normal

adding mrpeters' awesome favicon

Changing default hold error messages.

removing hold/circ history view for patrons until we can review policy concerns

Conflicts:

Open-ILS/src/templates/opac/myopac/holds.tt2

adding PINES help links

Conflicts:

Open-ILS/src/templates/opac/parts/login/help.tt2

updating the username/password hints for the opac

Conflicts:

Open-ILS/src/templates/opac/parts/login/form.tt2
Open-ILS/src/templates/opac/parts/login/password_hint.tt2

improving the default hold message for range-protected items

Enabling the display of 856 field URLs for related resources

Password Length Helpers

This branch provides length helpers when patrons are setting or resetting
passwords.

This assumes the default 7 characters, 1 number requirement that comes stock.
You would need to change the hard coded strings if you use a different password
requirement regex in Library Settings Editor.

Signed-off-by: Michael Peters <mpeters@emeralddata.net>

Remove the hardcoded width of locg dropdown

The hard coded 12em width causes longer library names to be cut
off.  This removes that restriction.

Signed-off-by: Michael Peters <mpeters@emeralddata.net>

removing journal title and bib call number from query type selector dropdown

Conflicts:

Open-ILS/src/templates/opac/parts/qtype_selector.tt2

adding newly designed PINES logos

removing bib level advanced search filter

Conflicts:

Open-ILS/src/templates/opac/parts/config.tt2

adding "Georgia PINES" to the title

adding footer and topnav links and logo

Conflicts:

Open-ILS/src/templates/opac/parts/footer.tt2
Open-ILS/src/templates/opac/parts/topnav_logo.tt2

attempt to sort libraries by system in tpac results

Includes tsbere's reccomendations to change Record.pm because
it overrides th sorting in AppUtils.pm

Signed-off-by: Chris Sharp <csharp@georgialibraries.org>
Conflicts:

Open-ILS/src/templates/opac/parts/record/copy_table.tt2

Bumping version numbers, adding Upgrade Script and Changelog

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

Bump OpenILS.pm version

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

Update core committer information on the staff client's About Page

senator has not contributed anything new for the past six months and is
therefore moved to the "Past Core Committer" listing per policies on:
http://wiki.evergreen-ils.org/doku.php?id=contributing:contributors

berick belongs to a new organization and this has been updated accordingly.

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

LP#1234927: Pluralizing link names to follow convention.

Based on feedback from Mike Rylander on the Launchpad ticket.

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

LP#1234927 Reporter needs Item -> Combined Aged and Active Circulations link

There is a need for item usage statistics that include more than just active
circulations.  I have included links to both Aged (patronless) Circulations
and to Combined Aged and Active Circulations sources.

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

LP#1371173: Reporter: Classic Item List needs link to Item source

Add a link on the reporter.classic_item_list fieldmapper definition
to asset.copy.

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

More additions to 2.7 Release Notes (part 2)

Add Steve Callender and move Yamil Suarez to the correct position alphabetically.

Also, note the location of the web client README.install file, but also
promise to update this by the next minor release of 2.7 series as a new
section of the primary README file.

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

More additions to 2.7 Release Notes

Additions include a brief description of the web client preview, the
addition of Sprint 1 funding partners in the acknowledgements section,
the addition of NC Cardinal in the acknolwedgements section for
funding an acq funds transfer fix, and the addition of another code
contributor whose patch was just merged over the past few days.

Signed-off-by: Kathy Lussier <klussier@masslnc.org>
Signed-off-by: Ben Shum <bshum@biblio.org>

lp1271198 Strip browse and facet parameters from links

Browse and facet parameters were sticking to links to author, subject and
series links on record and result pages, as well as some other links where
the search parameters were no longer necessary. This commit:

1. Adds browse_search_parms and facet_search_parms to header.tt2 as new
classes of search parameters that one may want to remove.

2. Removes these parameters from the author and subject links on the
record page. It also uses the same method for removing search parameters
from the author link on the search results page and on the searchbar
Browse link.

3. Fixes an issue where the advanced search links were inadvertently
stripping general search parameters.

4. Adds more parameters to the general_search_parms group so that we can
have cleaner URLs.

Signed-off-by: Kathy Lussier <klussier@masslnc.org>
Signed-off-by: Jason Stephenson <jstephenson@mvlc.org>

LP 1319560: Another fix for summary.tt2.

Argument "" isn't numeric in numeric gt (>) at
/openils/var/templates/opac/parts/record/summary.tt2 line 56.

It was apparently coming from the .size and not where I thought.

Signed-off-by: Jason Stephenson <jstephenson@mvlc.org>
Signed-off-by: Ben Shum <bshum@biblio.org>

LP 1319560: Fix the disappearing hold button.

Staff discovered an issue with the tt2 changes that caused the
place hold button to disappear when it shouldn't. Changing an &&
back to || seems to fix this.

Signed-off-by: Jason Stephenson <jstephenson@mvlc.org>
Signed-off-by: Ben Shum <bshum@biblio.org>

LP 1319560: Silence template error messages in Apache error log.

We get a number of messages like the following in our Apache error
logs on a constant basis:

Argument "" isn't numeric in addition (+) at
/openils/var/templates/opac/parts/record/navigation.tt2 line 6.
Argument "" isn't numeric in division (/) at
/openils/var/templates/opac/results.tt2 line 1.
Argument "" isn't numeric in numeric ge (>=) at
/openils/var/templates/opac/parts/record/navigation.tt2 line 38.
Argument "" isn't numeric in numeric gt (>) at
/openils/var/templates/opac/myopac/holds/edit.tt2 line 106.
Argument "" isn't numeric in numeric gt (>) at
/openils/var/templates/opac/myopac/holds.tt2 line 42.
Argument "" isn't numeric in numeric gt (>) at
/openils/var/templates/opac/parts/place_hold.tt2 line 98.
Argument "" isn't numeric in numeric gt (>) at
/openils/var/templates/opac/parts/record/summary.tt2 line 56.
Argument "" isn't numeric in numeric gt (>) at
/openils/var/templates/opac/parts/result/table.tt2 line 87.
Argument "" isn't numeric in numeric lt (<) at
/openils/var/templates/opac/myopac/hold_history.tt2 line 122.
Argument "" isn't numeric in numeric lt (<) at
/openils/var/templates/opac/myopa/result/table.tt2 line 87.
Argument "" isn't numeric in numeric lt (<) at
/openils/var/templates/opac/myopac/hold_history.tt2 line 122.
Argument "" isn't numeric in numeric lt (<) at
/openils/var/templates/opac/myopac/holds.tt2 line 215.

This commit adds some sanity checking to make those messages go
away.

Signed-off-by: Jason Stephenson <jstephenson@mvlc.org>
Signed-off-by: Ben Shum <bshum@biblio.org>

LP#1251347: Get the most recent session locale for Event description translation

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

Fix LP904860, Cannot print a patron address label

The sequence of events for printing an address label is as follows.
Staff client Code in the util/print.js file builds an HTML page of the
address label.  The page includes two vital components, 1) a source tag
to pull 'print_win.js' file from the server, and 2) an onload event
handler bound to the body tag to execute a 'print_init()' function that
is defined in the source tag.

However, print_init() is undefined because the source tag contains a
badly formed URL. It needs to be prefixed by 'oils://remote' so that the
new 'oils:' protocol can be used to pull the file remotely from the
server.

Signed-off-by: Steven Chan <schan@sitka.bclibraries.ca>
Signed-off-by: Ben Shum <bshum@biblio.org>

LP#1231634: Mark Floating Groups Config UI strings translatable.

Signed-off-by: Pasi Kallinen <pasi.kallinen@pttk.fi>
Signed-off-by: Ben Shum <bshum@biblio.org>

LP#1331127 Repair sort logic of previous issuances

Ensure that the list of previous issuances is sorted correctly (on
date_published) when looking for the previous serial.unit to update its
copy location (when serial.prev_issuance_copy_location is enabled).

The data comes sorted from the DB, but the sorting was lost during the
process of unique-ifying the list.  Ultimately, it was relying on the
order of hash keys, which is undefined.

Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Dan Wells <dbw2@calvin.edu>

LP 1369128: Add delete permission to fm_IDL.xml for class bre.

Signed-off-by: Jason Stephenson <jason@sigio.com>
Signed-off-by: Ben Shum <bshum@biblio.org>

LP#1370630: don't log mod_deflate actions by default

This patch comments out logging of mod_deflate actions,
as this is needed only for debugging.  Otherwise,
deflate_log will grow without bound, as it isn't dealt
with by typical default logrotate configs.

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

LP#1361801: Add required fields to serial issuance form

This commit prevents issuances from being created without a date_published
or holding_code. This has two benefits:

  1. Prevents uncaught exceptions related to submitting null values for
     either of those fields.
  2. Requiring date_published prevents an ugly error when trying to
     predict new issuances from an issuance without a date_published.

This commit does NOT provide any feedback to the user about which fields
are required. That would require further investigation into the Dojo/Dijit
objects being used for the edit form fields.

Signed-off-by: Remington Steed <rjs7@calvin.edu>
Signed-off-by: Jennifer Pringle <jpringle@sitka.bclibraries.ca>
Signed-off-by: Ben Shum <bshum@biblio.org>

LP#1246859: Improve username flow for staged users

Staged users in most cases have a username that consists of a UUID.

Rather than leave the UUID as their username or require staff to
clear the field and replace it with a barcode or other value, it
would be better to remove this username when loading the staged user
in the user editor.

This commit clears the username when loading a stage user if the
username looks like a UUID.

If a barcode has been staged for the loaded staged user, the
username will be populated with that value.

If there is no staged barcode, the username will remain blank and
will be populated using the standard method after staff enters a
value in the barcode field.

Signed-off-by: Jeff Godin <jgodin@tadl.org>
Signed-off-by: Ben Shum <bshum@biblio.org>

LP#1356576 repair sip date_format config typo

Avoids "Use of uninitialized value in bitwise or (|)" error.

Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Ben Shum <bshum@biblio.org>

LP1309131 - Fix series link

Clicking on a series link with a hyphen failed because the hyphen was
stripped and the two words are joined together. This branch changes
series.tts to replace special characters with a space, which is similar to
the links we build for authors.

Signed-off-by: Kathy Lussier <klussier@masslnc.org>
Signed-off-by: Ben Shum <bshum@biblio.org>

LP#1325720 Repair fund rollover year selector query

Dojo dojo stores track values as strings, so values used in data store
queries must be coerced into strings before performing the query.

This fixes the year query in the fund rollover UI so that the page is
able to render the results of a rollover without dying from an JS error
before completing.

Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Ben Shum <bshum@biblio.org>

LP#1292129: Removed deleted call numbers from the search results to properly show copy count.

The SQL in fm_IDL.xml for setting up the reporter source "Hold/Copy
Ratio per Bib" includes deleted call numbers which in turn can give
an incorrect copy count. The ratio appears to still be correct because
deleted copies are considered in the ratio calculation, but not the
copy count that is displayed.

Signed-off-by: Steven Callender <stevecallender@esilibrary.com>
Signed-off-by: Ben Shum <bshum@biblio.org>

LP 1350345: marc_export blows up on bad records.

Add some eval blocks around calls to MARC::Record->as_usmarc and
MARC::Record->as_xml_record to trap errors in bibliograpic and
authority records.

If MARC::Record reports an error, print the record id and the
error message and continue the export.

Signed-off-by: Jason Stephenson <jstephenson@mvlc.org>
Signed-off-by: Ben Shum <bshum@biblio.org>

LP#1358916: refuse to retrieve over-large MARC records via Z39.50

At least one malformed record discovered in the wild can
cause open-ils.search backends to balloon to over 3G of memory
consumption.  This patch works around that by refusing
to process any (MARC) Z39.50 results that are larger than the
MARC record maximum of 99,999 octets.

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

LP#1340947: Finish support in oils_ctl.sh for -y and -z options

Signed-off-by: Jason Boyer <jboyer1@library.in.gov>
Signed-off-by: Ben Shum <bshum@biblio.org>

LP#1357364: Explictly create the extension in the "public" schema.

Thanks to Thomas Berezansky for the suggestion!

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

LP#1357364: Allow for sites that may have the intarray extension already installed.

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

2.7 doc asciidoc fix

Signed-off-by: Angela Kilsdonk <akilsdonk@esilibrary.com>

Docs: updated root.txt to point to 2.7 release notes

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

2.7 documentation from ESI

Documentation for Delayed vs Canceled Items, Report Editing, Report Hints, and Copy View/Edit Links.  Documentation written by Erica Rohlfs, Equinox Software, and Sally Fortin, Athenaeum Educational Services.

Signed-off-by: Angela Kilsdonk <akilsdonk@esilibrary.com>

Release notes repair to fix PDF build

Commit bb0aed2 did not fix our problem with the PDF build. Removing
quotes from the problem area to see if it fixes the issue.

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

Documentation: Lost and Paid Status

Adding docs for lost and paid status. These changes also moves the copy
statuses documentation up from the old 1.6 docs with the addition of the
long overdue and lost and paid statuses and includes some fixes to the
bills documentation.

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

Docs: EDI admin section updated with new screenshots and SANs info

EDI admin section updated to include screenshots and information
regarding entering the SANs required for EDI to work.

Signed-off-by: Jennifer Pringle <jpringle@sitka.bclibraries.ca>
Signed-off-by: Yamil Suarez <yamil@yamil.com>

Docs: Fix filenames to match references in the docs - second time

This is just like the fix that Remington Steed sent in on Sept 4th.

These images were creating errors for me when I try to create
an epub version of the docs because they are all referenced as .jpg
but the filenames were .JPG.

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

Docs: Staff generated penalties documentation

Signed-off-by: Kate Butler <katebutler@rodgerslibrary.org>
Signed-off-by: Yamil Suarez <yamil@yamil.com>

Docs: Staff initials settings for patron notes

Signed-off-by: Kate Butler <katebutler@rodgerslibrary.org>
Signed-off-by: Yamil Suarez <yamil@yamil.com>

Docs: Documented change to Holds Shelf Expire Report

Thanks Joan Kranich

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

Minor reparis to release notes and merge parts doscs

We broke the Evergreen docs build and think it may be related to starting
a line with ``++V.1++''. This is an attempt to fix it.

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

Bitesize doc change: .xlsx report output

Signed-off-by: Angela Kilsdonk <akilsdonk@esilibrary.com>

LP#1350042 browser unit tests use repo fm_IDL2js.xsl

Generate JS-friendly IDL2js for browser client unit tests from the .xsl
file within the repo instead of referencing the installed version, which
may not yet exist on the machine.

Signed-off-by: Bill Erickson <berickxx@gmail.com>
Signed-off-by: Jason Etheridge <jason@esilibrary.com>

LP#1294269 Docs: Fixed small documentation formatting bugs

Signed-off-by: SnigdhaD <snigdha.dagar@gmail.com>
Signed-off-by: Yamil Suarez <yamil@yamil.com>

Acknowledge new contributor in 2.7 release notes

One more person to recognize now! Suzanne Paterno from 167f7f5

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

Translation updates - newpot

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

Translation updates - po files

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

LP#1241644: Remove xact_finish IS NULL checks from CLAIMSRETURNED and LONGOVERDUE

Makes open-ils.actor.user.checked_out(.count) results agree with
open-ils.storage.actor.user.checked_out(.count) results.

Signed-off-by: Jason Boyer <jboyer1@library.in.gov>
Signed-off-by: Michele Morgan <mmorgan@noblenet.org>
Signed-off-by: Ben Shum <bshum@biblio.org>

LP#1081551 Serials batch recv. dupe barcode check

Exit processing and return an event if the barcode on a new serial.unit
collides with an existing serial.unit or asset.copy barcode in the
serial receive API.

In the UI, detect the dupe barcode event and provide an error message to
the user.

Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Jennifer Pringle <jpringle@sitka.bclibraries.ca>
Signed-off-by: Ben Shum <bshum@biblio.org>

lp1335958 Remove parameters from current checkouts link

The limit and offset parameters can cause patrons to be stuck in a small
section of their current checkout lists. Let's remove those two parameters
when linking to that list from the patron dashboard.

Signed-off-by: Kathy Lussier <klussier@masslnc.org>
Signed-off-by: Jim Keenan <jkeenan@cwmars.org>
Signed-off-by: Ben Shum <bshum@biblio.org>

LP#1190508: Rename certain image files to avoid Ad Block Plus's blacklist

Ad Block Plus was blocking images with /images/adv_ in their pathname. It is
presumed that "adv" was short for "advertisement".  I have renamed the files
and the references to avoid the ABP blacklist.

Signed-off-by: Dan Pearl <dpearl@cwmars.org>
Signed-off-by: Kathy Lussier <klussier@masslnc.org>
Signed-off-by: Ben Shum <bshum@biblio.org>

LP#1350042 grid print CSV repair; all cols repair

* repaired grid Print CSV action, which had not been updated with the
  latest print API.

* repair grid show/hide all columns actions.

Signed-off-by: Bill Erickson <berickxx@gmail.com>
Signed-off-by: Ben Shum <bshum@biblio.org>

Move new color note to release notes 2.7

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

lp1261791 Adjustment to background color for mobile search links

Adding an id for the div surrounding mobile search links so that the
background color matches the color of the rest of the header.

Signed-off-by: Kathy Lussier <klussier@masslnc.org>
Signed-off-by: Ben Shum <bshum@biblio.org>

Release notes entry for new mobile-header-text-color

Adding a release notes entry for the addition of mobile-header-text-color
in colors.tt2.

Signed-off-by: Kathy Lussier <klussier@masslnc.org>
Signed-off-by: Ben Shum <bshum@biblio.org>

LP 1261791 - Mobile Search links for my account screens

Added links for Basic, Advanced and Browse Searching from all my Account pages in a less than 600px view.:

Added a new color for the mobile search links, so users can modify based on their background color.

Signed-off-by: Suzanne Paterno <paterno@noblenet.org>
Signed-off-by: Jason Stephenson <jstephenson@mvlc.org>

Docs: Fixed small typo in docs/reports/reporter_daemon.txt

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

Docs: Updated usage of autogen.sh to not use param 'u'

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

Docs: update to 'Address Alerts' feature content

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

Release notes cleanup

Cleanup to the 2.7 release notes, to include spelling corrections, moving
some content to the official docs, and adding acknowledgements at the end.

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