Protect against undefined value as ARRAY reference

We should not assume that the caller is going to supply an array
(empty or not) of addresses or cards in the input values for the
open-ils.actor.patron.update method.

Signed-off-by: Dan Scott <dscott@laurentian.ca>
Signed-off-by: Galen Charlton <gmc@esilibrary.com>

fix another POD glitch

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Signed-off-by: Dan Scott <dscott@laurentian.ca>

fix up POD

- avoids some noise when building the Perl modules for
  distributions that check for correctness when converting
  POD to man pages
- add a an author test, taken straight from the Test::Pod
  documentation, for POD correctness

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Signed-off-by: Dan Scott <dscott@laurentian.ca>

Merge remote branch 'working/user/berick/marc-stream-importer-read-repair'

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

Give CentOS / RHEL a chance at installing PostgreSQL 9.0

Evergreen 2.1 requires PostgreSQL 9.0, and we accordingly updated the
package names back in the summer to point at PostgreSQL 9.0, but nobody
running CentOS / RHEL brought to our attention that the repo target
needed to be updated as well. This, although untested, at least gives
CentOS / RHEL users a fighting chance of installing Evergreen master /
2.1 on their system.

Signed-off-by: Dan Scott <dscott@laurentian.ca>
Signed-off-by: Galen Charlton <gmc@esilibrary.com>

Fix broken Net::Z3950::SimpleServer 1.14 install

Pin the install of Net::Z3950::SimpleServer to the
known-to-work-with yaz 3.0.x version, 1.12. 1.14 introduced dependencies
on yaz 4.1.x that can't be easily worked around.

Signed-off-by: Dan Scott <dscott@laurentian.ca>
Signed-off-by: Galen Charlton <gmc@esilibrary.com>

Correct varname for PostgreSQL RPMs

On Fedora, Red Hat, and CentOS, we were still trying to install
the set of PostgreSQL RPMs for PostgreSQL 8.4 using PGSQL_84_RPMS.
This had been updated to PGSQL_90_RPMS to match the updated
database version - using the right varname resolves the problem
(at least on Fedora).

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

Stamping baseline schema for previous commit

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

Fix, as provided by Doug Kyle, for LP#822918

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

Stamped upgrade script for Grace Period Closed-Date Extension features

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

Grace period auto extension and backdate awareness

Grace period auto extension:

Default is "grace periods don't auto extend".

OU setting turns on grace period auto extension. By default they only do so
when the grace period ends on a closed date, but there are two modifiers to
change that.

The first modifier causes grace periods to extend for all closed dates that
they intersect. This is "grace periods are only consumed by open days."

The second modifier causes a grace period that ends just before a closed
day, with or without extension having happened, to include the closed day
(and any following it) as well. This is mainly so that a backdate into the
closed period following the grace period will assume the "best case" of the
item having been returned after hours on the last day of the closed date.

Backdate grace period awareness:

By moving grace period calculations into a shared location backdating can
check the grace period, including extensions. If it finds that the backdate
is within the grace period (before or after extension) then instead of
voiding the fines for after the backdate it instead voids all fines for the
circulation. This emulates what would have happened (no fines) if the item
had been checked in at the time the backdate was made to take effect.

Signed-off-by: Thomas Berezansky <tsbere@mvlc.org>
Signed-off-by: Mike Rylander <mrylander@gmail.com>

trival typo fix

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

control client dump statements through pref

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

TPac: Jump-to-details on first result page only

When activated, jump-to-record-details-on-1-hit should only come into
play on the first page of results.  In other words, if page 2 of a
search has 1 item, do not jump to the record details page for that item.
Instead, show page 2 of the search results.

Signed-off-by: Bill Erickson <berick@esilibrary.com>

TPac; start new facet "search" on page 0

When selecting a facet, thereby appending it to the current search,
start the search over at the first page.  In other words, don't drop
into page X of a new search, since there may be no page X of the new
search.

Signed-off-by: Bill Erickson <berick@esilibrary.com>

