From ae3f9db94807fa4f551ec61659fe03fb0ceb5107 Mon Sep 17 00:00:00 2001
From: nelo <nelo@verdnatura.es>
Date: Wed, 17 May 2017 08:19:03 +0200
Subject: [PATCH 1/2] =?UTF-8?q?VnAutocomplete=20ya=20no=20carga=20los=20da?=
 =?UTF-8?q?tos=20cuando=20los=20datos=20se=20pueden=20bandera.=20Creado=20?=
 =?UTF-8?q?el=20m=C3=A9todo=20card=20en=20client=20para=20listar=20el=20cl?=
 =?UTF-8?q?iente=20con=20sus=20datos=20para=20el=20combo.?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 client/client/src/basic-data/index.html   |  2 ++
 client/client/src/card/index.js           |  2 +-
 client/client/src/fiscal-data/index.html  |  3 ++
 client/core/src/autocomplete/index.js     | 22 +++++++++---
 services/client/common/models/Client.js   |  5 +--
 services/client/common/models/Client.json | 11 ++----
 services/client/common/scopes/card.js     | 39 ++++++++++++++++++++
 services/client/common/scopes/card.json   | 43 +++++++++++++++++++++++
 8 files changed, 110 insertions(+), 17 deletions(-)
 create mode 100644 services/client/common/scopes/card.js
 create mode 100644 services/client/common/scopes/card.json

diff --git a/client/client/src/basic-data/index.html b/client/client/src/basic-data/index.html
index 8e089af4b6..9fad9255b6 100644
--- a/client/client/src/basic-data/index.html
+++ b/client/client/src/basic-data/index.html
@@ -22,6 +22,7 @@
             <vn-horizontal>
                 <vn-textfield vn-one label="Email" field="$ctrl.client.email"></vn-textfield>
                 <vn-autocomplete vn-one
+                    initial-value="$ctrl.client.salesPerson"
                     field="$ctrl.client.salesPersonFk"
                     url="/client/api/Employees"
                     show-field="name"
@@ -29,6 +30,7 @@
                     label="Comercial">
                 </vn-autocomplete>
                 <vn-autocomplete vn-one
+                    initial-value="$ctrl.client.contactChannel"
                     field="$ctrl.client.contactChannelFk"
                     url="/client/api/ContactChannels"
                     show-field="name"
