Patch from Robert Soulliere to update settings-tester.pl

Changes include:

- Use Test::More perl module to replace eval/die tests for initial tests (as indicated in FIXME comment)
Note/question: should I add an install of the Test::More perl module in Makefile.install for Evergreen prerequisites?

- Add a statement to print postgresql version.

- Add an elsif during "driver" testing to say OK to SIP driver since this is used in telephony section. -- remove scary ERROR

- Add UUID::Tiny to perl module list of checks.
question: More perl module checks needed?

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

Patch from Thomas Berezansky addressing circ/hold config constraints, and their lack of usefulness.

The config.circ_matrix_matchpoint and config.hold_matrix_matchpoint tables have unique constraints that are intended to ensure that you have at most one entry in the table for each set of input conditions.

These constraints operate over a set of fields that can (in whole or part) be set to null, and thus don't do what they are intended to in most (if not basically all) cases.

[This patch] replaces the unique constraints with unique indexes. Nullable fields are coalesced into empty strings so that they can be matched against, and the index only applies to those rows set as active.

The included upgrade script cleans up the tables by taking each set of non-unique rows the unique index would fail on and setting all but the first (in id order) to active=false. If an administrator determines that the wrong row was left active they can toggle active off on the one that was left and back on for the correct one.

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

Remove syntax error from O:A:Storage:Publisher:authority and add strict pragma

Came across a line of dead code while debugging a related problem, and
noticed that strict/warnings wasn't in place, so I culled the dead code
and added use strict/use warnings accordingly (now that we no longer
have the syntax error caused by the dead code).

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

Create authorities that only contain controlled subfields (LP 712496)

Selecting "Create authority" on a field containing uncontrolled subfields
would generate an authority record that contained those uncontrolled
subfields. This would, in turn, prevent the "Validate" button from operating
correctly - if you added a 700 with $a and $c subfields, clicked "Create
authority", and then immediately clicked "Validate", the field would show
up as red.

Now we filter out the uncontrolled subfields before they get sent to
the "Create authority" function, avoiding this validation problem.

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

Ensure new authority ID subfield is inserted in the correct XUL DOM location

Addresses LP 712499. After creating an authority via the context menu
in the MARC editor, the new ID subfield ($0) would be created right after
the subfield on which the context menu was invoked. It turns out that it
was being placed in the wrong location, and one symptom was that the
Validate button would not validate the controlled field against the newly
created authority.

Now we hunt through the parent DOM nodes until we find the 'sf_box' element
and then we append the ID subfield to that node. We also eliminate some
duplicate code by defining a common function so that the problem can be
fixed in one stroke...

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

Prevent authority context menu on whitespace content

Continuing the refinement of LP 712538, if the subfield contains
only whitespace, treat it as though it were empty for the purposes
of the context menu.

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

Prevent authority context menu from being displayed on empty content (LP 712538)

It's confusing to have the authority context menu - particularly the
"Create authority" options - appear for empty content. Check to ensure
that the subfield being clicked on actually has some content before
displaying the authority context menu.

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

Fix ModsParser.pm bug involving invalid ISBNs.  See bug #697398.

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

remove debugging remnant

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

reset "no change" label whenever a specific row in the check-in interface is selected, but for now, only if the transaction has a balance owed

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

ported open-ils.actor.user.transaction.[fleshed.]retrieve to CStoreEditor both for cleanup and to take advantage of .authoritative.  Use authoritative version in staff client

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

portal tweaks; don't link to launchpad

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

make the browser Reload button aware of tab locking for unsaved data

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

clear shelf expire time and some other fields when un-cancelling a hold

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

Two binding fixes for manage items

The first fix is more or less cosmetic.  When selecting a target unit, both areas need to refresh in order to not show the selected unit's items in the main list.

The second fix involves handling cases where moving items from one unit to another does not empty the 'donor' unit.  In those cases, we need to update the contents of both the new and the old unit.

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

Authorities: control 610/611, do not control 4xx by default

The 4xx fields in bibliographic records have not been controlled
fields for some time (1988 for CAN/MARC, 1999 for USMARC per)
http://www.loc.gov/marc/bibliographic/bd4xx.html; and
http://www.oclc.org/bibformats/en/4xx/default.shtm also lists
these as obsolete. So, remove them from the default controlled
fields in Evergreen.

