refs #5878 feat: validate params in supplier model

modules/client/back/methods/client/updateFiscalData.js

@@ -128,9 +128,7 @@ module.exports = Self => {
         let tx;
         const myOptions = {};
         const models = Self.app.models;
-        const args = ctx.args;
+        const {args, req: {__: $t}} = ctx;
-        const userId = ctx.req.accessToken.userId;
-        const $t = ctx.req.__;
 
         if (typeof options == 'object')
             Object.assign(myOptions, options);

modules/supplier/back/methods/supplier/updateFiscalData.js

@@ -1,3 +1,65 @@
+const validatorBodyModel = require('../../../../client/back/methods/client/body_model_validator');
+
+const BODY_MODEL = [{
+    arg: 'name',
+    type: 'string'
+},
+{
+    arg: 'nif',
+    type: 'string'
+},
+{
+    arg: 'account',
+    type: 'any'
+},
+{
+    arg: 'sageTaxTypeFk',
+    type: 'any'
+},
+{
+    arg: 'sageWithholdingFk',
+    type: 'any'
+},
+{
+    arg: 'sageTransactionTypeFk',
+    type: 'any'
+},
+{
+    arg: 'postCode',
+    type: 'any'
+},
+{
+    arg: 'street',
+    type: 'any'
+},
+{
+    arg: 'city',
+    type: 'string'
+},
+{
+    arg: 'provinceFk',
+    type: 'any'
+},
+{
+    arg: 'countryFk',
+    type: 'any'
+},
+{
+    arg: 'supplierActivityFk',
+    type: 'string'
+},
+{
+    arg: 'healthRegister',
+    type: 'string'
+},
+{
+    arg: 'isVies',
+    type: 'boolean'
+},
+{
+    arg: 'isTrucker',
+    type: 'boolean'
+}];
 module.exports = Self => {
     Self.remoteMethod('updateFiscalData', {
         description: 'Updates fiscal data of a supplier',
@@ -14,65 +76,11 @@ module.exports = Self => {
             http: {source: 'path'}
         },
         {
-            arg: 'name',
+            arg: 'data',
-            type: 'string'
+            type: 'object',
-        },
+            http: {source: 'body'}
-        {
+        }
-            arg: 'nif',
+        ],
-            type: 'string'
-        },
-        {
-            arg: 'account',
-            type: 'any'
-        },
-        {
-            arg: 'sageTaxTypeFk',
-            type: 'any'
-        },
-        {
-            arg: 'sageWithholdingFk',
-            type: 'any'
-        },
-        {
-            arg: 'sageTransactionTypeFk',
-            type: 'any'
-        },
-        {
-            arg: 'postCode',
-            type: 'any'
-        },
-        {
-            arg: 'street',
-            type: 'any'
-        },
-        {
-            arg: 'city',
-            type: 'string'
-        },
-        {
-            arg: 'provinceFk',
-            type: 'any'
-        },
-        {
-            arg: 'countryFk',
-            type: 'any'
-        },
-        {
-            arg: 'supplierActivityFk',
-            type: 'string'
-        },
-        {
-            arg: 'healthRegister',
-            type: 'string'
-        },
-        {
-            arg: 'isVies',
-            type: 'boolean'
-        },
-        {
-            arg: 'isTrucker',
-            type: 'boolean'
-        }],
         returns: {
             arg: 'res',
             type: 'string',
@@ -86,13 +94,18 @@ module.exports = Self => {
 
     Self.updateFiscalData = async(ctx, supplierId) => {
         const models = Self.app.models;
-        const args = ctx.args;
+        const {args} = ctx;
         const supplier = await models.Supplier.findById(supplierId);
 
         // Remove unwanted properties
         delete args.ctx;
         delete args.id;
-
+        const {isValid, tag} = validatorBodyModel(BODY_MODEL, args.data);
-        return supplier.updateAttributes(args);
+        if (!isValid) {
+            const key = $t(`${tag}`);
+            throw new Error($t('Model is not valid', {key})
+            );
+        }
+        return supplier.updateAttributes(args.data);
     };
 };