thinktomorrow / chief

src/Concerns/Viewable/Viewable.php

Spacing +2 added lines, -2 removed lines

@@ -36,7 +36,7 @@ 
 
         // If no view has been created for this model, we try once again to fetch the content value if any. This will silently fail
         // if no content value is present. We consider the 'content' attribute to be a default for our copy.
-        return isset($this->content) ? (string) $this->content : '';
+        return isset($this->content) ? (string)$this->content : '';
     }
 
     public function setViewParent(ActsAsParent $parent): ViewableContract
@@ -64,7 +64,7 @@ 
         }
 
         if (config('thinktomorrow.chief.strict')) {
-            throw new NotFoundViewKey('Missing view key. Please add a [viewKey] property to ' . get_class($this));
+            throw new NotFoundViewKey('Missing view key. Please add a [viewKey] property to '.get_class($this));
         };
 
         return '';

src/Management/Application/StoringAndUpdatingFields.php

Spacing +3 added lines, -3 removed lines

@@ -26,7 +26,7 @@ 
 
             // 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;
                 }
@@ -36,7 +36,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);
@@ -48,7 +48,7 @@ 
 
     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) {

src/Management/Assistants/AssistedManager.php

Spacing +3 added lines, -3 removed lines

@@ -17,7 +17,7 @@ 
      */
     public function isAssistedBy(string $assistant): bool
     {
-        return !! $this->getAssistantClass($assistant);
+        return !!$this->getAssistantClass($assistant);
     }
 
     public function assistants(): array
@@ -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

@@ -23,10 +23,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();
@@ -54,14 +54,14 @@ 
             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), ],
                     ],
                     [],
                     [
                         '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 +6 added lines, -6 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);
         }
 
@@ -167,7 +167,7 @@ 
         $fields = $this->fields();
 
         foreach ($this->assistants() as $assistant) {
-            if (! method_exists($assistant, 'fields')) {
+            if (!method_exists($assistant, 'fields')) {
                 continue;
             }
 
@@ -206,7 +206,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);
@@ -244,7 +244,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

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 .= $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/Setup/Setup.php

Spacing +2 added lines, -2 removed lines

@@ -6,14 +6,14 @@
 {
     public function run()
     {
-        foreach ($this->tasks() as $task){
+        foreach ($this->tasks() as $task) {
             $this->runTask($task);
         }
     }
 
     private function runTask(Task $task)
     {
-        if($task->check()) return;
+        if ($task->check()) return;
 
         $task->run();
     }

Braces +3 added lines, -1 removed lines

@@ -13,7 +13,9 @@
 
     private function runTask(Task $task)
     {
-        if($task->check()) return;
+        if($task->check()) {
+            return;
+        }
 
         $task->run();
     }

src/Authorization/Console/GenerateRole.php

Spacing +2 added lines, -2 removed lines

@@ -42,7 +42,7 @@ 
             // Generate all permissions if only scope is passed
             if (false === strpos($permissionName, '-')) {
                 $cleanPermissionNames = array_merge($cleanPermissionNames, Permission::generate($permissionName));
-            } else {
+            }else {
                 // Trim the value
                 $cleanPermissionNames[] = $permissionName;
             }
@@ -55,6 +55,6 @@ 
             $role->givePermissionTo($cleanPermissionName);
         }
 
-        $this->info('Role ' . $role->name . ' was assigned the permissions: ' . implode(',', $cleanPermissionNames));
+        $this->info('Role '.$role->name.' was assigned the permissions: '.implode(',', $cleanPermissionNames));
     }
 }

src/Authorization/Console/CreateAdmin.php

Spacing +4 added lines, -4 removed lines

@@ -33,22 +33,22 @@
 
         if ($this->option('dev')) {
             $role = 'developer';
-        } else {
+        }else {
             $role = 'admin';
         }
         
         $this->createUser($firstname, $lastname, $email, $password, [$role]);
         
-        $this->info($firstname.' '.$lastname. ' succesfully added as admin user.');
+        $this->info($firstname.' '.$lastname.' succesfully added as admin user.');
     }
 
     private function settingPermissionsAndRoles()
     {
-        AuthorizationDefaults::permissions()->each(function ($permissionName) {
+        AuthorizationDefaults::permissions()->each(function($permissionName) {
             Artisan::call('chief:permission', ['name' => $permissionName]);
         });
 
-        AuthorizationDefaults::roles()->each(function ($defaultPermissions, $roleName) {
+        AuthorizationDefaults::roles()->each(function($defaultPermissions, $roleName) {
             Artisan::call('chief:role', ['name' => $roleName, '--permissions' => implode(',', $defaultPermissions)]);
         });