Merge 3.2 into 3.3

.travis.yml

@@ -34,6 +34,7 @@ matrix:
 before_script:
  - composer self-update || true
  - phpenv rehash
+ - phpenv config-rm xdebug.ini
  - git clone git://github.com/silverstripe-labs/silverstripe-travis-support.git ~/travis-support
  - "if [ \"$BEHAT_TEST\" = \"\" ]; then php ~/travis-support/travis_setup.php --source `pwd` --target ~/builds/ss; fi"
  - "if [ \"$BEHAT_TEST\" = \"1\" ]; then php ~/travis-support/travis_setup.php --source `pwd` --target ~/builds/ss --require silverstripe/behat-extension; fi"

code/model/SiteTreeLinkTracking.php

@@ -259,12 +259,13 @@ class SiteTreeLinkTracking_Parser {
 			// Local anchor.
 			$matches = array();
 			if(preg_match('/^#(.*)/i', $href, $matches)) {
+				$anchor = preg_quote($matches[1], '#');
 				$results[] = array(
 					'Type' => 'localanchor',
 					'Target' => null,
 					'Anchor' => $matches[1],
 					'DOMReference' => $link,
-					'Broken' => !preg_match("#(name|id)=\"{$matches[1]}\"#", $htmlValue->getContent())
+					'Broken' => !preg_match("#(name|id)=\"{$anchor}\"#", $htmlValue->getContent())
 				);
 
 				continue;

code/reports/BrokenLinksReport.php

@@ -112,7 +112,7 @@ class BrokenLinksReport extends SS_Report {
 					$stageLink = $item->AbsoluteLink();
 					return sprintf('%s <a href="%s">%s</a>',
 						$stageLink,
-						$liveLink ? $liveLink : $stageLink . '?stage=Stage',
+						$liveLink ? $liveLink : Controller::join_links($stageLink, '?stage=Stage'),
 						$liveLink ? '(live)' : '(draft)'
 					);
 				}

tests/model/SiteTreeLinkTrackingTest.php

@@ -20,6 +20,7 @@ class SiteTreeLinkTrackingTest extends SapphireTest {
 
 		$this->assertFalse($this->isBroken('<a name="anchor">anchor</a>'));
 		$this->assertFalse($this->isBroken('<a id="anchor">anchor</a>'));
+		$this->assertTrue($this->isBroken('<a href="##anchor">anchor</a>'));
 
 		$page = new Page();
 		$page->Content = '<a name="yes-name-anchor">name</a><a id="yes-id-anchor">id</a>';