Acq: misc minor bugfixes to various admin interfaces that didn't load
consistently.  Also a little more menu re-org.

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

no need to start a xact in lineitem search api call, especially if it's not closed out when done

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

specify @autoscroll=false for xul browser elements

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

Acq: minor menu reorganization. Move admin-oriented interfaces to admin menu.

Also remove "Purchase Order Events" from the acq menu: it's outmoded.

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

support pre-indexing normalizers for facet value normalization (negative pos value in the field-norm map)

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

Avoid spurious errors when searching more granular indexes

The storage service automatically generates methods for each top-level search
class, like open-ils.storage.authority.keyword.see_from.controlled.atomic, but
here we were trying to access more granular methods (which storage does not
know about) when a granular index was used. This generated an ugly error in
the logs.

To avoid these errors, we'll only look for cross references from the broad
index class to which the granular index belonds.

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

in patron editor, prevent Save and Save & Clone from working with an invalid form, or duplicate barcode or usrname

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

the dojo code now behind the util.date iso8601 formatter neither needs nor likes us appending timestamps to dates like that

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

Tiny bug in 16714; fixing it prevents a useless logline in some cases

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

In the unlikely event of an error in a payment transaction following
sucessful external credit card processing, keep the information from the
payment processor and
    a) attach it to the ilsevent that we return, and
    b) log it at the "error" loglevel

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

Make the estimated hold queue position method aware of the cut_in_line field

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

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

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

Better documentation of payment outcomes, particularly credit card declines

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

some minor code cleanup of the retrieve user by barcode method

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

avoid sending unnecessary disconnects to cstore

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

In qstore: support CASE expressions.

M    Open-ILS/include/openils/oils_buildq.h
M    Open-ILS/src/c-apps/oils_storedq.c
M    Open-ILS/src/c-apps/buildSQL.c

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

correct nullability test for certain WHERE clauses

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

missed a constraint change in the 0307 upgrade script

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

Serials schema adjustments designed by Dan Wells.  Essential features are moving subscription to attach directly to BRE, as the controlled peer to SRE, and allowing distributions to optionally link to an SRE in order to extend them.

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

a null boolean value should use the label for UNSET instead of False (the default)

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

troubleshooting DOM issue where too many columns in a treerow
some refactoring and comment cleanup.  toward having the need for treecell creation be determined on a column by column basis rather than examining the contents of a treerow in one go

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

$e may false (i.e. 0=SUCCESS).  it's undefined we need to avoid

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

forgot to remove old order-by when crafting the trunk version of the fifo-holds patch from James Fournie

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

Add helper handling to specify length of field, like: isbn_10 or isbn_13

"isbn" will still just get the first available value.

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

Logger not used

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

Bugfix fatal error: $self->editor is not defined

Logger wasn't being used either.

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

0306: Update JEDI template, edi_pusher logic tweaked

Old pusher was excluding valid target with "not in"
subquery.  Broke that out and added debugging so
that situation should be more obvious (if verbose).

Signed-off-by: Joe Atzberger <atz@esilibrary.com>
git-svn-id: svn://svn.open-ils.org/ILS/trunk@16681 dcc99617-32d9-48b4-a31d-7c20da2025e4

Improve the way information from payment card processors bubbles up to
callers of open-ils.circ.money.payment

This is done by replacing unneeded, weird data structures with ilsevents and
filling those events' payloads with result data from the processor.

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

Org unit setting and behavior for blocking hold requests if hold recipient has expired privileges

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

when configuring sort order for copy locations, allow local sorting on locations defined at parent orgs (in the UI, finally)

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

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.

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

Add column "left_operand" to query.expr_xcase view.

M    Open-ILS/src/sql/Pg/002.schema.config.sql
M    Open-ILS/src/sql/Pg/008.schema.query.sql
A    Open-ILS/src/sql/Pg/upgrade/0304.schema.query-expr-xcase-left-operand.sql
M    Open-ILS/examples/fm_IDL.xml

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

Forward port r16670 and r16672 to avoid directory creation errors in Dojo resource i18n script

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

Remove extraneous whitespace in ILS events POT file

We were grabbing everything - tabs, newlines, extra spaces - from
ils_events.xml, and we don't want to do that.

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

