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>

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>

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>

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>

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>

Append ?r= to rdetail url for non-TPAC staff

Now that TPAC is an option in the staff client, interfaces no longer
append '?r=' to the value of urls['opac_rdetail']. Therefore,
custom.js needs to be updated to include the '?r=' so that View in
Catalog and such does not break.

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>

tpac : uri escape added content key

The added content key may have spaces, etc.  URI escape the key to
prevent Net::HTTP from croaking in the URL.

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

tpac added content default type improvements

* In the pre-JS state, only choose a default display type when the type
is known to have data.

* If no default type is chosen for the main tab link, because no data
was available to the template, set the default from the first added
content type determined to be available via JS.

* Move added content JS to separate template file for easier override.

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

TPAC added content data retrieval sanity check

On the backend, avoid fetching added content data when initial request
returns 404.  This prevents edge case where a specific type may be
selected (e.g. via bookmark) even though no data exists for the type.

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

TPAC added content shows default tab

Use the first available content as the default tab to show when the main
added content tab is expanded.

Note, if no content is known to exist until after JS has run, it's still
possible to expand the main tab and have no content selected by default.

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

TPAC added content improvements

* Style the sub-tabs to indicate which is selected
* If the template knows there is no content, hide the main tab
* cleanup

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

TPAC added content integration

https://bugs.launchpad.net/evergreen/+bug/984963

This adds a new tab on the detail page called Additional Content
(suggestions welcome).  When the tab is expanded, available content is
presented to the user via a series of sub-tabs.

At the start of loading the record detail page, kick off a series of
asynchronous HTTP HEAD requests, one per type of added content.  At the
end of context loading, read the results of the HTTP requests for any
that have completed.  If the status for a type is 200, the type is
marked as available.  If it's not 200 (usually 404) it's marked as not
available.  Otherwise, it's marked as unknown.

In the template, available content produces a link the user can click
to view the content.  Non-available content produces no links.  Unknown
content produces a link that may lead to content or no content when
JS/dojo is disabled.  When dojo is enabled, unknown content sends a
series of JS queries to determine the state of the content
asynchronously and results in displayed links for any type that is later
determined to have content.

TODO: when the user expands the Additional Content tab, no type is
chosen by default.  A default (available) type needs to be selected on
initial display.

TODO: better styling, particularly in the sub-tabs.

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

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>

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>

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>

Create custom.js for staff client

Using custom.js, specify a jspac skin name for the staff client in a
tpac-aware fashion.

The "tadl-staff" skin will likely just be a copy of the default
skin, or possibly a symlink.

We want to be able to redirect requests for the "default" skin's xml
pages to tpac without redirecting staff.

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

Fix IDL and OU setting check for staged users

The IDL had references to sequences that do not exist:

staging.usr_stage_row_id_seq -- a typo, fixed

In the case of staging.billing_address_stage_row_id_seq, the
staging.billing_address_stage table is created with LIKE,
and uses the sequence staging.mailing_address_stage_row_id_seq

The OU setting check for the open-ils.actor.user.stage.create API
call was not passing an org unit, and would always fail.

We now pass the home_ou of the user being staged.

At this point, the opac.allow_pending_user OU setting type must
be manually created before its value can be set.

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>

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>

Teach Syndetic.pm to pass UPC in requests

Teach OpenILS::WWW::AddedContent::Syndetic to pass UPC when
requesting added content from the source.

Protect against warnings when no isbn/upc provided

Protect against Perl giving us uninitialized value warnings when
isbn or upc has not been provided.

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

Finish Added Content by record ID

OpenILS::WWW::AddedContent now supports fetching content by record ID.

To start, this lets us request content in a way that uses both ISBN
and UPC as keys, assuming the AC handler and provider support it.

The previous method of requesting AC data continues to be
supported, and should be unaffected.

Business::ISBN is used to clean ISBN values when fetching by record ID

If we find no valid ISBN and no UPC values in the record,
do not attempt to call an AC handler. This will need
to change when we have AC handlers that rely on other
possible keys.

Attempt to serve from cache before fetching keys from the bib
record in the database.

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

Start of AC by record ID from 2011 hackfest

Rough, untested cut. use rec id as they added content key;
fetch isbn/upc from full_rec; push full set of isbns and upcs
down to handler code

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

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>

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>

