Backport r10003 from trunk:

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/branches/rel_1_2@10004 dcc99617-32d9-48b4-a31d-7c20da2025e4

diff --git a/Open-ILS/web/opac/skin/default/js/rdetail.js b/Open-ILS/web/opac/skin/default/js/rdetail.js
index 5c7b45c..5933096 100644 (file)
--- 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'));
                }