From 14d2f7f5b8ef5e91a862754159b4a5cee4411c1c Mon Sep 17 00:00:00 2001
From: dbs <dbs@6d9bc8c9-1ec2-4278-b937-99fde70a366f>
Date: Thu, 12 Nov 2009 17:04:32 +0000
Subject: [PATCH] Improve the regex to account for linefeeds in multi-subfield
 ISSN tags

git-svn-id: svn://svn.open-ils.org/ILS-Contrib/conifer/trunk@715 6d9bc8c9-1ec2-4278-b937-99fde70a366f
---
 web/opac/skin/lul/xml/rdetail/rdetail_summary.xml | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/web/opac/skin/lul/xml/rdetail/rdetail_summary.xml b/web/opac/skin/lul/xml/rdetail/rdetail_summary.xml
index 7283b3eddb..114f50c586 100644
--- a/web/opac/skin/lul/xml/rdetail/rdetail_summary.xml
+++ b/web/opac/skin/lul/xml/rdetail/rdetail_summary.xml
@@ -94,7 +94,11 @@
 				    <script type='opac/slot-format'><![CDATA[
 					unHideMe($('tag022'));
 					var issn_raw = dojo.trim(dojox.data.dom.textContent(item));
-					var issn = issn_raw.replace(/^.*?(\d{4}).(\d{3}(\d|x)).*?$/i, "$1-$2");
+					// textContent returns multiple subfields concatenated with linefeeds,
+					// so our regex needs to match against linefeeds. "." does not match
+					// linefeeds, so get \s into the mix
+                                        var issn = issn_raw.replace(/^(\s|.)*?(\d{4}).(\d{3,4}[xX]?)(\s|.)*/, "$2-$3");
+
 					var ses = new OpenSRF.ClientSession('open-ils.resolver');	
 					var req = ses.request('open-ils.resolver.resolve_holdings.raw', 'issn', issn);	
 					req.oncomplete = function(r) {
-- 
2.11.0