Merge pull request #31 from devinweb/analysis-xg0A9N

Apply fixes from StyleCI

Author: darbaoui

config/hyperpay.php

@@ -29,5 +29,5 @@
      */
     'transaction_model' => 'Devinweb\LaravelHyperpay\Models\Transaction',
 
-    'notificationUrl' => '/hyperpay/webhook'
+    'notificationUrl' => '/hyperpay/webhook',
 ];

src/LaravelHyperpay.php

@@ -47,7 +47,6 @@ class LaravelHyperpay implements Hyperpay
      */
     protected $gateway_url = 'https://test.oppwa.com';
 
-
     /**
      * @var bool demand to register the user card
      */
@@ -180,15 +179,12 @@ public function paymentStatus(string $resourcePath, string $checkout_id)
         return $response;
     }
 
-    /**
-     *
-     */
     public function recurringPayment(string $registration_id, $amount, $checkout_id)
     {
         $result = (new HttpClient($this->client, $this->gateway_url.'/v1/registrations/'.$registration_id.'/payments', $this->config))->post(
             (new HttpParameters())->postRecurringPayment($amount, $this->redirect_url, $checkout_id),
         );
-        
+
         $response = (new HttpResponse($result, null, []))->recurringPayment();
 
         return $response;
@@ -221,7 +217,7 @@ public function addRedirectUrl($url)
     }
 
     /**
-     * Set the register user card information, to use it when we prepare the checkout
+     * Set the register user card information, to use it when we prepare the checkout.
      *
      * @return $this
      */

src/Support/HttpParameters.php

@@ -44,24 +44,25 @@ public function getParams($checkout_id): array
     }
 
     /**
-     * Generate the params that used in the recurring payment
-     * @param string $amount
-     * @param string $shopperResultUrl
-     * @param string $checkout_id That define the entity_id related to the registration id.
+     * Generate the params that used in the recurring payment.
+     *
+     * @param  string  $amount
+     * @param  string  $shopperResultUrl
+     * @param  string  $checkout_id  That define the entity_id related to the registration id.
      * @return array
      */
     public function postRecurringPayment($amount, $shopperResultUrl, $checkout_id)
     {
         $currency = config('hyperpay.currency');
 
         return array_merge([
-            "standingInstruction.mode"=>"REPEATED",
-            "standingInstruction.type" => "RECURRING",
-            "standingInstruction.source"=>"MIT",
-            "amount"=> $amount,
-            "currency" => $currency,
-            "paymentType"=>"PA",
-            "shopperResultUrl" => $shopperResultUrl
+            'standingInstruction.mode'=>'REPEATED',
+            'standingInstruction.type' => 'RECURRING',
+            'standingInstruction.source'=>'MIT',
+            'amount'=> $amount,
+            'currency' => $currency,
+            'paymentType'=>'PA',
+            'shopperResultUrl' => $shopperResultUrl,
         ], $this->getParams($checkout_id));
     }
 
@@ -94,16 +95,17 @@ protected function getBodyParameters($amount, Model $user, $heyPerPayConfig): ar
     }
 
     /**
-     * The init recurring payment params
+     * The init recurring payment params.
+     *
      * @return array
      */
     protected function registerPaymentData()
     {
         return [
-            "standingInstruction.mode"=>"INITIAL",
-            "standingInstruction.type" => "RECURRING",
-            "standingInstruction.source"=>"CIT",
-            "createRegistration"=>true,
+            'standingInstruction.mode'=>'INITIAL',
+            'standingInstruction.type' => 'RECURRING',
+            'standingInstruction.source'=>'CIT',
+            'createRegistration'=>true,
         ];
     }