From 218cafe43045e68373b487506822c554ac5c39af Mon Sep 17 00:00:00 2001 From: dbs Date: Wed, 9 Jul 2008 13:17:18 +0000 Subject: [PATCH] Enhance URL detection and linking just a tiny bit further: * Require a slash to avoid matching online URL for "HTTP: The Complete Reference" * Use same tests to determine whether the name is just the URL repeated (or a slight variation thereof) git-svn-id: svn://svn.open-ils.org/ILS/trunk@10003 dcc99617-32d9-48b4-a31d-7c20da2025e4 --- Open-ILS/web/opac/skin/default/js/rdetail.js | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/Open-ILS/web/opac/skin/default/js/rdetail.js b/Open-ILS/web/opac/skin/default/js/rdetail.js index 986a3dce94..63306b10f4 100644 --- a/Open-ILS/web/opac/skin/default/js/rdetail.js +++ b/Open-ILS/web/opac/skin/default/js/rdetail.js @@ -209,10 +209,11 @@ function _rdetailDraw(r) { var links = record.online_loc(); for( var i = 0; links && links.length > 0 && i < links.length; i = i + 2 ) { var href = links[i]; - if( href.match(/https?:|ftp:|mailto:/) ) { + // avoid matching "HTTP: The Complete Reference" + if( href.match(/https?:\/|ftps?:\/|mailto:/i) ) { unHideMe($('rdetail_online_row')); var name = '' + links[i+1]; - if(!name || name.match(/http/)) name = href; + if(!name || name.match(/https?:\/|ftps?:\/|mailto:/i)) name = href; $('rdetail_online').appendChild(elem('a', {href:href,'class':'classic_link'}, name)); $('rdetail_online').appendChild(elem('br')); } -- 2.43.2