print_custom: call sums func, add phone numbers

Call the print_do_sums function so that span-based summing will
work, and add a class-based replacement called tadlPhoneNumber
(accepts library shortname as an argument).

Signed-off-by: Jeff Godin <jgodin@tadl.org>

further tweaks to DB schema upgrade scripts

move Makefile for TADL skin out of the way

This way, we won't end up fighting with the Makefile generated
during the normal Evergreen installation process.

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

Merge remote-tracking branch 'tadl2/rel_2_5_1_tadl' into tadl_2_5

Merge remote-tracking branch 'tadl/rel_2_5' into tadl_2_5

Conflicts:
README

accumulated DB upgrade tweaks

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

Replace a bare URL in an 856 link label

If a bare URL has survived this far as an 856 link label, replace it
with something generic rather than displaying a difficult-to-wrap
URL.

Signed-off-by: Jeff Godin <jgodin@tadl.org>

Don't try to use Titlecase wrapper if not defined

Don't try to call the function passed from Lingua::EN::Titlecase
function if the tc variable is undefined.

This avoids having blank titles when the template is used in an
environment where EGWeb isn't passing the function to the templates.

Signed-off-by: Jeff Godin <jgodin@tadl.org>

Auto re-bill forgiven overdues on lost checkin

Auto re-bill forgiven overdues on checkin of a lost item.

This is done by looking for a Forgive payment with a special note,
then creating a new billing of type Overdue materials. Nothing is
done to the previous daily billings on the transaction.

Signed-off-by: Jeff Godin <jgodin@tadl.org>

Support forgive-vs void on Lost/Lost-checkin

Constants and strings for new OU settings:

circ.forgive_overdue_on_lost
circ.forgive_lost_on_checkin
circ.forgive_lost_proc_fee_on_checkin

These are intended for use as an alternative to their "void"
counterparts in situations where the library does not refund
fees and fines, and wishes funds taken in for one purpose or
billing type to not be re-applied toward a different billing
type as a result of voiding bills after they have been paid.

When circ.forgive_overdue_on_lost is set, attempt to make a
payment of type "Forgive" on the transaction for the amount
of outstanding overdue billings.

Only outstanding bills of type 1 (Overdue materials) will
be paid, and only the first contiguous grouping. If the
system finds an outstanding billing of type other than 1,
it will pay what it has found up to that point.

circ.forgive_overdue_on_lost has priority over the "void"
version of the same setting, if both happen to be set.

When circ.forgive_lost_on_checkin and/or
circ.forgive_lost_proc_fee_on_checkin is set, make a Forgive payment
on outstanding Lost and Lost Processing Fee bills on checkin of a
lost item.

New utility function:

outstanding_bills_for_circ (in AppUtils) accepts an editor and a
circ object, and will return a reference to an array of outstanding
billing objects. this has been broken out of the forgive_overdues
sub, as it will be useful elsewhere.

Signed-off-by: Jeff Godin <jgodin@tadl.org>
Conflicts:
Open-ILS/src/perlmods/lib/OpenILS/Application/AppUtils.pm
Open-ILS/src/perlmods/lib/OpenILS/Application/Cat/AssetCommon.pm
Open-ILS/src/perlmods/lib/OpenILS/Application/Circ/Circulate.pm

Display 245$a as Title Case in TPAC

Using Lingua::EN::Titlecase, perform a "Title Case" case transform on the 245$a
value as displayed in TPAC.

TODO: Make this configurable, allow it to fail gracefully when
Lingua::EN::Titlecase is not present.

Signed-off-by: Jeff Godin <jgodin@tadl.org>
Conflicts:
Open-ILS/src/templates/opac/parts/misc_util.tt2

Add username suggestion based on patron name

Add support for suggesting a username based on the patron's first
and last name.

Currently, the scheme for suggestion is a lowercased first initial +
last name, with any characters other than [a-z] removed.

No effort is made to suggest an available username, but if the
suggested username is already in use, the standard "username in use"
message is displayed next to the username field.

Staff can initiate the username suggestion by clicking the new
button next to the username field, and usernames are automatically
suggested when loading a staged user.

