anavel / crud

src/Http/routes.php

Spacing +1 added lines, -1 removed lines

@@ -5,7 +5,7 @@
         'prefix' => 'crud',
         'namespace' => 'Anavel\Crud\Http\Controllers'
     ],
-    function () {
+    function() {
         Route::get('/', [
             'as'   => 'anavel-crud.home',
             'uses' => 'HomeController@index'

src/helpers.php

Spacing +4 added lines, -4 removed lines

@@ -2,7 +2,7 @@ 
 
 use EasySlugger\Slugger;
 
-if (! function_exists('slugify')) {
+if (!function_exists('slugify')) {
     /**
      * Generate slug
      *
@@ -15,7 +15,7 @@ 
     }
 }
 
-if (! function_exists('uniqueSlugify')) {
+if (!function_exists('uniqueSlugify')) {
     /**
      * Generate unique slug
      *
@@ -28,7 +28,7 @@ 
     }
 }
 
-if (! function_exists('transcrud')) {
+if (!function_exists('transcrud')) {
     /**
      * Translate string but remove file key if translation not found
      *
@@ -37,7 +37,7 @@ 
      */
     function transcrud($text)
     {
-        $translation = trans('anavel-crud::models.' . $text);
+        $translation = trans('anavel-crud::models.'.$text);
 
         return str_replace('anavel-crud::models.', '', $translation);
     }

src/Abstractor/Eloquent/Model.php

Doc Comments +4 added lines, -1 removed lines

@@ -85,6 +85,9 @@ 
         return transcrud($this->name);
     }
 
+    /**
+     * @return string
+     */
     public function getModel()
     {
         return $this->model;
@@ -334,7 +337,7 @@ 
     }
 
     /**
-     * @param array $requestForm
+     * @param array $request
      * @return mixed
      */
     public function persist(Request $request)

Unused Use Statements -2 removed lines

@@ -16,8 +16,6 @@
 use Anavel\Crud\Abstractor\Exceptions\AbstractorException;
 use Illuminate\Http\Request;
 use Illuminate\Support\Collection;
-use League\Flysystem\Adapter\Local;
-use League\Flysystem\Filesystem;
 
 class Model implements ModelAbstractorContract
 {

Spacing +20 added lines, -20 removed lines

@@ -127,13 +127,13 @@ 
         }
 
         $customDisplayedColumns = $this->getConfigValue($action, 'display');
-        $customHiddenColumns = $this->getConfigValue($action, 'hide') ? : [];
+        $customHiddenColumns = $this->getConfigValue($action, 'hide') ?: [];
 
         $columns = array();
-        if (! empty($customDisplayedColumns) && is_array($customDisplayedColumns)) {
+        if (!empty($customDisplayedColumns) && is_array($customDisplayedColumns)) {
             foreach ($customDisplayedColumns as $customColumn) {
-                if (! array_key_exists($customColumn, $tableColumns)) {
-                    throw new AbstractorException("Column " . $customColumn . " does not exist on " . $this->getModel());
+                if (!array_key_exists($customColumn, $tableColumns)) {
+                    throw new AbstractorException("Column ".$customColumn." does not exist on ".$this->getModel());
                 }
 
                 $columns[$customColumn] = $tableColumns[$customColumn];
@@ -164,7 +164,7 @@ 
 
         $relations = collect();
 
-        if (! empty($configRelations)) {
+        if (!empty($configRelations)) {
             foreach ($configRelations as $relationName => $configRelation) {
                 if (is_int($relationName)) {
                     $relationName = $configRelation;
@@ -172,7 +172,7 @@ 
 
                 $config = [];
                 if ($configRelation !== $relationName) {
-                    if (! is_array($configRelation)) {
+                    if (!is_array($configRelation)) {
                         $config['type'] = $configRelation;
                     } else {
                         $config = $configRelation;
@@ -187,7 +187,7 @@ 
                 $secondaryRelations = $relation->getSecondaryRelations();
 
 
-                if (! $secondaryRelations->isEmpty()) {
+                if (!$secondaryRelations->isEmpty()) {
                     $relations->put($relationName,
                         collect(['relation' => $relation, 'secondaryRelations' => $secondaryRelations]));
                 } else {
@@ -209,7 +209,7 @@ 
     {
         $columns = $this->getColumns('list');
 
-        $fieldsPresentation = $this->getConfigValue('fields_presentation') ? : [];
+        $fieldsPresentation = $this->getConfigValue('fields_presentation') ?: [];
 
         $fields = array();
         foreach ($columns as $name => $column) {
@@ -244,7 +244,7 @@ 
     {
         $columns = $this->getColumns('detail');
 
-        $fieldsPresentation = $this->getConfigValue('fields_presentation') ? : [];
+        $fieldsPresentation = $this->getConfigValue('fields_presentation') ?: [];
 
         $fields = array();
         foreach ($columns as $name => $column) {
@@ -284,7 +284,7 @@ 
 
         $fields = array();
         foreach ($columns as $name => $column) {
-            if (! in_array($name, $this->getReadOnlyColumns())) {
+            if (!in_array($name, $this->getReadOnlyColumns())) {
                 $presentation = null;
                 if (array_key_exists($name, $this->fieldsPresentation)) {
                     $presentation = $this->fieldsPresentation[$name];
@@ -305,17 +305,17 @@ 
                     ->setConfig($config)
                     ->get();
 
-                if (! empty($this->instance) && ! empty($this->instance->getAttribute($name))) {
+                if (!empty($this->instance) && !empty($this->instance->getAttribute($name))) {
                     $field->setValue($this->instance->getAttribute($name));
                 }
 
                 $fields[$arrayKey][] = $field;
 
-                if (! empty($config['form_type']) && $config['form_type'] === 'file') {
+                if (!empty($config['form_type']) && $config['form_type'] === 'file') {
                     $field = $this->fieldFactory
                         ->setColumn($column)
                         ->setConfig([
-                            'name'         => $name . '__delete',
+                            'name'         => $name.'__delete',
                             'presentation' => null,
                             'form_type'    => 'checkbox',
                             'validation'   => null,
@@ -359,7 +359,7 @@ 
     {
         /** @var \ANavallaSuiza\Laravel\Database\Contracts\Manager\ModelManager $modelManager */
         $modelManager = App::make('ANavallaSuiza\Laravel\Database\Contracts\Manager\ModelManager');
-        if (! empty($this->instance)) {
+        if (!empty($this->instance)) {
             $item = $this->instance;
         } else {
             $item = $modelManager->getModelInstance($this->getModel());
@@ -371,7 +371,7 @@ 
             return;
         }
 
-        if (! empty($fields['main'])) {
+        if (!empty($fields['main'])) {
             $skipNext = false;
             foreach ($fields['main'] as $key => $field) {
                 if ($skipNext === true) {
@@ -382,19 +382,19 @@ 
 
                 if (get_class($field->getFormField()) === \FormManager\Fields\File::class) {
                     $handleResult = $this->handleField($request, $item, $fields, $key, 'main', $fieldName);
-                    if (! empty($handleResult['skipNext'])) {
+                    if (!empty($handleResult['skipNext'])) {
                         $skipNext = $handleResult['skipNext'];
                     }
-                    if (! empty($handleResult['requestValue'])) {
+                    if (!empty($handleResult['requestValue'])) {
                         $requestValue = $handleResult['requestValue'];
                     }
                 }
 
-                if (! $field->saveIfEmpty() && empty($requestValue)) {
+                if (!$field->saveIfEmpty() && empty($requestValue)) {
                     continue;
                 }
 
-                if (! empty($requestValue)) {
+                if (!empty($requestValue)) {
                     $item->setAttribute(
                         $fieldName,
                         $field->applyFunctions($requestValue)
@@ -408,7 +408,7 @@ 
         $this->setInstance($item);
 
 
-        if (! empty($relations = $this->getRelations())) {
+        if (!empty($relations = $this->getRelations())) {
             foreach ($relations as $relationKey => $relation) {
                 if ($relation instanceof Collection) {
                     $input = $request->input($relationKey);

src/Abstractor/Eloquent/Relation/Translation.php

Unused Use Statements -2 removed lines

@@ -3,8 +3,6 @@
 
 use Anavel\Crud\Abstractor\Eloquent\Relation\Traits\CheckRelationCompatibility;
 use Anavel\Crud\Contracts\Abstractor\Field;
-use App;
-use Illuminate\Http\Request;
 
 class Translation extends Relation
 {

Spacing +4 added lines, -4 removed lines

@@ -40,12 +40,12 @@ 
 
         $this->readConfig('edit');
 
-        if(empty($arrayKey)) {
+        if (empty($arrayKey)) {
             $arrayKey = $this->name;
         }
 
         $translationFields = [];
-        if (! empty($columns)) {
+        if (!empty($columns)) {
             foreach ($this->langs as $key => $lang) {
                 $tempFields = [];
                 foreach ($columns as $columnName => $column) {
@@ -65,7 +65,7 @@ 
 
                     $config = [
                         'name' => $columnName,
-                        'presentation' => ucfirst(transcrud($columnName)).' ['.$lang .']',
+                        'presentation' => ucfirst(transcrud($columnName)).' ['.$lang.']',
                         'form_type' => $formType,
                         'no_validate' => true,
                         'validation' => null,
@@ -105,7 +105,7 @@ 
      */
     public function persist(array $relationArray = null)
     {
-        if (! empty($relationArray)) {
+        if (!empty($relationArray)) {
             $currentTranslations = $this->eloquentRelation->get();
             $currentTranslations = $currentTranslations->keyBy('locale');
 

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') ?: [];
     }
 
     /**

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);

Unused Use Statements -1 removed lines

@@ -2,7 +2,6 @@
 namespace Anavel\Crud\Contracts\Abstractor;
 
 use Anavel\Crud\Abstractor\Exceptions\RelationException;
-use Illuminate\Http\Request;
 use Illuminate\Support\Collection;
 
 interface Relation

src/Abstractor/Eloquent/Relation/MiniCrudSingle.php

Spacing +7 added lines, -7 removed lines

@@ -26,7 +26,7 @@ 
      */
     public function getEditFields($arrayKey = null)
     {
-        if(empty($arrayKey)) {
+        if (empty($arrayKey)) {
             $arrayKey = $this->name;
         }
 
@@ -48,7 +48,7 @@ 
 
         $this->readConfig('edit');
 
-        if (! empty($columns)) {
+        if (!empty($columns)) {
             foreach ($columns as $columnName => $column) {
                 if (in_array($columnName, $readOnly, true)) {
                     continue;
@@ -58,7 +58,7 @@ 
 
                 $config = [
                     'name'         => $columnName,
-                    'presentation' => $this->name . ' ' . ucfirst(transcrud($columnName)),
+                    'presentation' => $this->name.' '.ucfirst(transcrud($columnName)),
                     'form_type'    => $formType,
                     'no_validate'  => true,
                     'validation'   => null,
@@ -73,7 +73,7 @@ 
                     ->setConfig($config)
                     ->get();
 
-                if (! empty($result->id)) {
+                if (!empty($result->id)) {
                     $field->setValue($result->getAttribute($columnName));
                 }
 
@@ -93,9 +93,9 @@ 
      */
     public function persist(array $relationArray = null)
     {
-        if (! empty($relationArray)) {
+        if (!empty($relationArray)) {
             $currentRelation = $this->eloquentRelation->getResults();
-            if (! empty($currentRelation)) {
+            if (!empty($currentRelation)) {
                 $relationModel = $currentRelation;
             } else {
                 $relationModel = $this->eloquentRelation->getRelated()->newInstance();
@@ -111,7 +111,7 @@ 
                 $relationModel->setAttribute($fieldKey, $fieldValue);
             }
 
-            if (! $shouldBeSkipped) {
+            if (!$shouldBeSkipped) {
                 $relationModel->save();
             }
         }

src/Abstractor/Eloquent/Relation/MiniCrud.php

Spacing +8 added lines, -8 removed lines

@@ -44,7 +44,7 @@ 
         $results = $this->eloquentRelation->getResults();
 
         $results->put('emptyResult', '');
-        if (! empty($columns)) {
+        if (!empty($columns)) {
             $readOnly = [Model::CREATED_AT, Model::UPDATED_AT];
             foreach ($results as $key => $result) {
                 $tempFields = [];
@@ -65,7 +65,7 @@ 
 
                     $config = [
                         'name'         => $columnName,
-                        'presentation' => $this->name . ' ' . ucfirst(transcrud($columnName)) . ' [' . $index . ']',
+                        'presentation' => $this->name.' '.ucfirst(transcrud($columnName)).' ['.$index.']',
                         'form_type'    => $formType,
                         'no_validate'  => true,
                         'validation'   => null,
@@ -87,12 +87,12 @@ 
                 }
 
                 $relationModel = $this->eloquentRelation->getRelated()->newInstance();
-                if (! empty($result)) {
+                if (!empty($result)) {
                     $relationModel = $result;
                 }
                 $this->modelAbstractor->setInstance($relationModel);
                 $secondaryRelations = $this->getSecondaryRelations();
-                if (! empty($secondaryRelations)) {
+                if (!empty($secondaryRelations)) {
                     foreach ($secondaryRelations as $secondaryRelationKey => $secondaryRelation) {
                         foreach ($secondaryRelation->getEditFields($secondaryRelationKey) as $editGroupName => $editGroup) {
                             if ($secondaryRelation->getType() === 'Anavel\Crud\Abstractor\Eloquent\Relation\Select') {
@@ -119,12 +119,12 @@ 
      */
     public function persist(array $relationArray = null)
     {
-        if (! empty($relationArray)) {
+        if (!empty($relationArray)) {
             $keyName = $this->eloquentRelation->getParent()->getKeyName();
             $currentRelations = $this->eloquentRelation->get()->keyBy($keyName);
 
             foreach ($relationArray as $relation) {
-                if (! empty($relation[$keyName])
+                if (!empty($relation[$keyName])
                     && ($currentRelations->has($relation[$keyName]))
                 ) {
                     $relationModel = $currentRelations->get($relation[$keyName]);
@@ -155,10 +155,10 @@ 
                     $relationModel->setAttribute($fieldKey, $fieldValue);
                 }
 
-                if (! $shouldBeSkipped) {
+                if (!$shouldBeSkipped) {
                     $relationModel->save();
 
-                    if (! $delayedRelations->isEmpty()) {
+                    if (!$delayedRelations->isEmpty()) {
                         foreach ($delayedRelations as $relationKey => $delayedRelation) {
                             /** @var RelationContract $secondaryRelation */
                             $secondaryRelation = $secondaryRelations->get($relationKey);

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) {