thinktomorrow / chief

src/Managers/Assistants/ModulesAssistant.php

Spacing +4 added lines, -4 removed lines

@@ -30,9 +30,9 @@ 
 
     public function routeModulesAssistant(string $action, $model = null, ...$parameters): ?string
     {
-        if(!in_array($action, ['modules-index','modules-edit','modules-update','modules-delete','modules-create','modules-store'])) return null;
+        if (!in_array($action, ['modules-index', 'modules-edit', 'modules-update', 'modules-delete', 'modules-create', 'modules-store'])) return null;
 
-        if(count($parameters) < 1 || ! reset($parameters) instanceof ActsAsChild) {
+        if (count($parameters) < 1 || !reset($parameters) instanceof ActsAsChild) {
             throw new \InvalidArgumentException('Missing arguments. Modules route definition requires two parameters: parent model and module model. ');
         }
 
@@ -51,9 +51,9 @@ 
         $modules = $parent->children();
 
         // Add their respective manager per class
-        $moduleManagers = $modules->groupBy(function ($child) {
+        $moduleManagers = $modules->groupBy(function($child) {
             return get_class($child);
-        })->map(function ($child, $class) {
+        })->map(function($child, $class) {
             return app(ModelToManagerMap::class)->find($class);
         });
 

Braces +3 added lines, -1 removed lines

@@ -30,7 +30,9 @@
 
     public function routeModulesAssistant(string $action, $model = null, ...$parameters): ?string
     {
-        if(!in_array($action, ['modules-index','modules-edit','modules-update','modules-delete','modules-create','modules-store'])) return null;
+        if(!in_array($action, ['modules-index','modules-edit','modules-update','modules-delete','modules-create','modules-store'])) {
+            return null;
+        }
 
         if(count($parameters) < 1 || ! reset($parameters) instanceof ActsAsChild) {
             throw new \InvalidArgumentException('Missing arguments. Modules route definition requires two parameters: parent model and module model. ');

src/Modules/Assistants/ModuleCrudAssistant.php

Spacing +6 added lines, -6 removed lines

@@ -31,7 +31,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);
     }
@@ -60,7 +60,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)
@@ -75,7 +75,7 @@ 
         return view('chief::back.managers.edit', [
             'model'      => $model,
             'managerViewModel' => $this->managerViewModel(),
-            '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)
                     ->viewData(['managerViewModel' => $this->managerViewModel()]);
@@ -95,7 +95,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)
@@ -105,12 +105,12 @@ 
         $model = static::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.');
     }
 }

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);
     }

src/Modules/Modules.php

Spacing +5 added lines, -5 removed lines

@@ -36,12 +36,12 @@ 
         // TODO: how to assign for pagebuilder?? with tag, via the chief relations config or is this default
         $moduleManagerClasses = $this->registeredManagers->tagged('module')->get();
 
-        $managers = collect(array_map(function($moduleManagerClass){
+        $managers = collect(array_map(function($moduleManagerClass) {
             return app($moduleManagerClass);
         }, $moduleManagerClasses));
 
         return $managers
-            ->reject(function($manager){ return !$manager->can('create'); })
+            ->reject(function($manager) { return !$manager->can('create'); })
             ->map(function($manager) {
                 return [
                     'label' => $manager->adminLabel('page_title'),
@@ -56,13 +56,13 @@ 
         // Get the modules that are set to be shareable.
         $moduleManagerClasses = $this->shareableModules();
 
-        $managers = collect(array_map(function($moduleManagerClass){
+        $managers = collect(array_map(function($moduleManagerClass) {
             return app($moduleManagerClass);
         }, $moduleManagerClasses));
 
         return $managers
-            ->reject(function($manager){ return !$manager->can('create-shared'); })
-            ->map(function($manager){
+            ->reject(function($manager) { return !$manager->can('create-shared'); })
+            ->map(function($manager) {
                 return [
                     'label' => $manager->adminLabel('page_title'),
                     'value' => $manager->route('create-shared')

src/Urls/ProvidesUrl/ProvidingUrl.php

Spacing +1 added lines, -1 removed lines

@@ -35,6 +35,6 @@
     /** @inheritdoc */
     public function baseUrlSegment(string $locale = null): string
     {
-        return BaseUrlSegment::find(isset(static::$baseUrlSegment) ? (array) static::$baseUrlSegment : [], $locale);
+        return BaseUrlSegment::find(isset(static::$baseUrlSegment) ? (array)static::$baseUrlSegment : [], $locale);
     }
 }

src/Urls/ProvidesUrl/BaseUrlSegment.php

Spacing +3 added lines, -3 removed lines

@@ -10,8 +10,8 @@ 
 {
     public static function find(array $segments, string $locale = null)
     {
-        if(count($segments) < 1) return '/';
-        if(count($segments) == 1) return reset($segments);
+        if (count($segments) < 1) return '/';
+        if (count($segments) == 1) return reset($segments);
 
         // Localized value
         if (($key = $locale ?? app()->getlocale()) && isset($segments[$key])) return $segments[$key];
@@ -31,7 +31,7 @@ 
      */
     public static function prepend(ProvidesUrl $model, string $slug, $locale): string
     {
-        $slugWithBaseSegment = $model->baseUrlSegment($locale) . '/' . $slug;
+        $slugWithBaseSegment = $model->baseUrlSegment($locale).'/'.$slug;
         $slugWithBaseSegment = trim($slugWithBaseSegment, '/');
 
         // If slug with base segment is empty string, it means that the passed slug was probably a "/" character.

Braces +12 added lines, -4 removed lines

@@ -10,14 +10,22 @@
 {
     public static function find(array $segments, string $locale = null)
     {
-        if(count($segments) < 1) return '/';
-        if(count($segments) == 1) return reset($segments);
+        if(count($segments) < 1) {
+            return '/';
+        }
+        if(count($segments) == 1) {
+            return reset($segments);
+        }
 
         // Localized value
-        if (($key = $locale ?? app()->getlocale()) && isset($segments[$key])) return $segments[$key];
+        if (($key = $locale ?? app()->getlocale()) && isset($segments[$key])) {
+            return $segments[$key];
+        }
 
         // Fallback localized value
-        if (($fallback_locale = config('app.fallback_locale')) && isset($segments[$fallback_locale])) return $segments[$fallback_locale];
+        if (($fallback_locale = config('app.fallback_locale')) && isset($segments[$fallback_locale])) {
+            return $segments[$fallback_locale];
+        }
 
         // Fallback to first entry
         return reset($segments);

src/ManagedModels/Assistants/ManagedModelDefaults.php

Spacing +3 added lines, -3 removed lines

@@ -44,14 +44,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) {

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>';