2023-02-13 07:02:34 +00:00
35 changed files with 533 additions and 191 deletions
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@ -5,19 +5,40 @@ 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).

+## [2306.01] - 2023-02-23
+
+### Added
+-
+
+### Changed
+-
+
+### Fixed
+-
+
 ## [2304.01] - 2023-02-09

 ### Added
 - (Rutas) Al descargar varias facturas se comprime en un zip
 - (Trabajadores -> Nuevo trabajador)  Nueva sección
+- (Tickets -> Adelantar tickets) Añadidos campos "líneas" y "litros" al ticket origen
+- (Tickets -> Adelantar tickets) Nuevo icono muestra cuando las agencias de los tickets origen/destino son distintas

 ### Changed
 - (Entradas -> Compras) Cambiados los campos "Precio Grouping/Packing" por "PVP" y "Precio" por "Coste"
 - (Artículos -> Últimas entradas) Cambiados los campos "P.P.U." y "P.P.P." por "PVP"
+- (Rutas -> Sumario/Tickets) Actualizados campos de los tickets
+- (Proveedores -> Crear/Editar) Permite añadir Proveedores con la misma razón social pero con países distintos
+- (Tickets -> Adelantar tickets) Cambiados selectores de estado por checks "Pendiente origen/destino"
+- (Tickets -> Adelantar tickets) Cambiado stock de destino a origen.

 ### Fixed
 - (Artículos -> Etiquetas) Permite intercambiar la relevancia entre dos etiquetas.
+- (Cliente -> Datos Fiscales) No se permite seleccionar 'Notificar vía e-mail' a los clientes sin e-mail
+- (Tickets -> Datos básicos) Permite guardar la hora de envío
 - (Tickets -> Añadir pago) Eliminado "null" en las referencias
+- (Tickets -> Adelantar tickets) Permite ordenar por importe
+- (Tickets -> Adelantar tickets) El filtrado por encajado muestra también los tickets sin tipo de encajado

 ## [2302.01] - 2023-01-26

--- a/back/models/notificationAcl.json
+++ b/back/models/notificationAcl.json
@ -6,6 +6,16 @@
 			"table": "util.notificationAcl"
 		}
 	},
