working/Evergreen.git
6 years agoStamping best-hold-order update script
Mike Rylander [Fri, 19 Apr 2013 20:19:11 +0000 (16:19 -0400)]
Stamping best-hold-order update script

Signed-off-by: Mike Rylander <mrylander@gmail.com>
6 years agoFix various Traditional and holds-go-home best-hold sort orders
Lebbeous Fogle-Weekley [Wed, 3 Apr 2013 20:35:51 +0000 (16:35 -0400)]
Fix various Traditional and holds-go-home best-hold sort orders

Use copy's call number's owning_lib instead of copy's circ_lib

    Should compare checkin lib to copy's (call number's) owning_lib, not
    hold request lib.

    You might think the comparison should be to acp.circ_lib, but that
    doesn't work with floating copies (for non-floaters, acp.circ_lib
    should be equal to acp.call_number.owning_lib).

approx is a more correct first determinant to give the behavior sites
are used to.

    hprox can cause copies to be too eager to go home when
    there are holds with that copy's circ lib as its request lib (if that's
    what you want, then you do pick or create a sort-order with hprox near
    the top).

Address a problem in the copy_has_not_been_home CTE.
    This expression was always meant to provide a TRUE or FALSE value as its
    lone result, but would return NULL in cases where copies had no transit
    history.

Use pickup_lib, not request_lib, as the determinant of
    nearness-to-home.  request_lib was used with the thinking that an item's
    "owning" patrons should have their wishes favored at holds-go-home time,
    even if where they wanted to send the copy was not actually home, but
    that's neither necessarily desired nor very intuitive.

Clear up holds-go-home logic with better code AND add TechRef
documentation with diagram in attempt to be as clear as possible.

Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
6 years agoConfirm URL_VERIFY_UPDATE_SETTINGS permission code is fixed
Mike Rylander [Fri, 19 Apr 2013 15:45:32 +0000 (11:45 -0400)]
Confirm URL_VERIFY_UPDATE_SETTINGS permission code is fixed

Jeff Davis spotted, and fixed, a problem with upgrade script 0754. If
someone has already applied that, this will address.

Signed-off-by: Mike Rylander <mrylander@gmail.com>
6 years agoUse correct perm code in upgrade scripts for SQL update 0754
Jeff Davis [Thu, 18 Apr 2013 23:09:40 +0000 (16:09 -0700)]
Use correct perm code in upgrade scripts for SQL update 0754

Signed-off-by: Jeff Davis <jdavis@sitka.bclibraries.ca>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
6 years agoMerge changes to Apache 2.4 example config files
Dan Scott [Fri, 19 Apr 2013 15:06:27 +0000 (11:06 -0400)]
Merge changes to Apache 2.4 example config files

Not only do we need the JSPAC->TPAC changes that went into the base
Apache example configs, we should also move to generating these files
instead of having hardcoded values. Longer term, we should probably just
generate the right example configs from one common base until Apache 2.2
is dead and gone, but this improves the Apache 2.4 experience greatly.

Signed-off-by: Dan Scott <dscott@laurentian.ca>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
6 years agoStamping upgrade for #deleted search modifier (and friends) fix
Mike Rylander [Thu, 18 Apr 2013 19:45:15 +0000 (15:45 -0400)]
Stamping upgrade for #deleted search modifier (and friends) fix

Signed-off-by: Mike Rylander <mrylander@gmail.com>
6 years agoQP: Repair edit_date, create_date, and bib_source filters
Lebbeous Fogle-Weekley [Wed, 17 Apr 2013 22:16:09 +0000 (18:16 -0400)]
QP: Repair edit_date, create_date, and bib_source filters

And also make records undeletable, where reingestion at undelete time
formerly balked in this manner:

> ERROR: duplicate key value violates unique constraint
> "record_attr_pkey" DETAIL: Key (id)=(1) already exists. CONTEXT: SQL
> statement "INSERT INTO metabib.record_attr (id, attrs) VALUES (NEW.id,
> new_attrs)" PL/pgSQL function "indexing_ingest_or_delete" line 129 at
> SQL statement

Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
6 years agoQP: Limit to deleted records for #deleted modifier at this level
Lebbeous Fogle-Weekley [Wed, 17 Apr 2013 21:24:59 +0000 (17:24 -0400)]
QP: Limit to deleted records for #deleted modifier at this level

Down in the DB's query_parser_fts procedure, we also know whether we're
in a deleted_search, and in those cases we skip all visibility and
similar testing accordingly.

Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
6 years agoBring back #deleted modifier post-revive-qp-fts
Mike Rylander [Fri, 5 Apr 2013 20:46:21 +0000 (16:46 -0400)]
Bring back #deleted modifier post-revive-qp-fts

Signed-off-by: Mike Rylander <mrylander@gmail.com>
Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
6 years agoTPAC schema.org: Add Organization types for contributors
Dan Scott [Sun, 7 Apr 2013 22:14:58 +0000 (18:14 -0400)]
TPAC schema.org: Add Organization types for contributors

Given a 110 / 710 field, when generating the record details for authors
and contributors we can declare an http://schema.org/Organization
itemtype rather than just jamming the name under the "contributor"
property. This is more in accordance with schema.org directions.

Signed-off-by: Dan Scott <dscott@laurentian.ca>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
6 years agoTPAC: Fix schema.org name / dates for authors
Dan Scott [Fri, 5 Apr 2013 20:10:01 +0000 (16:10 -0400)]
TPAC: Fix schema.org name / dates for authors

Per http://schema.org/Person, the birth date and death date are not
supposed to be part of the name of a Person. We can separate these out
correctly based on subfield d for 100 fields.

Signed-off-by: Dan Scott <dscott@laurentian.ca>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
6 years agoFix schema.org mapping for MusicAlbum, add Map
Dan Scott [Fri, 5 Apr 2013 17:21:47 +0000 (13:21 -0400)]
Fix schema.org mapping for MusicAlbum, add Map

