still to test

Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>

Change order: Saving filter sets

Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>

Filter loading: gracefully skip unknown fields, remove inital empty row

Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>

Change order: filter set loading works. still need saving.

Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>

Permission fixing

Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>

For the change order, let's do filter sets a little more generalized-like

Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>

Verify-all now means all-matching-my-search-terms, not necessarily all-in-uvs

Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>

implement the "process immediately" switch, hitherto unhooked up

Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>

Fix broken display of verification attempt in progress

Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>

TODO: deal with update_attempt_display() missing its target nodes now

scrolling is perfect vertically, but where's my horiz scrollbar? grr

Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>

Wasted hours on this

Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>

Corrections to the preceding commit

Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>

started verification review UI, also SCHEMA CHANGES

It just doesn't work for me to not have url_verify.url directly related
to url_verify.session.  When dealing with the "root" URL in a redirect chain,
you can get the related session through url_selector, but not when
you have any later URL in the chain.  The only way for IDL perms to work
would be to have a link to a view using a CTE to find the "root" URL.

That's too complex, so instead of that I've just added a session fkey on
url_verify.url.

Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>

lkasejf. It's the weekend.

Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>

URLVerify.pm; response throttling repairs

Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>

A note about open-ils.url_verify.verify_url for future reference

Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>

Verification sorta works but doesn't report progress correctly

Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>

these new bits untested, to be continued

Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>

Essentials for URL selecting interface

Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>

Fix IDL permissions that require jumps

Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>

Refactor create_session as dojo module.

Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>

Various fixes, largely UI

Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>

fix xpath generation to match what works

Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>

Call URL extraction phase from UI

Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>

Fix previously nonfunctional stored procedure url_verify.extract_urls(INT,INT)

Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>

whitespace

Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>

Permisison fixing

Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>

Pretty much finished session create UI but for cloning

Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>

Buckets and their items aren't designed to be PCRUD accessible,

so we need a handy view to link URL Verify Sessions to the bib
contained.  We can leverage this in flattener queries.

Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>

Fix subtle Perl issue

Not a syntax error that the compiler will catch, but see
"perldoc -f do" which will lead you do "perldoc perlsyn"

Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>

More UI work. Saved search selector & search scope OU selector & cosmetics

Still not actually functional.

Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>

use a perm that actually exists

Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>

Deal with moved publish_fieldmapper() method

This is a companion commit to
fac45ab9b1cb8924 / Move Fieldmapper API call to Application.pm

Without it, Flattener and Action/Trigger stop working with errors like
this:

[Mon Aug 20 13:50:18 2012] [error] [client XXX.XXX.XXX.XXX] Exception:
OpenSRF::EX::ERROR 2012-08-20T13:50:18 main -e:0 System ERROR:
Exception: OpenSRF::DomainObject::oilsMethodException
2012-08-20T13:50:18 OpenSRF::AppRequest
/usr/local/share/perl/5.10.1/OpenSRF/AppSession.pm:1064 <500>   *** Call
to [open-ils.fielder.flattened_search.execute.atomic] failed for session
[1345485018.767884163.96534353976], thread trace [1]:\nNo field by the
name publish_fieldmapper in Fieldmapper! at
/usr/local/share/perl/5.10.1/OpenILS/Utils/Fieldmapper.pm line
270.\n\n\n\n, referer:
http://XXXXXXX/eg/conify/global/actor/search_filter_group

Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>

check for session ownership and for previous searchitude

Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>

Get all search results, not just first 10

Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>

Beginnings for the big session kick-off UI.  Not yet functional.

Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>

ML methods to create sessions and do the searching/bucketing

We can't use PCRUD to create url_verify.session objects because a) you
couldn't trust the creator field if we allowed that, and b) the
container foreign key has a not-null constraint, so you have to create
that first, and you can't do that with PCRUD.

I've removed the C, U and D perms for PCRUD for url_verify.session, but
I left the R in case we wind up using that.

Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>

Add open-ils.url_verify service to example OpenSRF configs

Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>

New container type for url verification sessions

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

Function for ingesting URLs from container item + session

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

URLVerify.pm; docs; tested redirect max/loops and repairs

Signed-off-by: Bill Erickson <berick@esilibrary.com>

URLVerify.pm; docs; url domain looping; cleanup

