You can now have multiple rows in your MARC Expert Search

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

MARC expert search working

This just needs the JS row cloning so you can search on multiple terms
at a time.

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

It's ugly and hacky, but numeric search fully works (item barcode included)

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

Numeric search mostly works (except for item barcode)

Expert search is next

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

Don't show users who are already logged in a login form; that's confusing

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

Confirm CC payments with another page load, not confirm() popup

This does something gross. Your CC card number is rendered in a hidden
field on the last confirmation page.  At least it's all SSL only.

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

Show most up-to-date email address in update-email interface

plus cosmetic change to payment mini table

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

CC payment improvements:

Link from receipt after payment back to account summary.

Error reporting

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

Better to tell the user something rather than nothing

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

Remove ? icon by "Security Code" in payment form

It can go back in later if we want to make it actually do something.

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

Give the patrons the opportunity to see and change their e-mail address...

... before they submit fine payments.  This way they have a better
chance of getting a receipt e-mailed to them.

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

Persistent logins are more of a "stay logged in" feature than a "remember me"

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

Links to return to payment history from receipt print/email pages

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

Select-all checkboxes for fine payment

In the process, had to change the checkbox/cgi param name of the misc
fees to xact_misc and tell the middle layer to lump xact together with
xact_misc.  Should test carefully.

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

Empty case for payment history

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

YAOUS to limit OPAC payment history display to configured interval

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

Merge branch 'rel_2_1' of git.evergreen-ils.org:Evergreen into template-toolkit-opac-rel_2_1

move query params into the constants to better support current opac and TT-opac

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

Load embedded OPAC via SSL by default, w/ override option

* url_prefix option for using/forcing SSL.
* oils.secure_opac preference for doing such with the embedded OPAC

--------

To use the preference, you may want to include something like this in server/skin/custom.js:

// Force non-SSL for the OPAC
try {
    netscape.security.PrivilegeManager.enablePrivilege("UniversalXPConnect");
    var prefs = Components.classes['@mozilla.org/preferences-service;1'].getService(Components.interfaces['nsIPrefBranch']);
    if (!prefs.prefHasUserValue('oils.secure_opac')) {
        prefs.setBoolPref('oils.secure_opac',false);
    }
} catch(E) {
    alert('Error in custom.js trying to set oils.secure_opac preference to false: ' + E + '\n');
}

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

Metarecord list on results page

Added a new ?metarecord=<mrid> param to opac/results page to list
records associated with a given metarecord ID.  This is used as page 2
of the (pending) metarecord search process as well 'show in catalog'
for embedded staff client metarecord holds.

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

minor I18N text wrapping

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

Ubuntu Lucid needs MARC::Record from CPAN

As Ubuntu Lucid ships with MARC::Record 2.0.0, and Serials requires
2.0.1 or greater, we must install MARC::Record from CPAN or face failing
open-ils.search requests.

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

LP#813151 Copy Location sort order in Advanced Search

This patch addresses the unintuitive copy location sort order in
Advanced Search when searching at a specific branch. Thanks to
Dan Wells for the code suggestion which changes the order in the listbox
to alphabetical sort by asset.copy_location.name.
Tested, and confirmed working in 2.0.4 test and production environments.

Signed-off-by: Michael Peters <mrpeters@library.in.gov>
Signed-off-by: Thomas Berezansky <tsbere@mvlc.org>

NOVELIST/CHILIFRESH include statements missing a hash which was causing (none) errors.

Signed-off-by: James Fournie <jfournie@sitka.bclibraries.ca>
Signed-off-by: Thomas Berezansky <tsbere@mvlc.org>

adjust legacy xul column definitions for dates/timestamps to ensure correct  sorting

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

remove circ.util.std_map_row_to_columns; default to the one provided by list.js

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

