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

allow filter negation (e.g. -item_type(a) ) in dynamic filter query string building

format selector gets formats from CCVM fields, starting with 'mattype', falling back to item_type

added simple field/value filter to the RO fetch/cache code;  useful for limiting larger sets of config data like ccvm objects

use the more explicit get_foo instead of find_foo for pub/cache object lookup

move utils.tt2 to printnav.tt2 just to give it a more explicit name; i18n and cleanup

use org unit shortname for site param; capture site/depth in search builder to pass on to unapi retrieval, though there's still some template work to do w/ displaying the correct data there (for copy counts)

fetch bookbags inside xact to avoid replication delays, which can occur directly after bookbag-update actions

use unapi for record retrieval on details page; includes svf attrs for matttype, etc.; some i18n cleanup

Pull SVF attrs into displayed bib rec (results page only thus far).  use SVF value to determine format icon and icon label

particularly for running the catalog embedded in the staff client, which makes no visual indication of page progress, it's good to let the caller know something is happening w/ a search.  after a 1-second search delay, show a small progress spinny icon

inisial staff client integration in record details page w/ new staff js file; move footer and other js loading to their own templates; hide top-nav pane (my account summary) for embedded mode; load slim version of marc html (no external css; no print button)

pile of material type icons that are not yet mapped to codes.  renaming to follow

moving toward svf for mattype extraction; much media/material-type icon cleanup; icons are now accessed directly via code instead of inconsistent and map-requiring human names

added paging to payment history; added paging arrows to circ/holds/payments history links

make public bookbag name's links to the public view of the bookbag; minor tidying

improved holds history paging

repaired circ history paging

Support for password update

after clearing out a stale ses cookie, return user to originally requested resource instead directing home.  this prevents the case where going to 'my account' in presence of expired ses cookie redirects the user home

tie in select-all option in lists interfaces

removed debug log line

consistent w/ previous opac, default to barcode login and jump to username only if there is a barcode regex (and regex does not match)

oops, forgot to git-add w/ last commit

implemented username update

speed up and clean up the user holds out/available count summary call

oops, missed a wrapper decl. fix; repaired

More my-account cleanup/consistency

For consistency, make new sub-pages (payments history, preferences) load
as a separate page in the same path (prefs -> prefs_notify) instead of
either an ?expand= option or a sub-path page (prefs -> prefs/notify).

Let the my-account base page load the generic base page as a wrapper so
that my-account page are not required to load both and (for me, anyway)
make the inheritance more obvious.

circ history sorting newest to oldest, got paging working, but could use some work..

minor cleanup

added support for updating opt-in settings preferences

created inital notifications preference page; infrastructure for notify and search/settings preferences pages; prefs pages icons

initial payments history tab; I18n; some general cleanup

make next/prev search links not-visible instead of removing entirely to keep page position of links consistent

Use MultiSession for parallel sessions

In unapi.bre record retrieval, leverage OpenSRF::MultiSession for
fetching batches of records in parallel.

move to unapi.bre for bib/copy-count/slim-holdings retrieval on record results page

more rdetail extras cleanup; added support for authors display

consistent w/ seed skin (and to avoid total vertical domination), default to showing 10 copies-per-page on detail page

fixed odd bug w/ href attrs rendering the copyright symbol for &copy_limit=

More work on record details 'extras'

When an extra is chosen, anchor the page on the selected extra for focus
Lots of cleanup to create a consistent system for selecting and
displaying extras

update flesh depth

Fleshed out View MARC / Subjects Tabs

Record detail page has support for showing the 'extras' tabs.  Currently
supported tabs are View MARC (html) and extended subject info.  For
consistency w/ seed skin, subjects are now only shown in expanded
subjects area.

added fully-parallized (send 10 reqs simultaneously) versions of unapi.bre and direc

do some basic parallelizing optimization

more test options

