thinktomorrow / chief

src/OLD/Management/Details/HasDetails.php

Spacing +4 added lines, -4 removed lines

@@ -16,7 +16,7 @@ 
         $genericModelInstance = $this->modelInstance();
 
         // Generic model details
-        $id = Str::slug($this->registration->key() . ($this->hasExistingModel() ? '-' . $this->existingModel()->id : ''));
+        $id = Str::slug($this->registration->key().($this->hasExistingModel() ? '-'.$this->existingModel()->id : ''));
         $key = $this->registration->key();
         $labelSingular = property_exists($genericModelInstance, 'labelSingular') ? $genericModelInstance->labelSingular : Str::singular($key);
         $labelPlural = property_exists($genericModelInstance, 'labelPlural') ? $genericModelInstance->labelPlural : Str::plural($key);
@@ -32,10 +32,10 @@ 
         return new Details(
             $id,
             $key,
-            $labelSingular . '',
-            $labelPlural . '',
+            $labelSingular.'',
+            $labelPlural.'',
             $internal_label,
-            $title . ''
+            $title.''
         );
     }
 }

src/Nav/NavItem.php

Spacing +2 added lines, -2 removed lines

@@ -31,7 +31,7 @@ 
 
     public function isTagged($tags): bool
     {
-        return count(array_intersect($this->tags, (array) $tags)) > 0;
+        return count(array_intersect($this->tags, (array)$tags)) > 0;
     }
 
     public function isUntagged(): bool
@@ -41,7 +41,7 @@ 
 
     public function render(): string
     {
-        $output = '<a class="' . (isActiveUrl($this->url()) ? 'active' : '') . '" href="' . $this->url() . '">';
+        $output = '<a class="'.(isActiveUrl($this->url()) ? 'active' : '').'" href="'.$this->url().'">';
         $output .= ucfirst($this->label());
         $output .= '</a>';
 

src/Nav/Nav.php

Spacing +8 added lines, -8 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);
         }
 
