uccellolabs / uccello

app/Listeners/Core/AfterSaveEventListener.php

Spacing +1 added lines, -1 removed lines

@@ -20,7 +20,7 @@
     {
         if ($event->mode === 'create') {
             Entity::create([
-                'id' => (string) Str::uuid(),
+                'id' => (string)Str::uuid(),
                 'module_id' => $event->module->id,
                 'record_id' => $event->record->getKey(),
             ]);

app/Providers/RouteServiceProvider.php

Spacing +2 added lines, -2 removed lines

@@ -20,7 +20,7 @@ 
         parent::boot();
 
         // Bind domain
-        Route::bind('domain', function ($value) {
+        Route::bind('domain', function($value) {
             if (preg_match('`^[0-9]+$`', $value)) { // By id
                 $domain = Domain::findOrFail($value);
             } else { // By slug
@@ -30,7 +30,7 @@ 
         });
 
         // Bind module
-        Route::bind('module', function ($value) {
+        Route::bind('module', function($value) {
             if (preg_match('`^[0-9]+$`', $value)) { // By id
                 $module = Module::findOrFail($value);
             } else { // By name

app/Helpers/Uccello.php

Spacing +25 added lines, -25 removed lines

@@ -233,26 +233,26 @@ 
 
         if (is_numeric($nameOrId)) {
             // Use cache
-            $modules = Cache::rememberForever('modules_by_id', function () {
+            $modules = Cache::rememberForever('modules_by_id', function() {
                 $modulesGroupedById = collect();
                 Module::all()->map(function($item) use($modulesGroupedById) {
-                    $modulesGroupedById[$item->id] = $item;
+                    $modulesGroupedById[ $item->id ] = $item;
                     return $modulesGroupedById;
                 });
                 return $modulesGroupedById;
             });
-            return $modules[(string) $nameOrId] ?? null;
+            return $modules[ (string)$nameOrId ] ?? null;
         } else {
             // Use cache
-            $modules = Cache::rememberForever('modules_by_name', function () {
+            $modules = Cache::rememberForever('modules_by_name', function() {
                 $modulesGroupedByName = collect();
                 Module::all()->map(function($item) use($modulesGroupedByName) {
-                    $modulesGroupedByName[$item->name] = $item;
+                    $modulesGroupedByName[ $item->name ] = $item;
                     return $modulesGroupedByName;
                 });
                 return $modulesGroupedByName;
             });
-            return $modules[(string) $nameOrId] ?? null;
+            return $modules[ (string)$nameOrId ] ?? null;
         }
     }
 
@@ -270,26 +270,26 @@ 
 
         if (is_numeric($nameOrId)) {
             // Use cache
-            $uitypes = Cache::rememberForever('uitypes_by_id', function () {
+            $uitypes = Cache::rememberForever('uitypes_by_id', function() {
                 $uitypesGroupedById = collect();
                 Uitype::all()->map(function($item) use($uitypesGroupedById) {
-                    $uitypesGroupedById[$item->id] = $item;
+                    $uitypesGroupedById[ $item->id ] = $item;
                     return $uitypesGroupedById;
                 });
                 return $uitypesGroupedById;
             });
-            return $uitypes[(string) $nameOrId] ?? null;
+            return $uitypes[ (string)$nameOrId ] ?? null;
         } else {
             // Use cache
-            $uitypes = Cache::rememberForever('uitypes_by_name', function () {
+            $uitypes = Cache::rememberForever('uitypes_by_name', function() {
                 $uitypesGroupedByName = collect();
                 Uitype::all()->map(function($item) use($uitypesGroupedByName) {
-                    $uitypesGroupedByName[$item->name] = $item;
+                    $uitypesGroupedByName[ $item->name ] = $item;
                     return $uitypesGroupedByName;
                 });
                 return $uitypesGroupedByName;
             });
-            return $uitypes[(string) $nameOrId] ?? null;
+            return $uitypes[ (string)$nameOrId ] ?? null;
         }
     }
 
@@ -307,26 +307,26 @@ 
 
         if (is_numeric($nameOrId)) {
             // Use cache
-            $displaytypes = Cache::rememberForever('displaytypes_by_id', function () {
+            $displaytypes = Cache::rememberForever('displaytypes_by_id', function() {
                 $displaytypesGroupedById = collect();
                 Displaytype::all()->map(function($item) use($displaytypesGroupedById) {
-                    $displaytypesGroupedById[$item->id] = $item;
+                    $displaytypesGroupedById[ $item->id ] = $item;
                     return $displaytypesGroupedById;
                 });
                 return $displaytypesGroupedById;
             });
-            return $displaytypes[(string) $nameOrId] ?? null;
+            return $displaytypes[ (string)$nameOrId ] ?? null;
         } else {
             // Use cache
-            $displaytypes = Cache::rememberForever('displaytypes_by_name', function () {
+            $displaytypes = Cache::rememberForever('displaytypes_by_name', function() {
                 $displaytypesGroupedByName = collect();
                 Displaytype::all()->map(function($item) use($displaytypesGroupedByName) {
-                    $displaytypesGroupedByName[$item->name] = $item;
+                    $displaytypesGroupedByName[ $item->name ] = $item;
                     return $displaytypesGroupedByName;
                 });
                 return $displaytypesGroupedByName;
             });
-            return $displaytypes[(string) $nameOrId] ?? null;
+            return $displaytypes[ (string)$nameOrId ] ?? null;
         }
     }
 
@@ -344,26 +344,26 @@ 
 
         if (is_numeric($nameOrId)) {
             // Use cache
-            $capabilities = Cache::rememberForever('capabilities_by_id', function () {
+            $capabilities = Cache::rememberForever('capabilities_by_id', function() {
                 $capabilitiesGroupedById = collect();
                 Capability::all()->map(function($item) use($capabilitiesGroupedById) {
-                    $capabilitiesGroupedById[$item->id] = $item;
+                    $capabilitiesGroupedById[ $item->id ] = $item;
                     return $capabilitiesGroupedById;
                 });
                 return $capabilitiesGroupedById;
             });
-            return $capabilities[(string) $nameOrId] ?? null;
+            return $capabilities[ (string)$nameOrId ] ?? null;
         } else {
             // Use cache
-            $capabilities = Cache::rememberForever('capabilities_by_name', function () {
+            $capabilities = Cache::rememberForever('capabilities_by_name', function() {
                 $capabilitiesGroupedByName = collect();
                 Capability::all()->map(function($item) use($capabilitiesGroupedByName) {
-                    $capabilitiesGroupedByName[$item->name] = $item;
+                    $capabilitiesGroupedByName[ $item->name ] = $item;
                     return $capabilitiesGroupedByName;
                 });
                 return $capabilitiesGroupedByName;
             });
-            return $capabilities[(string) $nameOrId] ?? null;
+            return $capabilities[ (string)$nameOrId ] ?? null;
         }
     }
 
@@ -405,7 +405,7 @@ 
      * @param string $type
      * @return array
      */
-    public function getDatatableColumns(Module $module, $filterId=null, $type='list'): array
+    public function getDatatableColumns(Module $module, $filterId = null, $type = 'list'): array
     {
         $columns = [ ];
 

app/Http/Controllers/Core/Controller.php

Spacing +5 added lines, -5 removed lines

@@ -75,7 +75,7 @@ 
      * @param Module $module
      * @param boolean $forBlade
      */
-    protected function preProcess(?Domain &$domain, Module $module, Request $request, $forBlade=true)
+    protected function preProcess(?Domain &$domain, Module $module, Request $request, $forBlade = true)
     {
         // If we don't use multi domains, find the first one
         if (!uccello()->useMultiDomains()) {
@@ -157,7 +157,7 @@ 
     {
         $keyName = $getAdminModules ? 'modules_all' : 'modules_not_admin';
 
-        return Cache::rememberForever($keyName, function () use($getAdminModules) {
+        return Cache::rememberForever($keyName, function() use($getAdminModules) {
             $modules = [ ];
 
             $allModules = Module::all();
@@ -215,11 +215,11 @@ 
      */
     protected function getDomainsTreeHtml()
     {
-        return Cache::remember('users', 600, function () { // 10 minutes
+        return Cache::remember('users', 600, function() { // 10 minutes
             $domainsTreeHtml = '<ul class="tree tree-level-0">';
 
             $rootDomains = app('uccello')->getRootDomains();
-            foreach($rootDomains as $root) {
+            foreach ($rootDomains as $root) {
 
                 $descendants = $root->findDescendants()->get();
 
@@ -227,7 +227,7 @@ 
 
                 $html = $tree->render(
                     'ul',
-                    function ($node) {
+                    function($node) {
                         if (auth()->user()->hasRoleOnDomain($node)) {
                             $currentClass = '';
                             // if ($node->id === $this->domain->id) {

app/Http/Controllers/User/AccountController.php

Spacing +7 added lines, -7 removed lines

@@ -52,9 +52,9 @@ 
         });
 
         $validator = Validator::make(request()->all(), [
-            'username' => $rules['username'],
-            'name' => $rules['name'],
-            'email' => $rules['email'],
+            'username' => $rules[ 'username' ],
+            'name' => $rules[ 'name' ],
+            'email' => $rules[ 'email' ],
         ]);
 
         if ($validator->fails()) {
@@ -93,11 +93,11 @@ 
         if (request('avatar')) {
             $image = str_replace('data:image/png;base64,', '', request('avatar'));
             $image = str_replace(' ', '+', $image);
-            $imageName = 'user-' . $user->id . '.png';
+            $imageName = 'user-'.$user->id.'.png';
             $path = storage_path('app/public/avatar/');
-            $filepath = $path . $imageName;
+            $filepath = $path.$imageName;
 
-            if(!\File::isDirectory($path)){
+            if (!\File::isDirectory($path)) {
                 \File::makeDirectory($path, 0777, true, true);
             }
 
@@ -128,7 +128,7 @@ 
 
         $field = Field::where('module_id', ucmodule('user')->id)->where('name', 'password')->first();
         $password_rules = isset($field->data->rules) ? explode('|', $field->data->rules) : '';
-        $password_rules[] = 'confirmed';
+        $password_rules[ ] = 'confirmed';
 
         $validator = Validator::make(request()->all(), [
             'current_password' => function($attribute, $value, $fail) use ($user) {

app/Http/Controllers/Core/ListController.php

Spacing +7 added lines, -7 removed lines

@@ -51,7 +51,7 @@ 
             ->get();
 
         // Order by
-        $filterOrderBy = (array) $selectedFilter->order_by;
+        $filterOrderBy = (array)$selectedFilter->order_by;
 
         return $this->autoView(compact('datatableColumns', 'filters', 'selectedFilter', 'filterOrderBy'));
     }
@@ -115,7 +115,7 @@ 
             if ($relatedList && $relatedList->method) {
                 // Related list method
                 $method = $relatedList->method;
-                $recordIdsMethod = $method . 'RecordIds';
+                $recordIdsMethod = $method.'RecordIds';
 
                 // Get related records ids
                 $model = new $modelClass;
@@ -123,7 +123,7 @@ 
 
                 // Add the record id itself to be filtered
                 if ($relatedList->module_id === $relatedList->related_module_id && !empty($recordId) && !$filteredRecordIds->contains($recordId)) {
-                    $filteredRecordIds[] = (int)$recordId;
+                    $filteredRecordIds[ ] = (int)$recordId;
                 }
 
                 // Make the quer
@@ -134,7 +134,7 @@ 
             $records = $query->paginate($length);
         }
 
-        $records->getCollection()->transform(function ($record) use ($domain, $module) {
+        $records->getCollection()->transform(function($record) use ($domain, $module) {
             foreach ($module->fields as $field) {
                 // If a special template exists, use it. Else use the generic template
                 $uitype = uitype($field->uitype_id);
@@ -174,7 +174,7 @@ 
         $results = collect();
         if (method_exists($modelClass, 'getSearchResult') && property_exists($modelClass, 'searchableColumns')) {
             $searchResults = new Search();
-            $searchResults->registerModel($modelClass, (array) (new $modelClass)->searchableColumns);
+            $searchResults->registerModel($modelClass, (array)(new $modelClass)->searchableColumns);
             $results = $searchResults->search($q)->take(config('uccello.max_results.autocomplete', 10));
         }
 
@@ -195,9 +195,9 @@ 
         $savePageLength = $request->input('save_page_length');
 
         // Optional data
-        $data = [];
+        $data = [ ];
         if ($savePageLength) {
-            $data["length"] = $request->input('page_length');
+            $data[ "length" ] = $request->input('page_length');
         }
 
         $filter = Filter::firstOrNew([

app/Widgets/RelatedlistWidget.php

Spacing +5 added lines, -5 removed lines

@@ -20,19 +20,19 @@
     public function run()
     {
         // Get module
-        $module = ucmodule($this->config['module']);
+        $module = ucmodule($this->config[ 'module' ]);
 
         // Get record
         $modelClass = $module->model_class;
-        $record = $modelClass::find($this->config['record_id']);
+        $record = $modelClass::find($this->config[ 'record_id' ]);
 
         return view('uccello::widgets.relatedlist', [
             'config' => $this->config,
-            'domain' => ucdomain($this->config['domain']),
+            'domain' => ucdomain($this->config[ 'domain' ]),
             'module' => $module,
-            'data' => (object) $this->config['data'],
+            'data' => (object)$this->config[ 'data' ],
             'record' => $record,
-            'label' => $this->config['data']->label ?? $this->config['labelForTranslation'],
+            'label' => $this->config[ 'data' ]->label ?? $this->config[ 'labelForTranslation' ],
         ]);
     }
 }

database/migrations/2018_04_15_000028_add_default_data.php

Spacing +3 added lines, -3 removed lines

@@ -46,7 +46,7 @@ 
         ]);
 
         Entity::create([
-            'id' => (string) Str::uuid(),
+            'id' => (string)Str::uuid(),
             'module_id' => ucmodule('domain')->id,
             'record_id' => $domain->getKey(),
         ]);
@@ -61,7 +61,7 @@ 
         ]);
 
         Entity::create([
-            'id' => (string) Str::uuid(),
+            'id' => (string)Str::uuid(),
             'module_id' => ucmodule('profile')->id,
             'record_id' => $profile->getKey(),
         ]);
@@ -91,7 +91,7 @@ 
         $role->profiles()->attach($profile);
 
         Entity::create([
-            'id' => (string) Str::uuid(),
+            'id' => (string)Str::uuid(),
             'module_id' => ucmodule('role')->id,
             'record_id' => $role->getKey(),
         ]);

database/migrations/2018_04_15_000031_alter_role_module.php

Spacing +1 added lines, -1 removed lines

@@ -33,7 +33,7 @@
             'uitype_id' => uitype('boolean')->id,
             'displaytype_id' => displaytype('everywhere')->id,
             'sequence' => 3,
-            'data' => [ 'info' => 'field_info.see_descendants_records']
+            'data' => [ 'info' => 'field_info.see_descendants_records' ]
         ]);
 
         $filter = Filter::where('module_id', $roleModule->id)->where('name', 'filter.all')->first();