[taler-wallet-core] branch master updated: allow inline locations

[gnunet]

This is an automated email from the git hooks/post-receive script.

dold pushed a commit to branch master
in repository wallet-core.

The following commit(s) were added to refs/heads/master by this push:
     new e0c10a06 allow inline locations
e0c10a06 is described below

commit e0c10a06b1a1f2f91676f6846c4cf80af9e80b83
Author: Florian Dold <florian.dold@gmail.com>
AuthorDate: Sat Sep 5 18:09:48 2020 +0530

    allow inline locations
---
 packages/taler-wallet-core/src/types/talerTypes.ts | 85 +++++++++++++++++-----
 packages/taler-wallet-core/src/types/types-test.ts | 39 ++++++++++
 2 files changed, 106 insertions(+), 18 deletions(-)

diff --git a/packages/taler-wallet-core/src/types/talerTypes.ts 
b/packages/taler-wallet-core/src/types/talerTypes.ts
index b1061a1a..0b4afda1 100644
--- a/packages/taler-wallet-core/src/types/talerTypes.ts
+++ b/packages/taler-wallet-core/src/types/talerTypes.ts
@@ -265,10 +265,45 @@ export class AuditorHandle {
   url: string;
 }
 
+// Delivery location, losely modeled as a subset of
+// ISO20022's PostalAddress25.
+export interface Location {
+  // Nation with its own government.
+  country?: string;
+
+  // Identifies a subdivision of a country such as state, region, county.
+  country_subdivision?: string;
+
+  // Identifies a subdivision within a country sub-division.
+  district?: string;
+
+  // Name of a built-up area, with defined boundaries, and a local government.
+  town?: string;
+
+  // Specific location name within the town.
+  town_location?: string;
+
+  // Identifier consisting of a group of letters and/or numbers that
+  // is added to a postal address to assist the sorting of mail.
+  post_code?: string;
+
+  // Name of a street or thoroughfare.
+  street?: string;
+
+  // Name of the building or house.
+  building_name?: string;
+
+  // Number that identifies the position of a building on a street.
+  building_number?: string;
+
+  // Free-form address lines, should not exceed 7 elements.
+  address_lines?: string[];
+}
+
 export interface MerchantInfo {
   name: string;
-  jurisdiction: string | undefined;
-  address: string | undefined;
+  jurisdiction?: Location;
+  address?: Location;
 }
 
 export interface Tax {
@@ -306,12 +341,6 @@ export interface Product {
 
   // time indicating when this product should be delivered
   delivery_date?: Timestamp;
-
-  // where to deliver this product. This may be an URL for online delivery
-  // (i.e. 'http://example.com/download' or 'mailto:customer@example.com'),
-  // or a location label defined inside the proposition's 'locations'.
-  // The presence of a colon (':') indicates the use of an URL.
-  delivery_location?: string;
 }
 
 export interface InternationalizedString {
@@ -364,11 +393,6 @@ export class ContractTerms {
    */
   pay_deadline: Timestamp;
 
-  /**
-   * Delivery locations.
-   */
-  locations: any;
-
   /**
    * Maximum deposit fee covered by the merchant.
    */
@@ -384,6 +408,17 @@ export class ContractTerms {
    */
   merchant_pub: string;
 
+  /**
+   * Time indicating when the order should be delivered.
+   * May be overwritten by individual products.
+   */
+  delivery_date?: Timestamp;
+
+  /**
+   * Delivery location for (all!) products.
+   */
+  delivery_location?: Location;
+
   /**
    * List of accepted exchanges.
    */
@@ -1035,11 +1070,26 @@ export const codecForAuditorHandle = (): 
Codec<AuditorHandle> =>
     .property("url", codecForString())
     .build("AuditorHandle");
 
+
+export const codecForLocation = (): Codec<Location> =>
+  buildCodecForObject<Location>()
+    .property("country", codecOptional(codecForString()))
+    .property("country_subdivision", codecOptional(codecForString()))
+    .property("building_name", codecOptional(codecForString()))
+    .property("building_number", codecOptional(codecForString()))
+    .property("district", codecOptional(codecForString()))
+    .property("street", codecOptional(codecForString()))
+    .property("post_code", codecOptional(codecForString()))
+    .property("town", codecOptional(codecForString()))
+    .property("town_location", codecOptional(codecForString()))
+    .property("address_lines", codecOptional(codecForList(codecForString()))) 
+    .build("Location");
+
 export const codecForMerchantInfo = (): Codec<MerchantInfo> =>
   buildCodecForObject<MerchantInfo>()
     .property("name", codecForString())
-    .property("address", codecOptional(codecForString()))
-    .property("jurisdiction", codecOptional(codecForString()))
+    .property("address", codecOptional(codecForLocation()))
+    .property("jurisdiction", codecOptional(codecForLocation()))
     .build("MerchantInfo");
 
 export const codecForTax = (): Codec<Tax> =>
@@ -1063,8 +1113,6 @@ export const codecForProduct = (): Codec<Product> =>
     .property("quantity", codecOptional(codecForNumber()))
     .property("unit", codecOptional(codecForString()))
     .property("price", codecOptional(codecForString()))
-    .property("delivery_date", codecOptional(codecForTimestamp))
-    .property("delivery_location", codecOptional(codecForString()))
     .build("Tax");
 
 export const codecForContractTerms = (): Codec<ContractTerms> =>
@@ -1089,7 +1137,8 @@ export const codecForContractTerms = (): 
Codec<ContractTerms> =>
     .property("refund_deadline", codecForTimestamp)
     .property("wire_transfer_deadline", codecForTimestamp)
     .property("timestamp", codecForTimestamp)
-    .property("locations", codecForAny())
+    .property("delivery_location", codecOptional(codecForLocation()))
+    .property("delivery_date", codecOptional(codecForTimestamp))
     .property("max_fee", codecForString())
     .property("max_wire_fee", codecOptional(codecForString()))
     .property("merchant", codecForMerchantInfo())
diff --git a/packages/taler-wallet-core/src/types/types-test.ts 
b/packages/taler-wallet-core/src/types/types-test.ts
index afdc0184..183122ef 100644
--- a/packages/taler-wallet-core/src/types/types-test.ts
+++ b/packages/taler-wallet-core/src/types/types-test.ts
@@ -53,3 +53,42 @@ test("contract terms validation", (t) => {
 
   t.fail();
 });
+
+
+test("contract terms validation (locations)", (t) => {
+  const c = {
+    nonce: "123123123",
+    h_wire: "123",
+    amount: "EUR:1.5",
+    auditors: [],
+    exchanges: [{ master_pub: "foo", url: "foo" }],
+    fulfillment_url: "foo",
+    max_fee: "EUR:1.5",
+    merchant_pub: "12345",
+    merchant: {
+      name: "Foo",
+      address: {
+        country: "DE",
+      }
+    },
+    order_id: "test_order",
+    pay_deadline: { t_ms: 42 },
+    wire_transfer_deadline: { t_ms: 42 },
+    merchant_base_url: "https://example.com/pay";,
+    products: [],
+    refund_deadline: { t_ms: 42 },
+    summary: "hello",
+    timestamp: { t_ms: 42 },
+    wire_method: "test",
+    delivery_location: {
+      country: "FR",
+      town: "Rennes",
+    }
+  };
+
+  const r = codecForContractTerms().decode(c);
+
+  t.assert(r.merchant.address?.country === "DE");
+  t.assert(r.delivery_location?.country === "FR");
+  t.assert(r.delivery_location?.town === "Rennes");
+});

-- 
To stop receiving notification emails like this one, please contact
gnunet@gnunet.org.