Merge branch 'opac-tt-poc' of git+ssh://yeti.esilibrary.com/home/evergreen/evergreen-equinox into opac-tt-poc

more i18n goodness

avoid undef warnings

when authenticated, force the page to redirect the home page after the auth time has expired to protect patron data and  prevent potentially confusing post-search-submit redirects

Merge branch 'master' of git+ssh://yeti.esilibrary.com/home/evergreen/evergreen-equinox into opac-tt-poc

added option to open-ils.auth.session.retrieve to return the full cached object, which includes the auth time (cache timeout) of the cached user object

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

Add a note for future work for full hold count coverage

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

who needs this much noise in the javascript console? javascript.options.strict = false

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

capture credit card order_number

Uunique payment transaction ID for online payments

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

updated cstoreditor to fetch full cached object to get the authtime; capturing authtime in the web context

added option to open-ils.auth.session.retrieve to return the full cached object, which includes the auth time (cache timeout) of the cached user object

added sub to collect the HTML version of a record for display

hold actions work

Renewals working, with feedback displayed

make date format localize-able

pulling in more copy fields for display; added sample code

Merge branch 'opac-tt-poc' of git+ssh://yeti.esilibrary.com/home/evergreen/evergreen-equinox into opac-tt-poc

"Items Ready For Pickup" works

This uses the holds page of myopac with a special param.
Also, tiny bit of JS just to focus the username field on the login page.
Couldn't resist.

Merge branch 'opac-tt-poc' of ssh://senator@yeti.esilibrary.com/home/evergreen/evergreen-equinox into opac-tt-poc

"format" aka item_type (?) search working, I think

Judgment call. I think this just looks better.

make money string localizable

show grocery fines; define date format in a single location

heh, rolling bcak unintended addition of test code

added support for copy paging/sorting

when a secure page is requested insecurely, direct to login page, then redirect to the originally requested page after successful login.  This removes the need for explicit redirect_to and https:// in secure links

fixed holds sorting; my-account link redirects to my-account; i18n cleanup

work on myopac fines, displayed in account summary (myopac/main)

Merge branch 'opac-tt-poc' of ssh://senator@yeti.esilibrary.com/home/evergreen/evergreen-equinox into opac-tt-poc

myopac: more complete display of circs, holds, and fines

not completely complete. still developing.

Move USE * from topnav.tt2 to new header.tt2; also some myopac work

Minor progress in myopac/main and myopac/circs

Enable MFHD tests to find their testdata

When the MFHD tests wrapped in mfhd.t are invoked from outside of the directory,
they tests failed to find the mfhddata.txt file. Judicious use of the __FILE__
special literal and File::Basename::dirname() avoids that problem. Now we just
need to munge @INC to ensure that testlib.pm can be resolved (and hope that
no conflicts ever turn up).

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

localization fixed for plac hold UI

Merge branch 'opac-tt-poc' of git+ssh://yeti.esilibrary.com/home/evergreen/evergreen-equinox into opac-tt-poc

template cleanup;  localizing strings and mild style tweaks; more to come

repiared merge conflict

hold placement works better

moved some code out of the main catloader module into function-specific sub-modules.  Note, sub-modules still retain the OpenILS::WWW::EGCatLoader package name to ease subclassing of the top-level loader object

Teach authority_control_fields.pl about record ranges

Passing --record 10,000 times at the command line sucks, and
--all is not particularly realistic for large databases. Adding
--start_id and --end_id makes this script more usable.

Also provide reasonably full POD documentation, and rely on
Pod::Usage to provide help based on it when help is requested
or passed in options don't make sense.

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

Normalize ISBNs by stripping hyphens in search methods and in reporter.materialized_simple_record

We weren't normalizing ISBNs in search or in reporter.materialized_simple_record
which required users to exactly match the form of the ISBN as entered in the
MARC record to get a successful search result - pretty hit or miss.

The longer term fix in 2.0 and above is to make the ISBN search search against
the identifier|isbn index, which also gets ISBN10/ISBN13 equivalence goodness.
But this patch will be relatively easy to backport through to 1.6.1 and plugs
some of the holes in our exposed APIs and search points in the short term.

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

A start with record detail page

bump xulrunner version

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

Postgres 9.0 is more strict about variable names in functions. We will probably hit more of these, but these in particular break the OPAC.

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

advanced page doesn't need login first

repaired topnav merge conflict

start using a centrally hard-coded ctx.opac_root

the user account "dashboard"/mini-myopac works on every page now

whitespace and minor stuff

Merge branch 'opac-tt-poc' of ssh://senator@yeti.esilibrary.com/home/evergreen/evergreen-equinox into opac-tt-poc

some progress on the place_hold page; ou selector improvements, etc

login form cleanup and implementation of login-failure handling; more to come

added i18n filter support for translating text blocks via filters; more login form cleanup; moved some global USE's into topnav