Signed-off-by: Jeff Godin <jgodin@tadl.org>

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>

Add button to update expire date field

When updating/renewing a patron (expired or not), it would be
convenient to have a means of populating the expire date field with
a value of perm_interval + today. Currently, staff are required to
manually enter a date, or could toggle the profile from one profile
then back to the desired profile, but this is not ideal.

This commit adds a new button next to the expire date, which when
activated will re-calculate the date based on the current profile's
perm_interval and today's date.

This is similar to how the expire date is populated when creating a
new user, or when changing the profile group. A button is used here
so that the updating of the expire date is an intentional process,
not one that happens upon any edit.

Signed-off-by: Jeff Godin <jgodin@tadl.org>
Conflicts:
Open-ILS/web/js/dojo/openils/actor/nls/register.js

Treat empty username as invalid in user editor

There seems to be undesired interaction between the required
attribute and the isValid method on a dojo/dijit ValidationTextBox.
If both are set, isValid needs to check for an empty value,
otherwise the field is considered valid the moment it gains focus.

We don't want an empty username to be considered valid, because we
know that we will be unable to save the user.

We can work around this by teaching the isValid function to consider
an empty value as invalid.

Signed-off-by: Jeff Godin <jgodin@tadl.org>

Don't show contact invalidators for new users

Don't show contact invalidator buttons for new users. This affects
staged users and cloned users, where the contact fields have a value
when the editor initially loads.

Signed-off-by: Jeff Godin <jgodin@tadl.org>
Conflicts:
Open-ILS/web/js/ui/default/actor/user/register.js

Skip dup username check when username unchanged

Changing a username, then changing back to the user's current
username results in a duplicate username detection -- a false
positive due to the API call finding a user with that username --
the current user. This prevents you from saving any other changes
you may have made to the user.

This can be prevented by skipping the duplicate username check if
the user is not new and the username in the field matches the
username stored in the in-memory Javascript representation of the
user.

There's still a potential corner case which could be avoided by
using a new/modified API call to either accept an "except this
au.id" argument or to return the au.id of the found user, but I
don't see a need for that at this point.

Signed-off-by: Jeff Godin <jgodin@tadl.org>

Add alert in user editor for method errors

Add an alert() in the user editor so that method errors are not
silent.

Without this, a method error (such as when saving a patron fails at
the db level) is silent. Better to at least inform the user that
something is amiss.

Signed-off-by: Jeff Godin <jgodin@tadl.org>

Make deactivation of card optional

Make deactivation of current card optional when using Replace
Barcode function.

Signed-off-by: Jeff Godin <jgodin@tadl.org>

Add explicit sort to collections API with limit

Add an explicit ORDER BY au.id ASC when limiting collections API
calls. This ensures that the same users will be returned on
subsequent calls, assuming no other relevant changes in the data.

Signed-off-by: Jeff Godin <jgodin@tadl.org>

Add limit to penalty variant of collections call

Add a limit and sort order param to the API method
open-ils.collections.users_of_interest.warning_penalty.retrieve

When a limit is supplied, users are sorted by set_date (default
DESC, for most recently set). Supplying "ASC" as a param will sort
by oldest.

Signed-off-by: Jeff Godin <jgodin@tadl.org>
Conflicts:
Open-ILS/src/perlmods/lib/OpenILS/Application/Collections.pm

Show patron credit in patron summary

Show patron credit in patron summary

Signed-off-by: Jeff Godin <jgodin@tadl.org>

HACK: Ignore backdate on all SIP checkins

Very temporary workaround for LP 978287, which affects TADL on every
SIP checkin, since every SIP checkin is being backdated in our
environment.

A more elegant solution to be backported from master when it is
ready. For now, we use this workaround and try to avoid incorrect
overdue fines being generated this weekend.

Signed-off-by: Jeff Godin <jgodin@tadl.org>

Default to "record external" credit card payment

In the staff client, default to the "record externally processed
payment" for Credit Card payment type.

Signed-off-by: Jeff Godin <jgodin@tadl.org>

Adjust wording of SIP 'expired' message

