PartFire / MangoPayBundle

DependencyInjection/PartFireMangoPayExtension.php

Spacing +1 added lines, -1 removed lines

@@ -22,7 +22,7 @@
         $configuration = new Configuration();
         $config = $this->processConfiguration($configuration, $configs);
 
-        $loader = new Loader\YamlFileLoader($container, new FileLocator(__DIR__.'/../Resources/config'));
+        $loader = new Loader\YamlFileLoader($container, new FileLocator(__DIR__ . '/../Resources/config'));
         $loader->load('services.yml');
     }
 }

Models/DTOs/Translators/TransferTranslator.php

Braces +12 added lines, -8 removed lines

@@ -50,17 +50,21 @@
         $transfer->setDebitedWalledId($mangoPayTransfer->DebitedWalletId);
         $transfer->setCreditedWalledId($mangoPayTransfer->CreditedWalletId);
 
-        if (isset($mangoPayTransfer->Fees->Currency))
-            $transfer->setFeeCurrency($mangoPayTransfer->Fees->Currency);
+        if (isset($mangoPayTransfer->Fees->Currency)) {
+                    $transfer->setFeeCurrency($mangoPayTransfer->Fees->Currency);
+        }
 
-        if (isset($mangoPayTransfer->Fees->Amount))
-            $transfer->setFeeAmount($mangoPayTransfer->Fees->Amount);
+        if (isset($mangoPayTransfer->Fees->Amount)) {
+                    $transfer->setFeeAmount($mangoPayTransfer->Fees->Amount);
+        }
 
-        if (isset($mangoPayTransfer->DebitedFunds->Currency))
-            $transfer->setDebitedCurrency($mangoPayTransfer->DebitedFunds->Currency);
+        if (isset($mangoPayTransfer->DebitedFunds->Currency)) {
+                    $transfer->setDebitedCurrency($mangoPayTransfer->DebitedFunds->Currency);
+        }
 
-        if (isset($mangoPayTransfer->DebitedFunds->Amount))
-            $transfer->setDebitedAmount($mangoPayTransfer->DebitedFunds->Amount);
+        if (isset($mangoPayTransfer->DebitedFunds->Amount)) {
+                    $transfer->setDebitedAmount($mangoPayTransfer->DebitedFunds->Amount);
+        }
 
         return $transfer;
     }

Models/Adapters/UserQuery.php

Spacing +1 added lines, -1 removed lines

@@ -36,7 +36,7 @@
         Logger $logger,
         UserTranslator $userTranslator
     ) {
-        parent::__construct($clientId, $clientPassword, $baseUrl,$mangoPayApi, $logger);
+        parent::__construct($clientId, $clientPassword, $baseUrl, $mangoPayApi, $logger);
         $this->userTranslator = $userTranslator;
     }
 

Doc Comments +2 added lines, -2 removed lines

@@ -51,8 +51,8 @@
     }
 
     /**
-     * @param User $userDto
-     * @return User|PartFireException
+     * @param UserBase $userDto
+     * @return \PartFire\MangoPayBundle\Models\DTOs\UserNatural
      */
     public function create(UserBase $userDto)
     {

Models/Adapters/TransferQuery.php

Spacing +1 added lines, -1 removed lines

@@ -36,7 +36,7 @@
         Logger $logger,
         UserTranslator $userTranslator
     ) {
-        parent::__construct($clientId, $clientPassword, $baseUrl,$mangoPayApi, $logger);
+        parent::__construct($clientId, $clientPassword, $baseUrl, $mangoPayApi, $logger);
         $this->userTranslator = $userTranslator;
     }
 

Models/Adapters/WalletQuery.php

Spacing +3 added lines, -3 removed lines

@@ -45,7 +45,7 @@ 
         Logger $logger,
         WalletTranslator $walletTranslator
     ) {
-        parent::__construct($clientId, $clientPassword, $baseUrl,$mangoPayApi, $logger);
+        parent::__construct($clientId, $clientPassword, $baseUrl, $mangoPayApi, $logger);
         $this->walletTranslator = $walletTranslator;
     }
 
@@ -54,10 +54,10 @@ 
         $mangoWallet = $this->walletTranslator->convertDTOToMangoPayWallet($walletDto);
         try {
             $mangoWallet = $this->mangoPayApi->Wallets->Create($mangoWallet);
-        } catch(ResponseException $e) {
+        } catch (ResponseException $e) {
             $this->logger->addCritical($e->getMessage(), ['code' => $e->getCode(), 'details' => $e->GetErrorDetails()]);
             return new PartFireException($e->getMessage(), $e->getCode());
-        } catch(Exception $e) {
+        } catch (Exception $e) {
             $this->logger->addError($e->getMessage());
             return new PartFireException($e->getMessage(), $e->getCode());
         }

Models/Adapters/KycDocumentPageQuery.php