We had set LDR[06] = j to MusicRecording, but that is really meant for
individual songs. Use MusicAlbum instead, and per
http://schema.org/MusicAlbum, use a new MusicGroup itemtype with a
'byArtist' property for the primary artist rather than the generic
'accountablePerson'.

Also map LDR[06] = e to Map, because that seems like a safe bet.

Signed-off-by: Dan Scott <dscott@laurentian.ca>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
6 years agoStamping upgrade script for optional combined search by class
Ben Shum [Thu, 18 Apr 2013 17:45:54 +0000 (13:45 -0400)]
Stamping upgrade script for optional combined search by class

Signed-off-by: Ben Shum <bshum@biblio.org>
6 years agoAllow combined search to be optional per class
Mike Rylander [Tue, 16 Apr 2013 19:08:10 +0000 (15:08 -0400)]
Allow combined search to be optional per class

Relevance is thrown off for, in paticular, the keyword class when combined
search is used.  This is because the effect of an opaque blob of data, such
as the keyword|keyword index definition, is to applify the inclusion of spurious
(to the user) data in matching attempts.

This commit adds the ability to specify, per class, whether combined FTS should
be used, and turns this on for only the subject class by default.

Signed-off-by: Mike Rylander <mrylander@gmail.com>
Signed-off-by: Ben Shum <bshum@biblio.org>
6 years agoAcq: re-use more code for two ways of creating invoices (EDI and manual)
Lebbeous Fogle-Weekley [Wed, 20 Feb 2013 14:41:03 +0000 (09:41 -0500)]
Acq: re-use more code for two ways of creating invoices (EDI and manual)

This solves two problems.

 1) With EDI invoices, we had been failing to disencumber fund debits
    related to the invoiced lineitems, although that worked for manual
    invoices.
 2) With manual invoices, we would not automatically uncancel copies
    when the user decided to invoice them despite their canceled status.
    This was already working in EDI invoices though.  This is especially
    important since our schema lumps "backordered" in with "canceled,"
    and in theory backordered things do show up eventually.

There were earlier version of this commit out there with bugs that
prevented the EDI workflow from working correctly (the manual invoice
flow worked and still should).

Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
6 years agoApproximate number of search results
Kathy Lussier [Sun, 14 Apr 2013 04:38:53 +0000 (00:38 -0400)]
Approximate number of search results

The number of results listed on the search results page is often an
estimate when working with a large set. Let's add the word "about" to
reflect the fact that it is indeed an estimate.

At the same time, I moved the <strong> tags out of the code, adding a CSS
class in its place, moved the HTML tags out of the translatable
string (as recommended by Pasi Kallinen), and fixed a typo in the
stylesheet.

Signed-off-by: Kathy Lussier <klussier@masslnc.org>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
6 years agoMinor change to chrome/content/util/list.js.
Jason Stephenson [Thu, 18 Apr 2013 14:49:06 +0000 (10:49 -0400)]
Minor change to chrome/content/util/list.js.

Change the initializer of this.count_for_display from 0 to 1 in
order to match the counting of other lists in the client and to
provide something that staff are likely to prefer.

Signed-off-by: Jason Stephenson <jstephenson@mvlc.org>
6 years agoFixed bug in staff client offline mode.
Dmitry Nechai [Tue, 16 Apr 2013 21:33:57 +0000 (00:33 +0300)]
Fixed bug in staff client offline mode.

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

In list.js at start you create obj.columns for lineno, but later
you don`t process this value so it`s return default render
function.  This code repairs this.

Signed-off-by: Dmitry Nechai <nechai.dmitry@gmail.com>
Signed-off-by: Jason Stephenson <jstephenson@mvlc.org>
6 years agoPrevent compiler warning about unused numtype var
Dan Scott [Fri, 5 Apr 2013 05:53:55 +0000 (01:53 -0400)]
Prevent compiler warning about unused numtype var

There was a dangling variable left around that was making noise in the
compiler. Credit to Jeff Godin for the heads-up.

Signed-off-by: Dan Scott <dscott@laurentian.ca>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
6 years agoAddress SQL injection vulnerability in SQL ORM layer
Mike Rylander [Fri, 5 Apr 2013 05:52:16 +0000 (01:52 -0400)]
Address SQL injection vulnerability in SQL ORM layer

If the user-supplied value and the db column are both numbers
(jsonObject->type == JSON_NUMBER, get_primitive(field) == "number") then
don't quote. Otherwise, quote.

Signed-off-by: Mike Rylander <mrylander@gmail.com>
Signed-off-by: Dan Scott <dscott@laurentian.ca>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
6 years ago2.2.7-2.2.8 database upgrade script
Lebbeous Fogle-Weekley [Wed, 17 Apr 2013 02:24:07 +0000 (22:24 -0400)]
2.2.7-2.2.8 database upgrade script

Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>
Signed-off-by: Bill Erickson <berick@esilibrary.com>
6 years ago2.3.5 -> 2.3.6 SQL upgrade
Bill Erickson [Wed, 17 Apr 2013 19:47:59 +0000 (15:47 -0400)]
2.3.5 -> 2.3.6 SQL upgrade

Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>
6 years agoStamping upgrade script for reinstate tsconfigs.
Jason Stephenson [Wed, 17 Apr 2013 18:33:48 +0000 (14:33 -0400)]
Stamping upgrade script for reinstate tsconfigs.

Signed-off-by: Jason Stephenson <jstephenson@mvlc.org>
6 years agoReinstate required TEXT SEARCH CONFIGURATION data
Mike Rylander [Wed, 17 Apr 2013 18:04:11 +0000 (14:04 -0400)]
Reinstate required TEXT SEARCH CONFIGURATION data

