[mb-commits] branch, mbs-5696, updated. MBS-5696: Make ModBot leave schema independent URLs

The branch, mbs-5696 has been updated
       via  http://git.musicbrainz.org/gitweb/?p=musicbrainz-server/core.git;a=commit;h=fe9d7091c14c7f30b87f54b693bb7b5ed4b7db6b (commit)
      from  http://git.musicbrainz.org/gitweb/?p=musicbrainz-server/core.git;a=commit;h=e7d9990c74834bb387b39eb8e3e952700f04e1a3 (commit)

Summary of changes:
 lib/MusicBrainz/Server/Edit/Generic/Edit.pm |    2 +-
 lib/MusicBrainz/Server/Filters.pm           |    5 ++---
 t/lib/t/MusicBrainz/Server/Filters.pm       |    4 ++--
 3 files changed, 5 insertions(+), 6 deletions(-)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
commit fe9d7091c14c7f30b87f54b693bb7b5ed4b7db6b
Author: Oliver Charles <[email protected]>
Date:   Tue Jan 29 11:57:35 2013 +0000

    MBS-5696: Make ModBot leave schema independent URLs

diff --git a/lib/MusicBrainz/Server/Edit/Generic/Edit.pm b/lib/MusicBrainz/Server/Edit/Generic/Edit.pm
index 8bbbdd5..2bb1ccd 100644
--- a/lib/MusicBrainz/Server/Edit/Generic/Edit.pm
+++ b/lib/MusicBrainz/Server/Edit/Generic/Edit.pm
@@ -76,7 +76,7 @@ override 'accept' => sub
                 sprintf(
                     'The changes in this edit cause it to conflict with another artist. ' .
                     'You may need to merge this artist with "' . $conflict->name . '" ' .
-                    '(http://%s/artist/%s/).',
+                    '(//%s/artist/%s/).',
                     DBDefs->WEB_SERVER, $conflict->gid
                 )
             );
diff --git a/lib/MusicBrainz/Server/Filters.pm b/lib/MusicBrainz/Server/Filters.pm
index ba8f1da..f091b9d 100644
--- a/lib/MusicBrainz/Server/Filters.pm
+++ b/lib/MusicBrainz/Server/Filters.pm
@@ -110,7 +110,7 @@ sub _display_trimmed {
         : $encoded_url;
 
     $encoded_url = "http://$encoded_url"
-        unless $encoded_url =~ m{^https?://};
+        unless $encoded_url =~ m{^(?:https?:)?//};
 
     return qq{<a href="$encoded_url">$display_url</a>};
 }
@@ -142,10 +142,9 @@ sub format_editnote
 
     # The following taken from http://daringfireball.net/2010/07/improved_regex_for_matching_urls
     $html =~ s{
-    \b
     (                       # Capture 1: entire matched URL
       (?:
-        https?://               # http or https protocol
+        (?:https?:)?//               # http or https protocol
         |                       #   or
         www\d{0,3}[.]           # "www.", "www1.", "www2." … "www999."
         |                           #   or
diff --git a/t/lib/t/MusicBrainz/Server/Filters.pm b/t/lib/t/MusicBrainz/Server/Filters.pm
index 3c3ad28..80fd9f4 100644
--- a/t/lib/t/MusicBrainz/Server/Filters.pm
+++ b/t/lib/t/MusicBrainz/Server/Filters.pm
@@ -16,8 +16,8 @@ test 'Edit note syntax' => sub {
     is(format_editnote("http://musicbrainz.org"),
        '<a href="http://musicbrainz.org">http://musicbrainz.org</a>');
 
-    is(format_editnote("https://musicbrainz.org"),
-       '<a href="https://musicbrainz.org">https://musicbrainz.org</a>');
+    is(format_editnote("//musicbrainz.org"),
+       '<a href="//musicbrainz.org">//musicbrainz.org</a>');
 
     is(format_editnote("www.musicbrainz.org"),
        '<a href="http://www.musicbrainz.org">www.musicbrainz.org</a>');

-----------------------------------------------------------------------


hooks/post-receive
-- 
mb_server

_______________________________________________
MusicBrainz-commits mailing list
[email protected]
http://lists.musicbrainz.org/mailman/listinfo/musicbrainz-commits