thinktomorrow / chief

src/PageBuilder/PresentSections.php

Spacing +2 added lines, -2 removed lines

@@ -78,7 +78,7 @@ 
             $this->addModelToCollection($i, $child);
         }
 
-        return $this->sets->values()->map(function (ViewableContract $child) {
+        return $this->sets->values()->map(function(ViewableContract $child) {
             return ($this->withSnippets && method_exists($child, 'withSnippets'))
                 ? $child->withSnippets()->setViewParent($this->parent)->renderView()
                 : $child->setViewParent($this->parent)->renderView();
@@ -95,7 +95,7 @@ 
     {
         // Only published pages you fool!
         // TODO: check for assistant instead of method existence
-        if (method_exists($model, 'isPublished') && ! $model->isPublished()) {
+        if (method_exists($model, 'isPublished') && !$model->isPublished()) {
             return;
         }
 

src/Urls/MemoizedUrlRecord.php

Spacing +2 added lines, -2 removed lines

@@ -8,14 +8,14 @@
 {
     public static function findByModel(Model $model, string $locale = null): UrlRecord
     {
-        return chiefMemoize('url-records-find-by-model', function($model, $locale = null){
+        return chiefMemoize('url-records-find-by-model', function($model, $locale = null) {
             return parent::findByModel($model, $locale);
         }, [$model, $locale]);
     }
 
     public static function getByModel(Model $model)
     {
-        return chiefMemoize('url-records-get-by-model', function($model){
+        return chiefMemoize('url-records-get-by-model', function($model) {
             return parent::getByModel($model);
         }, [$model]);
     }

src/Urls/ChiefResponse.php

Spacing +11 added lines, -11 removed lines

@@ -14,41 +14,41 @@
 {
     public static function fromRequest(Request $request = null, $locale = null)
     {
-        if(!$request) $request = request();
-        if(!$locale) $locale = app()->getLocale();
+        if (!$request) $request = request();
+        if (!$locale) $locale = app()->getLocale();
 
         return static::fromSlug($request->path(), $locale);
     }
 
     public static function fromSlug(string $slug, $locale = null)
     {
-        if(!$locale) $locale = app()->getLocale();
+        if (!$locale) $locale = app()->getLocale();
 
-        try{
+        try {
             $urlRecord = UrlRecord::findBySlug($slug, $locale);
 
             $model = Morphables::instance($urlRecord->model_type)->find($urlRecord->model_id);
 
-            if($urlRecord->isRedirect()){
+            if ($urlRecord->isRedirect()) {
                 return static::createRedirect($model->url($locale));
             }
 
-            if(method_exists($model, 'isPublished') && ! $model->isPublished()){
+            if (method_exists($model, 'isPublished') && !$model->isPublished()) {
 
                 /** When admin is logged in and this request is in preview mode, we allow the view */
-                if( ! PreviewMode::fromRequest()->check()){
-                    throw new NotFoundHttpException('Model found for request ['. $slug .'] but it is not published.');
+                if (!PreviewMode::fromRequest()->check()) {
+                    throw new NotFoundHttpException('Model found for request ['.$slug.'] but it is not published.');
                 }
             }
 
             return new static($model->renderView(), 200);
         }
-        catch(UrlRecordNotFound | NotFoundMorphKey $e)
+        catch (UrlRecordNotFound | NotFoundMorphKey $e)
         {
-            if(config('thinktomorrow.chief.strict')){ throw $e; }
+            if (config('thinktomorrow.chief.strict')) { throw $e; }
         }
 
-        throw new NotFoundHttpException('No url or model found for request ['. $slug .'] for locale ['.$locale.'].');
+        throw new NotFoundHttpException('No url or model found for request ['.$slug.'] for locale ['.$locale.'].');
     }
 
     private static function createRedirect(string $url)

Braces +10 added lines, -5 removed lines

@@ -14,15 +14,21 @@ 
 {
     public static function fromRequest(Request $request = null, $locale = null)
     {
-        if(!$request) $request = request();
-        if(!$locale) $locale = app()->getLocale();
+        if(!$request) {
+            $request = request();
+        }
+        if(!$locale) {
+            $locale = app()->getLocale();
+        }
 
         return static::fromSlug($request->path(), $locale);
     }
 
     public static function fromSlug(string $slug, $locale = null)
     {
-        if(!$locale) $locale = app()->getLocale();
+        if(!$locale) {
+            $locale = app()->getLocale();
+        }
 
         try{
             $urlRecord = UrlRecord::findBySlug($slug, $locale);
@@ -42,8 +48,7 @@ 
             }
 
             return new static($model->renderView(), 200);
-        }
-        catch(UrlRecordNotFound | NotFoundMorphKey $e)
+        } catch(UrlRecordNotFound | NotFoundMorphKey $e)
         {
             if(config('thinktomorrow.chief.strict')){ throw $e; }
         }

src/Urls/UrlSlugField.php

Spacing +2 added lines, -2 removed lines

@@ -31,11 +31,11 @@
 
     private function rawSlugValue(): string
     {
-        if(!$this->urlRecord) return '';
+        if (!$this->urlRecord) return '';
 
         $slug = $this->urlRecord->slug;
 
-        if($this->startsWithBaseUrlSegment($slug)){
+        if ($this->startsWithBaseUrlSegment($slug)) {
             $slug = trim(substr($slug, strlen($this->baseUrlSegment)), '/');
         }
 

Braces +3 added lines, -1 removed lines

@@ -31,7 +31,9 @@
 
     private function rawSlugValue(): string
     {
-        if(!$this->urlRecord) return '';
+        if(!$this->urlRecord) {
+            return '';
+        }
 
         $slug = $this->urlRecord->slug;
 

src/Management/Application/StoringAndUpdatingFields.php

Spacing +5 added lines, -5 removed lines

@@ -20,11 +20,11 @@ 
         foreach ($manager->fieldsWithAssistantFields() as $field) {
 
             // Custom save methods
-            if($this->detectCustomSaveMethods($manager, $field)) continue;
+            if ($this->detectCustomSaveMethods($manager, $field)) continue;
 
             // Media fields are treated separately
             if ($field->ofType(FieldType::MEDIA, FieldType::DOCUMENT)) {
-                if (! isset($this->saveMethods['_files'])) {
+                if (!isset($this->saveMethods['_files'])) {
                     $this->saveMethods['_files'] = ['field' => new Fields([$field]), 'method' => 'uploadMedia'];
                     continue;
                 }
@@ -34,7 +34,7 @@ 
             }
 
             // Custom set methods - default is the generic setField() method.
-            $methodName = 'set'. ucfirst(camel_case($field->key())) . 'Field';
+            $methodName = 'set'.ucfirst(camel_case($field->key())).'Field';
             (method_exists($manager, $methodName))
                 ? $manager->$methodName($field, $request)
                 : $manager->setField($field, $request);
@@ -46,10 +46,10 @@ 
 
     protected function detectCustomSaveMethods(Manager $manager, Field $field): bool
     {
-        $saveMethodName = 'save'. ucfirst(camel_case($field->key())) . 'Field';
+        $saveMethodName = 'save'.ucfirst(camel_case($field->key())).'Field';
 
         // Custom save method on assistant
-        foreach($manager->assistants() as $assistant)
+        foreach ($manager->assistants() as $assistant)
         {
             if (method_exists($assistant, $saveMethodName)) {
                 $this->saveAssistantMethods[$field->key] = ['field' => $field, 'method' => $saveMethodName, 'assistant' => $assistant];

Braces +3 added lines, -1 removed lines

@@ -20,7 +20,9 @@
         foreach ($manager->fieldsWithAssistantFields() as $field) {
 
             // Custom save methods
-            if($this->detectCustomSaveMethods($manager, $field)) continue;
+            if($this->detectCustomSaveMethods($manager, $field)) {
+                continue;
+            }
 
             // Media fields are treated separately
             if ($field->ofType(FieldType::MEDIA, FieldType::DOCUMENT)) {

src/Management/Assistants/AssistedManager.php

Spacing +4 added lines, -4 removed lines

@@ -17,14 +17,14 @@ 
      */
     public function isAssistedBy(string $assistant): bool
     {
-        return !! $this->getAssistantClass($assistant);
+        return !!$this->getAssistantClass($assistant);
     }
 
     public function assistants(): array
     {
         $assistants = [];
 
-        foreach($this->assistants as $assistant){
+        foreach ($this->assistants as $assistant) {
             $assistants[] = $this->assistant($assistant);
         }
 
@@ -40,8 +40,8 @@ 
      */
     public function assistant(string $assistant): Assistant
     {
-        if (! $this->isAssistedBy($assistant)) {
-            throw new MissingAssistant('No assistant [' . $assistant . '] present on manager ' . get_class($this));
+        if (!$this->isAssistedBy($assistant)) {
+            throw new MissingAssistant('No assistant ['.$assistant.'] present on manager '.get_class($this));
         }
 
         $instance = app($this->getAssistantClass($assistant));

src/Management/Assistants/UrlAssistant.php

Spacing +5 added lines, -5 removed lines

@@ -32,10 +32,10 @@ 
 
     public function manager(Manager $manager)
     {
-        $this->manager  = $manager;
+        $this->manager = $manager;
 
-        if(! $manager->model() instanceof ProvidesUrl){
-            throw new \Exception('UrlAssistant requires the model interfaced by ' . ProvidesUrl::class . '.');
+        if (!$manager->model() instanceof ProvidesUrl) {
+            throw new \Exception('UrlAssistant requires the model interfaced by '.ProvidesUrl::class.'.');
         }
 
         $this->model = $manager->model();
@@ -59,7 +59,7 @@ 
             InputField::make('url-slugs')
                 ->validation(
                     [
-                        'url-slugs' => ['array', 'min:1', new UniqueUrlSlugRule(($this->model && $this->model->exists) ? $this->model : null),],
+                        'url-slugs' => ['array', 'min:1', new UniqueUrlSlugRule(($this->model && $this->model->exists) ? $this->model : null), ],
                     ],
                     [],
                     [
@@ -67,7 +67,7 @@ 
                         'url-slugs.*' => 'taalspecifieke link',
                     ])
                 ->view('chief::back._fields.url-slugs')
-                ->viewData(['fields' => UrlSlugFields::fromModel($this->model) ]),
+                ->viewData(['fields' => UrlSlugFields::fromModel($this->model)]),
         ]);
     }
 

src/Management/AbstractManager.php

Spacing +7 added lines, -7 removed lines

@@ -72,11 +72,11 @@ 
             $this->filters()->apply($builder);
 
             $results = $builder->get();
-        } else {
+        }else {
             $results = $model::all();
         }
 
-        return $results->map(function ($model) {
+        return $results->map(function($model) {
             return (new static($this->registration))->manage($model);
         });
     }
@@ -144,7 +144,7 @@ 
 
     public function guard($verb): Manager
     {
-        if (! $this->can($verb)) {
+        if (!$this->can($verb)) {
             NotAllowedManagerRoute::notAllowedVerb($verb, $this);
         }
 
@@ -166,8 +166,8 @@ 
     {
         $fields = $this->fields();
 
-        foreach($this->assistants() as $assistant){
-            if( ! method_exists($assistant, 'fields')) continue;
+        foreach ($this->assistants() as $assistant) {
+            if (!method_exists($assistant, 'fields')) continue;
 
             $fields = $fields->merge($assistant->fields());
         }
@@ -204,7 +204,7 @@ 
 
                 // Could be translatable field
                 if ($this->isTranslatableKey($field)) {
-                    $attribute = substr($field, strrpos($field, '.') + 1);
+                    $attribute = substr($field, strrpos($field, '.')+1);
                     $locale = substr($field, strlen('trans.'), 2);
 
                     return $this->model->getTranslationFor($attribute, $locale);
@@ -242,7 +242,7 @@ 
 
             // Make our media fields able to be translatable as well...
             if ($field->ofType(FieldType::MEDIA, FieldType::DOCUMENT)) {
-                throw new \Exception('Cannot process the ' . $field->key . ' media field. Currently no support for translatable media files. We should fix this!');
+                throw new \Exception('Cannot process the '.$field->key.' media field. Currently no support for translatable media files. We should fix this!');
             }
 
             // Okay so this is a bit odd but since all translations are expected to be inside the trans

Braces +3 added lines, -1 removed lines

@@ -167,7 +167,9 @@
         $fields = $this->fields();
 
         foreach($this->assistants() as $assistant){
-            if( ! method_exists($assistant, 'fields')) continue;
+            if( ! method_exists($assistant, 'fields')) {
+                continue;
+            }
 
             $fields = $fields->merge($assistant->fields());
         }

src/Fields/Fields.php

Indentation +1 added lines, -1 removed lines

@@ -47,7 +47,7 @@
     public function validate(array $data)
     {
         foreach($this->fields as $field){
-             $field->validator($data)->validate();
+                $field->validator($data)->validate();
         }
     }
 

Spacing +6 added lines, -6 removed lines

@@ -24,7 +24,7 @@ 
 
     public function first(): ?Field
     {
-        if(!$this->any()) return null;
+        if (!$this->any()) return null;
 
         return reset($this->fields);
     }
@@ -46,7 +46,7 @@ 
 
     public function validate(array $data)
     {
-        foreach($this->fields as $field){
+        foreach ($this->fields as $field) {
              $field->validator($data)->validate();
         }
     }
@@ -121,8 +121,8 @@ 
 
     public function offsetSet($offset, $value)
     {
-        if (! $value instanceof Field) {
-            throw new \InvalidArgumentException('Passed value must be of type ' . Field::class);
+        if (!$value instanceof Field) {
+            throw new \InvalidArgumentException('Passed value must be of type '.Field::class);
         }
 
         $this->fields[$offset] = $value;
@@ -142,7 +142,7 @@ 
     {
         $keyedFields = [];
 
-        foreach($fields as $field)
+        foreach ($fields as $field)
         {
             $keyedFields[$field->key] = $field;
         }
@@ -152,7 +152,7 @@ 
 
     private function validateFields(array $fields)
     {
-        array_map(function (Field $field) {
+        array_map(function(Field $field) {
         }, $fields);
     }
 

Braces +3 added lines, -1 removed lines

@@ -24,7 +24,9 @@
 
     public function first(): ?Field
     {
-        if(!$this->any()) return null;
+        if(!$this->any()) {
+            return null;
+        }
 
         return reset($this->fields);
     }