Also, add 610/611 to the controlled fields list (per LP #712467).

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

let due date show up correctly in the copy browser for circulating items

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

allow deletion of stat cat entries in the patron editor (or more accurately, rows in actor.stat_cat_entry_usr_map) by blanking out the value

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

circ.claim_return.mark_missing was superceded with circ.claim_return.copy_status.  It's still lingering in 1.6.1-2.0-upgrade-db.sql, and in other branches.  Thanks to bshum for noticing

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

exit finish_fines_and_voiding() if there is no open circ

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

Apply autogenerate barcode trigger to serial.unit to enable serial checkin

Serial checkin tried to use the '@@AUTO' macro to autogenerate barcodes,
but unfortunately the required trigger had only been defined on the parent
table asset.copy and not on the child table serial.unit. Here we define the
trigger on serial.unit to resolve that problem.

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

make sure that inline overdue fines generation completes before we attempt to void overdues fines in checkin, otherwise it's possible to create multiple services (storage and cstore) competing for the same DB row mid-transaction, resulting in cstore timeouts and rollback of checkin.  This could happen with backdated or amnesty checkins

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

Acq: Improving on r19351, don't let showCreatePane() be issued repeatedly to
fill the window with redundant dialogs, since a) it looks silly, and b) the
dialogs don't work when there are more than one of them. Thanks again Dan.

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

This should get this correct version of postgresql installed for clean
installations on debian-lenny, *if* your system has the backports repo enabled.

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

Acq: make the "new provider" button under Admin -> Server Admin ->
Acquisitions -> Providers not produce a dialog with a ton of mysterious
whitespace. Spotted by Dan Scott.

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

Serial: Little bug in the alt serials control item editor

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

use tabs & spaces consistently, bleh :)

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

In actor.usr_purge_data, default destination user to admin (id=1).

Destination user is where we move things like reports and acq picklists when
purging staff members.  However, if we tried to purge a patron that used to be
a staff member, then destination user was being set to null, which won't work
when updating things like requestor on holds.  The purge UI checks for the
STAFF_LOGIN permission on the user to be purged for determining whether to
prompt for a destination user or not.

Some alternatives to this change might include:
1) always asking for a destination user (and defaulting to admin or the
logged in staff member), or
2) robustifying the stored procedure and surrounding code to raise an
expected exception whenever this edge case hits, prompting the UI to try
again after asking for a destination user.

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

The new, ultra-basic tests caught this syntax error in BadDebt.pm

Yay for ultra-basic tests!

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

Add basic unit tests for Evergreen Perl modules

By "basic", I mean, "Can we use them without provoking a
syntax error?" and the answer is... all but one appear to
be clean, or have a legit reason for not being able to be
used normally. Stay tuned for the culprit...

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

Clean up Module::Build of Evergreen Perl modules

Give OpenILS.pm a head1 POD section to satisfy Module::Build's dist_abstract demand

Remove MYMETA.yml as it is a generated file.

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

Make Evergreen Perl modules installable via Module::Build to match OpenSRF

Build.PL gives us an install-time check on dependencies; right now
the required versions are not set, but we can update these easily
to start catching some of the problematic modules that have tripped
sites up in the past.

Build.PL really seems to want a top-level "OpenILS.pm" so add a placeholder
accordingly.

Adjust references to /src/perlmods/ to /src/perlmods/lib/ even though
a number of the affected scripts are probably cruft.

Use autoconf to provide the default paths in O:WWW:Web and O:WWW:Method;
next step is probably to teach autoconf to ask Build.PL to do that for
us to make the OpenILS Perl modules more independent.

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

Whitespace. gah.

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

Replace a hard-coded path in Cronscript.pm with a configure variable

As much as possible, we need to avoid hard-coded paths if we
ever want to be packagable.

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

Don't attempt to control 4xx fields in records

Per LP bug # 708029, it doesn't make sense to apply an
authority control on a 4xx field; so remove those entries
from the authority tag map.

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

Use the browse method instead of startwith for Manage Authorities, too

Now that the browse method is working properly, adopt it for the
Manage Authorities interface as well so that context around the
search term can be provided.

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

Correct authority browsing for reals

First, restore the >= enable before and after ranges in
authority_tag_sf_browse(), after I mistakenly removed it in
r19131; the second storage request for $after does not
stomp on the prior $before results, it simply gets pushed
onto the carefully constructed list of $before results,
ensuring that our target is in the middle of page 0.

Second, we're treating all of the "tag" members in the
method registration as list references now (for the purpose
of searching against 4xx/5xx in the .refs. variants), but
that was blowing up when we registered just a single tag as
a string and tried to treat the scalar as a list reference.
I could have checked to see if what we had incoming was a
reference and dance accordingly, but opted to just define
all single-tag entries as single-element arrays instead.
Applied the same to startwith.