Support SIP payment type 02 - credit card

Record SIP payments with payment type 02 as credit_card_payment in
Evergreen.

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

Add SIP support for Fine Items field

Add support for SIP Fine Items, msg 64 field AV.

Display outstanding balance, last_billing_type, and a description.

For circulation xacts, description is title / author from MODS.

For grocery xacts, description is last_billing_note.

Wishlist LP 1012328

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

Pass authtoken to OpenILS::SIP::Patron objects

To support retrieval of billing details, OpenILS::SIP::Patron
objects need an auth token.

Teach OpenILS::SIP->find_patron to pass patron objects an auth
token, and also move existing "find patron by what key: barcode or
database id?" logic into find_patron (and out of
OpenILS::SIP::Patron->new).

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

patron name border color for Notes

not Notes specifically, but this catches any non-alert non-blocking penalties
and prevents a colorless border from happening

Colors chosen from mrpeters' patch at
https://bugs.launchpad.net/evergreen/+bug/921817

Thanks!

Signed-off-by: Jason Etheridge <jason@esilibrary.com>
Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>
Conflicts:

Open-ILS/xul/staff_client/server/skin/patron_display.css

Documentation: Fix typo in upgrade instructions reported in launchpad bug #1028978.

https://bugs.launchpad.net/evergreen/+bug/1028978

Signed-off-by: Robert Soulliere <robert.soulliere@mohawkcollege.ca>

Add evergreen.get_locale_name() function to base schema

I added the evergreen.get_locale_name() function to the database schema
upgrades in 0723 but failed to add it to the base schema, resulting in
errors like: 'egweb: template error: undef error - Can't call method
"maketext"'. Get rid of that problem!

Signed-off-by: Dan Scott <dscott@laurentian.ca>
Signed-off-by: Thomas Berezansky <tsbere@mvlc.org>

2.2 upgrade missing vandelay.authority_match.quality column

This column slipped through the cracks of the 2.1 to 2.2 upgrade.

ALTER TABLE vandelay.authority_match ADD COLUMN quality INTEGER NOT NULL DEFAULT 0;

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

TPAC: Decode translated strings into UTF8

Without the _decode pragma, Locale::Maketext::Lexixcon did not
understand that it was being handed Unicode and generated some funky
output in the TPAC templates.

Signed-off-by: Dan Scott <dscott@laurentian.ca>
Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>

TPAC: Implement a locale picker

In situations in which more than a single locale is configured, display
a locale picker in the TPAC header based on the registered locales. We
set the eg_locale cookie if passed a set_eg_locale GET param. Default
the selection to the currently selected locale (if any) and resubmit the
current page request.

Grabs the localized locale names, if available, from the database;
otherwise falls back to the en-US version of the locale names.

The locale picker form resubmits the current page with any variables
that were present in the current page request, so that switching locales
should still maintain state.

Signed-off-by: Dan Scott <dscott@laurentian.ca>
Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>
Conflicts:

Open-ILS/web/css/skin/default/opac/style.css

Add constraint to ACQ PO state value

Limit 'state' values to

'new','pending','on-order','received','cancelled'

https://bugs.launchpad.net/evergreen/+bug/893193

Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Michael Peters <mrpeters@library.in.gov>
Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>
Conflicts:

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

Add component to *force* external browser use

For http/https urls that don't go to the host we are logged into.

Signed-off-by: Thomas Berezansky <tsbere@mvlc.org>
Signed-off-by: Michael Peters <mrpeters@library.in.gov>
Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>

Allow opening of links in default browser

And do so for the "Get Help With Evergreen" portal link.

Signed-off-by: Thomas Berezansky <tsbere@mvlc.org>
Signed-off-by: Michael Peters <mrpeters@library.in.gov>
Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>

LP: Allow user to change activation options for suspended holds.

Activation options were hidden from users when holds were in a suspended
state.

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

Fix LP800480, ACQ - Vendor Invoice Won't Save

1. Upon submitting the form, the client prepares two auxiliary data
objects containing service charges and line items. The two objects are
normally 'fleshed', ie, contain references to other data objects, but in
the versions sent to the server, they must be 'unfleshed', ie, the
referent objects are converted back to ID values. If the user resubmits
the form, the client should not unflesh again. However, the software for
preparing line items contain lines of code that unfleshes without
checking if the objects are already unflesh. When unfleshing a second
time, we get an uncaught reference error. Ironically, the software also
contains lines of code to do it correctly, so the errant lines of code
are also duplicates.

