Authentication block logging

* Login too-many-auth-failures message at Info level instead of Internal
* Log the username getting blocked

Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Galen Charlton <gmc@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>

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>

lp797321: Disable the Apply Payment button during payment processing to mitigate chance of duplicate payments.

I can't actually reproduce the creation of duplicate payments locally with
trunk/2.x, but there are reports of it, and this doesn't seem to hurt.  Not

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

Whitespace cleanup in aisle 3 (O:A:Reporter)

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

LP#740466 Sort report folders by name

Add order_by to Reporter.pm to sort report folders by name

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

LP#739444 Incrementing opac.dtd copyright date

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

Fix Hold Capture Check-In Mode, similar to 9641dcf040798aac1e274253c8ae8a722674bcc2

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

Fix some cases where View MARC upon colliding import broke in Vandelay

git-svn-id: svn://svn.open-ils.org/ILS/trunk@17763 dcc99617-32d9-48b4-a31d-7c20da2025e4
Signed-off-by: Jason Etheridge <jason@esilibrary.com>

stop hold permit check from crashing if age protection in effect

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

Revert "Wrap proximity refreshing in a transaction to avoid timespans where there is no such information"

This reverts commit 49d240fcb33ebea95c06bb2ae62689a10a261fd7.

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

Wrap proximity refreshing in a transaction to avoid timespans where there is no such information

git-svn-id: svn://svn.open-ils.org/ILS/branches/rel_1_6@20458 dcc99617-32d9-48b4-a31d-7c20da2025e4

fix disappearing cursor/caret for Check In

git-svn-id: svn://svn.open-ils.org/ILS/branches/rel_1_6@20317 dcc99617-32d9-48b4-a31d-7c20da2025e4

Patch from Jeff Davis addressing parameter usage issues

git-svn-id: svn://svn.open-ils.org/ILS/branches/rel_1_6@20247 dcc99617-32d9-48b4-a31d-7c20da2025e4

in the MARC Editor, put the callnumber and barcode textboxes for Fast Item Add on a separate row

git-svn-id: svn://svn.open-ils.org/ILS/branches/rel_1_6@20154 dcc99617-32d9-48b4-a31d-7c20da2025e4

font_helper in this version depends on JSAN being set up by my_init()

git-svn-id: svn://svn.open-ils.org/ILS/branches/rel_1_6@20141 dcc99617-32d9-48b4-a31d-7c20da2025e4

label interface, run persist_helper() before my_init(), since the latter depends on values set by the former

git-svn-id: svn://svn.open-ils.org/ILS/branches/rel_1_6@20108 dcc99617-32d9-48b4-a31d-7c20da2025e4

Backported version of r20094: Protect pcrud-ish retrieve from an empty result set

git-svn-id: svn://svn.open-ils.org/ILS/branches/rel_1_6@20097 dcc99617-32d9-48b4-a31d-7c20da2025e4

test for pre-cat slightly off in OPAC My Account, spotted by Steve Callender

git-svn-id: svn://svn.open-ils.org/ILS/branches/rel_1_6@20020 dcc99617-32d9-48b4-a31d-7c20da2025e4

Whoops: didn't mean to backport r19942 to the 1.6 branch. Not appropriate here.

git-svn-id: svn://svn.open-ils.org/ILS/branches/rel_1_6@19946 dcc99617-32d9-48b4-a31d-7c20da2025e4

Backport r19942 from trunk

Booking: This should a) make booking interfaces behave better when a staff
login session times out and is replaced by use of a re-auth dialog, and b)
wrap booking interfaces in a <browser> element with the attendant
back/forward/reload buttons that most similar interfaces already had.

git-svn-id: svn://svn.open-ils.org/ILS/branches/rel_1_6@19945 dcc99617-32d9-48b4-a31d-7c20da2025e4

Patch from Brandon Uhlman addressing SIP2 unavailable holds display.

The unavail_holds() function in OpenILS::SIP::Patron is a placeholder that returns an empty set, instead of an actual list of a patrons' holds not currently ready for pickup.

When an SC determines whether a patron has holds ready for pickup by comparing the cardinalities of hold_items() and unavail_holds() [*] using the existing unavail_holds() logic, it effectively reports every currently placed hold as ready for pickup, which is generally not true.

