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>
Conflicts:

Open-ILS/src/perlmods/lib/OpenILS/Application/Circ.pm

Revert "Fixing LP#1203734 - "Show Last Few Circulations" interface not counting archived circulations."

This reverts commit 4978ead0608a0c45a7e0f0c77380d0548a80e1e1.

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>

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>
Conflicts:

Open-ILS/src/templates/opac/parts/base.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>

Syncing kpac.xml with rel_2_5_1_kpac

Removing 4th of July KPAC link

Add GALILEO for Kids link to KPAC header.

Display Thanksgiving icon in KPAC.

Conflicts:

Open-ILS/examples/kpac.xml

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

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

Removing extra '|' character from footer links.

Merge branch 'rel_2_5_1' of git.evergreen-ils.org:/evergreen/pines into rel_2_5_1

LP#1311467: fix heading validation in bib editor

This patch fixes a regression whereby clicking the validate button
in the bib editor would result in no headings getting checked.

In the process, it enables 'startwith' to be used as a JSON query
operator for all types of predicates, not just the results of
transformed values.

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Signed-off-by: Mike Rylander <mrylander@gmail.com>

LP#1253163: stamping upgrade for authority.in-line-headings

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

Open-ILS/src/sql/Pg/002.schema.config.sql

LP#1253163: Replace dropped unique index, if we can

Not sure if this is necessary, but since we drop this index, we might
want to make an effort to recreate it if we can (but don't die if we
cannot).

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

LP#1253163: Make authority functions more truthful

One discovery we made on this bug was that PG can break if we
conveniently label functions as IMMUTABLE when they are now.  Let's
go ahead and reclass some authority functions as STABLE STRICT to
better fit reality.

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

LP#1253163: Materialize authority headings

There seems to be a bug in Postgres 9.3+ that does not like the
circuitous route we take to indexing authority headings.  So, we
get around this by storing the actual heading values instead of
indexing a function over the MARC.

Signed-off-by: Mike Rylander <mrylander@gmail.com>
Signed-off-by: Dan Wells <dbw2@calvin.edu>
Signed-off-by: Ben Shum <bshum@biblio.org>

LP#1253163: Replace dropped unique index, if we can

Not sure if this is necessary, but since we drop this index, we might
want to make an effort to recreate it if we can (but don't die if we
cannot).

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

LP#1253163: Make authority functions more truthful

One discovery we made on this bug was that PG can break if we
conveniently label functions as IMMUTABLE when they are now.  Let's
go ahead and reclass some authority functions as STABLE STRICT to
better fit reality.

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

LP#1253163: Materialize authority headings

There seems to be a bug in Postgres 9.3+ that does not like the
circuitous route we take to indexing authority headings.  So, we
get around this by storing the actual heading values instead of
indexing a function over the MARC.

Signed-off-by: Mike Rylander <mrylander@gmail.com>
Signed-off-by: Dan Wells <dbw2@calvin.edu>

LP#1311467: fix heading validation in bib editor

This patch fixes a regression whereby clicking the validate button
in the bib editor would result in no headings getting checked.

In the process, it enables 'startwith' to be used as a JSON query
operator for all types of predicates, not just the results of
transformed values.

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

Adding corrected wording for self-registration agreement.

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

Customizations to patron self-registration form and footer link.

Fix for problem where opac.hold_notify is null or blank
and no notification options are triggered.

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

Further tweaking KPAC holds bug fix.

Additional changes to hold bugs fix

Possible fix for KPAC Holds bug.

Conflicts:

Open-ILS/src/templates/kpac/getit.tt2

Removing due date visibility from public OPAC view.

Committing the current kpac.xml file.

Fixing LP#1203734 - "Show Last Few Circulations" interface not counting archived circulations.

open-ils.storage.asset.copy.circ_count was only pulling information from
action.circulation.  This changes that table to action.all_circulation instead.
It does not factor values in extend_reporter.legacy_circ_count.

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

2014 Summer Reading Program Graphics

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

Conflicts:

Open-ILS/examples/kpac.xml

Rewrite marc_export.in in support-scripts.

This commit rewrites marc_export.in in the support-scripts  src directory.
It will still be transformed into marc_export at build time.  This change
replaces the current marc_export scipt with a faster, DBI based alternative.

The replacement support nearly all of the features of the original, except
for the progress output.  Much of this information is harder to gather
with the new design.

This is a squashed, rebased commit of previous work.

Signed-off-by: Jason Stephenson <jstephenson@mvlc.org>

Add from_bare_hash to Fieldmapper.pm.

Teach the Perl Fieldmapper.pm to build objects from a hashref in a
similar manner to how the JS Fieldmapper already can.  You might use
it if you retrieved a database row as a hashref, like so:

my $hashref = {};
my $class = Fieldmapper::class_for_hint('bre');
my $bre = $class->from_bare_hash($hashref);

We also modify the Fieldmapper's properties method to sort the array of
field names it returns by position.  This makes the return value a lot
more useful for on the fly object construction and other fancy tricks.

Signed-off-by: Jason Stephenson <jstephenson@mvlc.org>

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.

If there are multiple CAPTURE-blocking penalties we could see dups on the pull list.  This avoids that.

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

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.

Staff were able to edit themselves.  This small logic reversal fixes that.