Finally, in r19331 I had used chop() to ensure that an
exact match for startwith would be returned as element 1 on
page 0, instead of appearing as the last element of page -1.
I had said that the right way to do this would be to naco_normalize()
the value to match the normalized afr.value, and so this is what
I have done. Rather than torturously using O:A:Storage:FTS to get
at the naco_normalize() definition, I moved the function into its
own Utils package and adjusted its usage accordingly through the
affected code. One step closer to single-sourcing the function
in the database, as well?

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

For more predictable authority browse results, use startwith until browse is sorted out

authority_tag_sf_browse() was returning unexpected results when only a small
set of authority records was loaded. This may be an indication of a corner
case, but until we sort that out the more straightforward startwith should
be used.

Also, chop the final character of the incoming search value to ensure
that an exact match is returned on page 0 instead of page -1. For example,
'Bacon, Jono.' becomes 'Bacon, Jono' for the search. (The right thing to
do, come to think of it, is probably to naco_normalize the incoming value;
perhaps next patch).

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

Address typos in Rollover and Propagation dialogue

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

Stay in sync with entity name changes

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

Serials: Add some missing permacrud permissions to IDL

7 classes (sdistn, siss, sunit, sin, sbsum, sssum, sisum) were missing IDL
perms.  Appropriate permissions were added to all but sin (serial_item_note).

For the sin class, since it's several leaps away from anything that points at
an org unit, and since no existing code deals with it via PermaCrud, its
PermaCrud section was simply removed.

Some classes retain open <retrieve /> permissions, and others do not.

No new permissions were created for this. All permissions are reused from
"higher" serials objects.

Credit to Mike Rylander for spotting the omissions.

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

Rename custom.js to custom.js.example.  This file should be under local control, and not trampled

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

Replace the native dump() function with a wrapper function that takes an optional 2nd parameter for log level (defaults to debug)
Log level: 0 None, 1 Error, 2 Warning, 3 Info, 4 debug
A global _dump_level variable gets consulted when the function is used.  The default is 2, Warning.
So by default, until we start changing things, this will suppress most of the existing dump statements.  Some are slipping through the cracks based on when the function replacement happens, but I'm happy with it.  We also prepend the dump msg with a label for the log level.

_dump_level could be locally customized with server/skin/custom.js

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

make the default sdump setup less noisy

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

effectively fixes tab locking with the MARC editor, and makes the previous fix with the patron editor redundant.  bleh :)

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

tweaking the debug output for page/tab locking

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

need to record the schema version in the upgrade script ... spotted by Thomas Berezansky

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

schema version bump required by previous commit

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

Patch from Thomas Berezansky addressing logical importance of various circulation and hold matrix matchpoint components.

Previous to this commit, INDB circ and holds use a pre-defined weighting set for rule ordering. This can be changed via replacing the relevant "find" functions in the database, but this is not easily done for most people.

The weight set for circ matchpoints is obtained based on the context ou of the circ (aka, where the circ is happening).

The weight set for hold matchpoints is obtained based on the item's circ library (aka, where the item lives).

Optionally, add an enabled circ.holds.weight_owner_not_circ internal flag to have the weight set for hold matchpoints be obtained based on the item's owning library (owner of the call number).

TODO: discuss promotion of circ.holds.weight_owner_not_circ to a Global Flag; wikified or docbook'd documentation and use-case examples.

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

Make window.print() configurable.  The "Mozilla Print" strategy is internally identified as 'webBrowserPrint', and it uses XPCOM for printing the contents of an HTML document.  However, in some cases we override this with a 'window.print' strategy to workaround bugs.  A problem with this has been that an actual bare javascript window.print() does not honor any of configured "gPrintSettings".  However, it is informed by (and in turn informs) a Mozilla preference called print.print_printer.  Right now window.print (if used from within the JSAN util.print library) only gets called in when in the Default printer context/role.  So we force print.print_printer to equal the configured printer for the Default role.  Have I mentioned that I hate printing in Mozilla?

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

enable the checkbox column so we can delete hard due dates

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

Serials: add "clone subscription" functionality to alt serials control

This feature clones subscriptions and all the objects related to them
that don't describe holdings.  Should help speed users' workflow when setting
up subscriptions.

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

patch to edi4r

This is a patch to edi4r which has been submitted; sticking
it here temporarily since at the moment it's unclear whether
edi4r still has an active maintainer.

This patch fixes a bug where question marks in the bibiographic
description ended up being escaped three times over.

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

remove redundant finalize