fix authentication failure

Ensure that the auth_limit values are read when open-ils.auth
initializes, not during oilsAuthInit().  This solves a problem
observed on brick-based setups where authentication could
incorrectly fail because the drone that processes
open-ils.auth.authenticate.complete didn't have the the block
count value initialized yet.

Also fixed type nit when compare the fail count to the block
limit.

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

Fix "0 holds created" message on successful hold

Caused by results not always being an array.

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

TPac: My List add/remove page anchor fix

Removed /g modifier from URL anchor regex to prevent URLs with multiple
page #-anchors.  This change primarily affects IE which includes #-anchors
in the referer header.

Signed-off-by: Bill Erickson <berick@esilibrary.com>

Really add *entire* selection list to a purchase order, when desired,

and generally do what a user would expect when, in Acq, the user
executes an operation on "all items".

This addresses LaunchPad bug #810030.

The fundamental issue is that an AcqLiTable, which is used in several
different ways in different interfaces, may be representing items on a
selection list (for example), but it doesn't necessarily know all of the
items that belong to that selection list.  It only knows what it has drawn.

This addresses that deficiency by teaching the getSelected() method
some new tricks so that it can consult the server in the various ways
appropriate to getting a list of /all/ the items on a selection list, a
purchase order, a virtual PO, a set of search results, etc.

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

Fix copy-paste-o: "su - postgres" to become postgres

Thanks to sal_ in IRC who caught this nugget!

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

README: Explicitly identify accounts for commands

Add a preamble to the beginning listing all of the accounts referenced
in the instructions (Linux accounts, PostgreSQL accounts, Evergreen
administrator accounts). Also move the Developer Instructions to the
start to follow the natural order of operations.

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

Avoid collecting facet data for non-facet fields

If a config.metabib_field (Admin -> Server Admin -> MARC Search/Facet
Fields) starts out as a facet field but is later changed, data for that
facet can still bubble up since it has already been extracted.  This
patch prevents facets for config.metabib_field entries whose
facet_field=false from getting collected/cached for search results.

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

TPac: jump to details page on single hit

When enabled, any time an OPAC search yields 1 result, the user will be
redirected to the record detail page for that result.  This change adds
2 new org unit settings, one to control the behavior in the staff
client, the other to control the public catalog.

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

Fix typo in lang.dtd: "instantenous"

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

i18n: Fix es-ES illegal use of '%' in lang.dtd

The '%' character is used to include previously defined blocks in DTD
files; would be nice if translate-toolkit gave a more obvious error to
alert us to this rather than saying the en-US doesn't validate and
dying.

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

Slightly updated about.html for rel_2_1

Not sure we want to actually keep listing individuals in this file -
there's a lot more credit to spread around - but at least I can bring it
somewhat up to date with current info.

Signed-off-by: Dan Scott <dscott@laurentian.ca>
Signed-off-by: Mike Rylander <mrylander@gmail.com>

pin version for upgrade script

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

Unwrapped upgrade script for security lockdown

Signed-off-by: Thomas Berezansky <tsbere@mvlc.org>
Signed-off-by: Jason Etheridge <jason@esilibrary.com>

Require permission or pref to debug in client

Disables all debug functions I know of unless:

1 - You have new DEBUG_CLIENT permission (after login)

This should work even on operator change.
Once a given debug interface is open you can change operator back.

2 - You have a client with the debug preference enabled

This causes the client in all places I touched to think you have DEBUG_CLIENT
This also allows debugging *before* login, including in the Standalone interface.

Debug functions I know of:

Debug box on login page (already protected by pref)
Debug items in Admin menu
    Venkman
    Javascript Shells
    Inspector
    Chrome List
CTRL+SHIFT+F7 debug box in overlays

Signed-off-by: Thomas Berezansky <tsbere@mvlc.org>
Signed-off-by: Jason Etheridge <jason@esilibrary.com>

Clear password from password_prompt post-login

Otherwise it remains as the plain text value.

