From db0cf385fe707091f219cc61fb4e12b5b1fe64d0 Mon Sep 17 00:00:00 2001 From: Torsten Grote Date: Mon, 30 Mar 2020 14:17:09 -0300 Subject: Fix amount serialization in PoS app --- .../java/net/taler/merchantpos/history/RefundFragment.kt | 15 +++++++++++---- .../java/net/taler/merchantpos/history/RefundManager.kt | 14 ++++++++++---- .../java/net/taler/merchantpos/payment/PaymentManager.kt | 5 +++-- merchant-terminal/src/main/res/values/strings.xml | 1 + 4 files changed, 25 insertions(+), 10 deletions(-) (limited to 'merchant-terminal') diff --git a/merchant-terminal/src/main/java/net/taler/merchantpos/history/RefundFragment.kt b/merchant-terminal/src/main/java/net/taler/merchantpos/history/RefundFragment.kt index 609eadd..7652ca4 100644 --- a/merchant-terminal/src/main/java/net/taler/merchantpos/history/RefundFragment.kt +++ b/merchant-terminal/src/main/java/net/taler/merchantpos/history/RefundFragment.kt @@ -28,6 +28,8 @@ import androidx.navigation.fragment.findNavController import com.google.android.material.snackbar.BaseTransientBottomBar.LENGTH_LONG import com.google.android.material.snackbar.Snackbar import kotlinx.android.synthetic.main.fragment_refund.* +import net.taler.common.Amount +import net.taler.common.AmountParserException import net.taler.common.fadeIn import net.taler.common.fadeOut import net.taler.common.navigate @@ -52,7 +54,7 @@ class RefundFragment : Fragment() { override fun onViewCreated(view: View, savedInstanceState: Bundle?) { val item = refundManager.toBeRefunded ?: throw IllegalStateException() - amountInputView.setText(item.amount.toString()) + amountInputView.setText(item.amount.amountStr) currencyView.text = item.amount.currency abortButton.setOnClickListener { findNavController().navigateUp() } refundButton.setOnClickListener { onRefundButtonClicked(item) } @@ -63,12 +65,17 @@ class RefundFragment : Fragment() { } private fun onRefundButtonClicked(item: HistoryItem) { - val inputAmount = amountInputView.text.toString().toDouble() - if (inputAmount > item.amountStr.toDouble()) { // TODO real Amount comparision + val inputAmount = try { + Amount.fromString(item.amount.currency, amountInputView.text.toString()) + } catch (e: AmountParserException) { + amountView.error = getString(R.string.refund_error_invalid_amount) + return + } + if (inputAmount > item.amount) { amountView.error = getString(R.string.refund_error_max_amount, item.amountStr) return } - if (inputAmount <= 0.0) { + if (inputAmount.isZero()) { amountView.error = getString(R.string.refund_error_zero) return } diff --git a/merchant-terminal/src/main/java/net/taler/merchantpos/history/RefundManager.kt b/merchant-terminal/src/main/java/net/taler/merchantpos/history/RefundManager.kt index 270b3b8..910116e 100644 --- a/merchant-terminal/src/main/java/net/taler/merchantpos/history/RefundManager.kt +++ b/merchant-terminal/src/main/java/net/taler/merchantpos/history/RefundManager.kt @@ -24,6 +24,7 @@ import com.android.volley.Request.Method.POST import com.android.volley.RequestQueue import com.android.volley.Response.ErrorListener import com.android.volley.Response.Listener +import net.taler.common.Amount import net.taler.merchantpos.config.ConfigManager import net.taler.merchantpos.config.MerchantRequest import org.json.JSONObject @@ -34,7 +35,7 @@ sealed class RefundResult { class Success( val refundUri: String, val item: HistoryItem, - val amount: Double, + val amount: Amount, val reason: String ) : RefundResult() } @@ -44,6 +45,10 @@ class RefundManager( private val queue: RequestQueue ) { + companion object { + val TAG = RefundManager::class.java.simpleName + } + var toBeRefunded: HistoryItem? = null private set @@ -57,14 +62,15 @@ class RefundManager( } @UiThread - internal fun refund(item: HistoryItem, amount: Double, reason: String) { + internal fun refund(item: HistoryItem, amount: Amount, reason: String) { val merchantConfig = configManager.merchantConfig!! val refundRequest = mapOf( "order_id" to item.orderId, - "refund" to "${item.amount.currency}:$amount", + "refund" to amount.toJSONString(), "reason" to reason ) val body = JSONObject(refundRequest) + Log.d(TAG, body.toString(4)) val req = MerchantRequest(POST, merchantConfig, "refund", null, body, Listener { onRefundResponse(it, item, amount, reason) }, ErrorListener { onRefundError() } @@ -76,7 +82,7 @@ class RefundManager( private fun onRefundResponse( json: JSONObject, item: HistoryItem, - amount: Double, + amount: Amount, reason: String ) { if (!json.has("contract_terms")) { diff --git a/merchant-terminal/src/main/java/net/taler/merchantpos/payment/PaymentManager.kt b/merchant-terminal/src/main/java/net/taler/merchantpos/payment/PaymentManager.kt index f83370e..054d7cd 100644 --- a/merchant-terminal/src/main/java/net/taler/merchantpos/payment/PaymentManager.kt +++ b/merchant-terminal/src/main/java/net/taler/merchantpos/payment/PaymentManager.kt @@ -71,9 +71,9 @@ class PaymentManager( val merchantConfig = configManager.merchantConfig!! val currency = merchantConfig.currency!! - val amount = order.total.toJSONString() val summary = order.summary val summaryI18n = order.summaryI18n +// val refundDeadline = Timestamp(System.currentTimeMillis() + HOURS.toMillis(2)) mPayment.value = Payment(order, summary, currency) @@ -82,12 +82,13 @@ class PaymentManager( "${FULFILLMENT_PREFIX}${URLEncoder.encode(summary, "UTF-8")}#$fulfillmentId" val body = JSONObject().apply { put("order", JSONObject().apply { - put("amount", amount) + put("amount", order.total.toJSONString()) put("summary", summary) if (summaryI18n != null) put("summary_i18n", order.summaryI18n) // fulfillment_url needs to be unique per order put("fulfillment_url", fulfillmentUrl) put("instance", "default") +// put("refund_deadline", JSONObject(mapper.writeValueAsString(refundDeadline))) put("products", order.getProductsJson()) }) } diff --git a/merchant-terminal/src/main/res/values/strings.xml b/merchant-terminal/src/main/res/values/strings.xml index 863ae6f..756ef38 100644 --- a/merchant-terminal/src/main/res/values/strings.xml +++ b/merchant-terminal/src/main/res/values/strings.xml @@ -53,6 +53,7 @@ Abort Give Refund Greater than order amount of %s + Invalid amount Needs to be positive amount Error processing refund Refund deadline has passed -- cgit v1.2.3