To fix problem 1, we remove the duplicate errant lines of code. We
define a helper unflesh() method and use it to replace the current lines
of code to unflesh.

2. When the user submits the form without filling in required data
fields in the invoice, including Vendor Invoice ID, the client does not
validate before making a service request. The server tries to complete
the database transaction, but gets an error.  When the response comes
back, the client shows the same form so that the user can retry.
However, the message alert to the user is not informative; it indicates
an error at the database level, but does not indicate the probable
reason.

To fix problem 2, We move the lines of code preparing the invoice object
earlier in the sequence.  We define a helper method mapValues() to
prepare the invoice object from the values in the UI widget object.
mapValues() will return an error object if it detects that required
values are null. We check for the error object, and will show an alert
message to the user and abort the submit operation early, allowing the
user to retry.

3. When an invoice with line items is saved, the invoice is re-rendered
with new buttons to allow receiving. It also contains a new button to
enable the user to view the line items in list format. However, it does
not work in a non-Firefox browser, because a debug statement using the
'dump()' function is left uncommented. (Inspection of all other
appearances of dump() show they are all commented out.) There is an
uncaught reference because dump() is not found.

Signed-off-by: James Fournie <jfournie@sitka.bclibraries.ca>
Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>

Documentation: Add line breaks in authorities chapter to fix
asciidoc formatting.

Signed-off-by: Robert Soulliere <robert.soulliere@mohawkcollege.ca>

Fix TPAC recognition of logged-in users via http

Fixes bug: LP 957375
tpac: catalog does not immediately recognize "stay logged in" users
https://bugs.launchpad.net/evergreen/+bug/957375

The issue: during the login process, the user is redirected to an
https connection and receives a cookie with the "secure" flag set.

Since this cookie is not sent over normal http connections,
following an external link or manually entering an http catalog url
will result in an unexpected "not logged in" experience.

Selecting the login link or any other action requiring login is
enough to return to the "logged in" experience, without a need to
re-enter credentials. Still, we can do better.

This affects users who have checked the persistent login checkbox
and those who have left it unchecked. Users selecting the persistent
login option are more likely to encounter the issue, especially if
the link they typically follow/enter is to a non-https catalog url.

The solution:

Add a new cookie (constant COOKIE_LOGGEDIN)
 - contains a "hint" that the user may be logged in already
 - set/cleared at login/logout time
 - contains no sensitive auth/session data
 - "secure" flag not set (sent for both http and https requests)

When a user's browser presents the COOKIE_LOGGEDIN cookie in a
request for a non-https URL, the user is automatically redirected to
the https version of that url.

At that time, if the user has a valid COOKIE_SES cookie set, they
will be recognized as a logged in user. If their COOKIE_SES value is
no longer valid, a logout is performed, clearing both cookies.

If for some reason the COOKIE_SES cookie is not present but the
COOKIE_LOGGEDIN is present, there is a harmless redirection to https
and the user is not logged in, but can log in via the usual means.

To test, after applying:
- log in to the TPAC
- navigate to http://example/eg/opac/home
- You should be redirected to https://example/eg/opac/home and you
  should see your name, count of checked out / on hold / etc items

Prior to this, the above steps would result in you remaining on the
http URL and seeing only a "Your Account Log In" button.

Signed-off-by: Jeff Godin <jgodin@tadl.org>
Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>

TPAC - repair editions statement display

Change && back to || and repair editions display in TPAC.

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

wording improvement for 2.2 release notes

Patch by Sally Fortin <sfortin@esilibrary.com> to fix
typo noticed by James Keenan.

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

JSPAC: Point to TPAC from <noscript> section

TPAC was built for browsers with no JavaScript. Ironically, we're still
pointing to the SlimPAC from the <noscript> section of the JSPAC... fix
that.

Signed-off-by: Dan Scott <dscott@laurentian.ca>
Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>

TPAC: Set autofocus appropriately for different contexts

At login, the basic search bar was fighting (and winning) for the
default focus rather than the username field. Additionally, in the
advanced search pages, no field was getting focus. Therefore, check to
see if we expect to be in a context where it makes sense to set the
default focus to the basic search input field; otherwise, set the focus
to the login username field or the appropriate advanced search field.

