Markup corrections to cataloging pages; Fixes for Bugs 2166 (" Book Jackets Not Showing on My Summary"), 2194 ("No results for Authority Search Shows blank record"), and 2196 ("Extra comma being added to record display")

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

koha-tmpl/intranet-tmpl/prog/en/includes/auth-finder-search.inc

@@ -3,55 +3,55 @@
             <input type="hidden" name="type" value="intranet" />
             <input type="hidden" name="nbstatements" value="<!-- TMPL_VAR NAME="nbstatements" -->" />
             <input type="hidden" name="index" value="<!-- TMPL_VAR NAME="index" -->" />
-            <h2>Search on</h2>
-                <p>
-                    <label>Authority type</label>
+                <fieldset class="rows"><ol><li>
+                    <span class="label">Authority type</span>
                     <!-- TMPL_VAR NAME="authtypecode" -->
                     <input type="hidden" name="authtypecode" value="<!-- TMPL_VAR NAME="authtypecode" -->" />
-                </p>
-                <p>
-                    <label>Main entry ($a only)</label>
+                </li>
+                <li>
+                    <label for="mainmainentry">Main entry ($a only)</label>
                     <input type="hidden" name="marclist" value="mainmainentry" />
                     <input type="hidden" name="and_or" value="and" />
                     <input type="hidden" name="excluding" value="" />
-                    <select name="operator">
+                    <select name="operator" id="mainmainentry">
                         <option value="contains">contains</option>
                         <option value="start">starts with</option>
                         <option value="is">is exactly</option>
                     </select>
                     <input type="text" name="value" value="<!-- TMPL_VAR name="mainmainstring" -->" />
-                </p>
-                <p>
-                    <label>Main entry</label>
+                </li>
+                <li>
+                    <label for="mainentry">Main entry</label>
                     <input type="hidden" name="marclist" value="mainentry" />
                     <input type="hidden" name="and_or" value="and" />
                     <input type="hidden" name="excluding" value="" />
-                    <select name="operator">
+                    <select name="operator" id="mainentry">
                         <option value="contains">contains</option>
                         <option value="start">starts with</option>
                         <option value="is">is exactly</option>
                     </select>
                     <input type="text" name="value" value="<!-- TMPL_VAR name="mainstring" -->" />
-                </p>
-                <p>
-                    <label>Anywhere</label>
+                </li>
+                <li>
+                    <label for="marclist">Anywhere</label>
                     <input type="hidden" name="marclist" value="" />
                     <input type="hidden" name="and_or" value="and" />
                     <input type="hidden" name="excluding" value="" />
-                    <select name="operator" >
+                    <select name="operator" id="marclist">
                         <option value="contains">contains</option>
                         <option value="start">starts with</option>
                         <option value="is">is exactly</option>
                     </select>
                     </select>
                     <input type="text" name="value" value="<!-- TMPL_VAR name="anystring" -->" />
-                </p>
-		<p>
-		 Sort by 
-		<select name="orderby">
+                </li>
+		<li>
+		 <label for="orderby">Sort by </label>
+		<select name="orderby" id="orderby">
 		<option value="">No order</option>
-            	<option value="HeadingAsc" selected="SELECTED">Heading Ascendant</option>
+            	<option value="HeadingAsc" selected="selected">Heading Ascendant</option>
             	<option value="HeadingDsc">Heading Descendant</option>
           	</select>
-             <input type="submit" value="Start search" class="submit" />
+		</li></ol></fieldset>
+            <fieldset class="action"> <input type="submit" value="Start search" class="submit" /> <a class="cancel close" href="#">Cancel</a></fieldset>
         </form>

koha-tmpl/intranet-tmpl/prog/en/includes/cataloging-toolbar.inc

@@ -38,7 +38,7 @@
 	//]]>
 	</script>
 	
-  <span id="newmenuc">
+  <div id="newmenuc">
     <!-- TMPL_IF NAME="NOTMARC" -->
 		<a id="newrecord" href="/cgi-bin/koha/cataloging/addbiblio.pl">New Record</a>
 	<!-- TMPL_ELSE -->
@@ -55,5 +55,5 @@
 			<input type="submit" value="Add Record Without Search" />
 		</form>
     <!-- /TMPL_IF -->  
-  </span>
+  </div>
 </div>

koha-tmpl/intranet-tmpl/prog/en/modules/authorities/auth_finder.tmpl