Signed-off-by: Mike Rylander <mrylander@gmail.com>
Signed-off-by: Jason Stephenson <jstephenson@mvlc.org>
6 years agoAddress hash/list thinko; Ensure initialization
Mike Rylander [Wed, 17 Apr 2013 16:08:41 +0000 (12:08 -0400)]
Address hash/list thinko; Ensure initialization

Two cascading defects were identified after being reported by users. The
first, which was only a potential issue, was an apparent lack of mod_perl
initialization for the item-age record feed.  While not the direct cause
of the error message, a lack of initialization here would manifest in the
same way.  So, we check that initialization.

Feed-generating callbacks were not being constructed properly due to a
simple thinko in the construction of the default record format list provided
by the SuperCat backend.  We need a list of two hashes, not a hash of two
keys.

Signed-off-by: Mike Rylander <mrylander@gmail.com>
Signed-off-by: Jason Stephenson <jstephenson@mvlc.org>
6 years agoTPAC bucket item retrieval operates in streaming mode
Bill Erickson [Tue, 9 Apr 2013 17:45:24 +0000 (13:45 -0400)]
TPAC bucket item retrieval operates in streaming mode

TPAC bucket item retreive fleshes bib records with large blobs of MARC
data.  When a bucket contains a few thousand items, the size of the
data passed around in atomic retreival mode will exceed the typical jabber
max stanza size and result in a failure.  Retrieve the records in
streaming mode instead.

Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Ben Shum <bshum@biblio.org>
6 years agoTPAC: Render record titles and authors consistently
Dan Scott [Thu, 11 Apr 2013 12:56:50 +0000 (05:56 -0700)]
TPAC: Render record titles and authors consistently

Yamil Suarez noticed that the titles and authors in the call number
browse results page were presented differently than the regular search
results. This change removes some of the hard-coded <em> tags and "bold"
classes, in favour of new "record_author" and "record_title" CSS
classes. Now sites can simply change the CSS in one place to keep things
consistent between the two search results pages.

Signed-off-by: Dan Scott <dscott@laurentian.ca>
Signed-off-by: Yamil Suarez <yamil@yamil.com>
Signed-off-by: Ben Shum <bshum@biblio.org>
6 years agoUse inline-block for opac-multiline-button style
Ben Shum [Wed, 27 Mar 2013 15:48:58 +0000 (11:48 -0400)]
Use inline-block for opac-multiline-button style

Follow up for LP1102934 which introduced CSS for opac-multiline-button.

The button appeared noticeably larger if using the english text for
"FAQs" on the login help button. Using inline-block for the style shrinks
it to fit more appropriately based on actual amount of text instead of
leaving it an unusually large looking button.

Signed-off-by: Ben Shum <bshum@biblio.org>
Signed-off-by: Kyle Tomita <ktomita@catalystitservices.com>
6 years agoTPAC Turn dashes into space in subject search links for better matching
Lebbeous Fogle-Weekley [Wed, 10 Apr 2013 17:23:28 +0000 (13:23 -0400)]
TPAC Turn dashes into space in subject search links for better matching

Thanks to Dan Wells for finding a bug and fixing it for demonstration
purposes.

Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>
Signed-off-by: Dan Wells <dbw2@calvin.edu>
6 years agoAdd summary creation to embedded alt dist editor
Dan Wells [Thu, 4 Apr 2013 20:43:38 +0000 (16:43 -0400)]
Add summary creation to embedded alt dist editor

In the Serial Control View, summaries are created automatically with
a new distribution. To avoid unhappy surprises, the embedded version
of the alt distribution editor should do the same.

Signed-off-by: Dan Wells <dbw2@calvin.edu>
Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>
6 years agoIgnore canceled holds during alt-queue generation
Bill Erickson [Wed, 3 Apr 2013 20:21:25 +0000 (16:21 -0400)]
Ignore canceled holds during alt-queue generation

When zero potential copies exist for a hold, the hold queue stats code
determines the size and shape of the queue by finding and sorting holds
that have the same type and target.  This patch forces the code to
ignore canceled holds when looking for similarly-shaped holds.

Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Jason Etheridge <jason@esilibrary.com>
6 years agoRemove a search change and add an invoicing note
Kathy Lussier [Mon, 1 Apr 2013 20:15:10 +0000 (16:15 -0400)]
Remove a search change and add an invoicing note

With lp1161601, we lost one of the new features from the qp work. Also, we
missed one new feature with invoices.

Signed-off-by: Kathy Lussier <klussier@masslnc.org>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
6 years agoRelease Note Additions
Kathy Lussier [Sat, 30 Mar 2013 02:09:48 +0000 (22:09 -0400)]
Release Note Additions

We should probably add the removal of JSPAC to the Release Notes. Also
added the following new features that were missing from the original
notes.

- Options to hide some user preferences from TPAC
- New fields for acq distribution formulas
- Public copy notes
- Direct access to item attribute editor
- Setting staff client search preferences

Signed-off-by: Kathy Lussier <klussier@masslnc.org>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
6 years agoMore typo fixing
Ben Shum [Wed, 20 Mar 2013 13:51:22 +0000 (09:51 -0400)]
More typo fixing

Change CVS to CSV as noticed by Remington Steed.

Signed-off-by: Ben Shum <bshum@biblio.org>
Signed-off-by: Kathy Lussier <klussier@masslnc.org>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
6 years agoMinor edit
Kathy Lussier [Wed, 20 Mar 2013 04:05:27 +0000 (00:05 -0400)]
Minor edit

Signed-off-by: Kathy Lussier <klussier@masslnc.org>
Signed-off-by: Ben Shum <bshum@biblio.org>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
6 years agoMinor edits to release notes and addition of new permissions
Kathy Lussier [Wed, 20 Mar 2013 03:50:48 +0000 (23:50 -0400)]
Minor edits to release notes and addition of new permissions

There are some new permissions that didn't get added to the original
release notes. Also, some minor edits.