Signed-off-by: Thomas Berezansky <tsbere@mvlc.org>
Signed-off-by: Jason Etheridge <jason@esilibrary.com>

i18n: Fix ru-RU lang.dtd problem

No ampersands at the end of a string! Breaks moz2po horribly.

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

pin version for upgrade script

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

Switch to Modal printing

Prevents an OS-level race condition on window focus.

Signed-off-by: Thomas Berezansky <tsbere@mvlc.org>
Signed-off-by: Jason Etheridge <jason@esilibrary.com>

Unwrapped upgrade script for print.custom_js_file

Signed-off-by: Thomas Berezansky <tsbere@mvlc.org>
Signed-off-by: Jason Etheridge <jason@esilibrary.com>

Remove in-template javascript, add script files

Adds support for a print_custom.js file and an org unit specified file.

If either exists and contains a print_custom function it will be called
before printing to allow for DOM manipulation.

If neither exists the print_win.js file will call all functions it has
defined.

In addition, if any code sets do_print to false the window will close
instead of attempting to print.

The print_custom function will receive the type of template used, if any.
This only applies to "Receipt Template" based print jobs, and the type in
question is the "Type" that shows in the template editor.

Functions defined in print_win.js:

print_init
    Main function, checks for print_custom and does final printing

print_do_sums
    Does summing of values.

print_check_alt
    Does swapping out of template blocks with display:none styling.

print_check_noprint
    Disables printing under certain circumstances.

For more information about usage of the print_do_sums, print_check_alt,
and print_check_noprint functions check the comments in print_win.js.

Signed-off-by: Thomas Berezansky <tsbere@mvlc.org>
Signed-off-by: Jason Etheridge <jason@esilibrary.com>

Stamped upgrade script for username limit settings

Signed-off-by: Bill Erickson <berick@esilibrary.com>

Unwrapped, badly named upgrade script

Signed-off-by: Thomas Berezansky <tsbere@mvlc.org>
Signed-off-by: Bill Erickson <berick@esilibrary.com>

Wording change on username error

Add that they may be restricted by policy.

Signed-off-by: Thomas Berezansky <tsbere@mvlc.org>
Signed-off-by: Bill Erickson <berick@esilibrary.com>

Username Change Limits

Default is "Only allowed to change username if it looks like a barcode".

There is a "Lock Username Changes" option to disallow username changing via
the OPAC entirely, useful if syncing usernames to an outside source.

There is also a username change limit disabling option that, if enabled and
the Lock option is disabled, allows the previous behaviour of unlimited
username changes.

Signed-off-by: Thomas Berezansky <tsbere@mvlc.org>
Signed-off-by: Bill Erickson <berick@esilibrary.com>

Username Rules

In TPac, enforce the "username can't look like a barcode" rule.
Add new setting for username validation. If set, usernames must match.
Add patron registration checks for usernames:
    No spaces
    If set, can look like a barcode (initial/no username picked)
    If set, can look like a username
    If both set, must look like one of them

Signed-off-by: Thomas Berezansky <tsbere@mvlc.org>
Signed-off-by: Bill Erickson <berick@esilibrary.com>

Require password to change email/username

Alter backend to check password period, not just for password changes.
Add form elements for asking for current password to JSPac and TPac.
Add handling for said form elements where needed.
Add handling for "incorrect password" events in TPac.

Signed-off-by: Thomas Berezansky <tsbere@mvlc.org>
Signed-off-by: Bill Erickson <berick@esilibrary.com>

TPac: Default Password Strength Rule

Every other password interface seems to have it.
Implemented as a single regex in this case.

Signed-off-by: Thomas Berezansky <tsbere@mvlc.org>
Signed-off-by: Bill Erickson <berick@esilibrary.com>

When workstation is invalid request a new seed

The original one may no longer be valid

Signed-off-by: Thomas Berezansky <tsbere@mvlc.org>
Signed-off-by: Bill Erickson <berick@esilibrary.com>

Make more auth values configurable

