Formatting and code cleanup

Author: bryannielsen

src/Api/Graph/Support/FieldtypeRegistrar.php

@@ -107,15 +107,15 @@ public function registerField(ChannelField $field)
             return $this->types[$field->field_name];
         }
 
-        if (!$fieldtype instanceof GeneratesGraphType) {
+        if (! $fieldtype instanceof GeneratesGraphType) {
             return null;
         }
 
         $name = "Field__{$field->field_name}";
         $typeDefinition = $fieldtype->generateGraphType($field);
 
-        if(!$typeDefinition instanceof GraphQLType) {
-            throw new \Exception("Generated GraphQL Type for field {$field->field_name} must extend ".GraphQLType::class.".");
+        if (! $typeDefinition instanceof GraphQLType) {
+            throw new \Exception("Generated GraphQL Type for field {$field->field_name} must extend ".GraphQLType::class.'.');
         }
 
         $typeDefinition->name = $name;

src/Bootstrap/LoadExpressionEngine.php

@@ -67,7 +67,7 @@ public function cli()
         // fake SERVER vars for CLI context
         $_SERVER['REMOTE_ADDR'] = '127.0.0.1';
         $this->constants['REQ'] = 'CLI';
-        $this->constants['EE_INSTALLED'] = file_exists($this->constants['SYSPATH'] . 'user/config/config.php');
+        $this->constants['EE_INSTALLED'] = file_exists($this->constants['SYSPATH'].'user/config/config.php');
 
         return $this;
     }

src/Commands/EECliCommand.php

@@ -7,20 +7,19 @@
 
 class EECliCommand extends Command
 {
-
     use CanAccessRestrictedClass;
 
     public $signature = 'eecli {args?*}';
 
-    public $description = "Run ExpressionEngine commands through Coilpack";
+    public $description = 'Run ExpressionEngine commands through Coilpack';
 
     private $cli;
 
     public function __construct()
     {
         parent::__construct();
 
-        if(($_SERVER['argv'][1] ?? '') != 'eecli') {
+        if (($_SERVER['argv'][1] ?? '') != 'eecli') {
             return;
         }
 
@@ -29,10 +28,10 @@ public function __construct()
 
         $this->setupCommand($_SERVER['argv'][2] ?? 'list');
 
-        foreach($_SERVER['argv'] as $arg) {
+        foreach ($_SERVER['argv'] as $arg) {
             // If we have a help flag we need to get out early so that it can be
             // handled by the ExpressionEngine Command and not by Symfony/Laravel
-            if($arg == '--help') {
+            if ($arg == '--help') {
                 return $this->handle();
             }
         }
@@ -42,7 +41,7 @@ protected function setupCommand($command)
     {
         $availableCommands = $this->callRestrictedMethod($this->cli, 'availableCommands');
 
-        if (!array_key_exists($command, $availableCommands)) {
+        if (! array_key_exists($command, $availableCommands)) {
             exit("Command '$command' not found.");
         }
 
@@ -61,23 +60,22 @@ protected function setupCommand($command)
         $this->callRestrictedMethod($commandClass, 'loadOptions');
         // Convert ExpressionEngine Command's option syntax into Laravel's and update the signature
         $options = $this->getRestrictedProperty($commandClass, 'commandOptions');
-        $this->signature = $this->signature . ' ' . implode(' ', array_map(function($option) {
-            if(strpos($option, 'verbose') !== false) {
+        $this->signature = $this->signature.' '.implode(' ', array_map(function ($option) {
+            if (strpos($option, 'verbose') !== false) {
                 return '';
             }
 
             $pieces = explode(',', $option);
-            if(count($pieces) == 2) {
-                $option = substr($pieces[1], 0, 1) . '|' . $pieces[0];
+            if (count($pieces) == 2) {
+                $option = substr($pieces[1], 0, 1).'|'.$pieces[0];
             }
-            return "{--".$option."=}";
+
+            return '{--'.$option.'=}';
         }, array_keys($options)));
     }
 
-
     public function handle(): int
     {
         return $this->cli->process();
     }
-
 }

src/Core.php

@@ -4,7 +4,6 @@
 
 class Core
 {
-
     use Traits\CanAccessRestrictedClass;
 
     protected $core;

src/Fieldtypes/Generic.php

@@ -61,7 +61,7 @@ public function apply(FieldContent $content, array $parameters = [])
         $data = $content->getAttribute('data');
 
         // Run pre_process if it exists
-        if(method_exists($handler, 'pre_process')) {
+        if (method_exists($handler, 'pre_process')) {
             $data = $handler->pre_process($data);
         }
 

src/Fieldtypes/Modifiers/File.php

@@ -22,9 +22,9 @@ public function handle(FieldtypeOutput $content, $parameters = [])
         $modified = (is_array($modified) && count($modified) === 1 && is_array(current($modified))) ? $modified[0] : $modified;
 
         // If a manipulation fails and returns a boolean we fallback to the original url
-        if(is_bool($modified)) {
+        if (is_bool($modified)) {
             $modified = [
-                'url' => $data['url'] ?? ''
+                'url' => $data['url'] ?? '',
             ];
         }
 

src/Traits/CanAccessRestrictedClass.php

@@ -2,8 +2,8 @@
 
 namespace Expressionengine\Coilpack\Traits;
 
-trait CanAccessRestrictedClass {
-
+trait CanAccessRestrictedClass
+{
     protected function getRestrictedProperty($object, $property)
     {
         $reflection = new \ReflectionClass($object);
@@ -28,5 +28,4 @@ public function callRestrictedMethod($object, $method, $parameters = [])
 
         return $method->invoke($object, ...$parameters);
     }
-
-}
+}

src/View/ModelTag.php

@@ -2,8 +2,8 @@
 
 namespace Expressionengine\Coilpack\View;
 
-use Expressionengine\Coilpack\Support\Parameter;
 use Expressionengine\Coilpack\Support\Arguments\ListArgument;
+use Expressionengine\Coilpack\Support\Parameter;
 
 abstract class ModelTag extends IterableTag
 {

src/View/Tags/Channel/Entries.php

@@ -194,7 +194,7 @@ public function getWithArgument($value)
     {
         // If the 'data' relation is requested we will append 'channel' and 'hiddenFields'
         // because these are necessary for properly displaying custom field data
-        if(in_array('data', explode('|', $value))) {
+        if (in_array('data', explode('|', $value))) {
             $value .= '|channel|hiddenFields';
         }