LaravelRUS / SleepingOwlAdmin

src/Console/Commands/UpdateCommand.php

Spacing +4 added lines, -4 removed lines

@@ -24,12 +24,12 @@
         collect([
             Installation\PublishAssets::class,
         ])
-            ->map(function ($installer) {
+            ->map(function($installer) {
                 return new $installer($this, $this->config);
             })
-            ->filter(function ($installer) {
-                return $this->option('force') ? true : ! $installer->installed();
-            })->each(function ($installer) {
+            ->filter(function($installer) {
+                return $this->option('force') ? true : !$installer->installed();
+            })->each(function($installer) {
                 $installer->install();
                 $installer->showInfo();
             });

resources/views/default/display/extensions/columns.blade.php

Spacing +1 added lines, -1 removed lines

@@ -22,7 +22,7 @@
             @foreach ($columns as $column)
                 <?php
                 $column->setModel($model);
-                if($column instanceof \SleepingOwl\Admin\Display\Column\Control) {
+                if ($column instanceof \SleepingOwl\Admin\Display\Column\Control) {
                     $column->initialize();
                 }
                 ?>

src/Display/DisplayTabbed.php

Doc Comments +1 added lines, -1 removed lines

@@ -65,7 +65,7 @@
     }
 
     /**
-     * @return TabInterface[]|DisplayTabsCollection
+     * @return \SleepingOwl\Admin\Form\FormElementsCollection
      */
     public function getTabs()
     {

Spacing +6 added lines, -6 removed lines

@@ -43,7 +43,7 @@ 
     {
         $this->initializeElements();
 
-        $activeTabs = $this->getTabs()->filter(function (TabInterface $tab) {
+        $activeTabs = $this->getTabs()->filter(function(TabInterface $tab) {
             return $tab->isActive();
         })->count();
 
@@ -57,7 +57,7 @@ 
      */
     public function setModelClass($class)
     {
-        $this->getTabs()->each(function (TabInterface $tab) use ($class) {
+        $this->getTabs()->each(function(TabInterface $tab) use ($class) {
             if ($tab instanceof DisplayInterface) {
                 $tab->setModelClass($class);
             }
@@ -137,7 +137,7 @@ 
      */
     public function setAction($action)
     {
-        $this->getTabs()->each(function (TabInterface $tab) use ($action) {
+        $this->getTabs()->each(function(TabInterface $tab) use ($action) {
             if ($tab instanceof FormInterface) {
                 $tab->setAction($action);
             }
@@ -149,7 +149,7 @@ 
      */
     public function setId($id)
     {
-        $this->getTabs()->each(function (TabInterface $tab) use ($id) {
+        $this->getTabs()->each(function(TabInterface $tab) use ($id) {
             if ($tab instanceof FormInterface) {
                 $tab->setId($id);
             }
@@ -163,7 +163,7 @@ 
      */
     public function validateForm(ModelConfigurationInterface $model)
     {
-        $this->getTabs()->each(function ($tab) use ($model) {
+        $this->getTabs()->each(function($tab) use ($model) {
             if ($tab instanceof FormInterface) {
                 $tab->validateForm($model);
             }
@@ -175,7 +175,7 @@ 
      */
     public function saveForm(ModelConfigurationInterface $model)
     {
-        $this->getTabs()->each(function (TabInterface $tab) use ($model) {
+        $this->getTabs()->each(function(TabInterface $tab) use ($model) {
             if ($tab instanceof FormInterface) {
                 $tab->saveForm($model);
             }

src/Form/FormDefault.php

Doc Comments +2 added lines, -2 removed lines

@@ -213,7 +213,7 @@ 
      * @deprecated 4.5.0
      * @see getElements()
      *
-     * @return Collection[]
+     * @return FormElementsCollection
      */
     public function getItems()
     {
@@ -301,7 +301,7 @@ 
      *
      * @param ModelConfigurationInterface $modelConfiguration
      *
-     * @return bool
+     * @return null|boolean
      */
     public function saveForm(ModelConfigurationInterface $modelConfiguration)
     {

Spacing +7 added lines, -7 removed lines

@@ -101,12 +101,12 @@ 
 
         parent::initialize();
 
-        if (! $this->hasHtmlAttribute('enctype')) {
+        if (!$this->hasHtmlAttribute('enctype')) {
             // Try to find upload element
-            $this->getElements()->each(function ($element) {
+            $this->getElements()->each(function($element) {
 
                 // TODO: this not works withs nested elements
-                if ($element instanceof Upload and ! $this->hasHtmlAttribute('enctype')) {
+                if ($element instanceof Upload and !$this->hasHtmlAttribute('enctype')) {
                     $this->setHtmlAttribute('enctype', 'multipart/form-data');
                 }
             });
@@ -229,7 +229,7 @@ 
      */
     public function setItems($items)
     {
-        if (! is_array($items)) {
+        if (!is_array($items)) {
             $items = func_get_args();
         }
 
@@ -256,7 +256,7 @@ 
      */
     public function setId($id)
     {
-        if (is_null($this->id) and ! is_null($id) and ($model = $this->getRepository()->find($id))) {
+        if (is_null($this->id) and !is_null($id) and ($model = $this->getRepository()->find($id))) {
             $this->id = $id;
 
             $this->setModel($model);
@@ -342,7 +342,7 @@ 
     protected function saveBelongsToRelations(Model $model)
     {
         foreach ($model->getRelations() as $name => $relation) {
-            if ($model->{$name}() instanceof BelongsTo && ! is_null($relation)) {
+            if ($model->{$name}() instanceof BelongsTo && !is_null($relation)) {
                 $relation->save();
                 $model->{$name}()->associate($relation);
             }
@@ -357,7 +357,7 @@ 
     protected function saveHasOneRelations(Model $model)
     {
         foreach ($model->getRelations() as $name => $relation) {
-            if ($model->{$name}() instanceof HasOneOrMany && ! is_null($relation)) {
+            if ($model->{$name}() instanceof HasOneOrMany && !is_null($relation)) {
                 if (is_array($relation) || $relation instanceof \Traversable) {
                     $model->{$name}()->saveMany($relation);
                 } else {

src/Form/FormElementsCollection.php

Spacing +3 added lines, -3 removed lines

@@ -12,9 +12,9 @@ 
      */
     public function onlyActive()
     {
-        return $this->filter(function ($element) {
+        return $this->filter(function($element) {
             if ($element instanceof FormElementInterface) {
-                return ! $element->isReadonly();
+                return !$element->isReadonly();
             }
 
             return true;
@@ -26,7 +26,7 @@ 
      */
     public function onlyVisible()
     {
-        return $this->filter(function ($element) {
+        return $this->filter(function($element) {
             if ($element instanceof FormElementInterface) {
                 return $element->isVisible();
             }

src/Display/DisplayTabsCollection.php

Spacing +3 added lines, -3 removed lines

@@ -13,11 +13,11 @@ 
      */
     public function onlyActive()
     {
-        return $this->filter(function (TabInterface $tab) {
+        return $this->filter(function(TabInterface $tab) {
             $element = $tab->getContent();
 
             if ($element instanceof FormElementInterface) {
-                return ! $element->isReadonly();
+                return !$element->isReadonly();
             }
 
             return true;
@@ -29,7 +29,7 @@ 
      */
     public function onlyVisible()
     {
-        return $this->filter(function (TabInterface $tab) {
+        return $this->filter(function(TabInterface $tab) {
             return $tab->isVisible();
         });
     }

src/Console/Commands/InstallCommand.php

Spacing +4 added lines, -4 removed lines

@@ -24,12 +24,12 @@
         collect([
             Installation\PublishAssets::class,
         ])
-            ->map(function ($installer) {
+            ->map(function($installer) {
                 return new $installer($this, $this->config);
             })
-            ->filter(function ($installer) {
-                return $this->option('force') ? true : ! $installer->installed();
-            })->each(function ($installer) {
+            ->filter(function($installer) {
+                return $this->option('force') ? true : !$installer->installed();
+            })->each(function($installer) {
                 $installer->install();
                 $installer->showInfo();
             });

src/Http/Controllers/AdminController.php

Doc Comments +3 added lines, -3 removed lines

@@ -305,7 +305,7 @@ 
      *
      * @param Request $request
      *
-     * @return bool
+     * @return boolean|null
      *
      * @throws \Symfony\Component\HttpKernel\Exception\NotFoundHttpException
      */
@@ -410,7 +410,7 @@ 
     }
 
     /**
-     * @param ModelConfigurationInterface|ModelConfiguration $model
+     * @param ModelConfigurationInterface $model
      * @param Request $request
      * @param int $id
      *
@@ -529,7 +529,7 @@ 
     /**
      * @param Request $request
      *
-     * @return null|string
+     * @return string
      */
     protected function getBackUrl(Request $request)
     {

Unused Use Statements -1 removed lines

@@ -8,7 +8,6 @@
 use Illuminate\Http\Response;
 use Illuminate\Routing\Controller;
 use Illuminate\Contracts\Support\Renderable;
-use Illuminate\Contracts\Validation\Validator;
 use Illuminate\Validation\ValidationException;
 use SleepingOwl\Admin\Contracts\FormInterface;
 use SleepingOwl\Admin\Contracts\ModelConfigurationInterface;

Spacing +18 added lines, -18 removed lines

@@ -43,8 +43,8 @@ 
         $this->navigation = $application['sleeping_owl.navigation'];
         $this->navigation->setCurrentUrl($request->url());
 
-        if (! Breadcrumbs::exists('home')) {
-            Breadcrumbs::register('home', function ($breadcrumbs) {
+        if (!Breadcrumbs::exists('home')) {
+            Breadcrumbs::register('home', function($breadcrumbs) {
                 $breadcrumbs->push(trans('sleeping_owl::lang.dashboard'), route('admin.dashboard'));
             });
         }
@@ -65,8 +65,8 @@ 
         }
 
         foreach ($breadcrumbs as  $breadcrumb) {
-            if (! Breadcrumbs::exists($breadcrumb['id'])) {
-                Breadcrumbs::register($breadcrumb['id'], function ($breadcrumbs) use ($breadcrumb) {
+            if (!Breadcrumbs::exists($breadcrumb['id'])) {
+                Breadcrumbs::register($breadcrumb['id'], function($breadcrumbs) use ($breadcrumb) {
                     $breadcrumbs->parent($breadcrumb['parent']);
                     $breadcrumbs->push($breadcrumb['title'], $breadcrumb['url']);
                 });
@@ -110,7 +110,7 @@ 
      */
     public function getDisplay(ModelConfigurationInterface $model)
     {
-        if (! $model->isDisplayable()) {
+        if (!$model->isDisplayable()) {
             abort(404);
         }
 
@@ -126,7 +126,7 @@ 
      */
     public function getCreate(ModelConfigurationInterface $model)
     {
-        if (! $model->isCreatable()) {
+        if (!$model->isCreatable()) {
             abort(404);
         }
 
@@ -146,7 +146,7 @@ 
      */
     public function postStore(ModelConfigurationInterface $model, Request $request)
     {
-        if (! $model->isCreatable()) {
+        if (!$model->isCreatable()) {
             abort(404);
         }
 
@@ -182,7 +182,7 @@ 
             $redirectPolicy = $model->getRedirect();
 
             /* Make redirect when use in model config && Fix editable redirect */
-            if ($redirectPolicy->get('create') == 'display' || ! $model->isEditable($newModel)) {
+            if ($redirectPolicy->get('create') == 'display' || !$model->isEditable($newModel)) {
                 $redirectUrl = $model->getDisplayUrl();
             }
 
@@ -219,7 +219,7 @@ 
     {
         $item = $model->getRepository()->find($id);
 
-        if (is_null($item) || ! $model->isEditable($item)) {
+        if (is_null($item) || !$model->isEditable($item)) {
             abort(404);
         }
 
@@ -243,7 +243,7 @@ 
         $editForm = $model->fireEdit($id);
         $item = $editForm->getModel();
 
-        if (is_null($item) || ! $model->isEditable($item)) {
+        if (is_null($item) || !$model->isEditable($item)) {
             abort(404);
         }
 
@@ -318,7 +318,7 @@ 
         $display = $model->fireDisplay();
 
         /** @var ColumnEditableInterface|null $column */
-        $column = $display->getColumns()->all()->filter(function ($column) use ($field) {
+        $column = $display->getColumns()->all()->filter(function($column) use ($field) {
             return ($column instanceof ColumnEditableInterface) and $field == $column->getName();
         })->first();
 
@@ -329,7 +329,7 @@ 
         $repository = $model->getRepository();
         $item = $repository->find($id);
 
-        if (is_null($item) || ! $model->isEditable($item)) {
+        if (is_null($item) || !$model->isEditable($item)) {
             abort(404);
         }
 
@@ -356,7 +356,7 @@ 
     {
         $item = $model->getRepository()->find($id);
 
-        if (is_null($item) || ! $model->isDeletable($item)) {
+        if (is_null($item) || !$model->isDeletable($item)) {
             abort(404);
         }
 
@@ -385,13 +385,13 @@ 
      */
     public function deleteDestroy(ModelConfigurationInterface $model, Request $request, $id)
     {
-        if (! $model->isRestorableModel()) {
+        if (!$model->isRestorableModel()) {
             abort(404);
         }
 
         $item = $model->getRepository()->findOnlyTrashed($id);
 
-        if (is_null($item) || ! $model->isRestorable($item)) {
+        if (is_null($item) || !$model->isRestorable($item)) {
             abort(404);
         }
 
@@ -420,13 +420,13 @@ 
      */
     public function postRestore(ModelConfigurationInterface $model, Request $request, $id)
     {
-        if (! $model->isRestorableModel()) {
+        if (!$model->isRestorableModel()) {
             abort(404);
         }
 
         $item = $model->getRepository()->findOnlyTrashed($id);
 
-        if (is_null($item) || ! $model->isRestorable($item)) {
+        if (is_null($item) || !$model->isRestorable($item)) {
             abort(404);
         }
 
@@ -552,7 +552,7 @@ 
      */
     protected function registerBreadcrumb($title, $parent)
     {
-        Breadcrumbs::register('render', function ($breadcrumbs) use ($title, $parent) {
+        Breadcrumbs::register('render', function($breadcrumbs) use ($title, $parent) {
             $breadcrumbs->parent($parent);
             $breadcrumbs->push($title);
         });

src/Model/ModelConfigurationManager.php

Doc Comments +1 added lines, -2 removed lines

@@ -327,7 +327,7 @@ 
     }
 
     /**
-     * @return null|string
+     * @return boolean
      */
     public function hasCustomControllerClass()
     {
@@ -505,7 +505,6 @@ 
      * @param string $event
      * @param bool $halt
      * @param Model|null $model
-     * @param array $args
      *
      * @return mixed
      */

Spacing +5 added lines, -5 removed lines

@@ -110,7 +110,7 @@ 
 
         $this->repository = app(RepositoryInterface::class, [$class]);
 
-        if (! $this->alias) {
+        if (!$this->alias) {
             $this->setDefaultAlias();
         }
     }
@@ -299,7 +299,7 @@ 
      */
     public function can($action, Model $model)
     {
-        if (! $this->checkAccess) {
+        if (!$this->checkAccess) {
             return true;
         }
 
@@ -331,7 +331,7 @@ 
      */
     public function hasCustomControllerClass()
     {
-        return ! is_null($controller = $this->getControllerClass()) and class_exists($controller);
+        return !is_null($controller = $this->getControllerClass()) and class_exists($controller);
     }
 
     /**
@@ -468,7 +468,7 @@ 
         $page->setPriority($priority);
 
         if ($badge) {
-            if (! ($badge instanceof BadgeInterface)) {
+            if (!($badge instanceof BadgeInterface)) {
                 $badge = new Badge($badge);
             }
 
@@ -511,7 +511,7 @@ 
      */
     public function fireEvent($event, $halt = true, Model $model = null, ...$payload)
     {
-        if (! isset(self::$dispatcher)) {
+        if (!isset(self::$dispatcher)) {
             return true;
         }