Amount of time seed is valid
Amount of time to keep failure count in memcache since last auth event
Number of failures before locking out auth attempts

Also, remove seed from memcache once it has been used once.

Signed-off-by: Thomas Berezansky <tsbere@mvlc.org>
Signed-off-by: Bill Erickson <berick@esilibrary.com>

Brute Force protection for authentication

Count auth failures in memcache.
If 10+ have occurred cause failure.

After 90 seconds of no activity counter resets.

Signed-off-by: Thomas Berezansky <tsbere@mvlc.org>
Signed-off-by: Bill Erickson <berick@esilibrary.com>

link asset.copy_template.status to config.copy_status in the IDL

If nothing else, this provides a dropdown for status in the copy
template editor (the one for Serials, under the admin menus, not the
one on the copy editing interface).

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

Make sure baseline schema is up to date

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

Fix update_db.sh to set eg_version correctly

psql wants --set=VARNAME=VALUE, we had -set VARNAME=VALUE; tested this
fix and it works with PostgreSQL 9.0.

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

TPAC: Fix searchbar typo by closing div

Nesting of divs was thrown off, resulting in the footer appearing in the
wrong place in search results / record display / etc, with the result
that the footer inherited the 974px width instead of full 100% width.

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

LP865817: Fix Located URI visibility test

Wrapped upgrade script for LP865817.

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

Correct Located URI visiblity

Located URIs should be visible when they exist at the search context org unit
or an ancestor thereof.  However, before the change embodied in this commit,
they effectively acted exactly like copies, making records visible anywhere
within the context+depth range.

An example, using the stock example OU hierarchy:

 * Before this change, a Located URI with an owner of BR1 would cause its
record to show up in a search at (or scoped to via a depth parameter) BR1,
SYS1 or CONS.

 * Before this change, a Located URI with an owner of SYS1 would NOT cause its
record to show up in a search at (or scoped to via a depth parameter) BR1!

 * After this change, a Located URI with an owner of BR1 will cause its record
to show up only in a search with a context OU of BR1, irrespective of depth
scoping.

 * After this change, a Located URI with an owner of SYS1 WILL cause its
record to show up in a search with a context OU of BR1 or SYS1, regardless of
depth scoping.

Signed-off-by: Mike Rylander <mrylander@gmail.com>
Signed-off-by: Dan Scott <dscott@laurentian.ca>

TPAC: No need for a 26px padding on the header bottom

26px on the bottom of the header was giving a bit too much space to the
header.

Signed-off-by: Dan Scott <dscott@laurentian.ca>
Signed-off-by: Bill Erickson <berick@esilibrary.com>

TPAC: Replace dashboard images with CSS

Also gets rid of a number of tables and a bunch of CSS definitions with
hard-coded widths that will cause trouble when longer translated terms
get dropped into place.

Signed-off-by: Dan Scott <dscott@laurentian.ca>
Signed-off-by: Bill Erickson <berick@esilibrary.com>

Tpac: expand facet display fix

When only one facet is collapsed, make sure we can expand it.

Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Dan Scott <dscott@laurentian.ca>

TPac: Facets on results page

Show search facets along the left side of the results page.  Clicking on
a facet re-runs the current search with the  new facet appeneded.
Selected facets are styled to indicate they are selected.  Clicking on a
selected facet removes that facet from the set of selected facets (i.e.
it backs out the facet).

TODO: update CSS to match default Evergreen skin

Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Dan Scott <dscott@laurentian.ca>

JSPAC: Fix a bug (I think) in orgIsMine(), and show more MFHD holdings ...

... when search scope would have it so.

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

Tweak the tab shape in TT OPAC

A slight modification to the "tab" elements throughout the OPAC to make them look more like tabs on a file folder.  I
found the rounded off bottoms less attractive, so I flattened them out to flow more evenly with the rest of the page.

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

Have TTPac utilize gradient friendly, transparent PNG files for the Evergreen logo

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

TPAC: Simple CSS gradients for the header/footer