Now fleshing copy parts, uris, and copy stat cats in 'direct' version
Added a full-feed version
Added an unapi.bre version

essentially functioning hold editor

Still needs a way to report failures to the user

link to editor from myopac/holds

annoying typo

flesh copy loc. for closer match to unapi

unapi test code

implemented holds history template w/ links to/from active holds page;  TODO: verify history call is returning the holds we want it to..

added circulation history backend and template; use |-style regex containers to reduce escapes in path checking

towards an opac hold editor

KCLS doesn't show an abstract high on the record detail page.

This is just a little juggling to reflect that

Ability to move records from temp list to bookbag

Show format icons in results and record detail pages

updated code comments

in bib attr (mra) fetcher, get the display label from the ccvm where appropriate and add to response object

removed debug log message

make all field_safe classes accessible via fetch/cache by default.  reduce page churn by creating then caching the fetch/cache and org-tree generator funcs and simply inserting them into the page context on each page load.  IOW, no need to redefine the funcs on each page load

added utility func for fetching extracted bre svf attributes (mra's)

as was recently done w/ existing kcls skin; hide the open transaction summary info in my-account, citing confusion and unnecessary additional information.  note, you can still see total balance owed

avoid adding site() to query if loc param is empty string

if there, show shelf expire date for available holds; minor var cleanup

pile of updated icons

implemented bookbag rss link

make presentation better by hiding as-yet-unready features, etc

also, made links out of the dashboard counts, but oddly the first one
("checked out") only works in IE, not Chrome or Firefox.  I suspect that
the complications of all the "position: absolute"-this and "position:
relative"-that make for fluky layout.

"Add to my list" becomes "remove from my list", and

it now works on the record detail page as well as the results page.
Also, I fixed a bug by which an HTML form was rendered wrongly, totally
breaking the display of the results page in IE and preventing the first
result on any page of search results from adding to the anonymous list.

smarter search term propagation, should avoid ARRAY(0xdeadbeef) type ...

... stuff in search boxes

bib record subject search terms: painstaking translation of BibTemplate to TT

the reference implementation I used is in
Open-ILS/web/opac/skin/default/xml/rdetail/rdetail_summary.xml starting
at line 93 as of this commit.

Only now that I've imitated the default skin's behavior exactly, I note
that the subject search links produced don't actually work on my test server.
Seems to be a local configuration issue though; will investigate.

Improve search term propagation from page to page

Also, for things like the author link on the record detail page, build a
link with 'qtype=author&query=JK Rowling' instead of 'query=au:JK Rowling'

The latter works, but causes the search form to look stupid.
Query type dropdown will be set to keyword by default, and the term will
be "au:JK Rowling", which will be a head scratcher for 99% of real
users.

If the TT stuff loads CGI w/o -utf8, that "instance" (not really instance)...

... will compete with use CGI in EGCatloader.pm. Really. Normally I enjoy
perl but the awful, horrendous, unspeakable black magic used in CGI.pm
has given me a great deal of trouble today.

Specifically, I could not really succeed in subclassing it. I had a kind
of half success before, but I could only really replace methods in my
subclass, but not call the methods unchanged from the original base
class (CGI).  This manifested in a bug where I had been using
the query_string() method preserve search terms across some pages, but
that method had stopped working since I introduced the CGI subclass.

In an attempt to fix that, I discovered this document
http://stein.cshl.org/WWW/CGI/#subclassing
and tried the advice therein, and several variations thereof, but
everything I produced caused mod_perl to consume memory until it keeled
over from OOM.

Finally, I discovered that I never really needed the subclass in the
first place, so long as 'use CGI' with no module parameters doesn't
happen in the same process as 'use CGI qw/-utf8/'.

integrated update-email address action into account prefs

CGIUTF8 was a horrible, subtle, dark failure

and I don't want to talk about it. Also, 'use CGI qw/-utf8/' suddenly
works.  Maybe it always did.

i18n and repaired holds estimated wait calculation.