diff --git a/code/model/SiteTree.php b/code/model/SiteTree.php index 261fdcfc..cc05249d 100644 --- a/code/model/SiteTree.php +++ b/code/model/SiteTree.php @@ -1764,9 +1764,9 @@ class SiteTree extends DataObject implements PermissionProvider,i18nEntityProvid $parentPage = $linkedPage->Parent; if($parentPage) { if($parentPage->ID) { - $parentPageLinks[] = "ID\">{$parentPage->Title}"; + $parentPageLinks[] = "ID\">{$parentPage->Title}"; } else { - $parentPageLinks[] = "ID\">" . + $parentPageLinks[] = "ID\">" . _t('SiteTree.TOPLEVEL', 'Site Content (Top Level)') . ""; } @@ -1814,7 +1814,7 @@ class SiteTree extends DataObject implements PermissionProvider,i18nEntityProvid ); $dependentTable->getConfig()->getComponentByType('GridFieldDataColumns') ->setFieldFormatting(array( - 'Title' => '$Title', + 'Title' => '$Title', 'AbsoluteLink' => '$value', )); } @@ -2754,4 +2754,4 @@ class SiteTree extends DataObject implements PermissionProvider,i18nEntityProvid self::$cache_permissions = array(); } -} \ No newline at end of file +} diff --git a/javascript/SilverStripeNavigator.js b/javascript/SilverStripeNavigator.js index 133a339b..f57a78b0 100644 --- a/javascript/SilverStripeNavigator.js +++ b/javascript/SilverStripeNavigator.js @@ -1,4 +1,4 @@ -public function windowName(suffix) { +function windowName(suffix) { var base = document.getElementsByTagName('base')[0].href.replace('http://','').replace(/\//g,'_').replace(/\./g,'_'); return base + suffix; } diff --git a/tasks/RemoveOrphanedPagesTask.php b/tasks/RemoveOrphanedPagesTask.php index d2aeaaf5..35fa534a 100644 --- a/tasks/RemoveOrphanedPagesTask.php +++ b/tasks/RemoveOrphanedPagesTask.php @@ -102,7 +102,7 @@ in the other stage:
) ); $label = sprintf( - '%s (#%d, Last Modified Date: %s, Last Modifier: %s, %s)', + '%s (#%d, Last Modified Date: %s, Last Modifier: %s, %s)', $orphan->ID, $orphan->Title, $orphan->ID, diff --git a/tests/model/SiteTreeHTMLEditorFieldTest.php b/tests/model/SiteTreeHTMLEditorFieldTest.php index 6e3eb5f3..1df60d53 100644 --- a/tests/model/SiteTreeHTMLEditorFieldTest.php +++ b/tests/model/SiteTreeHTMLEditorFieldTest.php @@ -53,16 +53,18 @@ class SiteTreeHtmlEditorFieldTest extends FunctionalTest { $editor->setValue(''); $editor->saveInto($sitetree); - $xml = new SimpleXMLElement($sitetree->Content); - $this->assertNotNull($xml['alt'], 'Alt tags are added by default.'); - $this->assertNotNull($xml['title'], 'Title tags are added by default.'); + $parser = new CSSContentParser($sitetree->Content); + $xml = $parser->getByXpath('//img'); + $this->assertEquals('', $xml[0]['alt'], 'Alt tags are added by default.'); + $this->assertEquals('', $xml[0]['title'], 'Title tags are added by default.'); - $editor->setValue('foo'); - $editor->saveInto($sitetree); + $editor->setValue('foo'); + $editor->saveInto($sitetree); - $xml = new SimpleXMLElement($sitetree->Content); - $this->assertNotNull('foo', $xml['alt'], 'Alt tags are preserved.'); - $this->assertNotNull('bar', $xml['title'], 'Title tags are preserved.'); + $parser = new CSSContentParser($sitetree->Content); + $xml = $parser->getByXpath('//img'); + $this->assertEquals('foo', $xml[0]['alt'], 'Alt tags are preserved.'); + $this->assertEquals('bar', $xml[0]['title'], 'Title tags are preserved.'); } public function testImageTracking() {