Better sort support for xul lists (see launchpad bug ://bugs.launchpad.net/evergreen/+bug/751745)
Example of supplying a sort_value function for item Create Date, and automatic sort_value for fm_columns-generated timestamp and money column definitions.
Need to provide more such functions, and eventually move everything over to fm_columns

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

Manage parts holds warning (and removal) w/ CSS

Use add/removeCSSClass to apply and remove holds parts selector
warning instead of inline style.border management, which (when using
'auto') causes problems in IE.

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

sort hold Queue Position column correctly in xul lists

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

Comment out line that breaks holds placement in IE

auto is not a valid option for a border, and IE faults on it.
Other browsers seem to ignore it, as far as I can tell.

Signed-off-by: Thomas Berezansky <tsbere@mvlc.org>
Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>

More better call number browser, uses supercat directly, doesn't use mvr
Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>

Also was missing the author search link

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

Call number shelf browser: fix prev/next links, wrong argument order in...

... call to open-ils.search.callnumber.browse (count and offset mixed
up)

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

Shelf browser!

Needs improvements: can't find jacket images by UPC yet

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

Cleaned up some file merge funkiness

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

More power to present friendly messages when hold placement fails

This area could use a little cleanup, elegance, but it can give end
users better messages for a diverse set of reasons why hold placement
failed.

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

Cleaned up some back-port merge mixups

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

Fix the wrongish way we were doing staff-assisted holds

Now the patron that a staff user wants to place a hold for is passed
around as a cookie

It's a little unclear when is best from a workflow perspective to kill
said cookie... hrmmm....

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

Alert or block holds on available copy at same pickup lib

This improves upon the earlier commit that attempted to deal with what
to do when either of these org_unit settings are true:

     circ.holds.hold_has_copy_at.alert
     circ.holds.hold_has_copy_at.block

The correct use of these settings are that when /any/ type of hold is
attempted at a given pickup lib, if either of those settings are true in
the context of said pickup lib, and there are copies in an available
status at the pickup lib already, inform the user before placing the
hold.  In the alert case, the user can place the hold anyway with
another click.  In the block case, there's an end on't.
Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>

Allow overriding some hold events when permissions apply

Works fine now for HOLD_EXISTS and HOLD_ITEM_CHECKOUT_OUT, at least

But this was also supposed to work with these org settings:
 circ.holds.hold_has_copy_at.alert
 circ.holds.hold_has_copy_at.block
which are supposed to define behavior when the user wants to place a
(copy- or volume-level only?) hold on an item at a branch that's not his
home_ou but his home_ou has an available copy.

Anyway, the open-ils.circ.title_hold.is_possible method is supposed to
return information about whether local copies are available to support
such a function, but it doesn't seem to do so.  I think.  Will revisit.

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

Preset correct default pickup lib for staff placing holds for patrons

Also, for hold placement, only enable org unit selection where
can_have_vols is true.

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

copy and volume level holds for staff

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

When coming from staff client patron interface,

pre-populate the patron barcode text box by which staff places a hold
for a patron.

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

Staff can now place title-type holds for patrons

Still need to put in copy, volume and issuance (and parts? maybe later) -type
holds.  Still need to deal with overridable events.

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

Wrapped up Novelist integration support

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

Load Chilifresh content after page load

Execute the CF JS after other inline JS has run.

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

Removed duplicate 'extras' tab

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

Avoid mixed content warnings on embedded content cafe

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

Support for embedded Content Cafe on details page

Added support for two new Apache environment variables
OILS_CONTENT_CAFE_USER and OILS_CONTENT_CAFE_PASS.  If set, a Content
Cafe pane will be loaded (via iframe) directly into a new "Summaries &
More" section on the record details page of the TT-pac, when the
section is expanded.  In addition to the C. Cafe content, we also show
the 520 bib summary in the same pane.

"Summaries & More" is only visible if either there is 520 data or there
is a suitable identifier (isbn or upc) and the content cafe
configuration is in place.

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

Meaningful title elements for record details and search results

Rather than the generic "Record Details" and "Search Results", populate
the <title> element with the actual title of the record or the search
query, respectively.

We punt on advanced search results for now, until we can parse the array
into something usable.

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

Showing hold/copy summary in details page

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

Chilifresh reviews for results and record pages

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

Pass ENV down to template environment

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

Added Chilifresh secure url to sample apache config

added notification of successful settings update to patron prefs page

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

honor hit count user pref on results page

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

Initial support for updating user prefs

Starting with hits-per-page and enable/disable circ/hold history

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

added read support for holds history setting

Read support for some user settings in setting page

Search hits per page and circ history opt-in displaying

TODO: add more settings
TODO: implement POST/write for updating values

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

Select all fines for CC payment by default

Format selector improvements

Consistent with format icon fall-thru, try mattype first to locate the
list of formats for search selection.  If no mattype configuration is
found, fall through to item_type.

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

cleanup:

1) fix broken INCLUDE statements in circ and hold histories