Signed-off-by: Kathy Lussier <klussier@masslnc.org>
Signed-off-by: Ben Shum <bshum@biblio.org>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
6 years agoFix some typos in the 2.4 release notes
Ben Shum [Tue, 19 Mar 2013 20:59:41 +0000 (16:59 -0400)]
Fix some typos in the 2.4 release notes

Signed-off-by: Ben Shum <bshum@biblio.org>
Signed-off-by: Kathy Lussier <klussier@masslnc.org>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
6 years agoUpdate for 2.4 beta1 release notes
Ben Shum [Tue, 19 Mar 2013 16:32:13 +0000 (12:32 -0400)]
Update for 2.4 beta1 release notes

- Move more release notes into 2.4 notes.
- Move notify_csv.txt into TechRef directory.
- Include entry for Linkchecker in 2.4 notes.

Signed-off-by: Ben Shum <bshum@biblio.org>
Signed-off-by: Kathy Lussier <klussier@masslnc.org>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
6 years agoInitial cut of 2.4 release notes.
Ben Shum [Wed, 20 Mar 2013 02:49:49 +0000 (22:49 -0400)]
Initial cut of 2.4 release notes.

Basically, the contents of RELEASE_NOTES_NEXT organized into
RELEASE_NOTES_2_4.txt and removed from RELEASE_NOTE_NEXT.

Move copy of alternate_graphic_fields to TechRef.

Signed-off-by: Ben Shum <bshum@biblio.org>
Signed-off-by: Kathy Lussier <klussier@masslnc.org>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
6 years agoRelease note template
Mike Rylander [Mon, 1 Apr 2013 17:28:48 +0000 (13:28 -0400)]
Release note template

Signed-off-by: Mike Rylander <mrylander@gmail.com>
6 years agoUpdated README to reflect the future
Mike Rylander [Mon, 1 Apr 2013 17:27:28 +0000 (13:27 -0400)]
Updated README to reflect the future

Signed-off-by: Mike Rylander <mrylander@gmail.com>
6 years agoproto-script for release note generation
Mike Rylander [Mon, 1 Apr 2013 17:20:26 +0000 (13:20 -0400)]
proto-script for release note generation

Signed-off-by: Mike Rylander <mrylander@gmail.com>
6 years agorelease notes FS structure
Mike Rylander [Mon, 1 Apr 2013 17:20:00 +0000 (13:20 -0400)]
release notes FS structure

Signed-off-by: Mike Rylander <mrylander@gmail.com>
6 years agostamping upgrade for field 264 reporter view
Dan Wells [Wed, 3 Apr 2013 19:30:40 +0000 (15:30 -0400)]
stamping upgrade for field 264 reporter view

Signed-off-by: Dan Wells <dbw2@calvin.edu>
6 years agoLP1071505 - Add 264 RDA tag to reporter views
Ben Shum [Thu, 31 Jan 2013 17:20:20 +0000 (12:20 -0500)]
LP1071505 - Add 264 RDA tag to reporter views

Some reports draw from reporter.materialized_simple_record table and these
were not made aware of the new 264 tag which contains publisher related
information.

Change the view for reporter.old_super_simple_record, which feeds into the
other reporter table.

The included upgrade SQL will update this view and then update the table using
the new view.

Signed-off-by: Ben Shum <bshum@biblio.org>
Signed-off-by: Dan Wells <dbw2@calvin.edu>
6 years agoPopulate variable needed for batch item receive
Dan Wells [Wed, 3 Apr 2013 16:44:23 +0000 (12:44 -0400)]
Populate variable needed for batch item receive

In the integrated alt editor, we load the sub. grid differently, so
'sub' was not being populated.  Since we should only ever have one
subscription showing at a time (in this view), we can simply assign
the item to 'sub' as it is received.

Signed-off-by: Dan Wells <dbw2@calvin.edu>
Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>
6 years agoSmall fixes for alt sub editor integration
Dan Wells [Wed, 3 Apr 2013 16:03:25 +0000 (12:03 -0400)]
Small fixes for alt sub editor integration

This commit applies some polish to the alt sub editor integration:

 - Prevent spinner from displaying continuously when adding a
   subscription
 - Hide the editor when changing subscriptions to limit confusion
   about what is loaded
 - Dynamically add a node to the tree view when adding a subscription
 - Make sure the 'View/Edit' tab gets populated when backing out of
   the stream or item view
 - Apply some default link styles

Signed-off-by: Dan Wells <dbw2@calvin.edu>
Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>
6 years agoBash script to be used after the main 2.3-2.4 db upgrade
Mike Rylander [Wed, 3 Apr 2013 17:19:18 +0000 (13:19 -0400)]
Bash script to be used after the main 2.3-2.4 db upgrade

This script, originally offered by Thomas Berezansky and tested by others,
and augmented by me to include more parallizable SQL, will reduce the
time spent upgrading a large Evergreen 2.3 database to 2.4 by several hours.

Signed-off-by: Mike Rylander <mrylander@gmail.com>
6 years agoRepair an issue in search filter group UI about filtering on owner OU
Lebbeous Fogle-Weekley [Wed, 3 Apr 2013 14:50:22 +0000 (10:50 -0400)]
Repair an issue in search filter group UI about filtering on owner OU

What was intended to be a filter against filter groups' owner field was
instead turned into a filter on filter group ID.

Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
6 years agoQP copy location group filter repair
Bill Erickson [Tue, 2 Apr 2013 20:02:26 +0000 (16:02 -0400)]
QP copy location group filter repair

Remove reference to nonexistent location group filter callback function.
Copy locations for filters are loaded inline.  No callback needed.

Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
6 years agoDocumentation: Add link checker documentation.
ESI [Tue, 2 Apr 2013 19:24:57 +0000 (15:24 -0400)]
Documentation: Add link checker documentation.

