From 62aa2818362ef7b594063c2efefc3c466dd0c4e1 Mon Sep 17 00:00:00 2001
From: Vlyan <vlyan26@gmail.com>
Date: Tue, 22 Dec 2020 13:58:06 +0100
Subject: [PATCH] working on 20Q drag n drop

---
 .../scripts/actors/twenty-questions-dialog.js | 95 +++++++++++++++++--
 system/scripts/actors/twenty-questions.js     | 11 ++-
 .../actors/twenty-questions-dialog.html       | 24 ++---
 3 files changed, 110 insertions(+), 20 deletions(-)

diff --git a/system/scripts/actors/twenty-questions-dialog.js b/system/scripts/actors/twenty-questions-dialog.js
index 9607092..d96e74b 100644
--- a/system/scripts/actors/twenty-questions-dialog.js
+++ b/system/scripts/actors/twenty-questions-dialog.js
@@ -16,6 +16,11 @@ export class TwentyQuestionsDialog extends FormApplication {
      */
     errors = {};
 
+    /**
+     * Cache for items (techniques, adv...)
+     */
+    cache = {};
+
     /**
      * Assign the default options
      * @override
@@ -43,6 +48,7 @@ export class TwentyQuestionsDialog extends FormApplication {
         this.actor = actor;
         this.object = new TwentyQuestions(actor);
         this.errors = this.object.validateForm();
+        this._constructCache();
     }
 
     /**
@@ -90,6 +96,7 @@ export class TwentyQuestionsDialog extends FormApplication {
             data: this.object.data,
             errors: this.errors,
             hasErrors: Object.keys(this.errors).length > 0,
+            cache: this.cache,
         };
     }
 
@@ -123,6 +130,14 @@ export class TwentyQuestionsDialog extends FormApplication {
             return;
         }
 
+        // Delete a dnd element
+        html.find(`.property-delete`).on("click", (event) => {
+            const stepName = $(event.currentTarget).parents(".tq-drag-n-drop")[0].name;
+            const itemId = $(event.currentTarget).parents(".property").data("propertyId");
+            this._deleteOwnedItem(stepName, itemId);
+            this.render(false);
+        });
+
         // Submit button
         html.find("#generate").on("click", async (event) => {
             this.object.toActor(this.actor);
@@ -134,10 +149,13 @@ export class TwentyQuestionsDialog extends FormApplication {
      * Handle dropped items
      */
     _onDropItem(type, event) {
-        console.log("*** _onDrop event", event, type);
         if (!["item", "technique", "peculiarity"].includes(type)) {
             return;
         }
+        if (event.target.name === "undefined") {
+            console.log("event.target.name is undefined");
+            return;
+        }
 
         // Try to extract the data
         // {type: "Item", id: "pC37smMSCqu3aSRM"}
@@ -147,15 +165,18 @@ export class TwentyQuestionsDialog extends FormApplication {
             if (data.type !== "Item") {
                 return;
             }
-
+            // Get item
             const item = game.items.get(data.id);
-            if (item || item.data.type !== type) {
+            if (!item || item.data.type !== type) {
                 return;
             }
+            // Add the item (step and cache)
+            this._addOwnedItem(item, event.target.name);
 
-            // TODO
-            console.log("** OK ", item);
-            // sub_type === 'peculiarity'
+            // TODO specific event (no added honor if tech selected etc)
+            console.log(this.object.data.step3.techniques, this.cache);
+
+            this.render(false);
         } catch (err) {
             console.warn(err);
         }
@@ -188,4 +209,66 @@ export class TwentyQuestionsDialog extends FormApplication {
         }
         this.render(false);
     }
+
+    /**
+     * Construct the cache tree with Items full object
+     */
+    _constructCache() {
+        this.cache = {};
+        TwentyQuestions.itemsList.forEach((stepName) => {
+            // Check if current step value is a array
+            const store = getProperty(this.object.data, stepName);
+            if (!store || !Array.isArray(store)) {
+                setProperty(this.object.data, stepName, []);
+            }
+
+            // Init cache if not exist
+            if (!hasProperty(this.cache, stepName)) {
+                setProperty(this.cache, stepName, []);
+            }
+
+            // Get linked Item, and store it in cache
+            const step = getProperty(this.object.data, stepName);
+            step.forEach((id, idx) => {
+                const item = game.items.get(id);
+                if (!item) {
+                    step.splice(idx, 1);
+                    return;
+                }
+                getProperty(this.cache, stepName).push(item);
+            });
+        });
+    }
+
+    /**
+     * Add a owned item reference in step and cache
+     * @private
+     */
+    _addOwnedItem(item, stepName) {
+        // Add to Step (uniq id only)
+        const step = getProperty(this.object.data, stepName);
+        if (step.some((e) => e === item.id)) {
+            return;
+        }
+        step.push(item.id);
+
+        // Add to cache
+        getProperty(this.cache, stepName).push(item);
+    }
+
+    /**
+     * Delete a owned item reference in step and cache
+     * @private
+     */
+    _deleteOwnedItem(stepName, itemId) {
+        // Delete from current step
+        let step = getProperty(this.object.data, stepName);
+        step = step.filter((e) => e !== itemId);
+        setProperty(this.object.data, stepName, step);
+
+        // Delete from cache
+        let cache = getProperty(this.cache, stepName);
+        cache = cache.filter((e) => e.id !== itemId);
+        setProperty(this.cache, stepName, cache);
+    }
 }
diff --git a/system/scripts/actors/twenty-questions.js b/system/scripts/actors/twenty-questions.js
index da1ef99..2d82859 100644
--- a/system/scripts/actors/twenty-questions.js
+++ b/system/scripts/actors/twenty-questions.js
@@ -25,6 +25,11 @@ export class TwentyQuestions {
         "step17.skill",
     ];
 
+    /**
+     * Shortcut for all Items to build cache
+     */
+    static itemsList = ["step3.techniques"];
+
     /**
      * Steps datas
      */
@@ -62,9 +67,9 @@ export class TwentyQuestions {
                 maho: false,
                 ninjutsu: false,
             },
-            techniques: "",
-            school_ability: "",
-            equipment: "",
+            techniques: [],
+            school_ability: [],
+            equipment: [],
             social_honor: 0,
         },
         step4: {
diff --git a/system/templates/actors/twenty-questions-dialog.html b/system/templates/actors/twenty-questions-dialog.html
index 0bded43..b308d25 100644
--- a/system/templates/actors/twenty-questions-dialog.html
+++ b/system/templates/actors/twenty-questions-dialog.html
@@ -236,17 +236,19 @@
         </div>
 
         <div>
-            <label>
-                {{localize 'l5r5e.twenty_questions.startech'}}
-                <textarea name="step3.techniques">{{data.step3.techniques}}</textarea>
-            </label>
-            <!-- TODO Drag n drop techniques-->
-            <div class="techniques 20q-droppable">
-                <span class="techniques 20q-droppable-text">
-                    drop item here
-                </span>
-                {{data.step3.techniques}}
-            </div>
+
+            {{localize 'l5r5e.twenty_questions.startech'}}
+            <fieldset name="step3.techniques" class="tq-drag-n-drop">
+                <ul class="item-list">
+                    {{#each cache.step3.techniques as |item id|}}
+                    {{> 'systems/l5r5e/templates/items/property/property-entry.html' item=item id=id }}
+                    {{/each}}
+                </ul>
+            </fieldset>
+
+            <fieldset class="techniques" name="step3.techniques" style="min-height: 50px">
+                <legend class="section-header"><i class="fa fa-arrow-down" aria-hidden="true"></i> {{ localize 'l5r5e.drophere' }}</legend>
+            </fieldset>
 
         </div>