Firefox, Safari, Chrome, Opera, and the Android browser support
gradients. Use them. iOS 5 will support gradients. IE does not,
naturally, but support is promised for IE 10.

Including better IE support.

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

TPAC: Remove the printnav bar from all screens

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

Revamp the search bar to a narrative, single-line style

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

Add the remaining extras back to the record details

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

TPAC: and then there were series

Also, tighten up spacing considerably.

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

Move the copies up higher

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

Remove "Expand all extras" as there are fewer extras to expand

Cleaner.

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

Going for the Jungle skin feel (like Amazon)

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

TPac: propagate searches to advanced search page

The link that takes users to the advanced searc page now propagates
any existing (well, all) search URL params.  The change required some
minor augmentation to the code that builds the advanced search form.

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

LP740320: Make <enter> submit the new bookbag dialog

For the keyboard jockeys out there - if the enter key doesn't submit,
the UI reviewer will not commit.

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

Initial dialog isn't quite wide enough, here's a hackish way to fix it

Signed-off-by: James Fournie <jfournie@sitka.bclibraries.ca>
Signed-off-by: Dan Scott <dscott@laurentian.ca>

Should probably use a more common button rather than the one for the advanced search

Signed-off-by: James Fournie <jfournie@sitka.bclibraries.ca>
Signed-off-by: Dan Scott <dscott@laurentian.ca>

This makes more sense as an alert; the Dojo dialog has no button which is not user-friendly.

Signed-off-by: James Fournie <jfournie@sitka.bclibraries.ca>
Signed-off-by: Dan Scott <dscott@laurentian.ca>

Markup-based dijit.Dialog implementation, inspired by and based on a patch from Joseph Lewis in https://bugs.launchpad.net/evergreen/+bug/787162

Signed-off-by: Mike Rylander <mrylander@gmail.com>
Signed-off-by: Dan Scott <dscott@laurentian.ca>

i18n: Merge en-GB translation from Launchpad

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

i18n: Merge en-CA translation from Launchpad

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

i18n: Merge pt-BR translation from Launchpad

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

i18n: Merge ru-RU translation from Launchpad

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

 i18n: Merge Armenian (hy-AM) translation from Launchpad

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

i18n: Merge cs-CZ translation from Launchpad

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

i18n: Merge es-ES translation from Launchpad

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

i18n: Merge fr-CA translation from Launchpad

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

Modified spine_labels.js to retain UTF-8 encoding

Dan Wells posted a fix to the DEV list for receipt printing that allowed the
receipt printer to print UTF-8 characters.  I used the ideas in that fix to
modify spine_labels.js.  In spine_labels.js, the url passed to the preview in
preview_xul_192 was encoding the url with the javascript encode() function.
This function does not preserve UTF-8 encodings.  I changed the function to
encodeURIComponent, and UTF-8 characters are now displayed properly by the
spine label printer code.

Signed-off-by: Liam Whalen <lwhalen@evergreen-dev.catalogue.nrcan.gc.ca>
Signed-off-by: Dan Scott <dscott@laurentian.ca>

Added missing Penalty module import for AssetCommon

The lack of import primarily affected the Action/Trigger MarkItemLost
reactor, which does not already import OpenILS::Utils::Penalty directly
or indirectly.

Signed-off-by: Bill Erickson <berick@esilibrary.com>

Remove an installation step: Listen 443 uncommenting

Every distribution known to man ships Apache listening on port 443, so
installers always have to comment out the "Listen 443" directive - so
why not just comment it out ourselves and save a step in the
installation instructions?

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

TPac: move google analytics import to head

Google Analytics documention recommends loading the JS in the page
<head>.

http://www.google.com/support/googleanalytics/bin/answer.py?answer=174090

Use as directed by default.

Also, moved the JS loading chunk to its own template to ease override by
those who do not wish to use as directed.

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

A slight modification to prevent clipping in the EDI Accounts interface

Just a few tweaks via CSS to make sure the footer notes don't clip off part of the main grid.

