Merge branch '6366-unifyTicketChangeState' of https://gitea.verdnatura.es/verdnatura/salix into 6366-unifyTicketChangeState

.vscode/settings.json

@@ -16,6 +16,7 @@
     },
     "cSpell.words": [
         "salix",
-        "fdescribe"
+        "fdescribe",
+        "Loggable"
     ]
 }

CHANGELOG.md

@@ -5,6 +5,13 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [2404.01] - 2024-01-25
+
+### Added
+### Changed
+### Fixed
+
+
 ## [2402.01] - 2024-01-11
 
 ### Added

back/methods/vn-user/privileges.js

@@ -68,7 +68,7 @@ module.exports = Self => {
             userToUpdate.hasGrant = hasGrant;
 
         if (roleFk) {
-            const role = await models.Role.findById(roleFk, {fields: ['name']}, myOptions);
+            const role = await models.VnRole.findById(roleFk, {fields: ['name']}, myOptions);
             const hasRole = await Self.hasRole(userId, role.name, myOptions);
 
             if (!hasRole)

back/methods/vn-user/specs/privileges.spec.js

@@ -70,7 +70,7 @@ describe('VnUser privileges()', () => {
         const tx = await models.VnUser.beginTransaction({});
 
         const options = {transaction: tx};
-        const agency = await models.Role.findOne({
+        const agency = await models.VnRole.findOne({
             where: {
                 name: 'agency'
             }

back/model-config.json

@@ -139,9 +139,6 @@
     "Warehouse": {
         "dataSource": "vn"
     },
-    "VnUser": {
-        "dataSource": "vn"
-    },
     "OsTicket": {
         "dataSource": "osticket"
     },
@@ -156,6 +153,12 @@
     },
     "ViaexpressConfig": {
         "dataSource": "vn"
+    },
+    "VnUser": {
+        "dataSource": "vn"
+    },
+    "VnRole": {
+        "dataSource": "vn"
     }
 }
 

back/models/dms-type.json

@@ -29,12 +29,12 @@
     "relations": {
     	"readRole": {
       		"type": "belongsTo",
-      		"model": "Role",
+      		"model": "VnRole",
       		"foreignKey": "readRoleFk"
     	},
     	"writeRole": {
             "type": "belongsTo",
-            "model": "Role",
+            "model": "VnRole",
             "foreignKey": "writeRoleFk"
     	}
   	},

back/models/image-collection.json

@@ -46,12 +46,12 @@
         },
         "readRole": {
             "type": "belongsTo",
-            "model": "Role",
+            "model": "VnRole",
             "foreignKey": "readRoleFk"
         },
         "writeRole": {
             "type": "belongsTo",
-            "model": "Role",
+            "model": "VnRole",
             "foreignKey": "writeRoleFk"
         }
     },
@@ -64,4 +64,3 @@
         }
     ]
 }
-  

back/models/notificationAcl.json

@@ -24,7 +24,7 @@
         },
         "role": {
             "type": "belongsTo",
-            "model": "Role",
+            "model": "VnRole",
             "foreignKey": "roleFk"
         }
     }

back/models/vn-role.json

@@ -0,0 +1,13 @@
+{
+    "name": "VnRole",
+    "base": "Role",
+	"validateUpsert": true,
+    "options": {
+		"mysql": {
+			"table": "account.role"
+		}
+	},
+    "mixins": {
+        "Loggable": true
+    }
+}

back/models/vn-user.json

@@ -7,6 +7,9 @@
 			"table": "account.user"
 		}
 	},