Already handled by OpenILS::Mapper, and has the effect
of making a parsing bug in edi4r worse by causing
question marks in the EDI data to be escaped multiple
times.

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

set the svn mimetype for these

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

Staff client portal page replacement by Michael Peters, with a few tweaks.

Thanks!

Developer's Certificate of Origin 1.1

By making a contribution to this project, I certify that:

(a) The contribution was created in whole or in part by me and I
    have the right to submit it under the open source license
    indicated in the file; or

(b) The contribution is based upon previous work that, to the best
    of my knowledge, is covered under an appropriate open source
    license and I have the right under that license to submit that
    work with modifications, whether created in whole or in part
    by me, under the same open source license (unless I am
    permitted to submit under a different license), as indicated
    in the file; or

(c) The contribution was provided directly to me by some other
    person who certified (a), (b) or (c) and I have not modified
    it.

(d) I understand and agree that this project and the contribution
    are public and that a record of the contribution (including all
    personal information I submit with it, including my sign-off) is
    maintained indefinitely and may be redistributed consistent with
    this project or the open source license(s) involved.

Signed-off-by: Michael Peters <mrpeters@library.in.gov>
git-svn-id: svn://svn.open-ils.org/ILS/trunk@19305 dcc99617-32d9-48b4-a31d-7c20da2025e4

Swap item/issuance labels in Subscription tab prediction

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

Clear out some unused code, which in some cases was causing "command bleed"

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

Fix broken stream functions, re-enable commented-out menu with correct options

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

This menu was for a different tab (subscriptions), comment out for now

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

fix the svn:mimetype properties for these

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

mark this as binary

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

Translation string in manage_dists.js : right problem, wrong fix

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

Fix checkin backdate assumptions

checkin_handle_backdate() did not properly interpret bill data (voided too much), so lets use void_overdues() instead

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

Tweaks to transparency and hue, courtesy of Michael Peters.  Thanks!

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

Small serial interface string fixes

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

Point to Fedora 14 as the preferred Fedora version

Fedora 13 will be out of service relatively soon; give
2.0 some legs if someone goes with Fedora.

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

Update README to include --admin-user / --admin-pass options for eg_db_config.pl

As the default Evergreen administrator account is now created with
a randomized user name and password, we have added the options to
eg_db_config.pl to reset those to something known and specific to
a given instance.

Also, Fedora 13 is getting close to retirement, update to Fedora 14 for
supported distros.

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

Teach the i18n Makefile how to handle serial.properties

Also check in the POT for serial.properties and some other
foobar.label matches for foobar.accesskey (does not have
an effect on properties files but hopefully if we're
consistent then the pattern will be evident for entities).

Noting in passing that there are a number of "foobar(s)"
comined singular/plural labels that should be split to
support translation.

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

Replace a few hardcoded strings in the serials JavaScript

There is, unfortunately, an awful lot of this left to fix.

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

Fix existing serial entities that didn't provide .label partners for .accesskey

The translate-toolkit project generates a POT file from an input DTD
that requires entities ending in .accesskey to have a corresponding
.label entity; this is a Mozilla convention for localization. Fix up
the serial interface strings that are localized to match this expectation.

Also remove two duplicate entities in lang.dtd that were flagged by
the build/i18n/tests/check_entities.py script (and which cause Launchpad
translation imports to break).

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

Expose (in a very basic way) existing serial item reset functionality, fix small bug in reset code (resetting an item with no unit)

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

Correct for over-zealous sdist editor field display code

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

Serials: A batch of improvements to the caption/pattern wizard

- Using a numeric frequency ($w) now pre-selects the regularity page
- Switch the regularity page from a box layout to a grid layout for legibility
- Simplify and unify month/date control pairs, enforcing correct limits
    on days in each month
- Scrollbars!
- If the user creates $y data in the regularity page, but then unchecks the
    "use specific regularity information" box, the $y stuff will be correctly
    excluded from the compiled pattern code.
- "Display in Holding Field" replaced with text that better explains what it
    means
- The wizard's dialog window has a title now
- In alternate serials control -> subscription details -> captions/patterns tab,
    if the user has already typed something in the Pattern Code field, they
    now get a warning that using the wizard will erase their existing work
    if they click the Wizard button
- The captions/pattern tab now treats pattern codes as immutable once created,
    which is what was apparently intended from the beginning.  See
    http://list.georgialibraries.org/pipermail/open-ils-dev/2010-May/006079.html

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

expose the print.always_print_silent preference in the Printer Settings Editor

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

make the Printer Settings Editor immune to the print.always_print_silent preference

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

