silverstripe / silverstripe-siteconfig

code/controller/SiteConfigLeftAndMain.php

Doc Comments +1 added lines, -1 removed lines

@@ -107,7 +107,7 @@
 	/**
 	 * Used for preview controls, mainly links which switch between different states of the page.
 	 *
-	 * @return ArrayData
+	 * @return SilverStripe\Model\FieldType\DBField
 	 */
     public function getSilverStripeNavigator()
     {

Spacing +2 added lines, -2 removed lines

@@ -70,7 +70,7 @@ 
 		)->setHTMLID('Form_EditForm');
 		$form->setValidationResponseCallback(function() use ($negotiator, $form) {
 			$request = $this->getRequest();
-			if($request->isAjax() && $negotiator) {
+			if ($request->isAjax() && $negotiator) {
 				$form->setupFormErrors();
 				$result = $form->forTemplate();
 
@@ -128,7 +128,7 @@ 
 		
 		try {
 			$siteConfig->write();
-		} catch(ValidationException $ex) {
+		} catch (ValidationException $ex) {
 			$form->sessionMessage($ex->getResult()->message(), 'bad');
 			return $this->getResponseNegotiator()->respond($this->request);
 		}

tests/behat/features/bootstrap/SilverStripe/SiteConfig/Test/Behaviour/ThemeContext.php

Doc Comments +6 added lines

@@ -54,6 +54,9 @@ 
         $this->requireFile(BASE_PATH . '/themes/' . $theme . '/templates/' . $template, $content);
     }
 
+    /**
+     * @param string $filename
+     */
     protected function requireFile($filename, $content)
     {
         // Already exists
@@ -71,6 +74,9 @@ 
         }
     }
 
+    /**
+     * @param string $dirname
+     */
     protected function requireDir($dirname)
     {
         // Directory doesn't exist, create it and mark it for deletion

code/model/SiteConfig.php

Spacing +1 added lines, -1 removed lines

@@ -165,7 +165,7 @@
     public function getAvailableThemes($baseDir = null)
     {
         $themes = SSViewer::get_themes($baseDir);
-        $disabled = (array)$this->config()->disabled_themes;
+        $disabled = (array) $this->config()->disabled_themes;
     
         foreach ($disabled as $theme) {
             if (isset($themes[$theme])) {

tests/behat/features/bootstrap/Context/FeatureContext.php

Spacing +2 added lines, -2 removed lines

@@ -33,7 +33,7 @@ 
 
         // Add extra contexts with more steps
         $this->useContext('ThemeContext', new \SilverStripe\Siteconfig\Test\Behaviour\ThemeContext($parameters));
-        if(!class_exists('SiteTree')) {
+        if (!class_exists('SiteTree')) {
             return;
         }
 
@@ -41,7 +41,7 @@ 
         $factory = $fixtureContext->getFixtureFactory();
         foreach (\ClassInfo::subclassesFor('SiteTree') as $id => $class) {
             $blueprint = \Injector::inst()->create('FixtureBlueprint', $class);
-            $blueprint->addCallback('afterCreate', function ($obj, $identifier, &$data, &$fixtures) {
+            $blueprint->addCallback('afterCreate', function($obj, $identifier, &$data, &$fixtures) {
                 $obj->publish('Stage', 'Live');
             });
             $factory->define($class, $blueprint);