When looking up a bind variable by name: escape any special characters
in the name before building the query.

M    Open-ILS/src/c-apps/oils_storedq.c

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

syntax

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

seed the UPDATE_PICKUP_LIB_FROM_TRANSIT perm

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

Trunk port of a 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 ]

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

When building string literals in SQL: escape special characters.

M    Open-ILS/src/c-apps/buildSQL.c

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

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

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

change the xulG tests to fix Save & Clone for new patrons

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

Change the return type of the open-ils.qstore.prepare method.

Formerly, this method merely returned a string to be used as
token for future references to the query.

Now it returns an object, with the following elements:

"token" -- the query token as returned formerly
"bind_variables" -- a (possibly empty) object containing a
list of bind variables and their various attributes, as
returned by the param_list method.

Rationale: an interactive query-building client will always
want a list of bind variables so that it can prompt the user
for values.  With this change to the return type, the client
can get such a list without having to make a separate call
to the param_list method.

M    Open-ILS/src/c-apps/oils_qstore.c

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

add important reporting-focused link to asset.copy_location

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

Hold requests in the middle layer now bubble up more specific information to
the caller in the event of failure.

The caller can find an ilsevent in the response received from calling
open-ils.circ.title_hold.is_possible, and this event will have a fail_part
attribute in its payload that can be mapped either to a) the new entities
in opac.dtd or b) values of the name column of the database table
config.standing_penalty, depending on why a hold request failed.

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

thinko on how ou settings are returned from open-ils.actor.ou_setting.ancestor_default; also, cache max_loops per pickup lib for the duration of the targetting run

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

Fix a pcrud bug.  Create method wasn't working because we were
foolishly trying to fetch the record that hadn't been created
yet, in order to test its ownership.

M    Open-ILS/src/c-apps/oils_sql.c

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

make sure we are not trying to use a "modern" class::dbi

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

logic thinko causing us to never use the restricted set of copies for this hold targeting round, and instead use all good copies, when in max_loop mode

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

drop copy transparency tables

These tables were never used.

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

thinko

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

Some python code is non-executable (script_x_check updated)

This corrects an overreach on the earlier patch.

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

</link> is not valid HTML

Although the DTD *may* specify xhtml, since in practice we are serving
with Content-Type: text/html and
  <meta http-equiv="Content-Type" content="text/html;charset=utf-8" />
we should avoid tags that are illegal in html.

These errors are reported in chrome as:
    "Unmatched </link> encountered.  Ignoring tag."

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

EDI Translator (relocated and combined)

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

Rewrote the implementation of the open-ils.qstore.columns method.

The old implementation examined the SELECT clause(s) as encoded
in the query.select_item and query.expression tables.  The new
implementation performs a dummy query, and then returns the
field names from the database result.

Advantages of the new implementation:

1. It doesn't get confused by wild cards in the SELECT clause.

2. It provides a field name for every column, including those
which are function calls, subqueries, or other expressions.
(Of course these names may not be very revealing, such as
"?column?".)

Disadvantages:

1. The resulting field names are not qualified by table name.

2. Additional overhead due to the additional database call.

M    Open-ILS/src/c-apps/oils_storedq.c

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

typo

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

In the patron editor, ensure that the Verify Password field matches the Password field to validate

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

Bugfix cleanup of stray tags

<strong> had mismatched nesting and </img> was completely orphaned.

Signed-off-by: Joe Atzberger <atz@esilibrary.com>
git-svn-id: svn://svn.open-ils.org/ILS/trunk@16638 dcc99617-32d9-48b4-a31d-7c20da2025e4

type is a required attributed for script and style tags.

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

Executable scripts

These were all found by ./build/tools/script_x_check.sh

Signed-off-by: Joe Atzberger <atz@esilibrary.com>
git-svn-id: svn://svn.open-ils.org/ILS/trunk@16636 dcc99617-32d9-48b4-a31d-7c20da2025e4

update GPL 2.0 text in the staff client license statement

Also fixed last reference to the FSF's old mailing address

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

updated license text with current GPL 2.0 text

NOTE: THIS IS NOT A SEMANTICALLY IMPORTANT CHANGE TO EVERGREEN'S LICENSE.