Signed-off-by: Michael Peters <mrpeters@library.in.gov>
Signed-off-by: Mike Rylander <mrylander@gmail.com>

Provide proper hrefs for links back from report debugging pages

Reimplementation of a fix provided by Michael Peters.  The originally submitted
patch included several fixes provided by an unrelated patch, and a regression
in build-time embedded tokens.

Original branch at: http://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/mrpeters-isl/clark_kent_back_links

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

LP856567: Mark HTML report output as Unicode-encoded

Non-ASCII characters in report ouput that has been generated in HTML
format appears to be corrupted when viewed in a browser. This can be
manually corrected by switching the "Encoding" browser option, but it
would be better to include <meta charset='utf-8'> in the output HTML to
tell the browser directly.

Signed-off-by: Dan Scott <dscott@laurentian.ca>
Signed-off-by: Mike Rylander <mrylander@gmail.com>

MFHD compression fails with pattern-less captions

The current code assumes that if you try to compress a range of
MFHD holdings, you actually have the necessary pattern information.
This commit introduces a very basic check to avoid the most common
failure case.

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

Teach update_db.sh to find the currently installed version number via eg_config and supply that when applying updates

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

Read STDIN up to record separator then stop

The initial (and only known) target for the importer is OCLC Connexion
exports, which export 1 record at a time, with each record ending with
the standard record separator.  There is no end-of-file /
end-of-transmission indicator, so the while() loop always blocks and
fails on the timeout alarm after the first record is read.

Until a more flexible solution is in place, roll back support for
reading multiple records via STDIN.  The original commit was

679506975f0efb6e2f6f17b0c19e3c53c5bfda44

Note, the feature being rolled back was not actually necessary to
implement the feature in the commit (file-based imports).  It was just
an added bonus.

Signed-off-by: Bill Erickson <berick@esilibrary.com>

LP857460: Get the generic class scheme when adding vols

In a default Evergreen instance, in the "Add Volumes" interface for a
bib record with both LC and Dewey call numbers, the list of call numbers
from which you can select appears to be limited to Dewey, even though
the "Bib Call #" field is populated with an LC call number.

The problem is that here we're grabbing the first entry from the
asset.call_number_classification table, and you might expect that to be
the "Generic" entry - however, it turns out to be the "Dewey" entry
because the entries are ordered alphabetically by .name(), not by
database ID. (You can test this be changing the name of "Dewey" to
"Vicious" and see that the "Generic" entry is chosen instead).

This commit changes the process to first search the acnc array for a
name() = "Generic"; and then if we don't find that, fall back to the
first entry in the classification scheme.

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

Removed old vandelay config from sample apache config

Bye, bye xmlent

Signed-off-by: Bill Erickson <berick@esilibrary.com>

TPac: Respect OU opac_visible flags

Unless we appear to be in staff mode. Then continue to ignore them.

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

TPac: Fix bolding of bad email/username

HTML filter the variable, not the entire language string.

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

TPac: Hide CC Payment info when not enabled

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

TPac Fixes - Missing Quotes

Start off real easy, add some missing quotes.

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

Merge branch 'master' of git.evergreen-ils.org:Evergreen

Stamping Upgrade Script for Suppress Transits

In addition to stamping, this commit also makes a few trivial
wording changes.

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

Create git .mailmap file for Evergreen

This is an initial .mailmap file for Evergreen.

The .mailmap file is used by git-shortlog to "coalesce together
commits by the same person".

In the output of git-shortlog, this eliminates duplicate entries
such as the username@UUID entries from the svn to git migration.

Compare "git shortlog -nes" with and without this .mailmap

Signed-off-by: Jeff Godin <jgodin@tadl.org>
Signed-off-by: Dan Scott <dscott@laurentian.ca>

Unwrapped Suppress Transits upgrade script

Signed-off-by: Thomas Berezansky <tsbere@mvlc.org>
Signed-off-by: Dan Wells <dbw2@calvin.edu>