working/Evergreen.git
8 years agoFix for LP894125 - reshelving interval collab/dyrcona/lp_894125_fix_reshelving
Ben Shum [Wed, 23 Nov 2011 20:14:32 +0000 (15:14 -0500)]
Fix for LP894125 - reshelving interval

Update to reshelving code to utilize actor.org_unit_ancestor_setting function
and properly retrieve org settings for "circ.reshelving_complete.interval" for
org unit or ancestors.

Signed-off-by: Ben Shum <bshum@biblio.org>
Signed-off-by: Jason Stephenson <jstephenson@mvlc.org>
8 years agoEnable record summary to be scrollable at larger font sizes
Dan Scott [Tue, 22 Nov 2011 18:33:07 +0000 (13:33 -0500)]
Enable record summary to be scrollable at larger font sizes

The record summary at the top of the cataloguing interfaces appears to
determine its size based on the "normal" font size at load; then, when a
larger font size is applied to the page, the text gets hidden by the
fixed size of the groupbox.  Ideally the groupbox would adjust to
accomodate the larger font size, but after trying many different
approaches the best short-term approach seems to be to enable overflow
so that scrollbars can appear when necessary.

Signed-off-by: Dan Scott <dscott@laurentian.ca>
Signed-off-by: Jason Etheridge <jason@esilibrary.com>
8 years agoLimit display to 856 with ind1 = 4 and ind2 = 0 or 1.
Jason Stephenson [Mon, 21 Nov 2011 18:47:30 +0000 (13:47 -0500)]
Limit display to 856 with ind1 = 4 and ind2 = 0 or 1.

Minor modification to Open-ILS/src/templates/opac/parts/
misc_util.tt2 that limits the display of 856 urls to that have
indicator 1 equal to 4 and indicator 2 equal to 0 or 1. These should
be those that point directly to the resource.

Signed-off-by: Jason Stephenson <jstephenson@mvlc.org>
Signed-off-by: Dan Scott <dscott@laurentian.ca>
8 years agoTPac: show non-asset.uri 856 URLs on detail page
Bill Erickson [Mon, 3 Oct 2011 20:31:45 +0000 (16:31 -0400)]
TPac: show non-asset.uri 856 URLs on detail page

Display "global" 856 URLs in record detail page.  A "global" URL in this
case is one that has no ownership and has not been extracted out as an
asset.uri.  As of right now, this is any 856 that has no $9, $w, or $n
subfield data.

Links are displayed directly after asset.uri links using the same style.

Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Dan Scott <dscott@laurentian.ca>
8 years agoAvoid redundant install_cpan_force dependencies
Dan Scott [Mon, 21 Nov 2011 17:41:03 +0000 (12:41 -0500)]
Avoid redundant install_cpan_force dependencies

The "install" target and each of the generic distro targets in
Makefile.install included install_cpan_force; make the "install" target
more useful by factoring out the install_libdbi dependency so that
"install" is truly common again (and we only add the install_cpan_force
dependency once).

Signed-off-by: Dan Scott <dscott@laurentian.ca>
8 years agoAdd install_yaz dependency to debian and ubuntu Makefile.install
Dan Scott [Mon, 21 Nov 2011 17:38:56 +0000 (12:38 -0500)]
Add install_yaz dependency to debian and ubuntu Makefile.install

To build SimpleServer 1.15 we need a version of yaz 4.2+, so add
install_yaz as a dependency for debian and ubuntu. This should have been
added in LP 872284 but was overlooked due to local testing and didn't
get any further testing before being committed.

Signed-off-by: Dan Scott <dscott@laurentian.ca>
8 years agobump xulrunner version
Jason Etheridge [Fri, 18 Nov 2011 16:15:04 +0000 (11:15 -0500)]
bump xulrunner version

Signed-off-by: Jason Etheridge <jason@esilibrary.com>
8 years agoSwitch to new Google Books API; make SSL friendly
Dan Wells [Thu, 17 Nov 2011 21:13:02 +0000 (16:13 -0500)]
Switch to new Google Books API; make SSL friendly

As implied in the title, this commit does two things.  First, it
switches to the new Google Books API (which is both imminent and
also necessary to make SSL calls work).  Though the information is
scant, from what I have read and experienced, we do not need an
API key to do searches and previews.  I have also not hit any kind
of unauthenticated limit in several days of heavy testing, so I
would figure we are safe (at this point) for normal end-user OPAC
browsing.

Second, all Google Book requests are now done over https.  This
eliminates the majority of mixed content warnings when browsing
securely, though you still get a warning when you actually do
preview a book.

In addition to possibly implementing protocol detection (rather
than doing https all the time as a "lowest" common denominator),
there are a few minor points where we might consider future changes.
Those points are commented within the code.

Signed-off-by: Dan Wells <dbw2@calvin.edu>
Signed-off-by: Dan Scott <dscott@laurentian.ca>
8 years agoStamp upgrade scripts for both unapi update and vandelay-to-bucket
Mike Rylander [Thu, 17 Nov 2011 19:46:41 +0000 (14:46 -0500)]
Stamp upgrade scripts for both unapi update and vandelay-to-bucket

Signed-off-by: Mike Rylander <mrylander@gmail.com>
8 years agoDefault to org tree root for bre unapi funcs
Bill Erickson [Tue, 8 Nov 2011 20:13:53 +0000 (15:13 -0500)]
Default to org tree root for bre unapi funcs

If the org unit ("site") parameter to unapi.biblio_record_entry_feed()
or unapi.bre() is equal to "-" or NULL, default to the root of the org
unit tree.  In cases where the caller does not care to limit by site,
determing the shortname of the root org node is one less step the client
now has to take.

Original patch from Mike Rylander.

Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
8 years agoVandelay; copy imported bib queue recs to bucket
Bill Erickson [Mon, 31 Oct 2011 21:33:23 +0000 (17:33 -0400)]
Vandelay; copy imported bib queue recs to bucket

Adds a "Copy to Bucket" option within the Vandelay queue interface.
User is given the option to name the bucket.  If the bucket already
exists, the records are added to the existing bucket.  Otherwise, a new
bucket is created and the records are added.

Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
8 years agoVandelay; API for creating buckets from queues
Bill Erickson [Mon, 31 Oct 2011 18:56:48 +0000 (14:56 -0400)]
Vandelay; API for creating buckets from queues