The attached patch assumes that unavailable holds are those that meet one of two sets of conditions:
- hold requests that are not cancelled (cancel_time = null), not fulfilled (fulfillment time = null), and not captured (capture_time = null)
- hold requests that are neither cancelled nor fulfilled (as above), are captured (capture_time != null), and for which, additionally, the current_copy does not have the status 'On holds shelf' (status = 8). The only items that should meet this condition are those that are in transit to a location where they will be added to the holds shelf.

The patch iterates through the items meeting each condition, and adds them to a master list of relevant holds, which it then processes (in the same way that hold_items is processed) and returns.

This patch was developed against and tested on rel_1_6_0 from SVN (~ 1.6.0.2), but should apply cleanly against current rel_1_6_0 and trunk; OpenSRF 1.2.2, PostgreSQL 8.3.8, Ubuntu Hardy 64-bit.

[*] The two cardinalities are returned by SIP message 64 as fixed-length integers in positions 35-38 and 55-58 respectively, so we don't need to iterate through the list ourseles.

git-svn-id: svn://svn.open-ils.org/ILS/branches/rel_1_6@19896 dcc99617-32d9-48b4-a31d-7c20da2025e4

Make sure "ol" propagates through BibTemplate subject search links for OPAC range hiding

git-svn-id: svn://svn.open-ils.org/ILS/branches/rel_1_6@19815 dcc99617-32d9-48b4-a31d-7c20da2025e4

Allow the "result_field" to be used as the default alias in the output column of a row or set returning transform

git-svn-id: svn://svn.open-ils.org/ILS/branches/rel_1_6@19802 dcc99617-32d9-48b4-a31d-7c20da2025e4

Prevent creation of report folders with empty folder names

Addresses LP # 720755. Clicking the "Create Folder" button when
no folder name has been supplied is now a no-op.

git-svn-id: svn://svn.open-ils.org/ILS/branches/rel_1_6@19744 dcc99617-32d9-48b4-a31d-7c20da2025e4

Backport r19682 from trunk

Booking: don't bail out completely on COPY_ALERT_MESSAGE when returning items

Provide overridability.

git-svn-id: svn://svn.open-ils.org/ILS/branches/rel_1_6@19685 dcc99617-32d9-48b4-a31d-7c20da2025e4

Remove duplicate permission.perm_list entries from data seed values

These duplicate perms somehow slipped into r18962 and need to
go away.

git-svn-id: svn://svn.open-ils.org/ILS/branches/rel_1_6@19676 dcc99617-32d9-48b4-a31d-7c20da2025e4

Draw pre-cataloged items (author and title) in My Account checkedout screen

Two problems: FETCH_MODS_FROM_COPY returns an object, so we weren't
getting to the point of fetching the raw copy itself, and there was
some confusion about pre-cataloged vs. non-cataloged items that is
hopefully straightened out now.

Addresses LP # 723895

git-svn-id: svn://svn.open-ils.org/ILS/branches/rel_1_6@19660 dcc99617-32d9-48b4-a31d-7c20da2025e4

Fix cloning of AutoField widgets: LP #728055

Cloning Action/Trigger event definitions was not working properly, as
all values were being copied from the original object using their display
strings rather than the underlying values.

Bill Erickson suggested a fix in IRC that works, so full credit to him
for this change!

git-svn-id: svn://svn.open-ils.org/ILS/branches/rel_1_6@19593 dcc99617-32d9-48b4-a31d-7c20da2025e4

correct the MOBTS view with regard to reservation additions (only affects reporting)

git-svn-id: svn://svn.open-ils.org/ILS/branches/rel_1_6@19579 dcc99617-32d9-48b4-a31d-7c20da2025e4

This should prevent some spurious errors with bill history (a race condition between list selection and the bib summary pane updating itself), but needs testing before we backport (apologies in advance for commiting untested).

git-svn-id: svn://svn.open-ils.org/ILS/branches/rel_1_6@19521 dcc99617-32d9-48b4-a31d-7c20da2025e4

Correct the install location for Dojo i18n bundles

We had been converting xx-YY into xx/yy for a directory structure,
but current evidence shows that it should be xx-yy. Hard to believe
we've overlooked this for so long. Also, with the exception of the
en-ca/en-gb/en-us locales, we should probably trim the locale to
just xx - but that's a different problem, and let's just focus on
getting this working to begin with.

Addresses Launchpad # 723489

git-svn-id: svn://svn.open-ils.org/ILS/branches/rel_1_6@19510 dcc99617-32d9-48b4-a31d-7c20da2025e4

