webstaff: default Check Number to 1

instead of 0, so that it doesn't speedbump payments (of any type)

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

Driver's license guidelines
Added on-screen tip to remind staff how driver's licenses should be
entered in PINES.

Signed-off-by: Terran McCanna <tmccanna@georgialibraries.org>

Hide Email Checkout Receipts Option

This doesn't work in the staff client, so hiding option from the patron
self-registration form and from the patron's My Account settings until
we have moved to the web client. (It still appears in the staff client,
so can be set there, but it will be virtually non-functional if set.)

Signed-off-by: Terran McCanna <tmccanna@georgialibraries.org>

Shelf Browser display - bug fix

Signed-off-by: Terran McCanna <tmccanna@georgialibraries.org>

Temporary Lists: Hide temporary list functionality until bug is resolved.

Signed-off-by: Terran McCanna <tmccanna@georgialibraries.org>

Hiding sort option for Popularity Adjusted Relevance.

Signed-off-by: Terran McCanna <tmccanna@georgialibraries.org>

LP#1360347 - Acq custom copy status at receiving

Current acq behavior is that received items get a
copy status of "In Process".  This status is typically
holdable and opac_visible, and some libraries have a need
for an intermediate status between "On Order" and "In Process".

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

Fix SIP timeouts due to invalid sessions

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

LP#1643932 - Fix search slowness when patron searching by username

Bug 1501781 introduced a feature to improve patron name searching,
but which was causing very slow queries when searching by username.
Adding an index to username solves the slowness problem.

Suggested by Mike Rylander.

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

LP#1586509 Bug fix to LP#1352542 caused extraneous blank line to appear in
spine label.  This affected LC call numbers that had only one cutter number
plus additional text following.

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

LP#1594937 Fix off-by-one display of closed dates

The switch to toISOString() to format dates introduced an off-by-one error in
the closed dates display, showing one extra day of closure due to the timezone
being ignored in toISOString().

toLocaleDateString() is the future of locale-sensitive date formats. In XUL,
because it is an old version of Firefox, it lacks locale sensitivity, but for
the purposes of the web staff client it's a good base to build on as even
Internet Explorer supports the locale and options arguments as of IE 11.

And for the immediate purposes of showing the right dates in the closed dates
editor, it works.

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

LP#1432753 Restore "All day" verbiage to Closed Dates editor

Commit ede7e78925 replaced the JSAN calls to util.date.formatted_date() with
inline date/time handling, in the process returning times with granularity to
the minute instead of to the second. This resulted in the test for "all day"
closings always failing.

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

LP#1528916 Patron Holds Ready/Total

Switch order or patron_stats().holds.ready and
patron_stats().holds.total in t_summary and patron
index.

Signed-off-by: Kyle Huckins <khuckins@catalystdevworks.com>
 Changes to be committed:
modified:   Open-ILS/src/templates/staff/circ/patron/index.tt2
modified:   Open-ILS/src/templates/staff/circ/patron/t_summary.tt2

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

Revert "Returning all selfcheck templates to stock versions."

This reverts commit 2d1fbbf5de79758b9c8b36863135554ecbdf9d18.

Returning all selfcheck templates to stock versions.

Since we only have bits and pieces of Spanish translations for our
customized KPAC right now, removing the parts that would display in
Spanish if Spanish has been selected in the OPAC. In the future when
all parts of the KPAC have been translated and graphic buttons have
been switched to text buttons that can be translated, we will then
need to go back in and re-add the language markup to allow the
translations to occur.

Signed-off-by: Terran McCanna <tmccanna@georgialibraries.org>

Hiding parent link from branch info pages to prevent confusion when patrons
see system hours (and other info) and think that they are the same as
branch hours.

Signed-off-by: Terran McCanna <tmccanna@georgialibraries.org>

LP#1647819 - Missing closing angle bracket in anon_list.tt2

Signed-off-by: Chris Sharp <csharp@georgialibraries.org>
Conflicts:
Open-ILS/src/templates/opac/parts/anon_list.tt2

LP#1647819 - Missing closing angle bracket in anon_list.tt2

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