* Adds a new biblio record entry bucket type of "vandelay_queue"
* Adds a new API call that allows callers to create new buckets (or add
to existing buckets), copying imported records from a vandelay queue
into the bucket.

open-ils.vandelay.bib_queue.to_bucket

Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
8 years agoRepair errant explode_array in upgrade script.
Bill Erickson [Wed, 16 Nov 2011 16:50:57 +0000 (11:50 -0500)]
Repair errant explode_array in upgrade script.

The upgrade script 0562.schema.copy_active_date.sql included 2 calls to
explode_array after explode_array had been removed from the baseline
schema.  This upgrade just copies the contents of the
action.hold_request_permit_test function from the baseline schema to
repair the function for anyone following the upgrade scripts.

Signed-off-by: Bill Erickson <berick@esilibrary.com>
8 years agoStamping upgrade script for copy visibility vs peer-bibs fix
Mike Rylander [Wed, 16 Nov 2011 17:00:48 +0000 (12:00 -0500)]
Stamping upgrade script for copy visibility vs peer-bibs fix

Signed-off-by: Mike Rylander <mrylander@gmail.com>
8 years agoDistinct cache_copy_visibility copy/record
Thomas Berezansky [Mon, 24 Oct 2011 14:36:39 +0000 (10:36 -0400)]
Distinct cache_copy_visibility copy/record

Because if you peer bib a copy to the bib it is on things don't like you.

Signed-off-by: Thomas Berezansky <tsbere@mvlc.org>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
8 years agoObey renewal rules for SIP2 Checkout
Thomas Berezansky [Tue, 18 Oct 2011 19:10:07 +0000 (15:10 -0400)]
Obey renewal rules for SIP2 Checkout

If local policy OR the selfcheck are set to not do renewals than don't
attempt them.

Also, return whether a renewal was attempted (success or failure) so that
the self check on the other end has a better clue what went on.

Signed-off-by: Thomas Berezansky <tsbere@mvlc.org>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
8 years agoImprovement for telephony: just-in-time event revalidation
Lebbeous Fogle-Weekley [Mon, 26 Sep 2011 16:06:07 +0000 (12:06 -0400)]
Improvement for telephony: just-in-time event revalidation

One of the shortcomings with using the Action/Trigger based telephony in
Evergreen until now was that while you might have overdue notices
generated and sent to a system where Asterisk runs for later calling,
but if the notice was generated on a Saturday night, and you have Asterisk
set up not to place any calls again until Monday morning, Asterisk has
no way of revalidating that call at the last minute.  That is, the
system could not determine whether the items that were overdue on
Saturday night are still overdue on Monday morning, and whether the call
should still be made.

Now we have a workable solution to that.

The eg-pbx-allocator.pl script, which takes call files for Asterisk from
a "staging" directory and slowly drips them onto Asterisk's spool can
now consult an open-ils.justintime which in turn asks open-ils.trigger
whether given events, enumerated within the call files themselves, are
still valid.

open-ils.trigger is designed to run as a private service, so that's why
we need a public service that doesn't do anything too sensitive.

This open-ils.justintime service can potentially be extended to offer other
just-in-time information to the allocator right before a call goes onto
Asterisk's spool. For example, that might be a good time to check the time
of day and make a late decision on which phone number to use for a given
user (day_phone, evening_phone, other_phone).

Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
8 years agoStamped upgrade for full circ count view fix
Mike Rylander [Tue, 15 Nov 2011 21:22:08 +0000 (16:22 -0500)]
Stamped upgrade for full circ count view fix

Signed-off-by: Mike Rylander <mrylander@gmail.com>
8 years agoUse DISTINCT in counts to avoid dupes
Thomas Berezansky [Thu, 29 Sep 2011 14:44:26 +0000 (10:44 -0400)]
Use DISTINCT in counts to avoid dupes

Otherwise you get:
Legacy count + 2(normal count)(aged count)

Signed-off-by: Thomas Berezansky <tsbere@mvlc.org>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
8 years agoFull Circ Count View Amplifies Legacy Circs
Dan Wells [Thu, 29 Sep 2011 13:23:49 +0000 (09:23 -0400)]
Full Circ Count View Amplifies Legacy Circs

The current extend_reported.full_circ_count sums the legacy circ count
column, but this results in amplifying the count by the number of current
circs in the system.  This commit adds the legacy count to the GROUP BY
instead of summing it.

Signed-off-by: Dan Wells <dbw2@calvin.edu>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
8 years agoSkip shelved holds at checkout auto-fill
Thomas Berezansky [Tue, 15 Nov 2011 16:08:00 +0000 (11:08 -0500)]
Skip shelved holds at checkout auto-fill

Already happened for volume/title hold, but target map checking wasn't
checking to see if the current copy was shelved.

Signed-off-by: Thomas Berezansky <tsbere@mvlc.org>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
8 years agoNet::Z3950::SimpleServer 1.15 should not require sedding
Dan Scott [Sun, 16 Oct 2011 19:48:38 +0000 (15:48 -0400)]
Net::Z3950::SimpleServer 1.15 should not require sedding

Checking both versions of the packages listed on ftp.indexdata.dk, the
hardcoded path to yaz-config does not exist in Makefile.PL - thus, we
should be able to drop the "sed" instruction for 1.15.

Signed-off-by: Dan Scott <dscott@laurentian.ca>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
8 years agoInstall yaz-4.2 and Net::Z3950::Simpleserver 1.15
Dan Scott [Sun, 16 Oct 2011 15:40:59 +0000 (11:40 -0400)]
Install yaz-4.2 and Net::Z3950::Simpleserver 1.15

