From f1c5f115dbbb51dff82491a9b5b02facc3b388f6 Mon Sep 17 00:00:00 2001
From: Jeff Davis <jdavis@sitka.bclibraries.ca>
Date: Mon, 8 Dec 2014 10:36:35 -0800
Subject: [PATCH] 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>
---
 src/od_pages_opac.coffee | 45 ++++++++++++++-------------------------------
 1 file changed, 14 insertions(+), 31 deletions(-)

diff --git a/src/od_pages_opac.coffee b/src/od_pages_opac.coffee
index e8d5105..ebd47cb 100644
--- a/src/od_pages_opac.coffee
+++ b/src/od_pages_opac.coffee
@@ -101,19 +101,13 @@ define [
 
 			return @ unless avail
 
-			$x =
-				if avail.available is undefined
-					$('<span>')
-						.css 'color', 'red'
-						.text 'No longer available'
-				else
-					tpl = _.template """
-					<span><%= n_avail %> of <%= n_owned %> available, <%= n_holds %> holds</span>
-					"""
-					$ tpl
-						n_avail: avail.copiesAvailable
-						n_owned: avail.copiesOwned
-						n_holds:  avail.numberOfHolds
+			tpl = _.template """
+			<span><%= n_avail %> of <%= n_owned %> available, <%= n_holds %> holds</span>
+			"""
+			$x = $ tpl
+				n_avail: avail.copiesAvailable
+				n_owned: avail.copiesOwned
+				n_holds:  avail.numberOfHolds
 
 			@find('.result_holdings_table .status')
 				.append $x
@@ -162,19 +156,13 @@ define [
 
 			return @ unless avail
 
-			$x =
-				if avail.available is undefined
-					$('<span>')
-						.css 'color', 'red'
-						.text 'No longer available'
-				else
-					tpl = _.template """
-					<span><%= n_avail %> of <%= n_owned %> available, <%= n_holds %> holds</span>
-					"""
-					$ tpl
-						n_avail: avail.copiesAvailable
-						n_owned: avail.copiesOwned
-						n_holds: avail.numberOfHolds
+			tpl = _.template """
+			<span><%= n_avail %> of <%= n_owned %> available, <%= n_holds %> holds</span>
+			"""
+			$x = $ tpl
+				n_avail: avail.copiesAvailable
+				n_owned: avail.copiesOwned
+				n_holds:  avail.numberOfHolds
 
 			@find '.status'
 				.append $x
@@ -185,11 +173,6 @@ define [
 		_replace_place_hold_link: (avail, type_of_interest) ->
 			return @ unless avail
 			
-			if avail.available is undefined
-				@find '.place_hold'
-					.remove()
-					.end()
-
 			# Find the place hold link that we want to replace
 			$a = @find '.place_hold > a'
 
-- 
2.11.0