Updated LICENSE.txt by pulling the current GPL 2.0 text
directly from http://www.gnu.org/licenses/old-licenses/gpl-2.0.txt

The changes are purely textual, and consist solely of:

* updating the FSF's mailing address
* changing references to the "GNU Library General Public License"
  to "GNU Lesser General Public License", per the FSF's current
  (and long-standing) practice
* whitespace changes

REPEAT: THIS IS NOT A RELICENSING

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

wrong URL

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

fix references to ersatz "as_unimarc" method in MARC::Record

No such method exists in MARC::Record; MARC::Record->as_usmarc()
should be used instead; think of as_usmarc() as if it were
named as_iso2709_blob().

As general note, UNIMARC support in Evergreen is currently
rather theoretical at this point.

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

no includes in fm_IDL.xml

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

explicitly allow SSI for these locations.  Important for Apache on Redhat, etc.

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

Recognize new "required" and "validate" attributes for <field> element.

M    Open-ILS/src/c-apps/idlval.c

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

1. Support function calls in the FROM clause.

2. Support wildcards in the SELECT clause.

WARNING: the presence of a wildcard in the SELECT clause
is likely to disrupt a GROUP BY by renumbering the columns.

Also: the "columns" method currently cannot return the
names of the columns into which a wild card is expanded.

M    Open-ILS/include/openils/oils_buildq.h
M    Open-ILS/src/c-apps/oils_storedq.c
M    Open-ILS/src/c-apps/buildSQL.c

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

protect against empty results from bib searches caused by
search timeouts.  This allows the API call to log the error
and return reasonable results

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

Split the ingest trigger logic into bits, simplifying optimization and configuration

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

Patch from Galen Charlton:

[This] patch adds a new XSL transform for indexing purpose that converts MARC21 880 fields (which are used for alternate graphic representations, i.e., vernacular representations of foreign language strings) to the base tags.

For example, if a MARC record for a Chinese book has

245.00 $6 880-01 $a Ba shi san nian duan pian xiao shuo xuan
880.00 $6 245-01/$1 $a\u516b\u5341\u4e09\u5e74\u77ed\u7bc7\u5c0f\u8aaa\u9078

this stylesheet will transform it to the equivalent of

245.00 $6 880-01 $a Ba shi san nian duan pian xiao shuo xuan
245.00 $6 245-01/$1 $a\u516b\u5341\u4e09\u5e74\u77ed\u7bc7\u5c0f\u8aaa\u9078

This allows an indexing XPath like //marc:datafield[@tag='245']/marc:subfield[@code='a'] to bring in both the vernacular and transliterated versions of the 245$a.

[ED: pretend the \u-encoded unicode are real characters ... curse you, Chrome, for not giving me the raw string!]

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

config.metabib_field_index_norm_map params must be valid json;  see also upgrade 0271

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

doh, too quick on the commit. :)  fix the dupe-barcode test

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

re-adds the duplicate-barcode check that was lost with the new patron editor.  TODO: disable the save buttons when there are form errors

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

Add the left_operand column to the query.expr_xbet view
(correcting an oversight).

M    Open-ILS/src/sql/Pg/002.schema.config.sql
M    Open-ILS/src/sql/Pg/008.schema.query.sql
A    Open-ILS/src/sql/Pg/upgrade/0298.schema.xbet-left-operand.sql
M    Open-ILS/examples/fm_IDL.xml

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

1. Add table: serial.caption_and_pattern

2. In serial.record entry: make "marc" row nullable.

M    Open-ILS/src/sql/Pg/002.schema.config.sql
A    Open-ILS/src/sql/Pg/upgrade/0297.schema.serial-caption-and-pattern.sql
M    Open-ILS/src/sql/Pg/210.schema.serials.sql
M    Open-ILS/examples/fm_IDL.xml

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

Patch from Galen Charlton:

Add foreign key to require that each row in metabib_field has a corresponding XML format.

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

I'm not a fan of I18N-effort on debug output meant for developers.  In this case, the wrong preferences file was being referenced, causing an error

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

Patch from Galen Charlton:

Fix glitch in definition of the new default identifier indexes that would cause bib ingest to fail.  Note that this affected only newly-created databases; the upgrade script for 0265 is correct.