Net::Z3950::Simpleserver 1.14+ requires yaz 4.2.x, as well as currently
suffering from a Makefile.PL problem that involves a hard-coded path to
yaz-config (https://rt.cpan.org/Public/Bug/Display.html?id=70444).
Provide a sed command to resolve the latter problem and upgrade our
"current version of yaz" to yaz 4.2.x.

Signed-off-by: Dan Scott <dscott@laurentian.ca>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
8 years agoFix a problem in the clear holds shelf list printer
Lebbeous Fogle-Weekley [Wed, 28 Sep 2011 20:50:03 +0000 (16:50 -0400)]
Fix a problem in the clear holds shelf list printer

This is a "special" interface that you get to via Admin -> For
Developers -> Local Administration in the staff client.

The addition features to one "side" of this hackish interface of label affix
and parts had caused breakage on the other "side."  This fixes that.

Also improve document printing, making it less likely the progress
dialog itself gets printed.

One day this will be replaced with something better.

Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
8 years agoStamped upgrade for asset.merge_record_assets fix
Mike Rylander [Tue, 15 Nov 2011 20:45:02 +0000 (15:45 -0500)]
Stamped upgrade for asset.merge_record_assets fix

Signed-off-by: Mike Rylander <mrylander@gmail.com>
8 years agoLP#849143 fix some gaps in bib record merging
Jason Etheridge [Tue, 13 Sep 2011 16:22:58 +0000 (12:22 -0400)]
LP#849143 fix some gaps in bib record merging

Tweaks asset.merge_record_assets such that it considers acquisition line items and user requests, serial subscriptions, booking resource types, biblio monograph parts, and biblio peer bib copy maps. I'm not handling vandelay.queued_record.imported_as or any other vandelay tables, but we may wish to do that too. We're also leaving acq.acq_lineitem_history alone.

Signed-off-by: Jason Etheridge <jason@esilibrary.com>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
8 years agoAligned the MARC editor subfields to 'start' fixes long subfield display
Liam Whalen [Tue, 1 Nov 2011 19:59:13 +0000 (15:59 -0400)]
Aligned the MARC editor subfields to 'start' fixes long subfield display

When viewing a record in the MARC editor that has extremely long
entries in a subfield, the output would align in the center of the
subfield making the output hard to read.  This patch adds an
align: 'start' option to both the subfield, and the subfield code
elements in the marcSubfield function to make the subfields align in a
 more readable manner.
Signed-off-by: Liam Whalen <lwhalen@evergreen-dev.catalogue.nrcan.gc.ca>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
8 years agoMerge remote-tracking branch 'eg-working/collab/phasefx/merged_bill_and_receipt_fixes'
Mike Rylander [Tue, 15 Nov 2011 20:36:26 +0000 (15:36 -0500)]
Merge remote-tracking branch 'eg-working/collab/phasefx/merged_bill_and_receipt_fixes'

8 years agoAcq: (un)receive w/o assets should not attempt a copy update
Bill Erickson [Fri, 23 Sep 2011 21:45:39 +0000 (17:45 -0400)]
Acq: (un)receive w/o assets should not attempt a copy update

When receiveing a lineitem for a PO that was activated without assets,
avoid any attempts to update said non-existent assets.

Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
8 years agoLP#848945 - Hide Empty Libs checkbox
Jason Etheridge [Tue, 13 Sep 2011 12:48:52 +0000 (08:48 -0400)]
LP#848945 - Hide Empty Libs checkbox

The "Show Libraries With Items" button in Holdings Maintenance shows all the
libs with items regardless of the library/depth filters. One problem is that
when the list refreshes (whenever an item/volume is edited, deleted, or added),
staff using this feature have to re-press the button, which can take a while for
some bibs. This branch removes this button and replaces it with a "sticky"
checkbox labelled "Hide Empty Libs".

This allows you to do almost the same thing; you can set the context library to
Consortium and the depth filter to the lowest depth, which would normally show
all libs, but then use the checkbox to prune out the ones that have no children
and no volumes(*). One possible use case that we'd lose is the ability to show a
specific set of libs that don't already have volumes in addition to all the libs
that do.

* - Also, the test for whether to hide an org or not is very simplistic, but
easy to implement. A more aggressive test would hide a library and all its
descendants if none of the descendants have volumes.

Signed-off-by: Jason Etheridge <jason@esilibrary.com>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
8 years agoPatch from Jeff Davis (SITKA) fixing cardless patron updates
Mike Rylander [Tue, 15 Nov 2011 20:04:08 +0000 (15:04 -0500)]
Patch from Jeff Davis (SITKA) fixing cardless patron updates

Quoth the bug report (https://bugs.launchpad.net/evergreen/+bug/851110):

  On our Evergreen 2.0.8 system, it sometimes arises that a patron's card gets deleted. If it was the patron's only card, it becomes impossible to add a new card to the account via the staff client.

  How to reproduce:
  1. Retrieve a patron record for a patron who has no associated card.
  2. Click the Edit button.
  3. Click Replace Barcode and enter a new barcode.
  4. Click Save. This does not work - the process hangs, and if you hit Reload, you will find that your changes were not saved.

  I believe this happens because some code in register.js assumes the patron has at least one card. I have a potential fix that this comment box is too small to contain ... uh, that is to say, I will post it momentarily.

Which he did, and I applied.

Signed-off-by: Mike Rylander <mrylander@gmail.com>
8 years agoThrow out non-digit barcodes for strict check
Thomas Berezansky [Thu, 22 Sep 2011 13:24:56 +0000 (09:24 -0400)]
Throw out non-digit barcodes for strict check

Characters like "." are valid for numbers, but not the algorithm.

Thus, we need to throw out "16.00" despite the check for
"16.00" == Number("16.00")
returning true.

Signed-off-by: Thomas Berezansky <tsbere@mvlc.org>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
8 years agoDon't check barcode without strict enabled
Thomas Berezansky [Thu, 22 Sep 2011 13:16:31 +0000 (09:16 -0400)]
Don't check barcode without strict enabled

Item status was, unlike other interfaces, doing the strict check, THEN
looking at the checkbox and if it was checked return.

This caused issues when the barcode crashes out the strict checking.

Signed-off-by: Thomas Berezansky <tsbere@mvlc.org>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
8 years agoFix JSPac Hold Editor Org Selector
Thomas Berezansky [Fri, 7 Oct 2011 14:03:34 +0000 (10:03 -0400)]
Fix JSPac Hold Editor Org Selector

Don't wipe out all changes just because the OU was changed.

Signed-off-by: Thomas Berezansky <tsbere@mvlc.org>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
8 years agoStamped upgrade script for hold_notification update via A/T cleanup module
Mike Rylander [Tue, 15 Nov 2011 17:18:53 +0000 (12:18 -0500)]
Stamped upgrade script for hold_notification update via A/T cleanup module

Signed-off-by: Mike Rylander <mrylander@gmail.com>
8 years agoHold notification creation A/T cleanup module
Bill Erickson [Thu, 9 Jun 2011 19:21:39 +0000 (15:21 -0400)]
Hold notification creation A/T cleanup module

New CreateHoldNotification Action/Trigger cleanup module that, when
linked to a hold-based event_definition, adds an entry into
action.hold_notification for each hold processed.  The main user-facing
change from this will be that the hold notification time / count in the
staff client will now take A/T notices into effect in addition to
manually created hold notifications.

Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
8 years agoFurther support for "circ.collections.exempt" setting
Bill Erickson [Tue, 18 Oct 2011 20:22:25 +0000 (16:22 -0400)]
Further support for "circ.collections.exempt" setting

Added support for skipping users that have the
"circ.collections.exempt" setting in the API call:

open-ils.collections.users_of_interest.warning_penalty.retrieve

This call is an alternate entry point for finding users to consider
for collections processing

Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
8 years agoReshelving update.
Steven Callender [Wed, 19 Oct 2011 18:51:14 +0000 (18:51 +0000)]
Reshelving update.

Update to the reshelving code to make it faster and utilize the new status change time field on asset.copy.

Signed-off-by: Steven Callender <stevecallender@esilibrary.com>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
8 years agoTPac: Fix glitch in displaying received holdings
Lebbeous Fogle-Weekley [Mon, 14 Nov 2011 18:18:30 +0000 (13:18 -0500)]
TPac: Fix glitch in displaying received holdings

Under "issues held" on the record detail page, holdings summaries for
received issues are nested in an additional layer of array as compared
to MFHD-based holdings, so compensate.

Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>
Signed-off-by: Bill Erickson <berick@esilibrary.com>
8 years agoClear shelf canceled holds notification fix
Bill Erickson [Mon, 24 Oct 2011 20:27:29 +0000 (16:27 -0400)]
Clear shelf canceled holds notification fix

After cancelling a batch of holds via the clear-shelf process, before we
send the holds off to action/trigger for potential notification
generation, re-fetch the holds from the DB to pick up the calculated
cancel_time value (i.e. get a real value for "now").  Otherwise, A/T may
die attempting to create a DateTime object by parsing "now".

Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
8 years agoTPac; hold success confirmation; redirect repairs
Bill Erickson [Wed, 2 Nov 2011 15:28:57 +0000 (11:28 -0400)]
TPac; hold success confirmation; redirect repairs

1. After a successful hold placement, provide feedback to the user that
the hold placement succeeded.  After placement, the user is taken to a
page (same page for failed holds) where the title(s) are listed along
with the success/failure information for each.

2. In some cases, the user may be redirected to the my account page
instead of the point of origin for holds placement.  This occurs
generally when the user is not already logged in and is asked to log in
prior to holds placement.

3. As a side effect, this change replaces one of the ornery
history.go(-1) actions with a true href/URL.

Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
8 years agoTPac; reduce vertical space caused by chilifresh link
Bill Erickson [Thu, 3 Nov 2011 14:18:12 +0000 (10:18 -0400)]
TPac; reduce vertical space caused by chilifresh link

CSS loaded from chilifresh forces a fixed height on the
.chili_review class, which causes significant vertical space gobbling in
the results and details pages.  Solution is to change the chili_review
container from a <div> to a <span>, where the height has no effect.

Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
8 years agoMake idlval build, rather than be dead code
Dan Scott [Sun, 23 Oct 2011 23:58:53 +0000 (16:58 -0700)]
Make idlval build, rather than be dead code

The tool "idlval" contained in Open-ILS/src/c-apps can be used to check
the correctness of fieldmapper definitions in files like
Open-ILS/examples/fm_IDL.xml - but currently does not have a build
target, nor does it compile cleanly due to the use of unspecified
OpenSRF header files. This commit adds a build target, fixes the compile
problems, and installs the tool in the bin directory.

Signed-off-by: Dan Scott <dscott@laurentian.ca>
Signed-off-by: Jason Stephenson <jstephenson@mvlc.org>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
8 years agoAdd link to cbs from bre via bre.source -> cbs.id in fm_IDL.xml.
Jason Stephenson [Tue, 8 Nov 2011 19:39:29 +0000 (14:39 -0500)]
Add link to cbs from bre via bre.source -> cbs.id in fm_IDL.xml.

Signed-off-by: Jason Stephenson <jstephenson@mvlc.org>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
8 years agoShow the patron account Create Date in both horizontal and vertical patron summary...
Jason Etheridge [Tue, 8 Nov 2011 20:01:43 +0000 (15:01 -0500)]
Show the patron account Create Date in both horizontal and vertical patron summary panes

Signed-off-by: Jason Etheridge <jason@esilibrary.com>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
8 years agoLP887822 mint_condition/Quality not set correctly
Jason Etheridge [Tue, 8 Nov 2011 23:42:18 +0000 (18:42 -0500)]
LP887822 mint_condition/Quality not set correctly

during item creation.

Specifically, the Javascript object for a new item (acp) does not explicitly set
the mint_condition field, leaving it null. The item attribute editor treats the
null as false (which it displays as "Mediocre" for that field). The database has
a not null constraint the corresponding column in asset.copy, and defaults to
true. So a new item in the attribute editor will show "Mediocre", but when
actually created, will become "Good".

This branch just sets mint_condition to true when an item is created.

Signed-off-by: Jason Etheridge <jason@esilibrary.com>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
8 years agoTpac; repair login link after password reset
Bill Erickson [Wed, 9 Nov 2011 15:13:28 +0000 (10:13 -0500)]
Tpac; repair login link after password reset

After a successful password reset, the UI offers a link to "Log In to My
Account".  Loging in this way would ultimately redirect the user back to
the password reset form.  This patch updates the Log In link to point to
myopac/main, which will direct the user to log in first,  instead of
pointing directly to /login, which, lacking any other information,
will return the user to the referer after logging in.

Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
8 years agoMerge remote-tracking branch 'eg-working/collab/berick/lp_888239_prevent_post-capture...
Mike Rylander [Tue, 15 Nov 2011 16:48:15 +0000 (11:48 -0500)]
Merge remote-tracking branch 'eg-working/collab/berick/lp_888239_prevent_post-capture_hold_suspend'

8 years agoTPac; propagate query params to adv search (results link)
Bill Erickson [Fri, 11 Nov 2011 18:22:34 +0000 (13:22 -0500)]
TPac; propagate query params to adv search (results link)

Update the "Advanced Search" link that shows on search results pages
(next to "Another Search") to propagate the current query into the
advanced search page, just like the Advanced Search link just above the
search bar.

Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
8 years agotpac; honor password reset matching email setting
Bill Erickson [Fri, 11 Nov 2011 20:25:49 +0000 (15:25 -0500)]
tpac; honor password reset matching email setting

If the "circ.password_reset_request_requires_matching_email" org unit
setting is enable, prompt the user to enter the email address associated
with the account and subsequently pass it on to the password_reset api
call.

Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
8 years agoSerials: in the holding code mini-wizard, offer a dropdown for seasons
Lebbeous Fogle-Weekley [Mon, 14 Nov 2011 20:31:14 +0000 (15:31 -0500)]
Serials: in the holding code mini-wizard, offer a dropdown for seasons

This spares the user from having to remember the MFHD code for the
season, when dealing with periodicals issued by season (21 - 24 for
Spring - Winter, respectively).

This also applies sets the "scrollOnFocus" attribute to false for the
dijits within the mini wizard, as the jumpy behavior you get in
less-than-fullscreen staff client windows is otherwise maddening.

Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
8 years agotpac; improve ssl and authentication redirection
Bill Erickson [Mon, 14 Nov 2011 21:27:05 +0000 (16:27 -0500)]
tpac; improve ssl and authentication redirection

Create a better separation between pages the require SSL and those
that also require authentication.  This change has a number of
beneficial side effects:

1. If a user is already logged in and is somehow redirected to the
non-SSL version of a page requiring SSL (via, for example, some external
link), they are simply taken to the SSL equivlent of the page first.  If
they are found to be logged in, we're done.  If not, they are directed to
the login page like usual.

2. password_reset page now requires SSL (but not auth, obviously)

3. Removed the unnecessary warning about logging out on a non-SSL
connection.  We now redirect the user to the SSL version of logout so,
if they are in fact logged in, they can be successfully logged out.

Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
8 years agoMake checkin retargeting parts-aware
Thomas Berezansky [Wed, 26 Oct 2011 13:45:54 +0000 (09:45 -0400)]
Make checkin retargeting parts-aware

Skip part holds on copies san parts
Skip part holds for parts not on the copy
Skip title holds for copies with parts

Signed-off-by: Thomas Berezansky <tsbere@mvlc.org>
Signed-off-by: Bill Erickson <berick@esilibrary.com>
8 years agoMore filtering of characters that could end up in PO JEDI template output
Lebbeous Fogle-Weekley [Tue, 11 Oct 2011 15:25:51 +0000 (11:25 -0400)]
More filtering of characters that could end up in PO JEDI template output

Backslashes cause problems too.  Let's JSON encode this thing and get it
right once and for all.

Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>
Signed-off-by: Ben Shum <bshum@biblio.org>
Signed-off-by: Bill Erickson <berick@esilibrary.com>
8 years agoSerials: allow freer input in holding code mini-wizard
Lebbeous Fogle-Weekley [Thu, 10 Nov 2011 22:22:22 +0000 (17:22 -0500)]
Serials: allow freer input in holding code mini-wizard

In the Subscription Details interface of the Alternate Serials Control
View, under the Issuance tab, if you create a new issuance you can
now enter things like "01/02" in (say) the month field to denote a combined
January/February issue.  All those fields are now free-form, whereas some of
them used to be numeric only.

Also, avoid an occasional issue where an error message about
dijit.form.DateTextBox not being a constructor would appear in the
JavaScript console.  The problem was in a race condition among some
co-dependent JavaScript files that were all being loaded at the same
time.

Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>
Signed-off-by: Dan Wells <dbw2@calvin.edu>
8 years agoLP88653 force SSL for requests from local XUL
Jason Etheridge [Thu, 10 Nov 2011 17:55:32 +0000 (12:55 -0500)]
LP88653 force SSL for requests from local XUL

Modifies RemoteRequest.js to convert any http:// request URL to https:// if the
requesting location has a chrome:// protocol.

Modifies util/network.js such that any use of util.network.request (as opposed
to util.network.simple_request) will assume "secure" for the resulting request.

Adds an example to the bottom of eg_vhost.conf for forcing SSL for an entire
site (OPAC and everything), courtesy of Dan Wells.

Signed-off-by: Jason Etheridge <jason@esilibrary.com>
Signed-off-by: Dan Wells <dbw2@calvin.edu>
8 years agoFedora 16 updates to README and Makefile.install
Dan Scott [Thu, 10 Nov 2011 06:04:50 +0000 (01:04 -0500)]
Fedora 16 updates to README and Makefile.install

Fedora 16 is out, we can improve the README and Makefile.install some to
better support it.

Signed-off-by: Dan Scott <dscott@laurentian.ca>
8 years agoTPac; disallow certain actions for captured holds collab/berick/lp_888239_prevent_post-capture_hold_suspend
Bill Erickson [Wed, 9 Nov 2011 20:55:50 +0000 (15:55 -0500)]
TPac; disallow certain actions for captured holds

In the hold edit page, where users can update the pickup library,
suspend holds, etc. disallow actions that don't make sense for holds
that have already been captured.  These include suspending the hold and
changing the expire time.

This patch also replaces the confusing "Cancel" button in the hold edit
interface (which does not cancel the hold) with a "Go Back" button which
returns to the holds list.

Signed-off-by: Bill Erickson <berick@esilibrary.com>
8 years agoTPac; batch hold update detects suspension of captured hold
Bill Erickson [Wed, 9 Nov 2011 20:54:21 +0000 (15:54 -0500)]
TPac; batch hold update detects suspension of captured hold

If the user attempts to suspend a captured hold via batch hold update,
warn that user af the update the some holds may not have been suspended
and why.

Signed-off-by: Bill Erickson <berick@esilibrary.com>
8 years agoPrevent hold suspencion for captured holds in API
Bill Erickson [Wed, 9 Nov 2011 20:51:40 +0000 (15:51 -0500)]
Prevent hold suspencion for captured holds in API

This codifies at the API layer the standard Evergreen practice of
disallowing suspension of holds that have already been captured.

Signed-off-by: Bill Erickson <berick@esilibrary.com>
8 years agoprevent suspend in transit initial poking
Bill Erickson [Wed, 9 Nov 2011 19:08:10 +0000 (14:08 -0500)]
prevent suspend in transit initial poking

Signed-off-by: Bill Erickson <berick@esilibrary.com>
8 years agoXML.ignoreWhitespace causes syntax errors
Thomas Berezansky [Wed, 9 Nov 2011 16:00:05 +0000 (11:00 -0500)]
XML.ignoreWhitespace causes syntax errors

So remove it.

Disabling pretty printing appears to be enough to fix the marc editor.

Signed-off-by: Thomas Berezansky <tsbere@mvlc.org>
Signed-off-by: Jason Etheridge <jason@esilibrary.com>
8 years agoFix a typo that snuck through the cracks in Cronscript.pm.in.
Jason Stephenson [Wed, 9 Nov 2011 14:37:44 +0000 (09:37 -0500)]
Fix a typo that snuck through the cracks in Cronscript.pm.in.

We had Opensrf::AppSession rather than OpenSRF::AppSession, which isn't
caught by the simple use_ok() tests that we largely rely on for our
current Perl test suite. In addition, when we manually tested the change
to Cronscript.pm.in, we didn't do both configure && make to ensure that
the new version of Cronscript.pm reflected the changes. Lots to learn,
luckily the method wasn't being used by anything else in Evergreen yet.

Signed-off-by: Jason Stephenson <jstephenson@mvlc.org>
Signed-off-by: Dan Scott <dscott@laurentian.ca>
8 years agoFix vanishing whitespace in marc editor
Thomas Berezansky [Mon, 7 Nov 2011 22:28:02 +0000 (17:28 -0500)]
Fix vanishing whitespace in marc editor

XML.ignoreWhitespace and XML.prettyPrinting cause whitespace to vanish.

So disable them at the top of marcedit.js.

Signed-off-by: Thomas Berezansky <tsbere@mvlc.org>
Signed-off-by: Jason Etheridge <jason@esilibrary.com>
8 years agolp849447: trival stylistic tweaks
Galen Charlton [Mon, 31 Oct 2011 15:46:48 +0000 (11:46 -0400)]
lp849447: trival stylistic tweaks

- rename variable avoid double-negative
- avoid 'Variable declared in conditional statement' perlcritic gripe

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
8 years agoProtect against slony replication lag, LP#621448
Mike Rylander [Tue, 8 Nov 2011 17:43:44 +0000 (12:43 -0500)]
Protect against slony replication lag, LP#621448

Make open-ils.circ.captured_holds.id_list.* API calls authoritative within
the staff client to avoid replication-lag related stale displays.  Inspired
by James Fournie at SITKA.

Signed-off-by: Mike Rylander <mrylander@gmail.com>
8 years agoREADME: Use consistent endings (none!) for titles/subtitles
Dan Scott [Mon, 7 Nov 2011 21:44:42 +0000 (16:44 -0500)]
README: Use consistent endings (none!) for titles/subtitles

Most, but not all, of the titles/subtitles had colons appended. This
looks really weird in the Table of Contents, so try to be consistently
unweird and have no appended colons.

Signed-off-by: Dan Scott <dscott@laurentian.ca>
8 years agoBackup the existing opensrf config files before copying
Dan Scott [Mon, 7 Nov 2011 21:38:32 +0000 (16:38 -0500)]
Backup the existing opensrf config files before copying

Per a suggestion from Wolf Halton <wolf.halton@gmail.com>, create backup
versions of the opensrf configuration files when we're copying the
Evergreen example versions over, so that we can retrieve the Jabber user
names and passwords and domains from the backups (if necessary).

Signed-off-by: Dan Scott <dscott@laurentian.ca>
8 years agoTPac; detail page "Search Results" page correction
Bill Erickson [Thu, 3 Nov 2011 19:51:41 +0000 (15:51 -0400)]
TPac; detail page "Search Results" page correction

On the detail page, when the user selects the "Search Results" link to
return the original search results, return to the correct page in the
search.  The correct page will be the page where the currently viewed
record sits within the search.  Normally, this is the last viewed
results page.  However, if the user does some paging within the detail
page, enough to cross a page boundry, "Search Results" will take them to
the new page (i.e. the page w/ the current record).

Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>
8 years agoTPac; sort items out list by due date, oldest first
Bill Erickson [Wed, 2 Nov 2011 19:45:06 +0000 (15:45 -0400)]
TPac; sort items out list by due date, oldest first

Sort items most in need of return (i.e. oldest due date) to top.  This
is done by replacing the call to "open-ils.actor.user.checked_out",
which does a lot more than we need, in a way that does not support
sorting/paging, with a simple json_query.

Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>
8 years agoClose All Tabs shortcut
Thomas Berezansky [Wed, 28 Sep 2011 23:00:56 +0000 (19:00 -0400)]
Close All Tabs shortcut

Enable CTRL+Click on the close tab "X" to close all tabs.

Signed-off-by: Thomas Berezansky <tsbere@mvlc.org>
Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>
8 years agoLP#860845 hold placement UI can break
Jason Etheridge [Tue, 27 Sep 2011 19:42:49 +0000 (15:42 -0400)]
LP#860845 hold placement UI can break

hold placement UI can break and give misleading error on Submit

Adds some defensive coding, but since I don't quite grok everything going on here, may not be the best fix.

Signed-off-by: Jason Etheridge <jason@esilibrary.com>
Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>
8 years agoSkip forced targeting of frozen holds
Thomas Berezansky [Tue, 13 Sep 2011 18:48:16 +0000 (14:48 -0400)]
Skip forced targeting of frozen holds

Otherwise a frozen hold can be manually retargeted by staff and captured.

Signed-off-by: Thomas Berezansky <tsbere@mvlc.org>
Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>
8 years agoAcq: List provider code in lineitem lists
Bill Erickson [Thu, 15 Sep 2011 18:57:19 +0000 (14:57 -0400)]
Acq: List provider code in lineitem lists

Along with selection list and purchase order, when an item is attached
to a purchase order, also show the provider code

Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>
8 years agoTPac; improved UPC extraction
Bill Erickson [Mon, 24 Oct 2011 17:52:14 +0000 (13:52 -0400)]
TPac; improved UPC extraction

Handle the case where a record has multiple UPC's.  When this occurs,
use the first UPC as the value for added content lookups as necessary.

Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>
8 years agoThis prevents the acq unified_search method from returning anything other
James Fournie [Tue, 1 Nov 2011 23:01:57 +0000 (16:01 -0700)]
This prevents the acq unified_search method from returning anything other
than the objects of the expected type.

This is to address LP 884991 in which Acq users get a perm error for each
result of a search that they don't have access to retrieve.  They shouldn't
even know about the existence of anything they don't have access to retrieve.

Signed-off-by: James Fournie <jfournie@sitka.bclibraries.ca>
Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>
8 years agoTpac; hold error message 18n string extraction repairs
Bill Erickson [Wed, 2 Nov 2011 13:47:04 +0000 (09:47 -0400)]
Tpac; hold error message 18n string extraction repairs

Make it possible for xgettext.pl (using
Locale::Maketext::Extract::Plugin::TT2) to extract hold error
message strings for translation.  The plugin is not quite
smart enough to extract wholly dynamic values from the templates, since
it (presumably) only parses the templates and does not execute them.
The solution is to wrap the strings in l() where they are defined.

Add an example for the EVENT_MSG_MAP for the event PATRON_EXCEEDS_FINES.

Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>
8 years agoCorrect oversight: replace old with new approach to upgrade script versioning
Lebbeous Fogle-Weekley [Fri, 4 Nov 2011 20:44:57 +0000 (16:44 -0400)]
Correct oversight: replace old with new approach to upgrade script versioning

Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>
8 years agoPreviously invisible copies showed as beige in the staff client, they stopped doing...
James Fournie [Tue, 20 Sep 2011 19:03:03 +0000 (12:03 -0700)]
Previously invisible copies showed as beige in the staff client, they stopped doing this in 2.0

This is because in 2.0 the copy counts logic was moved from open-ils.storage.biblio.record_entry.copy_count.staff
to an in-database function wrapped in open-ils.search.biblio.record.copy_count.staff.  Both methods returned various
counts including an 'unshadow' count, however the 'unshadow' in the new database function was not completed and it
has only been returning a simple copy count rather than a count of opac visible copies as it should be to
replicate the old behaviour.

Signed-off-by: James Fournie <jfournie@sitka.bclibraries.ca>
Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>
8 years agoAcq: better error message if trying to create PO without provider or agency
Lebbeous Fogle-Weekley [Thu, 6 Oct 2011 20:13:51 +0000 (16:13 -0400)]
Acq: better error message if trying to create PO without provider or agency

Before now, if you leave either the provider or ordering agency
fields blank in the "Create Purchase Order" dialog, you get a
nonsensical warning about prepayment.

This fixes that.

Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>
Signed-off-by: Bill Erickson <berick@esilibrary.com>
8 years agoModified authority.indexing_ingest_or_delete to update MARC data
Liam Whalen [Thu, 3 Nov 2011 20:53:57 +0000 (16:53 -0400)]
Modified authority.indexing_ingest_or_delete to update MARC data

The authority.indexing_ingest_or_delete trigger in the update script
was missing a call to authority.propagate_changes which is responsible
for updating the MARC data of records affected by an auhtority merge.
This patch adds the call to authority.propagate_changes as it appears
in the authority.indexing_ingest_or_delete trigger from the file
999.functions.global.sql.

Signed-off-by: Liam Whalen <lwhalen@nrcan-rncan.gc.ca>
Signed-off-by: Dan Scott <dscott@laurentian.ca>
8 years agoRepair Vandelay match_set_tree link
Bill Erickson [Fri, 21 Oct 2011 16:36:48 +0000 (12:36 -0400)]
Repair Vandelay match_set_tree link

ctx.base_path includes the "/eg" portion of the path, so no need to
duplicate that in the template.

Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
8 years agoAcq: fix a bug that sometimes prevents the general search page from loading
Lebbeous Fogle-Weekley [Thu, 6 Oct 2011 16:06:09 +0000 (12:06 -0400)]
Acq: fix a bug that sometimes prevents the general search page from loading

Within Acquisitions General Search, javascript included from one file
in the <head> relies on the variable "localeStrings" being initialized
by javascript included from a file in <body>.  Sometimes the wrong
javascript wins the race, you get an error about localeStrings being
undefined in the Javascript console, and the page stops loading.

Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>
Signed-off-by: Bill Erickson <berick@esilibrary.com>
8 years agoFine generator; raise multi-session mode recv timeout
Bill Erickson [Tue, 1 Nov 2011 21:02:44 +0000 (17:02 -0400)]
Fine generator; raise multi-session mode recv timeout

When fetching the batch of overdue circ IDs to process for fine
generation, wait a little longer for the data to arrive, since
it is a longer running DB query than most.

Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>
8 years agoforgot to update upgrade_log
Jason Etheridge [Tue, 1 Nov 2011 20:18:37 +0000 (16:18 -0400)]
forgot to update upgrade_log

Signed-off-by: Jason Etheridge <jason@esilibrary.com>
8 years agopin upgrade version number
Jason Etheridge [Tue, 1 Nov 2011 19:58:15 +0000 (15:58 -0400)]
pin upgrade version number

Signed-off-by: Jason Etheridge <jason@esilibrary.com>
8 years agoProtect SVF normalizers against NULL values
Mike Rylander [Tue, 1 Nov 2011 19:49:15 +0000 (15:49 -0400)]
Protect SVF normalizers against NULL values

Signed-off-by: Mike Rylander <mrylander@gmail.com>
Signed-off-by: Jason Etheridge <jason@esilibrary.com>
8 years agoForce Amazon AC requests to use 10-digit ISBNs
Dan Scott [Tue, 1 Nov 2011 17:29:25 +0000 (13:29 -0400)]
Force Amazon AC requests to use 10-digit ISBNs

Inspired by a patch submitted by Ian Bays <ian.bays@ptfs-europe.com>
via https://bugs.launchpad.net/evergreen/+bug/870171, this patch takes
the incoming key and, if it is longer than 10 characters (a 10-digit
ISBN or Amazon ID), attempts to normalize it and return a 10-digit ISBN.

We don't normalize all incoming keys because some sites are horribly
abusing the 020 MARC field to include an Amazon ID instead of an ISBN -
but that works in practice and we'd rather not break working
installations if we can avoid it.

Signed-off-by: Dan Scott <dscott@laurentian.ca>
8 years agoUse ne on line 631 of Actor.pm.
Jason Stephenson [Wed, 19 Oct 2011 20:47:03 +0000 (16:47 -0400)]
Use ne on line 631 of Actor.pm.

This is so that we stop getting errors like

Argument "1319057146.8881216485.4819607085" isn't numeric in numeric ne (!=) at
/usr/local/share/perl/5.10.1/OpenILS/Application/Actor.pm line 630.

in /openils/var/log/open-ils.actor_stderr.log

Signed-off-by: Jason Stephenson <jstephenson@mvlc.org>
Signed-off-by: Dan Scott <dscott@laurentian.ca>
8 years agoTpac; bookbag MARC retrieved via unapi
Bill Erickson [Tue, 1 Nov 2011 15:35:41 +0000 (11:35 -0400)]
Tpac; bookbag MARC retrieved via unapi

Make sure the bookbag interface has access to metabib.record_attr's by
retrieving and parsing the marcxml via unapi (get_records_and_facets).

Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
8 years agoTrivial whitespace touchup to Cronscript.pm.in
Dan Scott [Mon, 31 Oct 2011 20:56:55 +0000 (16:56 -0400)]
Trivial whitespace touchup to Cronscript.pm.in

Signed-off-by: Dan Scott <dscott@laurentian.ca>
8 years agoOpenILS::Utils::Cronscript can have only 1 session.
Jason Stephenson [Mon, 31 Oct 2011 20:36:27 +0000 (16:36 -0400)]
OpenILS::Utils::Cronscript can have only 1 session.

Work around the above limitation by using OpenSRF::AppSession
to create a session for open-ils.auth in the authenticate method.

Also requires us to manually check if we're bootstrapped, since
we can't rely on OpenILS::Utils::Cronscript->session to do it for
us.

Signed-off-by: Jason Stephenson <jstephenson@mvlc.org>
Signed-off-by: Dan Scott <dscott@laurentian.ca>
8 years agoAdd authentication capability to OpenILS::Utils::Cronscript.
Jason Stephenson [Mon, 31 Oct 2011 19:48:38 +0000 (15:48 -0400)]
Add authentication capability to OpenILS::Utils::Cronscript.

This allows the client script to call authenticate with a hash ref of
user information so that the user can login and get an authtoken if
needed.

It also stores the authtoken and authtime values for later retrieval.

Signed-off-by: Jason Stephenson <jstephenson@mvlc.org>
Signed-off-by: Dan Scott <dscott@laurentian.ca>
8 years agoTpac: show staff saved searches on no-results page
Lebbeous Fogle-Weekley [Mon, 31 Oct 2011 15:00:02 +0000 (11:00 -0400)]
Tpac: show staff saved searches on no-results page

Not having it there is awkward for the user who relies on the feature
and enters a search term that gets no results.

Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>
Signed-off-by: Bill Erickson <berick@esilibrary.com>
8 years agoTpac: staff saved searches expand/collapse, use right ou context for setting
Lebbeous Fogle-Weekley [Thu, 27 Oct 2011 22:20:00 +0000 (18:20 -0400)]
Tpac: staff saved searches expand/collapse, use right ou context for setting

1) Display staff saved searches collapsed by default. This makes the facet
display right underneath it more accessible in the case of long search
histories.

2) When checking the org unit setting for the limiting number of
searches to save, use the user's ws_ou as the context, since this is a
staff-client feature.

Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>
Signed-off-by: Bill Erickson <berick@esilibrary.com>
8 years agoA tweak to eg.conf to prevent against the BEAST exploit in TLS/SSL
Michael Peters [Thu, 27 Oct 2011 13:42:37 +0000 (09:42 -0400)]
A tweak to eg.conf to prevent against the BEAST exploit in TLS/SSL