+    "properties":{
+        "notificationFk": {
+            "id": true,
+            "type": "number"
+          },
+          "roleFk":{
+            "id": true,
+            "type": "number"
+          }
+    },
 	"relations": {
        "notification": {
            "type": "belongsTo",
--- a/back/models/notificationSubscription.js
+++ b/back/models/notificationSubscription.js
@ -0,0 +1,62 @@
+const UserError = require('vn-loopback/util/user-error');
+
+module.exports = Self => {
+    Self.observe('before save', async function(ctx) {
+        const models = Self.app.models;
+        const userId = ctx.options.accessToken.userId;
+        const user = await ctx.instance.userFk;
+        const modifiedUser = await getUserToModify(null, user, models);
+
+        if (userId != modifiedUser.id && userId != modifiedUser.bossFk)
+            throw new UserError('You dont have permission to modify this user');
+    });
+
+    Self.remoteMethod('deleteNotification', {
+        description: 'Deletes a notification subscription',
+        accepts: [
+            {
+                arg: 'ctx',
+                type: 'object',
+                http: {source: 'context'}
+            },
+            {
+                arg: 'notificationId',
+                type: 'number',
+                required: true
+            },
+        ],
+        returns: {
+            type: 'object',
+            root: true
+        },
+        http: {
+            verb: 'POST',
+            path: '/deleteNotification'
+        }
+    });
+
+    Self.deleteNotification = async function(ctx, notificationId) {
+        const models = Self.app.models;
+        const user = ctx.req.accessToken.userId;
+        const modifiedUser = await getUserToModify(notificationId, null, models);
+
+        if (user != modifiedUser.id && user != modifiedUser.bossFk)
+            throw new UserError('You dont have permission to modify this user');
+
+        await models.NotificationSubscription.destroyById(notificationId);
+    };
+
+    async function getUserToModify(notificationId, userFk, models) {
+        let userToModify = userFk;
+        if (notificationId) {
+            const subscription = await models.NotificationSubscription.findById(notificationId);
+            userToModify = subscription.userFk;
+        }
+        return await models.Worker.findOne({
+            fields: ['id', 'bossFk'],
+            where: {
+                id: userToModify
+            }
+        });
+    }
+};
--- a/back/models/notificationSubscription.json
+++ b/back/models/notificationSubscription.json
@ -7,15 +7,18 @@
 		}
 	},
    "properties": {
-        "notificationFk": {
+        "id": {
            "type": "number",
            "id": true,
-            "description": "Identifier"
+            "description": "Primary key"
+        },
+        "notificationFk": {
+            "type": "number",
+            "description": "Foreign key to Notification"
        },
        "userFk": {
            "type": "number",
-            "id": true,
-            "description": "Identifier"
+            "description": "Foreign key to Account"
        }
    },
 	"relations": {
--- a/back/models/specs/notificationSubscription.spec.js
+++ b/back/models/specs/notificationSubscription.spec.js
@ -0,0 +1,74 @@
+const models = require('vn-loopback/server/server').models;
+
+describe('loopback model NotificationSubscription', () => {
+    it('Should fail to delete a notification if the user is not editing itself or a subordinate', async() => {
+        const tx = await models.NotificationSubscription.beginTransaction({});
+
+        try {
+            const options = {transaction: tx};
+            const user = 9;
+            const notificationSubscriptionId = 2;
+            const ctx = {req: {accessToken: {userId: user}}};
+            const notification = await models.NotificationSubscription.findById(notificationSubscriptionId);
+
+            let error;
+
+            try {
+                await models.NotificationSubscription.deleteNotification(ctx, notification.id, options);
+            } catch (e) {
+                error = e;
+            }
+
+            expect(error.message).toContain('You dont have permission to modify this user');
+            await tx.rollback();
+        } catch (e) {
+            await tx.rollback();
+            throw e;
+        }
+    });
+
+    it('Should delete a notification if the user is editing itself', async() => {
+        const tx = await models.NotificationSubscription.beginTransaction({});
+
+        try {
+            const options = {transaction: tx};
+            const user = 9;
+            const notificationSubscriptionId = 4;
+            const ctx = {req: {accessToken: {userId: user}}};
+            const notification = await models.NotificationSubscription.findById(notificationSubscriptionId);
+
+            await models.NotificationSubscription.deleteNotification(ctx, notification.id, options);
+
+            const deletedNotification = await models.NotificationSubscription.findById(notificationSubscriptionId);
+
+            expect(deletedNotification).toBeNull();
+            await tx.rollback();
+        } catch (e) {
+            await tx.rollback();
+            throw e;
+        }
+    });
+
+    it('Should delete a notification if the user is editing a subordinate', async() => {
+        const tx = await models.NotificationSubscription.beginTransaction({});
+
+        try {
+            const options = {transaction: tx};
+            const user = 9;
+            const notificationSubscriptionId = 5;
+            const ctx = {req: {accessToken: {userId: user}}};
+            const notification = await models.NotificationSubscription.findById(notificationSubscriptionId);
+
+            await models.NotificationSubscription.deleteNotification(ctx, notification.id, options);
+
+            const deletedNotification = await models.NotificationSubscription.findById(notificationSubscriptionId);
+
+            expect(deletedNotification).toBeNull();
+            await tx.rollback();
+        } catch (e) {
+            await tx.rollback();
+            throw e;
+        }
+    });
+});
+
--- a/db/changes/230201/00-SupplierUniqueKey.sql
+++ b/db/changes/230201/00-SupplierUniqueKey.sql
@ -0,0 +1 @@
+ALTER TABLE `vn`.`supplier` ADD UNIQUE (name, countryFk);
--- a/db/changes/230401/00-acl_notifications.sql
+++ b/db/changes/230401/00-acl_notifications.sql
@ -0,0 +1,4 @@
+INSERT INTO `salix`.`ACL` (model,property,accessType,principalId)
+	VALUES 
+	('NotificationSubscription','*','*','employee'),
+	('NotificationAcl','*','READ','employee');
--- a/db/changes/230401/00-ticket_canAdvance.sql
+++ b/db/changes/230401/00-ticket_canAdvance.sql
@ -0,0 +1,110 @@
+DROP PROCEDURE IF EXISTS vn.ticket_canAdvance;
+
+DELIMITER $$
+$$
+CREATE DEFINER=`root`@`localhost` PROCEDURE `vn`.`ticket_canAdvance`(vDateFuture DATE, vDateToAdvance DATE, vWarehouseFk INT)
+BEGIN
+/**
+ * Devuelve los tickets y la cantidad de lineas de venta que se pueden adelantar.
+ *
+ * @param vDateFuture Fecha de los tickets que se quieren adelantar.
+ * @param vDateToAdvance Fecha a cuando se quiere adelantar.
+ * @param vWarehouseFk Almacén
+ */
+
+	DECLARE vDateInventory DATE;
+
+	SELECT inventoried INTO vDateInventory FROM vn.config;
+
+	DROP TEMPORARY TABLE IF EXISTS tmp.stock;
+	CREATE TEMPORARY TABLE tmp.stock
+		(itemFk INT PRIMARY KEY,
+		amount INT)
+		ENGINE = MEMORY;
+
+	INSERT INTO tmp.stock(itemFk, amount)
+		SELECT itemFk, SUM(quantity) amount FROM
+		(
+			SELECT itemFk, quantity
+				FROM vn.itemTicketOut
+				WHERE shipped >= vDateInventory
+					AND shipped < vDateFuture
+					AND warehouseFk = vWarehouseFk
+			UNION ALL
+			SELECT itemFk, quantity
+				FROM vn.itemEntryIn
+				WHERE landed >= vDateInventory
+					AND landed < vDateFuture
+					AND isVirtualStock = FALSE
+					AND warehouseInFk = vWarehouseFk
+			UNION ALL
+			SELECT itemFk, quantity
+				FROM vn.itemEntryOut
+				WHERE shipped >= vDateInventory
+					AND shipped < vDateFuture
+					AND warehouseOutFk = vWarehouseFk
+		) t
+		GROUP BY itemFk HAVING amount != 0;
+
+	DROP TEMPORARY TABLE IF EXISTS tmp.filter;
+	CREATE TEMPORARY TABLE tmp.filter
+        (INDEX (id))
+	SELECT s.ticketFk futureId,
+			t2.ticketFk id,
+			count(DISTINCT s.id) saleCount,
+            t2.state,
+			t2.isNotValidated,
+            st.name futureState,
+			st.isNotValidated futureIsNotValidated,
+			GROUP_CONCAT(DISTINCT ipt.code  ORDER BY ipt.code) futureIpt,
+			t2.ipt,
+			t.workerFk,
+			CAST(SUM(litros) AS DECIMAL(10,0)) futureLiters,
+			CAST(COUNT(*) AS DECIMAL(10,0)) `futureLines`,
+			t2.shipped,
+			t.shipped futureShipped,
+			t2.totalWithVat,
+			t.totalWithVat futureTotalWithVat,
+            t2.agency,
+            am.name futureAgency,
+            t2.lines,
+            t2.liters,
+            SUM((s.quantity <= IFNULL(st.amount,0))) hasStock
+		FROM vn.ticket t
+			JOIN vn.ticketState ts ON ts.ticketFk = t.id
+			JOIN vn.state st ON st.id = ts.stateFk
+			JOIN vn.saleVolume sv ON t.id = sv.ticketFk
+			JOIN (SELECT
+                        t2.id ticketFk,
+                        t2.addressFk,
+                        st.isNotValidated,
+                        st.name state,
+                        GROUP_CONCAT(DISTINCT ipt.code ORDER BY ipt.code) ipt,
+                        t2.shipped,
+                        t2.totalWithVat,
+                        am.name agency,
+                        CAST(SUM(litros) AS DECIMAL(10,0)) liters,
+                        CAST(COUNT(*) AS DECIMAL(10,0)) `lines`
+						FROM vn.ticket t2
+                            JOIN vn.saleVolume sv ON t2.id = sv.ticketFk
+							JOIN vn.sale s ON s.ticketFk = t2.id
+							JOIN vn.item i ON i.id = s.itemFk
+							JOIN vn.ticketState ts ON ts.ticketFk = t2.id
+							JOIN vn.state st ON st.id = ts.stateFk
+                            JOIN vn.agencyMode am ON t2.agencyModeFk = am.id
+							LEFT JOIN vn.itemPackingType ipt ON ipt.code = i.itemPackingTypeFk
+						WHERE t2.shipped BETWEEN vDateToAdvance AND util.dayend(vDateToAdvance)
+							AND t2.warehouseFk = vWarehouseFk
+						GROUP BY t2.id) t2 ON t2.addressFk = t.addressFk
+			JOIN vn.sale s ON s.ticketFk = t.id
+			JOIN vn.item i ON i.id = s.itemFk
+            JOIN vn.agencyMode am ON t.agencyModeFk = am.id
+			LEFT JOIN vn.itemPackingType ipt ON ipt.code = i.itemPackingTypeFk
+            LEFT JOIN tmp.stock st ON st.itemFk = s.itemFk
+		WHERE t.shipped BETWEEN vDateFuture AND util.dayend(vDateFuture)
+			AND t.warehouseFk = vWarehouseFk
+		GROUP BY t.id;
+
+	DROP TEMPORARY TABLE tmp.stock;
+END$$
+DELIMITER ;
--- a/db/changes/230401/00-uniqueKeyNotificationSubscription.sql
+++ b/db/changes/230401/00-uniqueKeyNotificationSubscription.sql
@ -0,0 +1,4 @@
+ALTER TABLE
+    `util`.`notificationSubscription`
+ADD
+    CONSTRAINT `notificationSubscription_UN` UNIQUE KEY (`notificationFk`, `userFk`);
--- a/db/changes/230401/00-updateIsToBeMailed.sql
+++ b/db/changes/230401/00-updateIsToBeMailed.sql
@ -0,0 +1,6 @@
+UPDATE `vn`.`client`
+    SET isToBeMailed = FALSE
+    WHERE
+        mailAddress is NULL
+        AND email is NULL
+        AND isToBeMailed = TRUE;
--- a/db/changes/230401/01-alter_notSubs.sql
+++ b/db/changes/230401/01-alter_notSubs.sql
@ -0,0 +1,7 @@
+ALTER TABLE `util`.`notificationSubscription`
+ADD `id` int(11) auto_increment NULL,
+DROP PRIMARY KEY,
+ADD CONSTRAINT PRIMARY KEY (`id`);
+
+ALTER TABLE `util`.`notificationSubscription`
+ADD KEY `notificationSubscription_ibfk_1` (`notificationFk`);
--- a/db/changes/230601/.gitkeep
+++ b/db/changes/230601/.gitkeep
--- a/db/dump/fixtures.sql
+++ b/db/dump/fixtures.sql
@ -1954,10 +1954,6 @@ INSERT INTO `vn`.`workerBusinessType` (`id`, `name`, `isFullTime`, `isPermanent`
        (100,   'INDEFINIDO A TIEMPO COMPLETO',                     1, 1, 1),
        (109,   'CONVERSION DE TEMPORAL EN INDEFINIDO T.COMPLETO',  1, 1, 1);

-INSERT INTO `vn`.`businessCategory` (`id`, `description`, `rate`)
-    VALUES
-        (1, 'basic employee', 1);
-
 UPDATE `vn`.`business` b
    SET `rate` = 7,
        `workerBusinessCategoryFk` = 1,
@ -2705,7 +2701,10 @@ INSERT INTO `util`.`notificationSubscription` (`notificationFk`, `userFk`)
    VALUES
        (1, 1109),
        (1, 1110),
-        (3, 1109);
+        (3, 1109),
+	    (1,9),
+        (1,3);
+

 INSERT INTO `vn`.`routeConfig` (`id`, `defaultWorkCenterFk`)
    VALUES
--- a/db/dump/structure.sql
+++ b/db/dump/structure.sql
@ -80203,4 +80203,3 @@ USE `vncontrol`;
 -- Dump completed on 2022-11-21  7:57:28


-
--- a/e2e/helpers/selectors.js
+++ b/e2e/helpers/selectors.js
@ -778,8 +778,8 @@ export default {
        ipt: 'vn-autocomplete[label="Destination IPT"]',
        tableIpt: 'vn-autocomplete[name="ipt"]',
        tableFutureIpt: 'vn-autocomplete[name="futureIpt"]',
-        futureState: 'vn-autocomplete[label="Origin Grouped State"]',
-        state: 'vn-autocomplete[label="Destination Grouped State"]',
+        futureState: 'vn-check[label="Pending Origin"]',
+        state: 'vn-check[label="Pending Destination"]',
        warehouseFk: 'vn-autocomplete[label="Warehouse"]',
        tableButtonSearch: 'vn-button[vn-tooltip="Search"]',
        moveButton: 'vn-button[vn-tooltip="Advance tickets"]',
@ -944,9 +944,9 @@ export default {
    routeSummary: {
        header: 'vn-route-summary > vn-card > h5',
        cost: 'vn-route-summary vn-label-value[label="Cost"]',
-        firstTicketID: 'vn-route-summary vn-tbody > vn-tr:nth-child(1) > vn-td:nth-child(2) > span',
+        firstTicketID: 'vn-route-summary vn-tbody > vn-tr:nth-child(1) > vn-td:nth-child(10) > span',
        firstTicketDescriptor: '.vn-popover.shown vn-ticket-descriptor',
-        firstAlias: 'vn-route-summary vn-tbody > vn-tr:nth-child(1) > vn-td:nth-child(3) > span',
+        firstAlias: 'vn-route-summary vn-tbody > vn-tr:nth-child(1) > vn-td:nth-child(5) > span',
        firstClientDescriptor: '.vn-popover.shown vn-client-descriptor',
        goToRouteSummaryButton: 'vn-route-summary > vn-card > h5 > a',

--- a/e2e/paths/05-ticket/22_advance.spec.js
+++ b/e2e/paths/05-ticket/22_advance.spec.js
@ -50,7 +50,7 @@ describe('Ticket Advance path', () => {
        await page.waitToClick(selectors.ticketAdvance.openAdvancedSearchButton);
        await page.autocompleteSearch(selectors.ticketAdvance.ipt, 'Horizontal');
        await page.waitToClick(selectors.ticketAdvance.submit);
-        await page.waitForNumberOfElements(selectors.ticketAdvance.table, 0);
+        await page.waitForNumberOfElements(selectors.ticketAdvance.table, 1);

        await page.waitToClick(selectors.ticketAdvance.openAdvancedSearchButton);
        await page.clearInput(selectors.ticketAdvance.ipt);
@ -62,7 +62,7 @@ describe('Ticket Advance path', () => {
        await page.waitToClick(selectors.ticketAdvance.openAdvancedSearchButton);
        await page.autocompleteSearch(selectors.ticketAdvance.futureIpt, 'Horizontal');
        await page.waitToClick(selectors.ticketAdvance.submit);
-        await page.waitForNumberOfElements(selectors.ticketAdvance.table, 0);
+        await page.waitForNumberOfElements(selectors.ticketAdvance.table, 1);

        await page.waitToClick(selectors.ticketAdvance.openAdvancedSearchButton);
        await page.clearInput(selectors.ticketAdvance.futureIpt);
@ -70,26 +70,36 @@ describe('Ticket Advance path', () => {
        await page.waitForNumberOfElements(selectors.ticketAdvance.table, 1);
    });

-    it('should search with the origin grouped state', async() => {
+    it('should search with the origin pending state', async() => {
        await page.waitToClick(selectors.ticketAdvance.openAdvancedSearchButton);
-        await page.autocompleteSearch(selectors.ticketAdvance.futureState, 'Free');
+        await page.waitToClick(selectors.ticketAdvance.futureState);
        await page.waitToClick(selectors.ticketAdvance.submit);
        await page.waitForNumberOfElements(selectors.ticketAdvance.table, 1);

        await page.waitToClick(selectors.ticketAdvance.openAdvancedSearchButton);
-        await page.clearInput(selectors.ticketAdvance.futureState);
+        await page.waitToClick(selectors.ticketAdvance.futureState);
+        await page.waitToClick(selectors.ticketAdvance.submit);
+        await page.waitForNumberOfElements(selectors.ticketAdvance.table, 0);
+
+        await page.waitToClick(selectors.ticketAdvance.openAdvancedSearchButton);
+        await page.waitToClick(selectors.ticketAdvance.futureState);
        await page.waitToClick(selectors.ticketAdvance.submit);
        await page.waitForNumberOfElements(selectors.ticketAdvance.table, 1);
    });

    it('should search with the destination grouped state', async() => {
        await page.waitToClick(selectors.ticketAdvance.openAdvancedSearchButton);
-        await page.autocompleteSearch(selectors.ticketAdvance.state, 'Free');
+        await page.waitToClick(selectors.ticketAdvance.state);
        await page.waitToClick(selectors.ticketAdvance.submit);
        await page.waitForNumberOfElements(selectors.ticketAdvance.table, 0);

        await page.waitToClick(selectors.ticketAdvance.openAdvancedSearchButton);
-        await page.clearInput(selectors.ticketAdvance.state);
+        await page.waitToClick(selectors.ticketAdvance.state);
+        await page.waitToClick(selectors.ticketAdvance.submit);
+        await page.waitForNumberOfElements(selectors.ticketAdvance.table, 1);
+
+        await page.waitToClick(selectors.ticketAdvance.openAdvancedSearchButton);
+        await page.waitToClick(selectors.ticketAdvance.state);
        await page.waitToClick(selectors.ticketAdvance.submit);
        await page.waitForNumberOfElements(selectors.ticketAdvance.table, 1);
    });
@ -116,42 +126,7 @@ describe('Ticket Advance path', () => {
        await page.waitForNumberOfElements(selectors.ticketAdvance.table, 1);
    });

-    it('should search in smart-table with stock', async() => {
-        await page.waitToClick(selectors.ticketAdvance.tableButtonSearch);
-        await page.write(selectors.ticketAdvance.tableStock, '5');
-        await page.waitForNumberOfElements(selectors.ticketAdvance.table, 2);
-
-        await page.waitToClick(selectors.ticketAdvance.tableButtonSearch);
-        await page.waitToClick(selectors.ticketAdvance.openAdvancedSearchButton);
-        await page.waitToClick(selectors.ticketAdvance.submit);
-        await page.waitForNumberOfElements(selectors.ticketAdvance.table, 1);
-    });
-
-    it('should search in smart-table with especified Lines', async() => {
-        await page.waitToClick(selectors.ticketAdvance.tableButtonSearch);
-        await page.write(selectors.ticketAdvance.tableLines, '0');
-        await page.keyboard.press('Enter');
-        await page.waitForNumberOfElements(selectors.ticketAdvance.table, 1);
-
-        await page.waitToClick(selectors.ticketAdvance.tableButtonSearch);
-        await page.waitToClick(selectors.ticketAdvance.openAdvancedSearchButton);
-        await page.waitToClick(selectors.ticketAdvance.submit);
-        await page.waitForNumberOfElements(selectors.ticketAdvance.table, 1);
-    });
-
-    it('should search in smart-table with especified Liters', async() => {
-        await page.waitToClick(selectors.ticketAdvance.tableButtonSearch);
-        await page.write(selectors.ticketAdvance.tableLiters, '0');
-        await page.keyboard.press('Enter');
-        await page.waitForNumberOfElements(selectors.ticketAdvance.table, 1);
-
-        await page.waitToClick(selectors.ticketAdvance.tableButtonSearch);
-        await page.waitToClick(selectors.ticketAdvance.openAdvancedSearchButton);
-        await page.waitToClick(selectors.ticketAdvance.submit);
-        await page.waitForNumberOfElements(selectors.ticketAdvance.table, 1);
-    });
-
-    it('should check the three last tickets and move to the future', async() => {
+    it('should check the one ticket and move to the present', async() => {
        await page.waitToClick(selectors.ticketAdvance.multiCheck);
        await page.waitToClick(selectors.ticketAdvance.moveButton);
        await page.waitToClick(selectors.ticketAdvance.acceptButton);
--- a/front/core/components/smart-table/index.js
+++ b/front/core/components/smart-table/index.js
@ -478,8 +478,8 @@ export default class SmartTable extends Component {

        const params = {q: JSON.stringify(stateFilter)};

-        this.$state.go(this.$state.current.name, params, {location: 'replace'});
-        this.refresh();
+        this.$state.go(this.$state.current.name, params, {location: 'replace'})
+            .then(() => this.refresh());
    }

    applySort() {
@ -499,8 +499,8 @@ export default class SmartTable extends Component {
        stateFilter.tableOrder = order;

        const params = {q: JSON.stringify(stateFilter)};
-        this.$state.go(this.$state.current.name, params, {location: 'replace'});
-        this.refresh();
+        this.$state.go(this.$state.current.name, params, {location: 'replace'})
+            .then(() => this.refresh());
    }

    filterSanitizer(field) {
@ -589,7 +589,7 @@ export default class SmartTable extends Component {
    refresh() {
        this.isRefreshing = true;
        this.model.refresh()
-            .then(() => this.isRefreshing = false);
+            .finally(() => this.isRefreshing = false);
    }
 }

--- a/front/core/components/smart-table/index.spec.js
+++ b/front/core/components/smart-table/index.spec.js
@ -160,7 +160,7 @@ describe('Component smartTable', () => {
    describe('applySort()', () => {
        it('should call the $state go and model refresh without making changes on the model order', () => {
            controller.$state = {
-                go: jest.fn(),
+                go: jest.fn().mockReturnValue(new Promise(resolve => resolve())),
                current: {
                    name: 'section'
                }
@ -171,13 +171,12 @@ describe('Component smartTable', () => {

            expect(controller.model.order).toBeUndefined();
            expect(controller.$state.go).toHaveBeenCalled();
-            expect(controller.refresh).toHaveBeenCalled();
        });

        it('should call the $state go and model refresh after setting model order according to the controller sortCriteria', () => {
            const orderBy = {field: 'myField', sortType: 'ASC'};
            controller.$state = {
-                go: jest.fn(),
+                go: jest.fn().mockReturnValue(new Promise(resolve => resolve())),
                current: {
                    name: 'section'
                }
@ -190,7 +189,6 @@ describe('Component smartTable', () => {

            expect(controller.model.order).toEqual(`${orderBy.field} ${orderBy.sortType}`);
            expect(controller.$state.go).toHaveBeenCalled();
-            expect(controller.refresh).toHaveBeenCalled();
        });
    });

@ -293,12 +291,10 @@ describe('Component smartTable', () => {
            controller.$inputsScope = {
                searchProps: {}
            };
-            jest.spyOn(controller, 'refresh');

            controller.defaultFilter();

            expect(controller.model.addFilter).toHaveBeenCalled();
-            expect(controller.refresh).toHaveBeenCalled();
        });
    });

--- a/loopback/locale/es.json
+++ b/loopback/locale/es.json
@ -261,5 +261,8 @@
 	"The DOCUWARE PDF document does not exists": "El documento PDF Docuware no existe",
    "It is not possible to modify tracked sales": "No es posible modificar líneas de pedido que se hayan empezado a preparar",
    "It is not possible to modify sales that their articles are from Floramondo": "No es posible modificar líneas de pedido cuyos artículos sean de Floramondo",
-    "It is not possible to modify cloned sales": "No es posible modificar líneas de pedido clonadas"
+    "It is not possible to modify cloned sales": "No es posible modificar líneas de pedido clonadas",
+	"A supplier with the same name already exists. Change the country.": "Un proveedor con el mismo nombre ya existe. Cambie el país.",
+	"There is no assigned email for this client": "No hay correo asignado para este cliente"
 }
+
--- a/modules/client/back/models/client.js
+++ b/modules/client/back/models/client.js
@ -141,6 +141,16 @@ module.exports = Self => {
        done();
    }

+    Self.validateAsync('isToBeMailed', isToBeMailed, {
+        message: 'There is no assigned email for this client'
+    });
+
+    function isToBeMailed(err, done) {
+        if (this.isToBeMailed == true && !this.email)
+            err();
+        done();
+    }
+
    Self.validateAsync('defaultAddressFk', isActive,
        {message: 'Unable to default a disabled consignee'}
    );
--- a/modules/route/back/methods/route/getTickets.js
+++ b/modules/route/back/methods/route/getTickets.js
@ -50,9 +50,12 @@ module.exports = Self => {
            am.name AS agencyModeName,
            u.nickname AS userNickname,
            vn.ticketTotalVolume(t.id) AS volume,
-            tob.description
+            tob.description,
+            GROUP_CONCAT(DISTINCT i.itemPackingTypeFk ORDER BY i.itemPackingTypeFk) ipt
        FROM vn.route r
            JOIN ticket t ON t.routeFk = r.id
+            JOIN vn.sale s ON s.ticketFk = t.id
+            JOIN vn.item i ON i.id = s.itemFk
            LEFT JOIN ticketState ts ON ts.ticketFk = t.id
            LEFT JOIN state st ON st.id = ts.stateFk
            LEFT JOIN warehouse wh ON wh.id = t.warehouseFk
@ -70,7 +73,9 @@ module.exports = Self => {
        const where = filter.where;
        where['r.id'] = filter.id;

-        stmt.merge(conn.makeSuffix(filter));
+        stmt.merge(conn.makeWhere(filter.where));
+        stmt.merge(conn.makeGroupBy('t.id'));
+        stmt.merge(conn.makeOrderBy(filter.order));

        const tickets = await conn.executeStmt(stmt, myOptions);

--- a/modules/route/front/summary/index.html
+++ b/modules/route/front/summary/index.html
@ -77,19 +77,35 @@
                <vn-thead>
                    <vn-tr>
                        <vn-th shrink>Order</vn-th>
-                        <vn-th number>Ticket id</vn-th>
-                        <vn-th>Alias</vn-th>
+                        <vn-th>Street</vn-th>
+                        <vn-th>City</vn-th>
+                        <vn-th shrink>PC</vn-th>
+                        <vn-th>Client</vn-th>
+                        <vn-th>Warehouse</vn-th>
                        <vn-th number shrink>Packages</vn-th>
                        <vn-th shrink>m³</vn-th>
-                        <vn-th>Warehouse</vn-th>
-                        <vn-th shrink>PC</vn-th>
-                        <vn-th>Street</vn-th>
+                        <vn-th shrink>Packaging</vn-th>
+                        <vn-th number>Ticket</vn-th>
                        <vn-th shrink></vn-th>
                    </vn-tr>
                </vn-thead>
                <vn-tbody>
                    <vn-tr ng-repeat="ticket in $ctrl.summary.tickets">
                        <vn-td shrink>{{ticket.priority | dashIfEmpty}}</vn-td>
+                        <vn-td expand title="{{ticket.address.street}}">{{ticket.street}}</vn-td>
+                        <vn-td expand>{{ticket.city}}</vn-td>
+                        <vn-td shrink>{{ticket.postalCode}}</vn-td>
+                        <vn-td>
+                            <span
+                                ng-click="clientDescriptor.show($event, ticket.clientFk)"
+                                class="link">
+                                {{ticket.nickname}}
+                            </span>
+                       </vn-td>
+                       <vn-td>{{ticket.warehouseName}}</vn-td>
+                        <vn-td number shrink>{{ticket.packages}}</vn-td>
+                        <vn-td shrink>{{ticket.volume}}</vn-td>
+                        <vn-td shrink>{{ticket.ipt}}</vn-td>
                        <vn-td number>
                            <span
                                ng-click="ticketDescriptor.show($event, ticket.id)"
@ -98,24 +114,12 @@
                            </span>
                        </vn-td>
                        <vn-td>
-                            <span 
-                                ng-click="clientDescriptor.show($event, ticket.clientFk)"
-                                class="link">
-                                {{ticket.nickname}}
-                            </span>
-                       </vn-td>
-                        <vn-td number shrink>{{ticket.packages}}</vn-td>
-                        <vn-td shrink>{{ticket.volume}}</vn-td>
-                        <vn-td>{{ticket.warehouseName}}</vn-td>
-                        <vn-td shrink>{{ticket.postalCode}}</vn-td>
-                        <vn-td expand title="{{ticket.address.street}}">{{ticket.street}}</vn-td>
-                        <vn-td shrink> 
-                            <vn-icon
-                                ng-if="ticket.notes.length"
-                                vn-tooltip="{{ticket.notes[0].description}}"
-                                icon="insert_drive_file"
-                                class="bright">
-                            </vn-icon>
+                            <vn-icon-button
+                                ng-if="::ticket.description"
+                                vn-tooltip="{{::ticket.description}}"
+                                icon="icon-notes"
+                                tabindex="-1">
+                            </vn-icon-button>
                        </vn-td>
                    </vn-tr>
                </vn-tbody>
--- a/modules/route/front/tickets/index.html
+++ b/modules/route/front/tickets/index.html
@ -59,8 +59,10 @@
                        <vn-th field="city">City</vn-th>
                        <vn-th field="postalCode" translate-attr="{title: 'Postcode'}" shrink>PC</vn-th>
                        <vn-th field="clientFk" expand>Client</vn-th>
+                        <vn-th field="warehouse" expand>Warehouse</vn-th>
                        <vn-th field="packages" shrink>Packages</vn-th>
                        <vn-th field="volume" shrink>m³</vn-th>
+                        <vn-th field="packaging" shrink>Packaging</vn-th>
                        <vn-th field="id" number>Ticket</vn-th>
                        <vn-th shrink></vn-th>
                        <vn-th shrink></vn-th>
@ -100,8 +102,10 @@
                                {{::ticket.nickname}}
                            </span>
                        </vn-td>
+                        <vn-td>{{ticket.warehouseName}}</vn-td>
                        <vn-td shrink>{{::ticket.packages}}</vn-td>
                        <vn-td shrink>{{::ticket.volume | number:2}}</vn-td>
+                        <vn-td shrink>{{::ticket.ipt}}</vn-td>
                        <vn-td number>
                            <span
                                ng-click="ticketDescriptor.show($event, ticket.id)"
--- a/modules/supplier/back/models/supplier.js
+++ b/modules/supplier/back/models/supplier.js
@ -16,10 +16,6 @@ module.exports = Self => {
        message: 'The social name cannot be empty'
    });

-    Self.validatesUniquenessOf('name', {
-        message: 'The supplier name must be unique'
-    });
-
    if (this.city) {
        Self.validatesPresenceOf('city', {
            message: 'City cannot be empty'
@ -117,6 +113,27 @@ module.exports = Self => {
            throw new UserError('You can not modify is pay method checked');
    });

+    Self.validateAsync('name', 'countryFk', hasSupplierSameName, {
+        message: 'A supplier with the same name already exists. Change the country.'
+    });
+
+    async function hasSupplierSameName(err, done) {
+        if (!this.name || !this.countryFk) done();
+        const supplier = await Self.app.models.Supplier.findOne(
+            {
+                where: {
+                    name: this.name,
+                    countryFk: this.countryFk
+                },
+                fields: ['id']
+            });
+
+        if (supplier && supplier.id != this.id)
+            err();
+
+        done();
+    }
+
    Self.observe('before save', async function(ctx) {
        const changes = ctx.data || ctx.instance;
        const orgData = ctx.currentInstance;
--- a/modules/ticket/back/methods/ticket/getTicketsAdvance.js
+++ b/modules/ticket/back/methods/ticket/getTicketsAdvance.js
@ -50,14 +50,14 @@ module.exports = Self => {
                required: false
            },
            {
-                arg: 'state',
-                type: 'string',
+                arg: 'isNotValidated',
+                type: 'boolean',
                description: 'Origin state',
                required: false
            },
            {
-                arg: 'futureState',
-                type: 'string',
+                arg: 'futureIsNotValidated',
+                type: 'boolean',
                description: 'Destination state',
                required: false
            },
@ -92,13 +92,23 @@ module.exports = Self => {
            case 'futureId':
                return {'f.futureId': value};
            case 'ipt':
-                return {'f.ipt': value};
+                return {or:
+                    [
+                        {'f.ipt': {like: `%${value}%`}},
+                        {'f.ipt': null}
+                    ]
+                };
            case 'futureIpt':
-                return {'f.futureIpt': value};
-            case 'state':
-                return {'f.stateCode': {like: `%${value}%`}};
-            case 'futureState':
-                return {'f.futureStateCode': {like: `%${value}%`}};
+                return {or:
+                    [
+                        {'f.futureIpt': {like: `%${value}%`}},
+                        {'f.futureIpt': null}
+                    ]
+                };
+            case 'isNotValidated':
+                return {'f.isNotValidated': value};
+            case 'futureIsNotValidated':
+                return {'f.futureIsNotValidated': value};
            }
        });

--- a/modules/ticket/back/methods/ticket/specs/getTicketsAdvance.spec.js
+++ b/modules/ticket/back/methods/ticket/specs/getTicketsAdvance.spec.js
@ -29,7 +29,7 @@ describe('TicketFuture getTicketsAdvance()', () => {
        }
    });

-    it('should return the tickets matching the origin grouped state', async() => {
+    it('should return the tickets matching the origin pending state', async() => {
        const tx = await models.Ticket.beginTransaction({});

        try {
@ -39,7 +39,7 @@ describe('TicketFuture getTicketsAdvance()', () => {
                dateFuture: tomorrow,
                dateToAdvance: today,
                warehouseFk: 1,
-                state: 'OK'
+                futureIsNotValidated: true
            };

            const ctx = {req: {accessToken: {userId: 9}}, args};
@ -54,7 +54,7 @@ describe('TicketFuture getTicketsAdvance()', () => {
        }
    });

-    it('should return the tickets matching the destination grouped state', async() => {
+    it('should return the tickets matching the destination pending state', async() => {
        const tx = await models.Ticket.beginTransaction({});

        try {
@ -64,13 +64,13 @@ describe('TicketFuture getTicketsAdvance()', () => {
                dateFuture: tomorrow,
                dateToAdvance: today,
                warehouseFk: 1,
-                futureState: 'FREE'
+                isNotValidated: true
            };

            const ctx = {req: {accessToken: {userId: 9}}, args};
            const result = await models.Ticket.getTicketsAdvance(ctx, options);

-            expect(result.length).toBeGreaterThan(0);
+            expect(result.length).toEqual(0);

            await tx.rollback();
        } catch (e) {
@ -89,7 +89,7 @@ describe('TicketFuture getTicketsAdvance()', () => {
                dateFuture: tomorrow,
                dateToAdvance: today,
                warehouseFk: 1,
-                ipt: 'Vertical'
+                ipt: 'V'
            };

            const ctx = {req: {accessToken: {userId: 9}}, args};
@ -114,7 +114,7 @@ describe('TicketFuture getTicketsAdvance()', () => {
                dateFuture: tomorrow,
                dateToAdvance: today,
                warehouseFk: 1,
-                tfIpt: 'Vertical'
+                tfIpt: 'V'
            };

            const ctx = {req: {accessToken: {userId: 9}}, args};
--- a/modules/ticket/front/advance-search-panel/index.html
+++ b/modules/ticket/front/advance-search-panel/index.html
@ -39,26 +39,18 @@
            </vn-autocomplete>
        </vn-horizontal>
        <vn-horizontal class="vn-px-lg">
-            <vn-autocomplete vn-one
-                data="$ctrl.groupedStates"
-                label="Origin Grouped State"
-                value-field="code"
-                show-field="name"
-                ng-model="filter.futureState">
-                <tpl-item>
-                    {{name}}
-                </tpl-item>
-            </vn-autocomplete>
-            <vn-autocomplete vn-one
-                data="$ctrl.groupedStates"
-                label="Destination Grouped State"
-                value-field="code"
-                show-field="name"
-                ng-model="filter.state">
-                <tpl-item>
-                    {{name}}
-                </tpl-item>
-            </vn-autocomplete>
+            <vn-check
+                vn-one
+                label="Pending Origin"
+                ng-model="filter.futureIsNotValidated"
+                triple-state="true">
+            </vn-check>
+            <vn-check
+                vn-one
+                label="Pending Destination"
+                ng-model="filter.isNotValidated"
+                triple-state="true">
+            </vn-check>
        </vn-horizontal>
        <vn-horizontal class="vn-px-lg">
            <vn-autocomplete
--- a/modules/ticket/front/advance-search-panel/index.js
+++ b/modules/ticket/front/advance-search-panel/index.js
@ -5,24 +5,9 @@ class Controller extends SearchPanel {
    constructor($, $element) {
        super($, $element);
        this.filter = this.$.filter;
-        this.getGroupedStates();
        this.getItemPackingTypes();
    }

-    getGroupedStates() {
-        let groupedStates = [];
-        this.$http.get('AlertLevels').then(res => {
-            for (let state of res.data) {
-                groupedStates.push({
-                    id: state.id,
-                    code: state.code,
-                    name: this.$t(state.code)
-                });
-            }
-            this.groupedStates = groupedStates;
-        });
-    }
-
    getItemPackingTypes() {
        let itemPackingTypes = [];
        const filter = {
--- a/modules/ticket/front/advance-search-panel/locale/es.yml
+++ b/modules/ticket/front/advance-search-panel/locale/es.yml
@ -1 +1,3 @@
 Advance tickets: Adelantar tickets
+Pending Origin: Pendiente origen
+Pending Destination: Pendiente destino
--- a/modules/ticket/front/advance/index.html
+++ b/modules/ticket/front/advance/index.html
@ -32,8 +32,8 @@
                <thead>
                    <tr second-header>
                        <td></td>
-                        <th colspan="5" translate>Origin</th>
-                        <th colspan="8" translate>Destination</th>
+                        <th colspan="9" translate>Origin</th>
+                        <th colspan="7" translate>Destination</th>
                    </tr>
                    <tr>
                        <th shrink>
@ -43,19 +43,30 @@
                                check-field="checked">
                            </vn-multi-check>
                        </th>
+                        <th shrink>
+                        </th>
                        <th field="futureId">
                            <span translate>ID</span>
                        </th>
                        <th field="futureShipped">
                            <span translate>Date</span>
                        </th>
-                        <th field="futureIpt" title="Item Packing Type">
+                        <th field="futureIpt" title="{{'Item Packing Type' | translate}}">
                            <span>IPT</span>
                        </th>
                        <th field="futureState">
                            <span translate>State</span>
                        </th>
-                        <th field="totalWithVat">
+                        <th field="futureLiters">
+                            <span translate>Liters</span>
+                        </th>
+                        <th field="hasStock">
+                            <span>Stock</span>
+                        </th>
+                        <th field="futureLines">
+                            <span translate>Lines</span>
+                        </th>
+                        <th field="futureTotalWithVat">
                            <span translate>Import</span>
                        </th>
                        <th separator field="id">
@ -64,7 +75,7 @@
                        <th field="shipped">
                            <span translate>Date</span>
                        </th>
-                        <th field="ipt" title="Item Packing Type">
+                        <th field="ipt" title="{{'Item Packing Type' | translate}}">
                            <span>IPT</span>
                        </th>
                        <th field="state">
@ -73,13 +84,10 @@
                        <th field="liters">
                            <span translate>Liters</span>
                        </th>
-                        <th field="hasStock">
-                            <span>Stock</span>
-                        </th>
                        <th field="lines">
                            <span translate>Lines</span>
                        </th>
-                        <th field="futureTotalWithVat">
+                        <th field="totalWithVat">
                            <span translate>Import</span>
                        </th>
                    </tr>
@ -92,6 +100,13 @@
                                vn-click-stop>
                            </vn-check>
                        </td>
+                        <td>
+                            <vn-icon
+                                ng-show="ticket.futureAgency !== ticket.agency"
+                                icon="icon-agency-term"
+                                vn-tooltip="{{$ctrl.agencies(ticket.futureAgency, ticket.agency)}}">
+                            </vn-icon>
+                        </td>
                        <td>
                            <span
                                ng-click="ticketDescriptor.show($event, ticket.futureId)"
@ -111,6 +126,9 @@
                                {{::ticket.futureState | dashIfEmpty}}
                            </span>
                        </td>
+                        <td>{{::ticket.futureLiters | dashIfEmpty}}</td>
+                        <td>{{::ticket.hasStock | dashIfEmpty}}</td>
+                        <td>{{::ticket.futureLines | dashIfEmpty}}</td>
                        <td>
                            <span class="chip {{$ctrl.totalPriceColor(ticket.futureTotalWithVat)}}">
                                {{::(ticket.futureTotalWithVat ? ticket.futureTotalWithVat : 0) | currency: 'EUR': 2}}
@ -136,7 +154,6 @@
                            </span>
                        </td>
                        <td>{{::ticket.liters | dashIfEmpty}}</td>
-                        <td>{{::ticket.hasStock | dashIfEmpty}}</td>
                        <td>{{::ticket.lines | dashIfEmpty}}</td>
                        <td>
                            <span class="chip {{$ctrl.totalPriceColor(ticket.totalWithVat)}}">
--- a/modules/ticket/front/advance/index.js
+++ b/modules/ticket/front/advance/index.js
@ -1,5 +1,6 @@
 import ngModule from '../module';
 import Section from 'salix/components/section';
+import './style.scss';

 export default class Controller extends Section {
    constructor($element, $) {
@ -128,6 +129,11 @@ export default class Controller extends Section {
        });
    }

+    agencies(futureAgency, agency) {
+        return this.$t(`Origin agency`, {agency: futureAgency}) +
+            '<br/>' + this.$t(`Destination agency`, {agency: agency});
+    }
+
    moveTicketsAdvance() {
        let ticketsToMove = [];
        this.checked.forEach(ticket => {
@ -157,6 +163,10 @@ export default class Controller extends Section {
            return {'liters': value};
        case 'lines':
            return {'lines': value};
+        case 'futureLiters':
+            return {'futureLiters': value};
+        case 'futureLines':
+            return {'futureLines': value};
        case 'ipt':
            return {'ipt': value};
        case 'futureIpt':
--- a/modules/ticket/front/advance/locale/es.yml
+++ b/modules/ticket/front/advance/locale/es.yml
@ -4,3 +4,6 @@ Advance confirmation: ¿Desea adelantar {{checked}} tickets?
 Success: Tickets movidos correctamente
 Lines: Líneas
 Liters: Litros
+Item Packing Type: Encajado
+Origin agency: "Agencia origen: {{agency}}"
+Destination agency: "Agencia destino: {{agency}}"
--- a/modules/ticket/front/advance/style.scss
+++ b/modules/ticket/front/advance/style.scss
@ -0,0 +1,7 @@
+@import "variables";
+
+vn-ticket-advance{
+    vn-icon {
+        color: #f7931e
+    }
+}
--- a/modules/ticket/front/basic-data/step-one/index.js
+++ b/modules/ticket/front/basic-data/step-one/index.js
@ -75,8 +75,10 @@ class Controller extends Component {
    }

    set shipped(value) {
+        if (new Date(this.ticket.shipped).toDateString() != value.toDateString())
+            value.setHours(0, 0, 0, 0);
+
        this.ticket.shipped = value;
-        this.ticket.shipped.setHours(0, 0, 0, 0);
        this.getLanded({
            shipped: value,
            addressFk: this.ticket.addressFk,
--- a/package.json
+++ b/package.json
@ -1,6 +1,6 @@
 {
    "name": "salix-back",
-    "version": "23.04.01",
+    "version": "23.06.01",
    "author": "Verdnatura Levante SL",
    "description": "Salix backend",
    "license": "GPL-3.0",
				`@ -0,0 +1 @@`
				ALTER TABLE `vn`.`supplier` ADD UNIQUE (name, countryFk);
				@ -80203,4 +80203,3 @@ USE `vncontrol`;
				`-- Dump completed on 2022-11-21 7:57:28`