Backport r19470 from trunk

Booking: fix the targeter in the I'm-reserving-this-exact-resource-with-this-
exact-barcode case.

It should have always worked like this. This should better mirror the behavior
in the holds targeter when placing a copy-level hold.  Basically don't fail
to target a resource for a reservation later just because it's not in an
available status right at this moment.

git-svn-id: svn://svn.open-ils.org/ILS/branches/rel_1_6@19473 dcc99617-32d9-48b4-a31d-7c20da2025e4

Normalize ISBNs by stripping hyphens in search methods and in reporter.materialized_simple_record

We weren't normalizing ISBNs in search or in reporter.materialized_simple_record
which required users to exactly match the form of the ISBN as entered in the
MARC record to get a successful search result - pretty hit or miss.

The longer term fix in 2.0 and above is to make the ISBN search search against
the identifier|isbn index, which also gets ISBN10/ISBN13 equivalence goodness.
But this patch will be relatively easy to backport through to 1.6.1 and plugs
some of the holes in our exposed APIs and search points in the short term.

git-svn-id: svn://svn.open-ils.org/ILS/branches/rel_1_6@19436 dcc99617-32d9-48b4-a31d-7c20da2025e4

Correct syntax error in BadDebt.pm

git-svn-id: svn://svn.open-ils.org/ILS/branches/rel_1_6@19435 dcc99617-32d9-48b4-a31d-7c20da2025e4

Fix ModsParser.pm bug involving invalid ISBNs.  See bug #697398.

git-svn-id: svn://svn.open-ils.org/ILS/branches/rel_1_6@19385 dcc99617-32d9-48b4-a31d-7c20da2025e4

backport r19364 from trunk for stat cat field value removal in patron editor

git-svn-id: svn://svn.open-ils.org/ILS/branches/rel_1_6@19366 dcc99617-32d9-48b4-a31d-7c20da2025e4

Backport r19288 : Fix checkin backdate assumptions

checkin_handle_backdate() did not properly interpret bill data (voided too much), so lets use void_overdues() instead

git-svn-id: svn://svn.open-ils.org/ILS/branches/rel_1_6@19348 dcc99617-32d9-48b4-a31d-7c20da2025e4

enable the checkbox column so we can delete hard due dates

git-svn-id: svn://svn.open-ils.org/ILS/branches/rel_1_6@19314 dcc99617-32d9-48b4-a31d-7c20da2025e4

improve performance of multi-class searches

Backport of trunk commit 18233.

For as yet unknown reasons, Postgres sometimes uses
nested loops instead of hash joins when joining subqueries
in a multi-class bib search.  This can result in searches
such as "author:rowling title:harry potter" never completing.

Removing the interior limit clause can both speed up the query
and make search results more accurate when combining queries
of two or more common words.

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
git-svn-id: svn://svn.open-ils.org/ILS/branches/rel_1_6@19263 dcc99617-32d9-48b4-a31d-7c20da2025e4

don't offer to cancel related transits that have already completed when canceling holds.  related to 19234, but data structure is different

git-svn-id: svn://svn.open-ils.org/ILS/branches/rel_1_6@19236 dcc99617-32d9-48b4-a31d-7c20da2025e4

Backport r19229 from trunk

Booking: these lines don't belong; they were erroneously copied and pasted
from a neighboring section of the file

git-svn-id: svn://svn.open-ils.org/ILS/branches/rel_1_6@19231 dcc99617-32d9-48b4-a31d-7c20da2025e4

Backport r19214 from trunk

Booking: Robert Soulliere spotted and fixed a bug in processing reservation-
related overdue fines.  See https://bugs.launchpad.net/evergreen/+bug/705061

git-svn-id: svn://svn.open-ils.org/ILS/branches/rel_1_6@19216 dcc99617-32d9-48b4-a31d-7c20da2025e4

Build and flatten a tree, correcting a sorting issue in some OU dropdowns.

The previous code assumed that work org units would be delivered in hierarchical order, but alas, they are not. Thus, we build the hierarchy and then flatten it, sorting at each level.
This will be non-fast with many work OUs, but the common case is a small set, which is not painful.

Further improvement is warranted when the above proves false.

git-svn-id: svn://svn.open-ils.org/ILS/branches/rel_1_6@19210 dcc99617-32d9-48b4-a31d-7c20da2025e4