diff --git a/client/client/src/card/index.js b/client/client/src/card/index.js
index 514950de79..a3c21dca4d 100644
--- a/client/client/src/card/index.js
+++ b/client/client/src/card/index.js
@@ -7,7 +7,7 @@ export const COMPONENT = {
     controllerAs: 'card',
     controller: function($http, $stateParams) {
         this.client = null;
-        $http.get(`/client/api/Clients/${$stateParams.id}?filter[include][account]`).then(
+        $http.get(`/client/api/Clients/${$stateParams.id}/card`).then(
             json => this.client = json.data
         );
 
diff --git a/client/client/src/fiscal-data/index.html b/client/client/src/fiscal-data/index.html
index acd389825a..8e74ad5712 100644
--- a/client/client/src/fiscal-data/index.html
+++ b/client/client/src/fiscal-data/index.html
@@ -20,6 +20,7 @@
             <vn-horizontal>
                 <vn-textfield vn-one label="Código postal" field="fiscal.client.postcode"></vn-textfield>
                 <vn-autocomplete vn-one
+                    initial-value="fiscal.client.province"
                     field="fiscal.client.provinceFk"
                     url="/client/api/Provinces"
                     show-field="name"
@@ -27,6 +28,7 @@
                     label="Provincia">
                 </vn-autocomplete>
                 <vn-autocomplete vn-one
+                    initial-value="fiscal.client.country"
                     field="fiscal.client.countryFk"
                     url="/client/api/Countries"
                     show-field="name"
@@ -42,6 +44,7 @@
             <vn-horizontal>
                 <vn-textfield vn-two label="IBAN" field="fiscal.client.iban"></vn-textfield>
                 <vn-autocomplete vn-two
+                    initial-value="fiscal.client.payMethod"
                     field="fiscal.client.payMethodFk"
                     url="/client/api/PayMethods"
                     show-field="name"
diff --git a/client/core/src/autocomplete/index.js b/client/core/src/autocomplete/index.js
index 9cb309650a..24db75bb2c 100644
--- a/client/core/src/autocomplete/index.js
+++ b/client/core/src/autocomplete/index.js
@@ -27,7 +27,6 @@ export default class Autocomplete extends Component {
         this.vnPopover = vnPopover;
 
         componentHandler.upgradeElement($element[0].firstChild);
-        this.requestItem();
     }
     set field(value) {
         this.locked = true;
@@ -37,6 +36,14 @@ export default class Autocomplete extends Component {
     get field() {
         return this.value;
     }
+    set initialValue(value) {
+        if (value) {
+            if (!this.data)
+                this.data = [];
+            this.data.push(value);
+        }
+    }
+
     mdlUpdate() {
         let mdlField = this.element.firstChild.MaterialTextfield;
         if (mdlField)
@@ -191,10 +198,7 @@ export default class Autocomplete extends Component {
         this.hasFocus = true;
         this.input.select();
 
-        if (this.data)
-            this.showPopover();
-        else
-            this.loadData();
+        this.loadData();
     }
     onBlur() {
         this.hasFocus = false;
@@ -340,6 +344,13 @@ export default class Autocomplete extends Component {
         this.item = item;
         this.mdlUpdate();
     }
+    $onChanges(changes) {
+        if (!changes.initialValue)
+            this.requestItem();
+        else
+            this.showItem(changes.initialValue);
+    }
+
 }
 Autocomplete.$inject = ['$element', '$scope', '$http', 'vnPopover'];
 
@@ -349,6 +360,7 @@ module.component('vnAutocomplete', {
         url: '@',
         showField: '@',
         valueField: '@',
+        initialValue: '<',
         field: '=',
         label: '@'
     },
diff --git a/services/client/common/models/Client.js b/services/client/common/models/Client.js
index 13990502b9..b7ad468d8a 100644
--- a/services/client/common/models/Client.js
+++ b/services/client/common/models/Client.js
@@ -1,7 +1,8 @@
-
 module.exports = function(Client) {
+    
+    require("../scopes/card.js")(Client);
+    
     // Validations
-
     Client.validatesUniquenessOf('name', {message: 'El nombre debe ser único'});
     Client.validatesUniquenessOf('fi', {message: 'El NIF/CIF debe ser único'});
     Client.validatesPresenceOf('socialName', {message: 'Debe especificarse la razón social'});
diff --git a/services/client/common/models/Client.json b/services/client/common/models/Client.json
index c61676656a..d00c9fd407 100644
--- a/services/client/common/models/Client.json
+++ b/services/client/common/models/Client.json
@@ -149,12 +149,5 @@
       "permission": "ALLOW"
     }
   ],
-  "methods": {},
-  "scopes": {
-    "test": {
-      "where": {
-        "name": "Verdnatura"
-      }
-    }
-  }
-}
+  "methods": {}
+}
\ No newline at end of file
diff --git a/services/client/common/scopes/card.js b/services/client/common/scopes/card.js
new file mode 100644
index 0000000000..74023ee316
--- /dev/null
+++ b/services/client/common/scopes/card.js
@@ -0,0 +1,39 @@
+module.exports = function(Client) {
+    // Validations
+    var card = require('./card.json');
+
+    Client.remoteMethod('card', {
+        description: 'Get client for card call',
+        accepts: {
+            arg: 'id',
+            type: 'number',
+            required: true,
+            description: 'Model id',
+            http: {source: 'path'}
+        },
+        returns: {
+            arg: 'data', 
+            type: 'Client',
+            root: true
+        },
+        http: {
+            verb: 'get',
+            path: '/:id/card'
+        }
+    });
+    
+    Client.card = function(id, cb) {
+        let filter = {
+            where: {
+                id: id
+            },
+            include: card
+        };
+
+        Client.find(filter, function(err, instances) {
+            if(!err) {
+                cb(null, instances[0]);
+            }
+        })
+    };
+};
\ No newline at end of file
diff --git a/services/client/common/scopes/card.json b/services/client/common/scopes/card.json
new file mode 100644
index 0000000000..84ca960d65
--- /dev/null
+++ b/services/client/common/scopes/card.json
@@ -0,0 +1,43 @@
+[
+    {
+        "relation": "salesPerson", 
+        "scope":{
+        "fields": {
+            "id": true, "name": true
+            }
+        }
+    },
+    {
+        "relation": "contactChannel", 
+        "scope":{
+            "fields": {
+                "id": true, "name": true
+            }
+        }
+    },
+    {
+        "relation": "province", 
+        "scope":{
+            "fields": {
+                "id": true, "name": true
+            }
+        }
+    },
+    {
+        "relation": "country", 
+        "scope":{
+            "fields": {
+                "id": true, "name": true
+            }
+        }
+    },
+    {
+        "relation": "payMethod", 
+        "scope":{
+            "fields": {
+                "id": true, "name": true
+            }
+        }
+    }
+]
+

From e4559c39594b097a1af67dcc0ec523ddfed664ad Mon Sep 17 00:00:00 2001
From: nelo <nelo@verdnatura.es>
Date: Wed, 17 May 2017 09:45:34 +0200
Subject: [PATCH 2/2] =?UTF-8?q?Correcci=C3=B3n=20error=20autocomplete=20Se?=
 =?UTF-8?q?parado=20meted=20activate=20Separado=20meted=20addresses?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 client/core/src/autocomplete/index.js         |  6 --
 services/client/common/models/Client.js       | 83 +------------------
 .../client/common/scopes/client/activate.js   | 29 +++++++
 .../client/common/scopes/client/addresses.js  | 49 +++++++++++
 .../client/common/scopes/{ => client}/card.js |  0
 .../common/scopes/{ => client}/card.json      |  0
 6 files changed, 81 insertions(+), 86 deletions(-)
 create mode 100644 services/client/common/scopes/client/activate.js
 create mode 100644 services/client/common/scopes/client/addresses.js
 rename services/client/common/scopes/{ => client}/card.js (100%)
 rename services/client/common/scopes/{ => client}/card.json (100%)

diff --git a/client/core/src/autocomplete/index.js b/client/core/src/autocomplete/index.js
index 24db75bb2c..422db8d420 100644
--- a/client/core/src/autocomplete/index.js
+++ b/client/core/src/autocomplete/index.js
@@ -344,12 +344,6 @@ export default class Autocomplete extends Component {
         this.item = item;
         this.mdlUpdate();
     }
-    $onChanges(changes) {
-        if (!changes.initialValue)
-            this.requestItem();
-        else
-            this.showItem(changes.initialValue);
-    }
 
 }
 Autocomplete.$inject = ['$element', '$scope', '$http', 'vnPopover'];
diff --git a/services/client/common/models/Client.js b/services/client/common/models/Client.js
index b7ad468d8a..a4bd9a6516 100644
--- a/services/client/common/models/Client.js
+++ b/services/client/common/models/Client.js
@@ -1,6 +1,8 @@
 module.exports = function(Client) {
     
-    require("../scopes/card.js")(Client);
+    require("../scopes/client/card.js")(Client);
+    require("../scopes/client/activate.js")(Client);
+    require("../scopes/client/addresses.js")(Client);
     
     // Validations
     Client.validatesUniquenessOf('name', {message: 'El nombre debe ser único'});
@@ -43,36 +45,6 @@ module.exports = function(Client) {
         }
     });
 
-    // Methods
-
-    Client.remoteMethod('activate', {
-        description: 'Activate or deactive client',
-        accepts: {
-            arg: 'id',
-            type: 'number',
-            required: true,
-            description: 'Model id',
-            http: {source: 'path'}
-        },
-        returns: {
-            arg: 'active', 
-            type: 'boolean'
-        },
-        http: {
-            verb: 'put',
-            path: '/:id/activate'
-        }
-    });
-    
-    Client.activate = function(id, cb) {
-        console.log(id);
-        Client.findById(id, function(err, client) {
-            if(!err) {
-                Client.update({id: client.id}, {active: !client.active}); 
-                cb(null, !client.active);
-            }
-        });
-    };
 
     // Basic filter
 
@@ -94,53 +66,4 @@ module.exports = function(Client) {
         };
     }
 
-    // Client addresses
-
-    Client.remoteMethod('addressesList', {
-        description: 'List items using a filter',
-        accessType: 'READ',
-        accepts: [
-            {
-                arg: 'id',
-                type: 'string',
-                required: true,
-                description: 'Model id',
-                http: {source: 'path'}
-            },
-            {
-                arg: 'filter',
-                type: 'object',
-                required: true,
-                description: 'Filter defining where',
-                http: function(ctx) {
-                    return ctx.req.query;
-                }
-            }
-        ],
-        returns: {
-            arg: 'data', 
-            type: ['Address'],
-            root: true
-        },
-        http: {
-            path: `/:id/addressesList`,
-            verb: 'get'
-        }
-    });
-
-    Client.addressesList = function(id, p, cb) {
-        let filter = {
-            where: {
-                clientFk: id,
-            },
-            skip: (p.page - 1) * p.size,
-            limit: p.size
-        };
-
-        Client.app.models.Address.find(filter, function(err, instances) {
-            if(!err) {
-                cb(null, instances);
-            }
-        })
-    };
 };
