LaravelRUS / SleepingOwlAdmin

src/Display/Column/DateTime.php

Spacing +2 added lines, -2 removed lines

@@ -74,8 +74,8 @@
      */
     protected function getFormatedDate($date)
     {
-        if (! is_null($date)) {
-            if (! $date instanceof Carbon) {
+        if (!is_null($date)) {
+            if (!$date instanceof Carbon) {
                 $date = Carbon::parse($date);
             }
 

src/Display/Extension/Columns.php

Spacing +6 added lines, -6 removed lines

@@ -91,10 +91,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);
             });
         }
 
@@ -116,7 +116,7 @@ 
      */
     public function set($columns)
     {
-        if (! is_array($columns)) {
+        if (!is_array($columns)) {
             $columns = func_get_args();
         }
 
@@ -162,7 +162,7 @@ 
 
     public function initialize()
     {
-        $this->all()->each(function (ColumnInterface $column) {
+        $this->all()->each(function(ColumnInterface $column) {
             $column->initialize();
         });
 
@@ -204,7 +204,7 @@ 
 
         $columns = $this->all();
 
-        if (! is_int(key($orders))) {
+        if (!is_int(key($orders))) {
             $orders = [$orders];
         }
 
@@ -212,7 +212,7 @@ 
             $columnIndex = array_get($order, 'column');
             $direction = array_get($order, 'dir', 'asc');
 
-            if (! $columnIndex && $columnIndex !== '0') {
+            if (!$columnIndex && $columnIndex !== '0') {
                 continue;
             }
 

src/Display/Extension/ColumnFilters.php

Spacing +4 added lines, -4 removed lines

@@ -35,7 +35,7 @@ 
      */
     public function set($columnFilters)
     {
-        if (! is_array($columnFilters)) {
+        if (!is_array($columnFilters)) {
             $columnFilters = func_get_args();
         }
 
@@ -173,13 +173,13 @@ 
 
         $display = $this->getDisplay();
 
-        if (! $display->getExtensions()->has('columns')) {
+        if (!$display->getExtensions()->has('columns')) {
             return;
         }
 
         $columns = $display->getColumns()->all();
 
-        if (! is_int(key($search))) {
+        if (!is_int(key($search))) {
             $search = [$search];
         }
 
@@ -216,7 +216,7 @@ 
 
     protected function prepareView()
     {
-        if (! in_array($this->getPlacement(), ['table.footer', 'table.header']) && $this->view == 'display.extensions.columns_filters_table') {
+        if (!in_array($this->getPlacement(), ['table.footer', 'table.header']) && $this->view == 'display.extensions.columns_filters_table') {
             $this->view = 'display.extensions.columns_filters';
             $this->setHtmlAttribute('class', 'table table-default');
         }

src/Display/ExtensionCollection.php

Spacing +5 added lines, -5 removed lines

@@ -15,7 +15,7 @@ 
      */
     public function placable()
     {
-        return $this->filter(function (DisplayExtensionInterface $extension) {
+        return $this->filter(function(DisplayExtensionInterface $extension) {
             return $extension instanceof Placable;
         });
     }
@@ -42,7 +42,7 @@ 
      */
     public function renderable()
     {
-        return $this->filter(function (DisplayExtensionInterface $extension) {
+        return $this->filter(function(DisplayExtensionInterface $extension) {
             return $extension instanceof Renderable;
         });
     }
@@ -52,7 +52,7 @@ 
      */
     public function sortByOrder()
     {
-        return $this->sortBy(function (DisplayExtensionInterface $extension) {
+        return $this->sortBy(function(DisplayExtensionInterface $extension) {
             return $extension->getOrder();
         });
     }
@@ -62,7 +62,7 @@ 
      */
     public function initialize()
     {
-        $this->each(function (DisplayExtensionInterface $extension) {
+        $this->each(function(DisplayExtensionInterface $extension) {
             if ($extension instanceof Initializable) {
                 $extension->initialize();
             }
@@ -76,7 +76,7 @@ 
      */
     public function modifyQuery(\Illuminate\Database\Eloquent\Builder $query)
     {
-        $this->each(function (DisplayExtensionInterface $extension) use ($query) {
+        $this->each(function(DisplayExtensionInterface $extension) use ($query) {
             $extension->modifyQuery($query);
         });
 

src/Display/DisplayDatatablesAsync.php

Doc Comments +3 added lines, -3 removed lines

@@ -118,7 +118,7 @@ 
     }
 
     /**
-     * @param mixed $distinct
+     * @param string|null $distinct
      *
      * @return $this
      */
@@ -162,7 +162,7 @@ 
     /**
      * Apply offset and limit to the query.
      *
-     * @param $query
+     * @param Builder $query
      * @param \Illuminate\Http\Request $request
      */
     protected function applyOffset($query, \Illuminate\Http\Request $request)
@@ -204,7 +204,7 @@ 
      * Convert collection to the datatables structure.
      *
      * @param \Illuminate\Http\Request $request
-     * @param array|Collection $collection
+     * @param Collection $collection
      * @param int $totalCount
      * @param int $filteredCount
      *

Spacing +4 added lines, -4 removed lines

@@ -26,7 +26,7 @@ 
     public static function registerRoutes(Router $router)
     {
         $routeName = 'admin.display.async';
-        if (! $router->has($routeName)) {
+        if (!$router->has($routeName)) {
             $router->get('{adminModel}/async/{adminDisplayName?}', [
                 'as'   => $routeName,
                 'uses' => 'SleepingOwl\Admin\Http\Controllers\DisplayController@async',
@@ -34,7 +34,7 @@ 
         }
 
         $routeName = 'admin.display.async.inlineEdit';
-        if (! $router->has($routeName)) {
+        if (!$router->has($routeName)) {
             $router->post('{adminModel}/async/{adminDisplayName?}', [
                 'as'   => $routeName,
                 'uses' => 'SleepingOwl\Admin\Http\Controllers\AdminController@inlineEdit',
@@ -203,7 +203,7 @@ 
         $totalCount = $query->count();
         $filteredCount = 0;
 
-        if (! is_null($this->distinct)) {
+        if (!is_null($this->distinct)) {
             $filteredCount = $query->distinct()->count($this->getDistinct());
         }
 
@@ -253,7 +253,7 @@ 
             return;
         }
 
-        $query->where(function (Builder $query) use ($search) {
+        $query->where(function(Builder $query) use ($search) {
             $columns = $this->getColumns()->all();
 
             foreach ($columns as $column) {

src/Http/Controllers/AdminController.php

Doc Comments +2 added lines, -2 removed lines

@@ -311,7 +311,7 @@ 
      * @param ModelConfigurationInterface $model
      * @param Request $request
      *
-     * @return bool
+     * @return boolean|null
      *
      * @throws \Symfony\Component\HttpKernel\Exception\NotFoundHttpException
      */
@@ -534,7 +534,7 @@ 
     /**
      * @param Request $request
      *
-     * @return null|string
+     * @return string
      */
     protected function getBackUrl(Request $request)
     {

Spacing +21 added lines, -21 removed lines

@@ -60,8 +60,8 @@ 
 
         $admin->navigation()->setCurrentUrl($request->getUri());
 
-        if (! $this->breadcrumbs->exists('home')) {
-            $this->breadcrumbs->register('home', function (Generator $breadcrumbs) {
+        if (!$this->breadcrumbs->exists('home')) {
+            $this->breadcrumbs->register('home', function(Generator $breadcrumbs) {
                 $breadcrumbs->push(trans('sleeping_owl::lang.dashboard'), route('admin.dashboard'));
             });
         }
@@ -118,7 +118,7 @@ 
      */
     public function getDisplay(ModelConfigurationInterface $model)
     {
-        if (! $model->isDisplayable()) {
+        if (!$model->isDisplayable()) {
             abort(404);
         }
 
@@ -138,7 +138,7 @@ 
      */
     public function getCreate(ModelConfigurationInterface $model)
     {
-        if (! $model->isCreatable()) {
+        if (!$model->isCreatable()) {
             abort(404);
         }
 
@@ -158,7 +158,7 @@ 
      */
     public function postStore(ModelConfigurationInterface $model, Request $request)
     {
-        if (! $model->isCreatable()) {
+        if (!$model->isCreatable()) {
             abort(404);
         }
 
@@ -196,7 +196,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();
             }
 
@@ -235,7 +235,7 @@ 
     {
         $item = $model->getRepository()->find($id);
 
-        if (is_null($item) || ! $model->isEditable($item)) {
+        if (is_null($item) || !$model->isEditable($item)) {
             abort(404);
         }
 
@@ -263,7 +263,7 @@ 
         $editForm = $model->fireEdit($id);
         $item = $editForm->getModel();
 
-        if (is_null($item) || ! $model->isEditable($item)) {
+        if (is_null($item) || !$model->isEditable($item)) {
             abort(404);
         }
 
@@ -342,7 +342,7 @@ 
 
         /* @var ColumnEditableInterface|null $column */
         if (is_callable([$display, 'getColumns'])) {
-            $column = $display->getColumns()->all()->filter(function ($column) use ($field) {
+            $column = $display->getColumns()->all()->filter(function($column) use ($field) {
                 return ($column instanceof ColumnEditableInterface) && $field == $column->getName();
             })->first();
         } else {
@@ -351,7 +351,7 @@ 
                     $content = $tab->getContent();
 
                     if ($content instanceof DisplayTable) {
-                        $column = $content->getColumns()->all()->filter(function ($column) use ($field) {
+                        $column = $content->getColumns()->all()->filter(function($column) use ($field) {
                             return ($column instanceof ColumnEditableInterface) && $field == $column->getName();
                         })->first();
                     }
@@ -360,7 +360,7 @@ 
 
                             //Return data-table if inside FormElements
                             if ($element instanceof DisplayTable) {
-                                $column = $element->getColumns()->all()->filter(function ($column) use ($field) {
+                                $column = $element->getColumns()->all()->filter(function($column) use ($field) {
                                     return ($column instanceof ColumnEditableInterface) && $field == $column->getName();
                                 })->first();
                             }
@@ -369,7 +369,7 @@ 
                             if ($element instanceof Column) {
                                 foreach ($element->getElements() as $columnElement) {
                                     if ($columnElement instanceof DisplayTable) {
-                                        $column = $columnElement->getColumns()->all()->filter(function ($column) use ($field) {
+                                        $column = $columnElement->getColumns()->all()->filter(function($column) use ($field) {
                                             return ($column instanceof ColumnEditableInterface) && $field == $column->getName();
                                         })->first();
                                     }
@@ -388,7 +388,7 @@ 
         $repository = $model->getRepository();
         $item = $repository->find($id);
 
-        if (is_null($item) || ! $model->isEditable($item)) {
+        if (is_null($item) || !$model->isEditable($item)) {
             abort(404);
         }
 
@@ -413,7 +413,7 @@ 
     {
         $item = $model->getRepository()->find($id);
 
-        if (is_null($item) || ! $model->isDeletable($item)) {
+        if (is_null($item) || !$model->isDeletable($item)) {
             abort(404);
         }
 
@@ -442,13 +442,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);
         }
 
@@ -477,13 +477,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);
         }
 
@@ -568,7 +568,7 @@ 
      */
     protected function registerBreadcrumb($title, $parent)
     {
-        $this->breadcrumbs->register('render', function (Generator $breadcrumbs) use ($title, $parent) {
+        $this->breadcrumbs->register('render', function(Generator $breadcrumbs) use ($title, $parent) {
             $breadcrumbs->parent($parent);
             $breadcrumbs->push($title);
         });
@@ -584,8 +584,8 @@ 
         $this->breadCrumbsData = $this->breadCrumbsData + (array) $model->getBreadCrumbs();
 
         foreach ($this->breadCrumbsData as  $breadcrumb) {
-            if (! $this->breadcrumbs->exists($breadcrumb['id'])) {
-                $this->breadcrumbs->register($breadcrumb['id'], function (Generator $breadcrumbs) use ($breadcrumb) {
+            if (!$this->breadcrumbs->exists($breadcrumb['id'])) {
+                $this->breadcrumbs->register($breadcrumb['id'], function(Generator $breadcrumbs) use ($breadcrumb) {
                     $breadcrumbs->parent($breadcrumb['parent']);
                     $breadcrumbs->push($breadcrumb['title'], $breadcrumb['url']);
                 });

src/Display/Filter/FilterCustom.php

Spacing +1 added lines, -1 removed lines

@@ -21,7 +21,7 @@
     {
         parent::__construct($name, $title);
 
-        if (! is_null($callback)) {
+        if (!is_null($callback)) {
             $this->setCallback($callback);
         }
     }

src/Display/Column/Custom.php

Spacing +2 added lines, -2 removed lines

@@ -32,7 +32,7 @@ 
     public function __construct($label = null, Closure $callback = null)
     {
         parent::__construct($label);
-        if (! is_null($callback)) {
+        if (!is_null($callback)) {
             $this->setCallback($callback);
         }
     }
@@ -66,7 +66,7 @@ 
      */
     public function getModelValue()
     {
-        if (! is_callable($callback = $this->getCallback())) {
+        if (!is_callable($callback = $this->getCallback())) {
             throw new \Exception('Invalid custom column callback');
         }
 

src/Display/DisplayTable.php

Indentation -1 removed lines

@@ -12,7 +12,6 @@
 
 /**
  * Class DisplayTable.
-
  * @method Columns getColumns()
  * @method $this setColumns(ColumnInterface|ColumnInterface[] $column)
  *

Spacing +3 added lines, -3 removed lines

@@ -71,7 +71,7 @@ 
         parent::initialize();
 
         if ($this->getModelConfiguration()->isRestorableModel()) {
-            $this->setApply(function (Builder $q) {
+            $this->setApply(function(Builder $q) {
                 return $q->withTrashed();
             });
         }
@@ -193,11 +193,11 @@ 
      */
     public function getCollection()
     {
-        if (! $this->isInitialized()) {
+        if (!$this->isInitialized()) {
             throw new \Exception('Display is not initialized');
         }
 
-        if (! is_null($this->collection)) {
+        if (!is_null($this->collection)) {
             return $this->collection;
         }