Merge pull request #10 from open-source-contributions/issue_#8

Resolves issue #8
zhooravell · Jul 1, 2020 · 56790ac · 56790ac
2 parents 3dbb2e3 + e058392
commit 56790ac
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 2 deletions.
diff --git a/.travis.yml b/.travis.yml
@@ -10,6 +10,7 @@ before_script:
   - travis_retry composer install --no-interaction --prefer-source
 
 script:
+  - vendor/bin/phpcs --standard=PSR2 src tests
   - vendor/bin/phpunit --coverage-clover=coverage.xml
   - vendor/bin/infection -s --threads=4
 

diff --git a/composer.json b/composer.json
@@ -14,6 +14,7 @@
     },
     "require-dev": {
         "infection/infection": "^0.15.3",
+        "squizlabs/php_codesniffer": "^3.5",
         "phpunit/phpunit": "^8.0"
     },
     "autoload": {

diff --git a/src/Factory/ResponseFactory.php b/src/Factory/ResponseFactory.php
@@ -44,8 +44,7 @@ public static function createFromRefundResponse(ResponseInterface $response): Re
     {
         $xml = self::responseToXML($response);
 
-        if (
-            !$xml->xpath('transaction_id') ||
+        if (!$xml->xpath('transaction_id') ||
             !$xml->xpath('mb_amount') ||
             !$xml->xpath('mb_currency') ||
             !$xml->xpath('status')