Thanks to Lee Dickens for running into this bug.

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

Patch from Galen Charlton: The attached patch adds triggers on biblio.record_entry and authority.record_entry to prevent the insertion or update of MARCXML that is not well-formed.  Since the various consumers of bre.marc and are.marc tend not to handle invalid XML all that well, these triggers serve as a (very basic) data integrity check.  libmail.georgialibraries.org/pipermail/open-ils-dev/2010-June/006116.html

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

Added a new login type "persist", as a peer of "opac", "staff", and "temp".
It is intended for sessions that may stay open for days or weeks at a time
even in the absence of activity.  The default timeout interval is defined
as two weeks in opensrf.xml, and may be overridden by the org unit
setting "auth.persistent_login_interval".

Timeout resets work a little differently for persistent logins.  They
have no effect unless the session is within ten minutes of expiring.  When
they do take effect, they reset the timeout to ten minutes, rather than to
the full length of the original timeout.  That way we can avoid rudely
interrupting an active session without extending it excessively.

The ten minute reset interval for persistent timeouts is currently
hard-coded.  With some further work it could be made configurable.

The timeout resets for the older login types still work the way they
always have.

------------

In order to make it easier to specify long timeout intervals, the
auth server now accepts PostgreSQL-style interval strings, such as
"15 minutes" or "2 weeks".  Such strings work for any of the login
types, and they work either in opensrf.xml or in the org unit setting
values.

If the timeout setting (in either context) is all digits, then it will
be interpreted as an integral number of seconds, as it has been in the
past.  So existing settings will almost certainly continue to work
without change.