2) move an opac "part" to the parts directory

empty cases for fines

Emailing receipts working, some tweaks to payment history page

Receipt printing also from payment history

Credit card payments working with printable receipts

Could still use e-mailed receipts

begin a sub to process payment

doh, it's history not location.history

remove in-template fine-totaling: mod_perl code was already doing the same

also:
    make that fine-totaling not use floating point math.
    don't provide payment form for <= 0 totals (we're not issuing refunds)
    last copyright date in footer always current year, not hardcoded

support paying all fines at once

Payment for mostly done. Still needs:

1) to be able to handle the upper "Pay Fines" button that will mean "all
my fines"

2) to use fixed point fine totaling math and bail on total <= 0

3) ... anything else?

start on cc payments

Fix an ugly grey bar in the record detail page

I have no idea what caused this or why this css adjustments fixes the
problem

Better redirect from place_hold

Before this, if you click place hold on something while not logged in,
you are redirected to the login page, which is good, and redirected
thence to the place hold page, which is good. But after actually
placing the hold you got redirected to the login page again, which is
not good.  Now you get sent back to the home page.

This may not be the ideal choice.  myopac/main may make more sense.
Also, there is still not any clear visual feedback for the user that
their hold placement succeeded.  That'll need to be addressed too.

serial issue display and holds work

removed unnecessary kcls record summary template file override

Clean up the rest of the non-entity ampersands in URLs

Continue the global change to entity ampersands in URLs
in the TT OPAC. This should be the last of it.

Signed-off-by: Dan Scott <dan@coffeecode.net>

Avoid empty ID attribute in SELECT element

The current code appears to always generate an empty id="" attribute,
as the "id" variable never gets populated. For now, protect against
the empty ID attribute.

Signed-off-by: Dan Scott <dan@coffeecode.net>

HTML compliance in result table

Convert & to &amp; in URLs, and change a SPAN element to a DIV
(because SPAN elements are not allowed to contain block-level
elements like DIV and TABLE).

Signed-off-by: Dan Scott <dan@coffeecode.net>

Less aggressive URL encoding

We were double-encoding URIs, as the url and uri filters in
TT happily encode % - which is fine the first time around,
but after you've already escaped everything as %nn the propagator
was serving it back to url to be escaped again on the following
page request.

The right way to do this might be to unescape the incoming query
string, then pass it on to url for escaping again - but for now,
using the entity version of & is good enough to begin with.

Signed-off-by: Dan Scott <dan@coffeecode.net>

Ensure ampersands in URLs are &amp; per HTML spec

The propagator variable defined in parts/header.tt2 holds the value
of the CGI query string - which, as it comes off the wire, is a simple
"&". The problem with this is perhaps best explained at
http://www.htmlhelp.com/tools/validator/problems.html#amp (and it
explains why "&copy_..." gets converted into a copyright symbol).

By passing the incoming query string through the TT url filter and
then replacing "&" with "&amp;" we avoid this problem at the source.
From there, we have to address the other locations in the code in
which we are manually appending GET variables.

Signed-off-by: Dan Scott <dan@coffeecode.net>

Fix unclosed quoted attribute in TT OPAC footer

Forgot to place the closing attribute quote on the footer image.

Signed-off-by: Dan Scott <dan@coffeecode.net>

Debrand the KCLS-specific hint about Library Elf in prefs settings

Even though prefs settings are not yet exposed, we can turn the
KCLS-specific comment into a brandable "hints" section in prefs
settings.

Signed-off-by: Dan Scott <dan@coffeecode.net>

Make refund policy for lost items a branding option

Different sites will have different policies for refunds
on lost items (and might not even provide refunds for lost
items, for that matter), so break out that policy description
into its own template and provide a KCLS-specific version of
it.

