tonyair/silverstripe-framework
1
0
Fork 0
mirror of https://github.com/silverstripe/silverstripe-framework synced 2024-10-22 12:05:37 +00:00
Code Issues Projects Releases Wiki Activity

BUG HTTP will now correctly pass over mailto: links when converting relative links to absolute (e.g. in Emails)

Browse Source
This commit is contained in:
Damian Mooyman 2013-04-18 14:25:51 +12:00
parent 4d70daa9e2
commit b6fc1d314e
2 changed files with 13 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
control/HTTP.php
View File

@ -42,6 +42,7 @@ class HTTP {
public static function absoluteURLs($html) {
$html = str_replace('$CurrentPageURL', $_SERVER['REQUEST_URI'], $html);
return HTTP::urlRewriter($html, function($url) {
if(stripos($url, 'mailto:') === 0) return $url;
return Director::absoluteURL($url, true);
});
}

12
tests/control/HTTPTest.php
View File

@ -187,6 +187,18 @@ class HTTPTest extends SapphireTest {
});
}
public function testEmailLinks() {
$this->withBaseURL('http://www.silverstripe.org/', function($test){
// links
$test->assertEquals(
'<a href=\'mailto:admin@silverstripe.org\'>Email Us</a>',
HTTP::absoluteURLs('<a href=\'mailto:admin@silverstripe.org\'>Email Us</a>')
);
});
}
/**
* Run a test while mocking the base url with the provided value
* @param string $url The base URL to use for this test