LaravelRUS / SleepingOwlAdmin

src/Form/Element/Upload.php

Spacing +1 added lines, -1 removed lines

@@ -31,7 +31,7 @@
 
         if (Request::input($this->getPath().'_remove')) {
             $this->setValue($this->getModel(), $this->getAttribute(), null);
-        } elseif (! is_null($value)) {
+        } elseif (!is_null($value)) {
             $this->setValue($this->getModel(), $this->getAttribute(), $value);
         }
     }

src/Providers/AliasesServiceProvider.php

Spacing +7 added lines, -7 removed lines

@@ -27,7 +27,7 @@ 
             'select' => \SleepingOwl\Admin\Display\Column\Filter\Select::class,
         ]);
 
-        $this->app->singleton('sleeping_owl.column_filter', function () use ($alias) {
+        $this->app->singleton('sleeping_owl.column_filter', function() use ($alias) {
             return $alias;
         });
     }
@@ -43,7 +43,7 @@ 
             'tree'            => \SleepingOwl\Admin\Display\DisplayTree::class,
         ]);
 
-        $this->app->singleton('sleeping_owl.display', function () use ($alias) {
+        $this->app->singleton('sleeping_owl.display', function() use ($alias) {
             return $alias;
         });
     }
@@ -68,7 +68,7 @@ 
             'treeControl' => \SleepingOwl\Admin\Display\Column\TreeControl::class,
         ]);
 
-        $this->app->singleton('sleeping_owl.table.column', function () use ($alias) {
+        $this->app->singleton('sleeping_owl.table.column', function() use ($alias) {
             return $alias;
         });
     }
@@ -79,7 +79,7 @@ 
             'checkbox'    => \SleepingOwl\Admin\Display\Column\Editable\Checkbox::class,
         ]);
 
-        $this->app->singleton('sleeping_owl.table.column.editable', function () use ($alias) {
+        $this->app->singleton('sleeping_owl.table.column.editable', function() use ($alias) {
             return $alias;
         });
     }
@@ -111,7 +111,7 @@ 
             'html'        => \SleepingOwl\Admin\Form\Element\Html::class,
         ]);
 
-        $this->app->singleton('sleeping_owl.form.element', function () use ($alias) {
+        $this->app->singleton('sleeping_owl.form.element', function() use ($alias) {
             return $alias;
         });
     }
@@ -124,7 +124,7 @@ 
             'panel'  => \SleepingOwl\Admin\Form\FormPanel::class,
         ]);
 
-        $this->app->singleton('sleeping_owl.form', function () use ($alias) {
+        $this->app->singleton('sleeping_owl.form', function() use ($alias) {
             return $alias;
         });
     }
@@ -138,7 +138,7 @@ 
             'related' => \SleepingOwl\Admin\Display\Filter\FilterRelated::class,
         ]);
 
