anavel / crud

src/Abstractor/Eloquent/Traits/ModelFields.php

Doc Comments +1 added lines, -1 removed lines

@@ -27,7 +27,7 @@
     /**
      * @param array $config
      * @param string $columnName
-     * @return array
+     * @return \Doctrine\DBAL\Schema\Column
      */
     public function setConfig(array $config, $columnName)
     {

Spacing +5 added lines, -5 removed lines

@@ -17,11 +17,11 @@
      */
     public function readConfig($action)
     {
-        $this->fieldsPresentation = $this->getConfigValue('fields_presentation') ? : [];
-        $this->formTypes = $this->getConfigValue($action, 'form_types') ? : [];
-        $this->validationRules = $this->getConfigValue($action, 'validation') ? : [];
-        $this->functions = $this->getConfigValue($action, 'functions') ? : [];
-        $this->defaults = $this->getConfigValue($action, 'defaults') ? : [];
+        $this->fieldsPresentation = $this->getConfigValue('fields_presentation') ?: [];
+        $this->formTypes = $this->getConfigValue($action, 'form_types') ?: [];
+        $this->validationRules = $this->getConfigValue($action, 'validation') ?: [];
+        $this->functions = $this->getConfigValue($action, 'functions') ?: [];
+        $this->defaults = $this->getConfigValue($action, 'defaults') ?: [];
     }
 
     /**

Unused Use Statements -2 removed lines

@@ -4,8 +4,6 @@
 namespace Anavel\Crud\Abstractor\Eloquent\Traits;
 
 
-use Anavel\Crud\Contracts\Abstractor\Field;
-
 trait ModelFields
 {
     protected $fieldsPresentation = [];

src/Contracts/Abstractor/Relation.php

Doc Comments -1 removed lines

@@ -55,7 +55,6 @@
     public function getSecondaryRelations();
 
     /**
-     * @param Model $relatedModel
      * @return Relation
      */
     public function setRelatedModel(\Illuminate\Database\Eloquent\Model $model);

src/Abstractor/Eloquent/Relation/SelectMultiple.php

Spacing +4 added lines, -4 removed lines

@@ -28,7 +28,7 @@ 
      */
     public function persist(array $relationArray = null)
     {
-        if (! empty($relationArray)) {
+        if (!empty($relationArray)) {
             /** @var \ANavallaSuiza\Laravel\Database\Contracts\Repository\Repository $repo */
             $repo = $this->modelManager->getRepository(get_class($this->eloquentRelation->getRelated()));
 
@@ -37,7 +37,7 @@ 
             $alreadyAssociated = $this->relatedModel->$relationName;
 
             $search = [];
-            if (! empty($relationArray[$relatedKeyName])) {
+            if (!empty($relationArray[$relatedKeyName])) {
                 $search = $relationArray[$relatedKeyName];
             }
             $results = $repo->pushCriteria(
@@ -53,7 +53,7 @@ 
                 $result->save();
             }
 
-            if (! $missing->isEmpty()) {
+            if (!$missing->isEmpty()) {
                 foreach ($missing as $result) {
                     $result->$keyName = null;
                     $result->save();
@@ -98,7 +98,7 @@ 
     {
         $results = $this->eloquentRelation->getResults();
 
-        if (! $results->isEmpty()) {
+        if (!$results->isEmpty()) {
             $values = [];
 
             foreach ($results as $result) {

src/Abstractor/Eloquent/Traits/HandleFiles.php

Doc Comments +3 added lines

@@ -9,6 +9,9 @@
 
 trait HandleFiles
 {
+    /**
+     * @param string $groupName
+     */
     protected function handleField(Request $request, $item, array $fields, $currentKey,  $groupName, $fieldName)
     {
         $modelFolder = $this->slug . DIRECTORY_SEPARATOR;

Spacing +16 added lines, -16 removed lines

@@ -11,18 +11,18 @@ 
 {
     protected function handleField(Request $request, $item, array $fields, $groupName, $fieldName)
     {
-        $modelFolder = $this->slug . DIRECTORY_SEPARATOR;
-        $basePath = base_path(DIRECTORY_SEPARATOR . 'public' . DIRECTORY_SEPARATOR .config('anavel-crud.uploads_path'));
-        $modelPath = $basePath . $modelFolder;
+        $modelFolder = $this->slug.DIRECTORY_SEPARATOR;
+        $basePath = base_path(DIRECTORY_SEPARATOR.'public'.DIRECTORY_SEPARATOR.config('anavel-crud.uploads_path'));
+        $modelPath = $basePath.$modelFolder;
         $skip = null;
         $requestValue = null;
-        if (! empty($fields["{$fieldName}__delete"])) {
+        if (!empty($fields["{$fieldName}__delete"])) {
             //We never want to save this field, it doesn't exist in the DB
             $skip = "{$fieldName}__delete";
 
 
             //If user wants to delete the existing file
-            if (! empty($request->input("{$groupName}.{$fieldName}__delete"))) {
+            if (!empty($request->input("{$groupName}.{$fieldName}__delete"))) {
                 $adapter = new Local($basePath);
                 $filesystem = new Filesystem($adapter);
                 if ($filesystem->has($item->$fieldName)) {
@@ -39,28 +39,28 @@ 
                 ];
             }
         }
-        if ($request->hasFile($groupName .'.'.$fieldName)) {
-            $fileName = pathinfo($request->file($groupName .'.'.$fieldName)->getClientOriginalName(), PATHINFO_FILENAME);
-            $extension = pathinfo($request->file($groupName .'.'.$fieldName)->getClientOriginalName(), PATHINFO_EXTENSION);
+        if ($request->hasFile($groupName.'.'.$fieldName)) {
+            $fileName = pathinfo($request->file($groupName.'.'.$fieldName)->getClientOriginalName(), PATHINFO_FILENAME);
+            $extension = pathinfo($request->file($groupName.'.'.$fieldName)->getClientOriginalName(), PATHINFO_EXTENSION);
 
-            $fileName = uniqid() . '_' . slugify($fileName);
-            if (! empty($extension)) {
-                $fileName .= '.' . $extension;
+            $fileName = uniqid().'_'.slugify($fileName);
+            if (!empty($extension)) {
+                $fileName .= '.'.$extension;
             }
 
 
-            $request->file($groupName .'.'.$fieldName)->move(
+            $request->file($groupName.'.'.$fieldName)->move(
                 $modelPath,
                 $fileName
             );
 
-            $requestValue = $modelFolder . $fileName;
-        } elseif (! empty($request->file($groupName .'.'.$fieldName)) && ! $request->file($groupName .'.'.$fieldName)->isValid()) {
-            throw new \Exception($request->file($groupName .'.'.$fieldName)->getErrorMessage());
+            $requestValue = $modelFolder.$fileName;
+        } elseif (!empty($request->file($groupName.'.'.$fieldName)) && !$request->file($groupName.'.'.$fieldName)->isValid()) {
+            throw new \Exception($request->file($groupName.'.'.$fieldName)->getErrorMessage());
         }
 
         //Avoid losing the existing filename if the user doesn't change it:
-        if (empty($requestValue) && (! empty($item->$fieldName))) {
+        if (empty($requestValue) && (!empty($item->$fieldName))) {
             $requestValue = $item->$fieldName;
         }
 

src/Abstractor/Eloquent/Relation/MiniCrud.php

Spacing +6 added lines, -6 removed lines

@@ -122,7 +122,7 @@ 
 
                 $config = [
                     'name' => $columnName,
-                    'presentation' => $this->name . ' ' . ucfirst(transcrud($columnName)),
+                    'presentation' => $this->name.' '.ucfirst(transcrud($columnName)),
                     'form_type' => $formType,
                     'no_validate' => true,
                     'validation' => null,
@@ -143,7 +143,7 @@ 
                     $field = $this->fieldFactory
                         ->setColumn($column)
                         ->setConfig([
-                            'name' => $columnName . '__delete',
+                            'name' => $columnName.'__delete',
                             'presentation' => null,
                             'form_type' => 'checkbox',
                             'no_validate' => true,
@@ -151,7 +151,7 @@ 
                             'functions' => null
                         ])
                         ->get();
-                    $fields[$columnName . '__delete'] = $field;
+                    $fields[$columnName.'__delete'] = $field;
                 }
             }
         }
@@ -195,12 +195,12 @@ 
                     $fieldName = $field->getName();
 
                     if (get_class($field->getFormField()) === \FormManager\Fields\File::class) {
-                        $handleResult = $this->handleField($request, $relationModel, $fieldsBase, $this->name . ".$relationIndex", $fieldName);
-                        if (! empty($handleResult['skip'])) {
+                        $handleResult = $this->handleField($request, $relationModel, $fieldsBase, $this->name.".$relationIndex", $fieldName);
+                        if (!empty($handleResult['skip'])) {
                             $skip = $handleResult['skip'];
                             unset($relationArray[$relationIndex][$skip]);
                         }
-                        if (! empty($handleResult['requestValue'])) {
+                        if (!empty($handleResult['requestValue'])) {
                             $relationArray[$relationIndex][$fieldName] = $handleResult['requestValue'];
                         }
                     }

src/Abstractor/Eloquent/Relation/SelectMultipleManyToMany.php

Spacing +1 added lines, -1 removed lines

@@ -28,7 +28,7 @@
      */
     public function persist(array $relationArray = null, Request $request)
     {
-        if (! empty($relationArray)) {
+        if (!empty($relationArray)) {
             $this->eloquentRelation->sync($relationArray[$this->eloquentRelation->getRelated()->getKeyName()]);
         }
     }

src/Http/Controllers/ModelController.php

Doc Comments +3 added lines

@@ -25,6 +25,9 @@
         $this->formGenerator = $formGenerator;
     }
     
+    /**
+     * @param string $methodName
+     */
     private function authorizeMethod(Model $modelAbstractor, $methodName)
     {
         if (array_key_exists('authorize', $config = $modelAbstractor->getConfig()) && $config['authorize'] === true) {

Spacing +9 added lines, -9 removed lines

@@ -38,8 +38,8 @@ 
      */
     private function customController(Model $modelAbstractor)
     {
-        if (! $this instanceof CustomController) { //Avoid infinite recursion
-            if (array_key_exists('controller', $config = $modelAbstractor->getConfig()) && (! empty($config['controller']))) {
+        if (!$this instanceof CustomController) { //Avoid infinite recursion
+            if (array_key_exists('controller', $config = $modelAbstractor->getConfig()) && (!empty($config['controller']))) {
                 /** @var CustomController $controller */
                 $controller = App::make($config['controller']);
                 $controller->setAbstractor($modelAbstractor);
@@ -62,7 +62,7 @@ 
 
         $this->authorizeMethod($modelAbstractor, 'adminIndex');
 
-        if (! empty($customController = $this->customController($modelAbstractor))) {
+        if (!empty($customController = $this->customController($modelAbstractor))) {
             return $customController->index($request, $model);
         }
 
@@ -102,7 +102,7 @@ 
 
         $this->authorizeMethod($modelAbstractor, 'adminCreate');
 
-        if (! empty($customController = $this->customController($modelAbstractor))) {
+        if (!empty($customController = $this->customController($modelAbstractor))) {
             return $customController->create($model);
         }
 
@@ -128,7 +128,7 @@ 
 
         $this->authorizeMethod($modelAbstractor, 'adminStore');
 
-        if (! empty($customController = $this->customController($modelAbstractor))) {
+        if (!empty($customController = $this->customController($modelAbstractor))) {
             return $customController->store($request, $model);
         }
 
@@ -162,7 +162,7 @@ 
 
         $this->authorizeMethod($modelAbstractor, 'adminShow');
 
-        if (! empty($customController = $this->customController($modelAbstractor))) {
+        if (!empty($customController = $this->customController($modelAbstractor))) {
             return $customController->show($model, $id);
         }
 
@@ -189,7 +189,7 @@ 
 
         $this->authorizeMethod($modelAbstractor, 'adminEdit');
 
-        if (! empty($customController = $this->customController($modelAbstractor))) {
+        if (!empty($customController = $this->customController($modelAbstractor))) {
             return $customController->edit($model, $id);
         }
 
@@ -219,7 +219,7 @@ 
 
         $this->authorizeMethod($modelAbstractor, 'adminUpdate');
 
-        if (! empty($customController = $this->customController($modelAbstractor))) {
+        if (!empty($customController = $this->customController($modelAbstractor))) {
             return $customController->update($request, $model, $id);
         }
 
@@ -254,7 +254,7 @@ 
 
         $this->authorizeMethod($modelAbstractor, 'adminDestroy');
 
-        if (! empty($customController = $this->customController($modelAbstractor))) {
+        if (!empty($customController = $this->customController($modelAbstractor))) {
             return $customController->destroy($request, $model, $id);
         }
 

src/Http/Controllers/HomeController.php

Spacing +1 added lines, -1 removed lines

@@ -23,7 +23,7 @@
             $modelAbstractor = $modelFactory->getByName($modelSlug);
             $config = $modelAbstractor->getConfig();
 
-            if (! array_key_exists('authorize', $config) || ($config['authorize'] === true && Gate::allows('adminIndex', $modelAbstractor->getInstance()) ||  $config['authorize'] === false)) {
+            if (!array_key_exists('authorize', $config) || ($config['authorize'] === true && Gate::allows('adminIndex', $modelAbstractor->getInstance()) || $config['authorize'] === false)) {
                 return new RedirectResponse(route('anavel-crud.model.index', $modelSlug));
             }
         }

src/View/Composers/SidebarComposer.php

Spacing +2 added lines, -2 removed lines

@@ -60,13 +60,13 @@
             } catch (\Exception $e) {
                 continue;
             }
-            $menuItems[$modelName]['isActive'] =  $item['isActive'];
+            $menuItems[$modelName]['isActive'] = $item['isActive'];
             $menuItems[$modelName]['name'] = $item['name'];
             $menuItems[$modelName]['items'][] = $item;
         }
 
         //Sort alphabetically de menu items
-        usort($menuItems, function ($itemA, $itemB) {
+        usort($menuItems, function($itemA, $itemB) {
             return strcmp($itemA['name'], $itemB['name']);
         });
         return $menuItems;