thinktomorrow / chief

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

src/Managers/Assistants/PreviewAssistant.php

Spacing +2 added lines, -2 removed lines

@@ -9,7 +9,7 @@ 
 {
     public function canPreviewAssistant(string $action, $model = null): bool
     {
-        if($action !== 'preview' || !$model) return false;
+        if ($action !== 'preview' || !$model) return false;
 
         return $model instanceof ProvidesUrl;
     }
@@ -27,7 +27,7 @@ 
      */
     public function routePreviewAssistant(string $action, $model = null, ...$parameters): ?string
     {
-        if(!$this->canPreviewAssistant($action, $model)) return null;
+        if (!$this->canPreviewAssistant($action, $model)) return null;
 
         return $model->url();
     }

Braces +6 added lines, -2 removed lines

@@ -9,7 +9,9 @@ 
 {
     public function canPreviewAssistant(string $action, $model = null): bool
     {
-        if($action !== 'preview' || !$model) return false;
+        if($action !== 'preview' || !$model) {
+            return false;
+        }
 
         return $model instanceof ProvidesUrl;
     }
@@ -27,7 +29,9 @@ 
      */
     public function routePreviewAssistant(string $action, $model = null, ...$parameters): ?string
     {
-        if(!$this->canPreviewAssistant($action, $model)) return null;
+        if(!$this->canPreviewAssistant($action, $model)) {
+            return null;
+        }
 
         return $model->url();
     }

src/Managers/Assistants/FragmentCrudAssistant.php

Spacing +1 added lines, -1 removed lines

@@ -55,7 +55,7 @@
     public function fragmentsIndex(string $ownerClass, $ownerId)
     {
 //        trap($ownerClass, $ownerId);
-        $models = static::managedModelClass()::whereIn('id',[2,3])->get();
+        $models = static::managedModelClass()::whereIn('id', [2, 3])->get();
 
         return view('chief::back.catalogpages._fragments', [
             'models'            => $models,

src/Managers/Assistants/CrudAssistant.php

Indentation +1 added lines, -1 removed lines

@@ -153,7 +153,7 @@
             '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' => new managerViewModel($this)]);
+                                ->viewData(['managerViewModel' => new managerViewModel($this)]);
             }),
         ]);
     }

Spacing +9 added lines, -9 removed lines

@@ -95,15 +95,15 @@ 
         // Default sorted by publish date
         $modelClass = static::managedModelClass();
         if (Schema::hasColumn((new $modelClass)->getTable(), 'published_at')) {
-            $filters = $filters->add(HiddenFilter::make('publish', function($query){
-                return $query->orderBy('published_at','DESC');
+            $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){
-                return $query->orderBy('updated_at','DESC');
+            $filters = $filters->add(HiddenFilter::make('updated', function($query) {
+                return $query->orderBy('updated_at', 'DESC');
             }));
         }
 
@@ -134,7 +134,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)
@@ -149,7 +149,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)
                              ->viewData(['managerViewModel' => new managerViewModel($this)]);
@@ -169,7 +169,7 @@ 
         $model->saveFields($model->fields(), $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($id, Request $request)
@@ -179,12 +179,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/Managers/Register/RegisteredManagers.php

Spacing +3 added lines, -3 removed lines

@@ -14,7 +14,7 @@ 
 
     public function tagged($tags): self
     {
-        $tags = (array) $tags;
+        $tags = (array)$tags;
 
         return $this->filter(function($value, $key) use($tags){
             return count(array_intersect($value, $tags)) > 0;
@@ -23,7 +23,7 @@ 
 
     public function notTagged($tags): self
     {
-        $tags = (array) $tags;
+        $tags = (array)$tags;
 
         return $this->filter(function($value, $key) use($tags){
             return count(array_intersect($value, $tags)) === 0;
@@ -32,7 +32,7 @@ 
 
     public function untagged($tags): self
     {
-        $tags = (array) $tags;
+        $tags = (array)$tags;
 
         return $this->filter(function($value, $key) use($tags){
             return empty($value);

src/Managers/Register/RegisterManager.php

Spacing +1 added lines, -1 removed lines

@@ -29,6 +29,6 @@
         );
 
         $this->container->make(ModelToManagerMap::class)->map($class::managedModelClass(), $class);
-        $this->container->make(RegisteredManagers::class)->tag($class, (array) $tags);
+        $this->container->make(RegisteredManagers::class)->tag($class, (array)$tags);
     }
 }

src/Managers/Routes/ManagedRoutes.php

Spacing +1 added lines, -1 removed lines

@@ -34,7 +34,7 @@
 
     public function push($managedRoutes): self
     {
-        return new static($this->managerClass, $this->name, array_merge($this->routes, (array) $managedRoutes));
+        return new static($this->managerClass, $this->name, array_merge($this->routes, (array)$managedRoutes));
     }
 
     public function getManagerClass(): string

src/Filters/Presets/CheckboxFilter.php

Spacing +1 added lines, -1 removed lines

@@ -28,7 +28,7 @@
 
     protected function viewData(): array
     {
-        return array_merge( parent::viewData(),[
+        return array_merge(parent::viewData(), [
             'options' => $this->options,
         ]);
     }