Fix account summary display in Evergreen 2.9+

As of Evergreen 2.9, the account summary table on the main My Account
page includes an additional "Account Expiration Date" row.  This broke
the OverDrive API integration layer.  This commit takes the extra row
into account when rewriting the account summary table.

(As a side effect, when OverDrive API integration is enabled, the
Account Expiration Date row will not be displayed on the main My Account
page, but it's still visible in the Account Preferences tab.)

Signed-off-by: Jeff Davis <jdavis@sitka.bclibraries.ca>

Change button label from "editHold" to "Edit Hold"

Signed-off-by: Jeff Davis <jdavis@sitka.bclibraries.ca>

Avoid potential runtime error on My Account main page

Signed-off-by: Jeff Davis <jdavis@sitka.bclibraries.ca>

Remove some unused code

Signed-off-by: Jeff Davis <jdavis@sitka.bclibraries.ca>

Flesh out install instructions

Thanks to Michael Peters for testing and providing README revisions!

Signed-off-by: Jeff Davis <jdavis@sitka.bclibraries.ca>

Check for session token before accessing Library Account API

This prevents a 404 error which can cause an "Authentication Required"
popup to appear in Chrome.

Signed-off-by: Jeff Davis <jdavis@sitka.bclibraries.ca>

Fix insertion of title identifier in availability URL

Signed-off-by: Jeff Davis <jdavis@sitka.bclibraries.ca>

Truncated totals on account summary due to bad regex

On the account summary page, OD API was using /\d+?/ to scrape item
totals.  In Javascript, this regular expression matches the minimum
possible match, i.e. one digit, even when the total had more than one
digit.  So, for example, if a patron had 10 items out, the checked out
totals on the account summary page in OD API would show 1 item out.
(The totals at the top right of the page are correct.)

This commit fixes the regex.

Signed-off-by: Jeff Davis <jdavis@sitka.bclibraries.ca>

LP#1434728: make password_required configurable per library

Signed-off-by: Jeff Davis <jdavis@sitka.bclibraries.ca>

checkout formats may be undefined, avoid error in that scenario

Signed-off-by: Jeff Davis <jdavis@sitka.bclibraries.ca>

Use jQuery.noConflict to avoid conflict with other libraries

jQuery's use of '$' as an alias was conflicting with Dojo's use of '$'
as a function name on the TPAC advanced search page.  We can resolve
this by making OD API depend on a wrapper function that invokes
jquery.noConflict().  There's an "official" way to do this using the
require.js map config, but the approach we're using here is simpler (and
actually works!).

Signed-off-by: Jeff Davis <jdavis@sitka.bclibraries.ca>

RT26522: Don't treat non-available items as a special case

Previously, when the 'available' element of a Library Availability API
response was undefined, we assumed that the title itself was no longer
part of the library's current Overdrive collection.  Thus, the Place
Hold link was removed and a "No longer available" message was displayed
in the Status column.  In fact, however, it may just be the case that
the item is currently in the collection, but all copies are checked out.
So it's preferable to retain the Place Hold link and show "0 of X
available" in the Status column -- in other words, don't treat undefined
availability as a special case.

Signed-off-by: Jeff Davis <jdavis@sitka.bclibraries.ca>

RT24863: Suppress OD API on grouped records (metarecords) in search results

Signed-off-by: Jeff Davis <jdavis@sitka.bclibraries.ca>

RT26084: Hide Suspend button on suspended holds

Signed-off-by: Jeff Davis <jdavis@sitka.bclibraries.ca>

remove hard-coded URLs for JS libraries

For ease of configuration and maintenance, paths for various required JS
libraries (require.js, jQuery, etc.) have been moved to Evergreen -- see
templates/opac/parts/js.tt2.

Signed-off-by: Jeff Davis <jdavis@sitka.bclibraries.ca>

configurable base URL for direct links to OverDrive site

Signed-off-by: Jeff Davis <jdavis@sitka.bclibraries.ca>

configurable base URL for product links

Signed-off-by: Jeff Davis <jdavis@sitka.bclibraries.ca>

Added .gitignore

Add license and copyright

This software was originally published on Google Code with an MIT
License and no copyright statment.  The present commit includes the
license within the source code (see LICENSE.txt) and assigns copyright
to the BC Libraries Cooperative.

Signed-off-by: Jeff Davis <jdavis@sitka.bclibraries.ca>

Comment out console.log in od_api module

Signed-off-by: Steven Chan <steven3416@gmail.com>

Complete the README file

Signed-off-by: Steven Chan <steven3416@gmail.com>

Fixed erroneous search links for author and title

Signed-off-by: Steven Chan <steven3416@gmail.com>

Refine the rendering of myopac account summary table and dashboard

Before, the table and dashboard were built to show empty e-item entries
that would sit there while the user waits for the several API requests
to finish before the total values were available to be placed in the
entries.  Now, the entries are built and shown after values are
available.

A solid gray bar shows in the dashboard during the wait interval. We
would have shown a progress bar, but it's too hard to get it to show
because of interference from the existing style sheet.

The first column of the summary table shows totals for physical items
and was suppressed until the arrival of totals for e-items, but in fact
the totals of physical items are already available from the old table.
Now, we show them immediately, and we don't show the second column at
all until it is filled with values.