Language files - a few more corrections.

Signed-off-by: Terran McCanna <tmccanna@georgialibraries.org>

Language - here are three files I missed including in the prior
collection of translation-related files.

Signed-off-by: Terran McCanna <tmccanna@georgialibraries.org>

OPAC Updates related to page text and translations. Also required are the
en-PINES.po custom text file and the es-ES.po translation file.

Signed-off-by: Terran McCanna <tmccanna@georgialibraries.org>
Conflicts:
Open-ILS/src/templates/opac/parts/advanced/search.tt2
Open-ILS/src/templates/opac/parts/pref_lib_display.tt2

Restoring selfcheck.js to stock version

Lists page - re-added note and link to Lists instructions

Signed-off-by: Terran McCanna <tmccanna@georgialibraries.org>

UI tweaks to the basic search form so that it looks better on smaller
screens.

Signed-off-by: Terran McCanna <tmccanna@georgialibraries.org>

OPAC 'preferred library' - removed question mark link

since it doesn't go to a meaningful page and confuses patrons.

(Terran)

OPAC advanced search page UI tweaks. (Terran)

Rolling prefs_notify.tt2 back to stock 2.11.1.

Checking out stock version of lists.tt2 to re-customize

Restoring stock advanced search template while we troubleshoot.

LP#1639782 - Add Active Date to Item Status Column Picker

Circ and Cataloging staff need a quick way to view the Active
Date of an item in the "Item Status" (F5) interface.  This
makes that column available for selection.

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

PINES already has 0977 applied, so comment it out.

Creating PINES pre-upgrade script.  Drop public.demographic view

Addition of new item format icon for direct audio (ie, Playaways and
similar items that are pre-loaded with audiobooks and do not require
access to a computer to use).

Signed-off-by: Terran McCanna <tmccanna@georgialibraries.org>

LP#1599634 - Remove DROP VIEW statement, add .sql to name.

When running the upgrade script, the DROP VIEW statement resulted in an
error since the view did not already exist.  You could add IF EXISTS, but
CREATE OR REPLACE VIEW is sufficient.

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

LP1599634 Circulation report source to include in-house(non cat), and non cat circ

Added a simple view and supporting IDL. This will allow all the circulation
data to output in the same report.

Signed-off-by: blake <blake@mobiusconsortium.org>
Signed-off-by: Chris Sharp <csharp@georgialibraries.org>
Conflicts:
Open-ILS/src/sql/Pg/090.schema.action.sql

LP#1610246 Classic Circulation View excluding circulations

The reporter.classic_current_circ view, which is available via
Open-ILS/src/sql/Pg/example.reporter-extension.sql, was excluding
circulations because it was assuming all users have a billing address
This branch corrects that with a LEFT JOIN.

No upgrade script will be included since these views are not assumed
to be installed by all Evergreen instances.  Please re-run the
example.reporter-extension.sql file to apply this change for your
instance.

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

Updates to KPAC Features (addition of TumbleBooks)

Signed-off-by: Terran McCanna <tmccanna@georgialibraries.org>

LP#1528647 Self-check only accepts user name value if regex
for barcode not set up

This patch was written by Bob Wicksall and posted to launchpad. I've
tested on my own server and packaged it for review here.

The fix allows a patron to log into the self-check interface with either
a barcode or a user name.

Signed-off-by: Terran McCanna <tmccanna@georgialibraries.org>
Revert "LP#1528647 Self-check only accepts user name value if regex"

Terran reports this not working as expected.

This reverts commit 0eb3a73c2fba4f609c0dab015a843a550e03516e.

LP#1528647 Self-check only accepts user name value if regex
for barcode not set up

This fix allows a patron to log into the self-check interface with either
a barcode or a user name.

Written by: Bob Wicksall <bwicksall@pls-net.org>

Signed-off-by: Terran McCanna <tmccanna@georgialibraries.org>
LP#1528647 Self-check only accepts user name value if regex for barcode
not set up

The original fix only worked if password was not required for self-check.
This fix allows it to work when password is required as well.