Adjust wording of SIP 'expired' message on patrons -- this is
displayed on selfcheck terminals.

Signed-off-by: Jeff Godin <jgodin@tadl.org>

TPAC: Add physical_dept arg/cookie

Add physical_dept variable to TPAC -- set by url argument, backed by
a session cookie.

This is accessible to TPAC templates as ctx.physical_dept, and can
be used by conditionals to show/hide UI components based on the
provided location of a catalog terminal within a library.

physical_dept is independent from, and can be used in combination
with, physical_loc: /eg/opac/home?physical_loc=2&physical_dept=teen

Signed-off-by: Jeff Godin <jgodin@tadl.org>
Conflicts:
Open-ILS/src/perlmods/lib/OpenILS/WWW/EGCatLoader.pm

Suggest password change on login if 4 digit PIN

Suggest that the user change their password at login if the password
is a four digit PIN. Do this by redirecting to the update_password
page with a CGI param to trigger an optional message.

Signed-off-by: Jeff Godin <jgodin@tadl.org>

Enable optional shared ses cookie for SSO-like use

Enable a shared ses cookie which is set at the domain level and can
then be used by other web based systems to verify an Evergreen user.
One example would be EZproxy with one of the CGI authentication
options.

TODO: move shared_ses_domain_regex into a single location

Signed-off-by: Jeff Godin <jgodin@tadl.org>

Add tadlHoldName macro to print_custom.js

Add tadlHoldName macro to print_custom.js, used for printing either
the patron alias or a string based on the patron first and last
name.

Signed-off-by: Jeff Godin <jgodin@tadl.org>

Add print_custom.js for tadlSlipDate

We use tadlSlipDate on hold slips. This replaces that element with a
formatted date 7 days in the future.

Originally from https://gist.github.com/179cd9a682a509537cb8

Signed-off-by: Jeff Godin <jgodin@tadl.org>

Enable clearing of an alias via OPAC

Enable clearing of the user's holdshelf alias via the OPAC.

Signed-off-by: Jeff Godin <jgodin@tadl.org>

Only request review sources TADL subscribes to

For Syndetic Solutions, only request those review
sources which TADL subscribes to. This includes
sources not requested by default, and removes
sources which we do not use. The ordering of
the review sources is also intentional.

Signed-off-by: Jeff Godin <jgodin@tadl.org>

Transform TADL patron barcodes in xul

Transform TADL patron barcodes in barcode_entry.xul, used when
retrieving a patron from the system.

Signed-off-by: Jeff Godin <jgodin@tadl.org>

Make patron alias unique test case insensitive

When testing a patron alias to determine if it is unique, ensure
that the comparison is done without regard for case.

Case is preserved, but when searching for an existing matching
alias, both values are lowercased using evergreen.lowercase().

Signed-off-by: Jeff Godin <jgodin@tadl.org>

Add ability for users to set their alias in TPAC

New option under Personal Information to display and allow editing
of a user's holdshelf alias.

Proof of concept, much potential for further enhancement.

Signed-off-by: Jeff Godin <jgodin@tadl.org>

Apply TADL-specific SIP patron barcode transforms

TADL has some SIP clients which are not capable of removing extra characters
from some scanned patron barcodes. Thus, we handle that on the SIP server side.

If no active card is found with the barcode provided in the SIP request, first
see if it starts with an uppercase letter. If so, lowercase it, then determine
if certain length requirements are met, and if so also truncate the value.
Attempt to find an active card via the resulting value, and return the usual
error if not found.

Signed-off-by: Jeff Godin <jgodin@tadl.org>

Skip Piwik tracking for staff

This change may be temporary. Since we do not currently have an
approach for incorporating Piwik tracking code in the catalog
interface within the staff client (within the oils:// protocol
wrapper), omit Piwik tracking code when is_staff is true.

Signed-off-by: Jeff Godin <jgodin@tadl.org>

Changes to mkurl to avoid "escaping" oils wrapper

Port changes to mkurl macro to avoid "escaping" the oils wrapper.

Observed symmptom was that we were no longer in "staff mode" in the
staff client's catalog interfaces after clicking certain links.