@@ -1,9 +1,12 @@
 <!-- TMPL_INCLUDE NAME="doc-head-open.inc" -->
 <title>Koha &rsaquo; Cataloguing Authority Plugin</title>
 <!-- TMPL_INCLUDE NAME="doc-head-close.inc" -->
+<style type="text/css">
+#custom-doc { width:51.46em;*width:50.17em;min-width:675px; margin:auto; text-align:left; } 
+</style>
 </head>
 <body>
-<div id="doc3" class="yui-t2">
+<div id="custom-doc" class="yui-t7">
    
    <div id="bd">
 	<div id="yui-main">

koha-tmpl/intranet-tmpl/prog/en/modules/authorities/searchresultlist-auth.tmpl

@@ -1,6 +1,9 @@
 <!-- TMPL_INCLUDE NAME="doc-head-open.inc" -->
 <title>Koha &rsaquo; Cataloguing Authority Plugin</title>
 <!-- TMPL_INCLUDE NAME="doc-head-close.inc" -->
+<style type="text/css">
+#custom-doc { width:51.46em;*width:50.17em;min-width:675px; margin:auto; text-align:left; } 
+</style>
 <script type="text/javascript">
 
 function jumpfull(page)
@@ -10,7 +13,7 @@ function jumpfull(page)
 </script>
 </head>
 <body>
-<div id="doc3" class="yui-t2">
+<div id="custom-doc" class="yui-t7">
 
    <div id="bd">
            <div id="yui-main">
@@ -18,9 +21,8 @@ function jumpfull(page)
 
     	<h1>Authority search results</h1>
     	<!-- TMPL_INCLUDE NAME="auth-finder-search.inc" -->
-    	<div id="resultnumber">
-            <div>
-                <!-- TMPL_IF name="displayprev" -->
+    	<!-- TMPL_IF name="total" --><div id="resultnumber">
+            <!-- TMPL_IF name="displayprev" -->
                     <a href="auth_finder.pl?startfrom=<!-- TMPL_VAR NAME="startfromprev" -->&amp;authtypecode=<!-- TMPL_VAR name="authtypecode" --><!-- TMPL_LOOP name="searchdata"-->&amp;<!-- TMPL_VAR name="term" -->=<!-- TMPL_VAR name="val"--><!-- /TMPL_LOOP -->&amp;op=do_search&amp;type=intranet&amp;index=<!-- TMPL_VAR name="index" -->&amp;tagid=<!-- TMPL_VAR name="tagid" -->">
                         &lt;&lt;
                     </a>
@@ -37,16 +39,15 @@ function jumpfull(page)
                     <a href="auth_finder.pl?startfrom=<!-- TMPL_VAR NAME="startfromnext" -->&amp;authtypecode=<!-- TMPL_VAR name="authtypecode" --><!-- TMPL_LOOP name="searchdata"-->&amp;<!-- TMPL_VAR name="term" -->=<!-- TMPL_VAR name="val"--><!-- /TMPL_LOOP -->&amp;op=do_search&amp;type=intranet&amp;index=<!-- TMPL_VAR name="index" -->&amp;tagid=<!-- TMPL_VAR name="tagid" -->">
                         &gt;&gt;</a>
                 <!-- /TMPL_IF -->
-            </div>
             <div>
-                <!-- TMPL_IF name="total" -->
-                    &nbsp;<b>Results <!-- TMPL_VAR NAME="from" --> to <!-- TMPL_VAR NAME="to" --> of <!-- TMPL_VAR NAME="total" --></b><br /><br />
+                
+                    <p><b>Results <!-- TMPL_VAR NAME="from" --> to <!-- TMPL_VAR NAME="to" --> of <!-- TMPL_VAR NAME="total" --></b></p>
                 <!-- TMPL_ELSE -->
-                    &nbsp; No results found<br />
-                <!-- /TMPL_IF -->
-            </div>
+                    
+                
+            </div><p>No results found</p><!-- /TMPL_IF -->
         </div>
-        <div id="resultlist">
+        <!-- TMPL_IF NAME="result" --><div id="resultlist">
             <table>
                 <tr>
                     <th>Summary</th>
@@ -74,8 +75,7 @@ function jumpfull(page)
     
         </div>
         <div id="resultnumber">
