From 17c9efbc0db8d68ab83c05e2f80cbe9a51a35f37 Mon Sep 17 00:00:00 2001
From: Joan Sanchez <joan@verdnatura.es>
Date: Fri, 2 Aug 2019 08:27:55 +0200
Subject: [PATCH] postcode autocompletion fixes

---
 .../components/autocomplete/autocomplete.js   | 17 +++++-
 .../client/front/address/create/index.spec.js |  1 -
 modules/client/front/address/edit/index.html  |  3 +-
 .../client/front/address/edit/index.spec.js   |  1 -
 modules/client/front/fiscal-data/index.html   | 58 ++++++++++---------
 modules/client/front/fiscal-data/index.js     | 23 +++++---
 6 files changed, 62 insertions(+), 41 deletions(-)

diff --git a/front/core/components/autocomplete/autocomplete.js b/front/core/components/autocomplete/autocomplete.js
index 45e28350b..91a88704a 100755
--- a/front/core/components/autocomplete/autocomplete.js
+++ b/front/core/components/autocomplete/autocomplete.js
@@ -1,6 +1,7 @@
 import ngModule from '../../module';
 import Input from '../../lib/input';
 import assignProps from '../../lib/assign-props';
+import {mergeWhere} from 'vn-loopback/util/filter';
 import './style.scss';
 
 /**
@@ -163,6 +164,8 @@ export default class Autocomplete extends Input {
             else
                 where[this.valueField] = this._field;
 
+            where = mergeWhere(where, this.fetchFunction);
+
             let filter = {
                 fields: this.$.dropDown.getFields(),
                 where: where
@@ -292,6 +295,17 @@ export default class Autocomplete extends Input {
         this.assignDropdownProps();
         this.$.dropDown.show(this.input, search);
     }
+
+    get fetchFunction() {
+        return this._fetchFunction;
+    }
+
+    set fetchFunction(value) {
+        this._fetchFunction = value;
+
+        if (value)
+            this.refreshSelection();
+    }
 }
 Autocomplete.$inject = ['$element', '$scope', '$http', '$transclude', '$translate', '$interpolate'];
 
@@ -317,7 +331,8 @@ ngModule.component('vnAutocomplete', {
         order: '@?',
         limit: '<?',
         translateFields: '<?',
-        searchFunction: '&?'
+        searchFunction: '&?',
+        fetchFunction: '<?'
     },
     transclude: {
         tplItem: '?tplItem'
diff --git a/modules/client/front/address/create/index.spec.js b/modules/client/front/address/create/index.spec.js
index 45b009de7..e840ec2d8 100644
--- a/modules/client/front/address/create/index.spec.js
+++ b/modules/client/front/address/create/index.spec.js
@@ -71,7 +71,6 @@ describe('Client', () => {
 
                 expect(controller.address.city).toEqual('New York');
                 expect(controller.address.provinceFk).toEqual(1);
-                expect(controller.address.countryFk).toEqual(2);
             });
         });
     });
diff --git a/modules/client/front/address/edit/index.html b/modules/client/front/address/edit/index.html
index b54e98901..c2dce8f37 100644
--- a/modules/client/front/address/edit/index.html
+++ b/modules/client/front/address/edit/index.html
@@ -60,9 +60,10 @@
                 url="/api/Postcodes/location"
                 fields="['code', 'townFk']"
                 field="$ctrl.address.postalCode"
+                where="{townFk: town.selection.id}"
                 selection="$ctrl.postcodeSelection"
                 search-function="{code: $search}"
-                where="{townFk: town.selection.id}"
+                fetch-function="{townFk: town.selection.id}"
                 order="code, townFk"
                 show-field="code"
                 value-field="code"
diff --git a/modules/client/front/address/edit/index.spec.js b/modules/client/front/address/edit/index.spec.js
index 9ce61bfe2..33aacb672 100644
--- a/modules/client/front/address/edit/index.spec.js
+++ b/modules/client/front/address/edit/index.spec.js
@@ -79,7 +79,6 @@ describe('Client', () => {
 
                 expect(controller.address.city).toEqual('New York');
                 expect(controller.address.provinceFk).toEqual(1);
-                expect(controller.address.countryFk).toEqual(2);
             });
         });
     });
diff --git a/modules/client/front/fiscal-data/index.html b/modules/client/front/fiscal-data/index.html
index 04bd1815f..67d7705ce 100644
--- a/modules/client/front/fiscal-data/index.html
+++ b/modules/client/front/fiscal-data/index.html
@@ -28,12 +28,39 @@
             </vn-textfield>
         </vn-horizontal>
         <vn-horizontal>
+            <vn-autocomplete vn-id="country" vn-one
+                field="$ctrl.client.countryFk"
+                url="/api/Countries"
+                show-field="country"
+                value-field="id"
+                label="Country">
+            </vn-autocomplete>
+            <vn-autocomplete vn-id="province" vn-one
+                field="$ctrl.client.provinceFk"
+                url="/api/Provinces"
+                where="{countryFk: country.selection.id}"
+                show-field="name"
+                value-field="id"
+                label="Province">
+            </vn-autocomplete>
+        </vn-horizontal>
+        <vn-horizontal>
+            <vn-autocomplete vn-id="town" vn-one 
+                label="City"
+                url="/api/Towns"
+                fields="['id', 'name']"
+                where="{provinceFk: province.selection.id}"
+                show-field="name"
+                value-field="name"
+                field="$ctrl.client.city">
+            </vn-autocomplete>
             <vn-autocomplete vn-id="postcode" vn-one
-                field="$ctrl.client.postcode"
-                on-change="$ctrl.setLocation()"
-                search-function="{code: $search}"
                 url="/api/Postcodes/location"
                 fields="['code', 'townFk']"
+                field="$ctrl.client.postcode"
+                selection="$ctrl.postcodeSelection"
+                search-function="{code: $search}"
+                where="{townFk: town.selection.id}"
                 show-field="code"
                 value-field="code"
                 label="Postcode">
@@ -42,31 +69,6 @@
                     ({{town.province.country.country}})
                 </tpl-item>
             </vn-autocomplete>
-            <vn-autocomplete vn-one
-                label="City"
-                url="/api/Towns"
-                show-field="name"
-                value-field="name"
-                field="$ctrl.client.city">
-            </vn-autocomplete>
-        </vn-horizontal>
-        <vn-horizontal>
-            <vn-autocomplete vn-one
-                disabled="true"
-                field="$ctrl.client.provinceFk"
-                url="/api/Provinces"
-                show-field="name"
-                value-field="id"
-                label="Province">
-            </vn-autocomplete>
-            <vn-autocomplete vn-one
-                disabled="true"
-                field="$ctrl.client.countryFk"
-                url="/api/Countries"
-                show-field="country"
-                value-field="id"
-                label="Country">
-            </vn-autocomplete>
         </vn-horizontal>
         <vn-horizontal pad-small-v>
             <vn-check
diff --git a/modules/client/front/fiscal-data/index.js b/modules/client/front/fiscal-data/index.js
index 77a753f92..d44b3db66 100644
--- a/modules/client/front/fiscal-data/index.js
+++ b/modules/client/front/fiscal-data/index.js
@@ -57,16 +57,21 @@ export default class Controller {
         }
     }
 
-    setLocation() {
-        const location = this.$.postcode.selection;
-        if (!location || !location.town) return;
-        const town = location.town;
-        const province = town.province;
-        const country = province.country;
+    get postcodeSelection() {
+        return this._postcodeSelection;
+    }
 
-        this.client.city = location.town.name;
-        this.client.provinceFk = province.id;
-        this.client.countryFk = country.id;
+    set postcodeSelection(selection) {
+        const hasValue = this._postcodeSelection;
+        this._postcodeSelection = selection;
+
+        if (!selection || !hasValue) return;
+
+        const town = selection.town;
+        const province = town.province;
+
+        this.address.city = town.name;
+        this.address.provinceFk = province.id;
     }
 }
 Controller.$inject = ['$scope', '$http', 'vnApp', '$translate'];