thinktomorrow / chief

src/Nav/Nav.php

Spacing +9 added lines, -9 removed lines

@@ -19,7 +19,7 @@ 
 
     public static function fromKeys($keys)
     {
-        $keys = (array) $keys;
+        $keys = (array)$keys;
         $collection = collect();
 
         /** @var Managers */
@@ -33,7 +33,7 @@ 
 
     public static function fromTags($tags)
     {
-        $tags = (array) $tags;
+        $tags = (array)$tags;
         $collection = collect();
 
         /** @var Managers */
@@ -52,9 +52,9 @@ 
 
     private static function fromManagers(Collection $collection)
     {
-        return new static(...$collection->reject(function ($manager) {
+        return new static(...$collection->reject(function($manager) {
             return !$manager->can('index');
-        })->map(function ($manager) {
+        })->map(function($manager) {
             return new NavItem($manager->details()->plural, $manager->route('index'), [
                 'key' => $manager->details()->key,
                 'tags' => app(Register::class)->filterByKey($manager->details()->key)->first()->tags()
@@ -64,7 +64,7 @@ 
 
     public function rejectKeys($keys)
     {
-        $keys = (array) $keys;
+        $keys = (array)$keys;
 
         foreach ($this->items as $k => $item) {
             if (in_array($item->details('key', ''), $keys)) {
@@ -77,7 +77,7 @@ 
 
     public function rejectTags($tags)
     {
-        $tags = (array) $tags;
+        $tags = (array)$tags;
 
         foreach ($this->items as $k => $item) {
             if (count(array_intersect($item->details('tags', []), $tags)) > 0) {
@@ -119,7 +119,7 @@ 
         $output = '';
 
         foreach ($this->items as $item) {
-            $output .= '<a class="' . (isActiveUrl($item->url()) ? 'active' : '') . '" href="'.$item->url().'">';
+            $output .= '<a class="'.(isActiveUrl($item->url()) ? 'active' : '').'" href="'.$item->url().'">';
             $output .= $title ?? ucfirst($item->title());
             $output .= '</a>';
         }
@@ -146,14 +146,14 @@ 
 
         $items = '';
         foreach ($this->items as $item) {
-            $items .= '<a class="' . (isActiveUrl($item->url()) ? 'active' : '') . '" href="'.$item->url().'">';
+            $items .= '<a class="'.(isActiveUrl($item->url()) ? 'active' : '').'" href="'.$item->url().'">';
             $items .= ucfirst($item->title());
             $items .= '</a>';
         }
 
         // Surround within vue dropdown
         $output = '<dropdown>';
-        $output .= '<span class="center-y nav-item" slot="trigger" slot-scope="{ toggle, isActive }" @click="toggle">'. ($title ?? 'Collecties') .'</span>';
+        $output .= '<span class="center-y nav-item" slot="trigger" slot-scope="{ toggle, isActive }" @click="toggle">'.($title ?? 'Collecties').'</span>';
         $output .= '<div v-cloak class="dropdown-box inset-s">';
         $output .= $items;
         $output .= '</div>';

src/Settings/Application/ChangeHomepage.php

Spacing +1 added lines, -1 removed lines

@@ -53,7 +53,7 @@
     {
         foreach ($flatReferences as $locale => $flatReferenceString) {
             if (!$flatReferenceString) {
-                throw new \InvalidArgumentException('Homepage setting value cannot be empty. Value for locale ' . $locale . ' is missing.');
+                throw new \InvalidArgumentException('Homepage setting value cannot be empty. Value for locale '.$locale.' is missing.');
             }
         }
     }

src/Fields/LocalizedFieldValidationRules.php

Spacing +3 added lines, -3 removed lines

@@ -27,7 +27,7 @@ 
                 // prepend the name with the expected trans.<locale>. string
                 $localizedAttr = (false !== strpos($attr, ':locale'))
                     ? str_replace(':locale', $locale, $attr)
-                    : 'trans.' . $locale . '.' . $attr;
+                    : 'trans.'.$locale.'.'.$attr;
 
                 $localizedAttr = $this->replaceBracketsByDots($localizedAttr);
 
@@ -49,13 +49,13 @@ 
      */
     public function influenceByPayload(array $data)
     {
-        if (! isset($data['trans'])) {
+        if (!isset($data['trans'])) {
             return $this;
         }
 
         // Remove locales that are considered empty in the request payload
         foreach ($data['trans'] as $locale => $values) {
-            if ($locale == $this->defaultLocale || ! is_array_empty($values)) {
+            if ($locale == $this->defaultLocale || !is_array_empty($values)) {
                 continue;
             }
 

src/Urls/UrlHelper.php

Spacing +4 added lines, -4 removed lines

@@ -15,21 +15,21 @@
      */
     public static function allOnlineModels(bool $onlySingles = false): array
     {
-        return chiefMemoize('all-online-models', function () use ($onlySingles) {
+        return chiefMemoize('all-online-models', function() use ($onlySingles) {
             $builder = UrlRecord::whereNull('redirect_id')->select('model_type', 'model_id')->groupBy('model_type', 'model_id');
 
             if ($onlySingles) {
                 $builder->where('model_type', 'singles');
             }
 
-            $liveUrlRecords = $builder->get()->mapToGroups(function ($record) {
+            $liveUrlRecords = $builder->get()->mapToGroups(function($record) {
                 return [$record->model_type => $record->model_id];
             });
 
             // Get model for each of these records...
-            $models = $liveUrlRecords->map(function ($record, $key) {
+            $models = $liveUrlRecords->map(function($record, $key) {
                 return Morphables::instance($key)->find($record->toArray());
-            })->map->reject(function ($model) {
+            })->map->reject(function($model) {
                 // Invalid references to archived or removed models where url record still exists.
                 return is_null($model) || !$model->isPublished();
             })->flatten();

src/Urls/UrlSlugFields.php

Spacing +6 added lines, -6 removed lines

@@ -18,7 +18,7 @@ 
 
     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');
 
@@ -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,7 +76,7 @@ 
      */
     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');
 
@@ -88,7 +88,7 @@ 
             $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/Media/UploadMedia.php

Spacing +14 added lines, -14 removed lines

@@ -50,7 +50,7 @@ 
 
     private function addFiles(HasMedia $model, string $type, array $files, array &$files_order, string $locale = null)
     {
-        if(!$this->actionExists($files, 'new')) return;
+        if (!$this->actionExists($files, 'new')) return;
 
         foreach ($files['new'] as $id => $file) {
             if (!$file) {
@@ -68,7 +68,7 @@ 
      */
     private function replaceFiles(HasMedia $model, array $files)
     {
-        if(!$this->actionExists($files, 'replace')) return;
+        if (!$this->actionExists($files, 'replace')) return;
 
         foreach ($files['replace'] as $id => $file) {
             if (!$file) {
@@ -86,7 +86,7 @@ 
      */
     private function removeFiles(HasMedia $model, array $files)
     {
-        if(!$this->actionExists($files, 'delete')) return;
+        if (!$this->actionExists($files, 'delete')) return;
 
         foreach ($model->assets()->whereIn('id', $files['delete'])->get() as $asset) {
             $asset->delete();
@@ -103,7 +103,7 @@ 
         if (is_string($file)) {
             $image_name = json_decode($file)->output->name;
             $asset      = $this->addAsset(json_decode($file)->output->image, $type, $locale, $image_name, $model);
-        } else {
+        }else {
             $image_name = $file->getClientOriginalName();
             $asset      = $this->addAsset($file, $type, $locale, $image_name, $model);
         }
@@ -126,7 +126,7 @@ 
 
         if (is_string($file)) {
             $asset = AssetUploader::uploadFromBase64($file, $filename);
-        } else {
+        }else {
             $asset = AssetUploader::upload($file, $filename);
         }
 
@@ -143,9 +143,9 @@ 
      */
     private function sluggifyFilename($filename): string
     {
-        $extension = substr($filename, strrpos($filename, '.') + 1);
+        $extension = substr($filename, strrpos($filename, '.')+1);
         $filename  = substr($filename, 0, strrpos($filename, '.'));
-        $filename  = str_slug($filename) . '.' . $extension;
+        $filename  = str_slug($filename).'.'.$extension;
 
         return $filename;
     }
@@ -161,9 +161,9 @@ 
                 if ($file instanceof UploadedFile && !$file->isValid()) {
                     if ($file->getError() == UPLOAD_ERR_INI_SIZE) {
                         throw new FileTooBigException(
-                            'Cannot upload file because it exceeded the allowed upload_max_filesize: upload_max_filesize is smaller than post size. ' .
-                            'upload_max_filesize: ' . (int)ini_get('upload_max_filesize') . 'MB, ' .
-                            'post_max_size: ' . (int)(ini_get('post_max_size')) . 'MB'
+                            'Cannot upload file because it exceeded the allowed upload_max_filesize: upload_max_filesize is smaller than post size. '.
+                            'upload_max_filesize: '.(int)ini_get('upload_max_filesize').'MB, '.
+                            'post_max_size: '.(int)(ini_get('post_max_size')).'MB'
                         );
                     }
                 }
@@ -178,16 +178,16 @@ 
         foreach ($files_by_type as $type => $files) {
             foreach ($files as $locale => $_files) {
                 if (!in_array($locale, config('translatable.locales'))) {
-                    throw new \InvalidArgumentException('Corrupt file payload. key is expected to be a valid locale [' . implode(',', config('translatable.locales', [])). ']. Instead [' . $locale . '] is given.');
+                    throw new \InvalidArgumentException('Corrupt file payload. key is expected to be a valid locale ['.implode(',', config('translatable.locales', [])).']. Instead ['.$locale.'] is given.');
                 }
 
                 if (!is_array($_files)) {
-                    throw new \InvalidArgumentException('A valid files entry should be an array of files, key with either [new, replace or delete]. Instead a ' . gettype($_files) . ' is given.');
+                    throw new \InvalidArgumentException('A valid files entry should be an array of files, key with either [new, replace or delete]. Instead a '.gettype($_files).' is given.');
                 }
 
                 foreach ($_files as $action => $file) {
                     if (!in_array($action, $actions)) {
-                        throw new \InvalidArgumentException('A valid files entry should have a key of either ['.implode(',', $actions).']. Instead ' . $action . ' is given.');
+                        throw new \InvalidArgumentException('A valid files entry should have a key of either ['.implode(',', $actions).']. Instead '.$action.' is given.');
                     }
                 }
             }
@@ -196,7 +196,7 @@ 
         foreach ($files_order_by_type as $type => $fileIdsCollection) {
             foreach ($fileIdsCollection as $locale => $commaSeparatedFileIds) {
                 if (!in_array($locale, config('translatable.locales'))) {
-                    throw new \InvalidArgumentException('Corrupt file payload. key for the file order is expected to be a valid locale [' . implode(',', config('translatable.locales', [])). ']. Instead [' . $locale . '] is given.');
+                    throw new \InvalidArgumentException('Corrupt file payload. key for the file order is expected to be a valid locale ['.implode(',', config('translatable.locales', [])).']. Instead ['.$locale.'] is given.');
                 }
             }
         }

Braces +9 added lines, -3 removed lines

@@ -50,7 +50,9 @@ 
 
     private function addFiles(HasMedia $model, string $type, array $files, array &$files_order, string $locale = null)
     {
-        if(!$this->actionExists($files, 'new')) return;
+        if(!$this->actionExists($files, 'new')) {
+            return;
+        }
 
         foreach ($files['new'] as $id => $file) {
             if (!$file) {
@@ -68,7 +70,9 @@ 
      */
     private function replaceFiles(HasMedia $model, array $files)
     {
-        if(!$this->actionExists($files, 'replace')) return;
+        if(!$this->actionExists($files, 'replace')) {
+            return;
+        }
 
         foreach ($files['replace'] as $id => $file) {
             if (!$file) {
@@ -86,7 +90,9 @@ 
      */
     private function removeFiles(HasMedia $model, array $files)
     {
-        if(!$this->actionExists($files, 'delete')) return;
+        if(!$this->actionExists($files, 'delete')) {
+            return;
+        }
 
         foreach ($model->assets()->whereIn('id', $files['delete'])->get() as $asset) {
             $asset->delete();

src/PageBuilder/UpdateSections.php

Spacing +7 added lines, -7 removed lines

@@ -91,7 +91,7 @@ 
     private function removeExistingSets()
     {
         foreach ($this->model->children() as $instance) {
-            if (! $instance instanceof StoredSetReference) {
+            if (!$instance instanceof StoredSetReference) {
                 continue;
             }
             $this->model->rejectChild($instance);
@@ -145,7 +145,7 @@ 
         }
 
         foreach ($this->text_modules['replace'] as $text_module) {
-            if (! $module = FlatReferenceFactory::fromString($text_module['id'])->instance()) {
+            if (!$module = FlatReferenceFactory::fromString($text_module['id'])->instance()) {
                 continue;
             }
 
@@ -155,10 +155,10 @@ 
                 continue;
             }
            
-            foreach($text_module['trans'] as $locale => $content)
+            foreach ($text_module['trans'] as $locale => $content)
             {
                 $purifier = new HTMLPurifier();
-                $sanitized_text = $purifier->purify( $content['content'] );
+                $sanitized_text = $purifier->purify($content['content']);
 
                 $text_module['trans'][$locale]['content'] = $sanitized_text;
             }
@@ -172,7 +172,7 @@ 
 
     private function removeTextualModule($module)
     {
-        if (! $module instanceof TextModule && ! $module instanceof PagetitleModule) {
+        if (!$module instanceof TextModule && !$module instanceof PagetitleModule) {
             throw new \Exception('Invalid request to remove non textual module');
         }
 
@@ -194,7 +194,7 @@ 
                 continue;
             }
 
-            $child = $children->first(function ($c) use ($reference) {
+            $child = $children->first(function($c) use ($reference) {
                 return $c->flatReference()->get() == $reference;
             });
 
@@ -245,7 +245,7 @@ 
     private function stripTagsBlacklist($value, $blacklist = [])
     {
         foreach ($blacklist as $tag) {
-            $value = preg_replace('/<\/?' . $tag . '(.|\s)*?>/', '', $value);
+            $value = preg_replace('/<\/?'.$tag.'(.|\s)*?>/', '', $value);
         }
 
         return $value;

src/Common/helpers.php

Spacing +14 added lines, -14 removed lines

@@ -74,7 +74,7 @@ 
         $empty = true;
 
         foreach ($values as $value) {
-            if (! $value || !trim($value)) {
+            if (!$value || !trim($value)) {
                 continue;
             }
             $empty = false;
@@ -84,7 +84,7 @@ 
     }
 }
 
-if (! function_exists('contract')) {
+if (!function_exists('contract')) {
     function contract($instance, $contract)
     {
         return $instance instanceof $contract;
@@ -113,7 +113,7 @@ 
             $text = cleanupHTML($text, $clean);
         }
         $teaser = substr($text, 0, $max);
-        return strlen($text) <= $max ? $teaser : $teaser . $ending;
+        return strlen($text) <= $max ? $teaser : $teaser.$ending;
     }
 }
 
@@ -166,13 +166,13 @@ 
         }
         // // cleanup HTML and any unwanted attributes
         $purifier = new HTMLPurifier();
-        $value = $purifier->purify( $value );
+        $value = $purifier->purify($value);
 
         /**
          * htmlPurifier converts characters to their encode equivalents. This is something
          * that we need to reverse after the htmlPurifier cleanup.
          */
-        $value  = str_replace('&amp;', '&', $value);
+        $value = str_replace('&amp;', '&', $value);
 
         return $value;
     }
@@ -240,16 +240,16 @@ 
             'scheme', 'host', 'port', 'path', 'query', '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;
             }
@@ -267,7 +267,7 @@ 
 
         $query = urldecode(http_build_query(array_merge($current_query, $query_params)));
 
-        return $baseurl . '?' . $query . $fragment;
+        return $baseurl.'?'.$query.$fragment;
     }
 }
 
@@ -276,9 +276,9 @@ 
     {
         $trace = debug_backtrace(DEBUG_BACKTRACE_IGNORE_ARGS, 1);
         if (php_sapi_name() == 'cli') {
-            print_r("\e[1;30m dumped at: " . str_replace(base_path(), '', $trace[0]['file']). ", line: " . $trace[0]['line'] . "\e[40m\n");
-        } else {
-            print_r("[dumped at: " . str_replace(base_path(), '', $trace[0]['file']). ", line: " . $trace[0]['line'] . "]\n");
+            print_r("\e[1;30m dumped at: ".str_replace(base_path(), '', $trace[0]['file']).", line: ".$trace[0]['line']."\e[40m\n");
+        }else {
+            print_r("[dumped at: ".str_replace(base_path(), '', $trace[0]['file']).", line: ".$trace[0]['line']."]\n");
         }
         return dd($var, ...$moreVars);
     }

src/Management/AbstractManager.php

Spacing +3 added lines, -3 removed lines

@@ -80,7 +80,7 @@ 
             $builder->orderBy('updated_at', 'DESC');
         }
 
-        return $builder->get()->map(function ($model) {
+        return $builder->get()->map(function($model) {
             return (new static($this->registration))->manage($model);
         });
     }
@@ -148,7 +148,7 @@ 
 
     public function guard($verb): Manager
     {
-        if (! $this->can($verb)) {
+        if (!$this->can($verb)) {
             NotAllowedManagerRoute::notAllowedVerb($verb, $this);
         }
 
@@ -171,7 +171,7 @@ 
         $fields = $this->fields();
 
         foreach ($this->assistants() as $assistant) {
-            if (! method_exists($assistant, 'fields')) {
+            if (!method_exists($assistant, 'fields')) {
                 continue;
             }