gordonbanderson / template-override

code/TemplateOverrideExtension.php

Indentation +15 added lines, -15 removed lines

@@ -6,22 +6,22 @@
 */
 class TemplateOverrideExtension extends DataExtension
 {
-    private static $db = array('AlternativeTemplate' => 'Varchar');
+	private static $db = array('AlternativeTemplate' => 'Varchar');
 
-    public function updateCMSFields(FieldList $fields)
-    {
-        $templatei18n = _t('TemplateOverride.TEMPLATE', 'Template');
-        $fields->addFieldToTab('Root.'.$templatei18n, new TextField('AlternativeTemplate',
-                _t('TemplateOverride.ALTERNATIVE_TEMPLATE_NAME', 'Alternative template name')));
+	public function updateCMSFields(FieldList $fields)
+	{
+		$templatei18n = _t('TemplateOverride.TEMPLATE', 'Template');
+		$fields->addFieldToTab('Root.'.$templatei18n, new TextField('AlternativeTemplate',
+				_t('TemplateOverride.ALTERNATIVE_TEMPLATE_NAME', 'Alternative template name')));
 
-        $info_field = new LiteralField(
-            $name = 'infofield',
-            $content = '<p class="message">'._t('TemplateOverride.INFO', 'If you wish to change'.
-                        ' the default template, type the name of the template here.  Otherwise '.
-                        ' the normal default template will be used.  Normally this will not '.
-                        ' require changing.').'</p>'
-        );
+		$info_field = new LiteralField(
+			$name = 'infofield',
+			$content = '<p class="message">'._t('TemplateOverride.INFO', 'If you wish to change'.
+						' the default template, type the name of the template here.  Otherwise '.
+						' the normal default template will be used.  Normally this will not '.
+						' require changing.').'</p>'
+		);
 
-        $fields->addFieldToTab('Root.'.$templatei18n, $info_field);
-    }
+		$fields->addFieldToTab('Root.'.$templatei18n, $info_field);
+	}
 }

code/PageControllerTemplateOverrideExtension.php

Indentation +28 added lines, -28 removed lines

@@ -2,35 +2,35 @@
 
 class PageControllerTemplateOverrideExtension extends Extension
 {
-    /*
+	/*
     If the alternative template exists, render that, otherwise render with the default template
     */
-    public function index()
-    {
-        return $this->useTemplateOverride();
-    }
+	public function index()
+	{
+		return $this->useTemplateOverride();
+	}
 
-    /**
-     * Render this page using the template override iff it exists.
-     *
-     * @return array An array suitable for SilverStripe to use the correct template
-     */
-    public function useTemplateOverride($data = null)
-    {
-        $template = $this->owner->AlternativeTemplate;
-        if (isset($template) && $template != '') {
-            if ($data) {
-                return $this->owner->customise(new ArrayData($data))
-                        ->renderWith(array($this->owner->AlternativeTemplate, $this->owner->ClassName, 'Page'));
-            } else {
-                return $this->owner->renderWith(array($this->owner->AlternativeTemplate, $this->owner->ClassName, 'Page'));
-            }
-        } else {
-            if ($data) {
-                return $data;
-            } else {
-                return array();
-            }
-        }
-    }
+	/**
+	 * Render this page using the template override iff it exists.
+	 *
+	 * @return array An array suitable for SilverStripe to use the correct template
+	 */
+	public function useTemplateOverride($data = null)
+	{
+		$template = $this->owner->AlternativeTemplate;
+		if (isset($template) && $template != '') {
+			if ($data) {
+				return $this->owner->customise(new ArrayData($data))
+						->renderWith(array($this->owner->AlternativeTemplate, $this->owner->ClassName, 'Page'));
+			} else {
+				return $this->owner->renderWith(array($this->owner->AlternativeTemplate, $this->owner->ClassName, 'Page'));
+			}
+		} else {
+			if ($data) {
+				return $data;
+			} else {
+				return array();
+			}
+		}
+	}
 }

tests/PageControllerTemplateOverrideExtensionTest.php

Indentation +42 added lines, -42 removed lines

