diff options
Diffstat (limited to 'src')
-rw-r--r-- | src/operations/refund.ts | 10 | ||||
-rw-r--r-- | src/types/talerTypes.ts | 22 | ||||
-rw-r--r-- | src/util/codec.ts | 5 |
3 files changed, 7 insertions, 30 deletions
diff --git a/src/operations/refund.ts b/src/operations/refund.ts index b91d42a3f..35384c087 100644 --- a/src/operations/refund.ts +++ b/src/operations/refund.ts @@ -35,7 +35,6 @@ import { initRetryInfo, CoinStatus, RefundReason, - RefundEventRecord, RefundState, PurchaseRecord, } from "../types/dbTypes"; @@ -44,10 +43,10 @@ import { parseRefundUri } from "../util/taleruri"; import { createRefreshGroup, getTotalRefreshCost } from "./refresh"; import { Amounts } from "../util/amounts"; import { - codecForMerchantOrderStatus, MerchantCoinRefundStatus, MerchantCoinRefundSuccessStatus, MerchantCoinRefundFailureStatus, + codecForMerchantOrderStatusPaid, } from "../types/talerTypes"; import { guardOperationException } from "./errors"; import { getTimestampNow } from "../util/time"; @@ -414,14 +413,9 @@ async function processPurchaseQueryRefundImpl( const refundResponse = await readSuccessResponseJsonOrThrow( request, - codecForMerchantOrderStatus(), + codecForMerchantOrderStatusPaid(), ); - if (refundResponse.order_status !== "paid") { - logger.error("can't refund unpaid order"); - return; - } - await acceptRefunds( ws, proposalId, diff --git a/src/types/talerTypes.ts b/src/types/talerTypes.ts index 2730da4b2..1dfcd80a2 100644 --- a/src/types/talerTypes.ts +++ b/src/types/talerTypes.ts @@ -839,15 +839,7 @@ export interface ExchangeRevealResponse { ev_sigs: ExchangeRevealItem[]; } -export type MerchantOrderStatus = - | MerchantOrderStatusPaid - | MerchantOrderStatusUnpaid; - interface MerchantOrderStatusPaid { - /** - * Has the payment for this order (ever) been completed? - */ - order_status: "paid"; /** * Was the payment refunded (even partially, via refund or abort)? @@ -931,11 +923,6 @@ export interface MerchantCoinRefundFailureStatus { export interface MerchantOrderStatusUnpaid { /** - * Has the payment for this order (ever) been completed? - */ - order_status: "unpaid"; - - /** * URI that the wallet must process to complete the payment. */ taler_pay_uri: string; @@ -1250,7 +1237,6 @@ export const codecForMerchantOrderStatusPaid = (): Codec< MerchantOrderStatusPaid > => makeCodecForObject<MerchantOrderStatusPaid>() - .property("order_status", makeCodecForConstString("paid")) .property("merchant_pub", codecForString) .property("refund_amount", codecForString) .property("refunded", codecForBoolean) @@ -1261,14 +1247,6 @@ export const codecForMerchantOrderStatusUnpaid = (): Codec< MerchantOrderStatusUnpaid > => makeCodecForObject<MerchantOrderStatusUnpaid>() - .property("order_status", makeCodecForConstString("unpaid")) .property("taler_pay_uri", codecForString) .property("already_paid_order_id", makeCodecOptional(codecForString)) .build("MerchantOrderStatusUnpaid"); - -export const codecForMerchantOrderStatus = (): Codec<MerchantOrderStatus> => - makeCodecForUnion<MerchantOrderStatus>() - .discriminateOn("order_status") - .alternative("paid", codecForMerchantOrderStatusPaid()) - .alternative("unpaid", codecForMerchantOrderStatusUnpaid()) - .build("MerchantOrderStatus"); diff --git a/src/util/codec.ts b/src/util/codec.ts index c468704b2..383a2d99e 100644 --- a/src/util/codec.ts +++ b/src/util/codec.ts @@ -176,6 +176,11 @@ class UnionCodecBuilder< const baseCodec = this.baseCodec; return { decode(x: any, c?: Context): R { + if (!c) { + c = { + path: [`(${objectDisplayName})`], + }; + } const d = x[discriminator]; if (d === undefined) { throw new DecodingError( |