Signed-off-by: Robert Soulliere <robert.soulliere@mohawkcollege.ca>
6 years agoUpdate LP1159768 - Client: Holds Client Controls Confusing
Kyle Tomita [Wed, 27 Mar 2013 22:31:17 +0000 (15:31 -0700)]
Update LP1159768 - Client: Holds Client Controls Confusing

The new string "Are you sure you want to cancel the hold(s)?"
has been added as the following variable,
staff.hold_list.cancel_hold_dialog.confirmation.

The string also needs to be translated to all supported languages.

Signed-off-by: Kyle Tomita <ktomita@catalystitservices.com>
Signed-off-by: Jason Stephenson <jstephenson@mvlc.org>
6 years agoLP1159768 - Client: Holds Client Controls Confusing
Kyle Tomita [Wed, 27 Mar 2013 20:31:03 +0000 (13:31 -0700)]
LP1159768 - Client: Holds Client Controls Confusing

The buttons were updated per Jason's suggestions to "No" and "Yes" with
a question added, "Are you sure you want to cancel the hold(s)?"

Signed-off-by: Kyle Tomita <ktomita@catalystitservices.com>
Signed-off-by: Jason Stephenson <jstephenson@mvlc.org>
6 years agoLP1159768 - Client: Holds Client Controls Confusing
Kyle Tomita [Wed, 27 Mar 2013 19:50:59 +0000 (12:50 -0700)]
LP1159768 - Client: Holds Client Controls Confusing

Changed the language from "Cancel" to "Cancel Action" and
"Apply" to "Apply Cancellation".  These need to be translated
to other supported languages.

Signed-off-by: Kyle Tomita <ktomita@catalystitservices.com>
Signed-off-by: Jason Stephenson <jstephenson@mvlc.org>
6 years agostamping upgrade script for authority validation fix
Mike Rylander [Tue, 2 Apr 2013 13:40:07 +0000 (09:40 -0400)]
stamping upgrade script for authority validation fix

Signed-off-by: Mike Rylander <mrylander@gmail.com>
6 years agoAvoid problems when auth recs are missing the 901c
Steven Callender [Fri, 15 Jun 2012 17:27:33 +0000 (13:27 -0400)]
Avoid problems when auth recs are missing the 901c

Fixed the authority.normalize_heading function to better handle
INT's when there is no 901 present. Now we look for the best-fit
control set instead of throwing a db-level error.

Signed-off-by: Steven Callender <stevecallender@esilibrary.com>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
6 years agoBest hold selection: fix a checkin-busting bug affecting Holds-go-home
Lebbeous Fogle-Weekley [Tue, 19 Mar 2013 20:13:35 +0000 (16:13 -0400)]
Best hold selection: fix a checkin-busting bug affecting Holds-go-home

Several subselects within SQL auxiliary queries can possibly return
multiple rows in situations where outer code requires that they return
only one or zero rows.

When you're using a best-hold order that includes htime or shtime, (such
as those with Holds-go-home) and if there are mutiple circs or transits
related to the copy in hand with certain characteristics, you can
trigger this bug.  The end user at the staff client gets one of those
nasty "Network error" dialogs, and the error in the SQL logs is:

> ERROR:  more than one row returned by a subquery used as an expression

Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
6 years agoFix bug in hold targeter introduced with Calculated Proximity Adjustments
Lebbeous Fogle-Weekley [Thu, 28 Mar 2013 19:08:37 +0000 (15:08 -0400)]
Fix bug in hold targeter introduced with Calculated Proximity Adjustments

The Calculated Proximity Adjustments feature introduced a bug in the
hold targeter where holds would not discount those copies from the
potentials list that aren't in either the Available or Reshelving
status.

This commit should restore the earlier, correct behavior.

Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
6 years agoStamping upgrade script for revive-qp-fts (LP 1161601).
Jason Stephenson [Mon, 1 Apr 2013 18:25:02 +0000 (14:25 -0400)]
Stamping upgrade script for revive-qp-fts (LP 1161601).

Signed-off-by: Jason Stephenson <jstephenson@mvlc.org>
6 years agoUpgrade script to bring back the stored proc
Mike Rylander [Thu, 28 Mar 2013 21:12:16 +0000 (17:12 -0400)]
Upgrade script to bring back the stored proc

Signed-off-by: Mike Rylander <mrylander@gmail.com>
Signed-off-by: Jason Stephenson <jstephenson@mvlc.org>
6 years agoRevert "Remove search.query_parser_fts from schema"
Mike Rylander [Mon, 25 Mar 2013 18:51:21 +0000 (14:51 -0400)]
Revert "Remove search.query_parser_fts from schema"

This reverts commit ab9fb958e387a20cfe9fafb6035fb72bc5f1fb3f.

Signed-off-by: Mike Rylander <mrylander@gmail.com>
Signed-off-by: Jason Stephenson <jstephenson@mvlc.org>
6 years agoRevert "Remove dependence on search.query_parser_fts proc"
Mike Rylander [Mon, 25 Mar 2013 18:50:55 +0000 (14:50 -0400)]
Revert "Remove dependence on search.query_parser_fts proc"

This reverts commit 6d8872cf120caf67ad6f65995b2c5155fa5ab652.

Conflicts:
Open-ILS/src/perlmods/lib/OpenILS/Application/Storage/Driver/Pg/QueryParser.pm

Signed-off-by: Mike Rylander <mrylander@gmail.com>
Signed-off-by: Jason Stephenson <jstephenson@mvlc.org>
6 years agoLP 1126545 repair staff 'return to results' tpac link
Bill Erickson [Fri, 29 Mar 2013 17:36:17 +0000 (13:36 -0400)]
LP 1126545 repair staff 'return to results' tpac link

Strip the extra "/" from the URL created by the "return to results"
button in the staff client TPAC view.  Havin the extra "/" in place
prevents the next/previous links from functioning.

Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Jason Etheridge <jason@esilibrary.com>
6 years agoAllow access to public hold notes via hold "blob"
Dan Wells [Fri, 15 Mar 2013 18:47:53 +0000 (14:47 -0400)]
Allow access to public hold notes via hold "blob"

