Fix incorrect handling of some specific links (#1489060)

CHANGELOG

@@ -1,6 +1,7 @@
 CHANGELOG Roundcube Webmail
 ===========================
 
+- Fix incorrect handling of some specific links (#1489060)
 - Fix incorrect handling of leading spaces in text wrapping
 - Fix unintentional messages list jumps on click in Internet Explorer (#1489056)
 - Fix list of required configuration options (#1489055)

program/lib/Roundcube/rcube_string_replacer.php

@@ -95,12 +95,12 @@ class rcube_string_replacer
             $attrib = (array)$this->options['link_attribs'];
             $attrib['href'] = $url_prefix . $url;
 
-            $i = $this->add($prefix . html::a($attrib, rcube::Q($url)) . $suffix);
+            $i = $this->add(html::a($attrib, rcube::Q($url)) . $suffix);
         }
 
         // Return valid link for recognized schemes, otherwise
         // return the unmodified string for unrecognized schemes.
-        return $i >= 0 ? $this->get_replacement($i) : $matches[0];
+        return $i >= 0 ? $prefix . $this->get_replacement($i) : $matches[0];
     }
 
     /**

tests/Framework/StringReplacer.php

@@ -37,6 +37,8 @@ class Framework_StringReplacer extends PHPUnit_Framework_TestCase
             array('http://link.com?(link)', '<a href="http://link.com?(link)">http://link.com?(link)</a>'),
             array('http://<test>', 'http://<test>'),
             array('http://', 'http://'),
+            array('1@1.com www.domain.tld', '<a href="mailto:1@1.com">1@1.com</a> <a href="http://www.domain.tld">www.domain.tld</a>'),
+            array(' www.domain.tld ', ' <a href="http://www.domain.tld">www.domain.tld</a> '),
         );
     }