@@ -2,51 +2,51 @@
 
 class PageControllerTemplateOverrideExtensionTest extends FunctionalTest
 {
-    protected static $fixture_file = 'template-override/tests/pages.yml';
+	protected static $fixture_file = 'template-override/tests/pages.yml';
 
-    public function testLayoutTemplateOveride()
-    {
-        $this->logInWithPermission('ADMIN');
-        $page = $this->objFromFixture('Page', 'page1');
-        $page->AlternativeTemplate = 'PageInnerTest';
-        $page->write();
-        $page->doPublish();
-        $response = $this->get('/'.$page->URLSegment);
-        $this->assertEquals(200, $response->getStatusCode());
+	public function testLayoutTemplateOveride()
+	{
+		$this->logInWithPermission('ADMIN');
+		$page = $this->objFromFixture('Page', 'page1');
+		$page->AlternativeTemplate = 'PageInnerTest';
+		$page->write();
+		$page->doPublish();
+		$response = $this->get('/'.$page->URLSegment);
+		$this->assertEquals(200, $response->getStatusCode());
 
-        // assert the the inner layout template has been used
-        $this->assertExactMatchBySelector('div.marker', array(
-            'INNER LAYOUT',
-        ));
-    }
+		// assert the the inner layout template has been used
+		$this->assertExactMatchBySelector('div.marker', array(
+			'INNER LAYOUT',
+		));
+	}
 
-    public function testOuterTemplateOveride()
-    {
-        $this->logInWithPermission('ADMIN');
-        $page = $this->objFromFixture('Page', 'page1');
-        $page->AlternativeTemplate = 'PageOuterTest';
-        $page->write();
-        $page->doPublish();
-        $response = $this->get('/'.$page->URLSegment);
-        $this->assertEquals(200, $response->getStatusCode());
+	public function testOuterTemplateOveride()
+	{
+		$this->logInWithPermission('ADMIN');
+		$page = $this->objFromFixture('Page', 'page1');
+		$page->AlternativeTemplate = 'PageOuterTest';
+		$page->write();
+		$page->doPublish();
+		$response = $this->get('/'.$page->URLSegment);
+		$this->assertEquals(200, $response->getStatusCode());
 
-        // show the the outer layout template has been used
-        $this->assertExactMatchBySelector('div.marker', array(
-            'OUTER OF LAYOUT',
-        ));
-    }
+		// show the the outer layout template has been used
+		$this->assertExactMatchBySelector('div.marker', array(
+			'OUTER OF LAYOUT',
+		));
+	}
 
-    public function testNoTemplateOverride()
-    {
-        $this->logInWithPermission('ADMIN');
-        $page = $this->objFromFixture('Page', 'page1');
-        $page->AlternativeTemplate = null;
-        $page->write();
-        $page->doPublish();
-        $response = $this->get('/'.$page->URLSegment);
-        $this->assertEquals(200, $response->getStatusCode());
-        $body = $response->getBody();
-        $this->assertNotContains('OUTER OF LAYOUT', $body);
-        $this->assertNotContains('INNER LAYOUT', $body);
-    }
+	public function testNoTemplateOverride()
+	{
+		$this->logInWithPermission('ADMIN');
+		$page = $this->objFromFixture('Page', 'page1');
+		$page->AlternativeTemplate = null;
+		$page->write();
+		$page->doPublish();
+		$response = $this->get('/'.$page->URLSegment);
+		$this->assertEquals(200, $response->getStatusCode());
+		$body = $response->getBody();
+		$this->assertNotContains('OUTER OF LAYOUT', $body);
+		$this->assertNotContains('INNER LAYOUT', $body);
+	}
 }

tests/TemplateOverrideExtensionTest.php

Indentation +14 added lines, -14 removed lines

@@ -2,19 +2,19 @@
 
 class TemplateOverrideExtensionTest extends SapphireTest
 {
-    protected static $fixture_file = 'template-override/tests/pages.yml';
+	protected static $fixture_file = 'template-override/tests/pages.yml';
 
-    public function testUpdateCMSFields()
-    {
-        $page = $this->objFromFixture('Page', 'page1');
-        $fields = $page->getCMSFields();
-        $tab = $fields->findOrMakeTab('Root.Template');
-        $fields = $tab->FieldList();
-        $names = array();
-        foreach ($fields as $field) {
-            $names[] = $field->getName();
-        }
-        $expected = array('AlternativeTemplate', 'infofield');
-        $this->assertEquals($expected, $names);
-    }
+	public function testUpdateCMSFields()
+	{
+		$page = $this->objFromFixture('Page', 'page1');
+		$fields = $page->getCMSFields();
+		$tab = $fields->findOrMakeTab('Root.Template');
+		$fields = $tab->FieldList();
+		$names = array();
+		foreach ($fields as $field) {
+			$names[] = $field->getName();
+		}
+		$expected = array('AlternativeTemplate', 'infofield');
+		$this->assertEquals($expected, $names);
+	}
 }