\ No newline at end of file
diff --git a/services/client/common/scopes/client/activate.js b/services/client/common/scopes/client/activate.js
new file mode 100644
index 0000000000..c3d0ff451e
--- /dev/null
+++ b/services/client/common/scopes/client/activate.js
@@ -0,0 +1,29 @@
+module.exports = function(Client){
+    Client.remoteMethod('activate', {
+        description: 'Activate or deactive client',
+        accepts: {
+            arg: 'id',
+            type: 'number',
+            required: true,
+            description: 'Model id',
+            http: {source: 'path'}
+        },
+        returns: {
+            arg: 'active', 
+            type: 'boolean'
+        },
+        http: {
+            verb: 'put',
+            path: '/:id/activate'
+        }
+    });
+    
+    Client.activate = function(id, cb) {
+        Client.findById(id, function(err, client) {
+            if(!err) {
+                Client.update({id: client.id}, {active: !client.active}); 
+                cb(null, !client.active);
+            }
+        });
+    };
+};
\ No newline at end of file
diff --git a/services/client/common/scopes/client/addresses.js b/services/client/common/scopes/client/addresses.js
new file mode 100644
index 0000000000..d050d52387
--- /dev/null
+++ b/services/client/common/scopes/client/addresses.js
@@ -0,0 +1,49 @@
+module.exports = function(Client){
+    Client.remoteMethod('addressesList', {
+        description: 'List items using a filter',
+        accessType: 'READ',
+        accepts: [
+            {
+                arg: 'id',
+                type: 'string',
+                required: true,
+                description: 'Model id',
+                http: {source: 'path'}
+            },
+            {
+                arg: 'filter',
+                type: 'object',
+                required: true,
+                description: 'Filter defining where',
+                http: function(ctx) {
+                    return ctx.req.query;
+                }
+            }
+        ],
+        returns: {
+            arg: 'data', 
+            type: ['Address'],
+            root: true
+        },
+        http: {
+            path: `/:id/addressesList`,
+            verb: 'get'
+        }
+    });
+
+    Client.addressesList = function(id, p, cb) {
+        let filter = {
+            where: {
+                clientFk: id,
+            },
+            skip: (p.page - 1) * p.size,
+            limit: p.size
+        };
+
+        Client.app.models.Address.find(filter, function(err, instances) {
+            if(!err) {
+                cb(null, instances);
+            }
+        })
+    };
+}
\ No newline at end of file
diff --git a/services/client/common/scopes/card.js b/services/client/common/scopes/client/card.js
similarity index 100%
rename from services/client/common/scopes/card.js
rename to services/client/common/scopes/client/card.js
diff --git a/services/client/common/scopes/card.json b/services/client/common/scopes/client/card.json
similarity index 100%
rename from services/client/common/scopes/card.json
rename to services/client/common/scopes/client/card.json