thinktomorrow / chief

src/ManagedModels/Application/DeleteManagedModel.php

Spacing +2 added lines, -2 removed lines

@@ -35,7 +35,7 @@ 
             // Mark the slug as deleted to avoid any conflict with newly created modules with the same slug.
             if ($model instanceof Module) {
                 $model->update([
-                    'slug' => $model->slug . $this->deletionAppendix(),
+                    'slug' => $model->slug.$this->deletionAppendix(),
                 ]);
             }
 
@@ -70,6 +70,6 @@ 
 
     private function deletionAppendix(): string
     {
-        return '_DELETED_' . time();
+        return '_DELETED_'.time();
     }
 }

src/ManagedModels/Assistants/SavingFields.php

Spacing +14 added lines, -14 removed lines

@@ -27,12 +27,12 @@ 
      */
     public function saveFields(Fields $fields, array $input, array $files): void
     {
-        foreach($fields as $field){
-            if($this->detectCustomSaveMethod($field)) {
+        foreach ($fields as $field) {
+            if ($this->detectCustomSaveMethod($field)) {
                 continue;
             }
 
-            if(!$field->isLocalized())
+            if (!$field->isLocalized())
             {
                 // Set standard non-localized attribute on the model
                 ($customSetMethod = $this->detectCustomSetMethod($field))
@@ -45,11 +45,11 @@ 
             // Dynamic localized values or standard translated
             // For standard translations we set value with the colon notation, e.g. title:en
             $this->foreachTranslationEntry(data_get($input, 'trans', []), function($locale, $key, $value) use($field){
-                if($key !== $field->getColumn()) return;
+                if ($key !== $field->getColumn()) return;
 
-                if($this->isFieldForDynamicValue($field)) {
+                if ($this->isFieldForDynamicValue($field)) {
                     $this->setDynamic($key, $value, $locale);
-                } else {
+                }else {
                     $this->{$field->getColumn().':'.$locale} = $value;
                 }
             });
@@ -58,8 +58,8 @@ 
         $this->save();
 
         // Custom save methods
-        foreach($fields as $field){
-            if($customSaveMethod = $this->detectCustomSaveMethod($field)) {
+        foreach ($fields as $field) {
+            if ($customSaveMethod = $this->detectCustomSaveMethod($field)) {
                 $this->$customSaveMethod($field, $input, $files);
             }
         }
@@ -72,8 +72,8 @@ 
 
     private function detectCustomSaveMethod(Field $field): ?string
     {
-        $saveMethodByKey = 'save' . ucfirst(Str::camel($field->getKey())) . 'Field';
-        $saveMethodByType = 'save' . ucfirst(Str::camel($field->getType()->get())) . 'Fields';
+        $saveMethodByKey = 'save'.ucfirst(Str::camel($field->getKey())).'Field';
+        $saveMethodByType = 'save'.ucfirst(Str::camel($field->getType()->get())).'Fields';
 
         foreach ([$saveMethodByKey, $saveMethodByType] as $saveMethod) {
             if (method_exists($this, $saveMethod)) {
@@ -86,7 +86,7 @@ 
 
     private function detectCustomSetMethod(Field $field): ?string
     {
-        $methodName = 'set' . ucfirst(Str::camel($field->getKey())) . 'Field';
+        $methodName = 'set'.ucfirst(Str::camel($field->getKey())).'Field';
 
         return (method_exists($this, $methodName)) ? $methodName : null;
     }
@@ -108,13 +108,13 @@ 
 
     private function saveImageFields(ImageField $field, array $input)
     {
-        app(ImageFieldHandler::class)->handle($this, $field, data_get($input, 'images.' . $field->getName(), []), $input);
+        app(ImageFieldHandler::class)->handle($this, $field, data_get($input, 'images.'.$field->getName(), []), $input);
     }
 
     private function foreachTranslationEntry(array $translations, \Closure $callback)
     {
-        foreach($translations as $locale => $trans){
-            foreach($trans as $key => $value){
+        foreach ($translations as $locale => $trans) {
+            foreach ($trans as $key => $value) {
                 call_user_func($callback, $locale, $key, $value);
             }
         }

Braces +3 added lines, -1 removed lines

@@ -45,7 +45,9 @@
             // Dynamic localized values or standard translated
             // For standard translations we set value with the colon notation, e.g. title:en
             $this->foreachTranslationEntry(data_get($input, 'trans', []), function($locale, $key, $value) use($field){
-                if($key !== $field->getColumn()) return;
+                if($key !== $field->getColumn()) {
+                    return;
+                }
 
                 if($this->isFieldForDynamicValue($field)) {
                     $this->setDynamic($key, $value, $locale);

src/Legacy/Pages/Page.php

Spacing +7 added lines, -7 removed lines

@@ -98,7 +98,7 @@ 
             return static::$managedModelKey;
         }
 
-        throw new \Exception('Missing required static property \'managedModelKey\' on ' . static::class . '.');
+        throw new \Exception('Missing required static property \'managedModelKey\' on '.static::class.'.');
     }
 
     /**
@@ -138,9 +138,9 @@ 
     public function modelReferenceLabel(): string
     {
         if ($this->exists) {
-            $status = !$this->isPublished() ? ' [' . $this->statusAsPlainLabel() . ']' : null;
+            $status = !$this->isPublished() ? ' ['.$this->statusAsPlainLabel().']' : null;
 
-            return $this->title ? $this->title . $status : '';
+            return $this->title ? $this->title.$status : '';
         }
 
         return '';
@@ -151,7 +151,7 @@ 
         $classKey = get_class($this);
         if (property_exists($this, 'labelSingular')) {
             $labelSingular = $this->labelSingular;
-        } else {
+        }else {
             $labelSingular = Str::singular($classKey);
         }
 
@@ -170,7 +170,7 @@ 
             $locale = app()->getLocale();
         }
         try {
-            $memoizedKey = $this->getMorphClass() . '-' . $this->id . '-' . $locale;
+            $memoizedKey = $this->getMorphClass().'-'.$this->id.'-'.$locale;
 
             if (isset(static::$cachedUrls[$memoizedKey])) {
                 return static::$cachedUrls[$memoizedKey];
@@ -226,11 +226,11 @@ 
     public function statusAsLabel()
     {
         if ($this->isPublished()) {
-            return '<a href="' . $this->url() . '" target="_blank"><em>online</em></a>';
+            return '<a href="'.$this->url().'" target="_blank"><em>online</em></a>';
         }
 
         if ($this->isDraft()) {
-            return '<a href="' . $this->url() . '" target="_blank" class="text-error"><em>offline</em></a>';
+            return '<a href="'.$this->url().'" target="_blank" class="text-error"><em>offline</em></a>';
         }
 
         if ($this->isArchived()) {

src/Setup/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);
     }

src/Modules/Assistants/ModuleCrudAssistant.php

Braces +3 added lines, -1 removed lines

@@ -31,7 +31,9 @@
 
     public function routeModuleCrudAssistant(string $action, $model = null, ...$parameters): ?string
     {
-        if(!in_array($action, ['create','store'])) return null;
+        if(!in_array($action, ['create','store'])) {
+            return null;
+        }
 
         return $this->generateRoute($action, $model, ... $parameters);
     }

Spacing +6 added lines, -6 removed lines

@@ -30,7 +30,7 @@ 
 
     public function routeModuleCrudAssistant(string $action, $model = null, ...$parameters): ?string
     {
-        if(!in_array($action, ['create','store'])) return null;
+        if (!in_array($action, ['create', 'store'])) return null;
 
         return $this->generateRoute($action, $model, ... $parameters);
     }
@@ -59,7 +59,7 @@ 
         $model->saveFields($model->fields()->tagged('create'), $request->all(), $request->allFiles());
 
         return redirect()->to($this->route('edit', $model))
-            ->with('messages.success', '<i class="fa fa-fw fa-check-circle"></i>  "' . $model->adminLabel('title') . '" is toegevoegd');
+            ->with('messages.success', '<i class="fa fa-fw fa-check-circle"></i>  "'.$model->adminLabel('title').'" is toegevoegd');
     }
 
     public function edit($id)
@@ -74,7 +74,7 @@ 
         return view('chief::back.managers.edit', [
             'manager'    => $this,
             'model'      => $model,
-            'fields'     => $model->fields()->map(function (Field $field) use ($model) {
+            'fields'     => $model->fields()->map(function(Field $field) use ($model) {
                 // TODO refactor so render method of field takes model and managerViewModel as arguments.
                 return $field->model($model);
             }),
@@ -93,7 +93,7 @@ 
         $model->saveFields($model->fields(), $request->all(), $request->allFiles());
 
         return redirect()->to($this->route('edit', $model))
-            ->with('messages.success', '<i class="fa fa-fw fa-check-circle"></i>  "' . $model->adminLabel('title') . '" is aangepast');
+            ->with('messages.success', '<i class="fa fa-fw fa-check-circle"></i>  "'.$model->adminLabel('title').'" is aangepast');
     }
 
     public function delete($id, Request $request)
@@ -103,12 +103,12 @@ 
         $model = $this->managedModelClass()::findOrFail($id);
 
         if ($request->get('deleteconfirmation') !== 'DELETE') {
-            return redirect()->back()->with('messages.warning', $model->adminLabel('title') . ' is niet verwijderd.');
+            return redirect()->back()->with('messages.warning', $model->adminLabel('title').' is niet verwijderd.');
         }
 
         app(DeleteManagedModel::class)->handle($model);
 
         return redirect()->to($this->route('index'))
-            ->with('messages.success', '<i class="fa fa-fw fa-check-circle"></i>  "' . $model->adminLabel('title') . '" is verwijderd.');
+            ->with('messages.success', '<i class="fa fa-fw fa-check-circle"></i>  "'.$model->adminLabel('title').'" is verwijderd.');
     }
 }

src/ManagedModels/Assistants/ManagedModelDefaults.php

Braces +3 added lines, -1 removed lines

@@ -44,7 +44,9 @@
 
     private function statusAsLabel(): string
     {
-        if(!$this instanceof ProvidesUrl) return '';
+        if(!$this instanceof ProvidesUrl) {
+            return '';
+        }
 
         if ($this->current_status === PageState::PUBLISHED) {
             return '<a href="' . $this->url() . '" target="_blank"><em>online</em></a>';

Spacing +6 added lines, -6 removed lines

@@ -20,7 +20,7 @@ 
 
     public function adminLabel(string $key, $default = null, array $replace = [])
     {
-        if (! ($value = data_get($this->adminLabels(), $key)) ) {
+        if (!($value = data_get($this->adminLabels(), $key))) {
             return $default;
         }
 
@@ -35,7 +35,7 @@ 
      */
     private function adminLabels(): array
     {
-        $singular = Str::of(static::managedModelKey())->singular()->replace('_',' ')->__toString();
+        $singular = Str::of(static::managedModelKey())->singular()->replace('_', ' ')->__toString();
 
         return [
             /**
@@ -45,7 +45,7 @@ 
              */
             'label' => $singular, // Label used to refer to the generic model
             'nav_label' => $singular, // label used in the chief navigation
-            'page_title' => Str::of(static::managedModelKey())->plural()->replace('_',' ')->__toString(), // Generic collection title, for example used on index
+            'page_title' => Str::of(static::managedModelKey())->plural()->replace('_', ' ')->__toString(), // Generic collection title, for example used on index
 
             /**
              * Instance labels
@@ -62,14 +62,14 @@ 
 
     private function statusAsLabel(): string
     {
-        if(!$this instanceof ProvidesUrl) return '';
+        if (!$this instanceof ProvidesUrl) return '';
 
         if ($this->current_status === PageState::PUBLISHED) {
-            return '<a href="' . $this->url() . '" target="_blank"><em>online</em></a>';
+            return '<a href="'.$this->url().'" target="_blank"><em>online</em></a>';
         }
 
         if ($this->current_status === PageState::DRAFT) {
-            return '<a href="' . $this->url() . '" target="_blank" class="text-error"><em>offline</em></a>';
+            return '<a href="'.$this->url().'" target="_blank" class="text-error"><em>offline</em></a>';
         }
 
         if ($this->current_status === PageState::ARCHIVED) {

src/ManagedModels/States/PageStatePresenter.php

Spacing +1 added lines, -1 removed lines

@@ -27,7 +27,7 @@
     {
         $class = $this->pageState->isOnline() ? 'text-success' : 'text-warning';
 
-        return '<span class="inline-xs stack-s ' . $class . '">' . $this->stateAsLabel() . '</span>';
+        return '<span class="inline-xs stack-s '.$class.'">'.$this->stateAsLabel().'</span>';
     }
 
     /**

src/ManagedModels/States/State/StateException.php

Spacing +5 added lines, -5 removed lines

@@ -6,26 +6,26 @@
 {
     public static function invalidState($state, $currentState, $stateMachine)
     {
-        return new self('Transition to state [' . $state . '] is not allowed from state [' . $currentState . '] or state does not exist on ' . get_class($stateMachine));
+        return new self('Transition to state ['.$state.'] is not allowed from state ['.$currentState.'] or state does not exist on '.get_class($stateMachine));
     }
 
     public static function malformedTransition($transition, $stateMachine)
     {
-        return new self('Transition [' . $transition . '] is malformed on ' . get_class($stateMachine) . '. It should contain both a [from:array] and [to:string] value.');
+        return new self('Transition ['.$transition.'] is malformed on '.get_class($stateMachine).'. It should contain both a [from:array] and [to:string] value.');
     }
 
     public static function invalidTransitionKey($transition, $stateMachine)
     {
-        return new self('unknown transition [' . $transition . '] on ' . get_class($stateMachine));
+        return new self('unknown transition ['.$transition.'] on '.get_class($stateMachine));
     }
 
     public static function invalidTransition($transition, $state, $stateMachine)
     {
-        return new self('Transition [' . $transition . '] cannot be applied from current state [' . $state . '] on ' . get_class($stateMachine));
+        return new self('Transition ['.$transition.'] cannot be applied from current state ['.$state.'] on '.get_class($stateMachine));
     }
 
     public static function invalidTransitionState($transition, $state, $stateMachine)
     {
-        return new self('Transition [' . $transition . '] contains a non existing [' . $state . '] on ' . get_class($stateMachine));
+        return new self('Transition ['.$transition.'] contains a non existing ['.$state.'] on '.get_class($stateMachine));
     }
 }

src/ManagedModels/States/Publishable/CheckPreviewTrait.php

Spacing +1 added lines, -1 removed lines

@@ -12,7 +12,7 @@
             Session::now('note.default', 'U bekijkt een preview.');
 
             return true;
-        } else {
+        }else {
             return false;
         }
     }