silverstripe / silverstripe-campaign-admin

src/AddToCampaignHandler_FormAction.php

Spacing +1 added lines, -1 removed lines

@@ -14,7 +14,7 @@
 {
     public function __construct()
     {
-        parent::__construct('addtocampaign', _t(__CLASS__.'.ADDTOCAMPAIGN', 'Add to Campaign'));
+        parent::__construct('addtocampaign', _t(__CLASS__ . '.ADDTOCAMPAIGN', 'Add to Campaign'));
         $this->setUseButtonTag(false);
         $this->addExtraClass('add-to-campaign-action');
         $this->addExtraClass('btn');

src/CampaignAdminList.php

Spacing +1 added lines, -1 removed lines

@@ -59,7 +59,7 @@
             "url" => $oneSetAction,
             "method" => "DELETE"
         ];
-        $data['data']['editFormSchemaEndpoint'] =  $schemaEndpoint;
+        $data['data']['editFormSchemaEndpoint'] = $schemaEndpoint;
 
         // Set summary columns
         $columns = [];

src/CampaignAdmin.php

Spacing +20 added lines, -20 removed lines

@@ -168,7 +168,7 @@ 
         $form->addExtraClass('form--padded');
 
         // Set callback response
-        $form->setValidationResponseCallback(function () use ($form) {
+        $form->setValidationResponseCallback(function() use ($form) {
             $schemaId = $this->Link('schema/EditForm');
             return $this->getSchemaResponse($form, $schemaId);
         });
@@ -215,7 +215,7 @@ 
                 'baseClass' => DataObject::getSchema()->baseDataClass($class),
                 'singular' => $item->i18n_singular_name(),
                 'plural' => $item->i18n_plural_name(),
-                'noItemsText' => _t(__CLASS__.'.NOITEMSTEXT', 'Add items from the {section} section', [
+                'noItemsText' => _t(__CLASS__ . '.NOITEMSTEXT', 'Add items from the {section} section', [
                     'section' => $item->i18n_plural_name(),
                 ]),
                 'items' => []
@@ -283,7 +283,7 @@ 
             'Created' => $changeSet->Created,
             'LastEdited' => $changeSet->LastEdited,
             'State' => $changeSet->State,
-            'StateLabel' => [ 'html' => $stateLabel ],
+            'StateLabel' => ['html' => $stateLabel],
             'IsInferred' => $changeSet->IsInferred,
             'canEdit' => $changeSet->canEdit(),
             'canPublish' => false,
@@ -338,8 +338,8 @@ 
             return [];
         }
 
-        $thumbnailWidth = (int)$this->config()->get('thumbnail_width');
-        $thumbnailHeight = (int)$this->config()->get('thumbnail_height');
+        $thumbnailWidth = (int) $this->config()->get('thumbnail_width');
+        $thumbnailHeight = (int) $this->config()->get('thumbnail_height');
         $hal = [
             '_links' => [
                 'self' => [
@@ -419,7 +419,7 @@ 
             $changesets = $changesets->filter('IsInferred', 0);
         }
         return $changesets
-            ->filterByCallback(function ($item) {
+            ->filterByCallback(function($item) {
                 /** @var ChangeSet $item */
                 return ($item->canView());
             });
@@ -626,19 +626,19 @@ 
             'campaignEditForm',
             $fields,
             FieldList::create(
-                FormAction::create('save', _t(__CLASS__.'SAVE', 'Save'))
+                FormAction::create('save', _t(__CLASS__ . 'SAVE', 'Save'))
                     ->setIcon('save')
                     ->setSchemaState([
                         'data' => [
-                            'pristineTitle' => _t(__CLASS__.'SAVED', 'Saved'),
+                            'pristineTitle' => _t(__CLASS__ . 'SAVED', 'Saved'),
                             'pristineIcon' => 'tick',
-                            'dirtyTitle' => _t(__CLASS__.'SAVE', 'Save'),
+                            'dirtyTitle' => _t(__CLASS__ . 'SAVE', 'Save'),
                             'dirtyIcon' => 'save',
                             'pristineClass' => 'btn-outline-primary',
                             'dirtyClass' => '',
                         ],
                     ]),
-                FormAction::create('cancel', _t(__CLASS__.'.CANCEL', 'Cancel'))
+                FormAction::create('cancel', _t(__CLASS__ . '.CANCEL', 'Cancel'))
                     ->setUseButtonTag(true)
             ),
             new RequiredFields('Name')
@@ -649,12 +649,12 @@ 
 
         // Set form action handler with ID included
         $form->setRequestHandler(
-            LeftAndMainFormRequestHandler::create($form, [ $id ])
+            LeftAndMainFormRequestHandler::create($form, [$id])
         );
 
         // Configure form to respond to validation errors with form schema
         // if requested via react.
-        $form->setValidationResponseCallback(function (ValidationResult $errors) use ($form, $id, $record) {
+        $form->setValidationResponseCallback(function(ValidationResult $errors) use ($form, $id, $record) {
             $schemaId = Controller::join_links(
                 $this->Link('schema'),
                 'campaignEditForm',
@@ -701,9 +701,9 @@ 
             'campaignCreateForm',
             $fields,
             FieldList::create(
-                FormAction::create('save', _t(__CLASS__.'.CREATE', 'Create'))
+                FormAction::create('save', _t(__CLASS__ . '.CREATE', 'Create'))
                     ->setIcon('plus'),
-                FormAction::create('cancel', _t(__CLASS__.'.CANCEL', 'Cancel'))
+                FormAction::create('cancel', _t(__CLASS__ . '.CANCEL', 'Cancel'))
                     ->setUseButtonTag(true)
             ),
             new RequiredFields('Name')
@@ -716,7 +716,7 @@ 
 
         // Configure form to respond to validation errors with form schema
         // if requested via react.
-        $form->setValidationResponseCallback(function (ValidationResult $errors) use ($form, $record) {
+        $form->setValidationResponseCallback(function(ValidationResult $errors) use ($form, $record) {
             $schemaId = $this->Link('schema/campaignCreateForm');
             return $this->getSchemaResponse($schemaId, $form, $errors);
         });
@@ -745,7 +745,7 @@ 
                 return Security::permissionFailure($this);
             }
             if (!$record || !$record->ID) {
-                $this->httpError(404, "Bad record ID #" . (int)$id);
+                $this->httpError(404, "Bad record ID #" . (int) $id);
             }
         } else {
             if (!ChangeSet::singleton()->canCreate()) {
@@ -764,13 +764,13 @@ 
             $form->saveInto($record, true);
             $record->write();
             $this->extend('onAfterSave', $record);
-            $message = _t(__CLASS__.'.SAVEDUP', 'Saved.');
+            $message = _t(__CLASS__ . '.SAVEDUP', 'Saved.');
             $form->setMessage($message, ValidationResult::TYPE_GOOD);
         } else {
-            $nameDuplicateMsg = _t(__CLASS__ . '.ERROR_DUPLICATE_NAME', 'Name "{Name}" already exists', '', [ 'Name' => $data['Name']]);
+            $nameDuplicateMsg = _t(__CLASS__ . '.ERROR_DUPLICATE_NAME', 'Name "{Name}" already exists', '', ['Name' => $data['Name']]);
             $errors = new ValidationResult();
             $errors->addFieldMessage('Name', $nameDuplicateMsg);
-            $message = _t(__CLASS__.'.SAVEDERROR', 'Error.');
+            $message = _t(__CLASS__ . '.SAVEDERROR', 'Error.');
             // Need to set the form message or the field message won't show up at all
             $form->setMessage($message, ValidationResult::TYPE_ERROR);
         }
@@ -828,7 +828,7 @@ 
                 ),
                 'category' => _t('SilverStripe\\Security\\Permission.CMS_ACCESS_CATEGORY', 'CMS Access'),
                 'help' => _t(
-                    __CLASS__.'.ACCESS_HELP',
+                    __CLASS__ . '.ACCESS_HELP',
                     'Allow viewing of the campaign publishing section.'
                 )
             )

src/AddToCampaignHandler.php

Spacing +3 added lines, -3 removed lines

@@ -120,7 +120,7 @@ 
                 'State' => ChangeSet::STATE_OPEN,
                 'IsInferred' => 0
             ])
-            ->filterByCallback(function ($item) {
+            ->filterByCallback(function($item) {
                 /** @var ChangeSet $item */
                 return $item->canView();
             });
@@ -158,7 +158,7 @@ 
      */
     protected function getObject($id, $class)
     {
-        $id = (int)$id;
+        $id = (int) $id;
         $class = ClassInfo::class_name($class);
 
         if (!$class
@@ -456,7 +456,7 @@ 
         if (is_array($data) && !empty($data['Campaign'])) {
             $campaignID = $data['Campaign'];
         } elseif (is_numeric($data)) {
-            $campaignID = (int)$data;
+            $campaignID = (int) $data;
         }
         if (empty($campaignID)) {
             throw $this->validationResult(_t(__CLASS__ . '.NONE_SELECTED', 'No campaign selected'));

tests/php/CampaignAdminTest.php

Spacing +10 added lines, -10 removed lines

@@ -94,7 +94,7 @@ 
         $results = $this->callProtectedMethod($admin, 'getListItems');
         $this->assertDOSEquals(
             [
-                [ 'Name' => 'changeset 1' ],
+                ['Name' => 'changeset 1'],
             ],
             $results
         );
@@ -104,8 +104,8 @@ 
         $results = $this->callProtectedMethod($admin, 'getListItems');
         $this->assertDOSEquals(
             [
-                [ 'Name' => 'changeset 1' ],
-                [ 'Name' => 'changeset 2' ],
+                ['Name' => 'changeset 1'],
+                ['Name' => 'changeset 2'],
             ],
             $results
         );
@@ -115,10 +115,10 @@ 
         $results = $this->callProtectedMethod($admin, 'getListItems');
         $this->assertDOSEquals(
             [
-                [ 'Name' => 'changeset 1' ],
-                [ 'Name' => 'changeset 2' ],
-                [ 'Name' => 'changeset 3' ],
-                [ 'Name' => 'changeset 4' ],
+                ['Name' => 'changeset 1'],
+                ['Name' => 'changeset 2'],
+                ['Name' => 'changeset 3'],
+                ['Name' => 'changeset 4'],
             ],
             $results
         );
@@ -128,8 +128,8 @@ 
         $results = $this->callProtectedMethod($admin, 'getListItems');
         $this->assertDOSEquals(
             [
-                [ 'Name' => 'changeset 1' ],
-                [ 'Name' => 'changeset 3' ],
+                ['Name' => 'changeset 1'],
+                ['Name' => 'changeset 3'],
             ],
             $results
         );
@@ -177,7 +177,7 @@ 
         $user = $this->objFromFixture(Member::class, 'mock_user');
         $this->logInAs($user);
 
-        $request->setRouteParams([ 'ID' => '1', 'Name' => 'show']);
+        $request->setRouteParams(['ID' => '1', 'Name' => 'show']);
         $response = $campaignAdmin->readCampaign($request);
         $status = $response->getStatusCode();