The exception -- an unlikely one -- is if the existing setting carries
a leading plus sign.  Under the old regime, a leading plus sign was
simply superfluous, and had no effect.  With the new version, a
leading plus sign means that the following number is to be treated as
a number of hours, rather than a number of seconds (just because
that's what PostgreSQL does with it).

Hence in the unlikely event that existing settings use a leading
plus sign, this change will make those timeouts 3600 times as long
as they should be.

If the timeout interval is expressed as anything other than a string
of all digits (possibly with leading and/or trailing white space), we
make a database call to get PostgreSQL to interpret it for us.  So the
convenience of using interval strings comes at the price of some
additional overhead.

--------------

Besides applying the changes to the C code, it will be necessary to
update the opensrf.xml file in order to define a default timeout
interval for the new login type.

M    Open-ILS/include/openils/oils_constants.h
M    Open-ILS/src/c-apps/oils_auth.c
M    Open-ILS/examples/opensrf.xml.example

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

Update a complete set of PO files in trunk

For all languages that have submitted any significant amount of translation,
commit PO files for every available POT or PO file (including our many new Dojo
I18N resource bundles)

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

Get the Makefile up to date with all of our new Dojo resource files

At some point, this Makefile has to become smarter, because there's a
whole lot of error-prone copying and pasting going on right now.

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

Stab at matching up reservation.js with the pluralized, JSON-compliant Dojo NLS file

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

Make booking reservation Dojo NLS file JSON-compliant

You can't use string concatentation based on the plurality of arguments
to create a translatable string. Solution: create a singular and a plural
version of the string, and use the logic in the program itself to pass in
the substitution variable to the correct respective string.

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

Smarter error handling in Dojo i18n script

Identify the troublesome file before allowing the traceback

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

More cleanup of Dojo JSON files

  * Double-quote all strings - no single-quoting or barewords
  * Don't escape dollar signs
  * JSON doesn't allow expressions like string continuations ("" + "")

reservations.js has some remaining issues, looks like functions were
defined for what should be simple string substitutions. dojo.string.substitute()
will solve this problem, I believe, with ${0}, ${1}, etc tokens for substitution

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

In Dojo i18n script: create output directories if they don't exist

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

Fix up acq Dojo NLS file to be strictly JSON compliant

simplejson chokes on non-compliant JSON:
  * Strings must be delimited in double-quotes
  * Joining strings with + is not allowed

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

Commit updates from Launchpad translation

Bringing all of the translated files up to date with what is in Launchpad.
Next steps: generate all of the updates from trunk and add all the new files

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

New function oilsUtilsIntervalToSeconds() to translate a string into a
number representing the number of seconds in an interval of time.

Simple integers are converted directly.  Anything else is passed to
PostgreSQL to be translated as an interval string.

M    Open-ILS/include/openils/oils_utils.h
M    Open-ILS/src/c-apps/oils_utils.c

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

Email/print methods for printing A/T templated bib information

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

Bugfixes: undef handling and legacy script support setting.

The legacy setting was a major bug.  It was not possible to disable legacy scripts
except by *removing* the setting.  That is, if you set it to 'false',
we failed to interpret that here in SIP to mean FALSE.  Instead we
looked at it as a non-zero-length string and therefore TRUE!

This patch also prevents warnings from unitialized values (undef concatenation), like:
Use of uninitialized value in concatenation (.) or string at /openils/lib/perl5/OpenILS/SIP/Patron.pm line 110.

Added a little formatting and whitespace cleanup to address display along w/ undef handling.

Corrected an error screen message also.

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

Checkin overhual for extensions, including item, patron and checkin underpinnings.

Extensions fields added to AUTOLOADable content.

Also fix checkin logic to check for NO_CHANGE and SUCCESS at necessary moments.

Update new code to match berick's #16544.

Add AUTOLOAD to Item and provide a slot to store the raw EG hold object.
Store the hold data down on the item (not the Checkin transaction level).

Allow SIP Patron lookup by usr ID (not barcode).
This is important because we need supplemental data when we find holds info at checkin.
The change is backwards compatible (w/ just 1 arg  it will still be treated as a barcode).

Propagate $inst_id down into checkout core, needed for hold routing logic.

Remove old accessor methods, also make some of the settings checks more flexible w/ regex.

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

Experimental implementation: renewal, block, enable.

Add sip_renewal (not implemented), correct log message w/ join

Patron block message and patron enable

This adds code for enable, still experimental.  It also adds the
client's block message to the patron alert, bracketed by <sip> tags.
The tags are necessary because enable has to be able to regexp out
the block alert message (including user-specified variable text).

Remove unused vars.
Whitespace cleanup and extra feedback data.

Return object on failure, not just 0

Signed-off-by: Joe Atzberger <atz@esilibrary.com>
git-svn-id: svn://svn.open-ils.org/ILS/trunk@16598 dcc99617-32d9-48b4-a31d-7c20da2025e4

Lots of cleanup, logging improvements, and comments.

One arrayref accessor updated to recommended style

Removed unused dependency.

Removed never-possible return statement.

Trying to use an arrayref in the log message results in a string like "ARRAY(0x2541870)".
I added a prebuilt string representation of the array for use in those messages.

More whitespace cleanup.
More flexible to_bool, extra comments;
Also added institution_id method.

Note: OpenILS::Event->new returns a hash, not a true object.

script_x_check.sh is executable.
Irony: this script could have been used to detect itself.

Typo in Cronscript POD.

Signed-off-by: Joe Atzberger <atz@esilibrary.com>
git-svn-id: svn://svn.open-ils.org/ILS/trunk@16597 dcc99617-32d9-48b4-a31d-7c20da2025e4

Basic check for missing DB upgrades

Run this after a update_db.sh failure to see what failed.
Or run in before update_db.sh to make sure you are ready to apply
new updates.

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

Add two little utility functions to convert from intervals to seconds.

M    Open-ILS/src/sql/Pg/999.functions.global.sql
M    Open-ILS/src/sql/Pg/002.schema.config.sql
A    Open-ILS/src/sql/Pg/upgrade/0293.function.interval-to-seconds.sql

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

thinko (unquoted string literal); be more explicity about circ lib comparisons

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

added granularity and user_data options to create_events_for_hook wrapper

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

Improve circ transaction performance by indexing NULL circs (Launchpad 587031)

Creating a unique index on target_copy in the action.circulation table where
the checkin_time is NULL will increase the speed of circ transactions,
particularly on systems with a lot of traffic. The uniqueness also
ensures that a given item will not be able to be checked out multiple times
concurrently.

Thanks to James Fournie from BC Sitka for suggesting the index in Launchpad
bug 587031!

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