Signed-off-by: Dan Scott <dscott@laurentian.ca>
Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>

Documentation:  Fix typo in upgrade instructions. Bug #1027752 reported  by Jeff Green.

Signed-off-by: Robert Soulliere <robert.soulliere@mohawkcollege.ca>

Documentation: remove those darn Windows carriage returns.

I am using linux based system for editing so I am not sure where they are coming from.

Signed-off-by: Robert Soulliere <robert.soulliere@mohawkcollege.ca>

Documentation: Migrate reports documentation from 2.1

Content reviewed by Jenny Turner. I converted to asciidoc and tested
processing. I also added index terms.

Signed-off-by: Robert Soulliere <robert.soulliere@mohawkcollege.ca>

Documentation: Format text to limit line length to 80 characters.

Signed-off-by: Robert Soulliere <robert.soulliere@mohawkcollege.ca>

Documentation: Fix various issues causing errors during processing.

Signed-off-by: Robert Soulliere <robert.soulliere@mohawkcollege.ca>

TPAC: Use % font size + bold to highlight login failure

Loved the idea to call more attention to login failures, but wanted to
try to stick to % for font sizes where possible. Testing suggested that
font-weight:bold and a little padding on top would help the display as
well.

Signed-off-by: Dan Scott <dscott@laurentian.ca>
Signed-off-by: Melissa Lefebvre <mlefebvre@biblio.org>

Tpac CSS: Login failure message

To draw attention to failed logins, the font size was increased
to 16px and the font color changed from black to red.

Signed-off-by: Melissa Lefebvre <mlefebvre@biblio.org>
Signed-off-by: Dan Scott <dscott@laurentian.ca>

TPAC: Physical description, now with spaces

Record details will regularly include physical descriptions like
"Physical Description: 1 sound disc :33 1/3 rpm, mono." where the colon
is jammed up against the content from the following subfield. Easily
fixed by using a space as the joiner for the subfields.

Signed-off-by: Dan Scott <dan@coffeecode.net>
Signed-off-by: Michael Peters <mrpeters@library.in.gov>

Move Prefix field ahead of Names in patron editor

In earlier versions of Evergreen (1.6 and earlier), the Prefix field
was used to enter title information, such as Mr., Mrs., etc. When the
field was eventually added to the newer patron registration screens
(2.0+), it was added before the patron's last name field. Ideally,
the prefix should be listed prior to the patron's first name instead.

In addition, address label issues by changing "Suffix/Title" back to
only "Suffix" and making "Prefix" into "Prefix/Title".

Note: string changes would normally not be allowed in a stable release,
however in this case the previous translation (if any) will still get
pulled from the IDL for the field in question, so English will not
suddenly start showing up in the patron registration form.

Signed-off-by: Ben Shum <bshum@biblio.org>
Signed-off-by: James Fournie <jfournie@sitka.bclibraries.ca>
Signed-off-by: Dan Scott <dscott@laurentian.ca>

Squelch uninitialized var warning from hold_copy_targeter

open-ils.storage_stderr.log was getting filled with entries like:

"Use of uninitialized value in concatenation (.) or string at
/usr/local/share/perl/5.10.1/OpenILS/Application/Storage/Publisher/action.pm"

... consuming disk space endlessly. Stop that!