Signed-off-by: Terran McCanna <tmccanna@georgialibraries.org>

LP#1528647 Self-check only accepts user name value if regex
for barcode not set up

This patch was written by Bob Wicksall and posted to launchpad. I've
tested on my own server and packaged it for review here.

The fix allows a patron to log into the self-check interface with either
a barcode or a user name.

Signed-off-by: Terran McCanna <tmccanna@georgialibraries.org>

KPAC SRP 2016 tweaks to pre-set search categories

KPAC files to promote Summer Reading Program for 2016.

Removing testing piece.

Actually, that never needed to be in parens.

Testing for now, so no commit;

Forgot a closing parens.

I've redone all of the kpac category links so they no longer use
a link shortener. I also rearranged some of the links so that there
are more options on the home page.

Signed-off-by: Terran McCanna <tmccanna@georgialibraries.org>
Conflicts:
Open-ILS/examples/kpac.xml

KPAC - Hold results not displaying item title and throwing log error

I believe the path comparison operator in the 'load' subroutine has
a flaw in logic caused by a path-naming problem. The path-matching
function recognizes the kpac/getit_results path as matching positively
for both the load_getit subroutine and the load_getit_results subroutine.

This patch reverses the order in which the matching occurs in an attempt
to resolve the conflict.

Signed-off-by: Terran McCanna <tmccanna@georgialibraries.org>

