thinktomorrow / chief

src/Shared/helpers.php

Spacing +9 added lines, -9 removed lines

@@ -20,7 +20,7 @@ 
         $manifestPath = '/chief-assets/back';
 
         // Manifest expects each entry to start with a leading slash - we make sure to deduplicate the manifest path.
-        $entry = str_replace($manifestPath, '', '/' . ltrim($filepath, '/'));
+        $entry = str_replace($manifestPath, '', '/'.ltrim($filepath, '/'));
 
         try {
             // Paths should be given relative to the manifestpath so make sure to remove the basepath
@@ -28,7 +28,7 @@ 
         } catch (\Exception $e) {
             \Illuminate\Support\Facades\Log::error($e);
 
-            return $manifestPath . $entry;
+            return $manifestPath.$entry;
         }
     }
 }
@@ -116,7 +116,7 @@ 
 
         $teaser = mb_substr($text, 0, $max, 'utf-8');
 
-        return strlen($text) <= $max ? $teaser : $teaser . $ending;
+        return strlen($text) <= $max ? $teaser : $teaser.$ending;
     }
 }
 
@@ -247,16 +247,16 @@ 
             'fragment',
         ], null), $parsed_url, $overrides);
 
-        $scheme = $parsed_url['scheme'] ? $parsed_url['scheme'] . '://' : null;
-        $port = $parsed_url['port'] ? ':' . $parsed_url['port'] : null;
-        $fragment = $parsed_url['fragment'] ? '#' . $parsed_url['fragment'] : null;
+        $scheme = $parsed_url['scheme'] ? $parsed_url['scheme'].'://' : null;
+        $port = $parsed_url['port'] ? ':'.$parsed_url['port'] : null;
+        $fragment = $parsed_url['fragment'] ? '#'.$parsed_url['fragment'] : null;
 
-        $baseurl = $scheme . $parsed_url['host'] . $port . $parsed_url['path'];
+        $baseurl = $scheme.$parsed_url['host'].$port.$parsed_url['path'];
         $current_query = [];
 
         $_query = explode('&', $parsed_url['query']);
 
-        array_map(function ($v) use (&$current_query) {
+        array_map(function($v) use (&$current_query) {
             if (!$v) {
                 return;
             }
@@ -274,7 +274,7 @@ 
 
         $query = urldecode(http_build_query(array_merge($current_query, $query_params)));
 
-        return $baseurl . '?' . $query . $fragment;
+        return $baseurl.'?'.$query.$fragment;
     }
 }
 

src/Admin/Nav/Nav.php

Spacing +2 added lines, -2 removed lines

@@ -30,7 +30,7 @@ 
 
     public function untagged(): self
     {
-        $filteredItems = array_filter($this->items, function(NavItem $navItem){
+        $filteredItems = array_filter($this->items, function(NavItem $navItem) {
             return $navItem->isUntagged();
         });
 
@@ -41,7 +41,7 @@ 
     {
         $instance = new static();
 
-        foreach($navItems as $item) {
+        foreach ($navItems as $item) {
             $instance->add($item);
         }
 

src/Managers/Assistants/FragmentsAssistant.php

Spacing +4 added lines, -4 removed lines

@@ -25,14 +25,14 @@ 
 
     public function routeFragmentsAssistant(string $action, $model = null, ...$parameters): ?string
     {
-        if(!in_array($action, ['fragments-index','fragments-reorder'])) return null;
+        if (!in_array($action, ['fragments-index', 'fragments-reorder'])) return null;
 
         return $this->generateRoute($action, $model, ...$parameters);
     }
 
     public function canFragmentsAssistant(string $action, $model = null): bool
     {
-        return in_array($action, ['fragments-index','fragments-reorder']);
+        return in_array($action, ['fragments-index', 'fragments-reorder']);
     }
 
     public function fragmentsIndex(Request $request, $id)
@@ -43,7 +43,7 @@ 
         $model = $modelClass::findOrFail($id);
 
         // Current fragments
-        $fragments = app(FragmentRepository::class)->getByOwner($model)->map(function(Fragmentable $model){
+        $fragments = app(FragmentRepository::class)->getByOwner($model)->map(function(Fragmentable $model) {
             return [
                 'model'    => $model,
                 'manager'  => app(Registry::class)->manager($model::managedModelKey()),
@@ -51,7 +51,7 @@ 
         });
 
         // Available fragments
-        $allowedFragments = array_map(function($fragmentableClass){
+        $allowedFragments = array_map(function($fragmentableClass) {
             $modelClass = app(Registry::class)->modelClass($fragmentableClass::managedModelKey());
             return [
                 'manager' => app(Registry::class)->manager($fragmentableClass::managedModelKey()),

Braces +3 added lines, -1 removed lines

@@ -25,7 +25,9 @@
 
     public function routeFragmentsAssistant(string $action, $model = null, ...$parameters): ?string
     {
-        if(!in_array($action, ['fragments-index','fragments-reorder'])) return null;
+        if(!in_array($action, ['fragments-index','fragments-reorder'])) {
+            return null;
+        }
 
         return $this->generateRoute($action, $model, ...$parameters);
     }

src/Managers/Assistants/FragmentAssistant.php

Spacing +7 added lines, -7 removed lines

@@ -32,18 +32,18 @@ 
 
     public function routeFragmentAssistant(string $action, $model = null, ...$parameters): ?string
     {
-        if(!in_array($action, ['fragment-edit','fragment-update','fragment-delete','fragment-create','fragment-store'])) {
+        if (!in_array($action, ['fragment-edit', 'fragment-update', 'fragment-delete', 'fragment-create', 'fragment-store'])) {
             return null;
         };
 
         $modelKey = $this->managedModelClass()::managedModelKey();
 
-        if(in_array($action, ['fragment-create', 'fragment-store'])) {
+        if (in_array($action, ['fragment-create', 'fragment-store'])) {
             if (!$model || !$model instanceof FragmentsOwner) {
                 throw new \Exception('Fragment route definition for '.$action.' requires the owning Model as second argument.');
             }
 
-            return route('chief.' . $modelKey . '.' . $action, array_merge([
+            return route('chief.'.$modelKey.'.'.$action, array_merge([
                 $model::managedModelKey(),
                 $model->modelReference()->id(),
             ], $parameters));
@@ -53,12 +53,12 @@ 
             throw new \Exception('Fragment route definition for '.$action.' requires the fragment model as second argument.');
         }
 
-        return route('chief.' . $modelKey . '.' . $action, $model->fragmentModel()->id);
+        return route('chief.'.$modelKey.'.'.$action, $model->fragmentModel()->id);
     }
 
     public function canFragmentAssistant(string $action, $model = null): bool
     {
-        return in_array($action, ['fragment-edit','fragment-update','fragment-delete','fragment-create','fragment-store']);
+        return in_array($action, ['fragment-edit', 'fragment-update', 'fragment-delete', 'fragment-create', 'fragment-store']);
     }
 
     public function fragmentCreate(Request $request, string $ownerKey, $ownerId)
@@ -145,13 +145,13 @@ 
         $model = $this->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.');
     }
 
     private function owner(string $ownerKey, $ownerId): FragmentsOwner

src/Managers/Assistants/CrudAssistant.php

Spacing +6 added lines, -6 removed lines

@@ -100,14 +100,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');
             }));
         }
@@ -141,7 +141,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(Request $request, $id)
@@ -172,7 +172,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(Request $request, $id)
@@ -182,12 +182,12 @@ 
         $model = $this->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.');
     }
 }