some interfaces (or at least org_unit_settings.js) was not infering the identifier value here.  This helps the filtering org select dijit in that interface

git-svn-id: svn://svn.open-ils.org/ILS/branches/rel_1_6@19206 dcc99617-32d9-48b4-a31d-7c20da2025e4

Enable truncation attribute in Z39.50 queries to be left out entirely

Per https://bugs.launchpad.net/evergreen/+bug/702695, some Z39.50
servers hate the truncation attribute (@attr 5=anything) and always
return 0 hits.

This gives you the ability to specify a negative value for the
truncation attribute and thereby prevent the truncation attribute
from being included in the Z39.50 query for a given server.

git-svn-id: svn://svn.open-ils.org/ILS/branches/rel_1_6@19178 dcc99617-32d9-48b4-a31d-7c20da2025e4

fix regression with org selector when specifying ?ol= and the org unit hiding setting is unset

git-svn-id: svn://svn.open-ils.org/ILS/branches/rel_1_6@19156 dcc99617-32d9-48b4-a31d-7c20da2025e4

backporting r16742,r17492 for proper dojo timestamp handling

git-svn-id: svn://svn.open-ils.org/ILS/branches/rel_1_6@19126 dcc99617-32d9-48b4-a31d-7c20da2025e4

Addressing https://bugs.launchpad.net/evergreen/+bug/696078 where deleted/inactive records are included in the freshmeat feed.

git-svn-id: svn://svn.open-ils.org/ILS/branches/rel_1_6@19108 dcc99617-32d9-48b4-a31d-7c20da2025e4

Patch from Robert Soulliere to backport Class::DBI version checking from 2.0: https://bugs.launchpad.net/evergreen/+bug/692717

git-svn-id: svn://svn.open-ils.org/ILS/branches/rel_1_6@19098 dcc99617-32d9-48b4-a31d-7c20da2025e4

Fix exception thrown in MARC editor searching for bib sources

As reported in https://bugs.launchpad.net/evergreen/+bug/695526,
the attempt to retrieve all of the config.bib_source values using
openils.PermaCrud was failing because the retrieveAll() method
relies on fieldmapper.IDL's setting of the
fieldmapper[hint].Identifier property for each class - but in the
1.6 releases, we're still using fmclasses in XUL instead of
fieldmapper.AutoIDL.

So, to fix the direct problem, we can simply set the property
ourselves on the desired class before invoking retrieveAll();
this avoids touching code paths that might have wider unintended
consequences.

git-svn-id: svn://svn.open-ils.org/ILS/branches/rel_1_6@19088 dcc99617-32d9-48b4-a31d-7c20da2025e4

commit 620f5d756619b84200add4c534a7cf22e1c0d037
further fixes to null indicator handling in flat-text MARC editor

[1] ensure that null indicators are converted to blanks
[2] fix glitch when supplying default indicator values

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
git-svn-id: svn://svn.open-ils.org/ILS/branches/rel_1_6@19059 dcc99617-32d9-48b4-a31d-7c20da2025e4

When the subfield delimiter is configured as "$" (the default), use {dollar} to represent a literal dollar sign.  This is apparently in line with other Breaker-format tools.  Fix suggested by Galen Charlton, thanks gmc!

git-svn-id: svn://svn.open-ils.org/ILS/branches/rel_1_6@19052 dcc99617-32d9-48b4-a31d-7c20da2025e4

addressing two Breaker-format issues: 1) supply an "empty" (space) value for indicators in the internal format if they are left out and 2) add indicator placeholders to incoming Breaker datafield lines if they seem to be missing.  Addresses https://bugs.launchpad.net/evergreen/+bug/693468

git-svn-id: svn://svn.open-ils.org/ILS/branches/rel_1_6@19050 dcc99617-32d9-48b4-a31d-7c20da2025e4

allow vandelay.replace_field() to handle a complex field spec by splitting and looping

git-svn-id: svn://svn.open-ils.org/ILS/branches/rel_1_6@19045 dcc99617-32d9-48b4-a31d-7c20da2025e4

Backport of r18989: boolean test for hard due force flag

git-svn-id: svn://svn.open-ils.org/ILS/branches/rel_1_6@18991 dcc99617-32d9-48b4-a31d-7c20da2025e4

repaired order-by in update_hard_due_dates function to prevent sql errors

