silverstripe / silverstripe-siteconfig

code/SiteConfig.php

Doc Comments +4 added lines, -1 removed lines

@@ -277,7 +277,7 @@ 
      * Can a user view this SiteConfig instance?
      *
      * @param Member $member
-     * @return boolean
+     * @return boolean|string
      */
     public function canView($member = null)
     {
@@ -372,6 +372,9 @@ 
         return false;
     }
 
+    /**
+     * @param Member $member
+     */
     public function canEdit($member = null)
     {
         if (!$member) {

Spacing +4 added lines, -4 removed lines

@@ -84,7 +84,7 @@ 
      */
     public function getCMSFields()
     {
-        $mapFn = function ($groups = []) {
+        $mapFn = function($groups = []) {
             $map = [];
             foreach ($groups as $group) {
                 // Listboxfield values are escaped, use ASCII char instead of »
@@ -173,15 +173,15 @@ 
             $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(
+                    "<p class=\"message warning\">"._t(
                         "SilverStripe\\CMS\\Model\\SiteTree.REMOVE_INSTALL_WARNING",
                         "Warning: You should remove install.php from this SilverStripe install for security reasons."
-                    ) . "</p>"
+                    )."</p>"
                 ),
                 "Title"
             );

code/SiteConfigLeftAndMain.php

Doc Comments +1 added lines, -1 removed lines

@@ -139,7 +139,7 @@
      *
      * @param array $data
      * @param Form $form
-     * @return String
+     * @return \SilverStripe\ORM\FieldType\DBHTMLText
      */
     public function save_siteconfig($data, $form)
     {

Spacing +2 added lines, -2 removed lines

@@ -99,12 +99,12 @@
             $actions,
             $validator
         )->setHTMLID('Form_EditForm');
-        $form->setValidationResponseCallback(function (ValidationResult $errors) use ($negotiator, $form) {
+        $form->setValidationResponseCallback(function(ValidationResult $errors) use ($negotiator, $form) {
             $request = $this->getRequest();
             if ($request->isAjax() && $negotiator) {
                 $result = $form->forTemplate();
                 return $negotiator->respond($request, array(
-                    'CurrentForm' => function () use ($result) {
+                    'CurrentForm' => function() use ($result) {
                         return $result;
                     }
                 ));

tests/bootstrap/mysite.php

Spacing +2 added lines, -2 removed lines

@@ -4,6 +4,6 @@
     define('BASE_PATH', getcwd());
 }
 
-require_once BASE_PATH . '/cms/tests/bootstrap/mysite.php';
+require_once BASE_PATH.'/cms/tests/bootstrap/mysite.php';
 
-copy(__DIR__ . '/fixtures/theme.yml.fixture', $projectPath . '/_config/dummytheme.yml');
+copy(__DIR__.'/fixtures/theme.yml.fixture', $projectPath.'/_config/dummytheme.yml');