-            <div>
-                <!-- TMPL_IF name="displayprev" -->
+            <!-- TMPL_IF name="displayprev" -->
                     <a href="auth_finder.pl?startfrom=<!-- TMPL_VAR NAME="startfromprev" -->&amp;authtypecode=<!-- TMPL_VAR name="authtypecode" --><!-- TMPL_LOOP name="searchdata"-->&amp;<!-- TMPL_VAR name="term" -->=<!-- TMPL_VAR name="val"--><!-- /TMPL_LOOP -->&amp;op=do_search&amp;type=intranetamp;&amp;index=<!-- TMPL_VAR name="index" -->&amp;tagid=<!-- TMPL_VAR name="tagid" -->">
                         &lt;&lt;
                     </a>
@@ -92,10 +92,9 @@ function jumpfull(page)
                     <a href="auth_finder.pl?startfrom=<!-- TMPL_VAR NAME="startfromnext" -->&amp;authtypecode=<!-- TMPL_VAR name="authtypecode" --><!-- TMPL_LOOP name="searchdata"-->&<!-- TMPL_VAR name="term" -->=<!-- TMPL_VAR name="val"--><!-- /TMPL_LOOP -->&amp;op=do_search&amp;type=intranet&amp;index=<!-- TMPL_VAR name="index" -->&amp;tagid=<!-- TMPL_VAR name="tagid" -->">
                         &gt;&gt;</a>
                 <!-- /TMPL_IF -->
-            </div>
-        </div>
+        </div><!-- /TMPL_IF -->
 </div>
 </div>
 </div>
 </div>
-<!-- TMPL_INCLUDE NAME="popup-bottom.inc" -->
+<!-- TMPL_INCLUDE NAME="intranet-bottom.inc" -->

koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/detail.tmpl