Signed-off-by: Steven Chan <steven3416@gmail.com>

Fix handling of errors for downloading Overdrive Read format

These sort of errors require parameters to be included with the
error-handling URL, and a different format for the notification box.

Signed-off-by: Steven Chan <steven3416@gmail.com>

Handle errors arising from downloading content

Any such errors will be embedded in the search parameters of the
myopac/circ page URL. We decode the various components and show them
inside a floating notification box, otherwise we render the page as
normally done.

Signed-off-by: Steven Chan <steven3416@gmail.com>

When decoding search parameters, convert pluses to spaces

Signed-off-by: Steven Chan <steven3416@gmail.com>

Use action dialogue to guide download link process

1. Define and use a downloadLink action dialogue
2. Auto-close done content box after 59 seconds
3. For other actions, auto-close after 2 seconds
4. Auto-focus on close button to allow for quick manual close

Signed-off-by: Steven Chan <steven3416@gmail.com>

Fix indentation
Signed-off-by: Steven Chan <steven3416@gmail.com>

Change how download button works to allow for right-clicking to work

On user clicking a download button, we convert the download URL to a content URL,
and then convert the button to a link with a new message. This will
allow the link to be used directly, ie, right-clicking to open in a new
tab.

The interaction is not perfect, for example, the content link will be
useful for only 60 seconds. A more elaborate interaction will need to be
devised.

Signed-off-by: Steven Chan <steven3416@gmail.com>

Fix regression error in checkout action button

Convert checkout link URLs to reverse proxy format to make it work
again.

Signed-off-by: Steven Chan <steven3416@gmail.com>

Fix format type mismatch

The format type that is listed on search results, record details, and
place hold form doesn't match what is listed in the selection option
list of the checkout dialogue.

To remove the mismatch, we need to use the format types found in the
checkout action object, which is gotten from Availability, rather than
the basic list found from Metadata.  The difference is presumably due to
Advantage accounts.

For search results or record details, we omit the call for Metadata and
substitute the format types from checkout action. For place hold form,
we cannot omit a call to Metadata, because it supplies title details as
well as format types. Therefore, we have to reschedule the asynchronous
calls to Metadata and Availability to run in parallel, wait for both
results, and then check to see if there is a checkout action in
Availability before being able to do the right thing.

Signed-off-by: Steven Chan <steven3416@gmail.com>

Rewording of direct link to Overdrive in holds/checkouts list
Signed-off-by: Steven Chan <steven3416@gmail.com>

Forgot to proxy the URLs in availability actions
Signed-off-by: Steven Chan <steven3416@gmail.com>

Add product ID to context of an action object

Potentially useful for a later refactor

Signed-off-by: Steven Chan <steven3416@gmail.com>

Implement a client-side blacklist

Hostnames in the URL can be blacklisted from running Overdrive functions.

Signed-off-by: Steven Chan <steven3416@gmail.com>

Improve comments for Moment configuration clause in config module
Signed-off-by: Steven Chan <steven3416@gmail.com>

Remove unused item_status() function
Signed-off-by: Steven Chan <steven3416@gmail.com>

Fix error in decoding search parameters in URL

Was not using a possible delimiter; was not decoding all entities

Signed-off-by: Steven Chan <steven3416@gmail.com>

Treat handle() as a method of routes object

Proxy URLs soon rather than late
This makes it easier to make the od_data module be primary over od_api.
Signed-off-by: Steven Chan <steven3416@gmail.com>

Auto-focus on place hold or checkout action buttons
Signed-off-by: Steven Chan <steven3416@gmail.com>

Replace progress text with a progress bar

As a way to do it, we slightly generalized the set_message method of the
action dialogue, to default to using a progress bar for the body text if
there is no text string specified as an argument nor as a body property
of the scenario. Thus, for an intent scenario, it is sufficient to use
the default body.

Signed-off-by: Steven Chan <steven3416@gmail.com>

Fix progress message in action dialogues

The jQuery ajax method does not implement the full promise interface; it
does not call notify and so the progress callback cannot work. As a
substitute, we pass the progress callback as the beforeSend callback,
which is executed before an ajax send.

Also, we use a progress bar instead of a progress text message.

Signed-off-by: Steven Chan <steven3416@gmail.com>

Fix manual logout of EG when OD token expires
Signed-off-by: Steven Chan <steven3416@gmail.com>

Correct an erroneous edit
Signed-off-by: Steven Chan <steven3416@gmail.com>

Add direct links to Overdrive in checkout and hold list
Signed-off-by: Steven Chan <steven3416@gmail.com>

Implement Overdrive data objects using data classes

A data class will make a new instance out of an OD object and endow it
with the methods to normalize the object so that it will be easier to use.
Currently, these methods are defined as ad-hoc functions within the
od_api module, making it lengthy and hard to understand.

Signed-off-by: Steven Chan <steven3416@gmail.com>

Fix delete an item in checkouts list; regression error
Signed-off-by: Steven Chan <steven3416@gmail.com>

Use Session class to implement a session cache
Signed-off-by: Steven Chan <steven3416@gmail.com>

Catchup
Signed-off-by: Steven Chan <steven3416@gmail.com>

Initial commit
Signed-off-by: Steven Chan <steven3416@gmail.com>