silverstripe / silverstripe-siteconfig

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

Unused Use Statements -8 removed lines

@@ -2,15 +2,7 @@
 
 namespace SilverStripe\Siteconfig\Test\Behaviour;
 
-use Behat\Behat\Context\ClosuredContextInterface;
-use Behat\Behat\Context\TranslatedContextInterface;
 use Behat\Behat\Context\BehatContext;
-use Behat\Behat\Context\Step;
-use Behat\Behat\Event\StepEvent;
-use Behat\Behat\Exception\PendingException;
-use Behat\Mink\Driver\Selenium2Driver;
-use Behat\Gherkin\Node\PyStringNode;
-use Behat\Gherkin\Node\TableNode;
 
 /**
  * Context used to create fixtures in the SilverStripe ORM.

Spacing +4 added lines, -4 removed lines

@@ -31,9 +31,9 @@ 
             throw new \InvalidArgumentException("Bad theme '$theme'");
         }
 
-        $this->requireDir(BASE_PATH . '/themes');
-        $this->requireDir(BASE_PATH . '/themes/' . $theme);
-        $this->requireDir(BASE_PATH . '/themes/' . $theme . '/templates');
+        $this->requireDir(BASE_PATH.'/themes');
+        $this->requireDir(BASE_PATH.'/themes/'.$theme);
+        $this->requireDir(BASE_PATH.'/themes/'.$theme.'/templates');
     }
 
     /**
@@ -51,7 +51,7 @@ 
         }
 
         $this->stepCreateTheme($theme);
-        $this->requireFile(BASE_PATH . '/themes/' . $theme . '/templates/' . $template, $content);
+        $this->requireFile(BASE_PATH.'/themes/'.$theme.'/templates/'.$template, $content);
     }
 
     protected function requireFile($filename, $content)

code/SiteConfigLeftAndMain.php

Unused Use Statements -1 removed lines

@@ -8,7 +8,6 @@
 use SilverStripe\Forms\HiddenField;
 use SilverStripe\Forms\LiteralField;
 use SilverStripe\Forms\Form;
-use SilverStripe\ORM\ValidationException;
 use SilverStripe\ORM\ArrayList;
 use SilverStripe\ORM\ValidationResult;
 use SilverStripe\View\ArrayData;

Spacing +2 added lines, -2 removed lines

@@ -57,7 +57,7 @@ 
 	public function init() {
 		parent::init();
         if (defined('CMS_DIR')) {
-            Requirements::javascript(CMS_DIR . '/client/dist/js/bundle.js');
+            Requirements::javascript(CMS_DIR.'/client/dist/js/bundle.js');
         }
 	}
 
@@ -96,7 +96,7 @@ 
 		)->setHTMLID('Form_EditForm');
 		$form->setValidationResponseCallback(function(ValidationResult $errors) use ($negotiator, $form) {
 			$request = $this->getRequest();
-			if($request->isAjax() && $negotiator) {
+			if ($request->isAjax() && $negotiator) {
 				$result = $form->forTemplate();
 				return $negotiator->respond($request, array(
 					'CurrentForm' => function() use($result) {

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

Unused Use Statements -3 removed lines

@@ -2,9 +2,6 @@
 
 namespace SilverStripe\Siteconfig\Test\Behaviour;
 
-use SilverStripe\BehatExtension\Context\SilverStripeContext;
-use SilverStripe\BehatExtension\Context\BasicContext;
-use SilverStripe\BehatExtension\Context\LoginContext;
 use SilverStripe\BehatExtension\Context\FixtureContext;
 use SilverStripe\Cms\Test\Behaviour;
 use SilverStripe\Core\ClassInfo;

Spacing +2 added lines, -2 removed lines

@@ -36,7 +36,7 @@ 
 
         // Add extra contexts with more steps
         $this->useContext('ThemeContext', new \SilverStripe\Siteconfig\Test\Behaviour\ThemeContext($parameters));
-        if(!class_exists('SilverStripe\\CMS\\Model\\SiteTree')) {
+        if (!class_exists('SilverStripe\\CMS\\Model\\SiteTree')) {
             return;
         }
 
@@ -44,7 +44,7 @@ 
         $factory = $fixtureContext->getFixtureFactory();
         foreach (ClassInfo::subclassesFor('SilverStripe\\CMS\\Model\\SiteTree') as $id => $class) {
             $blueprint = Injector::inst()->create('SilverStripe\\Dev\\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);

code/SiteConfig.php

Spacing +2 added lines, -2 removed lines

@@ -148,9 +148,9 @@
             $fields->makeFieldReadonly($titleField);
         }
 
-        if (file_exists(BASE_PATH . '/install.php')) {
+        if (file_exists(BASE_PATH.'/install.php')) {
             $fields->addFieldToTab("Root.Main", new LiteralField("InstallWarningHeader",
-                "<p class=\"message warning\">" . _t("SiteTree.REMOVE_INSTALL_WARNING",
+                "<p class=\"message warning\">"._t("SiteTree.REMOVE_INSTALL_WARNING",
                 "Warning: You should remove install.php from this SilverStripe install for security reasons.")
                 . "</p>"), "Title");
         }