Signed-off-by: Bill Erickson <berick@esilibrary.com>

URLVerify.pm; docs; url shuffling

Signed-off-by: Bill Erickson <berick@esilibrary.com>

URLVerify.pm; docs; url shuffling

Signed-off-by: Bill Erickson <berick@esilibrary.com>

URLVerify.pm; avoid re-processing same url repairs; honor delay=0

Signed-off-by: Bill Erickson <berick@esilibrary.com>

URLVerify.pm; avoid re-processing same url within the same attempt (thinko)

Signed-off-by: Bill Erickson <berick@esilibrary.com>

URLVerify.pm; avoid re-processing same url within the same attempt

Signed-off-by: Bill Erickson <berick@esilibrary.com>

URLVerify.pm; apply timeout to lwp useragent; comments

Signed-off-by: Bill Erickson <berick@esilibrary.com>

URLVerify.pm; move to lwp to support ftp and simplify

Signed-off-by: Bill Erickson <berick@esilibrary.com>

Do the same thing I did to fix constrain names in upgr scripts to baseline

Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>

url_verify schema repair / res-code constraint

Signed-off-by: Bill Erickson <berick@esilibrary.com>

url_verify seed data / sql manifest

Signed-off-by: Bill Erickson <berick@esilibrary.com>

url verify seed data repairs

Signed-off-by: Bill Erickson <berick@esilibrary.com>

URLVerify.pm; settings, tmp caching, cleanup / misc

Signed-off-by: Bill Erickson <berick@esilibrary.com>

url verify seed data / null constraint repairs

Signed-off-by: Bill Erickson <berick@esilibrary.com>

url_verify seed data (trigger)

Signed-off-by: Bill Erickson <berick@esilibrary.com>

url_verify perm/org setting seed data

Signed-off-by: Bill Erickson <berick@esilibrary.com>

url_verify perm/org setting seed data

Signed-off-by: Bill Erickson <berick@esilibrary.com>

url_verify perm/org setting seed data

Signed-off-by: Bill Erickson <berick@esilibrary.com>

URLVerify.pm; redirects / error handling

Signed-off-by: Bill Erickson <berick@esilibrary.com>

URLVerify.pm; more testing tweaks

Signed-off-by: Bill Erickson <berick@esilibrary.com>

Move Fieldmapper API call to Application.pm

Move export of opensrf.open-ils.system.fieldmapper API call from
Fieldmapper.pm into Application.pm with the rest of the shared API
calls.  This allows us to remove the OpenSRF::Application base from
Fieldmapper, which was causing some method name collisions.  For
example, if a Fieldmapper object had a field called "session", which,
incidentally, is also a method of OpenSRF::Application, the version
from OpenSRF::Application would get called instead of the Fieldmapper
version, since Fieldmapper methods are defined during AUTOLOAD.
Hilarity was guaranteed to ensue.

Signed-off-by: Bill Erickson <berick@esilibrary.com>

URLVerify.pm; initial testing tweaks

Signed-off-by: Bill Erickson <berick@esilibrary.com>

URLVerify.pm; resume options; docs

Signed-off-by: Bill Erickson <berick@esilibrary.com>

Liberalize res_code constraint

Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>

Constraints need unique names

Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>

URLVerify.pm; docs / cleanup

Signed-off-by: Bill Erickson <berick@esilibrary.com>

URLVerify.pm; firing batch url verify calls

Signed-off-by: Bill Erickson <berick@esilibrary.com>

URLVerify.pm getting started

Signed-off-by: Bill Erickson <berick@esilibrary.com>

Added is_event check to AppUtils

Signed-off-by: Bill Erickson <berick@esilibrary.com>

Add ON INSERT trigger to parse the URL as it is added

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

Add Rose::URI to the CPAN_MODULES list

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

Start building URL-Validation related funcitons

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

Schema and IDL changes for URL Verification functionality

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

Default notify by text receipt replaces to blank

So that when the option is disabled we don't leave behind things like
%notify_by_text% and %notify_by_text_msg%.

Signed-off-by: Thomas Berezansky <tsbere@mvlc.org>
Signed-off-by: Jason Etheridge <jason@esilibrary.com>

