taler-util: remove deprecated function

packages/merchant-backoffice-ui/src/paths/instance/templates/qr/QrPage.tsx

@@ -19,11 +19,9 @@
  * @author Sebastian Javier Marchano (sebasjm)
  */
 
-import { buildPayto, classifyTalerUri } from "@gnu-taler/taler-util";
 import { useTranslationContext } from "@gnu-taler/web-util/browser";
 import { h, VNode } from "preact";
 import { useState } from "preact/hooks";
-import { AsyncButton } from "../../../../components/exception/AsyncButton.js";
 import { QR } from "../../../../components/exception/QR.js";
 import {
   FormErrors,

packages/taler-util/src/taleruri.ts

@@ -168,66 +168,6 @@ export enum TalerUriAction {
   DevExperiment = "dev-experiment",
 }
 
-/**
- * Classify a taler:// URI.
- * @deprecated use parseTalerUri
- */
-export function classifyTalerUri(s: string): TalerUriType {
-  const sl = s.toLowerCase();
-  if (sl.startsWith("taler://restore/")) {
-    return TalerUriType.TalerRecovery;
-  }
-  if (sl.startsWith("taler+http://restore/")) {
-    return TalerUriType.TalerRecovery;
-  }
-  if (sl.startsWith("taler://pay/")) {
-    return TalerUriType.TalerPay;
-  }
-  if (sl.startsWith("taler+http://pay/")) {
-    return TalerUriType.TalerPay;
-  }
-  if (sl.startsWith("taler://pay-template/")) {
-    return TalerUriType.TalerPayTemplate;
-  }
-  if (sl.startsWith("taler+http://pay-template/")) {
-    return TalerUriType.TalerPayTemplate;
-  }
-  if (sl.startsWith("taler://tip/")) {
-    return TalerUriType.TalerTip;
-  }
-  if (sl.startsWith("taler+http://tip/")) {
-    return TalerUriType.TalerTip;
-  }
-  if (sl.startsWith("taler://refund/")) {
-    return TalerUriType.TalerRefund;
-  }
-  if (sl.startsWith("taler+http://refund/")) {
-    return TalerUriType.TalerRefund;
-  }
-  if (sl.startsWith("taler://withdraw/")) {
-    return TalerUriType.TalerWithdraw;
-  }
-  if (sl.startsWith("taler+http://withdraw/")) {
-    return TalerUriType.TalerWithdraw;
-  }
-  if (sl.startsWith(`taler://${talerActionPayPush}/`)) {
-    return TalerUriType.TalerPayPush;
-  }
-  if (sl.startsWith(`taler+http://${talerActionPayPush}/`)) {
-    return TalerUriType.TalerPayPush;
-  }
-  if (sl.startsWith(`taler://${talerActionPayPull}/`)) {
-    return TalerUriType.TalerPayPull;
-  }
-  if (sl.startsWith(`taler+http://${talerActionPayPull}/`)) {
-    return TalerUriType.TalerPayPull;
-  }
-  if (sl.startsWith("taler://dev-experiment/")) {
-    return TalerUriType.TalerDevExperiment;
-  }
-  return TalerUriType.Unknown;
-}
-
 interface TalerUriProtoInfo {
   innerProto: "http" | "https";
   rest: string;

packages/taler-wallet-cli/src/index.ts

@@ -21,7 +21,6 @@ import {
   AbsoluteTime,
   addPaytoQueryParams,
   AgeRestriction,
-  classifyTalerUri,
   codecForList,
   codecForString,
   CoreApiResponse,
@@ -32,13 +31,14 @@ import {
   j2s,
   Logger,
   parsePaytoUri,
+  parseTalerUri,
   PreparePayResultType,
   RecoveryMergeStrategy,
   sampleWalletCoreTransactions,
   setDangerousTimetravel,
   setGlobalLogLevelFromString,
   summarizeTalerErrorDetail,
-  TalerUriType,
+  TalerUriAction,
   WalletNotification,
 } from "@gnu-taler/taler-util";
 import { clk } from "@gnu-taler/taler-util/clk";
@@ -639,34 +639,32 @@ walletCli
       } else {
         uri = await readlinePrompt("Taler URI: ");
       }
-      const uriType = classifyTalerUri(uri);
+      const parsedTalerUri = parseTalerUri(uri);
-      switch (uriType) {
+      if (!parsedTalerUri) {
-        case TalerUriType.TalerPay:
+        throw Error("invalid taler URI");
+      }
+      switch (parsedTalerUri.type) {
+        case TalerUriAction.Pay:
           await doPay(wallet.client, uri, {
             alwaysYes: args.handleUri.autoYes,
           });
           break;
-        case TalerUriType.TalerTip:
+        case TalerUriAction.Tip: {
-          {
+          const res = await wallet.client.call(WalletApiOperation.PrepareTip, {
-            const res = await wallet.client.call(
-              WalletApiOperation.PrepareTip,
-              {
             talerTipUri: uri,
-              },
+          });
-            );
           console.log("tip status", res);
           await wallet.client.call(WalletApiOperation.AcceptTip, {
             walletTipId: res.walletTipId,
           });
-          }
           break;
-        case TalerUriType.TalerRefund:
+        }
+        case TalerUriAction.Refund:
           await wallet.client.call(WalletApiOperation.StartRefundQueryForUri, {
             talerRefundUri: uri,
           });
           break;
-        case TalerUriType.TalerWithdraw:
+        case TalerUriAction.Withdraw: {
-          {
           const withdrawInfo = await wallet.client.call(
             WalletApiOperation.GetWithdrawalDetailsForUri,
             {
@@ -693,16 +691,16 @@ walletCli
             },
           );
           console.log("accept withdrawal response", res);
-          }
           break;
-        case TalerUriType.TalerDevExperiment: {
+        }
+        case TalerUriAction.DevExperiment: {
           await wallet.client.call(WalletApiOperation.ApplyDevExperiment, {
             devExperimentUri: uri,
           });
           break;
         }
         default:
-          console.log(`URI type (${uriType}) not handled`);
+          console.log(`URI type (${parsedTalerUri}) not handled`);
           break;
       }
       return;
@@ -1683,3 +1681,6 @@ async function read(stream: NodeJS.ReadStream) {
 export function main() {
   walletCli.run();
 }
+function classifyTalerUri(uri: string) {
+  throw new Error("Function not implemented.");
+}