git-svn-id: svn://svn.open-ils.org/ILS/branches/rel_1_6@18988 dcc99617-32d9-48b4-a31d-7c20da2025e4

upgrade cleanup in prep for cutting 1.6.2.0

git-svn-id: svn://svn.open-ils.org/ILS/branches/rel_1_6@18976 dcc99617-32d9-48b4-a31d-7c20da2025e4

backport r14033 from trunk for unsetting (nullifying) a circ modifier

git-svn-id: svn://svn.open-ils.org/ILS/branches/rel_1_6@18970 dcc99617-32d9-48b4-a31d-7c20da2025e4

Patch from James Fournie to address https://bugs.launchpad.net/evergreen/+bug/622908 wherein we learn that related item physical description might be used as the main PD of the main item, if the main item lacks such a field in the MARC

git-svn-id: svn://svn.open-ils.org/ILS/branches/rel_1_6@18967 dcc99617-32d9-48b4-a31d-7c20da2025e4

Forward-port of r18962: Addressing https://bugs.launchpad.net/evergreen/+bug/638309 -- Thanks to Steve Callender for the original report and diagnosis

git-svn-id: svn://svn.open-ils.org/ILS/branches/rel_1_6@18963 dcc99617-32d9-48b4-a31d-7c20da2025e4

partial backport of r18957 to capture the fixes for vandelay.replace_field -- specifically, normalizing the XML for comparison when deciding if the "add" side of a replace should proceed

git-svn-id: svn://svn.open-ils.org/ILS/branches/rel_1_6@18961 dcc99617-32d9-48b4-a31d-7c20da2025e4

big thinko.  Going by the stock receipt template, this should be the workstation lib.  Destination is handled by a different variable

git-svn-id: svn://svn.open-ils.org/ILS/branches/rel_1_6@18952 dcc99617-32d9-48b4-a31d-7c20da2025e4

transit isn't in payload first time through with ROUTE_ITEM checkin, fallback on check.org

git-svn-id: svn://svn.open-ils.org/ILS/branches/rel_1_6@18948 dcc99617-32d9-48b4-a31d-7c20da2025e4

Enable Ingest to ingest multiple located URIs for a single org-unit at once

Previous logic incorrectly restricted a given bib record to have one located
URI per org-unit - unless you reingested it multiple times, each time adding
one more URI per org-unit.

We also add the strict pragma and adjust some syntax accordingly.

git-svn-id: svn://svn.open-ils.org/ILS/branches/rel_1_6@18938 dcc99617-32d9-48b4-a31d-7c20da2025e4

protect against empty control fields (bad MARC)

git-svn-id: svn://svn.open-ils.org/ILS/branches/rel_1_6@18917 dcc99617-32d9-48b4-a31d-7c20da2025e4

Be consistent with a misspelled field. Credit to Robert Soulliere for pointing
out my mistaken understanding of where this was already fixed and where it
wasn't.

git-svn-id: svn://svn.open-ils.org/ILS/branches/rel_1_6@18911 dcc99617-32d9-48b4-a31d-7c20da2025e4

Normalize spaces in notcontains advance search query terms to address bug https://bugs.launchpad.net/evergreen/+bug/677122

git-svn-id: svn://svn.open-ils.org/ILS/branches/rel_1_6@18908 dcc99617-32d9-48b4-a31d-7c20da2025e4

protect some of the supercat browse interfaces from unboundedness

E.g., browsing from a call number that contains forward slashes, e.g.,

"02/05/2004"

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
git-svn-id: svn://svn.open-ils.org/ILS/branches/rel_1_6@18886 dcc99617-32d9-48b4-a31d-7c20da2025e4

Backport some booking related commits from trunk.

r17980: more transaction cleanups
--
r18502: Booking: make the circ module avoid booking code if the booking service
isn't up

Previously, it was not possible to turn the booking service off and have
working circ.  Now you should be able to turn it off if you don't need it.
--
r18514: If we already know we're not using booking, skip a redundant and
expensive test for the possibility of capturing a hold. We're just going to
try actually doing it a little farther down.

git-svn-id: svn://svn.open-ils.org/ILS/branches/rel_1_6@18873 dcc99617-32d9-48b4-a31d-7c20da2025e4

back-porting cstoreditor fixes to help prevent transaction leaking

git-svn-id: svn://svn.open-ils.org/ILS/branches/rel_1_6@18871 dcc99617-32d9-48b4-a31d-7c20da2025e4