Signed-off-by: Jeff Godin <jgodin@tadl.org>

Remove nbsp div in facets

Remove the nbsp div in facets -- upstream uses a different approach
now. We may have need to port further facet changes, but this is it
for now.

Signed-off-by: Jeff Godin <jgodin@tadl.org>

Port facet count limiting from 2.5 stock

Port changes to facet count limiting (and remove the in-TT sort).

Signed-off-by: Jeff Godin <jgodin@tadl.org>

Display "MeLCat ID" (patron database id)

Display "MeLCat ID" (patron database id) to patrons under "My
Account".

Signed-off-by: Jeff Godin <jgodin@tadl.org>

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>

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>

Pulling in new version upgrade

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

LP#1251774 exit and alert on multiple payments per xact

If more than one payment for a transaction is received, exit the API
call early and return a MULTIPLE_PAYMENTS_FOR_XACT event.

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

Enforce one-payment-per-xact-per-call

There is no legitimate reason for a transaction to receive more than
one payment per call to open-ils.circ open-ils.circ.money.payment,
but we have seen the staff client generate such a data structure. This
leads to seemingly duplicate payments, and is bad all around.  So,
we will enforce the restriction by taking only the first payment per
xact in the list of payments.

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

Replace erroneous calls to $e->retrieve_authority_record($rec_id).

On lines 153 and 175 of OpenILS::Application::Cat::Authority calls
like the above should be $e->retrieve_authority_record_entry($rec_id).
This branch replaces the two misspelled calls with the proper ones.

Signed-off-by: Jason Stephenson <jstephenson@mvlc.org>
Signed-off-by: Mike Rylander <mrylander@gmail.com>

LP1093856 fix Fast Item Add with Z39.50 import

The first time I tried this fix I ran afoul of
https://bugs.launchpad.net/evergreen/+bug/787561

But a second attempt worked.  So I didn't kill
two birds with one stone, but this should take
care of 1093856.

Signed-off-by: Jason Etheridge <jason@esilibrary.com>
Signed-off-by: Mike Rylander <mrylander@gmail.com>

LP#1037171 Removed Expert Search paramters from subject links

After performing an Expert Search, if a user visits a record and then
clicks on one of the author, subject, or series search links within that
record, then they are given the results of the Expert Search again
instead of the relevant subject or author search.

This is happening because the Expert Search parameters are retained in
the URLs via the TT2 MACRO mkurl.  In particular, the tag parameter is
used to identify an Expert Search, so if it is present in a URL, than an
Expert Search will be attempted regardless of which other paramaters are
in the URL.

To fix this, I created two varialbes within header.tt2 above the
definition of mkurl.  These variables group common URL paramters for
Expert Searches and General Search parameters together.  These variables
can be used wherever mkurl is used to strip out unneeded search
paramters.

There is a third variable that is already present within many of these
pages named stop_parms.  While I initially tried to modify this
variable, I found that there are some instances where the general search
parameters should be retained while the Expert Search parameters must go.

This commit uses these three variables togehter as input into the third
arugment of the mkurl MACRO.  The third arugment specifies which
paramtes to remove from the previous URL before making a new URL based
on the pervious URL.

I also added the query parameter to the Place Hold's hold_source_page
URL variable.  I did this, so that the query terms would be preserved
once the user is done placing the hold or decides to cancle the placing
of the hold.

As well, I used the new variables to remove Expert Search paramters
from the Advanced Search links on the results and record pages.

Conflicts:

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

Signed-off-by: Liam Whalen <whalen.ld@gmail.com>
Signed-off-by: Mike Rylander <mrylander@gmail.com>

Docs: Remove ref to missing included file

This file was moved into the 2.5 release notes. If others believe it
belongs in the main documentation, it can easily be added in again. For
now, we will remove the reference because it only generates an asciidoc
warning.

Signed-off-by: Remington Steed <rjs7@calvin.edu>

Docs: Fix leveloffset bug, raise REL NOTES level

We should restrict our use of the asciidoc leveloffset command to the
root.txt file, since it is a global setting and cannot be set relative
to an existing offset.

