thinktomorrow / chief

src/ManagedModels/Application/DeleteModel.php

Indentation +1 added lines, -1 removed lines

@@ -39,7 +39,7 @@
             // so the user has a small window of recovery
 
             if($model instanceof HasAsset) {
-                 $this->detachAsset->detachAll($model);
+                    $this->detachAsset->detachAll($model);
             }
             // TODO: soft-delete the context of this model... (=fragments)
 //            Relation::deleteRelationsOf($model->getMorphClass(), $model->id);

Spacing +1 added lines, -1 removed lines

@@ -38,7 +38,7 @@
             // TODO: schedule for deletion instead of instantly deleting all relations and stuff...
             // so the user has a small window of recovery
 
-            if($model instanceof HasAsset) {
+            if ($model instanceof HasAsset) {
                  $this->detachAsset->detachAll($model);
             }
             // TODO: soft-delete the context of this model... (=fragments)

src/ManagedModels/Assistants/SavingFields.php

Braces +3 added lines, -1 removed lines

@@ -44,7 +44,9 @@
             // Dynamic localized values or standard translated
             // For standard translations we set value with the colon notation, e.g. title:en
             Form::foreachTrans(data_get($input, 'trans', []), function($locale, $key, $value) use($field){
-                if($key !== $field->getColumn()) return;
+                if($key !== $field->getColumn()) {
+                    return;
+                }
 
                 if($this->isFieldForDynamicValue($field)) {
                     $this->setDynamic($key, $value, $locale);

Spacing +13 added lines, -13 removed lines

@@ -31,12 +31,12 @@ 
     {
         [$input, $files] = $this->removeDuplicateFilePayload($input, $files);
 
-        foreach($fields as $field){
-            if($this->detectCustomSaveMethod($field)) {
+        foreach ($fields as $field) {
+            if ($this->detectCustomSaveMethod($field)) {
                 continue;
             }
 
-            if(!$field->isLocalized())
+            if (!$field->isLocalized())
             {
                 // Set standard non-localized attribute on the model
                 ($customSetMethod = $this->detectCustomSetMethod($field))
@@ -49,11 +49,11 @@ 
             // Dynamic localized values or standard translated
             // For standard translations we set value with the colon notation, e.g. title:en
             Form::foreachTrans(data_get($input, 'trans', []), function($locale, $key, $value) use($field){
-                if($key !== $field->getColumn()) return;
+                if ($key !== $field->getColumn()) return;
 
-                if($this->isFieldForDynamicValue($field)) {
+                if ($this->isFieldForDynamicValue($field)) {
                     $this->setDynamic($key, $value, $locale);
-                } else {
+                }else {
                     $this->{$field->getColumn().':'.$locale} = $value;
                 }
             });
@@ -62,8 +62,8 @@ 
         $this->save();
 
         // Custom save methods
-        foreach($fields as $field){
-            if($customSaveMethod = $this->detectCustomSaveMethod($field)) {
+        foreach ($fields as $field) {
+            if ($customSaveMethod = $this->detectCustomSaveMethod($field)) {
                 $this->$customSaveMethod($field, $input, $files);
             }
         }
@@ -76,8 +76,8 @@ 
 
     private function detectCustomSaveMethod(Field $field): ?string
     {
-        $saveMethodByKey = 'save' . ucfirst(Str::camel($field->getKey())) . 'Field';
-        $saveMethodByType = 'save' . ucfirst(Str::camel($field->getType()->get())) . 'Fields';
+        $saveMethodByKey = 'save'.ucfirst(Str::camel($field->getKey())).'Field';
+        $saveMethodByType = 'save'.ucfirst(Str::camel($field->getType()->get())).'Fields';
 
         foreach ([$saveMethodByKey, $saveMethodByType] as $saveMethod) {
             if (method_exists($this, $saveMethod)) {
@@ -90,7 +90,7 @@ 
 
     private function detectCustomSetMethod(Field $field): ?string
     {
-        $methodName = 'set' . ucfirst(Str::camel($field->getKey())) . 'Field';
+        $methodName = 'set'.ucfirst(Str::camel($field->getKey())).'Field';
 
         return (method_exists($this, $methodName)) ? $methodName : null;
     }
@@ -121,8 +121,8 @@ 
         $flatInput = Arr::dot($input);
         $flatFiles = Arr::dot($files);
 
-        foreach($flatInput as $key => $entry) {
-            if($this->isValidFile($entry) && array_key_exists($key, $flatFiles)) {
+        foreach ($flatInput as $key => $entry) {
+            if ($this->isValidFile($entry) && array_key_exists($key, $flatFiles)) {
                 Arr::forget($input, $key);
             }
         }

src/Site/Menu/ChiefMenu.php

Spacing +1 added lines, -1 removed lines

@@ -50,7 +50,7 @@
     public function sanitize(NodeCollection $collection): NodeCollection
     {
         if (!$this->includeHidden) {
-            $collection = $collection->shake(function ($node) {
+            $collection = $collection->shake(function($node) {
                 return !$node->hidden_in_menu && !$node->draft;
             });
         }

src/Site/Menu/Tree/PrepareMenuItemsForAdminSelect.php

Spacing +4 added lines, -4 removed lines

@@ -14,22 +14,22 @@
         $this->collection = $items;
 
         if ($model) {
-            $this->collection = $this->collection->prune(function ($node) use ($model) {
+            $this->collection = $this->collection->prune(function($node) use ($model) {
                 return !in_array($model->id, $node->pluckAncestors('id'));
             });
         }
 
-        $menu = $this->collection->mapRecursive(function ($node) {
+        $menu = $this->collection->mapRecursive(function($node) {
             $entry = $node->entry();
             $label = $entry->label;
-            $entry->label = $node->depth() != 0 ? (str_repeat('-', $node->depth())) . '>' : '';
+            $entry->label = $node->depth() != 0 ? (str_repeat('-', $node->depth())).'>' : '';
             $entry->label .= $label;
 
             return $node->replaceEntry($entry);
         });
 
         $menuitems = collect();
-        $menu->flatten()->each(function ($node) use ($menuitems) {
+        $menu->flatten()->each(function($node) use ($menuitems) {
             $menuitems[] = [
                 'label' => $node->label,
                 'id'    => $node->id,

src/Site/Menu/Tree/BuildMenuItemsTree.php

Spacing +2 added lines, -2 removed lines

@@ -27,7 +27,7 @@ 
             if ($item->ofType(MenuItem::TYPE_INTERNAL) && $owner = $item->owner) {
                 if (public_method_exists($owner, 'isArchived') && $owner->isArchived()) {
                     unset($items[$k]);
-                } else {
+                }else {
                     $item->url = $item->url();
 
                     // Extra info on admin page.
@@ -49,7 +49,7 @@ 
     {
         $collection = NodeCollection::fromArray($items);
 
-        $collection->mapRecursive(function ($node) {
+        $collection->mapRecursive(function($node) {
             return $node->replaceEntry((new MenuItem())->entry($node));
         });
 

src/Site/Menu/Application/UpdateMenu.php

Spacing +4 added lines, -4 removed lines

@@ -25,7 +25,7 @@ 
             $model->parent_id = ($request->input('allow_parent') && $request->input('parent_id')) ? $request->input('parent_id') : null;
             $model->order = $request->input('order', 0);
 
-            if($request->input('owner_reference')) {
+            if ($request->input('owner_reference')) {
                 $owner = ModelReference::fromString($request->input('owner_reference'));
                 $model->owner_type = $owner->className();
                 $model->owner_id = $owner->id();
@@ -58,13 +58,13 @@ 
         if (in_array($menu->order, $sequence)) {
             foreach ($sequence as $id => $order) {
                 if ($order < $menu->order) {
-                } else {
+                }else {
                     $sequence[$id]++;
                 }
             }
         }
 
-        $sequence = $sequence + [$menu->id => $menu->order];
+        $sequence = $sequence+[$menu->id => $menu->order];
 
         asort($sequence);
 
@@ -73,7 +73,7 @@ 
 
     private function reorder(array $sequence)
     {
-        array_walk($sequence, function ($order, $id) {
+        array_walk($sequence, function($order, $id) {
             MenuItem::withoutGlobalScope(SoftDeletingScope::class)
                 ->where('id', $id)
                 ->update(['order' => $order]);

src/Site/Menu/Application/CreateMenu.php

Spacing +1 added lines, -1 removed lines

@@ -26,7 +26,7 @@
             $model->parent_id = ($request->input('allow_parent') && $request->input('parent_id')) ? $request->input('parent_id') : null;
             $model->menu_type = $request->input('menu_type', 'main');
 
-            if($request->input('owner_reference')) {
+            if ($request->input('owner_reference')) {
                 $owner = ModelReference::fromString($request->input('owner_reference'));
                 $model->owner_type = $owner->className();
                 $model->owner_id = $owner->id();

src/Site/Menu/MenuItem.php

Spacing +2 added lines, -2 removed lines

@@ -70,7 +70,7 @@ 
 
     public function url($locale = null)
     {
-        if(!$locale) $locale = app()->getLocale();
+        if (!$locale) $locale = app()->getLocale();
 
         if ($this->ofType(static::TYPE_INTERNAL) && $owner = $this->owner) {
             return $owner->url($locale);
@@ -90,7 +90,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,
             'owner_type'     => $node->owner_type,

Braces +3 added lines, -1 removed lines

@@ -70,7 +70,9 @@
 
     public function url($locale = null)
     {
-        if(!$locale) $locale = app()->getLocale();
+        if(!$locale) {
+            $locale = app()->getLocale();
+        }
 
         if ($this->ofType(static::TYPE_INTERNAL) && $owner = $this->owner) {
             return $owner->url($locale);

src/Shared/Snippets/SnippetParser.php

Spacing +1 added lines, -1 removed lines

@@ -12,7 +12,7 @@
             return $value;
         }
 
-        $value = preg_replace_callback(static::$pattern, function ($matches) {
+        $value = preg_replace_callback(static::$pattern, function($matches) {
 
             // First entry of matches contains our full captured group, which we want to replace.
             // Second entry is the text itself, without the brackets