Based on recommedation from https://community.qualys.com/blogs/securitylabs/2011/10/17/mitigating-the-beast-attack-on-tls

Signed-off-by: Michael Peters <mrpeters@library.in.gov>
Signed-off-by: Galen Charlton <gmc@esilibrary.com>
8 years agoTpac: fix the doubly wrapped holds editor
Lebbeous Fogle-Weekley [Fri, 28 Oct 2011 14:40:03 +0000 (10:40 -0400)]
Tpac: fix the doubly wrapped holds editor

For some reason it must have made since in the past to wrap this
interface in both parts/base.tt2 and myopac/parts/base.tt2, but
nowadays that results in having extra headers and footers around
the actual page contents, so the layout looks broken.

This fixes that. No other interfaces were doing this double wrapping.

Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>
Signed-off-by: Bill Erickson <berick@esilibrary.com>
8 years agoTpac: Rename orig_loc to physical_loc, for clarity's sake.
Lebbeous Fogle-Weekley [Fri, 28 Oct 2011 17:49:42 +0000 (13:49 -0400)]
Tpac: Rename orig_loc to physical_loc, for clarity's sake.

This is an OPAC parameter not so much meant for use by patrons at home,
but for users at machines physically located in libraries.  In
conjunction with OpenILS::WWW::Redirector, it helps those machines
"remember" what branch they're in.

Anyway, the name orig_loc doesn't make the parameter's meaning quite as
clear as it could whne you encounter it in code.

Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>
Signed-off-by: Bill Erickson <berick@esilibrary.com>
8 years agoTpac: clear the per-process cache of org-unit settings on each page load
Lebbeous Fogle-Weekley [Thu, 27 Oct 2011 21:37:06 +0000 (17:37 -0400)]
Tpac: clear the per-process cache of org-unit settings on each page load

Any time saved is not worth the confusion of changes to org unit settings not
taking effect in the TPAC without an apache restart, or worse, taking
effect for some page loads but not others.

Actually authored by Bill.

Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>
Signed-off-by: Bill Erickson <berick@esilibrary.com>
8 years agoStamping upgrade script for YAOUS-target-when-closed
Mike Rylander [Fri, 28 Oct 2011 15:09:32 +0000 (11:09 -0400)]
Stamping upgrade script for YAOUS-target-when-closed

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