@@ -81,7 +81,7 @@ function verify_images() {
     <!-- /TMPL_IF -->
         <li><strong>Description:</strong>
                 <!-- TMPL_IF name="pages" --> <!-- /TMPL_IF --><!-- TMPL_VAR name="pages" --> <!-- TMPL_IF NAME="illus" --><!-- TMPL_VAR NAME="illus" --><!-- /TMPL_IF -->
-                <!-- TMPL_IF name="size" -->, <!-- /TMPL_IF --><!-- TMPL_VAR name="size" -->
+                <!-- TMPL_IF name="size" --><!-- TMPL_VAR name="size" --><!-- /TMPL_IF -->
         </li>
 <!-- TMPL_IF name="MARCURLS" --><li>
     

koha-tmpl/intranet-tmpl/prog/en/modules/cataloguing/addbooks.tmpl

@@ -49,7 +49,7 @@
             <td>
                 <p>
                     <a href="/cgi-bin/koha/catalogue/MARCdetail.pl?biblionumber=<!-- TMPL_VAR name="biblionumber" -->"><!-- TMPL_VAR NAME="title" --></a>
-                <!-- TMPL_VAR NAME="subtitle" -->
+                <!-- TMPL_VAR NAME="subtitle" --></p>
                 <!-- TMPL_IF name="summary" -->
                     <p><!-- TMPL_VAR name="summary" --></p>
                 <!-- TMPL_ELSE -->
@@ -93,10 +93,12 @@
                         <!-- TMPL_IF NAME="orderedcount" --> On order (<!-- TMPL_VAR NAME="orderedcount" -->)<!-- /TMPL_IF -->
                     </span>
                 </td>
-				<td> <a href="/cgi-bin/koha/catalogue/showmarc.pl?id=<!-- TMPL_VAR NAME="biblionumber" -->" title="MARC" rel="gb_page_center[600,500]">MARC</a> | <a href="/cgi-bin/koha/catalogue/showmarc.pl?viewas=card&id=<!-- TMPL_VAR NAME="biblionumber" -->" title="MARC" rel="gb_page_center[600,500]">Card</a>
+				<td> <a href="/cgi-bin/koha/catalogue/showmarc.pl?id=<!-- TMPL_VAR NAME="biblionumber" -->" title="MARC" rel="gb_page_center[600,500]">MARC</a> | <a href="/cgi-bin/koha/catalogue/showmarc.pl?viewas=card&amp;id=<!-- TMPL_VAR NAME="biblionumber" -->" title="MARC" rel="gb_page_center[600,500]">Card</a>
                 </td>
-                <td>
-                    <a href="/cgi-bin/koha/cataloguing/addbiblio.pl?biblionumber=<!-- TMPL_VAR NAME="biblionumber" -->" <!-- TMPL_UNLESS NAME="CAN_user_editcatalogue" --> class="disabled" onclick="return false;"<!-- /TMPL_UNLESS -->>Edit biblio</a>
+                <td><!-- TMPL_IF NAME="CAN_user_editcatalogue"--><a href="/cgi-bin/koha/cataloguing/addbiblio.pl?biblionumber=<!-- TMPL_VAR NAME="biblionumber" -->">Edit biblio</a>
+					<!-- TMPL_ELSE --><a href="/cgi-bin/koha/cataloguing/addbiblio.pl?biblionumber=<!-- TMPL_VAR NAME="biblionumber" -->" class="disabled" onclick="return false;">Edit biblio</a>
+					<!-- /TMPL_IF -->
+                    
                 </td>
                 <td>
                     <a href="/cgi-bin/koha/cataloguing/additem.pl?biblionumber=<!-- TMPL_VAR NAME="biblionumber" -->">Add holdings</a>
@@ -132,7 +134,7 @@
                 <!-- TMPL_VAR NAME="author" --></td>
                 <td><!-- TMPL_VAR NAME="isbn" --></td>
                 <td><!-- TMPL_VAR NAME="file" --></td>
-                <td> <a href="/cgi-bin/koha/catalogue/showmarc.pl?importid=<!-- TMPL_VAR NAME="id" -->" title="MARC" rel="gb_page_center[600,500]">MARC</a> | <a href="/cgi-bin/koha/catalogue/showmarc.pl?viewas=card&importid=<!-- TMPL_VAR NAME="id" -->" title="MARC" rel="gb_page_center[600,500]">Card</a>
+                <td> <a href="/cgi-bin/koha/catalogue/showmarc.pl?importid=<!-- TMPL_VAR NAME="id" -->" title="MARC" rel="gb_page_center[600,500]">MARC</a> | <a href="/cgi-bin/koha/catalogue/showmarc.pl?viewas=card&amp;importid=<!-- TMPL_VAR NAME="id" -->" title="MARC" rel="gb_page_center[600,500]">Card</a>
 				</td>
                 <td><!-- TMPL_IF NAME="NOTMARC" -->
                 <!-- TMPL_ELSE -->

koha-tmpl/opac-tmpl/prog/en/modules/opac-readingrecord.tmpl

@@ -1,5 +1,12 @@
 <!-- TMPL_INCLUDE name="doc-head-open.inc" --><!-- TMPL_IF NAME="LibraryNameTitle" --><!-- TMPL_VAR NAME="LibraryNameTitle" --><!-- TMPL_ELSE -->Koha Online<!-- /TMPL_IF --> Catalog &rsaquo;  Account for <!-- TMPL_LOOP name="BORROWER_INFO" --><!-- TMPL_VAR name="firstname" --> <!-- TMPL_VAR name="surname" --><!-- /TMPL_LOOP -->
 <!-- TMPL_INCLUDE NAME="doc-head-close.inc" -->
+<!-- TMPL_IF NAME="GoogleJackets" --><script type="text/javascript">
+//<![CDATA[
+$(document).ready(function(){
+    KOHA.Google.GetCoverFromIsbn();
+});
+//]]>
+</script><!-- /TMPL_IF -->
 </head>
 <body>
 <div id="doc3" class="yui-t1">

koha-tmpl/opac-tmpl/prog/en/modules/opac-results.tmpl

@@ -205,7 +205,7 @@ $(document).ready(function(){
                     <!-- TMPL_ELSE --><!-- TMPL_IF name="copyrightdate" --> <!-- TMPL_VAR name="copyrightdate" --><!-- /TMPL_IF --><!-- /TMPL_IF -->
                         <!-- TMPL_IF name="pages" -->. <!-- TMPL_VAR name="pages" --><!-- /TMPL_IF -->
                         <!-- TMPL_IF name="notes" -->, <!-- TMPL_VAR name="notes" --><!-- /TMPL_IF -->
-                        <!-- TMPL_IF name="size" -->, <!-- TMPL_VAR name="size" --><!-- /TMPL_IF -->
+                        <!-- TMPL_IF name="size" --> <!-- TMPL_VAR name="size" --><!-- /TMPL_IF -->
                         <!-- TMPL_IF name="timestamp" --> <i>(modified on <!-- TMPL_VAR name="timestamp" -->)</i><!-- /TMPL_IF -->
                 </span>
                 <!-- TMPL_IF NAME="copyrightdate" --><span class="results_summary"><span class="label">Date:</span><!-- TMPL_VAR NAME="copyrightdate" --></span><!-- COinS / OpenURL --><span class="Z3988" title="ctx_ver=Z39.88-2004&amp;rft_val_fmt=info%3Aofi%2Ffmt%3Akev%3Amtx%3Abook&amp;rft.au=<!-- TMPL_VAR NAME="author_nospan" -->&amp;rft.btitle=<!-- TMPL_VAR NAME="title_nospan" ESCAPE="url" -->&amp;rft.date=<!-- TMPL_VAR NAME="publicationyear" -->&amp;rft.tpages=<!-- TMPL_VAR NAME="size" -->&amp;rft.isbn=<!-- TMPL_VAR NAME="isbn"  ESCAPE="url" -->&amp;rft.aucorp=&amp;rft.place=<!-- TMPL_VAR NAME="place" -->&amp;rft.pub=<!-- TMPL_VAR NAME="publisher" ESCAPE="url" -->&amp;rft.edition=<!-- TMPL_VAR NAME="edition" -->&amp;rft.series=<!-- TMPL_VAR NAME="series" -->&amp;rft.genre="></span><!-- /TMPL_IF -->

koha-tmpl/opac-tmpl/prog/en/modules/opac-user.tmpl

@@ -42,6 +42,7 @@ $.tablesorter.addParser({
 					<!-- /TMPL_IF -->
 				<!-- /TMPL_IF -->
 			}); 
+<!-- TMPL_IF NAME="GoogleJackets" -->KOHA.Google.GetCoverFromIsbn();<!-- /TMPL_IF -->
             });
 //]]>
 </script>
@@ -60,10 +61,13 @@ $.tablesorter.addParser({
 		<!-- TMPL_IF NAME="patronupdate" --><div class="dialog message"><h3>Thank you!</h3><p>Your corrections have been submitted to the library, and a staff member will update your record as soon as possible.</p></div><!-- /TMPL_IF -->
 		
         <!-- TMPL_IF NAME="opacnote"-->
+		<div class="dialog message">
             <h3>Message from the library</h3>
             <p><!-- TMPL_VAR name="opacnote" --></p>
+		</div>
         <!-- /TMPL_IF -->
         <!-- TMPL_IF NAME="flagged" -->
+		<div class="dialog alert">
         <ul>
             <!-- TMPL_IF NAME="debarred" -->
                 <li><strong>Please note:</strong> Your account has been frozen. Usually the reason for freezing an account is old overdues or damage fees.If <a href="/cgi-bin/koha/opac-user.pl">your account page</a> shows your account to be clear, please consult a librarian.</li>
@@ -74,7 +78,7 @@ $.tablesorter.addParser({
             <!-- TMPL_IF NAME="lost" -->
                 <li><strong>Please note: </strong> Your library card has been marked as lost or stolen. If this is an error, please take your card to the circulation desk at your local library and the error will be corrected.</li>
             <!-- /TMPL_IF -->
-        </ul>
+        </ul></div>
         <!-- /TMPL_IF -->
         
 <div id="opac-user-views" class="toptabs">
@@ -229,7 +233,10 @@ $.tablesorter.addParser({
 <caption>Overdues <span class="count">(<!-- TMPL_VAR NAME="overdues_count" --> total)</span></caption>
 <!-- OVERDUES TABLE ROWS -->
 <tr>
-<th colspan="2">Title</th><th>Call No.</th>
+<!-- TMPL_IF EXPR="AmazonContent||GoogleJackets||BakerTaylorEnabled" --><th>&nbsp;</th><!-- /TMPL_IF -->
+        <th>Title</th>
+		<!-- TMPL_UNLESS NAME="item-level_itypes" --><th>Item Type</th> <!-- /TMPL_UNLESS -->
+<th>Call No.</th>
 <th>Due</th>
 <th>Renew</th>
 <!-- <th>Fine to Date</th> -->
@@ -237,10 +244,17 @@ $.tablesorter.addParser({
 
 <!-- TMPL_LOOP NAME="OVERDUES" -->
 <tr>
-<td><img src="<!-- TMPL_VAR NAME="themelang" -->/images/<!-- TMPL_VAR NAME="ccode" -->.gif" alt="<!-- TMPL_VAR NAME="ccode" -->" title="<!-- TMPL_VAR NAME="ccode" -->" /></td>
-<td><!-- TMPL_IF NAME="AmazonContent" --><!-- TMPL_IF NAME="isbn" --><a href="/cgi-bin/koha/opac-detail.pl?bib=<!-- TMPL_VAR NAME="biblionumber" ESCAPE="URL" -->"><img src="http://images.amazon.com/images/P/<!-- TMPL_VAR name="isbn" -->.01.THUMBZZZ.jpg" alt="" class="thumbnail" /></a><!-- /TMPL_IF --><!-- /TMPL_IF --> <a href="/cgi-bin/koha/opac-detail.pl?bib=<!-- TMPL_VAR NAME="biblionumber" -->"><!-- TMPL_VAR NAME="title" --></a><span class="item-details">
+<!-- TMPL_IF NAME="AmazonContent" --><td><a href="http://www.amazon.com/gp/reader/<!-- TMPL_VAR NAME="amazonisbn" -->/ref=sib_dp_pt/002-7879865-0184864#reader-link"><img border="0" src="http://images.amazon.com/images/P/<!-- TMPL_VAR name="amazonisbn" -->.01.THUMBZZZ.jpg" alt="Cover Image" /></a></td><!-- /TMPL_IF -->
+
+    <!-- TMPL_IF NAME="GoogleJackets" --><td><div style="block" title="<!-- TMPL_VAR NAME="biblionumber" ESCAPE="URL" -->" id="<!-- TMPL_VAR NAME="amazonisbn" -->" class="gbs-thumbnail"></div></td><!-- /TMPL_IF -->
+
+    <!-- TMPL_IF NAME="BakerTaylorEnabled" --><td><!-- TMPL_IF NAME="clean_isbn" --><a href="https://<!-- TMPL_VAR NAME="BakerTaylorBookstoreURL" ESCAPE="HTML" --><!-- TMPL_VAR NAME="clean_isbn" -->"><img alt="See Baker &amp; Taylor" src="<!-- TMPL_VAR NAME="BakerTaylorImageURL" ESCAPE="HTML" --><!-- TMPL_VAR NAME="clean_isbn" -->" /></a><!-- /TMPL_IF --></td><!-- /TMPL_IF -->
+
+<td><a href="/cgi-bin/koha/opac-detail.pl?bib=<!-- TMPL_VAR NAME="biblionumber" -->"><!-- TMPL_VAR NAME="title" --></a><span class="item-details">
 <!-- TMPL_VAR NAME="author" --></span></td>
-<td><!-- TMPL_VAR NAME="classification" --></td>
+
+ <!-- TMPL_UNLESS NAME="item-level_itypes" --><td><!-- TMPL_IF NAME="imageurl" --><img src="<!-- TMPL_VAR NAME="imageurl" -->" title="<!-- TMPL_VAR name="description" -->" alt="<!-- TMPL_VAR NAME="description" -->" /><!-- /TMPL_IF --> <!-- TMPL_VAR name="description" --></td><!-- /TMPL_UNLESS -->
+<td><!-- TMPL_VAR NAME="itemcallnumber" --></td>
 <td><!-- TMPL_VAR NAME="date_due" --></td>
 <td>
 <!-- TMPL_IF NAME="debarred" -->Account Frozen

opac/opac-user.pl

@@ -138,6 +138,23 @@ foreach my $issue ( @$issues ) {
     $issue->{date_due} = format_date($issue->{date_due});
     push @issuedat, $issue;
     $count++;
+	
+		# XISBN Stuff
+	my $xisbn=$issue->{'isbn'};
+	$xisbn =~ /(\d*[X]*)/;
+	$issue->{amazonisbn} = $1;		# FIXME: so it is OK if the ISBN = 'XXXXX' ?
+	my ($clean, $amazonisbn);
+	$amazonisbn = $1;
+	# these might be overkill, but they are better than the regexp above.
+	if (
+		$amazonisbn =~ /\b(\d{13})\b/ or
+		$amazonisbn =~ /\b(\d{10})\b/ or 
+		$amazonisbn =~ /\b(\d{9}X)\b/i
+	) {
+		$clean = $1;
+		$issue->{clean_isbn} = $1;
+	}
+	
 }
 
 $template->param( ISSUES       => \@issuedat );