Cut a long complicated function in two

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

minor error checking

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

api call for canceling and reseting trigger events

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

use the event def template for the bill note (thanks, miker) instead of using a param.  Created a system billing type of Notification Fee, also so none is needed via event param.  Added first billing type const (yay).

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

repaired event_params sequence config

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

added seed data entry for ApplyCircFee reactor

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

reactor to apply fees to a circulation

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

fixes the missing xulG.new_tab issue.  The problem was nested browsers/progress listeners

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

wrap authoritative rollbacks in a try so that a failed rollback won't leave a stale editor in the cache

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

added hook for lost.auto to handle items programatically marked lost (e.g. too long overdue).

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

safari does not comply with cookieManager's requests to remove the cookie.  dojo.cookie works in Safari.  Fixing the crucial cookies now.  more later.

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

In searchINPredicate function of oils_cstore.c: tighten
input validation.

1. Require that the operand of IN or NOT IN be either a
JSON_ARRAY or a JSON_OBJECT.

2. Require that an item in an IN list be either a
JSON_STRING or a JSON_NUMBER.

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

Really revert these files. Argh.

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

Revert accidental commit of record details enhancements; not yet approved

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

Now that we have localized org trees, let our tools pull them in
Note that for now I've cheated on the staff client chrome and book bags and forced them to en-US

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

initial actions dropdown.  some style tweaks

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

fixed label for consistency

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

Fix this missing getString() call, too.
dbs--

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

missing .getString.  This was breaking the spine label print preview

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

Mostly mplement issues with combined dates (eg, May/June). Still don't do combined years (2000/2001)

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

more layout tweaks

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

no need to special case the text widgets

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

Casting a null into a String results in a string with a value of "null"; change test accordingly

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

some display cleanup

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

In oils_cstore.c:

1. Verify that the BETWEEN operator receives
exactly two operands.

2. Validate the operator used in a simple predicate;
i.e. make sure it contains no semicolons or white space
(with the exception that "similar to" is allowed).
Purpose: prevent certain kinds of SQL injection.

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

don't set ischanged if value is not changing, otherwise all copies are always updated

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

let the _impl caller handle transaction commits

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

In searchPredicate(): make sure that there is only one
predicate at a time for a field (currently we silently
ignore anything but the first one).

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

no need to flesh fund, since we're grabbing those anyway.  set ischanged when item changes

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

added a copy (li_details) pane for creating a managing copies

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

added batch lineitem detail CUD method

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

this.user might be an event, check for that

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

In oils_cstore.c:

1. Applied the const qualifier to several parameters.

2. Added some return checking to ensure that errors bubble
up to the top.

3. Complain if an IN list is empty.

4. Complain if a predicate is empty.

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

added some user sesssion and object-list caching

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

added basic copy list grid (lineitem_details) linked from li table.  other small tweaks

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

pcrud entries for fund.  added some selector attrs and fixed an org_unit link

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

progress bar to show lineitem update progress.  some small style cleanup

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

added simple show/hide funcs

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

adding paging plugins to li table.  showing entry count in picklist view.

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

when searching bibs, don't start the cstore xact until after the results have returned to prevent connection timeout (and useless pinning of backend)

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

set state explicitly so caller will have the state in the retrieved LIs

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

Implement a NOT operator, expressed as "-not".

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

plug in li table to picklist display

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

reset table before next round of search results

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

display localized labels for li attrs. added button to return to li list

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

added attr def fetching function

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

started lineitem info pane, which shows marc record and (eventually) all lineitem attrs.  will also link to li_details

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

some lineitem table style

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

display:none implies invisible.  remove the visibilty part so un-hiding is simpler

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

moved lineitem table out to generic template for import.  this will eventually take the place of jubgrid in most cases.  still need to plug some final holes for basica functionality

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

added creator/editor to general picklist create

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

Support multiple volumes in a single year

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

updated IDL to match acq creator/editor columns

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

fleshing new creator/editor tracking fiels in PO, picklist, and lineitem

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

plugged in Bill Ott's lost item checkin functionality.  This includes the ability to void lost item fines, processing fines, and un-voiding existing overdue fines on the transaction when an item previously marked lost is checked in.  lib can also define a post-due-date interval during which these settings take effect.  Each are controlled by org-unit settings.  I still need to add Const.pm entries for billing types.  Thanks Billsvn diff Open-ILS/src/perlmods/OpenILS/Const.pm Open-ILS/src/perlmods/OpenILS/Application/Circ/Circulate.pm

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

better handling of sets of work_ou's.  will probably shove some of this logic into User.js at some point.  Added support for UPDATE_ORG_UNIT_SETTING_ALL perm checking

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

added new on-lost-return event org unit settings

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

incorrect table name in stored proc for updating materialized transaction summary

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

oops, wrong api name.  fixed

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

Correct a blunder (trying to free the same memory twice)

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

added method to return trigger events by user and type

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