This commit also raises the Release Notes section to the highest level
in order to prevent the lowest level subsections within it from being
lost. This is in response to asciidoc errors:

    WARNING: RELEASE_NOTES_2_5.txt: line 950: missing section: [sect5]

Signed-off-by: Remington Steed <rjs7@calvin.edu>

Document how to load concerto sample data

We might as well make it as easy as possible for people to use the
sample dataset that we created!

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

Add basic docs for testing with pgTAP

This can be expanded to describe how to create pgTAP tests later. For
now, let's help people actually set up their environment and test.

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

Fix untranslatable strings in several dijits:

AutoGrid, EditPane, FlattenerGrid, GridColumnPicker and HoldingCode.

[LFW: One string amended in conflict resolution; commit message edited
to wrap; one line of code moved (this.nls assignment in
GridColumnPicker.js).]

Signed-off-by: Pasi Kallinen <pasi.kallinen@pttk.fi>
Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>

LP1251424 - Fix for submit button when placing holds in staff client

I've moved the initializing of the cur_hold_barcode variable
to within the function rather than floating outside of it.
This seemed to fix an intermittent issue where the submit
button would not become activatable when first going to the
place hold screen via the staff client. To create the hold,
ths user had to cycle through the "Place hold by patron barcode"
option and then back to "Place hold for me" option.

It looked like behind the scenes for certain org units that
variable would not be set and the button will not open until
it at least has something. This causes it to be set on the initial
loading of the screen.

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

lp1012237 HOLD_ITEM_CHECKED_OUT display

Added a definition to the HOLD_ITEM_CHECKED_OUT event in ils_events.xml.
Trying to be generic between the opac and the staff client the message
reads, "The item you have attempted to place on hold is already checked
out to the requestor."

[LFW: I made a tiny spelling change to be consistent with other
appearances in Evergreen of the term "requestor".]

Signed-off-by: Garry Collum <gcollum@gmail.com>
Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>

Update version of release notes in 2.5 docs

Signed-off-by: Remington Steed <rjs7@calvin.edu>

Acq: Improvements to account-matching incoming EDI messages

The way the EDI fetcher works gives us a problem. That process iterates
over EDI accounts for which it has FTP host and credential information,
downloads documents from each of those sites, and files the messages
within those documents under the EDI account from which the login
credentials came. The problem is that in practice the exact same host
and login information is used by multiple accounts under the same
vendor, and files relating to these sub-accounts are commingled, so that
you can't make the decision about which messages should be filed under
which accounts based on the name of the document or its location. You
have to make that decision later, based on its contents.

We are already incompletely doing this, distinguishing between
sub-accounts under which we could file our messages when the vendor
specifies the buyer's SAN next to the specific sub-account number *and*
those sub-accounts belong to different Evergreen org units. We still
need ways to distinguish in other cases.

This will do what is natural for at least one vendor, and match the
message content against the vendacct field of the acq.edi_account table.

*Also,*

We were re-retrieving the working acq.edi_message row from the database
before writing it, throwing away possible changes to the object in hand
made by O::A::Acq::EDI::process_parsed_msg(). We should only do that in
the case where that function has raised an exception.

We were doing the same kind of thing in another place actually inside
process_parsed_msg() where we set the edi_message's purchase_order field
based on the first lineitem encountered if the message itself didn't
specify a valid PO identifier.

This supports making account-correction work for ORDRSP messages in
addition to INVOIC messages.

We also propagate that same correction to the provider and shipper
fields of any invoices that get created from said edi_message.

Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>
Signed-off-by: Kathy Lussier <klussier@masslnc.org>
Signed-off-by: Ben Shum <bshum@biblio.org>

TPAC: Remove fixed height from 'View My List' button

The TPAC button styles have been largely consolidated, but this little
outlier got missed.  It doesn't do anything of noticeable value in
recent browsers, and in fact hurts the display by causing the "Sort by"
widget to "hang" when resizing the window to certain intermediate
sizes.  Remove it.

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

Forward-port upgrade scripts from recent releases

In accordance with LP #894052.

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

Update OpenSRF minimum version in install docs

