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

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.3 upgrade script

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

Forward-port 2.5.7 upgrade script

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

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>

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>

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#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>

LP#1379824 Make PermaCrud.js disconnect() actually disconnect

The disconnect() method in PermaCrud.js was meant to wrap the
underlying session disconnect, but it never actually disconnected
the session.  This could lead to problems in long-living PermaCrud
objects, as they may think they are still connected when they are not
(the session remote_id is never cleared).

Also, remove a couple (now redundant) manual session.disconnect() calls.

Signed-off-by: Dan Wells <dbw2@calvin.edu>
Signed-off-by: Yamil Suarez <yamil@yamil.com>
Signed-off-by: Ben Shum <bshum@biblio.org>

LP#1305958 Change copy table header atts to scope attributes

Per http://webaim.org/techniques/tables/data, the current best practice for
marking up tables for accessibility is to use the "scope" attribute for
headers, rather than the error-prone and markup-heavy "headers" attribute on
table cells.

Signed-off-by: Dan Scott <dscott@laurentian.ca>
Signed-off-by: Chris Sharp <csharp@georgialibraries.org>
Signed-off-by: Ben Shum <bshum@biblio.org>

LP#1309664: Stamping upgrade script for new seed data for COM and SER

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

LP#1309664: Add some fixed field seed data for COM and SER

There are a few useful fixed fields we're missing mapping data for,
so this branch adds three: Freq, Regl, and File. There are certainly
more, but here's a bit that's of use to folks in the wild using the
fixed field editor improvements.

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

LP#1374551: Stamping upgrade script for new index on money.billing.voider

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

LP#1374551: Create index on money.billing.voider to speed user merge.

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

LP#1368314: Stamping upgrade script for RDA support to reporter.simple_record

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

LP#1368314: Add RDA support to reporter.simple_record.

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

LP#1306814: Make use of patron timeout setting for selfcheck

As described in the bug, the library setting "Self Check: Patron Login Timeout
(in seconds)" does not appear to work with the newer selfcheck interface.

It looks like some of it was already being pulled in, like the variable, but
the interface was not utilizing it.

This commit borrows from the old JS selfcheck and implements a default of
three minutes for selfcheck timeout otherwise handled by the library setting.

Signed-off-by: Ben Shum <bshum@biblio.org>
Signed-off-by: Michele Morgan <mmorgan@noblenet.org>

Translation updates - newpot

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

Translation updates - po files

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

Update script for update_pofiles

We were missing a few of the folders used for i18n purposes from
the update_pofiles script. Added the following entries:

FlattenerGrid.js
match_set.js
PCrudFilterPane.js
serial.js
tpac
URLVerify.js
vandelay.js

Also, change from Acq.js to acq.js.

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

LP#1252786: Add Workstation link to Aged and Active Circulations reports source.

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

LP#1374091: stamping upgrade script for speedier record_attr views

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

LP#1374091: Speed up metarecord version of unAPI mra call

The change to metabib.record_attr_flat in the previous commit caused
a regression in the plan used in unapi.mmr_mra.  This commit addresses
that by unrolling the mra view when used in context with metarecords,
which are shaped differently and require slightly different access
patterns.

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

LP#1374091: Redefine view to help PG make better plans

The metabib.record_attr_flat view previously pulled its data
through several other intermediate views.  While this improves
maintenance overhead, it ends up being an optimization fence
for certain data sets.  So, we pull the logic from the leaf
views up into the main view.  And, poof, 2 orders of magnitude
speedup on large data sets.

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

LP#1378575: teach action::hold_request about behind_desk

This patch teaches the CDBI class for the action.hold_request
table about the newish behind_desk column, preventing errors
like this from being thrown if a recent OpenSRF version is in
use:

Processing of hold failed:  Can't locate object method "behind_desk"
via package "action::hold_request" at /usr/local/share/perl/5.14.2/OpenILS/Application/Storage/CDBI.pm line 181.

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

LP#1314827: On login, don't allow referer-based redirect to external site

On /eg/opac/login, if no redirect_to param is provided, the TPAC will
attempt to use the referer (if any) as the redirect destination. This
leads to undesirable behavior if the referring URL is from an external
site.

Signed-off-by: Jeff Davis <jdavis@sitka.bclibraries.ca>
Signed-off-by: Ben Shum <bshum@biblio.org>

Documentation bug: corrected image file

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

Whitespace and non-weird emdash in docs

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

Docs: Change all internal links to correct format

Resolves LP#1294347

I just deleted one link since I couldn't find any trace of where it was intended to go.

acquisitions/selection_lists_po.txt:
"or use the link:federated.txt[View/Place Orders] menu item in the catalog."

If anyone ever finds where that is supposed to go, let me know and I'll fix it.

Signed-off-by: Josh Stompro <github@stompro.org>
Signed-off-by: Remington Steed <rjs7@calvin.edu>

LP#1347774 CStoreEditor anonymous PCRUD additions

* Exporter repairs : use export_to_level
* pcrud personality call argument mod repair
* log pcrud personality json_query attempts loudly to ease transition
* CStoreEditor anon/pcrud personality live test

* CStoreEditor anon pcrud default to undef authtoken:
The presence of an authtoken ("ANONYMOUS") in the editor can be
misleading at higher levels of the code, since it implies a
presumably functional authtoken has been provided somewhow.
Apply ANONYMOUS only as needed to API calls instead.

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

LP#1347774 Anonymous PCRUD mode

Support for anonymous access to public (field_safe=true) IDL data
via PCRUD without requiring an authtoken.  To use, pass an authtoken of
ANONYMOUS.

Includes initial CStoreEditor plugin for anon pcrud.

Signed-off-by: Mike Rylander <mrylander@gmail.com>
Signed-off-by: Bill Erickson <berickxx@gmail.com>

LP#1373693: Speed up call number browse search at top level org.

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

Docs: Removed redundant phrase as mentioned in bug #1294299

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

Docs: Updated 2 links that pointed to the old subversion repository

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

Docs: Removed a redundant phrase as mentioned in bug #1294299

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

Tidy search section of Using the OPAC

Primarily focused on restructuring the information around Boolean
searching, particularly as I find librarians want to know what the
Boolean operators are that they should use. However, cleaned up some
typos and formatting along the way.

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

Docs: for new "Imported As" column in Vandelay queue results view

Added explanation of the new "Imported As" menu in "MARC Batch Import/Export"
queue results view, also updated screenshots.

Thanks to Joan Kranich of C/W MARS, for helping research were to make these
changes.

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

Small typo fix for Blu-Ray spelling

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

Updated formats

Updated the formats under basic search according to version 2.6

Signed-off-by: Kathy Lussier <klussier@masslnc.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>