TPAC: Results - don't show the pubdate if we're showing full pubinfo

In the "show few details" mode, we show just the pubdate; in the "show
full details" mode, we're showing the full pubinfo and therefore do not
need to show the stubby pubdate.

Signed-off-by: Dan Scott <dscott@laurentian.ca>
Signed-off-by: Ben Shum <bshum@biblio.org>

TPAC: Display full publication info

In both the results table and record summary, we were failing to display
the place of publication - which is critical information for people
trying to track down the right edition or constructing a bibliography.

Grab the place of publication, and then reconstruct 260abc (avoiding
another XPath iteration and giving ourselves the ability to enhance the
schema.org microdata at the same time).

Signed-off-by: Dan Scott <dscott@laurentian.ca>
Signed-off-by: Ben Shum <bshum@biblio.org>

Enable 1-hit redirect for item barcode searches

Signed-off-by: Thomas Berezansky <tsbere@mvlc.org>
Signed-off-by: Jason Stephenson <jstephenson@mvlc.org>
Signed-off-by: Bill Erickson <berick@esilibrary.com>

Protect against fleshed circ_lib in legacy check_age_protect

Retrieving org settings with an org unit object as the context org unit
results in errors and dying.

This affects legacy scripts only.

Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Michael Peters <mrpeters@library.in.gov>

Sort of in line with 949322 but not absolutely necessary
This just supresses the message about what text was copied to the clipboard - globally!

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

lp949322 Copy Patron barcode to clipboard when clicking their name
CTRL+Click will open the patron record in a new tab

Signed-off-by: edoceo <code@edoceo.com>
This affects the patron label in Item Status -> Alt View -> Holds/Transits.  Looks good to me.

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

Various additions to the 2.3 release notes.

Signed-off-by: Kathy Lussier <klussier@masslnc.org>
Signed-off-by: Dan Scott <dscott@laurentian.ca>

Hide Print Page button on Simplified Pull List

On the Simplified Pull List interface, there is a button for "Print Page"
that shows up in the upper right corner. The suggestion is to hide that
button to avoid library staff printing just what's visible instead of using
the proper "Print Pull List" button that's actually on the page.

Signed-off-by: Ben Shum <bshum@biblio.org>
Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>

TPAC: Escape single quotes in MFHD record location

If an MFHD record contains an 852 field with a subfield containing a
single quote, the unescaped single quote is introduced directly into the
value of the JavaScript hash that uses single quotes as delimiters -
thereby generating a JS exception and preventing the staff client from
properly populating the MFHD Editor menu.

This commit escapes incoming single quotes from the MFHD location field
to prevent that from happening. The other fields in the JS hash are
numeric and therefore should not need escaping.

Signed-off-by: Dan Scott <dscott@laurentian.ca>
Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>
Signed-off-by: Dan Wells <dbw2@calvin.edu>

Remove defunct variable from 'jtitle' code

With the second part of the 'jtitle' code moved elsewhere,
we no longer need this extra '$q' variable.  No big deal, but
best to sweep up while we're in here.

Signed-off-by: Dan Wells <dbw2@calvin.edu>
Signed-off-by: Dan Scott <dscott@laurentian.ca>

Revert "Bug #1044721: QP handles explicit group+joiner badly"

This reverts commit b77bb9943009b81042b4ef1a6ac33ec9e3e96dd7.

Unfortunately this had side effects that escaped our manual testing
processes.  Especially with facets as reported by Ben Shum.

Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>
Signed-off-by: Dan Scott <dscott@laurentian.ca>

Journal title hackery - move to a more robust location

The fix for bug #1044721 broke the journal title hack, which probably
should have lived where this commit moves it - if, in the age of filter
groups, we decide whether it's worthwhile to keep it alive at all.

Signed-off-by: Dan Scott <dscott@laurentian.ca>
Signed-off-by: Dan Wells <dbw2@calvin.edu>

Upgrade bits for Vandelay Overlay Changes

Signed-off-by: Dan Wells <dbw2@calvin.edu>

Increase Overlay Speed for Standard Identifiers

Due to an inefficient query order and a confused query planner,
some overlay matching can end up going abysmally slow.  See LP
bug #1024095 for elaboration on the issue.

