Tweaked searches - ready to push to the public KPAC now (along with the graphics from the last commit)

Conflicts:

Open-ILS/examples/kpac.xml

Commented out Chilifresh patron reviews, as we don't use
them and the section title was appearing unnecessarily.

Forced Awards & Suggested Reads tab to always be open
in order to highlight NoveList Select content.

Oops!  Forgot to change the GALILEO links in the topnav.

Add links in reporter.classic_item_list to legacy_cat1 and legacy_cat2 views.

Currently, the reporter.classic_item_list view has the Legacy Cat 1 and 2
fields defined as "text", not "link", so all it will report out is the ID
number of the stat cat.  We want nullability control over stat cat entries,
so these need to be "link".

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

The "empty bib" deletion needs to exclude electronic resources (source <> 3)

LP1271218 fixes this issue, so reverting our workaround.

Revert "Commenting out a section of the notification preferences template that"

This reverts commit 87048c6278bc4ec3f59d78a23230cb51f7a7a0b1.

Conflicts:

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

This commit removes confirmation boxes popping up when a hyperlink is clicked

In PINES, we expect the behavior we've always had, so we revert the change.

Revert "Sort of in line with 949322 but not absolutely necessary"

This reverts commit e1a7501799119e0f305616af5bf97149d9bdf4c0.

Removing "Dynamic Catalog" link from the footer.

Can't vacuum within a transaction!  and commenting out a DROP INDEX.

Adding fix for KPAC hold placement.

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

Adding Terran's kpac.xml to update topics for post MLK weekend.

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

temporary workaround for holds error when user isn't logged in and doesn't select a pickup location

fixed problem with referencing wrong org selector

Adding workaround SQL for bug 1259665 to the post-upgrade script.

Adding PINES datawork to pre-upgrade script.

Commenting out a section of the notification preferences template that

causes an error to display.

Adding a copy of create_database_extensions.sql to re-create the PINES database during the upgrade.

Comment out portions of 2.4.3-2.5.0 upgrade script that cause errors.

Because of bug 1261355, we comment out 0795, 0841, 0842, and 0843
so we can get through the 2.4.3-2.5.0 upgrade script without errors.

See https://bugs.launchpad.net/evergreen/+bug/1261355 for details.

fixing some copy/paste typos

Moving from GIST to GIN indexes for text search in PINES

Adding permissions to LocalAdmins and GlobalAdmins to administer staff client toolbars.

Commenting out Bib Call Number and uncommenting Call Number Browse.

adding fields to be automatically stripped when importing MARC records

Needed some rearranging.

Re-adding library URLs, which were broken by a previous PINES-specific commit.

Attempt to fix default pickup location bug when user is not logged in.

Re-applied audience filter to restrict to juvenile audience types.

Modified style guide to emphasize call numbers in search results and item display.

Temporarily hid Add to List functionality until it can be further developed.

Bug fix to recognize user's default pickup library for holds when they are logged in.

Changed featured items for KPAC home page.

Adding KPAC link to header and footer.

PINES KPAC landing page doesn't include the Print button.

adding the beginnings of a post-upgrade script

Correcting file path for PINES logo on KPAC.

2.5.0 includes "Long Overdue", which adds its own copy status,

so we change our existing config.copy_status from "Long Overdue"
to "Long Overdue (Legacy)"

Re-adding the BEGIN;

Oops!  We need to define the eg_version.

PINES has already applied 0818.

Dropping a staged search function preventing the 2.3-2.4.0 script from completing.

Creating a PINES-specific version of the 2.3 to 2.4.0 upgrade script.

Includes indexes that must be dropped for the upgrade script to succeed.

Adding a pre-2.3-2.4.0 upgrade script to do some necessary datawork.

Removing "Bib Call Number" from the advanced numeric search options.

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>