get amount directly

Author: recca0120

src/Gateway.php

@@ -44,7 +44,6 @@ public function getDefaultParameters()
     }
 
     /**
-     * @param  array  $options
      * @return RequestInterface
      */
     public function purchase(array $options = [])
@@ -53,7 +52,6 @@ public function purchase(array $options = [])
     }
 
     /**
-     * @param  array  $options
      * @return RequestInterface
      */
     public function completePurchase(array $options = [])
@@ -62,7 +60,6 @@ public function completePurchase(array $options = [])
     }
 
     /**
-     * @param  array  $options
      * @return RequestInterface|NotificationInterface
      */
     public function acceptNotification(array $options = [])
@@ -71,7 +68,6 @@ public function acceptNotification(array $options = [])
     }
 
     /**
-     * @param  array  $options
      * @return RequestInterface
      */
     public function fetchTransaction(array $options = [])
@@ -80,7 +76,6 @@ public function fetchTransaction(array $options = [])
     }
 
     /**
-     * @param  array  $options
      * @return RequestInterface
      */
     public function refund(array $options = [])

src/Message/FetchTransactionRequest.php

@@ -58,7 +58,6 @@ public function sendData($data)
     }
 
     /**
-     * @param $data
      * @return array
      *
      * @throws RtnException

src/Message/PurchaseRequest.php

@@ -6,6 +6,7 @@
 use Omnipay\Common\Exception\InvalidRequestException;
 use Omnipay\Common\Message\AbstractRequest;
 use Omnipay\ECPay\Item;
+use Omnipay\ECPay\Traits\HasAmount;
 use Omnipay\ECPay\Traits\HasATMFields;
 use Omnipay\ECPay\Traits\HasATMOrCVSOrBARCODEFields;
 use Omnipay\ECPay\Traits\HasCreditFields;
@@ -32,6 +33,7 @@ class PurchaseRequest extends AbstractRequest
     use HasATMFields;
     use HasCVSOrBARCODEFields;
     use HasATMOrCVSOrBARCODEFields;
+    use HasAmount;
 
     protected $liveEndpoint = 'https://payment.ecpay.com.tw/Cashier/AioCheckOut/V5';
 
@@ -125,12 +127,14 @@ private function prepareItems()
         $currency = $this->getCurrency() ?: 'TWD';
 
         if (! $items) {
-            return [new Item([
-                'Name' => $this->getDescription(),
-                'Price' => $this->getAmount(),
-                'Currency' => $currency,
-                'Quantity' => 1,
-            ])];
+            return [
+                new Item([
+                    'Name' => $this->getDescription(),
+                    'Price' => $this->getAmount(),
+                    'Currency' => $currency,
+                    'Quantity' => 1,
+                ]),
+            ];
         }
 
         return array_map(static function ($item) use ($currency) {

src/Message/RefundRequest.php

@@ -4,6 +4,7 @@
 
 use Ecpay\Sdk\Exceptions\RtnException;
 use Omnipay\Common\Message\AbstractRequest;
+use Omnipay\ECPay\Traits\HasAmount;
 use Omnipay\ECPay\Traits\HasDefaults;
 use Omnipay\ECPay\Traits\HasECPay;
 use Omnipay\ECPay\Traits\HasMerchantTradeNo;
@@ -15,6 +16,7 @@ class RefundRequest extends AbstractRequest
     use HasDefaults;
     use HasMerchantTradeNo;
     use HasTotalAmount;
+    use HasAmount;
 
     public function setTradeNo($value)
     {

src/Traits/HasAmount.php

@@ -0,0 +1,11 @@
+<?php
+
+namespace Omnipay\ECPay\Traits;
+
+trait HasAmount
+{
+    public function getAmount()
+    {
+        return $this->getParameter('amount');
+    }
+}

tests/Message/AcceptNotificationRequestTest.php

@@ -52,8 +52,6 @@ public function testGetData()
 
     /**
      * @depends testGetData
-     *
-     * @param $results
      */
     public function testSendData($results)
     {

tests/Message/CompletePurchaseRequestTest.php

@@ -51,8 +51,6 @@ public function testGetData()
 
     /**
      * @depends testGetData
-     *
-     * @param $result
      */
     public function testSendData($result)
     {

tests/Message/FetchTransactionRequestTest.php

@@ -35,8 +35,6 @@ public function testGetData()
 
     /**
      * @depends testGetData
-     *
-     * @param $result
      */
     public function testSendData($result)
     {

tests/Message/RefundRequestTest.php

@@ -22,7 +22,7 @@ public function testGetData()
             'MerchantTradeNo' => '2821567410556',
             'TradeNo' => '1909021549160081',
             'Action' => 'R',
-            'TotalAmount' => '1000.00',
+            'TotalAmount' => '1000',
         ], $request->getData());
     }
 }

tests/Message/VoidRequestTest.php

@@ -22,7 +22,7 @@ public function testGetData()
             'MerchantTradeNo' => '2821567410556',
             'TradeNo' => '1909021549160081',
             'Action' => 'N',
-            'TotalAmount' => '1000.00',
+            'TotalAmount' => '1000',
         ], $request->getData());
     }
 }