thinktomorrow / chief

src/Site/Urls/Application/SaveUrlSlugs.php

Spacing +8 added lines, -8 removed lines

@@ -43,7 +43,7 @@ 
     private function saveRecord(ProvidesUrl $model, string $locale, ?string $slug, Collection $existingRecords, bool $strict = true)
     {
         // Existing ones for this locale?
-        $nonRedirectsWithSameLocale = $existingRecords->filter(function ($record) use ($locale) {
+        $nonRedirectsWithSameLocale = $existingRecords->filter(function($record) use ($locale) {
             return (
                 $record->locale == $locale &&
                 !$record->isRedirect()
@@ -52,7 +52,7 @@ 
 
         // If slug entry is left empty, all existing records will be deleted
         if (!$slug) {
-            $nonRedirectsWithSameLocale->each(function ($existingRecord) {
+            $nonRedirectsWithSameLocale->each(function($existingRecord) {
                 $existingRecord->delete();
             });
 
@@ -69,7 +69,7 @@ 
         }
 
         // Only replace the existing records that differ from the current passed slugs
-        $nonRedirectsWithSameLocale->each(function ($existingRecord) use ($slug) {
+        $nonRedirectsWithSameLocale->each(function($existingRecord) use ($slug) {
             if ($existingRecord->slug != $slug) {
                 $existingRecord->replaceAndRedirect(['slug' => $slug]);
             }
@@ -98,7 +98,7 @@ 
         $this->deleteIdenticalRedirects($sameExistingRecords, $locale, $slug);
 
         // Also delete any urls that match this locale and slug but are related to another model
-        if(!$this->strict) {
+        if (!$this->strict) {
             $this->deleteIdenticalRecords($model, $sameExistingRecords);
         }
     }
@@ -112,12 +112,12 @@ 
      */
     private function deleteIdenticalRedirects($existingRecords, $locale, $slug): void
     {
-        $existingRecords->filter(function ($record) use ($locale) {
+        $existingRecords->filter(function($record) use ($locale) {
             return (
                 $record->locale == $locale &&
                 $record->isRedirect()
             );
-        })->each(function ($existingRecord) use ($slug) {
+        })->each(function($existingRecord) use ($slug) {
             if ($existingRecord->slug == $slug) {
                 $existingRecord->delete();
             }
@@ -128,11 +128,11 @@ 
     {
         // The old homepage url should be removed since this is no longer in effect.
         // In case of any redirect to this old homepage, the last used redirect is now back in effect.
-        $existingRecords->reject(function ($existingRecord) use($model) {
+        $existingRecords->reject(function($existingRecord) use($model) {
             return (
                 $existingRecord->model_type == $model->getMorphClass() &&
                 $existingRecord->model_id == $model->id);
-        })->each(function ($existingRecord) {
+        })->each(function($existingRecord) {
 
             // TODO: if there is a redirect to this page, we'll take this one as the new url
             $existingRecord->delete();

src/ManagedModels/Assistants/ManagedModelDefaults.php

Spacing +6 added lines, -6 removed lines

@@ -21,7 +21,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;
         }
 
@@ -36,7 +36,7 @@ 
      */
     private function adminLabels(): array
     {
-        $singular = Str::of(static::managedModelKey())->singular()->replace('_',' ')->__toString();
+        $singular = Str::of(static::managedModelKey())->singular()->replace('_', ' ')->__toString();
 
         return [
             /**
@@ -46,7 +46,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
@@ -63,17 +63,17 @@ 
 
     private function onlineStatusAsLabel(): string
     {
-        if(!$this instanceof StatefulContract) return '';
+        if (!$this instanceof StatefulContract) return '';
 
         if ($this->stateOf(PageState::KEY) === PageState::PUBLISHED) {
             return $this instanceof ProvidesUrl
-                ? '<a href="' . $this->url() . '" target="_blank"><em>online</em></a>'
+                ? '<a href="'.$this->url().'" target="_blank"><em>online</em></a>'
                 : '<span><em>online</em></span>';
         }
 
         if ($this->stateOf(PageState::KEY) === PageState::DRAFT) {
             return $this instanceof ProvidesUrl
-                ? '<a href="' . $this->url() . '" target="_blank" class="text-error"><em>offline</em></a>'
+                ? '<a href="'.$this->url().'" target="_blank" class="text-error"><em>offline</em></a>'
                 : '<span class="text-error"><em>offline</em></span>';
         }
 

Braces +3 added lines, -1 removed lines

@@ -63,7 +63,9 @@
 
     private function onlineStatusAsLabel(): string
     {
-        if(!$this instanceof StatefulContract) return '';
+        if(!$this instanceof StatefulContract) {
+            return '';
+        }
 
         if ($this->stateOf(PageState::KEY) === PageState::PUBLISHED) {
             return $this instanceof ProvidesUrl

src/Managers/Assistants/ArchiveAssistant.php

Spacing +9 added lines, -9 removed lines

@@ -20,15 +20,15 @@ 
     public 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;
 
         try {
             $this->authorize('update-page');
@@ -36,15 +36,15 @@ 
             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($this->managedModelClass(), 'scopeArchived')) {
+            if (public_method_exists($this->managedModelClass(), 'scopeArchived')) {
                 return $this->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);
     }
@@ -64,7 +64,7 @@ 
 
             // Ok now get all urls from this model and point them to the new records
             foreach ($archivedUrlRecords 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);
@@ -84,7 +84,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(Request $request, $id)
@@ -97,7 +97,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 archiveIndex(Request $request)

Braces +6 added lines, -2 removed lines

@@ -28,7 +28,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;
+        }
 
         try {
             $this->authorize('update-page');
@@ -44,7 +46,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/ManagerDefaults.php

Spacing +5 added lines, -5 removed lines

@@ -28,8 +28,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;
             }
         }
@@ -39,8 +39,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;
             }
         }
@@ -56,7 +56,7 @@ 
             $parameters = array_merge((array)$modelId, $parameters);
         }
 
-        return route('chief.' . $this->managedModelClass()::managedModelKey() . '.' . $action, $parameters);
+        return route('chief.'.$this->managedModelClass()::managedModelKey().'.'.$action, $parameters);
     }
 
     private function guard(string $action, $model = null)

src/Managers/Assistants/PublishAssistant.php

Spacing +7 added lines, -7 removed lines

@@ -18,21 +18,21 @@ 
     public 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;
 
         try {
             $this->authorize('update-page');
@@ -40,7 +40,7 @@ 
             return false;
         }
 
-        if(!$model || !$model instanceof StatefulContract) return false;
+        if (!$model || !$model instanceof StatefulContract) return false;
 
         return PageState::make($model)->can($action);
     }
@@ -55,7 +55,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(Request $request, $id)
@@ -68,6 +68,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

@@ -25,14 +25,18 @@ 
 
     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;
+        }
 
         try {
             $this->authorize('update-page');
@@ -40,7 +44,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/CrudAssistant.php

Braces +9 added lines, -3 removed lines

@@ -39,7 +39,9 @@ 
 
     public function canCrudAssistant(string $action, $model = null): bool
     {
-        if(!in_array($action, ['index', 'create', 'store', 'edit', 'update', 'delete'])) return false;
+        if(!in_array($action, ['index', 'create', 'store', 'edit', 'update', 'delete'])) {
+            return false;
+        }
 
         try {
 
@@ -58,9 +60,13 @@ 
             return false;
         }
 
-        if(in_array($action, ['index', 'create', 'store'])) return true;
+        if(in_array($action, ['index', 'create', 'store'])) {
+            return true;
+        }
 
-        if(!$model || !$model instanceof StatefulContract) return true;
+        if(!$model || !$model instanceof StatefulContract) {
+            return true;
+        }
 
         // Model cannot be in deleted state for editing purposes.
         if(in_array($action, ['edit', 'update'])) {

Spacing +10 added lines, -10 removed lines

@@ -37,7 +37,7 @@ 
 
     public function canCrudAssistant(string $action, $model = null): bool
     {
-        if(!in_array($action, ['index', 'create', 'store', 'edit', 'update', 'delete'])) return false;
+        if (!in_array($action, ['index', 'create', 'store', 'edit', 'update', 'delete'])) return false;
 
         try {
 
@@ -56,12 +56,12 @@ 
             return false;
         }
 
-        if(in_array($action, ['index', 'create', 'store'])) return true;
+        if (in_array($action, ['index', 'create', 'store'])) return true;
 
-        if(!$model || !$model instanceof StatefulContract) return true;
+        if (!$model || !$model instanceof StatefulContract) return true;
 
         // Model cannot be in deleted state for editing purposes.
-        if(in_array($action, ['edit', 'update'])) {
+        if (in_array($action, ['edit', 'update'])) {
             return !($model->stateOf(PageState::KEY) == PageState::DELETED);
         }
 
@@ -99,14 +99,14 @@ 
         // Default sorted by publish date
         $modelClass = $this->managedModelClass();
         if (Schema::hasColumn((new $modelClass)->getTable(), 'published_at')) {
-            $filters = $filters->add(HiddenFilter::make('publish', function ($query) {
+            $filters = $filters->add(HiddenFilter::make('publish', function($query) {
                 return $query->orderBy('published_at', 'DESC');
             }));
         }
 
         // if model has no timestamps, updated_at doesn't exist
         if ((new $modelClass)->timestamps) {
-            $filters = $filters->add(HiddenFilter::make('updated', function ($query) {
+            $filters = $filters->add(HiddenFilter::make('updated', function($query) {
                 return $query->orderBy('updated_at', 'DESC');
             }));
         }
@@ -140,7 +140,7 @@ 
         $model->saveFields($model->fields()->notTagged('edit'), $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(Request $request, $id)
@@ -167,7 +167,7 @@ 
         $model->saveFields($model->fields()->notTagged('create'), $request->all(), $request->allFiles());
 
         return redirect()->to($this->route('index'))
-            ->with('messages.success', '<i class="fa fa-fw fa-check-circle"></i>  <a href="' . $this->route('edit', $model) . '">' . $model->adminLabel('title') . '</a> is aangepast');
+            ->with('messages.success', '<i class="fa fa-fw fa-check-circle"></i>  <a href="'.$this->route('edit', $model).'">'.$model->adminLabel('title').'</a> is aangepast');
     }
 
     public function delete(Request $request, $id)
@@ -177,12 +177,12 @@ 
         $this->guard('delete', $model);
 
         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(DeleteModel::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/Fields/Fields.php

Spacing +13 added lines, -13 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];
@@ -108,7 +108,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()) {
@@ -124,23 +124,23 @@ 
 
     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);
         }));
     }
@@ -148,14 +148,14 @@ 
     public function notTagged($tag): Fields
     {
         // TODO: to test this...
-        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();
         }));
     }
@@ -197,7 +197,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];
@@ -206,7 +206,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;
@@ -236,7 +236,7 @@ 
 
     private function validateFields(array $fields)
     {
-        array_map(function (Field $field) {
+        array_map(function(Field $field) {
         }, $fields);
     }
 

src/ManagedModels/Duplication/PageDuplicator.php

Spacing +2 added lines, -2 removed lines

@@ -53,13 +53,13 @@
 
             if ($child instanceof Module && $child->isPageSpecific()) {
                 $duplicatedChild = $child::create([
-                    'slug'           => $targetModel->title ? $targetModel->title . '-' . $child->slug : $child->slug . '-copy',
+                    'slug'           => $targetModel->title ? $targetModel->title.'-'.$child->slug : $child->slug.'-copy',
                     'owner_id'        => $targetModel->id,
                     'owner_type' => $targetModel->getMorphClass()
                 ]);
 
                 $this->moduleTemplateApplicator->handle($child, $duplicatedChild);
-            } else {
+            }else {
                 $duplicatedChild = $child;
             }
 

src/ManagedModels/Application/DuplicateContext.php

Spacing +2 added lines, -2 removed lines

@@ -22,12 +22,12 @@
 
     public function handle($sourceModel, $targetModel): void
     {
-        if(!$sourceModel instanceof FragmentsOwner || !$context = ContextModel::ownedBy($sourceModel)) return;
+        if (!$sourceModel instanceof FragmentsOwner || !$context = ContextModel::ownedBy($sourceModel)) return;
 
         $copiedContext = ContextModel::createForOwner($targetModel);
 
         /** @var FragmentModel $fragment */
-        foreach($context->fragments as $fragment) {
+        foreach ($context->fragments as $fragment) {
             $copiedFragment = $fragment->replicate(['context_id']);
             $copiedFragment->id = $this->fragmentRepository->nextId();
             $copiedFragment->context_id = $copiedContext->id;

Braces +3 added lines, -1 removed lines

@@ -22,7 +22,9 @@
 
     public function handle($sourceModel, $targetModel): void
     {
-        if(!$sourceModel instanceof FragmentsOwner || !$context = ContextModel::ownedBy($sourceModel)) return;
+        if(!$sourceModel instanceof FragmentsOwner || !$context = ContextModel::ownedBy($sourceModel)) {
+            return;
+        }
 
         $copiedContext = ContextModel::createForOwner($targetModel);