Spacing +3 added lines, -3 removed lines

@@ -45,7 +45,7 @@ 
         Logger $logger,
         WalletTranslator $walletTranslator
     ) {
-        parent::__construct($clientId, $clientPassword, $baseUrl,$mangoPayApi, $logger);
+        parent::__construct($clientId, $clientPassword, $baseUrl, $mangoPayApi, $logger);
         $this->walletTranslator = $walletTranslator;
     }
 
@@ -54,10 +54,10 @@ 
         $mangoWallet = $this->walletTranslator->convertDTOToMangoPayWallet($walletDto);
         try {
             $mangoWallet = $this->mangoPayApi->Wallets->Create($mangoWallet);
-        } catch(ResponseException $e) {
+        } catch (ResponseException $e) {
             $this->logger->addCritical($e->getMessage(), ['code' => $e->getCode(), 'details' => $e->GetErrorDetails()]);
             return new PartFireException($e->getMessage(), $e->getCode());
-        } catch(Exception $e) {
+        } catch (Exception $e) {
             $this->logger->addError($e->getMessage());
             return new PartFireException($e->getMessage(), $e->getCode());
         }

Models/Adapters/KycDocumentQuery.php

Spacing +5 added lines, -5 removed lines

@@ -34,7 +34,7 @@ 
         KycDocumentTranslator $kycDocumentTranslator
     ) {
         $this->kycDocumentTranslator = $kycDocumentTranslator;
-        parent::__construct($clientId, $clientPassword, $baseUrl,$mangoPayApi, $logger);
+        parent::__construct($clientId, $clientPassword, $baseUrl, $mangoPayApi, $logger);
     }
 
     public function create(KycDocument $kycDocumentDto, $shouldSubmit = false)
@@ -47,10 +47,10 @@ 
             }
             $mangoKycDocument = $this->mangoPayApi->Users->CreateKycDocument($UserId, $mangoKycDocument);
 
-        } catch(ResponseException $e) {
+        } catch (ResponseException $e) {
             $this->logger->addCritical($e->getMessage(), ['code' => $e->getCode(), 'details' => $e->GetErrorDetails()]);
             return new PartFireException($e->getMessage(), $e->getCode());
-        } catch(Exception $e) {
+        } catch (Exception $e) {
             $this->logger->addError($e->getMessage());
             return new PartFireException($e->getMessage(), $e->getCode());
         }
@@ -66,10 +66,10 @@ 
     {
         try {
             $mangoKycDocument = $this->mangoPayApi->Users->GetKycDocument($kycDocumentId);
-        } catch(ResponseException $e) {
+        } catch (ResponseException $e) {
             $this->logger->addCritical($e->getMessage(), ['code' => $e->getCode(), 'details' => $e->GetErrorDetails()]);
             return new PartFireException($e->getMessage(), $e->getCode());
-        } catch(Exception $e) {
+        } catch (Exception $e) {
             $this->logger->addError($e->getMessage());
             return new PartFireException($e->getMessage(), $e->getCode());
         }

Models/DTOs/UserLegal.php

Doc Comments +5 added lines, -5 removed lines

@@ -55,7 +55,7 @@ 
     }
 
     /**
-     * @param mixed $firstName
+     * @param string $firstName
      */
     public function setFirstName($firstName)
     {
@@ -71,7 +71,7 @@ 
     }
 
     /**
-     * @param mixed $lastName
+     * @param string $lastName
      */
     public function setLastName($lastName)
     {
@@ -87,7 +87,7 @@ 
     }
 
     /**
-     * @param mixed $legalRepresentativeBirthday
+     * @param integer $legalRepresentativeBirthday
      */
     public function setLegalRepresentativeBirthday($legalRepresentativeBirthday)
     {
@@ -103,7 +103,7 @@ 
     }
 
     /**
-     * @param mixed $nationality
+     * @param string $nationality
      */
     public function setNationality($nationality)
     {
@@ -119,7 +119,7 @@ 
     }
 
     /**
-     * @param mixed $countryOfResidence
+     * @param string $countryOfResidence
      */
     public function setCountryOfResidence($countryOfResidence)
     {

Models/DTOs/Translators/UserTranslator.php

Unused Use Statements -2 removed lines

@@ -15,9 +15,7 @@
 use MangoPay\UserLegal;
 use MangoPay\UserNatural;
 use MangoPay\Address;
-
 use PartFire\MangoPayBundle\MangoPayConstants;
-use PartFire\MangoPayBundle\Models\DTOs\UserBase;
 use PartFire\MangoPayBundle\Models\DTOs\Address as PFAddress;
 use PartFire\MangoPayBundle\Models\DTOs\UserNatural as PFUserNatural;
 use PartFire\MangoPayBundle\Models\DTOs\UserLegal as PFUserLegal;