This commit reorders the joins as originally suggested by Lebbeous,
then adds a specific index to coax the query planner into making the
best choices.  The example index here only targets the 02x identifier
fields, so other tag/subfield/substring(value) indexes will likely
be necessary for maximum benefit when matching on other fields
(e.g. 010).

These changes combined with the previous commit which replaces 'LIKE
ANY' with 'LIKE (... OR ...)' has shown great promise in testing.

Signed-off-by: Dan Wells <dbw2@calvin.edu>
Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>

Vandelay: Fix index-miss with MARC Imports using Match Sets

In some Postgres installations, an expression such as:

(value LIKE '13423488%' OR value LIKE '245425%') will use a btree index,
but

value LIKE ANY('{13423488%,245425%}'::TEXT[])

will not.

Missing such an index can make matching incoming bibs based on a field
that's present in most of your existing records terrifically slow.

Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>
Signed-off-by: Dan Wells <dbw2@calvin.edu>

Bug #1044721: QP handles explicit group+joiner badly

Although the following four queries should be equivalent:
  george  ||  fred
 (george) ||  fred
  george  || (fred)
 (george) || (fred)

The Pg QueryParser driver generates SQL queries that return different
numbers of results for each of those queries, with all desired results
appearing only with the first query. This seems to be because of the way
filters are added in the interface.

This patch adjusts the Search code in TPAC to add an explicit group
around the user-entered query, which enables the Pg QueryParser driver
to handle all four of the above queries properly.

This patch should also resolve the problems encountered with the patch
for bug #1040740, "Implicit ANDs should have higher precedence than
explicit ORs"

Signed-off-by: Jared Camins-Esakov <jcamins@cpbibliography.com>
Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>

Gracefully handle reporter class with no labels

LP 1045964 -- unable to clone report template

The template interface was failing while rendering the reporter sources
drop-down when a reporter source (IDL class) had no label.  This adds a
sanity check around that.

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

Fix Merge All Records in Bucket action

A xul file was not being prefixed with oils://remote

Note: This is a quick fix. Ideally we move the url to constants.js.

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

Define a few strings missing from cat.properties

By the power of build/i18n/test/check_properties.py ...

Signed-off-by: Dan Scott <dscott@laurentian.ca>
Signed-off-by: Bill Erickson <berick@esilibrary.com>

Importing translations for 2.3.rc1

Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Dan Scott <dscott@laurentian.ca>

Do the SQL upgrade dance for list pubdate in CSV

With a slight tweak to base the upgrade off of the event definition name
instead of the ID; possibly more resilient.

Signed-off-by: Dan Scott <dscott@laurentian.ca>

Add pub date to CSV output.

Change the template for the ContainerCSV reactor to add the publication
date (MARC 260$c) in the csv output in the same position that the previous
commit adds it to the html output.

Add an upgrade script to update the existing template for the ContainerCSV
reactor in action_trigger.event_definition.

Signed-off-by: Jason Stephenson <jstephenson@mvlc.org>
Signed-off-by: Dan Scott <dscott@laurentian.ca>

TPAC: Add the ability to set a different default locale

Installations with multiple sites might desire different default locales
for their skins. This adds the Apache config directive
"OILSWebDefaultLocale", which enables you to set the default locale on a
global and a per-skin basis.

Note that this prevents the browser Accept-language header from having
any effect; sadly that header is not often used in real life.

Signed-off-by: Dan Scott <dscott@laurentian.ca>
Signed-off-by: Art Rhyno <art632000@yahoo.ca>
Conflicts:
Open-ILS/examples/apache/eg_vhost.conf

Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Dan Scott <dscott@laurentian.ca>

TPAC: Make call number browse record links retain search lib

In addition to maintaining scope for the call number browse search
results, we also need to maintain the scope for the links for the call
number records in the results themselves (for example, the "retrieve
this record" link, or the "search for this author" link).

Signed-off-by: Dan Scott <dscott@laurentian.ca>
Signed-off-by: Ben Shum <bshum@biblio.org>

Documentation: removie file reference to line items since section was moved to another file.

Signed-off-by: Robert Soulliere <robert.soulliere@mohawkcollege.ca>