-        $this->app->singleton('sleeping_owl.display.filter', function () use ($alias) {
+        $this->app->singleton('sleeping_owl.display.filter', function() use ($alias) {
             return $alias;
         });
     }

src/Display/Column/NamedColumn.php

Doc Comments +1 added lines, -1 removed lines

@@ -71,7 +71,7 @@
     /**
      * Get column value from instance.
      *
-     * @param Collection|Model|Closure $instance
+     * @param Model $instance
      * @param string           $name
      *
      * @return mixed

Spacing +4 added lines, -4 removed lines

@@ -73,8 +73,8 @@ 
      */
     public function setOrderable($orderable = true)
     {
-        if ($orderable !== false && ! $orderable instanceof OrderByClauseInterface) {
-            if (! is_string($orderable) && ! $orderable instanceof Closure) {
+        if ($orderable !== false && !$orderable instanceof OrderByClauseInterface) {
+            if (!is_string($orderable) && !$orderable instanceof Closure) {
                 $orderable = $this->getName();
             }
 
@@ -118,11 +118,11 @@ 
 
         if ($instance instanceof Collection) {
             $instance = $instance->pluck($part);
-        } elseif (! is_null($instance)) {
+        } elseif (!is_null($instance)) {
             $instance = $instance->getAttribute($part);
         }
 
-        if (! empty($parts) && ! is_null($instance)) {
+        if (!empty($parts) && !is_null($instance)) {
             return $this->getValueFromObject($instance, implode('.', $parts));
         }
 

src/Display/Display.php

Doc Comments +1 added lines, -1 removed lines

@@ -308,7 +308,7 @@
     }
 
     /**
-     * @return \Illuminate\Foundation\Application|mixed
+     * @return RepositoryInterface
      * @throws \Exception
      */
     protected function makeRepository()

Spacing +5 added lines, -5 removed lines

@@ -167,7 +167,7 @@ 
         $this->repository = $this->makeRepository();
         $this->repository->with($this->with);
 
-        $this->extensions->each(function (DisplayExtensionInterface $extension) {
+        $this->extensions->each(function(DisplayExtensionInterface $extension) {
             if ($extension instanceof Initializable) {
                 $extension->initialize();
             }
@@ -201,7 +201,7 @@ 
             $this->title,
         ];
 
-        $this->getExtensions()->each(function (DisplayExtensionInterface $extension) use (&$titles) {
+        $this->getExtensions()->each(function(DisplayExtensionInterface $extension) use (&$titles) {
             if (method_exists($extension, $method = 'getTitle')) {
                 $titles[] = call_user_func([$extension, $method]);
             }
@@ -267,7 +267,7 @@ 
 
         $blocks = [];
 
-        $placableExtensions = $this->getExtensions()->filter(function ($extension) {
+        $placableExtensions = $this->getExtensions()->filter(function($extension) {
             return $extension instanceof Placable;
         });
 
@@ -280,7 +280,7 @@ 
 
         foreach ($blocks as $block => $data) {
             foreach ($data as $html) {
-                if (! empty($html)) {
+                if (!empty($html)) {
                     $view->getFactory()->startSection($block);
                     echo $html;
                     $view->getFactory()->yieldSection();
@@ -338,7 +338,7 @@ 
     {
         $repository = app($this->repositoryClass, [$this->modelClass]);
 
-        if (! ($repository instanceof RepositoryInterface)) {
+        if (!($repository instanceof RepositoryInterface)) {
             throw new \Exception('Repository class must be instanced of [RepositoryInterface]');
         }
 

src/Display/Extension/Columns.php

Doc Comments +1 added lines, -1 removed lines

@@ -145,7 +145,7 @@
     }
 
     /**
-     * @param string|\Illuminate\View\View $view
+     * @param string $view
      *
      * @return $this
      */

Spacing +4 added lines, -4 removed lines

@@ -88,10 +88,10 @@ 
         $this->controlActive = false;
 
         if ($this->isInitialize()) {
-            $this->columns = $this->columns->filter(function ($column) {
+            $this->columns = $this->columns->filter(function($column) {
                 $class = get_class($this->getControlColumn());
 
-                return ! ($column instanceof $class);
+                return !($column instanceof $class);
             });
         }
 
@@ -113,7 +113,7 @@ 
      */
     public function set($columns)
     {
-        if (! is_array($columns)) {
+        if (!is_array($columns)) {
             $columns = func_get_args();
         }
 
@@ -179,7 +179,7 @@ 
 
     public function initialize()
     {
-        $this->all()->each(function (ColumnInterface $column) {
+        $this->all()->each(function(ColumnInterface $column) {
             $column->initialize();
         });
 

src/Navigation/Page.php

Doc Comments +1 added lines, -1 removed lines

@@ -83,7 +83,7 @@
     }
 
     /**
-     * @return Closure
+     * @return \Closure
      */
     public function getAccessLogic()
     {

Spacing +5 added lines, -5 removed lines

@@ -31,7 +31,7 @@ 
      */
     public function getModelConfiguration()
     {
-        if (! $this->hasModel()) {
+        if (!$this->hasModel()) {
             return;
         }
 
@@ -43,7 +43,7 @@ 
      */
     public function hasModel()
     {
-        return ! is_null($this->model) and class_exists($this->model);
+        return !is_null($this->model) and class_exists($this->model);
     }
 
     /**
@@ -87,9 +87,9 @@ 
      */
     public function getAccessLogic()
     {
-        if (! is_callable($this->accessLogic)) {
+        if (!is_callable($this->accessLogic)) {
             if ($this->hasModel()) {
-                return function () {
+                return function() {
                     return $this->getModelConfiguration()->isDisplayable();
                 };
             }
@@ -107,7 +107,7 @@ 
     {
         $data = $this->toArray();
 
-        if (! is_null($view)) {
+        if (!is_null($view)) {
             return view($view, $data)->render();
         }
 

src/Repository/TreeRepository.php

Doc Comments +7 added lines, -7 removed lines

@@ -203,7 +203,7 @@ 
      *
      * @param $id
      * @param $parentId
-     * @param $left
+     * @param integer $left
      * @param $right
      */
     protected function move($id, $parentId, $left, $right)
@@ -300,7 +300,7 @@ 
      * Call several methods and get first result.
      *
      * @param $instance
-     * @param $methods
+     * @param string[] $methods
      *
      * @return mixed
      * @throws Exception
@@ -320,9 +320,9 @@ 
      *
      * @param $root
      * @param $parentId
-     * @param $left
+     * @param integer $left
      *
-     * @return mixed
+     * @return integer
      */
     protected function recursiveReorder($root, $parentId, $left)
     {
@@ -341,7 +341,7 @@ 
      * Recursive reoder simple tree type.
      *
      * @param $data
-     * @param $parentId
+     * @param string|null $parentId
      */
     protected function recursiveReorderSimple(array $data, $parentId)
     {
@@ -363,7 +363,7 @@ 
      * Get children for simple tree type structure.
      *
      * @param $collection
-     * @param $id
+     * @param string|null $id
      *
      * @return Collection
      */
@@ -385,7 +385,7 @@ 
 
     /**
      * Create simple tree type structure.
-     * @return static
+     * @return Collection
      */
     protected function createSimpleTree()
     {

src/Traits/FormElements.php

Doc Comments +1 added lines, -1 removed lines

@@ -214,7 +214,7 @@
     /**
      * @param $object
      *
-     * @return mixed
+     * @return Model
      */
     protected function getElementContainer($object)
     {

Spacing +8 added lines, -8 removed lines

@@ -23,7 +23,7 @@ 
      */
     public function initializeElements()
     {
-        $this->getElements()->each(function ($element) {
+        $this->getElements()->each(function($element) {
             if ($element instanceof Initializable) {
                 $element->initialize();
             }
@@ -41,7 +41,7 @@ 
 
         foreach ($this->getElements() as $element) {
             if ($element instanceof ElementsInterface) {
-                if (! is_null($found = $element->getElement($path))) {
+                if (!is_null($found = $element->getElement($path))) {
                     return $found;
                 }
             }
@@ -135,7 +135,7 @@ 
      */
     protected function setModelForElements(Model $model)
     {
-        $this->getElements()->each(function ($element) use ($model) {
+        $this->getElements()->each(function($element) use ($model) {
             $element = $this->getElementContainer($element);
 
             if ($element instanceof WithModel) {
@@ -153,7 +153,7 @@ 
      */
     protected function getValidationRulesFromElements(array $rules = [])
     {
-        $this->getElements()->onlyActive()->each(function ($element) use (&$rules) {
+        $this->getElements()->onlyActive()->each(function($element) use (&$rules) {
             $element = $this->getElementContainer($element);
 
             if ($element instanceof Validable) {
@@ -171,7 +171,7 @@ 
      */
     protected function getValidationMessagesForElements(array $messages = [])
     {
-        $this->getElements()->onlyActive()->each(function ($element) use (&$messages) {
+        $this->getElements()->onlyActive()->each(function($element) use (&$messages) {
             $element = $this->getElementContainer($element);
 
             if ($element instanceof Validable) {
@@ -189,7 +189,7 @@ 
      */
     protected function getValidationLabelsForElements(array $labels = [])
     {
-        $this->getElements()->onlyActive()->each(function ($element) use (&$labels) {
+        $this->getElements()->onlyActive()->each(function($element) use (&$labels) {
             $element = $this->getElementContainer($element);
 
             if ($element instanceof Validable) {
@@ -202,7 +202,7 @@ 
 
     protected function saveElements()
     {
-        $this->getElements()->onlyActive()->each(function ($element) {
+        $this->getElements()->onlyActive()->each(function($element) {
             $element = $this->getElementContainer($element);
 
             if ($element instanceof FormElementInterface) {
@@ -213,7 +213,7 @@ 
 
     protected function afterSaveElements()
     {
-        $this->getElements()->onlyActive()->each(function ($element) {
+        $this->getElements()->onlyActive()->each(function($element) {
             $element = $this->getElementContainer($element);
 
             if ($element instanceof FormElementInterface) {

src/Display/Filter/FilterField.php

Spacing +2 added lines, -2 removed lines

@@ -38,8 +38,8 @@
             $relationName = implode('.', $parts);
         }
 
-        if (! is_null($relationName)) {
-            $query->whereHas($relationName, function ($q) use ($name, $value) {
+        if (!is_null($relationName)) {
+            $query->whereHas($relationName, function($q) use ($name, $value) {
                 $this->buildQuery($q, $name, $value);
             });
         } else {