Porto-SAP / Hello-API

app/Port/Email/Abstracts/MailsAbstract.php

Doc Comments -1 removed lines

@@ -37,7 +37,6 @@
     /**
      * MailsAbstract constructor.
      *
-     * @param \Illuminate\Mail\Mailer $mail
      */
     public function __construct()
     {

Spacing +2 added lines, -2 removed lines

@@ -61,7 +61,7 @@ 
      */
     public function send($data = [])
     {
-        if(!$this->fromEmail || !$this->toEmail){
+        if (!$this->fromEmail || !$this->toEmail) {
             throw new EmailIsMissedException();
         }
 
@@ -70,7 +70,7 @@ 
         // check if sending emails is enabled and if this is not running a testing environment
         if (Config::get('mail.enabled')) {
 
-            Mail::queue('EmailTemplates.' . $this->template, $data, function ($m) {
+            Mail::queue('EmailTemplates.' . $this->template, $data, function($m) {
                 $m->from($this->fromEmail, $this->fromName);
                 $m->to($this->toEmail, $this->toName)
                     ->subject($this->subject);

app/Port/Criterias/Eloquent/CountCriteria.php

Spacing +1 added lines, -1 removed lines

@@ -37,6 +37,6 @@
      */
     public function apply($model, PrettusRepositoryInterface $repository)
     {
-        return DB::table($model->getModel()->getTable())->select('*', DB::raw('count('.$this->field.') as total_count'))->groupBy($this->field);
+        return DB::table($model->getModel()->getTable())->select('*', DB::raw('count(' . $this->field . ') as total_count'))->groupBy($this->field);
     }
 }

app/Port/Broadcast/Providers/BroadcastServiceProvider.php

Spacing +2 added lines, -2 removed lines

@@ -19,8 +19,8 @@
         /*
          * Authenticate the user's personal channel...
          */
-        Broadcast::channel('App.User.*', function ($user, $userId) {
-            return (int) $user->id === (int) $userId;
+        Broadcast::channel('App.User.*', function($user, $userId) {
+            return (int)$user->id === (int)$userId;
         });
     }
 }

app/Port/Middleware/Http/Localization.php

Spacing +1 added lines, -1 removed lines

@@ -38,7 +38,7 @@
         $locale = $request->header('Content-Language');
 
         // if the header is missed
-        if(!$locale){
+        if (!$locale) {
             // take the default local language
             $locale = $this->app->config->get('app.locale');
         }

app/Port/Provider/Providers/MainServiceProvider.php

Spacing +1 added lines, -1 removed lines

@@ -18,7 +18,7 @@
      */
     public function register()
     {
-        $this->app->bind('PortButler', function () {
+        $this->app->bind('PortButler', function() {
             return $this->app->make(PortButler::class);
         });
     }

app/Port/Provider/Traits/PortServiceProviderTrait.php

Doc Comments +2 added lines, -1 removed lines

@@ -48,6 +48,7 @@ 
      * By default Laravel takes (server/database/factories) as the
      * path to the factories, this function changes the path to load
      * the factories from the infrastructure directory.
+     * @param string $customPath
      */
     public function changeTheDefaultDatabaseModelsFactoriesPath($customPath)
     {
@@ -138,7 +139,7 @@ 
     }
 
     /**
-     * @param $directory
+     * @param string $directory
      */
     private function loadConfigs($directory)
     {

Spacing +6 added lines, -6 removed lines

@@ -29,7 +29,7 @@ 
     public function debugDatabaseQueries($log = true, $terminal = false)
     {
         if (Config::get('database.query_debugging')) {
-            DB::listen(function ($event) use ($terminal, $log) {
+            DB::listen(function($event) use ($terminal, $log) {
                 $fullQuery = vsprintf(str_replace(['%', '?'], ['%%', '%s'], $event->sql), $event->bindings);
 
                 $text = $event->connectionName . ' (' . $event->time . '): ' . $fullQuery;
@@ -52,7 +52,7 @@ 
      */
     public function changeTheDefaultDatabaseModelsFactoriesPath($customPath)
     {
-        App::singleton(\Illuminate\Database\Eloquent\Factory::class, function ($app) use ($customPath) {
+        App::singleton(\Illuminate\Database\Eloquent\Factory::class, function($app) use ($customPath) {
             $faker = $app->make(\Faker\Generator::class);
 
             return \Illuminate\Database\Eloquent\Factory::construct($faker, base_path() . $customPath);
@@ -75,7 +75,7 @@ 
             $allServiceProviders[] = PortButler::buildMainServiceProvider($containersNamespace, $containerName);
         }
 
-        return array_unique($allServiceProviders) ? : [];
+        return array_unique($allServiceProviders) ?: [];
     }
 
     /**
@@ -104,7 +104,7 @@ 
 
             if (File::isDirectory($containerMigrationDirectory)) {
 
-                App::afterResolving('migrator', function ($migrator) use ($containerMigrationDirectory) {
+                App::afterResolving('migrator', function($migrator) use ($containerMigrationDirectory) {
                     foreach ((array)$containerMigrationDirectory as $path) {
                         $migrator->path($path);
                     }
@@ -201,7 +201,7 @@ 
 
         // if DataArray `\League\Fractal\Serializer\DataArraySerializer` do noting since it's set by default by the Dingo API
         if ($serializerName !== 'DataArray') {
-            app('Dingo\Api\Transformer\Factory')->setAdapter(function () use ($serializerName) {
+            app('Dingo\Api\Transformer\Factory')->setAdapter(function() use ($serializerName) {
                 switch ($serializerName) {
                     case 'JsonApi':
                         $serializer = new \League\Fractal\Serializer\JsonApiSerializer(Config::get('api.domain'));
@@ -234,7 +234,7 @@ 
             ->registerMiddlewareGroups($middlewareGroups);
 
         // Registering Route Middleware's
-        foreach ($routeMiddlewares as $key => $routeMiddleware){
+        foreach ($routeMiddlewares as $key => $routeMiddleware) {
             $this->app['router']->middleware($key, $routeMiddleware);
         }
 

app/Port/Configs/repository.php

Spacing +1 added lines, -1 removed lines

@@ -220,7 +220,7 @@
     */
     'generator'  => [
         'basePath'      => env('SRC_PATH'),
-        'rootNamespace' => env('ROOT_NAMESPACE').'\\',
+        'rootNamespace' => env('ROOT_NAMESPACE') . '\\',
         'paths'         => [
             'models'       => 'Entities',
             'repositories' => 'Repositories',

app/Port/Routes/Traits/RoutesServiceProviderTrait.php

Doc Comments +1 added lines, -1 removed lines

@@ -113,7 +113,7 @@
     /**
      * @param \Symfony\Component\Finder\SplFileInfo $file
      *
-     * @return  mixed
+     * @return  string
      */
     private function getRouteFileNameWithoutExtension(SplFileInfo $file)
     {

Spacing +4 added lines, -4 removed lines

@@ -55,7 +55,7 @@ 
                 $apiVersionNumber = $this->getRouteFileVersionNumber($file);
 
                 $this->apiRouter->version('v' . $apiVersionNumber,
-                    function (DingoApiRouter $router) use ($file, $containerPath, $containersNamespace) {
+                    function(DingoApiRouter $router) use ($file, $containerPath, $containersNamespace) {
 
                         $controllerNamespace = $containersNamespace . '\\Containers\\' . basename($containerPath) . '\\UI\API\Controllers';
 
@@ -68,7 +68,7 @@ 
                             'limit'      => Config::get('api.limit'),
                             // The API limit expiry time.
                             'expires'    => Config::get('api.limit_expires'),
-                        ], function ($router) use ($file) {
+                        ], function($router) use ($file) {
 
                             require $file->getPathname();
 
@@ -102,7 +102,7 @@ 
                 $this->webRouter->group([
                     'middleware' => ['web'],
                     'namespace'  => $controllerNamespace,
-                ], function (LaravelRouter $router) use ($file) {
+                ], function(LaravelRouter $router) use ($file) {
                     require $file->getPathname();
                 });
             }
@@ -143,7 +143,7 @@ 
     protected function hashIds()
     {
         if (Config::get('hello.hash-id')) {
-            Route::bind('id', function ($id, $route) {
+            Route::bind('id', function($id, $route) {
                 return Hashids::decode($id)[0];
             });
         }

app/Port/Tests/PHPUnit/Traits/TestingTrait.php

Doc Comments +2 added lines, -2 removed lines

@@ -32,13 +32,13 @@
     public $loggedInTestingUser;
 
     /**
-     * @param        $endpoint
+     * @param        string $endpoint
      * @param string $verb
      * @param array  $data
      * @param bool   $protected
      * @param array  $headers
      *
-     * @return  mixed
+     * @return  Response
      * @throws \Symfony\Component\Debug\Exception\UndefinedMethodException
      */
     public function apiCall($endpoint, $verb = 'get', array $data = [], $protected = true, array $headers = [])

Unused Use Statements -1 removed lines

@@ -7,7 +7,6 @@
 use App\Containers\Authorization\Tasks\AttachRoleTask;
 use App\Containers\User\Actions\CreateUserAction;
 use Dingo\Api\Http\Response as DingoAPIResponse;
-
 use Illuminate\Http\Response;
 use Illuminate\Http\UploadedFile;
 use Illuminate\Support\Arr as LaravelArr;

Spacing +1 added lines, -1 removed lines

@@ -355,7 +355,7 @@
      */
     public function injectEndpointId($endpoint, $id)
     {
-        if(Config::get('hello.hash-id')){
+        if (Config::get('hello.hash-id')) {
             $id = Hashids::encode($id);
         }