LP1272316 - Calculate proximity in the DB for speed

We need to calculate and store a (potenially adjusted) proxmity for
each hold and copy for use in targetting and optimization of op
capture.  Before this commit, we do that within the hold target
code itself.  Now we'll do it when the hold-copy map is inserted,
because we have the same information available at that time as we
have in the targeter.  This will both speed up the apparent cost of
the calculation, because it avoids the cost of a network round-trip,
and the total number of SELECTs we must issue, because the proximity
value will now be cached for use by the proximity map function.

Backward compatability is retained for the create_prox_map() function
in case any other code is depending on that.

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

Tweaked to recognize libraries that aren't allowed to be hold pickup locations

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

Followup to make ou_ancestor_setting() even faster

Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>

Speed up these settings lookups in the hold targeter that get repeated a ...

... lot when there are lots of copies for a hold.

Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>

LP1271218 - clarify how opt-in user notifications are shown in my account

As reported in LP127128, end users may not need to be shown the message that
"No notification preferences are configured" when in fact this message is only
displayed when the user opt-in notification settings are not configured.

So, alter this to only show the table and div when there are settings to be
configured and otherwise show nothing.

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

Always show Email Address label to staff

When placing holds staff commonly change out patrons, and pointing staff
towards the "My Account" area makes no sense to begin with.

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

LP1271218 fixes this issue, so reverting our workaround.

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

This reverts commit 87048c6278bc4ec3f59d78a23230cb51f7a7a0b1.

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.

Break out of focus() in cat/z3950.js if obj.active_services is undefined.

Signed-off-by: Jason Stephenson <jstephenson@mvlc.org>

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.

Merge branch 'rel_2_5_0' into rel_2_5_1

Conflicts:
ChangeLog
Open-ILS/src/perlmods/lib/OpenILS.pm
Open-ILS/src/perlmods/lib/OpenILS/Application.pm
Open-ILS/src/sql/Pg/002.schema.config.sql
Open-ILS/xul/staff_client/chrome/content/main/about.html
Open-ILS/xul/staff_client/defaults/preferences/prefs.js
README
configure.ac

Pull in missing 2.5.0 upgrade file

This file was never copied upstream, so it didn't make it into this
branch.  Adding it manually now.

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

Bumping version numbers, adding Upgrade Script and Changelog

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

Changes to smooth out make_release

1) Make the fake 'HeadURL' into a URL which works with our web repo.
Note that this has 'tags/' hardcoded in, so it only works with true
release branches (e.g. rel_x_y_z) not "working" branches
(e.g. rel_x_y).

2) Change range selector when building log to exclude commits from
rel_2_4.

3) Switch the XULRunner download order so that we can remove them
automatically during make_release and manually afterwards.

4) Add a pause between tagging and building for manual tweaks.

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

Bump OpenILS.pm version

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

Fix boolean lists; Better atom regex; Caching

First, we didn't need to make $last_type local, and it broke explicit
grouping anyway.  That's removed, and we now reset that (and a few more
like it) at calls to the top level parse() method.  This introduces a
situation where a long list of booleans could cause query plan problems,
so we limit the plan depth to 40 (20 ||'d conditions).

Second, we are smarter about finding the boundary of atoms.  Previous
to this commit, and curly brace could send the parser into a tailspin
from which it would not recover.  Now we use alternation instead of
a character class, which is much safer with the default multi-character
float syntax specifier.

Third, as a catch-all, if we can't parse the remained of a query we
now simply say so (when in debug mode) and go away, instead of risking
an infinite loop.  We do this via a final, unqualified "else" clause
in decompose().

Finally, instead of building 10+ regexp objects on each query parse,
cache them per QP subclass and reuse them.

Signed-off-by: Mike Rylander <mrylander@gmail.com>
Signed-off-by: Dan Wells <dbw2@calvin.edu>

Change "Alias" to "Holds Alias" in Patron Search.

This same field is referred to as "OPAC/Staff Client Holds Alias" in
the Register Patron interface.  This change is for consistency in labeling.

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

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

LP#1238015 hide patron credit summary when disabled

When the circ.disable_patron_credit / "Disable Patron Credit" org unit
setting is disabled, also hide the patron credit summary information in
the bills interface.

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

Re-apply the changes provided by  0802 for backport

Preserve record order of subfields for authority heading extraction

When extracting headings from authority records we currently read the
subfields of a tag in configuration order.  We should, instead, read
them in record order, to preserve the desired sorting properties that
the cataloger has encoded in the record.

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

Fix copy_table HTML markup errors

TD elements take a "headers" attribute, not a "header" attribute. Also,
we have a few typos in identifying the header attributes.

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

LP#1254146 gracefully handle custom trees sans entries

Avoid references to custom tree entries when no tree entries for a
custom tree exist.  Prevents:

egweb: template error: undef error - Can't call method "org_unit" on an
undefined value at
/usr/local/share/perl/5.10.1/OpenILS/WWW/EGCatLoader/Util.pm line 185.\n

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

Improve facet styling

1) Facets currently display very poorly when the number reaches 3-4
digits.  Facet data is more or less a two column table, so let's style
it that way to make it more robust.

2) Rearrange some styles to reduce redundancy, and allow overflow
where possible.

3) Change the selected highlight to look a little better.

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

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.