logic error (inversion) in retargetting of previous copy when no other holds are looking at it

git-svn-id: svn://svn.open-ils.org/ILS/branches/rel_1_6@18860 dcc99617-32d9-48b4-a31d-7c20da2025e4

Patch from Jeff Godin, backported from trunkified version:

In the staff client, the holds context menu option "Edit PickupLibrary" displays and allows selection of org units that it shouldn't.

Org units whose type has can_have_users = FALSE are not valid pickup locations.

Using the default types as an example, CONS and SYS1 are invalid pickup locations.

The OPAC enforces this when allowing patrons to edit a hold.

The staff client does not enforce this when staff edit a hold.

The can_have_users() method of the au object returns a string 't' or 'f' which is then compared to 0.

The attached patch corrects this issue. ( foo == 0 becomes !isTrue(foo) ).

git-svn-id: svn://svn.open-ils.org/ILS/branches/rel_1_6@18856 dcc99617-32d9-48b4-a31d-7c20da2025e4

merge r18716 from rel_2_0 to cache this setting.  Thanks to James Fournie for https://bugs.launchpad.net/evergreen/+bug/677311  We should probably do this as a cookie instead for even fewer network calls.  I can't see the setting changing all that frequently

git-svn-id: svn://svn.open-ils.org/ILS/branches/rel_1_6@18806 dcc99617-32d9-48b4-a31d-7c20da2025e4

back-porting 17292: added custom internal_id handler to SIP patron module

git-svn-id: svn://svn.open-ils.org/ILS/branches/rel_1_6@18777 dcc99617-32d9-48b4-a31d-7c20da2025e4

backport r18757 from trunk for opac org hiding extension to place hold pickup lib menu

git-svn-id: svn://svn.open-ils.org/ILS/branches/rel_1_6@18758 dcc99617-32d9-48b4-a31d-7c20da2025e4

Trim leading and trailing whitespace from metabib.full_rec values - fixes ISSN quicksearch (thanks for noticing this, Dan Wells!)

git-svn-id: svn://svn.open-ils.org/ILS/branches/rel_1_6@18752 dcc99617-32d9-48b4-a31d-7c20da2025e4

asset.uri needs a seed entry for queries of its ID values

To fix a problem with ingesting URIs, Ingest.pm was changed to reflect
the maximum asset.uri.id value rather than the asset.call_number.id
value. However, with no entries in asset.uri, the query returned a
NULL object which broke ingest entirely. Adding a seed entry avoids
this problem.

git-svn-id: svn://svn.open-ils.org/ILS/branches/rel_1_6@18749 dcc99617-32d9-48b4-a31d-7c20da2025e4

Reverting and extending r18472 to correct implementation thinko

git-svn-id: svn://svn.open-ils.org/ILS/branches/rel_1_6@18736 dcc99617-32d9-48b4-a31d-7c20da2025e4

Backport r17447 from trunk: fix wrong-headed permissions tests in booking