If a hold note is marked as "public", it seems reasonable that it
should be visible to anyone who can see that hold, regardless of
who created the note.

Signed-off-by: Dan Wells <dbw2@calvin.edu>
Signed-off-by: Kyle Tomita <ktomita@catalystitservices.com>
6 years agoFix reversed logic for hold "blob" notes
Dan Wells [Fri, 15 Mar 2013 18:42:24 +0000 (14:42 -0400)]
Fix reversed logic for hold "blob" notes

Hold notes (at least at the data level) allow for both staff and
"private" notes to be attached. This code intended to show "private"
notes to the hold owner and staff notes to staff, but had the logic
reversed.

Signed-off-by: Dan Wells <dbw2@calvin.edu>
Signed-off-by: Kyle Tomita <ktomita@catalystitservices.com>
6 years agotpac : honor page locale in record detail copy grid
Bill Erickson [Mon, 25 Mar 2013 19:19:14 +0000 (15:19 -0400)]
tpac : honor page locale in record detail copy grid

The query used to load the record detail copy grid now uses the same
locale value set as the default for all cstore-editor calls.  This
solves the issue where copy location names, statuses, etc. are not
properly translated.

Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Pasi Kallinen <pasi.kallinen@pttk.fi>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
6 years agoStamping upgrade script for aoupa constraint fix
Mike Rylander [Wed, 27 Mar 2013 19:50:40 +0000 (15:50 -0400)]
Stamping upgrade script for aoupa constraint fix

Signed-off-by: Mike Rylander <mrylander@gmail.com>
6 years agoOrg unit prox adjustment tweaks
Lebbeous Fogle-Weekley [Thu, 14 Mar 2013 17:58:47 +0000 (13:58 -0400)]
Org unit prox adjustment tweaks

1) UI change - pre-select highest OU where user can edit

There's a dropdown in the org unit proximity adjustment interface that
affects the scope of what you can see in the table below.  Before, that
dropdown would be set to the user's home UO (say, BR1 in the example org
tree).  But you might have the permission to make changes everywhere, so
you add a proximity adjustment that affects BR3 in some way.  As soon as
you'd refresh the interface, your new proximity adjustment would not be
visible, because your scope selector would default to BR1.

So now the dropdown defaults to the first context where you have the
permission, which in all but eccentric setups should be the highest OU
(and therefore broadest scope) where you have the permission.

This could also improve other interfaces in the future.  Anything that
invokes openils.User.buildPermOrgSelector() can pass -1 as the third
parameter to get the new behavior.

2) Change UNIQUE index at DB layer.

This prevents the creation of rows that would give the hold targeter
conflicting directives.

Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
6 years agoUndo holding lib to SRE linkage (alt dist editor)
Dan Wells [Mon, 25 Mar 2013 21:39:19 +0000 (17:39 -0400)]
Undo holding lib to SRE linkage (alt dist editor)

Since the legacy record dropdown piggy-backs on the OPAC display, it
doesn't work in the alt interface (which has no clear OPAC context).

Worse, this leads to a Javascript error on the first editor load for
the distribution list, preventing subsequent editors.

A better fix will require more consideration, but for now, let's
revert to the unformatted display used in 2.3.

Also, this commit removes an unnecessary conditional (it was always
true).

Signed-off-by: Dan Wells <dbw2@calvin.edu>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
6 years agoMore authority fixed-field cleanup
Mike Rylander [Fri, 22 Mar 2013 14:40:59 +0000 (10:40 -0400)]
More authority fixed-field cleanup

Even with the efforts by many to add and harmonize the authority fixed-field
setup, and maintain functionality, there were some missing bits.  This commit
should take care of those by realigning the remaining field names and allwowing
those fields to be saved properly.

Signed-off-by: Mike Rylander <mrylander@gmail.com>
Signed-off-by: Bill Erickson <berick@esilibrary.com>
6 years agoMARC strip fields for Acquisitions
Bill Erickson [Thu, 21 Mar 2013 13:45:39 +0000 (09:45 -0400)]
MARC strip fields for Acquisitions

Added MARC strip field group selector to acquisitions upload interface
in the same manner as the group selector for non-acq Vandelay

Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
6 years agoLP#1160343: remove unused open-ils.circ settings
Galen Charlton [Tue, 26 Mar 2013 15:20:18 +0000 (11:20 -0400)]
LP#1160343: remove unused open-ils.circ settings

The circ_modifier and billing_types lists in the
open-ils.circ section of opensrf.xml are no longer
used.  This commit removes them from the opensrf.xml
example config.  It also removes the method
open-ils.circ.billing_type.retrieve.all, which has
not be used by any other code once in-DB billing types
were introduced.

Any external clients that used the method that was
removed should instead use
open-ils.circ.billing_type.ranged.retrieve.all.

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Signed-off-by: Bill Erickson <berick@esilibrary.com>
6 years agoLP 1150458 DB upgrade stamping
Bill Erickson [Wed, 20 Mar 2013 20:42:10 +0000 (16:42 -0400)]
LP 1150458 DB upgrade stamping

Signed-off-by: Bill Erickson <berick@esilibrary.com>
6 years agoChanged the way authority tags were being pulled to specifically use an aggregate...
Steven Callender [Thu, 7 Mar 2013 14:28:08 +0000 (09:28 -0500)]
Changed the way authority tags were being pulled to specifically use an aggregate array.

There appears to have been a change in postgres at some point between 9.0 and 9.1
to the aggregate method. Because of this, postgres was not returning the proper
results when pulling tags for authorities. This change will force postgres to do
a proper aggregate array call and return the correct results.

