SherifElfadaly / Laravel-Api-Skeleton

src/Modules/Roles/Repositories/RoleRepository.php

Spacing +2 added lines, -2 removed lines

@@ -26,7 +26,7 @@ 
      */
     public function detachPermissions(mixed $role): bool
     {
-        $role = ! filter_var($role, FILTER_VALIDATE_INT) ? $role : $this->find($role);
+        $role = !filter_var($role, FILTER_VALIDATE_INT) ? $role : $this->find($role);
         $role->permissions()->detach();
 
         return true;
@@ -41,7 +41,7 @@ 
      */
     public function attachPermissions(mixed $role, array $permissionIds): bool
     {
-        $role = ! filter_var($role, FILTER_VALIDATE_INT) ? $role : $this->find($role);
+        $role = !filter_var($role, FILTER_VALIDATE_INT) ? $role : $this->find($role);
         $role->permissions()->attach($permissionIds);
 
         return true;

src/Modules/Reporting/Services/ReportService.php

Spacing +2 added lines, -2 removed lines

@@ -47,9 +47,9 @@
         /**
          * Check report existance and permission.
          */
-        if (! $report) {
+        if (!$report) {
             Errors::notFound('report');
-        } elseif (! $skipPermission && ! $this->userService->can($report->view_name, 'report')) {
+        } elseif (!$skipPermission && !$this->userService->can($report->view_name, 'report')) {
             Errors::noPermissions();
         }
 

src/Modules/Reporting/Repositories/ReportRepository.php

Spacing +1 added lines, -1 removed lines

@@ -30,7 +30,7 @@
      */
     public function renderReport(mixed $report, array $conditions = [], int $perPage = 0): mixed
     {
-        $report = ! filter_var($report, FILTER_VALIDATE_INT) ? $report : $this->find($report);
+        $report = !filter_var($report, FILTER_VALIDATE_INT) ? $report : $this->find($report);
         /**
          * Fetch data from the report based on the given conditions.
          */

src/Modules/Core/Http/Middleware/CheckPermissions.php

Spacing +2 added lines, -2 removed lines

@@ -57,7 +57,7 @@ 
         $permission          = $routeActions[1];
 
         $this->auth->shouldUse('api');
-        if (! in_array($permission, $skipLoginCheck)) {
+        if (!in_array($permission, $skipLoginCheck)) {
             $this->authMiddleware->handle($request, function ($request) use ($modelName, $skipPermissionCheck, $permission) {
                 $user             = $this->auth->user();
                 $isPasswordClient = $user->token() ? $user->token()->client->password_client : false;
@@ -67,7 +67,7 @@ 
                 }
     
                 if ($isPasswordClient && (in_array($permission, $skipPermissionCheck) || $this->userService->can($permission, $modelName))) {
-                } elseif (! $isPasswordClient && $user->tokenCan($modelName.'-'.$permission)) {
+                } elseif (!$isPasswordClient && $user->tokenCan($modelName.'-'.$permission)) {
                 } else {
                     Errors::noPermissions();
                 }

src/Modules/Core/Traits/Translatable.php

Spacing +1 added lines, -1 removed lines

@@ -40,7 +40,7 @@
             return $values;
         }
 
-        if (! $primaryLocale || ! is_object($values) || ! property_exists($values, $primaryLocale)) {
+        if (!$primaryLocale || !is_object($values) || !property_exists($values, $primaryLocale)) {
             return $values ? (isset($values->$fallbackLocale) ? $values->$fallbackLocale : $values) : '';
         }
 

src/Modules/Core/Console/Commands/GenerateDocCommand.php

Spacing +8 added lines, -8 removed lines

@@ -124,7 +124,7 @@ 
         ];
 
 
-        if (! $skipLoginCheck || ! in_array($method, $skipLoginCheck)) {
+        if (!$skipLoginCheck || !in_array($method, $skipLoginCheck)) {
             $route['headers']['Authorization'] = 'Bearer {token}';
         }
     }
@@ -183,7 +183,7 @@ 
         if ($className) {
             $reflectionClass  = new \ReflectionClass($className);
         } elseif (in_array($reflectionMethod->getName(), ['store', 'update'])) {
-            $className = 'App\\Modules\\' . ucfirst($module) . '\\Http\\Requests\\Store'  . ucfirst($modelName);
+            $className = 'App\\Modules\\'.ucfirst($module).'\\Http\\Requests\\Store'.ucfirst($modelName);
             $reflectionClass  = new \ReflectionClass($className);
         }
 
@@ -210,8 +210,8 @@ 
     {
         $errors = [];
         foreach (\Module::all() as $module) {
-            $nameSpace = 'App\\Modules\\' . $module['basename'];
-            $class = $nameSpace . '\\Errors\\'  . $module['basename'] . 'Errors';
+            $nameSpace = 'App\\Modules\\'.$module['basename'];
+            $class = $nameSpace.'\\Errors\\'.$module['basename'].'Errors';
             $reflectionClass = new \ReflectionClass($class);
             foreach ($reflectionClass->getMethods() as $method) {
                 $methodName       = $method->name;
@@ -257,16 +257,16 @@ 
      */
     protected function getModels($modelName, &$docData, $reflectionClass)
     {
-        if ($modelName && ! Arr::has($docData['models'], $modelName)) {
+        if ($modelName && !Arr::has($docData['models'], $modelName)) {
             $repo = call_user_func_array("\Core::{$modelName}", []);
-            if (! $repo) {
+            if (!$repo) {
                 return;
             }
             
             $modelClass = get_class($repo->model);
             $nameSpaceArr = explode('\\', $modelClass);
             array_pop($nameSpaceArr);
-            $factory = implode('\\', $nameSpaceArr) . '\\Database\Factories\\' . ucfirst($modelName) . 'Factory';
+            $factory = implode('\\', $nameSpaceArr).'\\Database\Factories\\'.ucfirst($modelName).'Factory';
             $model = App::make($factory)->make();
 
             $property = $reflectionClass->getProperty('modelResource');
@@ -280,7 +280,7 @@ 
                 $body = $this->getMethodBody($reflectionMethod);
                 foreach ($relationMethods as $relationMethod) {
                     $relation = $reflectionMethod->getName();
-                    if (strpos($body, '$this->' . $relationMethod) && $relation !== 'morphedByMany') {
+                    if (strpos($body, '$this->'.$relationMethod) && $relation !== 'morphedByMany') {
                         $relations[] = $relation;
                         break;
                     }

src/Modules/Core/Console/Commands/MakeModuleCommand.php

Spacing +17 added lines, -17 removed lines

@@ -91,7 +91,7 @@ 
         $this->container['slug']        = Str::slug($this->argument('slug'));
         $this->container['name']        = Str::studly($this->container['slug']);
         $this->container['version']     = '1.0';
-        $this->container['description'] = 'This is the description for the ' . $this->container['name'] . ' module.';
+        $this->container['description'] = 'This is the description for the '.$this->container['name'].' module.';
         $this->container['location']    = config('modules.default_location');
         $this->container['provider']    = config("modules.locations.{$this->container['location']}.provider");
         $this->container['basename']    = Str::studly($this->container['slug']);
@@ -124,7 +124,7 @@ 
 
         $progress->finish();
 
-        event($this->container['slug'] . '.module.made');
+        event($this->container['slug'].'.module.made');
 
         $this->info("\nModule generated successfully.");
     }
@@ -144,7 +144,7 @@ 
         }
 
         $directory = module_path(null, $this->container['basename'], $location);
-        $source    = __DIR__ . '/Stubs/Module';
+        $source    = __DIR__.'/Stubs/Module';
 
         $this->files->makeDirectory($directory);
 
@@ -163,7 +163,7 @@ 
                 $subPath = str_replace($search, $replace, $subPath);
             }
 
-            $filePath = $directory . '/' . $subPath;
+            $filePath = $directory.'/'.$subPath;
             
             // if the file is module.json, replace it with the custom manifest file name
             if ($file->getFilename() === 'module.json' && $manifest) {
@@ -178,7 +178,7 @@ 
             
             $dir = dirname($filePath);
             
-            if (! $this->files->isDirectory($dir)) {
+            if (!$this->files->isDirectory($dir)) {
                 $this->files->makeDirectory($dir, 0755, true);
             }
 
@@ -213,22 +213,22 @@ 
         ];
 
         $replace = [
-            ucfirst($modelNameSingular) . 'Factory',
+            ucfirst($modelNameSingular).'Factory',
             $modelNameSingular,
             $this->container['slug'],
             ucfirst($modelName),
             ucfirst($modelNameSingular),
-            ucfirst($modelName) . 'DatabaseSeeder',
-            ucfirst($modelName) . 'TableSeeder',
-            ucfirst($modelNameSingular) . 'Controller',
-            ucfirst($modelNameSingular) . 'Service',
-            ucfirst($modelNameSingular) . 'ServiceInterface',
-            ucfirst($modelNameSingular) . 'Repository',
-            ucfirst($modelNameSingular) . 'RepositoryInterface',
-            ucfirst($modelName) . 'Errors',
-            'Store' . ucfirst($modelNameSingular),
+            ucfirst($modelName).'DatabaseSeeder',
+            ucfirst($modelName).'TableSeeder',
+            ucfirst($modelNameSingular).'Controller',
+            ucfirst($modelNameSingular).'Service',
+            ucfirst($modelNameSingular).'ServiceInterface',
+            ucfirst($modelNameSingular).'Repository',
+            ucfirst($modelNameSingular).'RepositoryInterface',
+            ucfirst($modelName).'Errors',
+            'Store'.ucfirst($modelNameSingular),
             ucfirst($modelNameSingular),
-            ucfirst($modelNameSingular) . 'Observer',
+            ucfirst($modelNameSingular).'Observer',
             Str::snake($modelName),
             $modelName,
         ];
@@ -252,6 +252,6 @@ 
         $modelName = $this->container['slug'];
         $migrationName = Str::camel($modelName);
         $migrationName = Str::snake($migrationName);
-        return $this->callSilent('make:module:migration', ['slug' => $modelName, 'name' => 'create_' . $migrationName . '_table']);
+        return $this->callSilent('make:module:migration', ['slug' => $modelName, 'name' => 'create_'.$migrationName.'_table']);
     }
 }

src/Modules/Core/Console/Commands/PassportInstallCommand.php

Spacing +1 added lines, -1 removed lines

@@ -32,7 +32,7 @@
     {
         $this->call('passport:keys', ['--force' => $this->option('force'), '--length' => $this->option('length')]);
         $oauthClient = \Core::oauthCLients()->first(['password_client' => 1]);
-        if (! $oauthClient) {
+        if (!$oauthClient) {
             $oauthClient = $client->createPasswordGrantClient(
                 null,
                 config('app.name'),

src/Modules/Core/BaseClasses/BaseService.php

Spacing +4 added lines, -4 removed lines

@@ -42,7 +42,7 @@ 
     {
         $translatable = $this->repo->model->translatable ?? [];
         $filters = $this->constructFilters($conditions, $local);
-        $sortBy = in_array($sortBy, $translatable) ? $sortBy . '->' . $local : $sortBy;
+        $sortBy = in_array($sortBy, $translatable) ? $sortBy.'->'.$local : $sortBy;
 
         if ($trashed) {
             return $this->deleted(['and' => $filters], $perPage ?? 15, $sortBy ?? 'created_at', $desc ?? true);
@@ -213,7 +213,7 @@ 
                 /**
                  * Prepare key based on the the requested lang if it was translatable.
                  */
-                $key = in_array($key, $translatable) ? $key . '->' . $local : $key;
+                $key = in_array($key, $translatable) ? $key.'->'.$local : $key;
 
                 /**
                  * Convert 0/1 or true/false to boolean in case of not foreign key.
@@ -248,11 +248,11 @@ 
                      * Default string filteration.
                      */
                 } elseif ($value) {
-                    $key = 'LOWER(' . $key . ')';
+                    $key = 'LOWER('.$key.')';
                     $value = strtolower($value);
                     $filters[$key] = [
                         'op' => 'like',
-                        'val' => '%' . $value . '%'
+                        'val' => '%'.$value.'%'
                     ];
                 }
             }