[open-ils-commits] r10003 - trunk/Open-ILS/web/opac/skin/default/js

[svn at svn.open-ils.org]

Author: dbs
Date: 2008-07-09 09:17:18 -0400 (Wed, 09 Jul 2008)
New Revision: 10003

Modified:
   trunk/Open-ILS/web/opac/skin/default/js/rdetail.js
Log:
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)


Modified: trunk/Open-ILS/web/opac/skin/default/js/rdetail.js
===================================================================
--- trunk/Open-ILS/web/opac/skin/default/js/rdetail.js	2008-07-09 03:50:07 UTC (rev 10002)
+++ trunk/Open-ILS/web/opac/skin/default/js/rdetail.js	2008-07-09 13:17:18 UTC (rev 10003)
@@ -209,10 +209,11 @@
 	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'));
 		}