thinktomorrow / chief

src/ManagedModels/Application/DuplicateContext.php

Spacing +2 added lines, -2 removed lines

@@ -22,12 +22,12 @@
 
     public function handle($sourceModel, $targetModel): void
     {
-        if(!$sourceModel instanceof FragmentsOwner || !$context = ContextModel::ownedBy($sourceModel)) return;
+        if (!$sourceModel instanceof FragmentsOwner || !$context = ContextModel::ownedBy($sourceModel)) return;
 
         $copiedContext = ContextModel::createForOwner($targetModel);
 
         /** @var FragmentModel $fragment */
-        foreach($context->fragments as $fragment) {
+        foreach ($context->fragments as $fragment) {
             $copiedFragment = $fragment->replicate(['context_id']);
             $copiedFragment->id = $this->fragmentRepository->nextId();
             $copiedFragment->context_id = $copiedContext->id;

Braces +3 added lines, -1 removed lines

@@ -22,7 +22,9 @@
 
     public function handle($sourceModel, $targetModel): void
     {
-        if(!$sourceModel instanceof FragmentsOwner || !$context = ContextModel::ownedBy($sourceModel)) return;
+        if(!$sourceModel instanceof FragmentsOwner || !$context = ContextModel::ownedBy($sourceModel)) {
+            return;
+        }
 
         $copiedContext = ContextModel::createForOwner($targetModel);
 

src/Managers/Assistants/FragmentAssistant.php

Spacing +7 added lines, -7 removed lines

@@ -31,18 +31,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));
@@ -52,12 +52,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)
@@ -144,13 +144,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(DeleteModel::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/Shared/ModelReferences/ModelReference.php

Spacing +6 added lines, -6 removed lines

@@ -24,16 +24,16 @@ 
     public static function fromString(string $reference): self
     {
         if (false == strpos($reference, '@')) {
-            throw new \InvalidArgumentException('Invalid reference composition. A model reference should consist of schema <class>@<id>. [' . $reference . '] was passed instead.');
+            throw new \InvalidArgumentException('Invalid reference composition. A model reference should consist of schema <class>@<id>. ['.$reference.'] was passed instead.');
         }
 
         list($className, $id) = explode('@', $reference);
 
-        if("" === $id) {
-            throw new \InvalidArgumentException('Missing id on model reference. [' . $reference . '] was passed.');
+        if ("" === $id) {
+            throw new \InvalidArgumentException('Missing id on model reference. ['.$reference.'] was passed.');
         }
 
-        return new static($className, (int) $id);
+        return new static($className, (int)$id);
     }
 
     /**
@@ -45,7 +45,7 @@ 
     {
         $className = $this->className();
 
-        if($this->id == 0) {
+        if ($this->id == 0) {
             return new $className($attributes);
         }
 
@@ -67,7 +67,7 @@ 
 
     public function get(): string
     {
-        return $this->className . '@' . $this->id;
+        return $this->className.'@'.$this->id;
     }
 
     public function equals($other): bool

src/Shared/Helpers/Form.php

Spacing +2 added lines, -2 removed lines

@@ -13,8 +13,8 @@
      */
     public static function foreachTrans(array $translations, \Closure $callback)
     {
-        foreach($translations as $locale => $trans){
-            foreach($trans as $key => $value){
+        foreach ($translations as $locale => $trans) {
+            foreach ($trans as $key => $value) {
                 call_user_func($callback, $locale, $key, $value);
             }
         }

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

Spacing +12 added lines, -12 removed lines

@@ -26,12 +26,12 @@ 
      */
     public function saveFields(Fields $fields, array $input, array $files): void
     {
-        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))
@@ -44,11 +44,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;
                 }
             });
@@ -57,8 +57,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);
             }
         }
@@ -71,8 +71,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)) {
@@ -85,7 +85,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;
     }
@@ -107,6 +107,6 @@ 
 
     private function saveImageFields(ImageField $field, array $input)
     {
-        app(ImageFieldHandler::class)->handle($this, $field, data_get($input, 'images.' . $field->getName(), []), $input);
+        app(ImageFieldHandler::class)->handle($this, $field, data_get($input, 'images.'.$field->getName(), []), $input);
     }
 }

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);

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));
         });