Signed-off-by: Steven Callender <stevecallender@esilibrary.com>
Signed-off-by: Bill Erickson <berick@esilibrary.com>
6 years agoDB upgrade script for 2.2.7
Lebbeous Fogle-Weekley [Wed, 20 Mar 2013 17:58:09 +0000 (13:58 -0400)]
DB upgrade script for 2.2.7

Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>
Signed-off-by: Bill Erickson <berick@esilibrary.com>
6 years ago2.3.4 -> 2.3.5 DB upgrade script
Bill Erickson [Wed, 20 Mar 2013 14:21:22 +0000 (10:21 -0400)]
2.3.4 -> 2.3.5 DB upgrade script

Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>
6 years agoPlaceholder DB upgrade for 2.3.4
Bill Erickson [Wed, 20 Mar 2013 15:35:30 +0000 (11:35 -0400)]
Placeholder DB upgrade for 2.3.4

Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Ben Shum <bshum@biblio.org>
6 years agoPlaceholder DB upgrade for 2.2.6
Bill Erickson [Wed, 20 Mar 2013 15:35:17 +0000 (11:35 -0400)]
Placeholder DB upgrade for 2.2.6

Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Ben Shum <bshum@biblio.org>
6 years agoTPAC - Syntax error breaks place-hold page
Lebbeous Fogle-Weekley [Tue, 19 Mar 2013 18:42:11 +0000 (14:42 -0400)]
TPAC - Syntax error breaks place-hold page

A recently introduced <label> tag had a ' mismatched with a ", breaking
the pickup library selector on the TPAC place-hold interface.

Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>
Signed-off-by: Galen Charlton <gmc@esilibrary.com>
6 years agoUpdating POT files
Mike Rylander [Fri, 15 Mar 2013 19:53:34 +0000 (19:53 +0000)]
Updating POT files

Signed-off-by: Mike Rylander <mrylander@gmail.com>
6 years agoTranslation updates!
Mike Rylander [Fri, 15 Mar 2013 19:14:05 +0000 (15:14 -0400)]
Translation updates!

Signed-off-by: Mike Rylander <mrylander@gmail.com>
6 years agoBumping OpenILS.pm version number
Mike Rylander [Fri, 15 Mar 2013 18:47:19 +0000 (14:47 -0400)]
Bumping OpenILS.pm version number

Signed-off-by: Mike Rylander <mrylander@gmail.com>
6 years agoSIP re-login fetches correct auth session
Bill Erickson [Fri, 3 Aug 2012 19:08:51 +0000 (15:08 -0400)]
SIP re-login fetches correct auth session

After a login session expires and the SIP server logs in again to
acquire a new authtoken, be sure to use the new authtoken when fetching
the new session object.  Otherwise, the login_session is left undef and
you may see errors like this when checking in items:

Can't call method "ws_ou" on an undefined value at
/usr/local/share/perl/5.10.1/OpenILS/SIP/Transaction/Checkin.pm line 77.

Incidentally, the checkin code is the only code that references
$sip_handler->{login_session}.  And the problem listed above only occurs
when the login session expires just prior to a checkin call.  In any
other scenario, the login_session would ultimately get repaired.

Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Galen Charlton <gmc@esilibrary.com>
6 years agoTPAC - Account preferences headings
Melissa Lefebvre [Mon, 25 Feb 2013 15:42:15 +0000 (10:42 -0500)]
TPAC - Account preferences headings

Currently, all tabs in My Account show a heading once clicked(Items Checked
Out shows "Current Items Checked Out" heading when clicked); however,
under the Account Preferences for a change of username, password, and
email address no headings appear.

Div classes were added to update_password.tt2, update_username.tt2, and
update_email.tt2 for consistancy in the My Account area.

Signed-off-by: Melissa Lefebvre <mlefebvre@biblio.org>
Signed-off-by: Ben Shum <bshum@biblio.org>
Signed-off-by: Galen Charlton <gmc@esilibrary.com>
6 years agoLP#1155496: fix typo
Galen Charlton [Fri, 15 Mar 2013 15:35:25 +0000 (11:35 -0400)]
LP#1155496: fix typo

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Signed-off-by: Ben Shum <bshum@biblio.org>
6 years agoadd typo fix back
Galen Charlton [Fri, 15 Mar 2013 17:18:45 +0000 (13:18 -0400)]
add typo fix back

I missed this when I was resolving a merge confict
with the previous patch.

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
6 years agoFix some typos
Galen Charlton [Fri, 15 Mar 2013 17:07:28 +0000 (13:07 -0400)]
Fix some typos

Except for the typo in staff client's multiclass_search_help.html,
the rest are in comments.

Signed-off-by: Pasi Kallinen <pasi.kallinen@pttk.fi>
Conflicts:

Open-ILS/examples/apache/startup.pl

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
6 years agoFix detection of empty Syndetics AC data elements
Jeff Godin [Thu, 14 Mar 2013 18:15:28 +0000 (14:15 -0400)]
Fix detection of empty Syndetics AC data elements

Due to a change in how the vendor responds to requests for data
elements which are not present for a requested title, we need to
change how we detect "empty" data elements in the "data_exists"
function.

First, we check for the new "No Data Available" title element, then
we check for the previously-used value of "error".

This change resolves the symptom where the catalog would display
tabs for "Author Notes", "Excerpt", "Table of Contents", on every
record in the catalog, even when clicking each tab would show that
the content was not present / available.

Signed-off-by: Jeff Godin <jgodin@tadl.org>
Signed-off-by: Ben Shum <bshum@biblio.org>
6 years agoLP#1078222 - Add transparency to newtab2.PNG
Michael Peters [Wed, 14 Nov 2012 17:06:38 +0000 (12:06 -0500)]
LP#1078222 - Add transparency to newtab2.PNG

Some users noted that this image looked bad with white
background when changing to a different colored "welcome"
screen.

This simply converts the icon to a transparent PNG file.