+    "mixins": {
+        "Loggable": true
+    },
     "resetPasswordTokenTTL": "604800",
 	"properties": {
 		"id": {
@@ -63,7 +66,7 @@
 	"relations": {
 		"role": {
 			"type": "belongsTo",
-			"model": "Role",
+			"model": "VnRole",
 			"foreignKey": "roleFk"
 		},
 		"roles": {

db/changes/235001/00-updateACL_Role_VnRole.sql

@@ -0,0 +1,6 @@
+INSERT INTO salix.ACL (model,property,accessType,permission,principalType,principalId) VALUES
+	 ('VnRole','*','READ','ALLOW','ROLE','employee'),
+	 ('VnRole','*','WRITE','ALLOW','ROLE','it');
+
+DELETE FROM`salix`.`ACL`  WHERE model='Role';
+

db/changes/240001/00-fixInvoiceCorrectionConstraintsName.sql

@@ -0,0 +1,7 @@
+ALTER TABLE `vn`.`invoiceCorrection` DROP FOREIGN KEY `cplusInvoiceTyoeFk`;
+ALTER TABLE `vn`.`invoiceCorrection` DROP FOREIGN KEY `invoiceCorrectionType_Fk33`;
+ALTER TABLE `vn`.`invoiceCorrection` DROP FOREIGN KEY `invoiceCorrection_ibfk_1`;
+
+ALTER TABLE `vn`.`invoiceCorrection` ADD CONSTRAINT `siiTypeInvoiceOut_FK` FOREIGN KEY (`siiTypeInvoiceOutFk`) REFERENCES `vn`.`siiTypeInvoiceOut`(id) ON UPDATE CASCADE;
+ALTER TABLE `vn`.`invoiceCorrection` ADD CONSTRAINT `invoiceCorrectionType_FK` FOREIGN KEY (`invoiceCorrectionTypeFk`) REFERENCES `vn`.`invoiceCorrectionType`(id) ON UPDATE CASCADE;
+ALTER TABLE `vn`.`invoiceCorrection` ADD CONSTRAINT `cplusRectificationType_FK` FOREIGN KEY (`cplusRectificationTypeFk`) REFERENCES `vn`.`cplusRectificationType`(id) ON UPDATE CASCADE;

db/changes/240001/00-getTaxBases.sql

@@ -0,0 +1,33 @@
+DELIMITER $$
+$$
+CREATE OR REPLACE DEFINER=`root`@`localhost` PROCEDURE `vn`.`getTaxBases`()
+BEGIN
+/**
+*   Calcula y devuelve en número de bases imponibles postivas y negativas
+*	Requiere la tabla temporal tmp.ticketToInvoice(id)
+*
+*	returns tmp.taxBases
+*/
+
+	CREATE OR REPLACE TEMPORARY TABLE tmp.ticket
+		(KEY (ticketFk))
+		ENGINE = MEMORY
+		SELECT id ticketFk
+			FROM tmp.ticketToInvoice;
+
+	CALL ticket_getTax(NULL);
+
+	DROP TEMPORARY TABLE IF EXISTS tmp.taxBases;
+	CREATE TEMPORARY TABLE tmp.taxBases
+		ENGINE = MEMORY
+		SELECT
+                SUM(taxableBase > 0) as positive,
+				SUM(taxableBase < 0) as negative
+			FROM(
+				SELECT SUM(taxableBase) taxableBase
+					FROM tmp.ticketTax
+					GROUP BY pgcFk
+				) t;
+
+END$$
+DELIMITER ;

db/changes/240001/01-newHasAnyPositiveBase.sql

@@ -0,0 +1,30 @@
+DELIMITER $$
+$$
+CREATE OR REPLACE DEFINER=`root`@`localhost` FUNCTION `vn`.`hasAnyPositiveBase`() RETURNS tinyint(1)
+    DETERMINISTIC
+BEGIN
+
+/**
+*	Calcula si existe alguna base imponible positiva
+*	Requiere la tabla temporal tmp.ticketToInvoice(id) para getTaxBases()
+*
+*	returns BOOLEAN
+*/
+
+	DECLARE hasAnyPositiveBase BOOLEAN;
+
+    CALL getTaxBases();
+
+	SELECT positive INTO hasAnyPositiveBase
+        FROM tmp.taxBases
+        LIMIT 1;
+
+	DROP TEMPORARY TABLE
+        tmp.ticketTax,
+        tmp.ticket,
+        tmp.taxBases;
+
+	RETURN hasAnyPositiveBase;
+
+END$$
+DELIMITER ;

db/changes/240001/01-refactorHasAnyNegativeBase.sql

@@ -0,0 +1,32 @@
+DELIMITER $$
+$$
+CREATE OR REPLACE DEFINER=`root`@`localhost` FUNCTION `vn`.`hasAnyNegativeBase`() RETURNS tinyint(1)
+    DETERMINISTIC
+BEGIN
+
+/**
+*	Calcula si existe alguna base imponible negativa
+*	Requiere la tabla temporal tmp.ticketToInvoice(id) para getTaxBases()
+*
+*	returns BOOLEAN
+*/
+
+	DECLARE hasAnyNegativeBase BOOLEAN;
+
+    CALL getTaxBases();
+
+	SELECT negative INTO hasAnyNegativeBase
+        FROM tmp.taxBases
+        LIMIT 1;
+
+	DROP TEMPORARY TABLE
+        tmp.ticketTax,
+        tmp.ticket,
+        tmp.taxBases;
+
+	RETURN hasAnyNegativeBase;
+
+END$$
+DELIMITER ;
+
+

db/changes/240201/00-ticketSmsToClientSms.sql

@@ -0,0 +1,9 @@
+ALTER TABLE `vn`.`clientSms` ADD `ticketFk` int(11) NULL;
+ALTER TABLE `vn`.`clientSms` ADD CONSTRAINT `clientSms_FK_2` FOREIGN KEY (`ticketFk`) REFERENCES `vn`.`ticket`(`id`) ON DELETE RESTRICT ON UPDATE CASCADE;
+
+INSERT INTO`vn`.`clientSms` (`clientFk`, `smsFk`, `ticketFk`)
+	SELECT `t`.`clientFk`, `s`.`smsFk`, `s`.`ticketFk`
+		FROM `vn`.`clientSms` `s`
+			JOIN `vn`.`ticket` `t` ON `t`.`id` = `s`.`ticketFk`;
+
+RENAME TABLE `vn`.`ticketSms` TO `vn`.`ticketSms__`;

db/dump/fixtures.sql

@@ -606,6 +606,7 @@ INSERT INTO `vn`.`invoiceOutSerial` (`code`, `description`, `isTaxed`, `taxAreaF
         ('T', 'Española rapida',            1, 'NATIONAL',  0,  'quick'),
         ('V', 'Intracomunitaria global',    0, 'CEE',       1,  'global'),
         ('M', 'Múltiple nacional',          1, 'NATIONAL',  0,  'quick'),
+        ('R', 'Rectificativa',              1, 'NATIONAL',  0,   NULL),
         ('E', 'Exportación rápida',         0, 'WORLD',     0,  'quick');
 
 INSERT INTO `vn`.`invoiceOut`(`id`, `serial`, `amount`, `issued`,`clientFk`, `created`, `companyFk`, `dued`, `booked`, `bankFk`, `hasPdf`)
@@ -613,7 +614,7 @@ INSERT INTO `vn`.`invoiceOut`(`id`, `serial`, `amount`, `issued`,`clientFk`, `cr
         (1, 'T', 1026.24,   util.VN_CURDATE(),                              1101, util.VN_CURDATE(),                              442, util.VN_CURDATE(),                              util.VN_CURDATE(), 1, 0),
         (2, 'T', 121.36,    util.VN_CURDATE(),                              1102, util.VN_CURDATE(),                              442, util.VN_CURDATE(),                              util.VN_CURDATE(), 1, 0),
         (3, 'T', 8.88,      util.VN_CURDATE(),                              1103, util.VN_CURDATE(),                              442, util.VN_CURDATE(),                              util.VN_CURDATE(), 1, 0),
-        (4, 'T', 8.88,      util.VN_CURDATE(),                              1103, util.VN_CURDATE(),                              442, util.VN_CURDATE(),                              util.VN_CURDATE(), 1, 0),
+        (4, 'T', 8.88,      util.VN_CURDATE(),                              1104, util.VN_CURDATE(),                              442, util.VN_CURDATE(),                              util.VN_CURDATE(), 1, 0),
         (5, 'A', 8.88,      DATE_ADD(util.VN_CURDATE(), INTERVAL -1 MONTH), 1103, DATE_ADD(util.VN_CURDATE(), INTERVAL -1 MONTH), 442, DATE_ADD(util.VN_CURDATE(), INTERVAL -1 MONTH), DATE_ADD(util.VN_CURDATE(), INTERVAL -1 MONTH), 1, 0);
 
 UPDATE `vn`.`invoiceOut` SET ref = 'T1111111' WHERE id = 1;
@@ -719,7 +720,7 @@ INSERT INTO `vn`.`route`(`id`, `time`, `workerFk`, `created`, `vehicleFk`, `agen
 INSERT INTO `vn`.`ticket`(`id`, `priority`, `agencyModeFk`,`warehouseFk`,`routeFk`, `shipped`, `landed`, `clientFk`,`nickname`, `addressFk`, `refFk`, `isDeleted`, `zoneFk`, `zonePrice`, `zoneBonus`, `created`, `weight`)
     VALUES
         (1 , 3,     1,  1, 1,       DATE_ADD(util.VN_CURDATE(), INTERVAL -1 MONTH), DATE_ADD(DATE_ADD(util.VN_CURDATE(),INTERVAL -1 MONTH), INTERVAL +1 DAY), 1101, 'Bat cave',                                      121, NULL,     0, 1,      5, 1, DATE_ADD(util.VN_CURDATE(), INTERVAL -1 MONTH),      1),
-        (2 , 1,     1,  1, 1,       DATE_ADD(util.VN_CURDATE(), INTERVAL -1 MONTH), DATE_ADD(DATE_ADD(util.VN_CURDATE(),INTERVAL -1 MONTH), INTERVAL +1 DAY), 1104, 'Stark tower',                                   124, NULL,     0, 1,      5, 1, DATE_ADD(util.VN_CURDATE(), INTERVAL -1 MONTH),      2),
+        (2 , 1,     1,  1, 1,       DATE_ADD(util.VN_CURDATE(), INTERVAL -1 MONTH), DATE_ADD(DATE_ADD(util.VN_CURDATE(),INTERVAL -1 MONTH), INTERVAL +1 DAY), 1101, 'Bat cave',                                      1,   NULL,     0, 1,      5, 1, DATE_ADD(util.VN_CURDATE(), INTERVAL -1 MONTH),      2),
         (3 , 1,     7,  1, 6,       DATE_ADD(util.VN_CURDATE(), INTERVAL -2 MONTH), DATE_ADD(DATE_ADD(util.VN_CURDATE(),INTERVAL -2 MONTH), INTERVAL +1 DAY), 1104, 'Stark tower',                                   124, NULL,     0, 3,      5, 1, DATE_ADD(util.VN_CURDATE(), INTERVAL -2 MONTH),      NULL),
         (4 , 3,     2,  1, 2,       DATE_ADD(util.VN_CURDATE(), INTERVAL -3 MONTH), DATE_ADD(DATE_ADD(util.VN_CURDATE(),INTERVAL -3 MONTH), INTERVAL +1 DAY), 1104, 'Stark tower',                                   124, NULL,     0, 9,      5, 1, DATE_ADD(util.VN_CURDATE(), INTERVAL -3 MONTH),      NULL),
         (5 , 3,     3,  3, 3,       DATE_ADD(util.VN_CURDATE(), INTERVAL -4 MONTH), DATE_ADD(DATE_ADD(util.VN_CURDATE(),INTERVAL -4 MONTH), INTERVAL +1 DAY), 1104, 'Stark tower',                                   124, NULL,     0, 10,     5, 1, DATE_ADD(util.VN_CURDATE(), INTERVAL -4 MONTH),      NULL),
@@ -3020,3 +3021,17 @@ INSERT INTO `vn`.`docuwareTablet` (`tablet`,`description`)
 	VALUES
         ('Tablet1','Jarvis tablet'),
 	    ('Tablet2','Avengers tablet');
+
+INSERT INTO `vn`.`sms` (`id`, `senderFk`, `sender`, `destination`, `message`, `statusCode`, `status`, `created`) 
+    VALUES (1, 66, '111111111', '0001111111111', 'Lorem ipsum dolor sit amet, consectetur adipiscing elit.', 0, 'OK', util.VN_CURDATE()),
+        (2, 66, '222222222', '0002222222222', 'Lorem ipsum dolor sit amet, consectetur adipiscing elit.', 0, 'PENDING', util.VN_CURDATE()),
+        (3, 66, '333333333', '0003333333333', 'Lorem ipsum dolor sit amet, consectetur adipiscing elit.', 0, 'ERROR', util.VN_CURDATE()),
+        (4, 66, '444444444', '0004444444444', 'Lorem ipsum dolor sit amet, consectetur adipiscing elit.', 0, 'OK', util.VN_CURDATE());
+
+INSERT INTO `vn`.`clientSms` (`id`, `clientFk`, `smsFk`, `ticketFk`) 
+    VALUES(1, 1103, 1, NULL),
+        (2, 1103, 2, NULL),
+        (3, 1103, 3, 32),
+        (4, 1103, 4, 32),
+        (13, 1101, 1, NULL),
+        (14, 1101, 4, 27);

e2e/paths/05-ticket/18_index_payout.spec.js

@@ -35,7 +35,7 @@ describe('Ticket index payout path', () => {
         await page.waitToClick(selectors.ticketsIndex.openAdvancedSearchButton);
         await page.write(selectors.ticketsIndex.advancedSearchClient, '1101');
         await page.keyboard.press('Enter');
-        await page.waitForNumberOfElements(selectors.ticketsIndex.anySearchResult, 9);
+        await page.waitForNumberOfElements(selectors.ticketsIndex.anySearchResult, 10);
         await page.waitToClick(selectors.ticketsIndex.firstTicketCheckbox);
         await page.waitToClick(selectors.ticketsIndex.secondTicketCheckbox);
 

e2e/paths/09-invoice-out/01_summary.spec.js

@@ -28,7 +28,6 @@ describe('InvoiceOut summary path', () => {
 
     it('should contain the tax breakdown', async() => {
         const firstTax = await page.waitToGetProperty(selectors.invoiceOutSummary.taxOne, 'innerText');
-
         const secondTax = await page.waitToGetProperty(selectors.invoiceOutSummary.taxTwo, 'innerText');
 
         expect(firstTax).toContain('10%');
@@ -37,10 +36,9 @@ describe('InvoiceOut summary path', () => {
 
     it('should contain the tickets info', async() => {
         const firstTicket = await page.waitToGetProperty(selectors.invoiceOutSummary.ticketOne, 'innerText');
-
         const secondTicket = await page.waitToGetProperty(selectors.invoiceOutSummary.ticketTwo, 'innerText');
 
         expect(firstTicket).toContain('Bat cave');
-        expect(secondTicket).toContain('Stark tower');
+        expect(secondTicket).toContain('Bat cave');
     });
 });

front/core/directives/rule.js

@@ -23,7 +23,6 @@ export function directive($translate, $window) {
         let rule = $attrs.rule.split('.');
         let modelName = rule.shift();
         let fieldName = rule.shift();
-
         let split = $attrs.ngModel.split('.');
         if (!fieldName) fieldName = split.pop() || null;
         if (!modelName) modelName = firstUpper(split.pop() || '');

loopback/common/mixins/loggable.js

@@ -0,0 +1,12 @@
+const LoopBackContext = require('loopback-context');
+async function handleObserve(ctx) {
+    ctx.options.httpCtx = LoopBackContext.getCurrentContext();
+}
+module.exports = function(Self) {
+    let Mixin = {
+        'before save': handleObserve,
+        'before delete': handleObserve,
+    };
+    for (const [listener, handler] of Object.entries(Mixin))
+        Self.observe(listener, handler);
+};

loopback/common/models/loggable.js

@@ -1,15 +0,0 @@
-const LoopBackContext = require('loopback-context');
-
-module.exports = function(Self) {
-    Self.setup = function() {
-        Self.super_.setup.call(this);
-    };
-
-    Self.observe('before save', async function(ctx) {
-        ctx.options.httpCtx = LoopBackContext.getCurrentContext();
-    });
-
-    Self.observe('before delete', async function(ctx) {
-        ctx.options.httpCtx = LoopBackContext.getCurrentContext();
-    });
-};

loopback/common/models/loggable.json

@@ -1,5 +0,0 @@
-{
-  "name": "Loggable",
-  "base": "VnModel",
-  "validateUpsert": true
-}

loopback/locale/en.json

@@ -45,6 +45,7 @@
 	"Extension format is invalid": "Extension format is invalid",
 	"NO_ZONE_FOR_THIS_PARAMETERS": "NO_ZONE_FOR_THIS_PARAMETERS",
 	"This client can't be invoiced": "This client can't be invoiced",
+	"You must provide the correction information to generate a corrective invoice": "You must provide the correction information to generate a corrective invoice",
 	"The introduced hour already exists": "The introduced hour already exists",
 	"Invalid parameters to create a new ticket": "Invalid parameters to create a new ticket",
 	"Concept cannot be blank": "Concept cannot be blank",
@@ -178,7 +179,8 @@
 	"The renew period has not been exceeded": "The renew period has not been exceeded",
 	"You can not use the same password": "You can not use the same password",
 	"Valid priorities": "Valid priorities: %d",
-	"Negative basis of tickets": "Negative basis of tickets: {{ticketsIds}}",
+	"hasAnyNegativeBase": "Negative basis of tickets: {{ticketsIds}}",
+	"hasAnyPositiveBase": "Positive basis of tickets: {{ticketsIds}}",
 	"This ticket cannot be left empty.": "This ticket cannot be left empty. %s",
 	"Social name should be uppercase": "Social name should be uppercase",
 	"Street should be uppercase": "Street should be uppercase",

loopback/locale/es.json

@@ -72,6 +72,7 @@
 	"The secret can't be blank": "La contraseña no puede estar en blanco",
 	"We weren't able to send this SMS": "No hemos podido enviar el SMS",
 	"This client can't be invoiced": "Este cliente no puede ser facturado",
+    "You must provide the correction information to generate a corrective invoice": "Debes informar la información de corrección para generar una factura rectificativa",
 	"This ticket can't be invoiced": "Este ticket no puede ser facturado",
 	"You cannot add or modify services to an invoiced ticket": "No puedes añadir o modificar servicios a un ticket facturado",
 	"This ticket can not be modified": "Este ticket no puede ser modificado",
@@ -305,7 +306,8 @@
 	"Mail not sent": "Se ha producido un fallo al enviar la factura al cliente [{{clientId}}]({{{clientUrl}}}), por favor revisa la dirección de correo electrónico",
 	"The renew period has not been exceeded": "El periodo de renovación no ha sido superado",
 	"Valid priorities": "Prioridades válidas: %d",
-	"Negative basis of tickets": "Base negativa para los tickets: {{ticketsIds}}",
+	"hasAnyNegativeBase": "Base negativa para los tickets: {{ticketsIds}}",
+	"hasAnyPositiveBase": "Base positivas para los tickets: {{ticketsIds}}",
 	"You cannot assign an alias that you are not assigned to": "No puede asignar un alias que no tenga asignado",
 	"This ticket cannot be left empty.": "Este ticket no se puede dejar vacío. %s",
 	"The company has not informed the supplier account for bank transfers": "La empresa no tiene informado la cuenta de proveedor para transferencias bancarias",
@@ -333,5 +335,6 @@
 	"This user does not have an assigned tablet": "Este usuario no tiene tablet asignada",
 	"Incorrect pin": "Pin incorrecto.",
 	"You already have the mailAlias": "Ya tienes este alias de correo",
-	"The alias cant be modified": "Este alias de correo no puede ser modificado"
+	"The alias cant be modified": "Este alias de correo no puede ser modificado",
+	"No tickets to invoice": "No hay tickets para facturar"
 }

loopback/server/model-config.json

@@ -25,20 +25,19 @@
     "FieldAcl": {
         "dataSource": "vn"
     },
-    "Role": {
-        "dataSource": "vn",
-        "options": {
-            "mysql": {
-                "table": "salix.Role"
-            }
-        }
-    },
     "RoleMapping": {
         "dataSource": "vn",
         "options": {
             "mysql": {
                 "table": "salix.RoleMapping"
             }
+        },
+        "relations": {
+            "role": {
+                "type": "belongsTo",
+                "model": "VnRole",
+                "foreignKey": "roleId"
+            }
         }
     },
     "Schema": {

modules/account/back/models/ldap-config.js

@@ -239,7 +239,7 @@ module.exports = Self => {
 
             // Prepare data
 
-            let roles = await $.Role.find({
+            let roles = await $.VnRole.find({
                 fields: ['id', 'name', 'description']
             });
             let roleRoles = await $.RoleRole.find({

modules/account/back/models/role-inherit.json

@@ -15,12 +15,12 @@
 	"relations": {
 		"owner": {
 			"type": "belongsTo",
-			"model": "Role",
+			"model": "VnRole",
 			"foreignKey": "role"
 		},
 		"inherits": {
 			"type": "belongsTo",
-			"model": "Role",
+			"model": "VnRole",
 			"foreignKey": "inheritsFrom"
 		}
 	}

modules/account/back/models/role-role.json

@@ -14,12 +14,12 @@
 	"relations": {
 		"owner": {
 			"type": "belongsTo",
-			"model": "Role",
+			"model": "VnRole",
 			"foreignKey": "role"
 		},
 		"inherits": {
 			"type": "belongsTo",
-			"model": "Role",
+			"model": "VnRole",
 			"foreignKey": "inheritsFrom"
 		}
 	}

modules/account/front/acl/create/index.html

@@ -15,7 +15,7 @@
             <vn-autocomplete
                 label="Role"
                 ng-model="$ctrl.acl.principalId"
-                url="Roles"
+                url="VnRoles"
                 id-field="name"
                 value-field="name"
                 vn-focus>

modules/account/front/acl/search-panel/index.html

@@ -4,7 +4,7 @@
         <vn-autocomplete
             label="Role"
             ng-model="filter.principalId"
-            url="Roles"
+            url="VnRoles"
             value-field="name">
         </vn-autocomplete>
         <vn-autocomplete

modules/account/front/create/index.html

@@ -30,7 +30,7 @@
             <vn-autocomplete
                 label="Role"
                 ng-model="$ctrl.user.roleFk"
-                url="Roles"
+                url="VnRoles"
                 rule="VnUser">
             </vn-autocomplete>
             <vn-textfield

modules/account/front/privileges/index.html

@@ -22,7 +22,7 @@
             <vn-autocomplete
                 label="Role"
                 ng-model="$ctrl.user.roleFk"
-                url="Roles">
+                url="VnRoles">
             </vn-autocomplete>
         </vn-vertical>
     </vn-card>

modules/account/front/role/basic-data/index.html

@@ -1,25 +1,27 @@
 <vn-watcher
     vn-id="watcher"
-    url="Roles"
+    url="VnRoles"
     data="$ctrl.role"
     form="form">
 </vn-watcher>
 <form
     name="form"
     ng-submit="watcher.submit()"
+
     class="vn-w-md">
     <vn-card class="vn-pa-lg">
         <vn-vertical>
             <vn-textfield
                 label="Name"
                 ng-model="$ctrl.role.name"
-                rule
+                rule="VnRole.name"
                 vn-focus>
             </vn-textfield>
             <vn-textfield
             label="Description"
             ng-model="$ctrl.role.description"
-                rule>
+            rule="VnRole.description"
+               >
             </vn-textfield>
         </vn-vertical>
     </vn-card>

modules/account/front/role/card/index.js

@@ -3,7 +3,7 @@ import ModuleCard from 'salix/components/module-card';
 
 class Controller extends ModuleCard {
     reload() {
-        this.$http.get(`Roles/${this.$params.id}`)
+        this.$http.get(`VnRoles/${this.$params.id}`)
             .then(res => this.role = res.data);
     }
 }

modules/account/front/role/card/index.spec.js

@@ -1,6 +1,6 @@
 import './index';
 
-describe('component vnRoleCard', () => {
+fdescribe('component vnRoleCard', () => {
     let controller;
     let $httpBackend;
 
@@ -15,7 +15,7 @@ describe('component vnRoleCard', () => {
         it('should reload the controller data', () => {
             controller.$params.id = 1;
 
-            $httpBackend.expectGET('Roles/1').respond('foo');
+            $httpBackend.expectGET('VnRoles/1').respond('foo');
             controller.reload();
             $httpBackend.flush();
 

modules/account/front/role/create/index.html

@@ -1,6 +1,6 @@
 <vn-watcher
     vn-id="watcher"
-    url="Roles"
+    url="VnRoles"
     data="$ctrl.role"
     insert-mode="true"
     form="form">
@@ -14,13 +14,13 @@
             <vn-textfield
                 label="Name"
                 ng-model="$ctrl.role.name"
-                rule
+                rule="VnRole.name"
                 vn-focus>
             </vn-textfield>
             <vn-textfield
                 label="Description"
                 ng-model="$ctrl.role.description"
-                rule>
+                rule="VnRole.description">
             </vn-textfield>
         </vn-vertical>
     </vn-card>

modules/account/front/role/descriptor/index.js

@@ -11,7 +11,7 @@ class Controller extends Descriptor {
     }
 
     onDelete() {
-        return this.$http.delete(`Roles/${this.id}`)
+        return this.$http.delete(`VnRoles/${this.id}`)
             .then(() => this.$state.go('account.role'))
             .then(() => this.vnApp.showSuccess(this.$t('Role removed')));
     }

modules/account/front/role/descriptor/index.spec.js

@@ -1,6 +1,6 @@
 import './index';
 
-describe('component vnRoleDescriptor', () => {
+fdescribe('component vnRoleDescriptor', () => {
     let controller;
     let $httpBackend;
 
@@ -18,7 +18,7 @@ describe('component vnRoleDescriptor', () => {
             controller.$state.go = jest.fn();
             jest.spyOn(controller.vnApp, 'showSuccess');
 
-            $httpBackend.expectDELETE('Roles/1').respond();
+            $httpBackend.expectDELETE('VnRoles/1').respond();
             controller.onDelete();
             $httpBackend.flush();
 

modules/account/front/role/main/index.html

@@ -1,6 +1,6 @@
 <vn-crud-model
     vn-id="model"
-    url="Roles"
+    url="VnRoles"
     filter="::$ctrl.filter"
     limit="20">
 </vn-crud-model>

modules/account/front/role/subroles/index.html

@@ -40,7 +40,7 @@
         <vn-autocomplete
             label="Role"
             ng-model="$ctrl.addData.inheritsFrom"
-            url="Roles"
+            url="VnRoles"
             vn-focus>
         </vn-autocomplete>
     </tpl-body>

modules/account/front/role/summary/index.js

@@ -6,7 +6,7 @@ class Controller extends Component {
         this._role = value;
         this.$.summary = null;
         if (!value) return;
-        this.$http.get(`Roles/${value.id}`)
+        this.$http.get(`VnRoles/${value.id}`)
             .then(res => this.$.summary = res.data);
     }
 

modules/account/front/search-panel/index.html

@@ -19,7 +19,7 @@
                 vn-one
                 label="Role"
                 ng-model="filter.roleFk"
-                url="Roles"
+                url="VnRoles"
                 value-field="id"
                 show-field="name">
             </vn-autocomplete>

modules/claim/back/models/claim-beginning.json

@@ -1,6 +1,9 @@
 {
     "name": "ClaimBeginning",
-    "base": "Loggable",
+    "base": "VnModel",
+    "mixins": {
+        "Loggable": true
+    },
     "options": {
         "mysql": {
             "table": "claimBeginning"

modules/claim/back/models/claim-development.json

@@ -1,6 +1,9 @@
 {
     "name": "ClaimDevelopment",
-    "base": "Loggable",
+    "base": "VnModel",
+    "mixins": {
+        "Loggable": true
+    },
     "options": {
         "mysql": {
             "table": "claimDevelopment"

modules/claim/back/models/claim-dms.json

@@ -1,6 +1,9 @@
 {
     "name": "ClaimDms",
-    "base": "Loggable",
+    "base": "VnModel",
+    "mixins": {
+        "Loggable": true
+    },
     "options": {
         "mysql": {
             "table": "claimDms"

modules/claim/back/models/claim-end.json

@@ -1,6 +1,9 @@
 {
     "name": "ClaimEnd",
-    "base": "Loggable",
+    "base": "VnModel",
+    "mixins": {
+        "Loggable": true
+    },
     "options": {
         "mysql": {
             "table": "claimEnd"

modules/claim/back/models/claim-observation.json

@@ -1,6 +1,9 @@
 {
     "name": "ClaimObservation",
-    "base": "Loggable",
+    "base": "VnModel",
+    "mixins": {
+        "Loggable": true
+    },
     "options": {
         "mysql": {
             "table": "claimObservation"

modules/claim/back/models/claim-state.json

@@ -1,6 +1,9 @@
 {
     "name": "ClaimState",
-    "base": "Loggable",
+    "base": "VnModel",
+    "mixins": {
+        "Loggable": true
+    },
     "options": {
         "mysql": {
             "table": "claimState"
@@ -32,7 +35,7 @@
     "relations": {
     	"writeRole": {
       		"type": "belongsTo",
-      		"model": "Role",
+      		"model": "VnRole",
       		"foreignKey": "roleFk"
         }
     },

modules/claim/back/models/claim.json

@@ -1,6 +1,9 @@
 {
     "name": "Claim",
-    "base": "Loggable",
+    "base": "VnModel",
+    "mixins": {
+        "Loggable": true
+    },
     "options": {
         "mysql": {
             "table": "claim"

modules/client/back/methods/client/canBeInvoiced.js

@@ -1,7 +1,7 @@
 const UserError = require('vn-loopback/util/user-error');
 
 module.exports = function(Self) {
-    Self.remoteMethodCtx('canBeInvoiced', {
+    Self.remoteMethod('canBeInvoiced', {
         description: 'Change property isEqualizated in all client addresses',
         accessType: 'READ',
         accepts: [

modules/client/back/methods/client/specs/consumption.spec.js

@@ -16,7 +16,7 @@ describe('client consumption() filter', () => {
             };
             const result = await models.Client.consumption(ctx, filter, options);
 
-            expect(result.length).toEqual(10);
+            expect(result.length).toEqual(11);
 
             await tx.rollback();
         } catch (e) {
@@ -49,7 +49,7 @@ describe('client consumption() filter', () => {
             const thirdRow = result[2];
 
             expect(result.length).toEqual(3);
-            expect(firstRow.quantity).toEqual(10);
+            expect(firstRow.quantity).toEqual(11);
             expect(secondRow.quantity).toEqual(15);
             expect(thirdRow.quantity).toEqual(20);
 

modules/client/back/model-config.json

@@ -113,9 +113,6 @@
     "SageTransactionType": {
         "dataSource": "vn"
     },
-    "TicketSms": {
-        "dataSource": "vn"
-    },
     "TpvError": {
         "dataSource": "vn"
     },

modules/client/back/models/address.json

@@ -1,7 +1,10 @@
 {
     "name": "Address",
     "description": "Client addresses",
-    "base": "Loggable",
+    "base": "VnModel",
+    "mixins": {
+        "Loggable": true
+    },
     "options": {
         "mysql": {
             "table": "address"

modules/client/back/models/client-contact.json

@@ -1,7 +1,10 @@
 {
     "name": "ClientContact",
     "description": "Client phone contacts",
-    "base": "Loggable",
+    "base": "VnModel",
+    "mixins": {
+        "Loggable": true
+    },
     "options": {
         "mysql": {
             "table": "clientContact"

modules/client/back/models/client-dms.json

@@ -1,6 +1,9 @@
 {
     "name": "ClientDms",
-    "base": "Loggable",
+    "base": "VnModel",
+    "mixins": {
+        "Loggable": true
+    },
     "options": {
         "mysql": {
             "table": "clientDms"

modules/client/back/models/client-informa.json

@@ -1,6 +1,9 @@
 {
 	"name": "ClientInforma",
-	"base": "Loggable",
+	"base": "VnModel",
+    "mixins": {
+        "Loggable": true
+    },
 	"log": {
 		"model":"ClientLog",
 		"relation": "client",

modules/client/back/models/client-observation.json

@@ -1,7 +1,10 @@
 {
   	"name": "ClientObservation",
 	"description": "Client notes",
-	"base": "Loggable",
+	"base": "VnModel",
+    "mixins": {
+        "Loggable": true
+    },
 	"options": {
 		"mysql": {
 			"table": "clientObservation"

modules/client/back/models/client-sample.json

@@ -1,6 +1,9 @@
 {
     "name": "ClientSample",
-    "base": "Loggable",
+    "base": "VnModel",
+    "mixins": {
+        "Loggable": true
+    },
     "options": {
         "mysql": {
         "table": "clientSample"

modules/client/back/models/client-sms.json

@@ -21,6 +21,11 @@
             "type": "belongsTo",
             "model": "Sms",
             "foreignKey": "smsFk"
+        },
+        "ticket": {
+            "type": "belongsTo",
+            "model": "Ticket",
+            "foreignKey": "ticketFk"
         }
     }
 }

modules/client/back/models/client.json

@@ -1,6 +1,9 @@
 {
 	"name": "Client",
-	"base": "Loggable",
+	"base": "VnModel",
+    "mixins": {
+        "Loggable": true
+    },
 	"options": {
 		"mysql": {
 			"table": "client"

modules/client/back/models/greuge.json

@@ -1,6 +1,9 @@
 {
     "name": "Greuge",
-    "base": "Loggable",
+    "base": "VnModel",
+    "mixins": {
+        "Loggable": true
+    },
     "options": {
         "mysql": {
             "table": "greuge"

modules/client/back/models/recovery.json

@@ -1,6 +1,9 @@
 {
     "name": "Recovery",
-    "base": "Loggable",
+    "base": "VnModel",
+    "mixins": {
+        "Loggable": true
+    },
     "options": {
         "mysql": {
             "table": "recovery"

modules/client/back/models/role-credit-limit.json

@@ -19,7 +19,7 @@
     "relations": {
         "role": {
             "type": "belongsTo",
-            "model": "Role",
+            "model": "VnRole",
             "foreignKey": "roleFk"
         }
     }

modules/client/back/models/ticket-sms.json

@@ -1,28 +0,0 @@
-{
-    "name": "TicketSms",
-	"base": "VnModel",
-    "options": {
-        "mysql": {
-            "table": "ticketSms"
-        }
-    },
-    "properties": {
-        "smsFk": {
-            "type": "number",
-            "id": true,
-            "description": "Identifier"
-        }
-    },
-    "relations": {
-        "ticket": {
-            "type": "belongsTo",
-            "model": "Ticket",
-            "foreignKey": "ticketFk"
-        },
-        "sms": {
-            "type": "belongsTo",
-            "model": "Sms",
-            "foreignKey": "smsFk"
-        }
-    }
-}

modules/client/front/sms/index.html

@@ -1,40 +1,2 @@
-<vn-crud-model
+<vn-card>
-    vn-id="model"
-    url="ClientSms"
-    link="{clientFk: $ctrl.$params.id}"
-    filter="::$ctrl.filter"
-    data="clientSmsList"
-    limit="20"
-    auto-load="true">
-</vn-crud-model>
-<vn-data-viewer model="model">
-    <vn-card class="vn-w-lg">
-        <vn-table model="model" auto-load="false">
-            <vn-thead>
-                <vn-tr>
-                    <vn-th field="senderFk">Sender</vn-th>
-                    <vn-th field="destination" number>Destination</vn-th>
-                    <vn-th field="message">Message</vn-th>
-                    <vn-th field="status">Status</vn-th>
-                    <vn-th field="created" expand>Created</vn-th>
-                </vn-tr>
-            </vn-thead>
-            <vn-tbody>
-                <vn-tr ng-repeat="clientSms in clientSmsList">
-                    <vn-td>
-                        <span class="link" ng-click="workerDescriptor.show($event, clientSms.sms.senderFk)">
-                            {{::clientSms.sms.sender.name}}
-                        </span>
-                    </vn-td>
-                    <vn-td number expand>{{::clientSms.sms.destination}}</vn-td>
-                    <vn-td expand vn-tooltip="{{::clientSms.sms.message}}">{{::clientSms.sms.message}}</vn-td>
-                    <vn-td>{{::clientSms.sms.status}}</vn-td>
-                    <vn-td shrink-datetime>{{::clientSms.sms.created | date:'dd/MM/yyyy HH:mm'}}</vn-td>
-                </vn-tr>
-            </vn-tbody>
-        </vn-table>
 </vn-card>
-</vn-data-viewer>
-<vn-worker-descriptor-popover
-    vn-id="worker-descriptor">
-</vn-worker-descriptor-popover>

modules/client/front/sms/index.js

@@ -1,32 +1,14 @@
 import ngModule from '../module';
 import Section from 'salix/components/section';
 
-export default class Controller extends Section {
+class Controller extends Section {
     constructor($element, $) {
         super($element, $);
+    }
 
-        this.filter = {
+    async $onInit() {
-            fields: ['id', 'smsFk'],
+        this.$state.go('client.card.summary', {id: this.$params.id});
-            include: {
+        window.location.href = await this.vnApp.getUrl(`Customer/${this.$params.id}/sms`);
-                relation: 'sms',
-                scope: {
-                    fields: [
-                        'senderFk',
-                        'sender',
-                        'destination',
-                        'message',
-                        'statusCode',
-                        'status',
-                        'created'],
-                    include: {
-                        relation: 'sender',
-                        scope: {
-                            fields: ['name']
-                        }
-                    }
-                }
-            }
-        };
     }
 }
 

modules/entry/back/models/buy.json

@@ -1,6 +1,9 @@
 {
     "name": "Buy",
-    "base": "Loggable",
+    "base": "VnModel",
+    "mixins": {
+        "Loggable": true
+    },
     "options": {
         "mysql": {
         "table": "buy"

modules/entry/back/models/entry-observation.json

@@ -1,6 +1,9 @@
 {
     "name": "EntryObservation",
-    "base": "Loggable",
+    "base": "VnModel",
+    "mixins": {
+        "Loggable": true
+    },
     "options": {
         "mysql": {
             "table": "entryObservation"

modules/entry/back/models/entry.json

@@ -1,6 +1,9 @@
 {
     "name": "Entry",
-    "base": "Loggable",
+    "base": "VnModel",
+    "mixins": {
+        "Loggable": true
+    },
     "options": {
         "mysql": {
             "table": "entry"

modules/invoiceIn/back/models/invoice-in-tax.json

@@ -1,6 +1,9 @@
 {
     "name": "InvoiceInTax",
-    "base": "Loggable",
+    "base": "VnModel",
+    "mixins": {
+        "Loggable": true
+    },
     "options": {
         "mysql": {
             "table": "invoiceInTax"

modules/invoiceIn/back/models/invoice-in.json

@@ -1,6 +1,9 @@
 {
     "name": "InvoiceIn",
-    "base": "Loggable",
+    "base": "VnModel",
+    "mixins": {
+        "Loggable": true
+    },
     "options": {
         "mysql": {
             "table": "invoiceIn"

modules/invoiceOut/back/methods/invoiceOut/createManualInvoice.js

@@ -85,7 +85,7 @@ module.exports = Self => {
                     throw new UserError(`A ticket with an amount of zero can't be invoiced`);
 
                 // Validates ticket nagative base
-                const hasNegativeBase = await getNegativeBase(ticketId, myOptions);
+                const hasNegativeBase = await getNegativeBase(maxShipped, clientId, companyId, myOptions);
                 if (hasNegativeBase && company.code == 'VNL')
                     throw new UserError(`A ticket with a negative base can't be invoiced`);
             } else {
@@ -162,10 +162,13 @@ module.exports = Self => {
         return result.invoiceable;
     }
 
-    async function getNegativeBase(ticketId, options) {
+    async function getNegativeBase(maxShipped, clientId, companyId, options) {
         const models = Self.app.models;
-        const query = 'SELECT vn.hasSomeNegativeBase(?) AS base';
+        await models.InvoiceOut.rawSql('CALL invoiceOut_exportationFromClient(?,?,?)',
-        const [result] = await models.InvoiceOut.rawSql(query, [ticketId], options);
+            [maxShipped, clientId, companyId], options
+        );
+        const query = 'SELECT vn.hasAnyNegativeBase() AS base';
+        const [result] = await models.InvoiceOut.rawSql(query, [], options);
 
         return result.base;
     }

modules/invoiceOut/back/methods/invoiceOut/invoiceClient.js

@@ -80,6 +80,7 @@ module.exports = Self => {
                 invoiceType,
                 args.companyFk,
                 args.invoiceDate,
+                null,
                 options
             );
 

modules/invoiceOut/back/methods/invoiceOut/specs/transferinvoice.spec.js

@@ -2,7 +2,7 @@
 const models = require('vn-loopback/server/server').models;
 const LoopBackContext = require('loopback-context');
 
-describe('InvoiceOut tranferInvoice()', () => {
+describe('InvoiceOut transferInvoice()', () => {
     const activeCtx = {
         accessToken: {userId: 5},
         http: {
@@ -23,20 +23,29 @@ describe('InvoiceOut tranferInvoice()', () => {
         const tx = await models.InvoiceOut.beginTransaction({});
         const options = {transaction: tx};
         const args = {
-            id: '1',
+            id: '4',
-            ref: 'T4444444',
+            refFk: 'T4444444',
             newClientFk: 1,
-            cplusRectificationId: 1,
+            cplusRectificationTypeFk: 1,
-            siiTypeInvoiceOutId: 1,
+            siiTypeInvoiceOutFk: 1,
-            invoiceCorrectionTypeId: 1
+            invoiceCorrectionTypeFk: 1
         };
         ctx.args = args;
         try {
+            const {clientFk: oldClient} = await models.InvoiceOut.findById(args.id, {fields: ['clientFk']});
+            const invoicesBefore = await models.InvoiceOut.find({}, options);
             const result = await models.InvoiceOut.transferInvoice(
                 ctx,
                 options);
+            const invoicesAfter = await models.InvoiceOut.find({}, options);
+            const rectificativeInvoice = invoicesAfter[invoicesAfter.length - 2];
+            const newInvoice = invoicesAfter[invoicesAfter.length - 1];
 
             expect(result).toBeDefined();
+            expect(invoicesAfter.length - invoicesBefore.length).toEqual(2);
+            expect(rectificativeInvoice.clientFk).toEqual(oldClient);
+            expect(newInvoice.clientFk).toEqual(args.newClientFk);
+
             await tx.rollback();
         } catch (e) {
             await tx.rollback();
@@ -49,20 +58,44 @@ describe('InvoiceOut tranferInvoice()', () => {
         const options = {transaction: tx};
         const args = {
             id: '1',
-            ref: 'T1111111',
+            refFk: 'T1111111',
             newClientFk: 1101,
-            cplusRectificationId: 1,
+            cplusRectificationTypeFk: 1,
-            siiTypeInvoiceOutId: 1,
+            siiTypeInvoiceOutFk: 1,
-            invoiceCorrectionTypeId: 1
+            invoiceCorrectionTypeFk: 1
         };
         ctx.args = args;
         try {
             await models.InvoiceOut.transferInvoice(
                 ctx,
                 options);
+            await tx.rollback();
         } catch (e) {
             expect(e.message).toBe(`Select a different client`);
             await tx.rollback();
         }
     });
+
+    it('should throw an UserError when it is refund', async() => {
+        const tx = await models.InvoiceOut.beginTransaction({});
+        const options = {transaction: tx};
+        const args = {
+            id: '1',
+            refFk: 'T1111111',
+            newClientFk: 1102,
+            cplusRectificationTypeFk: 1,
+            siiTypeInvoiceOutFk: 1,
+            invoiceCorrectionTypeFk: 1
+        };
+        ctx.args = args;
+        try {
+            await models.InvoiceOut.transferInvoice(
+                ctx,
+                options);
+            await tx.rollback();
+        } catch (e) {
+            expect(e.message).toContain(`This ticket is already a refund`);
+            await tx.rollback();
+        }
+    });
 });

modules/invoiceOut/back/methods/invoiceOut/transferInvoice.js

@@ -12,7 +12,7 @@ module.exports = Self => {
                 description: 'Issued invoice id'
             },
             {
-                arg: 'ref',
+                arg: 'refFk',
                 type: 'string',
                 required: true
             },
@@ -22,17 +22,17 @@ module.exports = Self => {
                 required: true
             },
             {
-                arg: 'cplusRectificationId',
+                arg: 'cplusRectificationTypeFk',
                 type: 'number',
                 required: true
             },
             {
-                arg: 'siiTypeInvoiceOutId',
+                arg: 'siiTypeInvoiceOutFk',
                 type: 'number',
                 required: true
             },
             {
-                arg: 'invoiceCorrectionTypeId',
+                arg: 'invoiceCorrectionTypeFk',
                 type: 'number',
                 required: true
             },
@@ -50,14 +50,14 @@ module.exports = Self => {
     Self.transferInvoice = async(ctx, options) => {
         const models = Self.app.models;
         const myOptions = {userId: ctx.req.accessToken.userId};
-        const args = ctx.args;
+        const {id, refFk, newClientFk, cplusRectificationTypeFk, siiTypeInvoiceOutFk, invoiceCorrectionTypeFk} = ctx.args;
         let tx;
         if (typeof options == 'object')
             Object.assign(myOptions, options);
 
-        const {clientFk} = await models.InvoiceOut.findById(args.id);
+        const {clientFk} = await models.InvoiceOut.findById(id);
 
-        if (clientFk == args.newClientFk)
+        if (clientFk == newClientFk)
             throw new UserError(`Select a different client`);
 
         if (!myOptions.transaction) {
@@ -65,10 +65,10 @@ module.exports = Self => {
             myOptions.transaction = tx;
         }
         try {
-            const filterRef = {where: {refFk: args.ref}};
+            const filterRef = {where: {refFk: refFk}};
             const tickets = await models.Ticket.find(filterRef, myOptions);
             const ticketsIds = tickets.map(ticket => ticket.id);
-            await models.Ticket.refund(ctx, ticketsIds, null, myOptions);
+            const refundTickets = await models.Ticket.refund(ctx, ticketsIds, null, myOptions);
 
             const filterTicket = {where: {ticketFk: {inq: ticketsIds}}};
 
@@ -82,20 +82,16 @@ module.exports = Self => {
             const clonedTicketIds = [];
 
             for (const clonedTicket of clonedTickets) {
-                await clonedTicket.updateAttribute('clientFk', args.newClientFk, myOptions);
+                await clonedTicket.updateAttribute('clientFk', newClientFk, myOptions);
                 clonedTicketIds.push(clonedTicket.id);
             }
 
-            const invoiceIds = await models.Ticket.invoiceTickets(ctx, clonedTicketIds, myOptions);
+            const invoiceCorrection =
-            const [invoiceId] = invoiceIds;
+                {correctedFk: id, cplusRectificationTypeFk, siiTypeInvoiceOutFk, invoiceCorrectionTypeFk};
+            const refundTicketIds = refundTickets.map(ticket => ticket.id);
 
-            await models.InvoiceCorrection.create({
+            await models.Ticket.invoiceTickets(ctx, refundTicketIds, invoiceCorrection, myOptions);
-                correctingFk: invoiceId,
+            const [invoiceId] = await models.Ticket.invoiceTickets(ctx, clonedTicketIds, null, myOptions);
-                correctedFk: args.id,
-                cplusRectificationTypeFk: args.cplusRectificationId,
-                siiTypeInvoiceOutFk: args.siiTypeInvoiceOutId,
-                invoiceCorrectionTypeFk: args.invoiceCorrectionTypeId
-            }, myOptions);
 
             if (tx) {
                 await tx.commit();

modules/invoiceOut/back/models/invoice-correction.json

@@ -16,13 +16,43 @@
             "type": "number"
         },
         "cplusRectificationTypeFk": {
-            "type": "number"
+            "type": "number",
+            "required": true
         },
         "siiTypeInvoiceOutFk": {
-            "type": "number"
+            "type": "number",
+            "required": true
         },
         "invoiceCorrectionTypeFk": {
-            "type": "number"
+            "type": "number",
+            "required": true
+        },
+        "relations": {
+            "correcting": {
+                "type": "belongsTo",
+                "model": "InvoiceOut",
+                "foreignKey": "correctingFk"
+            },
+            "corrected": {
+                "type": "belongsTo",
+                "model": "InvoiceOut",
+                "foreignKey": "correctedFk"
+            },
+            "cplusRectificationType": {
+                "type": "belongsTo",
+                "model": "cplusRectificationType",
+                "foreignKey": "cplusRectificationTypeFk"
+            },
+            "siiTypeInvoiceOut": {
+                "type": "belongsTo",
+                "model": "siiTypeInvoiceOut",
+                "foreignKey": "siiTypeInvoiceOutFk"
+            },
+            "invoiceCorrectionType": {
+                "type": "belongsTo",
+                "model": "invoiceCorrectionType",
+                "foreignKey": "invoiceCorrectionTypeFk"
+            }
         }
     }
 }

modules/invoiceOut/back/models/sii-type-invoice-out.json

@@ -12,6 +12,9 @@
             "type": "number",
             "description": "Identifier"
         },
+        "code": {
+            "type": "string"
+        },
         "description": {
             "type": "string"
         }

modules/invoiceOut/front/descriptor-menu/index.html

@@ -7,7 +7,8 @@
 <vn-crud-model
     auto-load="true"
     url="SiiTypeInvoiceOuts"
-    data="siiTypeInvoiceOut">
+    data="siiTypeInvoiceOuts"
+    where="{code: {like: 'R%'}}">
 </vn-crud-model>
 <vn-crud-model
     auto-load="true"
@@ -185,6 +186,9 @@
     vn-id="transferInvoice"
     title="transferInvoice"
     on-accept="$ctrl.transferInvoice()">
+    <tpl-title translate>
+        transferInvoice
+    </tpl-title>
     <tpl-body>
         <section class="transferInvoice">
             <vn-horizontal>
@@ -197,8 +201,7 @@
                     show-field="name"
                     value-field="id"
                     search-function="{or: [{id: $search}, {name: {like: '%'+ $search +'%'}}]}"
-                ng-model="$ctrl.invoiceOut.client.id"
+                    ng-model="$ctrl.clientId"
-                initial-data="$ctrl.invoiceOut.client.id"
                     order="id">
                     <tpl-item>
                         #{{id}} - {{::name}}
@@ -213,7 +216,7 @@
                     value-field="id"
                     ng-model="$ctrl.cplusRectificationType"
                     search-function="{or: [{id: $search}, {description: {like: '%'+ $search +'%'}}]}"
-                label="Cplus Type">
+                    label="Rectificative type">
                     <tpl-item>
                         {{::description}}
                     </tpl-item>
@@ -222,14 +225,17 @@
             <vn-horizontal>
                 <vn-autocomplete
                     vn-one
-            vn-id="cplusInvoiceType"
+                    vn-id="siiTypeInvoiceOut"
-            data="siiTypeInvoiceOut"
+                    data="siiTypeInvoiceOuts"
                     show-field="description"
                     value-field="id"
+                    fields="['id','code','description']"
                     required="true"
                     ng-model="$ctrl.siiTypeInvoiceOut"
-            search-function="{or: [{id: $search}, {description: {like: '%'+ $search +'%'}}]}"
                     label="Class">
+                    <tpl-item>
+                        {{::code}} - {{::description}}
+                    </tpl-item>
                 </vn-autocomplete>
                 <vn-autocomplete
                     vn-one

modules/invoiceOut/front/descriptor-menu/index.js

@@ -129,15 +129,15 @@ class Controller extends Section {
     transferInvoice() {
         const params = {
             id: this.invoiceOut.id,
-            ref: this.invoiceOut.ref,
+            refFk: this.invoiceOut.ref,
-            newClientFk: this.invoiceOut.client.id,
+            newClientFk: this.clientId,
-            cplusRectificationId: this.cplusRectificationType,
+            cplusRectificationTypeFk: this.cplusRectificationType,
-            siiTypeInvoiceOutId: this.siiTypeInvoiceOut,
+            siiTypeInvoiceOutFk: this.siiTypeInvoiceOut,
-            invoiceCorrectionTypeId: this.invoiceCorrectionType
+            invoiceCorrectionTypeFk: this.invoiceCorrectionType
         };
         this.$http.post(`InvoiceOuts/transferInvoice`, params).then(res => {
             const invoiceId = res.data;
-            this.vnApp.showSuccess(this.$t('Invoice trasfered!'));
+            this.vnApp.showSuccess(this.$t('Transferred invoice'));
             this.$state.go('invoiceOut.card.summary', {id: invoiceId});
         });
     }

modules/invoiceOut/front/descriptor-menu/locale/es.yml

@@ -22,4 +22,5 @@ The email can't be empty: El correo no puede estar vacío
 The following refund tickets have been created: "Se han creado los siguientes tickets de abono: {{ticketIds}}"
 Refund...: Abono...
 Transfer invoice to...: Transferir factura a...
-Cplus Type: Cplus Tipo
+Rectificative type: Tipo rectificativa
+Transferred invoice: Factura transferida

modules/invoiceOut/front/index/manual/index.html

@@ -6,6 +6,7 @@
         auto-load="true"
         url="InvoiceOutSerials"
         data="invoiceOutSerials"
+        where="{code: {neq: 'R'}}"
         order="code">
     </vn-crud-model>
     <vn-crud-model

modules/item/back/models/item-barcode.json

@@ -1,6 +1,9 @@
 {
   "name": "ItemBarcode",
-  "base": "Loggable",
+  "base": "VnModel",
+    "mixins": {
+        "Loggable": true
+    },
   "options": {
       "mysql": {
         "table": "itemBarcode"

modules/item/back/models/item-botanical.json

@@ -1,6 +1,9 @@
 {
     "name": "ItemBotanical",
-    "base": "Loggable",
+    "base": "VnModel",
+    "mixins": {
+        "Loggable": true
+    },
     "options": {
         "mysql": {
           "table": "itemBotanical"

modules/item/back/models/item-shelving.json

@@ -1,6 +1,9 @@
 {
 	"name": "ItemShelving",
-	"base": "Loggable",
+	"base": "VnModel",
+    "mixins": {
+        "Loggable": true
+    },
 	"options": {
 		"mysql": {
 			"table": "itemShelving"

modules/item/back/models/item-tag.json

@@ -1,6 +1,9 @@
 {
     "name": "ItemTag",
-    "base": "Loggable",
+    "base": "VnModel",
+    "mixins": {
+        "Loggable": true
+    },
     "options": {
         "mysql": {
             "table": "itemTag"

modules/item/back/models/item-tax-country.json

@@ -1,6 +1,9 @@
 {
     "name": "ItemTaxCountry",
-    "base": "Loggable",
+    "base": "VnModel",
+    "mixins": {
+        "Loggable": true
+    },
     "options": {
         "mysql": {
           "table": "itemTaxCountry"

modules/item/back/models/item.json

@@ -1,6 +1,9 @@
 {
     "name": "Item",
-    "base": "Loggable",
+    "base": "VnModel",
+    "mixins": {
+        "Loggable": true
+    },
     "options": {
         "mysql": {
             "table": "item"

modules/route/back/models/route.json

@@ -1,6 +1,9 @@
 {
     "name": "Route",
-	"base": "Loggable",
+	"base": "VnModel",
+    "mixins": {
+        "Loggable": true
+    },
     "options": {
         "mysql": {
             "table": "route"

modules/shelving/back/models/shelving.json

@@ -1,6 +1,9 @@
 {
     "name": "Shelving",
-	"base": "Loggable",
+	"base": "VnModel",
+    "mixins": {
+        "Loggable": true
+    },
     "options": {
         "mysql": {
             "table": "shelving"

modules/supplier/back/models/supplier-account.json

@@ -1,6 +1,9 @@
 {
     "name": "SupplierAccount",
-    "base": "Loggable",
+    "base": "VnModel",
+    "mixins": {
+        "Loggable": true
+    },
     "options": {
         "mysql": {
             "table": "supplierAccount"

modules/supplier/back/models/supplier-address.json

@@ -1,7 +1,10 @@
 {
     "name": "SupplierAddress",
     "description": "Supplier addresses",
-    "base": "Loggable",
+    "base": "VnModel",
+    "mixins": {
+        "Loggable": true
+    },
     "options": {
         "mysql": {
             "table": "supplierAddress"

modules/supplier/back/models/supplier-contact.json

@@ -1,6 +1,9 @@
 {
     "name": "SupplierContact",
-    "base": "Loggable",
+    "base": "VnModel",
+    "mixins": {
+        "Loggable": true
+    },
     "options": {
         "mysql": {
             "table": "supplierContact"

modules/supplier/back/models/supplier.json

@@ -1,6 +1,9 @@
 {
     "name": "Supplier",
-    "base": "Loggable",
+    "base": "VnModel",
+    "mixins": {
+        "Loggable": true
+    },
     "options": {
         "mysql": {
             "table": "supplier"

modules/ticket/back/methods/sale/refund.js

@@ -19,7 +19,7 @@ module.exports = Self => {
             }
         ],
         returns: {
-            type: ['number'],
+            type: ['object'],
             root: true
         },
         http: {
@@ -54,7 +54,7 @@ module.exports = Self => {
 
             if (tx) await tx.commit();
 
-            return refundsTicket[0];
+            return refundsTicket;
         } catch (e) {
             if (tx) await tx.rollback();
             throw e;

modules/ticket/back/methods/sale/specs/canEdit.spec.js

@@ -102,7 +102,7 @@ describe('sale canEdit()', () => {
             try {
                 const options = {transaction: tx};
 
-                const role = await models.Role.findOne({
+                const role = await models.VnRole.findOne({
                     where: {
                         name: roleEnabled.principalId
                     }
@@ -159,7 +159,7 @@ describe('sale canEdit()', () => {
             try {
                 const options = {transaction: tx};
 
-                const role = await models.Role.findOne({
+                const role = await models.VnRole.findOne({
                     where: {
                         name: roleEnabled.principalId
                     }

modules/ticket/back/methods/sale/specs/refund.spec.js

@@ -23,9 +23,9 @@ describe('Sale refund()', () => {
         try {
             const options = {transaction: tx};
 
-            const refundedTicket = await models.Sale.refund(ctx, salesIds, servicesIds, withWarehouse, options);
+            const refundedTickets = await models.Sale.refund(ctx, salesIds, servicesIds, withWarehouse, options);
 
-            expect(refundedTicket).toBeDefined();
+            expect(refundedTickets).toBeDefined();
 
             await tx.rollback();
         } catch (e) {
@@ -42,11 +42,11 @@ describe('Sale refund()', () => {
             const options = {transaction: tx};
             const ticketsBefore = await models.Ticket.find({}, options);
 
-            const ticket = await models.Sale.refund(ctx, salesIds, servicesIds, withWarehouse, options);
+            const tickets = await models.Sale.refund(ctx, salesIds, servicesIds, withWarehouse, options);
 
             const refundedTicket = await models.Ticket.findOne({
                 where: {
-                    id: ticket.id
+                    id: tickets[0].id
                 },
                 include: [
                     {

modules/ticket/back/methods/ticket/canBeInvoiced.js

@@ -10,20 +10,20 @@ module.exports = function(Self) {
                 description: 'The tickets id',
                 type: ['number'],
                 required: true
+            },
+            {
+                arg: 'isRectificative',
+                description: 'If it is rectificative',
+                type: 'boolean'
             }
         ],
-        returns: {
-            arg: 'data',
-            type: 'boolean',
-            root: true
-        },
         http: {
             path: `/canBeInvoiced`,
             verb: 'get'
         }
     });
 
-    Self.canBeInvoiced = async(ctx, ticketsIds, options) => {
+    Self.canBeInvoiced = async(ctx, ticketsIds, isRectificative, options) => {
         const myOptions = {};
         const $t = ctx.req.__; // $translate
 
@@ -34,26 +34,14 @@ module.exports = function(Self) {
             where: {
                 id: {inq: ticketsIds}
             },
-            fields: ['id', 'refFk', 'shipped', 'totalWithVat', 'companyFk']
+            fields: ['id', 'refFk', 'shipped', 'totalWithVat']
         }, myOptions);
-        const [firstTicket] = tickets;
-        const companyFk = firstTicket.companyFk;
-
-        const query =
-                `SELECT COUNT(*) isSpanishCompany
-                    FROM supplier s
-                        JOIN country c ON c.id = s.countryFk
-                            AND c.code = 'ES'
-                    WHERE s.id = ?`;
-        const [supplierCompany] = await Self.rawSql(query, [companyFk], options);
-
-        const isSpanishCompany = supplierCompany?.isSpanishCompany;
-
-        const [result] = await Self.rawSql('SELECT hasAnyNegativeBase() AS base', null, options);
-        const hasAnyNegativeBase = result?.base && isSpanishCompany;
-        if (hasAnyNegativeBase)
-            throw new UserError($t('Negative basis of tickets', {ticketsIds: ticketsIds}));
 
+        const taxBaseFunction = isRectificative ? 'hasAnyPositiveBase' : 'hasAnyNegativeBase';
+        const [hasAnyIncorrectBase] =
+            await Self.rawSql(`SELECT ${taxBaseFunction}() AS hasBasesProblem`, null, options);
+        if (hasAnyIncorrectBase?.hasBasesProblem)
+            throw new UserError($t(taxBaseFunction, {ticketsIds: ticketsIds}));
         const today = Date.vnNew();
 
         tickets.some(ticket => {
@@ -70,7 +58,5 @@ module.exports = function(Self) {
             if (ticketsIds.length == 1 && priceZero)
                 throw new UserError(`A ticket with an amount of zero can't be invoiced`);
         });
-
-        return true;
     };
 };

modules/ticket/back/methods/ticket/invoiceTickets.js

@@ -10,7 +10,13 @@ module.exports = function(Self) {
                 description: 'The tickets id',
                 type: ['number'],
                 required: true
+            },
+            {
+                arg: 'invoiceCorrection',
+                description: 'The invoice correction',
+                type: 'object',
             }
+
         ],
         returns: {
             type: ['object'],
@@ -22,7 +28,7 @@ module.exports = function(Self) {
         }
     });
 
-    Self.invoiceTickets = async(ctx, ticketsIds, options) => {
+    Self.invoiceTickets = async(ctx, ticketsIds, invoiceCorrection, options) => {
         const models = Self.app.models;
         const date = Date.vnNew();
         date.setHours(0, 0, 0, 0);
@@ -41,10 +47,10 @@ module.exports = function(Self) {
         let invoicesIds = [];
         try {
             const tickets = await models.Ticket.find({
+                fields: ['id', 'clientFk', 'companyFk', 'addressFk'],
                 where: {
                     id: {inq: ticketsIds}
-                },
+                }
-                fields: ['id', 'clientFk', 'companyFk']
             }, myOptions);
 
             const [firstTicket] = tickets;
@@ -55,22 +61,20 @@ module.exports = function(Self) {
             if (!isSameClient)
                 throw new UserError(`You can't invoice tickets from multiple clients`);
 
-            const client = await models.Client.findById(clientId, {
+            const {hasToInvoiceByAddress} = await models.Client.findById(clientId, {
-                fields: ['id', 'hasToInvoiceByAddress']
+                fields: ['hasToInvoiceByAddress']
             }, myOptions);
 
-            if (client.hasToInvoiceByAddress) {
+            let ticketsByAddress = hasToInvoiceByAddress
-                const query = `
+                ? Object.values(tickets.reduce((group, {id, addressFk}) => {
-                    SELECT DISTINCT addressFk
+                    group[addressFk] = group[addressFk] ?? [];
-                        FROM ticket t
+                    group[addressFk].push(id);
-                        WHERE id IN (?)`;
+                    return group;
-                const result = await Self.rawSql(query, [ticketsIds], myOptions);
+                }, {}))
+                : [ticketsIds];
 
-                const addressIds = result.map(address => address.addressFk);
+            for (const ticketIds of ticketsByAddress)
-                for (const address of addressIds)
+                invoicesIds.push(await createInvoice(ctx, companyId, ticketIds, invoiceCorrection, myOptions));
-                    await createInvoice(ctx, companyId, ticketsIds, address, invoicesIds, myOptions);
-            } else
-                await createInvoice(ctx, companyId, ticketsIds, null, invoicesIds, myOptions);
 
             if (tx) await tx.commit();
         } catch (e) {
@@ -85,9 +89,8 @@ module.exports = function(Self) {
         return invoicesIds;
     };
 
-    async function createInvoice(ctx, companyId, ticketsIds, address, invoicesIds, myOptions) {
+    async function createInvoice(ctx, companyId, ticketsIds, invoiceCorrection, myOptions) {
         const models = Self.app.models;
-
         await models.Ticket.rawSql(`
             CREATE OR REPLACE TEMPORARY TABLE tmp.ticketToInvoice
             (PRIMARY KEY  (id))
@@ -95,11 +98,8 @@ module.exports = function(Self) {
                 SELECT id
                     FROM vn.ticket
                     WHERE id IN (?)
-                        ${address ? `AND addressFk = ${address}` : ''}
         `, [ticketsIds], myOptions);
-
+        return models.Ticket.makeInvoice(ctx, 'R', companyId, Date.vnNew(), invoiceCorrection, myOptions);
-        const invoiceId = await models.Ticket.makeInvoice(ctx, 'R', companyId, Date.vnNew(), myOptions);
-        invoicesIds.push(invoiceId);
     }
 };