thinktomorrow / chief

src/PageBuilder/PageBuilder.php

Spacing +10 added lines, -10 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types = 1);
+declare(strict_types=1);
 
 namespace Thinktomorrow\Chief\PageBuilder;
 
@@ -42,7 +42,7 @@ 
         $availableChildren = AvailableChildren::forParent($model);
 
         // Only include modules of this model or shared ones.
-        $modules = $availableChildren->onlyModules()->reject(function ($module) use ($model) {
+        $modules = $availableChildren->onlyModules()->reject(function($module) use ($model) {
             return $module->owner_id != null && $module->owner_id != $model->id;
         });
 
@@ -51,7 +51,7 @@ 
         $available_sets = FlatReferencePresenter::toGroupedSelectValues($availableChildren->onlySets())->toArray();
 
         // Current sections
-        $sections = $model->children()->map(function ($section, $index) use($model) {
+        $sections = $model->children()->map(function($section, $index) use($model) {
 //            if ($section instanceof TranslatableContract) {
 //
 //            }
@@ -88,14 +88,14 @@ 
         $trans = [];
 
         // Has dynamic attributes
-        if(public_method_exists($model, 'dynamic')) {
+        if (public_method_exists($model, 'dynamic')) {
             foreach (config('thinktomorrow.chief.locales') as $locale) {
                 $trans[$locale] = [];
 
-                foreach($model->rawDynamicValues() as $key => $values) {
-                    if(is_array($values)) {
-                        foreach($values as $dynamicLocale => $dynamicValue) {
-                            if($dynamicLocale === $locale) {
+                foreach ($model->rawDynamicValues() as $key => $values) {
+                    if (is_array($values)) {
+                        foreach ($values as $dynamicLocale => $dynamicValue) {
+                            if ($dynamicLocale === $locale) {
                                 $trans[$locale][$key] = $dynamicValue;
                             }
                         }
@@ -105,9 +105,9 @@ 
         }
 
         // Astrotomic translatable
-        if($model instanceof Translatable) {
+        if ($model instanceof Translatable) {
             foreach (config('thinktomorrow.chief.locales') as $locale) {
-                if(!isset($trans[$locale])) $trans[$locale] = [];
+                if (!isset($trans[$locale])) $trans[$locale] = [];
 
                 $trans[$locale] = array_merge($trans[$locale], $model->getTranslation($locale)->toArray());
             }

Braces +3 added lines, -1 removed lines

@@ -107,7 +107,9 @@
         // Astrotomic translatable
         if($model instanceof Translatable) {
             foreach (config('thinktomorrow.chief.locales') as $locale) {
-                if(!isset($trans[$locale])) $trans[$locale] = [];
+                if(!isset($trans[$locale])) {
+                    $trans[$locale] = [];
+                }
 
                 $trans[$locale] = array_merge($trans[$locale], $model->getTranslation($locale)->toArray());
             }

src/PageBuilder/Relations/AvailableChildren.php

Spacing +4 added lines, -4 removed lines

@@ -36,14 +36,14 @@ 
 
     public function onlyModules(): Collection
     {
-        return $this->collection()->reject(function ($item) {
+        return $this->collection()->reject(function($item) {
             return ($item instanceof ActsAsParent);
         });
     }
 
     public function onlyPages(): Collection
     {
-        return $this->collection()->filter(function ($item) {
+        return $this->collection()->filter(function($item) {
             return ($item instanceof ActsAsParent);
         });
     }
@@ -98,12 +98,12 @@ 
 //        }
 
         // Remove duplicate models
-        $collection = $collection->unique(function ($model) {
+        $collection = $collection->unique(function($model) {
             return (new FlatReference(get_class($model), $model->id))->get();
         });
 
         // Filter out our parent
-        return $this->collection = $collection->reject(function ($item) {
+        return $this->collection = $collection->reject(function($item) {
             return ($item instanceof $this->parent && $item->id == $this->parent->id);
         })->values();
     }

src/Managers/Assistants/CrudAssistant.php

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