Improve float avoidance in outstanding bills

Simplify and improve floating point issue avoidance in
outstanding_bills_for_circ: fewer parentheses, better results.

Signed-off-by: Jeff Godin <jgodin@tadl.org>

Log bills found during lost checkin with forgive

Log outstanding bills found during lost checkin with forgive.

Signed-off-by: Jeff Godin <jgodin@tadl.org>

Support forgive of Lost bills on checkin

Add support for making a Forgive payment on outstanding Lost and
Lost Processing Fee bills on checkin of a lost item.

Signed-off-by: Jeff Godin <jgodin@tadl.org>

Move outstanding_bills_for_circ to AppUtils

Move the outstanding_bills_for_circ sub to
OpenILS::Application::Apputils, since we intend to call it from at
least two different packages.

Signed-off-by: Jeff Godin <jgodin@tadl.org>

Merge branch 'my_tadl' into lost_forgive_not_void

Call outstanding_bills_for_circ differently

Our arguments don't include a ref to the class/self when we're calling
outstanding_bills_for_circ from within its own package.

For now, rename to a private-looking sub and change what arguments we
look for.

Signed-off-by: Jeff Godin <jgodin@tadl.org>

"find outstanding bills for circ" gets its own sub

outstanding_bills_for_circ accepts an editor and a circ object,
and will return a reference to an array of outstanding billing
objects. this has been broken out of the forgive_overdues sub,
as it will be useful elsewhere.

"forgive" overdues when marking an item lost

New option for forgiving (as opposed to voiding) overdues on
a circ when marking the item lost:

When circ.forgive_overdue_on_lost is set, attempt to make a
payment of type "Forgive" on the transaction for the amount
of outstanding overdue billings.

Only outstanding bills of type 1 (Overdue materials) will
be paid, and only the first contiguous grouping. If the
system finds an outstanding billing of type other than 1,
it will pay what it has found up to that point.

circ.forgive_overdue_on_lost has priority over the "void"
version of the same setting, if both happen to be set.

Signed-off-by: Jeff Godin <jgodin@tadl.org>

const/strings for YAOUS: "forgive" lost billings

Constants and strings for new OU settings:

circ.forgive_overdue_on_lost
circ.forgive_lost_on_checkin
circ.forgive_lost_proc_fee_on_checkin

These are intended for use as an alternative to their "void"
counterparts in situations where the library does not refund
fees and fines, and wishes funds taken in for one purpose or
billing type to not be re-applied toward a different billing
type as a result of voiding bills after they have been paid.

Signed-off-by: Jeff Godin <jgodin@tadl.org>

Fix IDL and OU setting check for staged users

The IDL had references to sequences that do not exist:

staging.usr_stage_row_id_seq -- a typo, fixed

In the case of staging.billing_address_stage_row_id_seq, the
staging.billing_address_stage table is created with LIKE,
and uses the sequence staging.mailing_address_stage_row_id_seq

The OU setting check for the open-ils.actor.user.stage.create API
call was not passing an org unit, and would always fail.

We now pass the home_ou of the user being staged.

At this point, the opac.allow_pending_user OU setting type must
be manually created before its value can be set.

Signed-off-by: Jeff Godin <jgodin@tadl.org>

Fix syntax error in default skin rdetail.js

Fix a syntax error in the default skin's rdetail.js file.

Signed-off-by: Jeff Godin <jgodin@tadl.org>

Fix broken ref to colors.css for tadl-legacy skin

The colors.css file lives in the theme, not the skin.

Another mistaken substitution that went unnoticed during testing
-- the OPAC looked so CLEAN!

Signed-off-by: Jeff Godin <jgodin@tadl.org>

Fix image path broken by over-zealous replacement

Over-zealous substitution on my part.

There is no /images/tadl-legacy, just /images/tadl

Signed-off-by: Jeff Godin <jgodin@tadl.org>