Fix for spurious warning (LP bug #707114)

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

i18n support for a few OPAC strings

Enable translation of the "More copies listed in full record details"
message for the search results library / call number / item /status
lines

Also provide translation support for the hold queue status message,
including singular / plural variants (most languages do not provide
the equivalent of the "item(s)" idiom in English).

Instead of burying the display of hold queue status with a hard
coded "if (false)" test, turn it into a top-level variable for
a bit more exposure.

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

correct XPath attribute test syntax

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

handle deleting line item from selection list more carefully

Once a line item is attached to a purchase order, removing it
from a selection list should mean detaching it from the list,
not deleting it outright.

The open-ils.acq.lineitem.delete method can be removed
once the old jubgrid code is finally purged.

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

more overzealous unsaved data prompts

onKeypress catches all sorts of stuff, like the control+w for closing tabs.  Not good.  Ideally we'd test event.isChar, but that's broken in Mozilla:
https://bugzilla.mozilla.org/show_bug.cgi?id=312552

Instead we look to see if control, alt, or meta are being held down.  Not perfect, but good enough for now.  For example, won't ignore tabs for jumping from field to field, and won't realize the hotkey for clipboard pasting should trigger the unsaved data state.

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

Protect against volumes with many copies attached

Bill Erickson has good data for testing these cases.

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

Use holdings_xml-full for the copy display

Less to munge and transfer over the wire = win win win!

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

slimpac issue where deleted copies are visisble, spotted by Steve Callendar and Jason Etheridge

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

Adopt the Conifer local call number display code as an example of complex client-side customization

It's useful, too - library / call number / copy location / copy status on
search results, and easily tweaked from the out of the box defaults.

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

fix spurious unsaved data prompt with repeated use of same patron editor (only set tab lock once with patron editor, since we only unlock once upon save)

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

Part 3 of a more secure default set up

With this commit, the hardcoded default barcode is replaced by an
MD5 hash of a random string of numbers, preventing the barcode from
being used as a known login name in the OPAC interface (which would
remove one of the factors required in a brute forcing of the account
credentials).

You probably don't want to change the barcode for the admin user,
but if you feel the need you can use the usual patron editor in
the staff client.

Thanks to Thomas Berezansky for suggesting this additional change.

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

Acq: the "Load Catalog Record IDs" interface failed on files produced in
Windows due to not stripping out carriage returns. Fixed.

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

Acq: use the acqlimad table instead of its ancestor acqliad table to
populate a dropdown for the Export Single Attribute List function of lists
of lineitems

The acqliad table may appear to have duplicate entries since it's a parent, and
the point of the function that's trying to use it is just to export lists of
ISBNs or UPCs, so acqlimad is a better fit.

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

don't offer to cancel related transits that have already completed when canceling holds

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

Booking: these lines don't belong; they were erroneously copied and pasted
from a neighboring section of the file

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

whack-a-mole, was getting a null instead of an empty array here, but may be a red herring

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

if this user does not have permission to edit an existing user, disable the save and clone buttons

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

remove disallowed permission groups from the perm group selector in user registration; see bug 699914

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

let amnesty checkin mode trump the restore overdues on lost checkin setting

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

Booking: Robert Soulliere spotted and fixed a bug in processing reservation-
related overdue fines.  See https://bugs.launchpad.net/evergreen/+bug/705061

This patch tests successfully for me. Thanks Robert!

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

Build and flatten a tree, correcting a sorting issue in some OU dropdowns.

The previous code assumed that work org units would be delivered in hierarchical order, but alas, they are not.  Thus, we build the hierarchy and then flatten it, sorting at each level.  This will be non-fast with many work OUs, but the common case is a small set, which is not painful.

Further improvement is warranted when the above proves false.

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

bug #684467: more bulletproofing of naco_normalize

Also preparing for backporting to rel_2_0.

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

drop name check constraint on config.hard_due_due

See https://bugs.launchpad.net/evergreen/+bug/704951 for context
and possible future direction.

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

Avoid escaping issues in authority.normalize_heading() by parameterizing the query

Long story short: MARC subfield values containing backslashes caused noise
and in some cases painful errors. Using spi_prepare/spi_exec_query is the
safest way of handling escaping, rather than adding more regexes and munging
the data before it even gets to naco_normalize().

Most painful case was <subfield code="a">Foo, Bar\</subfield> - the trailing
slash ended up escaping the enclosing single quote (because PostgreSQL isn't
configured by default with strict conformance to SQL escaping rules yet) and
threw an error.

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