Signed-off-by: Dan Scott <dbs@coffeecode.net>

Debrand lowhits purchase request reference to KCLS

While we break this out to a separate template, perhaps we
could pull the library name from context instead of having
it hardcoded. It will probably depend on each instance, ergo
hardcoding the generic "your library" for now should be fine.

Signed-off-by: Dan Scott <dbs@coffeecode.net>

Pull call number from XML holdings, not MARC record

KCLS uses call numbers in the MARC record, but most other sites
use the call numbers from asset.call_number. For now, just display
the first returned call number; eventually the display will probably
need to be adjusted to handle call number / shelving location / library
/ status like the current AjaxPAC does.

Also note that ##URI## call numbers should be filtered out but are
not yet.

Signed-off-by: Dan Scott <dan@coffeecode.net>

Debrand the login form

Breaking the password hint and login help sections out into
separate templates enables sites to adjust the hint and
login help links without touching the rest of the interface.

Signed-off-by: Dan Scott <dbs@coffeecode.net>

Debrand the OPAC footer, home page, and topnav links

Restore the copyright message and Powered by Evergreen logo
in the footer, and provide obvious example links instead of
the KCLS-specific links.

Also, replace the KCLS home page with the main Evergreen logo.

Signed-off-by: Dan Scott <dan@coffeecode.net>

Create a templates_kcls to hold KCLS branding of TT OPAC

The debranding of the TT OPAC begins with the top navigation.
  * Replace the KCLS logo with the Evergreen logo (even though
    white background on black doesn't look great, it's something)
  * Replace the KCLS specific top links with example links.
  * Split the logo and the links sections out into separate
    templates so that we can maximize commonality of the
    topnav.tt2 template.

For now, all files containing the string 'kcls' have been
copied into the templates_kcls directory. As we move through
the templates, we may refactor them in similar to how we just
refactored topnav.tt2.

Signed-off-by: Dan Scott <dbs@coffeecode.net>

Internationalize more of the TT OPAC (records and results)

There are a number of hardcoded strings remaining in the TT OPAC.
This is just some of the lowhanging fruit; more work needs to be
done on the more complex cases involving HTML and placeholders.

Signed-off-by: Dan Scott <dan@coffeecode.net>
Signed-off-by: berick <berick@esilibrary.com>

Enable unAPI support in TT OPAC

We have enough information available from the context and date
plugin to construct valid unAPI IDs - so let's do it and take
one more XXX off the table. (This is good for Zotero support!)

Sadly, Zotero doesn't seem to recognize more than one unAPI
ID on the search results page; in theory it should give the
user the ability to add all of the search results at once,
but in practice it doesn't display anything.

At the same time, we can remove the duplicate unAPI placeholders
and remove unnecessary <span> elements.

Signed-off-by: Dan Scott <dan@coffeecode.net>
Signed-off-by: berick <berick@esilibrary.com>

ContentCafe summary of avaible item content

Added "available_json" support for Content Cafe.  Returns and array of
available content for a given item, where the content type is mapped to
one of the Evergreen standard added content types, e.g. "summary",
"anotes", "reviews", ...

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

use medium-sized jacket image on record details page

Add excerpt AC and avoid rate-limiting for OpenLibrary content

OpenLibrary added rate-limiting (100 cover requests per 5 minutes)
for cover images requested by ISBN; using the OpenLibrary book
API to pull the list of direct cover URLs avoids the rate limit.

Also, add excerpt added content calls for OpenLibrary.

Signed-off-by: Dan Scott <dbs@coffeecode.net>
git-svn-id: svn://svn.open-ils.org/ILS/trunk@20354 dcc99617-32d9-48b4-a31d-7c20da2025e4

force empty value for the default 'Any' option to avoid using the option text as a valid value

generic filters in advanced search using SVF

incidentally, this reveals that my database has some *weird* languages
represented in it

one more minor thing when showing fines on reservations

if there's not a record associated with the xact (like in the case of
a reservation against a non-cataloged resource), don't try to show a
link

fines on reservations no longer break myopac/main (my account)

if no query is sent in the search, return the user to the referring page

show No Subjects when, well, there are no subjects; cleanup