dbs [Fri, 13 Aug 2010 19:11:38 +0000 (19:11 +0000)]
Grant permissions related to booking/reservations to circulators and admins
This should make for a better out-of-the-box experience for Evergreen adopters.
The permissions in the upgrade script are outside of the transaction to avoid
rolling back the entire upgrade if a site has already assigned these
permissions.
TODO: Document the new permissions as part of the 1.6.1 release notes so
that sites can alter the upgrade script according to their desires if they
don't like the defaults.
dbs [Thu, 12 Aug 2010 18:21:55 +0000 (18:21 +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:32 +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:45:11 +0000 (07:45 +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:22 +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:26 +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:38:07 +0000 (20:38 +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:39 +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.
Get the 1.6.0.4-1.6.0.5 updates into the rel_1_6/rel_1_6_0 stream
This may end up in someone who has followed the entire 1.6.0.x
upgrade train applying these two updates twice, but it shouldn't
hurt - and ensures that someone who jumped to 1.6.1.0 or 1.6.1.1
gets these important updates.
r15562 contained a fix for this to rel_1_6_0, but apparently I never
forward-ported that "fix". Unfortunately, this leads to broken interfaces in
the i18n build (bug re-found by Robert Souilliere in the 1.6.1.1 release).
senator [Mon, 28 Jun 2010 21:35:59 +0000 (21:35 +0000)]
Patch from Don McMorris. Good catch, Don.
He writes:
In the branch rel_1_6 version of 950.data.seed-values.sql [1], lines
138-142 create the OU setting
'circ.booking_reservation.default_elbow_room' via the query:
INSERT INTO actor.org_unit_setting (org_unit, name, value) VALUES (
(SELECT id FROM actor.org_unit WHERE parent_ou IS NULL),
'circ.booking_reservation.default_elbow_room',
'"1 day"'
);
However, it appears that actor.org_unit does not start to get
populated until line 853. As such, the "SELECT id FROM actor.org_unit
WHERE parent_ou IS NULL" portion of the insert will return NULL, and
actor.org_unit_setting.org_unit has a 'NOT NULL' contraint - as such,
the insert fails.
erickson [Mon, 14 Jun 2010 16:14:23 +0000 (16:14 +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:50:24 +0000 (13:50 +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.
dbs [Fri, 11 Jun 2010 03:32:22 +0000 (03:32 +0000)]
Merge r16605 and r16607 from trunk for Dojo i18n enhancements:
* Create output directories if they do no exist (r16605)
* Identify problem file when generating POT from Dojo resource files (r16607)
miker [Thu, 10 Jun 2010 19:15:55 +0000 (19:15 +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:48:37 +0000 (18:48 +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 16:15:49 +0000 (16:15 +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
senator [Mon, 7 Jun 2010 15:17:41 +0000 (15:17 +0000)]
Booking: make check-in op-capture behave more reasonably when elbow_room is
zero, and give the OU setting for elbow_room a default value of 1 day since
that's probably a more reasonable value than zero anyway.
Also add a smidge of documentation explaining what elbow room is for.
miker [Fri, 4 Jun 2010 14:41:42 +0000 (14:41 +0000)]
protect the main transaction from existing permissions; individually insert permissions to avoid missing some; do not pin permission IDs, just push them into the protected range
phasefx [Thu, 3 Jun 2010 15:08:50 +0000 (15:08 +0000)]
backport changesets 16564, 16566, 16567: fix logic error that could result in partial list of available libs for workstation registration and fix bug where attempting to use the home lib as a menu default when the home lib is not among the working location and descendeants
erickson [Tue, 1 Jun 2010 20:40:25 +0000 (20:40 +0000)]
backporting 16544: in the SIP server plugin, don't connect and start a transaction by default with every cstore handle. only connect/begin when necessary. also, making use of new CStoreEditor::init() call for setting up the editor environment when initial load was done before the IDL was available. Also, back-porting Joe A's CStoreEditor cleanup changes, namely the new init() routine.
erickson [Fri, 21 May 2010 20:03:20 +0000 (20:03 +0000)]
back-porting a number of local dojo library fixes and enhancements. of note, autogrid caching and paging for faster paged grid display. (turning on the pager in some config interfaces to follow).
erickson [Fri, 7 May 2010 18:54:14 +0000 (18:54 +0000)]
patch from James Fournie to repair some faulty assumptions in the previous legacy script SIP fix; additionally, I added legacy_script_support override option to oils_sip.xml.exmple to allow use of in-db circ-mod configs even if the circ code is still using circ scritps
erickson [Mon, 3 May 2010 15:36:46 +0000 (15:36 +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.
Bring the password reset interface to the Craftsman skin
Along the way, we make the placement slightly less hard-coded so that we
can better support custom skins that might not match default - although
we're still assuming that the login box will be a table with a tbody, you
can use the config.ids.login.tbody variable to point at the ID you care
about for your skin.
Clean up after ourselves by clearing the values of the barcode / username
text fields if the user hits Cancel or submits a request that was handled
by the backend service.
senator [Wed, 14 Apr 2010 18:54:15 +0000 (18:54 +0000)]
Booking workflow fix for reservation op-capture
Only capture items for reservation upon checkin if the reservation in question
is starting within the interval defined by the elbow room value. Elbow room
value can be controller per booking resource type, or if none is set, an
org unit setting defines the elbow room interval. This allows new
circulations to happen more easily between the time a reservation is made and
the time a reservation actually starts.
Backport r16211 from trunk: Turn "next page" / "previous page" symbols into entities
Makes it easier for sites to customize this aspect of the look and feel of their skins if they support multiple languages.