pass hint instead of class as core_type

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

hide magical ##URI## call numbers from the copy browser in the staff client

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

provide for a editing audit trail on picklists, POs and lineitems (JUBs)

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

removed the old, convoluted work_perm_org code.  Now using the stored procedure for calculating range of influence.  updated api name to hopefully be more clear

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

For usr_has_perm* functions: super_user has all permissions
everywhere (even permissions that don't exist).

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

more cleanup, plugged in get-selected operation, so back to original functionality

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

Add several functions for identifying the org units for which
a given user has a given permission.

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

In oils_cstore.c:

1. Restore the quoting of numeric literals where the associated
column is known to be textual.

2. Wherever possible: replace calls to jsonObjectToSimpleString()
with calls to jsonObjectGetString().  Each replacement saves a malloc
and a free.

3. At one spot in searchWHERE: detect and complain about input where
the class name is empty.

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

changed cookie base to the more generic oilsBasePath

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

move away from jubgrid in bib search for greater flexibility of display.  stream LI's from search call for faster grid population.  swap form and results display for better screen use

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

provide option to return new lineitems and associated attrs from search call

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

Mark tests as implemented, now that volume changes based on date work.

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

Add code to support setting volume number based on date rather
than number of issues per volume.

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

due to some odd interactions with IE and dojo and the fact that opensrf.js is not dojo-ized, load opensrf.js in the usual fashion first.  may want to do this for all non-dojo-ized js files, but so far, loading this file alone seems to fix ie6

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

first pass at MFHD test suite

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

minor cleanups to calculating volume numbers

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

Apply r12520 to version of MODS3.3 XSL stored in database - thanks miker

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

Minor stylistic cleanups

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

Check in revision 1.27 of MARC21slim2MODS33.xsl (for 648 -> <subject> mapping)
Note: 242$i change was not mentioned in MODS mapping revision notes; was this
    a local Evergreen customization?

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

These now validate with XML schema:
   xmllint --schema docbook-5.0/xsd/docbook.xsd --xinclude --noout index.xml
A slash at the end of the XLink namespace declaration is no good; whoops.
And the placeholder chapters wanted more content. Fine, throw in some index terms.

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

bump template version to reflect new join choice functionality

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

Fix two bugs that I introduced into the Google Preview support code

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

apply proper I18N-ification to new template builder functionality

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

Pure whitespace, nothing more

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

Typo: Anscestor -> Ancestor

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

Remove generated POT files to avoid confusion

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

adding "Enable nullability selection" switch to allow complete control over join types in templates

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

Further tightened input validation.  In all cases where we call
searchWHERE(), we now respond to a NULL return by propagating
the error up the calling chain (instead of generating
defective SQL).

Also: replaced one call to jsonObjectToSimpleString() with a
call to jsonObjectGetString(), avoiding a malloc and free.

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

Tightened the input validation in searchWHERE().  It now complains
about an empty JSON object or empty JSON array, instead of
constructing a doomed WHERE clause.

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

teach parallel_pg_loader.pl to set the sequence on non-auto tables after loading data into them

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

Redo MFHD, MFHD::Holding, and MFHD::Caption so that they are enhanced MARC::Record (for MFHD) and MARC::Field (for the other two) objects.

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

In searchWHERE(): plugged a security hole that invited SQL injection.

The use of a leading plus sign was originally intended to allow
references to boolean columns in a WHERE clause, without requiring
an explicit comparison to true or false.  E.g. "WHERE col" instead
of the more prosaic "WHERE col = TRUE".

However the old code worked by simply concatenating unsanitized
strings, leaving the door open for SQL injection.

The new code attempts to verify that the last string to be appended
looks like an SQL identifier, with no extra SQL syntax.

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

fix PATRON_BARCODE macro in bills_main_view template by sending fleshed usr

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

fix for Cancel in patron editor for existing patron

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

Enable slimpac and dynamic OPAC to have localized org trees

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

Enable get_org_tree to return an org tree with a locale based on a locale code,
rather than a reference to a variable :)

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

teach clark how to use a separate state_store db connection for recording report metadata

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

When inserting a literal value into a SELECT statement:
whenever possible, leave the value unquoted if it is known to be
numeric, i.e. it is carried as a JSON_NUMBER, regardless of the
datatype as inferred from the associated column.  Reason: so that
the test_json_query utility (which currently doesn't look up the
datatypes of the columns) can generate the correct SQL most of
the time.  This approach should also be slightly faster, since it
bypasses some hashed lookups.

2. As part of the implementation of the change described above:
combine searchSimplePredicate() and searchWriteSimplePredicate()
into a single function, so that the JSON type is known when it's
time do decide whether to add quotes.  This change is benign because
the latter function was called only by the former anyway.

3. Several minor rearrangements and optimizations.

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

forgot to decrement a counter for the chunk size.. since performance was pretty spiffy with the bug, this probably argues for using larger chunk sizes

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