@@ -55,21 +55,21 @@ 
 
     public function render(): string
     {
-        return array_reduce($this->items, function($carry, $item){
-            return $carry .'<li>'. $item->render() . '</li>';
+        return array_reduce($this->items, function($carry, $item) {
+            return $carry.'<li>'.$item->render().'</li>';
         } ,'');
     }
 
     public function renderDropdown(string $label): string
     {
-        $items = array_reduce($this->items, function($carry, $item){
-            return $carry . $item->render();
+        $items = array_reduce($this->items, function($carry, $item) {
+            return $carry.$item->render();
         } ,'');
 
         // Surround with vue dropdown
         $output = '<dropdown>';
-        $output .= '<span class="center-y nav-item" slot="trigger" slot-scope="{ toggle, isActive }" @click="toggle">' . $label . '</span>';
-        $output .= '<div v-cloak class="dropdown-box inset-s">' . $items . '</div></dropdown>';
+        $output .= '<span class="center-y nav-item" slot="trigger" slot-scope="{ toggle, isActive }" @click="toggle">'.$label.'</span>';
+        $output .= '<div v-cloak class="dropdown-box inset-s">'.$items.'</div></dropdown>';
 
         return $output;
     }

src/Menu/MenuItem.php

Spacing +2 added lines, -2 removed lines

@@ -104,7 +104,7 @@ 
             if ($item->ofType(static::TYPE_INTERNAL) && $page = $item->page) {
                 if (public_method_exists($page, 'isArchived') && $page->isArchived()) {
                     unset($items[$k]);
-                } else {
+                }else {
                     $item->url = self::composePageUrl($item, $page);
                     $item->page_label = public_method_exists($page, 'menuLabel') ? $page->menuLabel() : '';
                     $item->hidden_in_menu = $page->hidden_in_menu;
@@ -153,7 +153,7 @@ 
             'id'             => $node->id,
             'type'           => $node->type,
             'label'          => $node->label,
-            'page_label'     => $node->page_label,                       // Extra info when dealing with internal links
+            'page_label'     => $node->page_label, // Extra info when dealing with internal links
             'url'            => $node->url,
             'order'          => $node->order,
             'page_id'        => $node->page_id,

src/Urls/Field/Url.php

Spacing +3 added lines, -3 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types = 1);
+declare(strict_types=1);
 
 namespace Thinktomorrow\Chief\Urls\Field;
 
@@ -19,9 +19,9 @@ 
 
         // Push update to homepage setting value
         // TODO: we should just fetch the homepages and push that instead...
-        UrlRecord::getByModel($model)->reject(function ($record) {
+        UrlRecord::getByModel($model)->reject(function($record) {
             return ($record->isRedirect() || !$record->isHomepage());
-        })->each(function ($record) {
+        })->each(function($record) {
             app(ChangeHomepage::class)->onUrlChanged($record);
         });
     }

src/Urls/Field/UrlSlugFields.php

Spacing +9 added lines, -9 removed lines

@@ -18,14 +18,14 @@ 
 
     public static function redirectsFromModel(ProvidesUrl $model)
     {
-        $records = MemoizedUrlRecord::getByModel($model)->reject(function ($record) {
+        $records = MemoizedUrlRecord::getByModel($model)->reject(function($record) {
             return !$record->isRedirect();
         })->sortByDesc('created_at');
 
         $fields = new static([]);
 
         foreach ($records as $record) {
-            $key = 'redirects-' . $record->locale . '-' . $record->slug;
+            $key = 'redirects-'.$record->locale.'-'.$record->slug;
             $fields[$key] = UrlSlugField::make($key)
                 ->setUrlRecord($record)
                 ->setFullUrl($model->resolveUrl($record->locale, $record->slug));
@@ -60,10 +60,10 @@ 
         $fields = new static([]);
 
         foreach ($locales as $locale) {
-            $fields['url-slugs.' . $locale] = UrlSlugField::make('url-slugs.' . $locale)
+            $fields['url-slugs.'.$locale] = UrlSlugField::make('url-slugs.'.$locale)
                 ->setBaseUrlSegment($model->baseUrlSegment($locale))
-                ->prepend($model->resolveUrl($locale, $model->baseUrlSegment($locale)) . '/')
-                ->name('url-slugs[' . $locale . ']')
+                ->prepend($model->resolveUrl($locale, $model->baseUrlSegment($locale)).'/')
+                ->name('url-slugs['.$locale.']')
                 ->label($locale);
         }
 
@@ -76,19 +76,19 @@ 
      */
     private static function fillWithExistingValues(ProvidesUrl $model, self $fields): void
     {
-        $records = UrlRecord::getByModel($model)->reject(function ($record) {
+        $records = UrlRecord::getByModel($model)->reject(function($record) {
             return $record->isRedirect();
         })->sortBy('locale');
 
         foreach ($records as $record) {
-            if (!isset($fields['url-slugs.' . $record->locale])) {
+            if (!isset($fields['url-slugs.'.$record->locale])) {
                 continue;
             }
 
-            $fields['url-slugs.' . $record->locale]
+            $fields['url-slugs.'.$record->locale]
                 ->setUrlRecord($record)
                 ->setBaseUrlSegment($model->baseUrlSegment($record->locale))
-                ->prepend($model->resolveUrl($record->locale, $model->baseUrlSegment($record->locale)) . '/');
+                ->prepend($model->resolveUrl($record->locale, $model->baseUrlSegment($record->locale)).'/');
         }
     }
 }

src/Urls/Field/UrlSlugField.php

Spacing +2 added lines, -2 removed lines

@@ -42,7 +42,7 @@ 
     {
         return $this->fullUrl
             ? $this->fullUrl
-            : $this->prepend . $this->getValue();
+            : $this->prepend.$this->getValue();
     }
 
     public function setFullUrl(string $fullUrl)
@@ -94,7 +94,7 @@ 
             'baseUrlSegment' => $this->baseUrlSegment,
             'hint'           => null, // Hint placeholder to show url hint when it already exists
             'is_homepage'    => ($this->getValue() === '/'),
-            'show'           => !!$this->getValue(),// show input field or not
+            'show'           => !!$this->getValue(), // show input field or not
         ];
     }
 }

src/Urls/UrlHelper.php

Spacing +5 added lines, -5 removed lines

@@ -89,7 +89,7 @@ 
 
     public static function modelsByType(array $types, Model $ignoredModel = null, bool $online = true)
     {
-        $models = chiefMemoize('all-online-models-' . implode('_', $types), function () use ($types, $online) {
+        $models = chiefMemoize('all-online-models-'.implode('_', $types), function() use ($types, $online) {
             $builder = UrlRecord::whereNull('redirect_id')
                 ->select('model_type', 'model_id')
                 ->groupBy('model_type', 'model_id');
@@ -98,11 +98,11 @@ 
                 $builder->whereIn('model_type', $types);
             }
 
-            return $builder->get()->mapToGroups(function ($record) {
+            return $builder->get()->mapToGroups(function($record) {
                 return [$record->model_type => $record->model_id];
-            })->map(function ($record, $key) {
+            })->map(function($record, $key) {
                 return Morphables::instance($key)->find($record->toArray());
-            })->map->reject(function ($model) use ($online) {
+            })->map->reject(function($model) use ($online) {
                 if ($online) {
                     return is_null($model) || (public_method_exists($model, 'isPublished') && !$model->isPublished());
                 } // Invalid references to archived or removed models where url record still exists.
@@ -112,7 +112,7 @@ 
         });
 
         if ($ignoredModel) {
-            $models = $models->reject(function ($model) use ($ignoredModel) {
+            $models = $models->reject(function($model) use ($ignoredModel) {
                 return (get_class($model) === get_class($ignoredModel) && $model->id === $ignoredModel->id);
             });
         }

src/Media/Application/AbstractMediaFieldHandler.php

Spacing +4 added lines, -4 removed lines

@@ -140,10 +140,10 @@ 
             return $filename;
         }
 
-        $extension = substr($filename, strrpos($filename, '.') + 1);
+        $extension = substr($filename, strrpos($filename, '.')+1);
         $filename = substr($filename, 0, strrpos($filename, '.'));
 
-        return Str::slug($filename) . '.' . $extension;
+        return Str::slug($filename).'.'.$extension;
     }
 
     protected function sort(HasAsset $model, MediaField $field, array $input)
@@ -169,8 +169,8 @@ 
         $values = isset($fileIdInput[$key])
             ? $fileIdInput[$key]
             : (
-                isset($fileIdInput['files-' . $key])
-                ? $fileIdInput['files-' . $key]
+                isset($fileIdInput['files-'.$key])
+                ? $fileIdInput['files-'.$key]
                 : ''
             );