From 62bc005534c8f83849291c29148af8160fa85f70 Mon Sep 17 00:00:00 2001
From: Pieter Vander Vennet <pietervdvn@posteo.net>
Date: Sun, 28 Jul 2024 04:00:13 +0200
Subject: [PATCH] Theme: tool_library: improve membership handling

---
 assets/layers/tool_library/tool_library.json | 28 +++++++++++++++++---
 1 file changed, 25 insertions(+), 3 deletions(-)

diff --git a/assets/layers/tool_library/tool_library.json b/assets/layers/tool_library/tool_library.json
index 015472846..c52d1ba93 100644
--- a/assets/layers/tool_library/tool_library.json
+++ b/assets/layers/tool_library/tool_library.json
@@ -52,6 +52,12 @@
           "then": {
             "en": "A <b>membership is required</b> to use this tool library"
           }
+        },
+        {
+          "if": "membership=optional",
+          "then": {
+            "en": "A <b>membership is possible</b> but not required to use this tool library"
+          }
         }
       ]
     },
@@ -72,18 +78,30 @@
         "en": "A membership costs {charge:membership}"
       }
     },
-
     {
       "question": {
         "en": "Is a fee asked to borrow tools?"
       },
       "id": "fee",
       "mappings": [
+        {
+          "if": {
+            "and": [
+              "fee=no",
+              "membership=required"
+            ]
+          },
+          "then": {
+            "en": "Borrowing tools is free if one has a membership"
+          },
+          "hideInAnswer": "membership!=required"
+        },
         {
           "if": "fee=no",
           "then": {
             "en": "Borrowing tools is free"
-          }
+          },
+          "hideInAnswer": "membership=required"
         },
         {
           "if": "fee=yes",
@@ -144,7 +162,11 @@
       "charge:membership": {
         "inverted": true,
         "quantity": "duration",
-        "denominations": ["years","months","weeks"]
+        "denominations": [
+          "years",
+          "months",
+          "weeks"
+        ]
       }
     }
   ]