Consider 100 holds instead of 10 at op-capture

when using fifo holds strategy and rules which limit which copies can
transit for holds (such as age hold protection), your 10 next holds
may all be unsuitable for capture.

TADL had previously run with this change in production without issue
(though we had limited it to "only when fifo is in use" since others
in the consortium were not using fifo holds.

Signed-off-by: Jeff Godin <jgodin@tadl.org>

Configure staff client to use tadl-legacy skin

Using standard options in custom.js, direct the staff client to
use the "tadl-legacy" opac skin.

Signed-off-by: Jeff Godin <jgodin@tadl.org>

Change hardcoded paths in tadl-legacy skin

s/tadl/tadl-legacy/ on hardcoded skin paths in tadl-legacy skin.

Signed-off-by: Jeff Godin <jgodin@tadl.org>

Copy tadl skin to tadl-legacy

Preparing for redirection of old skin names to the current
tadlv5 skin, and we do not want to change the skin in the
staff client. The new "tadlstaff" skin is not yet ready,
thus we copy the tadl skin to tadl-legacy with the desire that
it be short-lived.

Following commits will adjust paths within the tadl-legacy skin
and adjust the staff client to use the tadl-legacy skin.

Signed-off-by: Jeff Godin <jgodin@tadl.org>

Handle new-style licenses as patron barcodes

Handle new-style drivers licenses when used as patron barcodes by
using the first 13 characters (lowercased) when the "raw" barcode
is either 25 or 27 characters long. Previously we only handled 25
character barcodes.

Signed-off-by: Jeff Godin <jgodin@tadl.org>

Patron barcodes: handle drivers license, state ID

Special handling for drivers license and state ID
patron barcodes.

These changes have been in production for some time, but were
not previously checked in to revision control.

Signed-off-by: Jeff Godin <jgodin@tadl.org>

Specify TADL skin choices in custom.js

Use custom.js to specify TADL skin selections for the staff
client. Also, include custom.js in index.xhtml "new tab" page.

These changes have been in production for some time, but were
not previously checked in to revision control.

Signed-off-by: Jeff Godin <jgodin@tadl.org>

Add optional in-library navigation bar

Support optional in-library navigation bar sourced as an iframe
when cookie "inlib" is set. Initial setting of cookie is easily
done by loading index.xml with inlib=1 as a URL param.

Signed-off-by: Jeff Godin <jgodin@tadl.org>

Jacket image aspect ratio, subject sidebar style

Fix aspect ratio on jacket images (especially CDs) by
setting height to auto.

Add styling for sidebar subject div.

Signed-off-by: Jeff Godin <jgodin@tadl.org>

Prevent cleanup of "dead" serial units for now

We (TADL) have serial units with no serial items.

There is logic in the unitize_items subroutine in
OpenILS::Application::Serial which "cleans up"
these "dead" units.

While we look into why we have units with no items,
we're disabling this deletion so that we don't
have need to undelete big batches of still-circulating
serial units.

Signed-off-by: Jeff Godin <jgodin@tadl.org>

Only request review sources TADL subscribes to

For Syndetic Solutions, only request those review
sources which TADL subscribes to. This includes
sources not requested by default, and removes
sources which we do not use. The ordering of
the review sources is also intentional.

Signed-off-by: Jeff Godin <jgodin@tadl.org>

Change comment style in bibtemplate / inline JS

Inline / BibTemplate Javascript with // style comments can
cause problems for us in MSIE. This commit removes some
commented-out code and changes some comments from // comments
to /* comments */

Signed-off-by: Jeff Godin <jgodin@tadl.org>

Teach tadlv5 to display jackets by record ID

Teach the tadlv5 skin how to display jackets images by record ID.

Signed-off-by: Jeff Godin <jgodin@tadl.org>

Change references from tadlv4 to tadlv5

Change references in include statements from tadlv4 to tadlv5

Signed-off-by: Jeff Godin <jgodin@tadl.org>

Fork tadlv4 skin to tadlv5 skin

The tadlv5 skin will add jacket images by record ID.

Signed-off-by: Jeff Godin <jgodin@tadl.org>

Teach OPAC (jspac) jacket images by record id

Teach the Javascript OPAC how to retrieve jacket images by
record id.

Signed-off-by: Jeff Godin <jgodin@tadl.org>

Removed unused, commented-out function

Removed unused, commented-out function: an earlier iteration of
buildISBNSrc

Signed-off-by: Jeff Godin <jgodin@tadl.org>

Teach Syndetic.pm to pass UPC in requests

Teach OpenILS::WWW::AddedContent::Syndetic to pass UPC when
requesting added content from the source.

Protect against warnings when no isbn/upc provided

Protect against Perl giving us uninitialized value warnings when
isbn or upc has not been provided.

Signed-off-by: Jeff Godin <jgodin@tadl.org>

Finish Added Content by record ID

OpenILS::WWW::AddedContent now supports fetching content by record ID.

To start, this lets us request content in a way that uses both ISBN
and UPC as keys, assuming the AC handler and provider support it.

The previous method of requesting AC data continues to be
supported, and should be unaffected.

Business::ISBN is used to clean ISBN values when fetching by record ID

If we find no valid ISBN and no UPC values in the record,
do not attempt to call an AC handler. This will need
to change when we have AC handlers that rely on other
possible keys.

Attempt to serve from cache before fetching keys from the bib
record in the database.

Signed-off-by: Jeff Godin <jgodin@tadl.org>

Start of AC by record ID from 2011 hackfest

Rough, untested cut. use rec id as they added content key;
fetch isbn/upc from full_rec; push full set of isbns and upcs
down to handler code

Signed-off-by: Jeff Godin <jgodin@tadl.org>

LP#845763 remote added content breaks

remote added content breaks when using locally served content

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

Include reshelving count in multi-lib summary

Also include reshelving counts in multi-lib summary, so that
we don't say "0 available at Example Branch 1" without further
information "(plus 2 recently returned)".

Signed-off-by: Jeff Godin <jgodin@tadl.org>

Don't change &amp; to &Amp;, etc.

Fix toTitleCase() to not uppercase HTML character entities.

Signed-off-by: Jeff Godin <jgodin@tadl.org>

Add reshelving counts, suggest hold when no copies

Rather than count "reshelving" copies as "available", note
that there are X available "plus Y recently returned".

Adjust logic to display unavailable count when there are no
available copies, and to include a suggestion to (try) placing
a hold.

Signed-off-by: Jeff Godin <jgodin@tadl.org>

Fix logic -- no more "additional copies" repeating

Fix logic for max_lines handling to avoid repeated output of
"additional copies" message.

Also, remove max_lines limit on record detail page.

Signed-off-by: Jeff Godin <jgodin@tadl.org>

Increase size of home search box

Increase size of home search box, overriding element-level
style declaration.

Signed-off-by: Jeff Godin <jgodin@tadl.org>

Remove locale picker from home, new inputs css

Remove locale picker from home search page.

Freshen CSS for input elements, based on wjr's work on dev website.

Signed-off-by: Jeff Godin <jgodin@tadl.org>

Use 245$a for title, and use Title Case

Shift to using 245$a for title in record detail summary.

Use Title Case for both search results and record detail.

Signed-off-by: Jeff Godin <jgodin@tadl.org>

Add "Additional persons" field (700 tags)

Add "Additional persons" to record detail summary for display
of 700 tags from the MARC record.

Signed-off-by: Jeff Godin <jgodin@tadl.org>

Combine pub date and publisher into one field

Combine publication date and publisher into one field,
"Publication info".

Signed-off-by: Jeff Godin <jgodin@tadl.org>

Change the display of subjects in the sidebar

For the sidebar display of subjects, use one link per subject
item, not the "increasingly specific" multiple link style of
the record detail summary.

Also minor other formatting changes, such as using <li>. Change
to slot.parentNode reference for unHideMe.

Signed-off-by: Jeff Godin <jgodin@tadl.org>

Extras as "tabs", hide some useless AC content

Use a "tab" effect to bring more attention to extras such as
added content reviews, author notes.

Hide useless / duplicated content such as the title and author
in added content reviews (as returned by Syndetic Solutions).

Signed-off-by: Jeff Godin <jgodin@tadl.org>

Only show series if there is content

Only show series if there is content available for that field.

Also, show series by default without need to select "More Info".

Signed-off-by: Jeff Godin <jgodin@tadl.org>

Add subjects to sidebar

Add bibtemplate style subjects to the sidebar. Shows only in
record detail view, no apparent problems being present-but-
not-visible elsewhere.

Signed-off-by: Jeff Godin <jgodin@tadl.org>

Update hardcoded tadlv3 paths to tadlv4

Update hardcoded paths in files to reference tadlv4 instead
of tadlv3.

Signed-off-by: Jeff Godin <jgodin@tadl.org>

Fork tadlv3 skin to tadlv4 skin

Fork tadlv3 to tadlv4 to begin work.

Signed-off-by: Jeff Godin <jgodin@tadl.org>

hs18738 checked for expired accounts with SIP

Fix link display even when no physical copies

This is a bugfix: We were failing to create any copy / uri
summary if there were zero physical copies. Now, instead of
"return", we "return output.innerHTML" to display the links
which have been generated previously (but not output).

At this time, we don't want to do that in the record detail
page, so we continue to bail there regardless.

Signed-off-by: Jeff Godin <jgodin@tadl.org>

Fix bug in copy summary display on detail page

Copy summaries with multiple libraries were having their
list of libraries overwritten by the last library in the
list.

Thanks to wjr for noting the issue. This fixes it by moving
a mis-placed curly brace.

Previous commit fixed search results but not record detail.

Signed-off-by: Jeff Godin <jgodin@tadl.org>

Fix bug in copy summary display

Copy summaries with multiple libraries were having their
list of libraries overwritten by the last library in the
list.

Thanks to wjr for noting the issue. This fixes it by moving
a mis-placed curly brace.

Signed-off-by: Jeff Godin <jgodin@tadl.org>

Use available copy summary in record detail page

Having removed the copy info matrix/table from the record detail
page, we now replace it with the new available copy summary from
the search results page.

When there are copies at multiple libraries, display a total
count of available copies at each library.

When there are copies available at only one library (which also
happens when search is scoped to one library), display a count
of available copies grouped by copy shelving location and call
number label.

Signed-off-by: Jeff Godin <jgodin@tadl.org>

Copy summary group by call number, add max_lines

Group copy summary by call number in addition to circ lib and
copy location.

Also, support setting of max_lines for limiting the number
of call numbers output -- useful for bibs with lots of volumes
at a single library -- to prevent the search results page from
growing too long.

Signed-off-by: Jeff Godin <jgodin@tadl.org>

Summarize copy availability in search results

Summary of available copies in search results.

Next, need to display call numbers.

Signed-off-by: Jeff Godin <jgodin@tadl.org>

Show reviews/anotes by default if found

If the added content handler found reviews or author notes,
show that information by default (first reviews, then author
notes if there were no reviews).

If neither are found, show nothing by default -- but things
like Table of Contents or Excerpts will still be available
by clicking the proper link.

Signed-off-by: Jeff Godin <jgodin@tadl.org>

Remove copy info matrix from rdetail page

Removing copy info matrix.

Next up: bibtemplate-style copy information to replace
copy info matrix.

Signed-off-by: Jeff Godin <jgodin@tadl.org>

GRPL-style more/less info button in rdetail

Add GRPL-style more/less info button in rdetail view.

Signed-off-by: Jeff Godin <jgodin@tadl.org>

Remove some borders/color, adjust a missed color

Remove border around "home" page search, remove border around
search results.r

Remove some color from top and bottom of search results.

Override a missed color which was resulting in the default green
color on "low search results".

Signed-off-by: Jeff Godin <jgodin@tadl.org>

Adjust include paths from tadlv2 to tadlv3

Adjust include paths which reference tadlv2 to reference tadlv3
instead. We'd rather not rely on default AND the previous skin
revision.

Signed-off-by: Jeff Godin <jgodin@tadl.org>

Forking tadlv2 to tadlv3 skin

Breaking ground on tadlv3 skin, starting with what is currently
the tadlv2 skin.

Signed-off-by: Jeff Godin <jgodin@tadl.org>

Merge branch 'prod/rel_2_1/deployed' of yeti.esilibrary.com:tadl/evergreen into tadl_skin_changes

Merge branch 'prod/rel_2_1/deployed' of yeti.esilibrary.com:tadl/evergreen into my_tadl

perl modules from asp-tadl-app1, minus OpenILS/SIP/Patron.pm

more modified perl modules from asp-tadl-sip1

snapshot of /usr/local/share/perl/5.10.1/OpenILS/SIP/Patron.pm from asp-tadl-sip1

Adjust styling of css_button class

Adjust the styling of the css_button class, especially changes
in margin and changing of display to inline-block so as not to
overlap with other buttons or other display elements.

Signed-off-by: Jeff Godin <jgodin@tadl.org>

Fix: recs without online_loc break results display

Don't try to check a substring of rec.online_loc()[0] if there
are no entries returned by rec.online_loc() to begin with.

Manifests result_info_2 (with the prev/next links) not being
visible on the bottom of the search results page when there
are any bibs present in the results which have no online_loc()
entries.

Signed-off-by: Jeff Godin <jgodin@tadl.org>

Use a default auri label instead of http:// urls

Use a default auri label in search results and record summary
pages. This is mostly to prevent the symptoms of LP 918020
but may be more generally useful as well.

Also, use css_button class for BibTemplate-generated auri links
in a record summary.

Signed-off-by: Jeff Godin <jgodin@tadl.org>

Reference tadlv2 skin-local myopac.js

Use the skin-local myopac.js instead of that from the
default skin.

Signed-off-by: Jeff Godin <jgodin@tadl.org>

Display Series, Hold Count in record summary

Re-adding pre-bibtemplate Series display to record summary,
re-adding CGI-backed Hold Count.

These are both prime candidates for being replaced using
newly-available features.

Signed-off-by: Jeff Godin <jgodin@tadl.org>

Use library local patron barcode regex in myopac

Use library local patron barcode regex in myopac, so that
username vs barcode detection will be correct.

The regex used at login is maintained outside of the skin,
so we're already set there.

Signed-off-by: Jeff Godin <jgodin@tadl.org>

Hide copy counts for online-only bibs in results

Hide copy counts in search results for bibs which appear to be
online-only.

Signed-off-by: Jeff Godin <jgodin@tadl.org>

Fork rresult.js from default skin

This is the rresult.js from the default skin.

Signed-off-by: Jeff Godin <jgodin@tadl.org>

Enhance display of online-only bibs

On search results:
* Hide "place hold" links for bibs which appear to have TADL
  ezproxy auris.
* Simplify physical description for online-only bibs in search
  results
* Hide format icons for online-only bibs
* Add some padding to result_table_format_cell

On record summary:
* Change auri links to use the css_button class.
* Hide things which only make sense for physical holdings:
  * place hold
  * copy info displays
  * call number browse

Signed-off-by: Jeff Godin <jgodin@tadl.org>

Use medium item image for record summary

Switch to "medium" size item image in record summary.

Signed-off-by: Jeff Godin <jgodin@tadl.org>

Use skin-local rdetail.js, result_common.js

Start using the skin-local rdetail.js and result_common.js
files. Previously we were still referencing those from the
default skin.

Signed-off-by: Jeff Godin <jgodin@tadl.org>

Display TADL ezproxy OPAC links in search results

Feature the auri when it appears to be a TADL ezproxy link.

Signed-off-by: Jeff Godin <jgodin@tadl.org>

Hide item image in results when error loading

Hide the item image in results table when there is an
error loading it -- typically a 404.

Signed-off-by: Jeff Godin <jgodin@tadl.org>

Change "Place Hold" links to css buttons

Make "Place Hold" more visible by using CSS to turn
the link into something that resembles a button.

Signed-off-by: Jeff Godin <jgodin@tadl.org>

Add online payment link

Add online payment link to My Fines in the OPAC for
the tadlv2 skin.

Signed-off-by: Jeff Godin <jgodin@tadl.org>

Fork myopac.js in prep for online payment link

This is myopac.js from the default skin, before changes.

Signed-off-by: Jeff Godin <jgodin@tadl.org>

TADL CSS changes - layout and color

tadl_layout.css overrides and adds to default/layout.css
tadl_colors.css is a copy of theme/tadl/css/colors.css

tadl_colors.css should be trimmed down and unintentional
overrides/conflicts with theme/default/css/colors.css
eliminated.

Signed-off-by: Jeff Godin <jgodin@tadl.org>

Remove extra / PINES-specific links from footer

Remove extra links (and PINES-specific links) from footer.

Signed-off-by: Jeff Godin <jgodin@tadl.org>

Updated TADL logo for tadlv2 skin

Updated copy of TADL logo in tadlv2 skin.

Increase logo display size on search result / detail pages --
anywhere common/logo.xml is used.

Signed-off-by: Jeff Godin <jgodin@tadl.org>

Disable Google Chrome Frame, Google Books

In both the tadlstaff and the tadlv2 skins:

* Disable Google Chrome Frame by removing the div and script
* Disable Google Books links, preview by setting existing
  boolean values to false

Signed-off-by: Jeff Godin <jgodin@tadl.org>

Adding two new skins: tadlstaff, tadlv2

Two new skins, forked from default.

Currently, both are identical and reference the css and js files
from the default skin.

Intent is to fork css and js if and when needed.

Signed-off-by: Jeff Godin <jgodin@tadl.org>

Reshelving fix.

Signed-off-by: Steven Callender <stevecallender@esilibrary.com>

OPAC customizations including password reset text change.

Signed-off-by: Steven Callender <stevecallender@esilibrary.com>

Configue credit card processing to default to the recording setting.

Signed-off-by: Steven Callender <stevecallender@esilibrary.com>

corrected combined.js after running autogen.sh

changes since migration, but not sure where these came from yet

custom OPAC skin at the time of migration

helpspot 18373

https://gist.github.com/179cd9a682a509537cb8

Added Content module for pointing to another Evergreen instance

Needs tweaks to opensrf.xml, for example:

        <added_content>
            <!-- load the OpenLibrary added content module -->
            <module>OpenILS::WWW::AddedContent::Evergreen</module>
            <base_url>http://grpl.michiganevergreen.org/opac/extras/ac</base_url>

Patron editor changes

Originating from https://github.com/grpl-eg/rel_2_1.git:

aad962182fc75967a763a66bf32208065fd4b773
39a67d5523f80aaf119b537f5665c0772958e6f7
8bac903b2e512d3625553f2693b100f3cc60f9a0

Stock rel_2_1 patron editor copied into TADL specific directories as a baseline

Bumping version numbers, adding Upgrade Script and Changelog

Signed-off-by: Thomas Berezansky <tsbere@mvlc.org>

Stamping upgrade script for copy visibility vs peer-bibs fix

Signed-off-by: Mike Rylander <mrylander@gmail.com>