(Stealing Lebbeous' footwork and providing a slightly different solution.)

Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Ben Shum <bshum@biblio.org>
Signed-off-by: Dan Scott <dscott@laurentian.ca>

Searchbar spacing: move inline CSS into a style

In keeping with our attempt to control styles via the CSS file rather
than inline overrides, define a class for the searchbar and style that.

Signed-off-by: Dan Scott <dscott@laurentian.ca>
Signed-off-by: Melissa Lefebvre <mlefebvre@biblio.org>

TPAC: Search wrapper spacing

Currently the search box options are "smooshed up" against the
links toolbar.  The following changes address this.

Padding was added to style.css search_box_wrapper in order to
bring the "Search the Catalog" down lower from the link toolbar.

Padding was also added to the searchbar.tt2 in order to bring the
search filter options down lower from the "Search the Catalog"
wording.

Signed-off-by: Melissa Lefebvre <mlefebvre@biblio.org>
Signed-off-by: Dan Scott <dscott@laurentian.ca>

increase the default width for the xul list line number column

Signed-off-by: Jason Etheridge <jason@esilibrary.com>
Signed-off-by: Jason Stephenson <jstephenson@mvlc.org>
Signed-off-by: Bill Erickson <berick@esilibrary.com>

Documentation: Add Patron Bills section to Patron Management chapter.

Converted to asciidoc by Robert Soulliere

Signed-off-by: Robert Soulliere <robert.soulliere@mohawkcollege.ca>

Documentation: Add Circulation, Patron management Chapter

Content provided by Jennifer Turner at PALS. Converted to Asciidoc by Robert Soulliere. Robert also added indexing terms. Appendices for documentation cleaned up as well.

Signed-off-by: Robert Soulliere <robert.soulliere@mohawkcollege.ca>

Set unix file format to remove trailing "\r"s in file.

Signed-off-by: Robert Soulliere <robert.soulliere@mohawkcollege.ca>

Add Group Serials Issues Chapter from Sally Fortin at Equinox Software. Source: esilibrary.com/esi/availableDocs.php.

Tested in documentation  processing.

Signed-off-by: Robert Soulliere <robert.soulliere@mohawkcollege.ca>

Update Evergreen upgrade instructions to reflect the 2.2.0 release.

Update download reference to point to 2.2.0. Also moved
database updates to the end of the process. It seemed out of order since
the database update scripts were run before the download of the 2.2 tar file.

Signed-off-by: Robert Soulliere <robert.soulliere@mohawkcollege.ca>

Remove antiquated 1.6 directory from docs folder.

This was used for early DocBook testing and is no longer needed.

Signed-off-by: Robert Soulliere <robert.soulliere@mohawkcollege.ca>
Signed-off-by: Dan Scott <dan@coffeecode.net>

Convert docs files from DOS to UNIX format.

Signed-off-by: Ben Shum <bshum@biblio.org>
Signed-off-by: Robert Soulliere <robert.soulliere@mohawkcollege.ca>

Improve Fedora prereqs and instructions

Fedora 16 and 17 need the makensis package.

As well, make the Makefile.install target "fedora" to match the OpenSRF
target, and adjust the docs accordingly. This should lead to fewer
changes in the future as Fedora 18, 19, etc roll out.

Signed-off-by: Dan Scott <dan@coffeecode.net>

DOC: fix a couple typos

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

README: Bump OpenSRF prerequisite to 2.1 for Evergreen 2.2+

Also use a title that fits into the DIG documentation more smoothly, and
be consistent about sentence-style capitalization of sub-headings.

Signed-off-by: Dan Scott <dan@coffeecode.net>

Further integration: link README to server install docs

We will now have the INSTALL file linking to the README file which in
turn leads to the docs/installation/server_installation.text file. But
it all works!

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

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

Merge the Evergreen docs into the code repository

Rather than maintaining two separate repositories, with dual maintenance
of what are essentially single-sourced files in the release notes and
the install instructions, make better use of the docs/ subfolder of the
Evergreen repository. If this crazy scheme works, DIG can commit to the
docs/ subfolder, and devs can commit there too.

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

Merge remote-tracking branch 'docbook/master' into docs_merge

Conflicts:
README

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

When a user views holds in their account, clicking "only available"
on the holds screen or "Ready for Pickup" in the navbar will add an
"available=1" parameter. However, if the user then attempts to return
to a list of all holds, the "available" parameter is not cleared, and
the user will only see holds available for pickup.

This commit will clear those parameters.

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

Cache propagated funds in fund rollover action

Repair display of propagated funds grid which was likely broken with
b9bb6d47.  Propagated funds must be collected and added to the local
cache for grid getter functions to find them.

https://bugs.launchpad.net/evergreen/+bug/1011789

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

Repair PCrudFilter localeStrings variable collisions

Rename PCrudFilter's localeStrings global variable to avoid name
collisions with UI scripts that define the same variable.

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

TPac: Disable caching for auth-required pages

To make back buttons less likely to show sensitive info.

Signed-off-by: Thomas Berezansky <tsbere@mvlc.org>
Signed-off-by: Michael Peters <mrpeters@library.in.gov>
Signed-off-by: Dan Scott <dscott@laurentian.ca>

Silence undef string concatenation warning in AutoSuggest

Addressing LP #1023651 reported by Dan Scott.

When building a cache key, use processed input values that are
guaranteed to be defined rather than raw ones.

This will mean brief cache misses for AutoSuggest once applied, but
the default life of AutoSuggest cache entries is only 5 minutes anyway.

Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>
Signed-off-by: Dan Scott <dscott@laurentian.ca>

Fix titles/tabs in Account Preferences subpages

Adjust the value of myopac_page in the Notification Preferences and
Search Preferences pages so that the page title and tab
color/hilight will be correct.

Signed-off-by: Jeff Godin <jgodin@tadl.org>
Signed-off-by: Jason Stephenson <jstephenson@mvlc.org>
Signed-off-by: Dan Scott <dan@coffeecode.net>

Repaired typo in example rsyslog config file

Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Dan Scott <dan@coffeecode.net>

TPAC: Handle multiple matches for an XPath expression

Per LP 1009980, "If a record has multiple subfield b's in the 260 field,
tpac will not display the publisher or publication date in the record
details page and will not display the publisher on the search results
page".

This is because we're calling textContent on a nodeset, rather than an
individual node, and therefore get null back rather than any content.

To avoid this, always expect a nodeset and iterate over it to populate a
list of the strings.  To maintain the same semantics of expecting a
single text string back for, say, args.pubdate, we define the list as
args.pubdates (plural name), and then grab the first item from the list
and populate that as args.pubdate (singular name).

Signed-off-by: Dan Scott <dan@coffeecode.net>
Signed-off-by: Ben Shum <bshum@biblio.org>

Avoid problem with 2.1 -> 2.2 upgrade script issuing error

The error is:
    > ERROR: cannot ALTER TABLE "thesaurus" because it has pending
    > trigger events.

Reported by Justin Hopkins, George Duimovich, and Bill Erickson.

Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>
Signed-off-by: Justin Hopkins <hopkinsju@gmail.com>

TPac: Add paging to My Lists

- Added paging to My Lists based on the code from the Checkout History tab.
- Fixed a bug where adding more than 10 items to the Temporary list failed
- After adding paging renaming a list on page 2+ would fail.  Fixed.
- Redirect after saving a list now leaves you on the same page
- Paging was bleeding between My Lists, Holds History and Circ History.
  Fixed.

List contents were also limited to the first 10.  I have upped the limit to
1000 rather than having nested paging.  This should be revisited.

CSV download of lists is still limited to the first 10.  Not sure where to
fix this.

Signed-off-by: Bob Wicksall <bwicksall@pls-net.org>
Signed-off-by: Ben Shum <bshum@biblio.org>
Signed-off-by: Jason Stephenson <jstephenson@mvlc.org>
Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>

Teach the autosuggest web service to cache suggestions where appropriate

Should spare some DB load on sites.

LFW: I'm not sure the update to the Apache config made any difference,
but it doesn't hurt.

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

tpac: repair some straggling bookbag CGI param names

We missed some going from "id" to "bbid"

Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Jason Stephenson <jstephenson@mvlc.org>

lp#1014664: add new columns to CDBI table definitions

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

Fix version numbers in 2.1 -> 2.2 upgrade script

Justin Hopkins spotted the problem, and offered a fix in LP #1015807.
I'm just doing it slightly differently here.  Thanks Justin!

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

TPAC: Add record detail navigation to page bottom

A usability rule of thumb suggests that navigation on long pages should
not require you to scroll back up to the top of the page from the
bottom; that gets old fast. We already do this on search results pages;
bring the same good practice to record details.

Signed-off-by: Dan Scott <dan@coffeecode.net>
Signed-off-by: Dan Wells <dbw2@calvin.edu>

Security fix: For auth, give same stacktrace for all cases of LOGIN_FAILED

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

Security fix: Prevent login by deleted and barred users

An existing comment in the code suggested that we thought we were already
keeping barred users out.  LP #1010671 brings up that deleted users were
not being kept out.

Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>
Signed-off-by: Mike Rylander <mrylander@gmail.com>

Security fix for Launchpad Bug 1003052.

Bug reported by James Fournie:

Revoking the UPDATE_MARC permission doesn't actually seem to prevent a
user from editing a record. Our use case is that we would like a user
to create new records but not edit existing MARC records.

Changing CREATE_MARC to UPDATE_MARC in OpenILS::Application::Cat's
biblio_record_replace_marc() method seems to fix the problem.

Signed-off-by: Jason Stephenson <jstephenson@mvlc.org>
Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>

Fix in-transit hold retarget

Tell the abort transit function to not commit our editor on us, basically.

Signed-off-by: Thomas Berezansky <tsbere@mvlc.org>
Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>

Remove unsafe thesaurus/control-set mapping

Evergreen 2.2 and beyond includes new Authority Control Set functionality
intended to allow the definition of locally defined authority control
headings.  The primary mechanism for determining which Control Set is used
by any given authority record is to look at the thesaurus fixed field
(mnemonic: Subj), which currently must be uniquely linked to a Control Set.
Failing that (in the case of no thesaurus code at all), there is a backup
mechanism: if the thesaurus value for the authority record is missing or not
used by any configured Control Set then Evergreen will look at the first Main
Entry (1xx) field in the record, and find a control set which uses this field.
This is, obviously, not perfect, but for non-overlapping authority field
schemes, it works very well.

However, the seed data for Evergreen currently includes the "no attempt to
code" value of "|" in the set of thesauri that point to the LoC Control Set.
This value ("|") is also the default value for the Subj fixed field.  This
means that the fallback mechanism is very likely never reached, and anything
short of fully specifying an appropriate thesaurus code for any non-LoC
authority record will run afoul of a similar outcome as the bib-data loss
situation described in bug #983487.

We can avoid this situation in most cases by /not/ tying the "|" thesaurus
value to the LoC control set.

This commit accomplishes this goal by allowing the Control Set for a thesaurus
to be NULL, removing the Control Set for "|" and removing the Control Set
recorded for records with "|" as the thesaurus value, thereby forcing Main
Entry inspection on any use of the authority record where Control Set is
required.

Signed-off-by: Mike Rylander <mrylander@gmail.com>
Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>

Serial Note Sort Fix

Serial notes are currently sorting backwards from what is intended
(and what is expected based on other interfaces).  This was caused
by adding a sort to the middle-layer code which resulted in an
unwanted double reverse.

[whitespace adjustment by LFW per request of DBS]

Signed-off-by: Dan Wells <dbw2@calvin.edu>
Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>

fix double-click in Holdings Maintenance

so that it won't attempt to spawn the Item Editor for non-items

Signed-off-by: Jason Etheridge <jason@esilibrary.com>
Signed-off-by: Jason Stephenson <jstephenson@mvlc.org>
Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>

Show Vandelay Buckets in Manage Record Buckets

The 'Manage Record Buckets' view is currently filtered to show
only 'staff_client' type buckets.  This expands the filter to
include 'vandelay_queue' buckets.

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

Load Vandelay queue list in Inspect Queue

When the Inspect Queue tab is opened, ensure the list of queues for the
default selected type (bib queues) is loaded.

https://bugs.launchpad.net/evergreen/+bug/1010636

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

LP#845096 - coded_value_map_id_seq doesn't have a proper last_value

Thanks to Kathy Lussier for the original report.  Kathy discovered
that users were unable to add new Coded Value Maps via the staff client.

With some log digging, I discovered the culprit was a rash of
primary key errors.

With help from Dan Scott, I was able to rectify this via a quick bit
of SQL which properly sets the last_value in the sequence.

Thanks, Kathy and Dan!

Signed-off-by: Michael Peters <mrpeters@library.in.gov>
Signed-off-by: Jason Stephenson <jstephenson@mvlc.org>
Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>

Fix broken non-RSS bookbag feeds

This commit addresses LP#1010036 by defining a stub 'description()'
method.  Lack of such a method was causing internal server errors
for other kinds of feeds.

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

LP1009752: activate purchase orders with patron requests

Simple typo: "actor_hold_request" should be "action_hold_request".

Signed-off-by: Dan Scott <dan@coffeecode.net>

Update 2.1.2 DB schema upgrade script

As part of the march to 2.1.2, and in time for 2.2.0, update the
2.1.1-2.1.2 update script.

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

Add TPAC translations

We're still working out our translation update process; new PO files can
fairly easily slip through the net. Fix that for TPAC, which is a pretty
big deal for 2.2.

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