dbs [Thu, 12 Aug 2010 18:22:12 +0000 (18:22 +0000)]
Show the "Active?" checkbox as part of the required (minimal) set of fields
As we're registering users, it's probably a good idea to ensure that staff
can set the user to active at the same time that they enter the minimal
patron information.
dbs [Thu, 12 Aug 2010 17:50:56 +0000 (17:50 +0000)]
Enable "Delete address" button to work in rel_1_6
In some ways the inverse of r17023, the else clause that enabled
the delete button to be enabled was never being reached because
the preceding clauses caught all cases.
Moving it outside of the foreach() loop entirely seems to resolve the problem.
dbs [Thu, 12 Aug 2010 07:46:01 +0000 (07:46 +0000)]
Remove broken old JavaScript in Google Book preview code
I /think/ this was supposed to change the title of the "Preview" link to
"Read this online" or the like in the case that the full text is available,
but it was relying on a function that didn't exist (setText) and trying
to reference an ID that didn't exists. So it broke in those rare cases
where Google Books did provide the full text.
gmc [Tue, 10 Aug 2010 21:22:44 +0000 (21:22 +0000)]
bug 592777: allow authoritative version of open-ils.circ.retrieve
Part of a fix to avoid race condition that can occur
when patron renews an item in the OPAC in a database
that uses pgpool and replication, which sometimes
results in an erroneous 'action_circulation_not_found' error.
dbs [Mon, 9 Aug 2010 13:15:56 +0000 (13:15 +0000)]
Patch from Ben Ostrowsky <ben@esilibrary.com> to specify "staff account" in proxied pages
This change will make it a bit more clear that we're not looking for a
patron barcode/PIN in the Selfcheck Login screen (and make the wording
consistent across similar login pages).
gmc [Fri, 6 Aug 2010 20:35:57 +0000 (20:35 +0000)]
bug 532217: work around caching issue resulting doubled title display
Quick hack shamelessly borrowed from Dan Scott to fix problem
of title being displayed twice on bib details page when back
button is used in OPAC or staff client.
This is a temporary fix in lieu of rewriting the bib details
display to use BibTemplate exclusively.
gmc [Fri, 6 Aug 2010 01:22:56 +0000 (01:22 +0000)]
bug 614150: bail out on ACTOR_USER_NOT_FOUND
This fixes a bug where uploading an offline checkout
that refers to a missing patron returns an INTERNAL_SERVER_ERROR
instead of ACTOR_USER_NOT_FOUND. More generally, this avoids
an exception in case case where a circ operation is made
without checking the existence of the patron record beforehand.
dbs [Fri, 18 Jun 2010 04:48:03 +0000 (04:48 +0000)]
Backport security fix r16747 from trunk
1. Disable fleshing for PCRUD. Otherwise fleshing would provide a
back door whereby a user could see stuff he has no permission to see.
2. For the id_list method: strip out the "flesh_fields" entry, not
the "flesh_columns" entry (which doesn't exist). This actually makes
no difference, but if we're going to do something useless, we might
as well do it right.
erickson [Mon, 14 Jun 2010 15:43:05 +0000 (15:43 +0000)]
removed unused method retrieved from method_lookup. apart from being unused, the call was attempting to fetch a nonexistent method ('auth' vs. 'authority') and hilarity ensued
erickson [Fri, 11 Jun 2010 13:49:45 +0000 (13:49 +0000)]
added a number of info messages to the action/trigger runner and server code. the messages provide summary data about what event defs / hooks are being processed and when/if they complete or timeout.
miker [Thu, 10 Jun 2010 19:16:27 +0000 (19:16 +0000)]
Patch from James Fournie of SITKA:
There was some discussion about problems with holds fulfillment at the
holds roundtable at EG2010. I am pleased to share this patch with the
community which has been thoroughly tested by the folks at
Thompson-Nicola Regional District Library. (thanks guys!)
Background:
Evergreen's default out-of-the-box behaviour for holds fulfillment is
a gas-saving method. Holds are fulfilled by proximity. In a
multibranch library, holds are fulfilled at the local branch first.
Many libraries, particularly single branch libraries may be ok with
this, but it may be problematic for other libraries.
Imagine a scenario where you have a large central branch and a small
rural branch of the same library system. At the large branch, there
are many copies of Popular New DVD with lots of holds. There are no
copies at the rural branch. Patrons at the small rural branch who
want to pick up Popular New DVD at their home branch may never get
their hold fulfilled because the copies will stay at the large branch
as long as there are holds for pickup there.
This patch adds an org unit setting that changes the opportunistic
check-in so that items checked in will be assigned to holds by request
date first, rather than proximity. This setting can be applied to
any level of the org tree, so in some situations you may even want to
activate FIFO for large libraries, but leave the original setting for
smaller libraries with less traffic who want to keep their copies more
local.
Also credit to Jeff Godin who thought of the same patch and
contributed the setting name "holds FIFO" for the setting
[ NOTE: Implications of mixed FIFO and non-FIFO environments that are
not sufficiently segregated by the use of Hard Boundaries for Holds
present a potenial for user confusion. Beware that mixing FIFO and
non-FIFO settings within a resource-sharing group will likely result in
severe imbalance of hold fulfillment, though further configuration,
development, tuning and testing may be able to mitigate these issues.
--miker ]
erickson [Thu, 10 Jun 2010 18:49:14 +0000 (18:49 +0000)]
updated report param editor to handle join types embedded in the field name. this bug caused sporadic failed rendering of the report editor params widgets
erickson [Tue, 8 Jun 2010 19:14:27 +0000 (19:14 +0000)]
back-porting: protect against empty results from bib searches caused by search timeouts. This allows the API call to log the error and return reasonable results
erickson [Mon, 3 May 2010 15:37:24 +0000 (15:37 +0000)]
backporting 16376: fixed bug where updating the email address resulted in updating the username instead of the email address on the local copy of the user object in the opac. this bug likely affected nothing.
Backporting r16204: Patch from Galen Charlton. This patch adds additional calls to escape_xml to handle cases where patron or library data could contain ampersand or other characters that need to be converted to entities. Issue discovered by Bibliomation; patch includes contributions by Ben Ostrowsky.
miker [Sat, 27 Mar 2010 17:35:48 +0000 (17:35 +0000)]
Patch from Dan Wells which allows restriction of renewal when the item in question is needed to fulfill a hold.
There was concern initially about whether a patrons own holds should be ignored, but that is not the case in scripted circ rules, so the behavior, as implemented by Dan, is correct.
phasefx [Fri, 26 Mar 2010 16:02:24 +0000 (16:02 +0000)]
textbox support for oils_persist (to fix stickiness in the label interface). trunk already has this but is so drastically different that I'm afraid to backport
phasefx [Tue, 16 Mar 2010 19:56:27 +0000 (19:56 +0000)]
make batch renewal use synchronous calls again to better handle exceptions. The original push for synchronous calls here was for performance, so we may need to revisit
miker [Tue, 16 Mar 2010 14:17:34 +0000 (14:17 +0000)]
Patch from Galen Charlton: removes empty controlfield elements, which can be produced in a couple different ways during the ingest of slightly invalid MARC records
phasefx [Mon, 15 Mar 2010 09:52:49 +0000 (09:52 +0000)]
Fix some broken survey handling. The patron editor (legacy, haven't checked newer one yet) was only saving the response for the first question in any given survey, and the Info/Other -> Survey display screen was likewise
mishandling survey responses and giving an ugly error for multi-question surveys.
miker [Wed, 10 Mar 2010 19:22:00 +0000 (19:22 +0000)]
Reworking of a patch from Ben Ostrowsky: add an OILSProxyLoginOU Directory config option for apache to supply a pinned workstation OU for permission checking
dbs [Tue, 9 Mar 2010 11:49:02 +0000 (11:49 +0000)]
Backport r15661 from trunk: Apply patch from Galen Charlton to prevent legacy 852 fields from being exported during bib+holdings export
From https://bugs.launchpad.net/evergreen/+bug/528424