added support for login via barcode and persistent logins; capturing login failures; use cache infrastructure for fetching/caching cmf objects

Patch from Thomas Berezansky addressing his recent dynamic circ/hold field weighting patch.

Due to an oversight of auto-casting on my part dynamic weighting may incorrectly order similar rows that differ only in org unit or permission group.

Bad me for not testing for that case before now.

The attached patch should fix this. All it does is replace two functions, and in them replaces one INT variable with a NUMERIC(6,2) and a pile of integer constants with numeric constants (by virtue of adding .0 to them).

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

Merge branch 'opac-tt-poc' of ssh://senator@yeti.esilibrary.com/home/evergreen/evergreen-equinox into opac-tt-poc

Merge branch 'master' of ssh://senator@yeti.esilibrary.com/home/evergreen/evergreen-equinox into opac-tt-poc

misc work on the results page, including:

hold placement link,
result prev/next page links on top and bottom,
edition and physical description,
add in previously missing filtersort template

Merge branch 'opac-tt-poc' of git+ssh://yeti.esilibrary.com/home/evergreen/evergreen-equinox into opac-tt-poc

Reduce hard-coded strings in Serial Control interfaces

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

allow item status update if either status-specific perm is allowed or UPDATE_COPY is allowed.  fire off related A/T events after the initial transaction has been committed

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

remove duplicate variable def

Merge branch 'master' of git+ssh://yeti.esilibrary.com/home/evergreen/evergreen-equinox into opac-tt-poc

if no locale config is present in oils_web, assume en_US exists and is valid

hey, maybe I should commit sometime today

results page partly working. woof.

tweak selector used for font_helper() CSS classes

Makes the flat-text MARC editor respect the global font settings,
resolving bug https://bugs.launchpad.net/evergreen/+bug/678078

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

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

Merge branch 'master' of ssh://senator@yeti.esilibrary.com/home/evergreen/evergreen-equinox into opac-tt-poc

home page looks like kcls prod version now rather than dev

basic search "works" inasmuch as it leads to a GET request against the
results page as it should. not that the results page itself works yet.

login page working (just for login, not the extras like "remember me")

ctx.media_prefix for all images and css

Render entities as calls to l(). Goodbye DTD

preparing a could of docs for entitization (fix wrong ent references)

Shove the POC out of the way, move the KCLS-derived stuff in

Big move of css, images, js

Remove a ton of javascript. A little more to come

Never got the place_hold page in until just now

parts/place_hold.tt2 needs css/js attention already seen by other
templates in parts/

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

kill a lot of inline styling, add some more js stubs, fix a bug in advanced.tt2

correct some style= that should be class=

utils changes

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

started moving some inline styles into css classes

Here's the deal so far: all the inline styles are covered by one class
in semiauto.css, but those were created programatically and reflect no
understanding of the relationship between the purpose of any given
element that's styled and what its style is.  These auto-generated
styles also are kind of redundant, since with many of them, instead of
using the style with a long definition, you could use a combination of
smaller styles.

The existing markup uses many slightly different style attributes, so
there's plenty of stuff like "width: 12", "width: 13", "width: 14", etc,
so there's no way to map a lot of this to semantically meaningful names.
some of these classes are just going to end up with names like
"wide-fourteen" that are no better than inline styles.

But others, as we find them manually, can be grouped and renamed into
something meaningful, and sometimes style can be put into prexisting id
and class definitions. Should be a couple more hours work.

semiauto.css reflects progress with the generated classes.

changes to style.css reflect early places where style could be moved
into pre-existing classes.

move temporary kcls format-icon table from mod_perl to template land

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

Merge branch 'opac-tt-poc' of git+ssh://yeti.esilibrary.com/home/evergreen/evergreen-equinox into opac-tt-poc

Integrate Locale::Maketext for string translations

Added support for configuring supported locales and specifying message
catalogs as gettext .po or .mo files.  All locales default to native
template strings when no translation is provided.

Updated oils_web config and added some inline template examples

Merge branch 'master' of ssh://senator@yeti.esilibrary.com/home/evergreen/evergreen-equinox into opac-tt-poc

a thought about the add/remove css funcs

JS stubbed out, mostly. See longer commit msg.

Here's the deal with JS so far.

A few instances of onfocus, onblur, onmouseout, and onmouseover have been
left alone, as they're doing simple cosmetic things.

A lot of instances of onclick remain, and a couple onchange, but these
now either a) just do cosmetic things (hide/unhide) or b) call stubbed
out functions prepared in misc.js

That doesn't mean we won't still totally rip out lots of this JS, but
it leaves us a hint as to what is supposed to happen where.  I can
easily see us losing track of that later.  So anyway, many of the
stubbed functions will probably be replaced with purely server side
functionality later, but until then, they can serve as a kind of
documentation.