thinktomorrow / chief

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) use($ownerType, $ownerId){
                 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/Modules/Assistants/ModuleCudAssistant.php

Spacing +5 added lines, -5 removed lines

@@ -91,7 +91,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)
@@ -106,7 +106,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()]);
@@ -126,7 +126,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)
@@ -136,12 +136,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.');
     }
 }

src/Modules/ModuleManager.php

Spacing +1 added lines, -1 removed lines

@@ -159,7 +159,7 @@
             if (is_array_empty($translation)) {
 
                 // Nullify all values
-                $trans[$locale] = array_map(function ($value) {
+                $trans[$locale] = array_map(function($value) {
                     return null;
                 }, $translation);
                 continue;

src/Fields/Fields.php

Spacing +12 added lines, -12 removed lines

@@ -40,8 +40,8 @@ 
 
     public function find(string $key): Field
     {
-        if(!isset($this->fields[$key])) {
-            throw new \InvalidArgumentException('No field found by key ' . $key);
+        if (!isset($this->fields[$key])) {
+            throw new \InvalidArgumentException('No field found by key '.$key);
         }
 
         return $this->fields[$key];
@@ -101,7 +101,7 @@ 
                 continue;
             }
 
-            $method = 'get' . ucfirst($key);
+            $method = 'get'.ucfirst($key);
 
             // Reject from list if value does not match expected one
             if ($value && $value == $field->$method()) {
@@ -117,30 +117,30 @@ 
 
     public function render(): string
     {
-        return array_reduce($this->fields, function (string $carry, Field $field) {
-            return $carry . $field->render();
+        return array_reduce($this->fields, function(string $carry, Field $field) {
+            return $carry.$field->render();
         }, '');
     }
 
     public function keyed($key): Fields
     {
-        $keys = (array) $key;
+        $keys = (array)$key;
 
-        return new static(array_filter($this->fields, function (Field $field) use ($keys) {
+        return new static(array_filter($this->fields, function(Field $field) use ($keys) {
             return in_array($field->getKey(), $keys);
         }));
     }
 
     public function tagged($tag): Fields
     {
-        return new static(array_filter($this->fields, function (Field $field) use ($tag) {
+        return new static(array_filter($this->fields, function(Field $field) use ($tag) {
             return $field->tagged($tag);
         }));
     }
 
     public function untagged(): Fields
     {
-        return new static(array_filter($this->fields, function (Field $field) {
+        return new static(array_filter($this->fields, function(Field $field) {
             return $field->untagged();
         }));
     }
@@ -182,7 +182,7 @@ 
     public function offsetGet($offset)
     {
         if (!isset($this->fields[$offset])) {
-            throw new \RuntimeException('No field found by key [' . $offset . ']');
+            throw new \RuntimeException('No field found by key ['.$offset.']');
         }
 
         return $this->fields[$offset];
@@ -191,7 +191,7 @@ 
     public function offsetSet($offset, $value)
     {
         if (!$value instanceof Field) {
-            throw new \InvalidArgumentException('Passed value must be of type ' . Field::class);
+            throw new \InvalidArgumentException('Passed value must be of type '.Field::class);
         }
 
         $this->fields[$offset] = $value;
@@ -221,7 +221,7 @@ 
 
     private function validateFields(array $fields)
     {
-        array_map(function (Field $field) {
+        array_map(function(Field $field) {
         }, $fields);
     }
 

src/Managers/Assistants/RedactorFileUploadAssistant.php

Spacing +3 added lines, -3 removed lines

@@ -43,16 +43,16 @@
             $filename = $filePayload['filename'];
 
             if (!$asset = AssetUploader::uploadFromBase64($base64EncodedFile, $filename)) {
-                $responseContent['file-' . rand(1 - 999)] = [
+                $responseContent['file-'.rand(1-999)] = [
                     'error'    => true,
-                    'messages' => 'Afbeelding [' . $filename . '] kan niet worden opgeladen.',
+                    'messages' => 'Afbeelding ['.$filename.'] kan niet worden opgeladen.',
                 ];
                 continue;
             }
 
             app(AddAsset::class)->add($model, $asset, MediaType::CONTENT, $request->input('locale', app()->getLocale()));
 
-            $responseContent['file-' . $asset->id] = [
+            $responseContent['file-'.$asset->id] = [
                 'url' => $asset->url(),
                 'id'  => $asset->id,
             ];

src/Managers/Assistants/UrlAssistant.php

Spacing +1 added lines, -1 removed lines

@@ -41,7 +41,7 @@
         if ($urlRecord->isRedirect()) {
             return 'Deze link bestaat reeds als redirect. Deze redirect zal bijgevolg worden verwijderd.';
         }
-        return 'Deze link bestaat reeds. Kies een andere of <a target="_blank" href="' . $this->editRouteOfOtherModel($urlRecord) . '">pas de andere pagina aan</a>.';
+        return 'Deze link bestaat reeds. Kies een andere of <a target="_blank" href="'.$this->editRouteOfOtherModel($urlRecord).'">pas de andere pagina aan</a>.';
     }
 
     private function editRouteOfOtherModel(UrlRecord $urlRecord): string

src/Managers/Assistants/ManagerDefaults.php

Spacing +5 added lines, -5 removed lines

@@ -26,8 +26,8 @@ 
 
     public function route(string $action, $model = null, ...$parameters): string
     {
-        foreach(DiscoverTraitMethods::belongingTo(static::class, 'route') as $method) {
-            if(null !== ($route = $this->$method($action, $model, ...$parameters))){
+        foreach (DiscoverTraitMethods::belongingTo(static::class, 'route') as $method) {
+            if (null !== ($route = $this->$method($action, $model, ...$parameters))) {
                 return $route;
             }
         }
@@ -40,8 +40,8 @@ 
 
     public function can(string $action, $model = null): bool
     {
-        foreach(DiscoverTraitMethods::belongingTo(static::class, 'can') as $method) {
-            if(true === $this->$method($action, $model)){
+        foreach (DiscoverTraitMethods::belongingTo(static::class, 'can') as $method) {
+            if (true === $this->$method($action, $model)) {
                 return true;
             }
         }
@@ -83,7 +83,7 @@ 
             $parameters = array_merge((array)$modelId, $parameters);
         }
 
-        return route('chief.' . static::routeKey() . '.' . $action, $parameters);
+        return route('chief.'.static::routeKey().'.'.$action, $parameters);
     }
 
     private function guard(string $action)

src/Managers/Assistants/ArchiveAssistant.php

Spacing +9 added lines, -9 removed lines

@@ -19,25 +19,25 @@ 
     public static function routesArchiveAssistant(): array
     {
         return [
-            ManagedRoute::post('archive','archive/{id}'),
-            ManagedRoute::post('unarchive','unarchive/{id}'),
+            ManagedRoute::post('archive', 'archive/{id}'),
+            ManagedRoute::post('unarchive', 'unarchive/{id}'),
             ManagedRoute::get('archive_index'),
         ];
     }
 
     public function canArchiveAssistant(string $action, $model = null): bool
     {
-        if(!in_array($action, ['archive', 'unarchive', 'archive_index'])) return false;
+        if (!in_array($action, ['archive', 'unarchive', 'archive_index'])) return false;
 
-        if($action === 'archive_index') {
+        if ($action === 'archive_index') {
             // Archive index is only visitable when there is at least one model archived.
-            if(public_method_exists(static::managedModelClass(), 'scopeArchived')) {
+            if (public_method_exists(static::managedModelClass(), 'scopeArchived')) {
                 return static::managedModelClass()::archived()->count() > 0;
             }
             return false;
         }
 
-        if(!$model || !$model instanceof StatefulContract) return false;
+        if (!$model || !$model instanceof StatefulContract) return false;
 
         return PageState::make($model)->can($action);
     }
@@ -56,7 +56,7 @@ 
 
             // Ok now get all urls from this model and point them to the new records
             foreach (UrlRecord::getByModel($model) as $urlRecord) {
-                if ($targetRecord = $targetRecords->first(function ($record) use ($urlRecord) {
+                if ($targetRecord = $targetRecords->first(function($record) use ($urlRecord) {
                     return ($record->locale == $urlRecord->locale && !$record->isRedirect());
                 })) {
                     $urlRecord->redirectTo($targetRecord);
@@ -68,7 +68,7 @@ 
 
         Audit::activity()->performedOn($model)->log('archived');
 
-        return redirect()->to($this->route('index'))->with('messages.success', $model->adminLabel('title') . ' is gearchiveerd.');
+        return redirect()->to($this->route('index'))->with('messages.success', $model->adminLabel('title').' is gearchiveerd.');
     }
 
     public function unarchive($id, Request $request)
@@ -81,7 +81,7 @@ 
 
         Audit::activity()->performedOn($model)->log('unarchived');
 
-        return redirect()->to($this->route('index'))->with('messages.success', $model->adminLabel('title') . ' is uit het archief gehaald.');
+        return redirect()->to($this->route('index'))->with('messages.success', $model->adminLabel('title').' is uit het archief gehaald.');
     }
 
     public function archive_index(Request $request)

Braces +6 added lines, -2 removed lines

@@ -27,7 +27,9 @@ 
 
     public function canArchiveAssistant(string $action, $model = null): bool
     {
-        if(!in_array($action, ['archive', 'unarchive', 'archive_index'])) return false;
+        if(!in_array($action, ['archive', 'unarchive', 'archive_index'])) {
+            return false;
+        }
 
         if($action === 'archive_index') {
             // Archive index is only visitable when there is at least one model archived.
@@ -37,7 +39,9 @@ 
             return false;
         }
 
-        if(!$model || !$model instanceof StatefulContract) return false;
+        if(!$model || !$model instanceof StatefulContract) {
+            return false;
+        }
 
         return PageState::make($model)->can($action);
     }

src/Managers/Assistants/PublishAssistant.php

Spacing +7 added lines, -7 removed lines

@@ -17,23 +17,23 @@ 
     public static function routesPublishAssistant(): array
     {
         return [
-            ManagedRoute::post('publish','publish/{id}'),
-            ManagedRoute::post('unpublish','unpublish/{id}'),
+            ManagedRoute::post('publish', 'publish/{id}'),
+            ManagedRoute::post('unpublish', 'unpublish/{id}'),
         ];
     }
 
     public function routePublishAssistant(string $action, $model = null, ...$parameters): ?string
     {
-        if(!$this->canPublishAssistant($action, $model)) return null;
+        if (!$this->canPublishAssistant($action, $model)) return null;
 
         return $this->generateRoute($action, $model, ...$parameters);
     }
 
     public function canPublishAssistant(string $action, $model = null): bool
     {
-        if(!in_array($action, ['publish', 'unpublish'])) return false;
+        if (!in_array($action, ['publish', 'unpublish'])) return false;
 
-        if(!$model || !$model instanceof StatefulContract) return false;
+        if (!$model || !$model instanceof StatefulContract) return false;
 
         return PageState::make($model)->can($action);
     }
@@ -48,7 +48,7 @@ 
 
         Audit::activity()->performedOn($model)->log('published');
 
-        return redirect()->to($this->route('index'))->with('messages.success', $model->adminLabel('title') . ' is online geplaatst.');
+        return redirect()->to($this->route('index'))->with('messages.success', $model->adminLabel('title').' is online geplaatst.');
     }
 
     public function unpublish($id, Request $request)
@@ -61,6 +61,6 @@ 
 
         Audit::activity()->performedOn($model)->log('unpublished');
 
-        return redirect()->to($this->route('index'))->with('messages.success', $model->adminLabel('title') . ' is offline gehaald.');
+        return redirect()->to($this->route('index'))->with('messages.success', $model->adminLabel('title').' is offline gehaald.');
     }
 }

Braces +9 added lines, -3 removed lines

@@ -24,16 +24,22 @@
 
     public function routePublishAssistant(string $action, $model = null, ...$parameters): ?string
     {
-        if(!$this->canPublishAssistant($action, $model)) return null;
+        if(!$this->canPublishAssistant($action, $model)) {
+            return null;
+        }
 
         return $this->generateRoute($action, $model, ...$parameters);
     }
 
     public function canPublishAssistant(string $action, $model = null): bool
     {
-        if(!in_array($action, ['publish', 'unpublish'])) return false;
+        if(!in_array($action, ['publish', 'unpublish'])) {
+            return false;
+        }
 
-        if(!$model || !$model instanceof StatefulContract) return false;
+        if(!$model || !$model instanceof StatefulContract) {
+            return false;
+        }
 
         return PageState::make($model)->can($action);
     }