Porto-SAP / Hello-API

app/Containers/Email/Services/GenerateEmailConfirmationUrlService.php

Doc Comments +1 added lines, -1 removed lines

@@ -27,7 +27,7 @@
      * @param $email
      * @param $password
      *
-     * @return mixed
+     * @return string
      */
     public function run(User $user)
     {

app/Containers/Email/Services/SendConfirmationEmailService.php

Doc Comments +1 added lines, -1 removed lines

@@ -16,7 +16,7 @@
 
     /**
      * @param \App\Containers\User\Models\User $user
-     * @param                                  $confirmationUrl
+     * @param                                  string $confirmationUrl
      */
     public function run(User $user, $confirmationUrl)
     {

app/Containers/Payments/Services/PaymentsFactory.php

Doc Comments +1 added lines, -1 removed lines

@@ -23,7 +23,7 @@
 
     /**
      * @param \App\Containers\User\Models\User $user
-     * @param                                  $amount
+     * @param                                  integer $amount
      * @param string                           $currency
      *
      * @return  mixed

Unused Use Statements -1 removed lines

@@ -8,7 +8,6 @@
 use App\Containers\Payments\Contracts\Chargeable;
 use App\Containers\Payments\Exceptions\ObjectNonChargeableException;
 use App\Containers\Payments\Exceptions\PaymentMethodNotFoundException;
-use App\Containers\Payments\Exceptions\UserNotSetInThePaymentServiceException;
 use Illuminate\Support\Facades\App;
 
 /**

app/Containers/Stripe/Actions/CreateStripeAccountAction.php

Unused Use Statements -1 removed lines

@@ -6,7 +6,6 @@
 use App\Containers\Stripe\Repositories\Eloquent\StripeAccountRepository;
 use App\Containers\User\Models\User;
 use App\Port\Action\Abstracts\Action;
-use Auth;
 
 /**
  * Class CreateStripeAccountAction.

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 +1 added lines, -1 removed lines

@@ -57,7 +57,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/Exception/Abstracts/Exception.php

Doc Comments +1 added lines, -1 removed lines

@@ -40,7 +40,7 @@
     /**
      * Exception constructor.
      *
-     * @param null            $message
+     * @param string            $message
      * @param null            $errors
      * @param null            $statusCode
      * @param int             $code

app/Port/Provider/Traits/AutoRegisterServiceProvidersTrait.php

Unused Use Statements -2 removed lines

@@ -3,8 +3,6 @@
 namespace App\Port\Provider\Traits;
 
 use App;
-use DB;
-use Log;
 
 /**
  * Class AutoRegisterServiceProvidersTrait.

app/Port/Provider/Traits/PortServiceProviderTrait.php

Doc Comments +1 added lines

@@ -47,6 +47,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)
     {

Spacing +4 added lines, -4 removed lines

@@ -27,7 +27,7 @@ 
     {
         if (Config::get('database.query_debugging')) {
             // TODO: might not work in laravel 5.2 - check the events section of the upgrading giud and test it
-            DB::listen(function ($query, $bindings, $time, $connection) use ($terminal, $log) {
+            DB::listen(function($query, $bindings, $time, $connection) use ($terminal, $log) {
                 $fullQuery = vsprintf(str_replace(['%', '?'], ['%%', '%s'], $query), $bindings);
 
                 $text = $connection . ' (' . $time . '): ' . $fullQuery;
@@ -50,7 +50,7 @@ 
      */
     public function changeTheDefaultDatabaseModelsFactoriesPath($customPath)
     {
-        $this->app->singleton(\Illuminate\Database\Eloquent\Factory::class, function ($app) use ($customPath) {
+        $this->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);
@@ -81,7 +81,7 @@ 
             $allServiceProviders[] = PortButler::buildMainServiceProvider($containersNamespace, $containerName);
         }
 
-        return array_unique($allServiceProviders) ? : [];
+        return array_unique($allServiceProviders) ?: [];
     }
 
     /**
@@ -101,7 +101,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'));

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

Doc Comments +4 added lines, -4 removed lines

@@ -28,10 +28,10 @@ 
     public $loggedInTestingUser;
 
     /**
-     * @param           $endpoint
+     * @param           string $endpoint
      * @param string    $verb
      * @param array     $data
-     * @param bool|true $protected
+     * @param boolean $protected
      * @param array     $header
      *
      * @throws \Symfony\Component\Debug\Exception\UndefinedMethodException
@@ -116,7 +116,7 @@ 
     /**
      * @param null $userDetails
      *
-     * @return mixed
+     * @return App\Containers\User\Models\User
      */
     public function registerAndLoginTestingUser($userDetails = null)
     {
@@ -143,7 +143,7 @@ 
     }
 
     /**
-     * @param $keys
+     * @param string[] $keys
      * @param $response
      */
     public function assertResponseContainKeys($keys, $response)