Signed-off-by: Michael Peters <mrpeters@library.in.gov>
Signed-off-by: Galen Charlton <gmc@esilibrary.com>
6 years agofix typo preventing transit abort error dialogs
Jason Etheridge [Fri, 22 Jun 2012 16:41:20 +0000 (12:41 -0400)]
fix typo preventing transit abort error dialogs

Signed-off-by: Jason Etheridge <jason@esilibrary.com>
Signed-off-by: Galen Charlton <gmc@esilibrary.com>
6 years agoRename "Search Preferences" tab to "Search and History Preferences".
Jason Stephenson [Mon, 26 Nov 2012 20:04:59 +0000 (15:04 -0500)]
Rename "Search Preferences" tab to "Search and History Preferences".

This change better reflects the complete meaning of the preference tab.

Signed-off-by: Jason Stephenson <jstephenson@mvlc.org>
Signed-off-by: Ben Shum <bshum@biblio.org>
6 years agoLP1054322 - libparent-perl not needed for Ubuntu Precise
Ben Shum [Fri, 22 Feb 2013 05:46:17 +0000 (00:46 -0500)]
LP1054322 - libparent-perl not needed for Ubuntu Precise

The newer version of Perl 5.14 which comes with Ubuntu Precise already
contains the contents of libparent-perl, which is now an obsolete package.

Remove this dependency from the extra debs installed for Debian and Ubuntu
but then add to the extras for Debian Squeeze and create a new extras just
for Ubuntu Lucid so that it continues to be installed on those systems and
not on Ubuntu Precise.

Signed-off-by: Ben Shum <bshum@biblio.org>
Signed-off-by: Galen Charlton <gmc@esilibrary.com>
6 years agoLP1103068 - Show titles and authors for pre-cat items
Ben Shum [Thu, 14 Mar 2013 21:40:04 +0000 (17:40 -0400)]
LP1103068 - Show titles and authors for pre-cat items

As noted in LP1103068, TPAC does not currently show anything when viewing a
pre-cataloged item in the my account summary of items checked out.

If the call number is -1 (pre-cat), then show the dummy title and dummy author
values, which fields are hopefully relevant as long as staff were good to put
something meaningful in those during creation of the pre-cat.

Also, wrap the title in a check for attrs.title.

Signed-off-by: Ben Shum <bshum@biblio.org>
Signed-off-by: Michael Peters <mpeters@emeralddata.net>
6 years agoLP1116258 - Hide download circ history if none to be found
Ben Shum [Fri, 15 Mar 2013 05:04:06 +0000 (01:04 -0400)]
LP1116258 - Hide download circ history if none to be found

As noted in LP1116258, if there is no circ history for a patron, but they
click on the Download CSV button, it leads them to unhappy server errors.

So, hide the Download CSV button unless the number of circs > 0.

Signed-off-by: Ben Shum <bshum@biblio.org>
Signed-off-by: Pasi Kallinen <pasi.kallinen@pttk.fi>
6 years agoLP#791881: make it possible to save copy location translations
Galen Charlton [Thu, 10 Jan 2013 17:27:30 +0000 (12:27 -0500)]
LP#791881: make it possible to save copy location translations

TranslatorPopup requires that the i18n fieldmapper class be loaded.

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Signed-off-by: Ben Shum <bshum@biblio.org>
6 years agoTPAC: Address severe accessibility issues
Dan Scott [Thu, 14 Mar 2013 21:52:09 +0000 (17:52 -0400)]
TPAC: Address severe accessibility issues

Using the Chrome Accessibility Dev Tools extension to run accessibility
audits against the TPAC turned up some "severe" issues, such as input
widgets lacking labels. In most cases the text for the labels was
present, but it needed to be associated with a <label> element. In some
other cases, it was easier to use the aria-label attribute to give the
widget an accessible label. Finally, we can use the placeholder
attribute where warranted as an accessible label.

These changes address the first set of accessibility issues I came
across via the audits and could address quickly.

Signed-off-by: Dan Scott <dscott@laurentian.ca>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
6 years agoFix spelling error of "Successfully" in ils_events.xml
Michael Peters [Fri, 15 Mar 2013 13:57:55 +0000 (09:57 -0400)]
Fix spelling error of "Successfully" in ils_events.xml

LP#1155495 "card transaction was processed successfuly" is
spelled incorrectly.   This corrects that misspelling.

Signed-off-by: Michael Peters <mpeters@emeralddata.net>
Signed-off-by: Galen Charlton <gmc@esilibrary.com>
6 years agoUse a consistent spelling of "Pickup" throughout the code
Michael Peters [Fri, 15 Mar 2013 13:47:26 +0000 (09:47 -0400)]
Use a consistent spelling of "Pickup" throughout the code

LP#1155494 In myopac/holds.tt2 the text "PickUp Location" is
used, and everywhere else it's "Pickup Location".
Pick one spelling and stick to it.

Signed-off-by: Michael Peters <mpeters@emeralddata.net>
Signed-off-by: Galen Charlton <gmc@esilibrary.com>
6 years agoEnable dojo in TPAC if using newer Novelist API
Ben Shum [Fri, 22 Feb 2013 05:03:58 +0000 (00:03 -0500)]
Enable dojo in TPAC if using newer Novelist API

Enable dojo if the apache variable for Novelist URL is set. This avoids
issues such as needing to have dojo enabled via either AutoSuggest or
Google Books preview, which may not be enabled by every TPAC.

Signed-off-by: Ben Shum <bshum@biblio.org>
Signed-off-by: Bill Erickson <berick@esilibrary.com>
6 years agoSupport new-style Novelist for TPAC.
Bill Erickson [Thu, 11 Oct 2012 20:09:12 +0000 (16:09 -0400)]
Support new-style Novelist for TPAC.

Updates the Novelist loading script to use the newer Novelist API.

Includes minor change to example/apache/eg_vhost.conf for environment
variables.

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