Fix for org unit order in OPAC holdings list (adapted from
http://git.evergreen-ils.org/?p=evergreen/pines.git;a=commit;h=adc566e152d10796b73c992a16ae46e69cc3ab77)

Signed-off-by: Terran McCanna <tmccanna@georgialibraries.org>

Adding PG 9.4 installation targets

Second piece to the bug fix for group holds...

Signed-off-by: Terran McCanna <tmccanna@georgialibraries.org>

adding a/t environment to new event def

More changes to post-upgrade data file.

LP#1463166 Removed multiple perms

The use of multiple perms in creating the widget was causing problems.
I changed the permlist back to a single perm.

Signed-off-by: Liam Whalen <liam.whalen@bc.libraries.coop>

LP#1463166 Fix Batch update of Funds in Selection Lists

Selection lists are currently unable to batch update funds.  The problem
stems from the fact that the Fund selection list is not populated in the
copy rows.  New code used to limit the number of funds, in the dropdown
list, to those funds available to the user was added for purchase
orders.  However, these changes were not added to Selection Lists.  This
code adds the new call to initBatchUpdater to the selection list code.

Signed-off-by: Liam Whalen <liam.whalen@bc.libraries.coop>

Temporary modifications to My Account > Holds History until more
permanent fixes are developed.

Hid paging because it is broken. See:
https://bugs.launchpad.net/evergreen/+bug/1422927

Showing "Canceled" instead of "Active" for canceled holds. See:
https://bugs.launchpad.net/evergreen/+bug/1275118

Signed-off-by: Terran McCanna <tmccanna@georgialibraries.org>

Adding customization to disable the "Clear Holds Shelf" function.

adding login redirection page

Adding post-upgrade datawork script

Adding PINES-ified notification wordings.

These changes undo a couple of small things I tried to tweak on the web
client that didn't work the way I hoped they would.

Signed-off-by: Terran McCanna <tmccanna@georgialibraries.org>

Changes to splash screen and receipt template defaults for web client.

(Note: Could not test on terran-testbox since it is not https-enabled,
so I could not log in to the web client.)

Signed-off-by: Terran McCanna <tmccanna@georgialibraries.org>

Self-Check changes in preparation for upgrade:
Changed wording on some of the on-screen directions.

Signed-off-by: Terran McCanna <tmccanna@georgialibraries.org>

OPAC changes in preparation for upgrade:

1. Added instructions to circ history page in My Account for turning
on your circulation history.

2. Modified behavior or NoveList tab on item details page so that it
is closed by default in the staff client, and it will close when a
different tab is opened to reduce page scrolling.

Signed-off-by: Terran McCanna <tmccanna@georgialibraries.org>

KPAC changes in preparation for upgrade:

1. Biggest change was to move the library selector box to the main
search panel instead of on the side - had to modify quite a few files
in order to make this work and pass the values correctly.

2. Also changed the feature icons to use links instead of search terms
to allow more control over search limits and sorting and to see if
caching the searches would improve response speed.

3. A number of smaller UI changes.

Signed-off-by: Terran McCanna <tmccanna@georgialibraries.org>

Selfcheck Cookie Path

Change base.xul/base.js to support a "cookie path" option to limit what paths
will use the login cookie, then tell the selfcheck interface to use it.

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

Additional changes to the self-check interface.

Signed-off-by: Terran McCanna <tmccanna@georgialibraries.org>

Images for new "no image available" functionality.

Lines 94-104 in eg_vhost.conf need to be un-commented.

Signed-off-by: Terran McCanna <tmccanna@georgialibraries.org>

Fixes to OPAC holds & patron self-registration pages

Signed-off-by: Terran McCanna <tmccanna@georgialibraries.org>

These are modifications to the Evergreen self-check interface to:

1. Fix bugs with printing holds and fines.
2. Improve page layout overall, and with consideration for
   1024x768 resolution monitors that a lot of libraries use.
3. Hide hold status from view on holds table.
4. Hide buttons/fields from screen when user is not logged in.

Signed-off-by: Terran McCanna <tmccanna@georgialibraries.org>
Conflicts:
Open-ILS/src/templates/circ/selfcheck/banner.tt2

Further adjustments to staff client splash page so it will fit onto
older 1024x768 resolution monitors better.

Signed-off-by: Terran McCanna <tmccanna@georgialibraries.org>

Modifications to patron self-registration form to make it
display better and change library locator link.

Signed-off-by: Terran McCanna <tmccanna@georgialibraries.org>
Conflicts:
Open-ILS/src/templates/opac/register.tt2

Miscellaneous OPAC UI tweaks for spacing / layout.

Signed-off-by: Terran McCanna <tmccanna@georgialibraries.org>
Conflicts:
Open-ILS/src/templates/opac/parts/org_selector.tt2
Open-ILS/src/templates/opac/parts/record/copy_table.tt2
Open-ILS/src/templates/opac/parts/searchbar.tt2

Numerous local customizations to My Account pages for layout and
to add on-screen assistance to patrons.

Signed-off-by: Terran McCanna <tmccanna@georgialibraries.org>
Conflicts:
Open-ILS/src/templates/opac/myopac/circs.tt2
Open-ILS/src/templates/opac/myopac/holds.tt2
Open-ILS/src/templates/opac/myopac/prefs.tt2
Open-ILS/src/templates/opac/myopac/prefs_settings.tt2

Staff Client tweaks to splash page:
- Added link to PINES Documentation
- Changed logo at top to PINES logo
- Adjusted spacing
- Updated copyright date

Also, hid footer on OPAC pages while viewing from
within the staff client because it clutters the page
without adding any value.

Signed-off-by: Terran McCanna <tmccanna@georgialibraries.org>
Conflicts:
Open-ILS/web/opac/locale/en-US/lang.dtd
Open-ILS/xul/staff_client/server/index.xhtml

Improved list display in OPAC (title, description, annotations).

Signed-off-by: Terran McCanna <tmccanna@georgialibraries.org>

Removed added content from the KPAC for now since it doesn't
display properly. Possibly add back in later if we can get
the NoveList content to pull in right.

Signed-off-by: Terran McCanna <tmccanna@georgialibraries.org>

Improvements to KPAC list display, especially adding
display of list annotations that were not shown.

Signed-off-by: Terran McCanna <tmccanna@georgialibraries.org>
Conflicts:
Open-ILS/src/templates/kpac/parts/record_row.tt2
Open-ILS/src/templates/kpac/results.tt2

Updated kpac.xml configuration file to be ready to roll out after upgrade.

Signed-off-by: Terran McCanna <tmccanna@georgialibraries.org>
Conflicts:
Open-ILS/examples/kpac.xml

Adding corrected wording for self-registration agreement.

Signed-off-by: Terran McCanna <tmccanna@georgialibraries.org>

Customizations to patron self-registration form and footer link.

Conflicts:
Open-ILS/src/templates/opac/parts/footer.tt2
Open-ILS/src/templates/opac/register.tt2

PINES has already applied 0940

correcting typo causing Chilifresh to continue to display

Correcting a typo that caused the copy_table.tt2 template to break

Enable Halloween holiday feature in KPAC.

Signed-off-by: Terran McCanna <tmccanna@georgialibraries.org>
Conflicts:
Open-ILS/examples/kpac.xml

Conflicts:
Open-ILS/examples/kpac.xml

Change links old library map to new interactive library locator map.

Signed-off-by: Terran McCanna <tmccanna@georgialibraries.org>

Updated link to old library map to new library locator map in KPAC.

Signed-off-by: Terran McCanna <tmccanna@georgialibraries.org>

Added PINES Quick Reports icon and link to Evergreen Staff Client
main page. The link will open in a browser window.

Signed-off-by: Terran McCanna <tmccanna@georgialibraries.org>

Enabled Google Analytics for OPAC usage tracking.

Adds B4 Book Log icon to KPAC search results page and
KPAC item details page.

(This is a PINES-only customization.)

Adds B4 Book Log icon and link to KPAC search results page and
KPAC item details page.

(This is a PINES-only customization.)

Conflicts:
Open-ILS/src/templates/kpac/parts/record_row.tt2
Open-ILS/src/templates/kpac/record.tt2

Adding GALILEO in-library link logic to topnav links.

PINES/GALILEO avoid auth redirect within library

If the physical_loc is set, indicating the patron is physically at a
branch (via IP redirect), link directly to the GALILEO site with no
authentication pass-through.  GALILEO does not requires PINES auth
if the client is coming from a known IP address.

Signed-off-by: Bill Erickson <berick@esilibrary.com>
Conflicts:
Open-ILS/src/templates/opac/parts/footer.tt2

LP#1446860 Staff were able to edit their own accounts.

This change reverses what appears to be a thinko in the
original programming.  If the editing user is the same
as the user being edited, disable the Save button.

Signed-off-by: Chris Sharp <csharp@georgialibraries.org>
Conflicts:
Open-ILS/web/js/ui/default/actor/user/register.js

LP#1446860 - Correct mistaken logic in previous fix.

See https://bugs.launchpad.net/evergreen/+bug/1446860/comments/2
for reference.  The logic was not enforcing the "if the editing
staff member doesn't have permission to edit a user in this
permission group" check.  This corrects that.

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

LP#1446860 Staff were able to edit their own accounts.

This change reverses what appears to be a thinko in the
original programming.  If the editing user is the same
as the user being edited, disable the Save button.

Signed-off-by: Chris Sharp <csharp@georgialibraries.org>
Conflicts:
Open-ILS/web/js/ui/default/actor/user/register.js

branding the built-in selfcheck interface for PINES

Sadly, GALILEO/EBSCO does not use HTTPS, so this will create errors for patrons who authenticate via PINES.  Reverting to HTTP.

Make users_of_interest test for defined actor.usr.card values

If the open-ils.collections.users_of_interest API encounters a patron that
has a null "card" value in actor.usr the API will crash with an error like:

Received Exception:
Name: osrfMethodException
Status:  *** Call to [open-ils.collections.users_of_interest.retrieve] failed for session [1416515351.333446.1416515351406], thread trace [1]:
Can't call method "barcode" on an undefined value at /usr/local/share/perl/5.14.2/OpenILS/Application/Collections.pm line 299.

This patches users_of_interest to be sure that the user has a defined value
for actor.usr.card before proceeding to pull out the barcode for the results
of the API call.

Signed-off-by: Michael Peters <mpeters@emeralddata.net>
Conflicts:
Open-ILS/src/perlmods/lib/OpenILS/Application/Collections.pm

Header and Footer display changes to improve the way they display
on mobile devices and smaller-width windows.

Signed-off-by: Terran McCanna <tmccanna@georgialibraries.org>

restoring commented out checkbox to allow multiple hold requests on the same title