Evergreen now requires OpenSRF 2.2.1 or later, so it makes sense to be
clear about it.

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

Update upgrade instructions for 2.5.0 release

- Bump up minimum OpenSRF to 2.2.1
- Replace all 2.5-beta1 markers (and few stray 2.4 markers) with 2.5.0

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

Translation updates - newpot

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

Translation updates - po files

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

Don't auto-logout TPAC in staff client

Wrap the meta refresh in TPAC within a check for ctx.is_staff, so
that idle TPAC tabs will not log out when running within the staff
client.

This is intended to resolve the primary cause of symptoms mentioned
in bug 1036318 "OPAC timeout within the client"

Signed-off-by: Jeff Godin <jgodin@tadl.org>
Signed-off-by: Jason Stephenson <jstephenson@mvlc.org>

Fix typos in 2.5 release notes

These typos were fixed in the separate (now deleted) files, but only
after the collective file had been built.  This commit transfers those
fixes to the built file.

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

Delete separate NEXT release notes

These files have been carefully woven together into a single document,
so it is time to part with these bits and pieces.

Also, add a DONOTREMOVE file to preserve the SIP folder in the same
fashion as the others.

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

Remove manual TPAC locale applications

OpenILS::Utils::CStoreEditor::default_locale is no longer set, since the
opensrf client locale is now global.  Remove all references to this
variable.

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

Default per-request TPAC locale

Set the OpenSRF default client locale for each TPAC request to ensure
all outbound OpenSRF calls use the requested locale.  When done, reset
the locale to the OpenSRF default.

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

Improve locale picker positioning

The locale picker was dependant on some header padding which was
removed.  Let's position it more directly.

Also, add a small left margin to the 'dashboard' area.

Signed-off-by: Dan Wells <dbw2@calvin.edu>
Signed-off-by: Pasi Kallinen <pasi.kallinen@pttk.fi>

Only one of these tests was valid, removing the bad one

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

Live pgTAP test for 0847, authority overlay generator fixes

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

Stamping 0847: improved auth overlay generation

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

Add indicators; Remove junk tags

First, we were not supplying authority tag indicators when generating the
overlay template.  The xml merge function does not currently transfer
indicators, but we need to do this anyway because newer MARC::Field version
balk at fields with no indicators.

Second, PG 9.1+ adds wrapper tags to xml content that is passed to XMLELEMENT
as an array.  So, we'll use XMLAGG to get around that.

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

Add --refresh mode to completely rewrite linking

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

Use all subfield values to link authority records to bibs

Given an Evergreen instance with two authority records loaded, one
being a more specific than the other via a repeated subdivision subfield,
we must make sure that we use all the bib-supplied subfield values when
attempting to auto-link to the correct authority.  Otherwise, the "shorter"
authority record may be selected as appropriate, and data in the bib record
would be lost.

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

Clean up 2.5 upgrade instructions

This commit cleans up a few tiny issues and a large formatting bug,
fixes a filename, and reorders the linux distros to put the newest ones
first. The content related changes are:

  - fix filename in 'tar' command
  - reorder linux distros (in two places)
  - fix various punctuation (add comma; remove colon in header)

And the AsciiDoc changes:

  - add asciidoc '+'s before and after 'indexterm' commands to fix
    broken ordered list formatting
  - remove an unnecessary asciidoc '+'
  - correct the length of an asciidoc block header line (doesn't change
    behavior, but helps readability of the text file)

Signed-off-by: Remington Steed <rjs7@calvin.edu>

Stamping 0846: overlay subfield order fix

Respect source XML subfield order during overlay

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

Respect source XML subfield order during overlay

When adding specific subfields to a field as described by an add
or replace rule we were adding them in rule-order.  Instead, they
should be added in source-XML physical order.  This commit does
that.

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

Stamping upgrade script for 0845: browse pivots should be stable.

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

Make browse pivot DB funcs STABLE

Since these are just convenience functions doing only selects, these
should be STABLE safe, and making this change allows browse to be
functional on production-size data sets.

