thinktomorrow / chief

src/Fields/FieldName.php

Spacing +1 added lines, -1 removed lines

@@ -79,7 +79,7 @@
             return $value;
         }
 
-        $value = str_replace('.', '][', $value) . ']';
+        $value = str_replace('.', '][', $value).']';
 
         return substr_replace($value, '', strpos($value, ']'), 1);
     }

src/Fields/ValidationRules/ValidatesExistingAssetAttributes.php

Spacing +1 added lines, -1 removed lines

@@ -56,7 +56,7 @@
     protected function validateAssetMimetypes(Asset $asset, $parameters)
     {
         return (in_array($asset->getMimeType(), $parameters) ||
-            in_array(explode('/', $asset->getMimeType())[0] . '/*', $parameters));
+            in_array(explode('/', $asset->getMimeType())[0].'/*', $parameters));
     }
 
     protected function validateAssetMax(Asset $asset, $parameters)

src/FlatReferences/FlatReference.php

Spacing +1 added lines, -1 removed lines

@@ -39,7 +39,7 @@
 
     public function get(): string
     {
-        return $this->className . '@' . $this->id;
+        return $this->className.'@'.$this->id;
     }
 
     public function equals($other): bool

src/FlatReferences/FlatReferenceFactory.php

Spacing +2 added lines, -2 removed lines

