cardinity / cardinity-sdk-php

src/Method/ResultObjectMapper.php

Spacing +1 added lines, -1 removed lines

@@ -73,7 +73,7 @@
     {
         $parts = explode('_', $field);
         array_map('ucfirst', $parts);
-        $name = 'set' . implode('', $parts);
+        $name = 'set'.implode('', $parts);
 
         return $name;
     }

src/Method/ResultObject.php

Spacing +2 added lines, -2 removed lines

@@ -106,7 +106,7 @@ 
     private function classGetters($class)
     {
         $methods = get_class_methods($class);
-        return array_filter($methods, function ($value) use ($methods) {
+        return array_filter($methods, function($value) use ($methods) {
             if ($value == 'getErrors') {
                 return false;
             }
@@ -134,6 +134,6 @@ 
         $parts = array_map('ucfirst', $parts);
         $property = implode('', $parts);
 
-        return 'set' . ucfirst($property);
+        return 'set'.ucfirst($property);
     }
 }

src/Exception/InvalidAttributeValue.php

Spacing +1 added lines, -1 removed lines

@@ -11,7 +11,7 @@
 
     public function __construct($message, ConstraintViolationListInterface $violations)
     {
-        $message .= ' Violations: ' . $violations->__toString();
+        $message .= ' Violations: '.$violations->__toString();
         parent::__construct($message);
 
         $this->violations = $violations;

src/Exception/Request.php

Spacing +1 added lines, -1 removed lines

@@ -15,7 +15,7 @@
      */
     public function __construct(\Exception $previous = null, ResultObjectInterface $result = null)
     {
-        $this->message .= ' Response data: ' . serialize($result);
+        $this->message .= ' Response data: '.serialize($result);
         parent::__construct($this->message, $this->code, $previous);
 
         $this->result = $result;

src/Method/Payment/Payment.php

Doc Comments +14 added lines, -14 removed lines

@@ -90,7 +90,7 @@ 
 
     /**
      * Gets the value of id.
-     * @return mixed
+     * @return string
      */
     public function getId()
     {
@@ -109,7 +109,7 @@ 
 
     /**
      * Gets the value of amount.
-     * @return mixed
+     * @return double
      */
     public function getAmount()
     {
@@ -128,7 +128,7 @@ 
 
     /**
      * Gets the value of currency.
-     * @return mixed
+     * @return string
      */
     public function getCurrency()
     {
@@ -147,7 +147,7 @@ 
 
     /**
      * Gets the value of created.
-     * @return mixed
+     * @return string
      */
     public function getCreated()
     {
@@ -166,7 +166,7 @@ 
 
     /**
      * Gets the value of type.
-     * @return mixed
+     * @return string
      */
     public function getType()
     {
@@ -185,7 +185,7 @@ 
 
     /**
      * Gets the value of live.
-     * @return mixed
+     * @return boolean
      */
     public function getLive()
     {
@@ -204,7 +204,7 @@ 
 
     /**
      * Gets the value of settle.
-     * @return mixed
+     * @return boolean
      */
     public function getSettle()
     {
@@ -223,7 +223,7 @@ 
 
     /**
      * Gets the value of status.
-     * @return mixed
+     * @return string
      */
     public function getStatus()
     {
@@ -242,7 +242,7 @@ 
 
     /**
      * Gets the value of error.
-     * @return mixed
+     * @return string
      */
     public function getError()
     {
@@ -261,7 +261,7 @@ 
 
     /**
      * Gets the value of orderId.
-     * @return mixed
+     * @return string
      */
     public function getOrderId()
     {
@@ -280,7 +280,7 @@ 
 
     /**
      * Gets the value of description.
-     * @return mixed
+     * @return string
      */
     public function getDescription()
     {
@@ -299,7 +299,7 @@ 
 
     /**
      * Gets the value of country.
-     * @return mixed
+     * @return string
      */
     public function getCountry()
     {
@@ -318,7 +318,7 @@ 
 
     /**
      * Gets the value of paymentMethod.
-     * @return mixed
+     * @return string
      */
     public function getPaymentMethod()
     {
@@ -357,7 +357,7 @@ 
     /**
      * @deprecated method is deprecated and shouldn't be used.
      * Gets the value of authorizeData.
-     * @return mixed
+     * @return string
      */
     public function getAuthorizeData()
     {

Unused Use Statements -1 removed lines

@@ -3,7 +3,6 @@
 namespace Cardinity\Method\Payment;
 
 use Cardinity\Method\ResultObject;
-use Cardinity\Method\Payment\ThreeDS2Data;
 use Cardinity\Method\Payment\ThreeDS2AuthorizationInformation;
 
 class Payment extends ResultObject

Spacing +1 added lines, -1 removed lines

@@ -405,7 +405,7 @@
      */
     public function setThreeds2Data(
         ThreeDS2AuthorizationInformation $threeDS2AuthorizationInformation
-    ){
+    ) {
         $this->threeDS2AuthorizationInformation = $threeDS2AuthorizationInformation;
     }
 

src/Method/Payment/Create.php

Braces +9 added lines, -3 removed lines

@@ -174,9 +174,15 @@
                 'message' => 'The value {{ value }} is not a valid {{ type }}.'
             ]),
         ];
-        if ($isRequired) array_unshift($inside_array, new Assert\NotBlank());
-        if ($length) array_push($inside_array, new Assert\Length($length));
-        if ($args) array_push($inside_array, $args);
+        if ($isRequired) {
+            array_unshift($inside_array, new Assert\NotBlank());
+        }
+        if ($length) {
+            array_push($inside_array, new Assert\Length($length));
+        }
+        if ($args) {
+            array_push($inside_array, $args);
+        }
         
         return $isRequired 
             ? new Assert\Required($inside_array)

Doc Comments +3 added lines

@@ -166,6 +166,9 @@
         ]); 
     }
 
+    /**
+     * @param string $typeValue
+     */
     private function buildElement($typeValue, bool $isRequired=false, $length=0, $args=0)
     {
         $inside_array = [

Spacing +5 added lines, -5 removed lines

@@ -43,11 +43,11 @@ 
     {
         return new Assert\Collection([
             'amount' =>  $this->buildElement('float', 1),
-            'currency' => $this->buildElement('string', 1, ['min' => 3,'max' => 3]),
+            'currency' => $this->buildElement('string', 1, ['min' => 3, 'max' => 3]),
             'settle' => $this->buildElement('bool'),
-            'order_id' => $this->buildElement('string', 0, ['min' => 2,'max' => 50]),
+            'order_id' => $this->buildElement('string', 0, ['min' => 2, 'max' => 50]),
             'description' => $this->buildElement('string', 0, ['max' => 255]),
-            'country' => $this->buildElement('string', 1, ['min' => 2,'max' => 2]),
+            'country' => $this->buildElement('string', 1, ['min' => 2, 'max' => 2]),
             'payment_method' => new Assert\Required([
                 new Assert\Type([
                     'type' => 'string',
@@ -80,7 +80,7 @@ 
                     ]),
                     'exp_year' => $this->buildElement(
                         'integer', 1, 
-                        ['min' => 4,'max' => 4],
+                        ['min' => 4, 'max' => 4],
                         new Assert\Range(['min' => date('Y')])
                     ),
                     'exp_month' => $this->buildElement('integer', 1),
@@ -166,7 +166,7 @@ 
         ]); 
     }
 
-    private function buildElement($typeValue, bool $isRequired=false, $length=0, $args=0)
+    private function buildElement($typeValue, bool $isRequired = false, $length = 0, $args = 0)
     {
         $inside_array = [
             new Assert\Type([

src/Method/Payment/Finalize.php

Spacing +1 added lines, -1 removed lines

@@ -16,7 +16,7 @@
      * @param STRING authorize data 'cres' or 'authorize_data'
      * @param BOOL should it be 3D secure v2 ?
      */
-    public function __construct(string $paymentId, string $authorizeData, $isV2=false)
+    public function __construct(string $paymentId, string $authorizeData, $isV2 = false)
     {
         $this->paymentId = $paymentId;
         $this->authorizeData = $authorizeData;