From a5f052d69c6457ad0289fdcb56398ea1fabedc2a Mon Sep 17 00:00:00 2001
From: Sebastian <sebasjm@gmail.com>
Date: Mon, 29 Aug 2022 11:32:07 -0300
Subject: using CTA for manual withdrawal

---
 .../src/cta/TermsOfServiceSection.tsx              |  28 ++-
 .../src/cta/Withdraw/index.ts                      |  12 +-
 .../src/cta/Withdraw/state.ts                      | 218 +++++++++++++++++++--
 .../src/cta/Withdraw/test.ts                       |  10 +-
 .../src/cta/Withdraw/views.tsx                     |  13 +-
 5 files changed, 239 insertions(+), 42 deletions(-)

(limited to 'packages/taler-wallet-webextension/src/cta')

diff --git a/packages/taler-wallet-webextension/src/cta/TermsOfServiceSection.tsx b/packages/taler-wallet-webextension/src/cta/TermsOfServiceSection.tsx
index 5decb1632..b60c86021 100644
--- a/packages/taler-wallet-webextension/src/cta/TermsOfServiceSection.tsx
+++ b/packages/taler-wallet-webextension/src/cta/TermsOfServiceSection.tsx
@@ -48,7 +48,9 @@ export function TermsOfServiceSection({
         return (
           <Fragment>
             {terms.status === "notfound" && (
-              <section>
+              <section
+                style={{ justifyContent: "space-around", display: "flex" }}
+              >
                 <WarningText>
                   <i18n.Translate>
                     Exchange doesn&apos;t have terms of service
@@ -62,7 +64,9 @@ export function TermsOfServiceSection({
       return (
         <Fragment>
           {terms.status === "notfound" && (
-            <section>
+            <section
+              style={{ justifyContent: "space-around", display: "flex" }}
+            >
               <WarningText>
                 <i18n.Translate>
                   Exchange doesn&apos;t have terms of service
@@ -71,7 +75,9 @@ export function TermsOfServiceSection({
             </section>
           )}
           {terms.status === "new" && (
-            <section>
+            <section
+              style={{ justifyContent: "space-around", display: "flex" }}
+            >
               <Button
                 variant="contained"
                 color="success"
@@ -84,7 +90,9 @@ export function TermsOfServiceSection({
             </section>
           )}
           {terms.status === "changed" && (
-            <section>
+            <section
+              style={{ justifyContent: "space-around", display: "flex" }}
+            >
               <Button
                 variant="contained"
                 color="success"
@@ -102,13 +110,13 @@ export function TermsOfServiceSection({
     return (
       <Fragment>
         {ableToReviewTermsOfService && (
-          <section>
+          <section style={{ justifyContent: "space-around", display: "flex" }}>
             <LinkSuccess upperCased onClick={() => onReview(true)}>
               <i18n.Translate>Show terms of service</i18n.Translate>
             </LinkSuccess>
           </section>
         )}
-        <section>
+        <section style={{ justifyContent: "space-around", display: "flex" }}>
           <CheckboxOutlined
             name="terms"
             enabled={reviewed}
@@ -129,7 +137,7 @@ export function TermsOfServiceSection({
   return (
     <Fragment>
       {terms.status !== "notfound" && !terms.content && (
-        <section>
+        <section style={{ justifyContent: "space-around", display: "flex" }}>
           <WarningBox>
             <i18n.Translate>
               The exchange reply with a empty terms of service
@@ -138,7 +146,7 @@ export function TermsOfServiceSection({
         </section>
       )}
       {terms.status !== "accepted" && terms.content && (
-        <section>
+        <section style={{ justifyContent: "space-around", display: "flex" }}>
           {terms.content.type === "xml" && (
             <TermsOfService>
               <ExchangeXmlTos doc={terms.content.document} />
@@ -160,14 +168,14 @@ export function TermsOfServiceSection({
         </section>
       )}
       {reviewed && ableToReviewTermsOfService && (
-        <section>
+        <section style={{ justifyContent: "space-around", display: "flex" }}>
           <LinkSuccess upperCased onClick={() => onReview(false)}>
             <i18n.Translate>Hide terms of service</i18n.Translate>
           </LinkSuccess>
         </section>
       )}
       {terms.status !== "notfound" && (
-        <section>
+        <section style={{ justifyContent: "space-around", display: "flex" }}>
           <CheckboxOutlined
             name="terms"
             enabled={reviewed}
diff --git a/packages/taler-wallet-webextension/src/cta/Withdraw/index.ts b/packages/taler-wallet-webextension/src/cta/Withdraw/index.ts
index b12e8df3b..7425dbd29 100644
--- a/packages/taler-wallet-webextension/src/cta/Withdraw/index.ts
+++ b/packages/taler-wallet-webextension/src/cta/Withdraw/index.ts
@@ -23,15 +23,20 @@ import * as wxApi from "../../wxApi.js";
 import {
   Props as TermsOfServiceSectionProps
 } from "../TermsOfServiceSection.js";
-import { useComponentState } from "./state.js";
+import { useComponentStateFromParams, useComponentStateFromURI } from "./state.js";
 import { CompletedView, LoadingExchangeView, LoadingInfoView, LoadingUriView, SuccessView } from "./views.js";
 
 
-export interface Props {
+export interface PropsFromURI {
   talerWithdrawUri: string | undefined;
   cancel: () => Promise<void>;
 }
 
+export interface PropsFromParams {
+  amount: string;
+  cancel: () => Promise<void>;
+}
+
 export type State =
   | State.Loading
   | State.LoadingUriError
@@ -93,4 +98,5 @@ const viewMapping: StateViewMap<State> = {
   success: SuccessView,
 };
 
-export const WithdrawPage = compose("Withdraw", (p: Props) => useComponentState(p, wxApi), viewMapping)
+export const WithdrawPageFromURI = compose("WithdrawPageFromURI", (p: PropsFromURI) => useComponentStateFromURI(p, wxApi), viewMapping)
+export const WithdrawPageFromParams = compose("WithdrawPageFromParams", (p: PropsFromParams) => useComponentStateFromParams(p, wxApi), viewMapping)
diff --git a/packages/taler-wallet-webextension/src/cta/Withdraw/state.ts b/packages/taler-wallet-webextension/src/cta/Withdraw/state.ts
index 849dd5cca..3b138e74d 100644
--- a/packages/taler-wallet-webextension/src/cta/Withdraw/state.ts
+++ b/packages/taler-wallet-webextension/src/cta/Withdraw/state.ts
@@ -15,16 +15,210 @@
  */
 
 
-import { Amounts } from "@gnu-taler/taler-util";
+import { Amounts, parsePaytoUri } from "@gnu-taler/taler-util";
 import { TalerError } from "@gnu-taler/taler-wallet-core";
 import { useState } from "preact/hooks";
 import { useAsyncAsHook } from "../../hooks/useAsyncAsHook.js";
 import { buildTermsOfServiceState } from "../../utils/index.js";
 import * as wxApi from "../../wxApi.js";
-import { Props, State } from "./index.js";
+import { PropsFromURI, PropsFromParams, State } from "./index.js";
 
-export function useComponentState(
-  { talerWithdrawUri, cancel }: Props,
+export function useComponentStateFromParams(
+  { amount, cancel }: PropsFromParams,
+  api: typeof wxApi,
+): State {
+
+  const [ageRestricted, setAgeRestricted] = useState(0);
+
+  const exchangeHook = useAsyncAsHook(api.listExchanges);
+
+  const exchangeHookDep =
+    !exchangeHook || exchangeHook.hasError || !exchangeHook.response
+      ? undefined
+      : exchangeHook.response;
+
+  const chosenAmount = Amounts.parseOrThrow(amount);
+
+  // get the first exchange with the currency as the default one
+  const exchange = exchangeHookDep ? exchangeHookDep.exchanges.find(e => e.currency === chosenAmount.currency) : undefined
+  /**
+   * For the exchange selected, bring the status of the terms of service
+   */
+  const terms = useAsyncAsHook(async () => {
+    if (!exchange) return undefined
+
+    const exchangeTos = await api.getExchangeTos(exchange.exchangeBaseUrl, ["text/xml"]);
+
+    const state = buildTermsOfServiceState(exchangeTos);
+
+    return { state };
+  }, [exchangeHookDep]);
+
+  /**
+   * With the exchange and amount, ask the wallet the information
+   * about the withdrawal
+   */
+  const amountHook = useAsyncAsHook(async () => {
+    if (!exchange) return undefined
+
+    const info = await api.getExchangeWithdrawalInfo({
+      exchangeBaseUrl: exchange.exchangeBaseUrl,
+      amount: chosenAmount,
+      tosAcceptedFormat: ["text/xml"],
+    });
+
+    const withdrawAmount = {
+      raw: Amounts.parseOrThrow(info.withdrawalAmountRaw),
+      effective: Amounts.parseOrThrow(info.withdrawalAmountEffective),
+    }
+
+    return { amount: withdrawAmount };
+  }, [exchangeHookDep]);
+
+  const [reviewing, setReviewing] = useState<boolean>(false);
+  const [reviewed, setReviewed] = useState<boolean>(false);
+
+  const [withdrawError, setWithdrawError] = useState<TalerError | undefined>(
+    undefined,
+  );
+  const [doingWithdraw, setDoingWithdraw] = useState<boolean>(false);
+  const [withdrawCompleted, setWithdrawCompleted] = useState<boolean>(false);
+
+  if (!exchangeHook) return { status: "loading", error: undefined }
+  if (exchangeHook.hasError) {
+    return {
+      status: "loading-uri",
+      error: exchangeHook,
+    };
+  }
+
+  if (!exchange) {
+    return {
+      status: "loading-exchange",
+      error: {
+        hasError: true,
+        operational: false,
+        message: "ERROR_NO-DEFAULT-EXCHANGE",
+      },
+    };
+  }
+
+  async function doWithdrawAndCheckError(): Promise<void> {
+    if (!exchange) return;
+
+    try {
+      setDoingWithdraw(true);
+
+      const response = await wxApi.acceptManualWithdrawal(
+        exchange.exchangeBaseUrl,
+        Amounts.stringify(amount),
+      );
+
+      setWithdrawCompleted(true);
+    } catch (e) {
+      if (e instanceof TalerError) {
+        setWithdrawError(e);
+      }
+    }
+    setDoingWithdraw(false);
+  }
+
+  if (!amountHook) {
+    return { status: "loading", error: undefined }
+  }
+  if (amountHook.hasError) {
+    return {
+      status: "loading-info",
+      error: amountHook,
+    };
+  }
+  if (!amountHook.response) {
+    return { status: "loading", error: undefined };
+  }
+  if (withdrawCompleted) {
+    return { status: "completed", error: undefined };
+  }
+
+  const withdrawalFee = Amounts.sub(
+    amountHook.response.amount.raw,
+    amountHook.response.amount.effective,
+  ).amount;
+  const toBeReceived = amountHook.response.amount.effective;
+
+  const { state: termsState } = (!terms
+    ? undefined
+    : terms.hasError
+      ? undefined
+      : terms.response) || { state: undefined };
+
+  async function onAccept(accepted: boolean): Promise<void> {
+    if (!termsState || !exchange) return;
+
+    try {
+      await api.setExchangeTosAccepted(
+        exchange.exchangeBaseUrl,
+        accepted ? termsState.version : undefined,
+      );
+      setReviewed(accepted);
+    } catch (e) {
+      if (e instanceof Error) {
+        //FIXME: uncomment this and display error
+        // setErrorAccepting(e.message);
+      }
+    }
+  }
+
+  const mustAcceptFirst =
+    termsState !== undefined &&
+    (termsState.status === "changed" || termsState.status === "new");
+
+  const ageRestrictionOptions: Record<string, string> | undefined = "6:12:18"
+    .split(":")
+    .reduce((p, c) => ({ ...p, [c]: `under ${c}` }), {});
+
+  if (ageRestrictionOptions) {
+    ageRestrictionOptions["0"] = "Not restricted";
+  }
+
+  //TODO: calculate based on exchange info
+  const ageRestrictionEnabled = false;
+  const ageRestriction = ageRestrictionEnabled ? {
+    list: ageRestrictionOptions,
+    value: String(ageRestricted),
+    onChange: async (v: string) => setAgeRestricted(parseInt(v, 10)),
+  } : undefined;
+
+  return {
+    status: "success",
+    error: undefined,
+    exchangeUrl: exchange.exchangeBaseUrl,
+    toBeReceived,
+    withdrawalFee,
+    chosenAmount,
+    ageRestriction,
+    doWithdrawal: {
+      onClick:
+        doingWithdraw || (mustAcceptFirst && !reviewed)
+          ? undefined
+          : doWithdrawAndCheckError,
+      error: withdrawError,
+    },
+    tosProps: !termsState
+      ? undefined
+      : {
+        onAccept,
+        onReview: setReviewing,
+        reviewed: reviewed,
+        reviewing: reviewing,
+        terms: termsState,
+      },
+    mustAcceptFirst,
+    cancel,
+  };
+}
+
+export function useComponentStateFromURI(
+  { talerWithdrawUri, cancel }: PropsFromURI,
   api: typeof wxApi,
 ): State {
   const [ageRestricted, setAgeRestricted] = useState(0);
@@ -50,21 +244,6 @@ export function useComponentState(
       ? undefined
       : uriInfoHook.response;
 
-  // const { amount, thisExchange } = useMemo(() => {
-  // if (!uriHookDep)
-  //   return {
-  //     amount: undefined,
-  //     thisExchange: undefined,
-  //     thisCurrencyExchanges: [],
-  //   };
-
-  // const { uriInfo } = uriHookDep;
-
-  // const amount = uriHookDep ? Amounts.parseOrThrow(uriHookDep.amount) : undefined;
-  // const thisExchange = uriHookDep?.thisExchange;
-
-  // return { amount, thisExchange };
-  // }, [uriHookDep]);
 
   /**
    * For the exchange selected, bring the status of the terms of service
@@ -118,6 +297,7 @@ export function useComponentState(
   }
 
   const { amount, thisExchange } = uriInfoHook.response
+
   const chosenAmount = Amounts.parseOrThrow(amount);
 
   if (!thisExchange) {
diff --git a/packages/taler-wallet-webextension/src/cta/Withdraw/test.ts b/packages/taler-wallet-webextension/src/cta/Withdraw/test.ts
index dd3f6c9c7..c72f906e5 100644
--- a/packages/taler-wallet-webextension/src/cta/Withdraw/test.ts
+++ b/packages/taler-wallet-webextension/src/cta/Withdraw/test.ts
@@ -27,7 +27,7 @@ import {
 import { ExchangeWithdrawDetails } from "@gnu-taler/taler-wallet-core";
 import { expect } from "chai";
 import { mountHook } from "../../test-utils.js";
-import { useComponentState } from "./state.js";
+import { useComponentStateFromURI } from "./state.js";
 
 const exchanges: ExchangeListItem[] = [
   {
@@ -56,7 +56,7 @@ describe("Withdraw CTA states", () => {
   it("should tell the user that the URI is missing", async () => {
     const { getLastResultOrThrow, waitNextUpdate, assertNoPendingUpdate } =
       mountHook(() =>
-        useComponentState({ talerWithdrawUri: undefined, cancel: async () => { null } }, {
+        useComponentStateFromURI({ talerWithdrawUri: undefined, cancel: async () => { null } }, {
           listExchanges: async () => ({ exchanges }),
           getWithdrawalDetailsForUri: async ({ talerWithdrawUri }: any) => ({
             amount: "ARS:2",
@@ -88,7 +88,7 @@ describe("Withdraw CTA states", () => {
   it("should tell the user that there is not known exchange", async () => {
     const { getLastResultOrThrow, waitNextUpdate, assertNoPendingUpdate } =
       mountHook(() =>
-        useComponentState({ talerWithdrawUri: "taler-withdraw://", cancel: async () => { null } }, {
+        useComponentStateFromURI({ talerWithdrawUri: "taler-withdraw://", cancel: async () => { null } }, {
           listExchanges: async () => ({ exchanges }),
           getWithdrawalDetailsForUri: async ({ talerWithdrawUri }: any) => ({
             amount: "EUR:2",
@@ -122,7 +122,7 @@ describe("Withdraw CTA states", () => {
   it("should be able to withdraw if tos are ok", async () => {
     const { getLastResultOrThrow, waitNextUpdate, assertNoPendingUpdate } =
       mountHook(() =>
-        useComponentState({ talerWithdrawUri: "taler-withdraw://", cancel: async () => { null } }, {
+        useComponentStateFromURI({ talerWithdrawUri: "taler-withdraw://", cancel: async () => { null } }, {
           listExchanges: async () => ({ exchanges }),
           getWithdrawalDetailsForUri: async ({ talerWithdrawUri }: any) => ({
             amount: "ARS:2",
@@ -188,7 +188,7 @@ describe("Withdraw CTA states", () => {
   it("should be accept the tos before withdraw", async () => {
     const { getLastResultOrThrow, waitNextUpdate, assertNoPendingUpdate } =
       mountHook(() =>
-        useComponentState({ talerWithdrawUri: "taler-withdraw://", cancel: async () => { null } }, {
+        useComponentStateFromURI({ talerWithdrawUri: "taler-withdraw://", cancel: async () => { null } }, {
           listExchanges: async () => ({ exchanges }),
           getWithdrawalDetailsForUri: async ({ talerWithdrawUri }: any) => ({
             amount: "ARS:2",
diff --git a/packages/taler-wallet-webextension/src/cta/Withdraw/views.tsx b/packages/taler-wallet-webextension/src/cta/Withdraw/views.tsx
index 5949076e0..850bd6e8f 100644
--- a/packages/taler-wallet-webextension/src/cta/Withdraw/views.tsx
+++ b/packages/taler-wallet-webextension/src/cta/Withdraw/views.tsx
@@ -122,14 +122,17 @@ export function SuccessView(state: State.Success): VNode {
             <div
               style={{
                 display: "flex",
+                alignItems: "center",
               }}
             >
               <i18n.Translate>Exchange</i18n.Translate>
-              <SvgIcon
-                title="Edit"
-                dangerouslySetInnerHTML={{ __html: editIcon }}
-                color="black"
-              />
+              <Link>
+                <SvgIcon
+                  title="Edit"
+                  dangerouslySetInnerHTML={{ __html: editIcon }}
+                  color="black"
+                />
+              </Link>
             </div>
           }
           text={<ExchangeDetails exchange={state.exchangeUrl} />}
-- 
cgit v1.2.3