@@ -7,7 +7,7 @@ 
     public static function fromString(string $reference): FlatReference
     {
         if (false == strpos($reference, '@')) {
-            throw new \InvalidArgumentException('Invalid reference composition. A flat reference should honour schema <class>@<id>. [' . $reference . '] was passed instead.');
+            throw new \InvalidArgumentException('Invalid reference composition. A flat reference should honour schema <class>@<id>. ['.$reference.'] was passed instead.');
         }
 
         list($className, $id) = explode('@', $reference);
@@ -16,7 +16,7 @@ 
         $instance->{$instance->getKeyName()} = $id;
 
         if (!method_exists($instance, 'flatReference')) {
-            throw new \InvalidArgumentException('Instance created from model reference [' . $reference . '] was expected to have a method of flatReference() but is has not.');
+            throw new \InvalidArgumentException('Instance created from model reference ['.$reference.'] was expected to have a method of flatReference() but is has not.');
         }
 
         return $instance->flatReference();

src/Sets/Set.php

Spacing +4 added lines, -4 removed lines

@@ -49,7 +49,7 @@ 
 
         // If no view has been created for this page collection, we try once again to fetch the content value if any. This will silently fail
         // if no content value is present. We don't consider the 'content' attribute to be a default as we do for module.
-        return $this->map(function ($item) {
+        return $this->map(function($item) {
             return ($this->viewParent && $item instanceof ViewableContract)
                 ? $item->setViewParent($this->viewParent)->renderView()
                 : ($item->content ?? '');
@@ -82,7 +82,7 @@ 
     {
         $currentPage = $currentPage ?? request()->get('page', 1);
         $path = request()->path();
-        $items = array_slice($this->all(), ($currentPage - 1) * $perPage);
+        $items = array_slice($this->all(), ($currentPage-1) * $perPage);
 
         return (new \Illuminate\Pagination\Paginator($items, $perPage ?? $this->paginateSetting('perPage', '12'), $currentPage))->setPath($path);
     }
@@ -98,8 +98,8 @@ 
     public function paginate($perPage = null, $currentPage = null): Paginator
     {
         $currentPage = $currentPage ?? request()->get('page', 1);
-        $path = '/' . request()->path();
-        $items = array_slice($this->all(), ($currentPage - 1) * $perPage, $perPage);
+        $path = '/'.request()->path();
+        $items = array_slice($this->all(), ($currentPage-1) * $perPage, $perPage);
 
         return (new \Illuminate\Pagination\LengthAwarePaginator($items, $this->paginateSetting('total', $this->count()), $perPage ?? $this->paginateSetting('perPage', '12'), $currentPage))->setPath($path);
     }

src/Sets/SetReference.php

Spacing +5 added lines, -5 removed lines

@@ -36,7 +36,7 @@ 
     {
         // Constraints
         if (!isset($values['action'])) {
-            throw new \InvalidArgumentException('Set reference array is missing required values for the "action" keys. Given: ' . print_r($values, true));
+            throw new \InvalidArgumentException('Set reference array is missing required values for the "action" keys. Given: '.print_r($values, true));
         }
 
         return new static(
@@ -51,14 +51,14 @@ 
     {
         $sets = config('thinktomorrow.chief.sets', []);
 
-        return collect($sets)->map(function ($set, $key) {
+        return collect($sets)->map(function($set, $key) {
             return SetReference::fromArray($key, $set);
         });
     }
 
     public static function find($key): ?SetReference
     {
-        return static::all()->filter(function ($ref) use ($key) {
+        return static::all()->filter(function($ref) use ($key) {
             return $ref->key() == $key;
         })->first();
     }
@@ -150,11 +150,11 @@ 
     private static function validateAction($class, $method)
     {
         if (!class_exists($class)) {
-            throw new \InvalidArgumentException('The class [' . $class . '] isn\'t a valid class reference or does not exist in the chief-settings.sets config entry.');
+            throw new \InvalidArgumentException('The class ['.$class.'] isn\'t a valid class reference or does not exist in the chief-settings.sets config entry.');
         }
 
         if (!method_exists($class, $method)) {
-            throw new \InvalidArgumentException('The method [' . $method . '] does not exist on the class [' . $class . ']. Make sure you provide a valid method to the action value in the chief-settings.sets config entry.');
+            throw new \InvalidArgumentException('The method ['.$method.'] does not exist on the class ['.$class.']. Make sure you provide a valid method to the action value in the chief-settings.sets config entry.');
         }
     }
 

src/Sets/StoredSetReference.php

Spacing +2 added lines, -2 removed lines

@@ -35,12 +35,12 @@
 
     public function toReference(): SetReference
     {
-        $reference = SetReference::all()->first(function ($setReference) {
+        $reference = SetReference::all()->first(function($setReference) {
             return $setReference->key() == $this->key;
         });
 
         if (!$reference) {
-            throw new \Exception('No query set found by key [' . $this->key . ']. Make sure that this ' . $this->key . ' set is added to the chief.sets config array.');
+            throw new \Exception('No query set found by key ['.$this->key.']. Make sure that this '.$this->key.' set is added to the chief.sets config array.');
         }
 
         return $reference;

src/Authorization/Permission.php

Spacing +3 added lines, -3 removed lines

@@ -26,15 +26,15 @@
     {
         $abilities = ['view', 'create', 'update', 'delete'];
 
-        return array_map(function ($val) use ($scope) {
-            return $val . '-' . $scope;
+        return array_map(function($val) use ($scope) {
+            return $val.'-'.$scope;
         }, $abilities);
     }
 
     public static function getPermissionsForIndex()
     {
         $permissions = $temp = [];
-        self::all()->each(function ($permission) use (&$permissions, &$temp) {
+        self::all()->each(function($permission) use (&$permissions, &$temp) {
             $model = explode("_", $permission->name, 2)[1];
             $temp[$model][$permission->id] = explode("_", $permission->name, 2)[0];
             $permissions = $temp;

src/Pages/Console/GeneratePage.php

Spacing +21 added lines, -21 removed lines

@@ -37,10 +37,10 @@ 
 
         // Set required paths
         $this->dirs = ['base' => $this->settings['base_path'] ?? base_path()];
-        $this->dirs['model'] = $this->settings['model_path'] ?? $this->dirs['base'] . '/' . config('thinktomorrow.chief.domain.path', 'src');
-        $this->dirs['views'] = $this->settings['views_path'] ?? $this->dirs['base'] . '/resources/views';
-        $this->dirs['controller'] = $this->settings['controller_path'] ?? $this->dirs['base'] . '/app/Http/Controllers';
-        $this->files['routes'] = $this->settings['routes_file'] ?? $this->dirs['base'] . '/routes/web.php';
+        $this->dirs['model'] = $this->settings['model_path'] ?? $this->dirs['base'].'/'.config('thinktomorrow.chief.domain.path', 'src');
+        $this->dirs['views'] = $this->settings['views_path'] ?? $this->dirs['base'].'/resources/views';
+        $this->dirs['controller'] = $this->settings['controller_path'] ?? $this->dirs['base'].'/app/Http/Controllers';
+        $this->files['routes'] = $this->settings['routes_file'] ?? $this->dirs['base'].'/routes/web.php';
     }
 
     public function handle()
@@ -62,19 +62,19 @@ 
     private function publishModel()
     {
         $this->publishFile(
-            __DIR__ . '/stubs/model.php.stub',
-            $to = $this->dirs['model'] . '/' . ucfirst($this->plural) . '/' . ucfirst($this->singular) . '.php',
+            __DIR__.'/stubs/model.php.stub',
+            $to = $this->dirs['model'].'/'.ucfirst($this->plural).'/'.ucfirst($this->singular).'.php',
             'model'
         );
 
-        $this->addToConfig('collections', [$this->plural => $this->guessNamespace() . '\\' . ucfirst($this->singular)]);
+        $this->addToConfig('collections', [$this->plural => $this->guessNamespace().'\\'.ucfirst($this->singular)]);
     }
 
     private function publishController()
     {
         $this->publishFile(
-            __DIR__ . '/stubs/controller.php.stub',
-            $to = $this->dirs['controller'] . '/' . ucfirst($this->plural) . '/' . ucfirst($this->singular) . '.php',
+            __DIR__.'/stubs/controller.php.stub',
+            $to = $this->dirs['controller'].'/'.ucfirst($this->plural).'/'.ucfirst($this->singular).'.php',
             'controller'
         );
     }
@@ -113,8 +113,8 @@ 
     private function modelTraits()
     {
         return [
-            '\\' . Publishable::class,
-            '\\' . Sortable::class,
+            '\\'.Publishable::class,
+            '\\'.Sortable::class,
             'q' => 'Proceed.',
         ];
     }
@@ -129,7 +129,7 @@ 
     protected function publishFile($from, $to, $type)
     {
         if ($this->filesystem->exists($to) && !$this->option('force')) {
-            if (!$this->confirm('File [' . $to . '] already exists? Overwrite this file?')) {
+            if (!$this->confirm('File ['.$to.'] already exists? Overwrite this file?')) {
                 return;
             }
         }
@@ -168,7 +168,7 @@ 
 
         $to = str_replace($this->dirs['base'], '', realpath($to));
 
-        $this->line('<info>Copied ' . $type . '</info> <comment>[' . $from . ']</comment> <info>To</info> <comment>[' . $to . ']</comment>');
+        $this->line('<info>Copied '.$type.'</info> <comment>['.$from.']</comment> <info>To</info> <comment>['.$to.']</comment>');
     }
 
     protected function replacePlaceholders($content)
@@ -186,17 +186,17 @@ 
 
     private function generateImportStatements(): string
     {
-        return collect(['\\' . Page::class])
-            ->map(function ($statement) {
-                return 'use ' . $statement . ";\n    ";
+        return collect(['\\'.Page::class])
+            ->map(function($statement) {
+                return 'use '.$statement.";\n    ";
             })->implode('');
     }
 
     private function generateTraitStatements(): string
     {
         return collect($this->chosenTraits)
-            ->map(function ($statement) {
-                return 'use ' . $statement . ";\n    ";
+            ->map(function($statement) {
+                return 'use '.$statement.";\n    ";
             })->implode('');
     }
 
@@ -215,12 +215,12 @@ 
 
         // We make an estimated guess based on the project name. At Think Tomorrow, we
         // have a src folder which is PSR-4 namespaced by the project name itself.
-        return str_replace('\\\\', '\\', ucfirst(config('thinktomorrow.chief.domain.namespace', 'App')) . '\\' . ucfirst($this->plural));
+        return str_replace('\\\\', '\\', ucfirst(config('thinktomorrow.chief.domain.namespace', 'App')).'\\'.ucfirst($this->plural));
     }
 
     private function addToConfig($configKey, $value)
     {
-        $current_values = config('thinktomorrow.chief.' . $configKey);
+        $current_values = config('thinktomorrow.chief.'.$configKey);
 
         if (is_array($current_values)) {
             $value = array_merge($current_values, $value);
@@ -235,7 +235,7 @@ 
 
         // Find value - note: this regex does not work for nested arrays!
         // Also creates array with the non-short syntax.
-        $content = preg_replace('/[\'|"]' . $key . '[\'|"] ?=> ?(\[[^\]]*\]|[\'|"].*[\'|"])/', var_export($value, true), $content);
+        $content = preg_replace('/[\'|"]'.$key.'[\'|"] ?=> ?(\[[^\]]*\]|[\'|"].*[\'|"])/', var_export($value, true), $content);
 
         file_put_contents($file, $content);
     }