This should have been backported to rel_1_6 immediately. :-(

git-svn-id: svn://svn.open-ils.org/ILS/branches/rel_1_6@18733 dcc99617-32d9-48b4-a31d-7c20da2025e4

Backport r18688 from trunk: booking bugfix

git-svn-id: svn://svn.open-ils.org/ILS/branches/rel_1_6@18690 dcc99617-32d9-48b4-a31d-7c20da2025e4

do not use TRUNCATE when refreshing reporter.materialized_simple_record

Previous behavior would break Slony replication after doing a
bib load.  Since a deletion is slower than a truncate, if you're
not using Slony replication, you may prefer to truncate rmsr
prior to calling reporter.enable_materialized_simple_record_trigger.

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
git-svn-id: svn://svn.open-ils.org/ILS/branches/rel_1_6@18679 dcc99617-32d9-48b4-a31d-7c20da2025e4

Bacport r18657: syntax problem spotted by Jason Etheridge

git-svn-id: svn://svn.open-ils.org/ILS/branches/rel_1_6@18659 dcc99617-32d9-48b4-a31d-7c20da2025e4

Backporting r18652: Teach vandelay.replace_field to be a little smarter by allowing simple cases of both replacing and regexp-testing the same subfield

git-svn-id: svn://svn.open-ils.org/ILS/branches/rel_1_6@18654 dcc99617-32d9-48b4-a31d-7c20da2025e4

fix user password reset request time column def

Needs to be a timestamp with time zone; fixes a bug
where it was interpreted as a UTC time, throwing off
the calculation of the expiration of the password reset
request.

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
git-svn-id: svn://svn.open-ils.org/ILS/branches/rel_1_6@18628 dcc99617-32d9-48b4-a31d-7c20da2025e4

backporting r18609 patch from bshum for catalog search in button bar

git-svn-id: svn://svn.open-ils.org/ILS/branches/rel_1_6@18611 dcc99617-32d9-48b4-a31d-7c20da2025e4

cut down on reloading of IDL classes

git-svn-id: svn://svn.open-ils.org/ILS/branches/rel_1_6@18605 dcc99617-32d9-48b4-a31d-7c20da2025e4

add the Batch Bib Update module to startup.pl

git-svn-id: svn://svn.open-ils.org/ILS/branches/rel_1_6@18595 dcc99617-32d9-48b4-a31d-7c20da2025e4

Backport r18535 from trunk: hard due date config UIs

git-svn-id: svn://svn.open-ils.org/ILS/branches/rel_1_6@18588 dcc99617-32d9-48b4-a31d-7c20da2025e4

Patch from Steve Callender to address a booking-related collection agency reporting problem

git-svn-id: svn://svn.open-ils.org/ILS/branches/rel_1_6@18582 dcc99617-32d9-48b4-a31d-7c20da2025e4

Backporting r18561 from trunk: Make vandelay.add_field smarter about applying add rules when no target field exists before hand

git-svn-id: svn://svn.open-ils.org/ILS/branches/rel_1_6@18563 dcc99617-32d9-48b4-a31d-7c20da2025e4

backport r18498-18499 from trunk for staff-initiated age overdue circ/item to Lost functionality, with upgrade script

git-svn-id: svn://svn.open-ils.org/ILS/branches/rel_1_6@18528 dcc99617-32d9-48b4-a31d-7c20da2025e4

missing clear_expired_circ_history.srfsh was breaking the build, copied from trunk

git-svn-id: svn://svn.open-ils.org/ILS/branches/rel_1_6@18526 dcc99617-32d9-48b4-a31d-7c20da2025e4

Backport r18516: Ensure call numbers are returned to the holdings editor in ascending sorted order

James Fournie in https://bugs.launchpad.net/evergreen/+bug/635121 reported
that call numbers were being displayed in the order that they had been
added to the database, rather than in sorted call number label order.

Although I have been unable to reproduce this problem on two different
test systems, the patch he provided for 1.6.1 (which fixes the problem on their
test system) shouldn't hurt other systems.

git-svn-id: svn://svn.open-ils.org/ILS/branches/rel_1_6@18521 dcc99617-32d9-48b4-a31d-7c20da2025e4

adjust user tests that get done when retargeting holds

Fixes bug where a user having the maximum number of
active hold requests allowed to them by policy would
prevent items from being targeted to fill their requests.

Backport of rel_2_0 commits r18189, r18197, r18201, and
r18229 by Mike Rylander and Galen Charlton.

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
git-svn-id: svn://svn.open-ils.org/ILS/branches/rel_1_6@18507 dcc99617-32d9-48b4-a31d-7c20da2025e4

This column does not exist in 1.6.1, no good trying to DROP it.

git-svn-id: svn://svn.open-ils.org/ILS/branches/rel_1_6@18497 dcc99617-32d9-48b4-a31d-7c20da2025e4

Prevent ingest errors when asset.uri ID value outstrips asset.call_number

We were basing IDs in asset.uri on the max ID of asset.call_number,
which occasionally led to major ingest problems as attempts to insert
the next ID into asset.uri failed because that ID already existed. Using
the max ID from asset.uri for asset.uri inserts seems to make more sense
and does resolve that problem.

The bigger problem of not using the sequences that are already on these
tables to generate the IDs will not be an issue in 2.0, when we move to
in-database ingest and can use the normal lastval() approach to populate
asset.uri_call_number_map with the new values.

git-svn-id: svn://svn.open-ils.org/ILS/branches/rel_1_6@18482 dcc99617-32d9-48b4-a31d-7c20da2025e4

include more information in the debugging dump; correct the "back to index" link

git-svn-id: svn://svn.open-ils.org/ILS/branches/rel_1_6@18471 dcc99617-32d9-48b4-a31d-7c20da2025e4