Signed-off-by: Dan Wells <dbw2@calvin.edu>
Signed-off-by: Chris Sharp <csharp@georgialibraries.org>
Signed-off-by: Jason Stephenson <jstephenson@mvlc.org>

Handle Missing copy status (4) in checkin_via_barcode2 in circ.util.js.

We treate just like available and reshelving.

Signed-off-by: Jason Stephenson <jstephenson@mvlc.org>
Signed-off-by: Kathy Lussier <klussier@masslnc.org>
Signed-off-by: Dan Wells <dbw2@calvin.edu>

Make sure that # can be used in auth browse

Because we use URLs to call the SuperCat browse API, we need to
be more careful about encoding parameters.  #, in particular, breaks
auth browse because it's seen as a URL fragment separator by the browser
and web server.

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

Touch-up record detail extras display

1) Remove unsightly gaps, which gave it an overly "stripey"
appearance.

2) Switch off-green arrow graphic with a text-based arrow character.

3) Simplify unnecessarily complex positioning styles.

4) Lighten and add subtle border to give depth and definition to
section headers.

Signed-off-by: Dan Wells <dbw2@calvin.edu>
Signed-off-by: Dan Scott <dscott@laurentian.ca>

Match main-content right margin to left

Having a margin on one side but not the other just looks "off". Let's
match it on the right.

Signed-off-by: Dan Wells <dbw2@calvin.edu>
Signed-off-by: Dan Scott <dscott@laurentian.ca>

Tweak TPAC header layout styles

The TPAC header relies heavily on fixed pixel measurements and relative
positioning, which makes it fragile and overly difficult to customize.

To improve the situation, we can greatly reduce the number of styles
while still achieving the same overall effect.

Signed-off-by: Dan Wells <dbw2@calvin.edu>
Signed-off-by: Dan Scott <dscott@laurentian.ca>

RDA / TPAC: Add 264 for pubinfo graphic 880 lookups

Per https://bugs.launchpad.net/evergreen/+bug/1243746 the publication
info in search results was coming up empty for RDA records because they
use 264 for publication info instead of 260.

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

Docs 2.5: Fix floating groups heading level (2nd try)

The first attempt successfully pushed the Floating Groups chapter to
its proper place in the table of contents, but it also mistakenly pushed
many following chapters down one heading level. This commit follows the
approach already common in the root.txt file, and also undoes the
previous approach of shifting the heading level within the chapter file
itself. I suspect the previous approach only failed because it lacked a
blank line before the final 'leveloffset' command, but this new
approach is better because it is already in common use.

Signed-off-by: Remington Steed <rjs7@calvin.edu>

Serials Docs: Update for new features in 2.5

This commit adds a "Notes" section to the chapter on Receiving, including
a subsection on using the new "alerts" feature in the receiving interface.
It also adds a new screenshot for the "Notes" section and updates two
screenshots that have changed in 2.5. Finally, this commit updates the
Serials Copy Template doc to reflect the new 2.5 Floating Groups feature.

Signed-off-by: Remington Steed <rjs7@calvin.edu>

Serials Docs: Improve format, fix typos, and update

This commit makes several minor changes to the Serials doc files, mostly
changes in formatting or corrections of bugs/typos. This list covers
most of the changes:

    - Add row on 'receiving' to Serials interfaces comparison table
    - Improve formatting of actionable items (using bold and arrows),
      e.g. Click on *First Menu* -> *Next Menu* -> *Last Menu*
    - Standardize formatting of form field names (using italics)
    - Improve unclear wording
    - Correct an outdated description of copy template 'status' field
      (changed before 2.3)
    - Change heading level for a subsection

Signed-off-by: Remington Steed <rjs7@calvin.edu>

Docs 2.5: Fix floating groups heading level

The new Floating Groups docs are acting like a Docs Section rather than
a Docs Chapter. This commit should bump all the headings down one level
so they will take their appropriate place in the TOC.

Signed-off-by: Remington Steed <rjs7@calvin.edu>

LP#1240207: Spellchecked the docs

Signed-off-by: Ben Ostrowsky <sylvar@gmail.com>
Signed-off-by: Remington Steed <rjs7@calvin.edu>