feat(ticket): add option 'refund all' in descriptor menu

db/changes/10450-april/00-aclSaleRefund.sql

@@ -0,0 +1,3 @@
+UPDATE `salix`.`ACL`
+	SET `property`='refund'
+WHERE `model`='Sale' AND `property`='payBack';

db/changes/10450-april/00-ticket_doRefund.sql

@@ -0,0 +1,113 @@
+DROP PROCEDURE IF EXISTS vn.ticket_doRefund;
+
+DELIMITER $$
+$$
+CREATE DEFINER=`root`@`localhost` PROCEDURE `vn`.`ticket_doRefund`(IN vOriginTicket INT, OUT vNewTicket INT)
+BEGIN
+	
+	DECLARE vDone BIT DEFAULT 0;
+	DECLARE vCustomer MEDIUMINT;
+	DECLARE vWarehouse TINYINT;
+	DECLARE vCompany MEDIUMINT;
+	DECLARE vAddress MEDIUMINT;
+    DECLARE vRefundAgencyMode INT;
+    DECLARE vItemFk INT;
+    DECLARE vQuantity DECIMAL (10,2);
+    DECLARE vConcept VARCHAR(50);
+    DECLARE vPrice DECIMAL (10,2);
+    DECLARE vDiscount TINYINT;
+	DECLARE vSaleNew INT;
+    DECLARE vSaleMain INT;
+    DECLARE vZoneFk INT;
+   	DECLARE vDescription VARCHAR(50);
+  	DECLARE vTaxClassFk INT;
+   	DECLARE vTicketServiceTypeFk INT;
+    
+    DECLARE cSales CURSOR FOR
+		SELECT *
+			FROM tmp.sale;
+		
+	DECLARE cTicketServices CURSOR FOR
+		SELECT *
+			FROM tmp.ticketService;	
+    
+    DECLARE CONTINUE HANDLER FOR NOT FOUND SET vDone = 1;
+	
+    SELECT id INTO vRefundAgencyMode
+		FROM agencyMode WHERE `name` = 'ABONO';
+
+	SELECT clientFk, warehouseFk, companyFk, addressFk 
+			INTO vCustomer, vWarehouse, vCompany, vAddress
+		FROM ticket 
+        WHERE id = vOriginTicket;
+    
+    SELECT id INTO vZoneFk 
+		FROM zone WHERE agencyModeFk = vRefundAgencyMode
+        LIMIT 1;
+        
+	INSERT INTO vn.ticket (
+			clientFk,
+			shipped,
+			addressFk,
+			agencyModeFk,
+			nickname,
+			warehouseFk,
+			companyFk,
+			landed,
+            zoneFk
+		)
+		SELECT
+					vCustomer,
+					CURDATE(),
+					vAddress,
+					vRefundAgencyMode,
+					a.nickname,
+					vWarehouse,
+					vCompany,
+					CURDATE(),
+                    vZoneFk
+			FROM address a
+			WHERE a.id = vAddress;
+
+	SET vNewTicket = LAST_INSERT_ID();
+
+	SET vDone := 0;	
+	OPEN cSales;
+    FETCH cSales INTO vSaleMain, vItemFk, vQuantity, vConcept, vPrice, vDiscount;
+
+	WHILE NOT vDone DO
+    
+		INSERT INTO vn.sale(ticketFk, itemFk, quantity, concept, price, discount)
+			VALUES( vNewTicket, vItemFk, vQuantity, vConcept, vPrice, vDiscount );
+            
+        SET vSaleNew = LAST_INSERT_ID();
+        
+        INSERT INTO vn.saleComponent(saleFk,componentFk,`value`) 
+        SELECT vSaleNew,componentFk,`value`  
+			FROM vn.saleComponent 
+            WHERE saleFk = vSaleMain;
+        
+		FETCH cSales INTO vSaleMain, vItemFk, vQuantity, vConcept, vPrice, vDiscount;
+        
+	END WHILE;
+	CLOSE cSales;
+
+	SET vDone := 0;	
+	OPEN cTicketServices;
+    FETCH cTicketServices INTO vDescription, vQuantity, vPrice, vTaxClassFk, vTicketServiceTypeFk;
+
+	WHILE NOT vDone DO
+    
+		INSERT INTO vn.ticketService(description, quantity, price, taxClassFk, ticketFk, ticketServiceTypeFk)
+			VALUES(vDescription, vQuantity, vPrice, vTaxClassFk, vNewTicket, vTicketServiceTypeFk);
+        
+		FETCH cTicketServices INTO vDescription, vQuantity, vPrice, vTaxClassFk, vTicketServiceTypeFk;
+        
+	END WHILE;
+	CLOSE cTicketServices;
+
+	INSERT INTO vn.ticketRefund(refundTicketFk, originalTicketFk)
+		VALUES(vNewTicket, vOriginTicket);
+
+END$$
+DELIMITER ;

e2e/helpers/selectors.js

@@ -571,7 +571,7 @@ export default {
         moreMenuUnmarkReseved: 'vn-item[name="unreserve"]',
         moreMenuUpdateDiscount: 'vn-item[name="discount"]',
         moreMenuRecalculatePrice: 'vn-item[name="calculatePrice"]',
-        moreMenuPayBack: 'vn-item[name="payBack"]',
+        moreMenuRefund: 'vn-item[name="refund"]',
         moreMenuUpdateDiscountInput: 'vn-input-number[ng-model="$ctrl.edit.discount"] input',
         transferQuantityInput: '.vn-popover.shown vn-table > div > vn-tbody > vn-tr > vn-td-editable > span > text',
         transferQuantityCell: '.vn-popover.shown vn-table > div > vn-tbody > vn-tr > vn-td-editable',

e2e/paths/05-ticket/01-sale/02_edit_sale.spec.js

@@ -212,10 +212,10 @@ describe('Ticket Edit sale path', () => {
         await page.accessToSection('ticket.card.sale');
     });
 
-    it('should select the third sale and create a pay back', async() => {
+    it('should select the third sale and create a refund', async() => {
         await page.waitToClick(selectors.ticketSales.firstSaleCheckbox);
         await page.waitToClick(selectors.ticketSales.moreMenu);
-        await page.waitToClick(selectors.ticketSales.moreMenuPayBack);
+        await page.waitToClick(selectors.ticketSales.moreMenuRefund);
         await page.waitForState('ticket.card.sale');
     });
 

loopback/locale/es.json

@@ -219,7 +219,7 @@
 	"The worker has a marked absence that day": "El trabajador tiene marcada una ausencia ese día",
 	"You can not modify is pay method checked": "No se puede modificar el campo método de pago validado",
 	"Can't transfer claimed sales": "No puedes transferir lineas reclamadas",
-	"You don't have privileges to create pay back": "No tienes permisos para crear un abono",
+	"You don't have privileges to create refund": "No tienes permisos para crear un abono",
 	"The item is required": "El artículo es requerido",
 	"The agency is already assigned to another autonomous": "La agencia ya está asignada a otro autónomo",
 	"date in the future": "Fecha en el futuro",

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

@@ -1,14 +1,14 @@
 const UserError = require('vn-loopback/util/user-error');
 
 module.exports = Self => {
-    Self.remoteMethodCtx('payBack', {
+    Self.remoteMethodCtx('refund', {
         description: 'Create ticket with the selected lines changing the sign to the quantites',
         accessType: 'WRITE',
         accepts: [{
             arg: 'sales',
             description: 'The sales',
             type: ['object'],
-            required: true
+            required: false
         },
         {
             arg: 'ticketId',
@@ -21,12 +21,12 @@ module.exports = Self => {
             root: true
         },
         http: {
-            path: `/payBack`,
+            path: `/refund`,
             verb: 'post'
         }
     });
 
-    Self.payBack = async(ctx, sales, ticketId, options) => {
+    Self.refund = async(ctx, sales, ticketId, options) => {
         const myOptions = {};
         let tx;
 
@@ -47,21 +47,58 @@ module.exports = Self => {
             const hasValidRole = isClaimManager || isSalesAssistant;
 
             if (!hasValidRole)
-                throw new UserError(`You don't have privileges to create pay back`);
+                throw new UserError(`You don't have privileges to create refund`);
 
-            for (let sale of sales)
-                salesIds.push(sale.id);
+            if (sales) {
+                for (let sale of sales)
+                    salesIds.push(sale.id);
 
-            const query = `
+                const query = `
                 DROP TEMPORARY TABLE IF EXISTS tmp.sale;
+
                 CREATE TEMPORARY TABLE tmp.sale
                     SELECT s.id, s.itemFk, - s.quantity, s.concept, s.price, s.discount
                         FROM sale s
                         WHERE s.id IN (?);
-                CALL vn.ticket_doRefund(?, @newTicket);
-                DROP TEMPORARY TABLE tmp.sale;`;
 
-            await Self.rawSql(query, [salesIds, ticketId], myOptions);
+                CREATE TEMPORARY TABLE tmp.ticketService(
+                    description VARCHAR(50),
+                    quantity DECIMAL (10,2),
+                    price DECIMAL (10,2),
+                    taxClassFk INT,
+                    ticketServiceTypeFk INT
+                );
+                                               
+                CALL vn.ticket_doRefund(?, @newTicket);
+
+                DROP TEMPORARY TABLE tmp.sale;
+                DROP TEMPORARY TABLE tmp.ticketService;`;
+
+                await Self.rawSql(query, [salesIds, ticketId], myOptions);
+            } else {
+                const query = `
+                DROP TEMPORARY TABLE IF EXISTS tmp.sale;
+                DROP TEMPORARY TABLE IF EXISTS tmp.ticketService;
+
+                CREATE TEMPORARY TABLE tmp.sale
+                    SELECT s.id, s.itemFk, - s.quantity, s.concept, s.price, s.discount
+                        FROM sale s
+                            JOIN ticket t ON t.id = s.ticketFk 
+                        WHERE t.id IN (?);
+
+                CREATE TEMPORARY TABLE tmp.ticketService
+                    SELECT ts.description, - ts.quantity, ts.price, ts.taxClassFk, ts.ticketServiceTypeFk 
+                        FROM ticketService ts 
+                        WHERE ts.ticketFk IN (?);
+
+                CALL vn.ticket_doRefund(?, @newTicket);
+
+                DROP TEMPORARY TABLE tmp.sale;
+                DROP TEMPORARY TABLE tmp.ticketService;`;
+
+                await Self.rawSql(query, [ticketId, ticketId, ticketId], myOptions);
+            }
+
             const [newTicket] = await Self.rawSql('SELECT @newTicket id', null, myOptions);
             ticketId = newTicket.id;
 

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

@@ -1,6 +1,6 @@
 const models = require('vn-loopback/server/server').models;
 
-describe('sale payBack()', () => {
+describe('sale refund()', () => {
     it('should create ticket with the selected lines changing the sign to the quantites', async() => {
         const tx = await models.Sale.beginTransaction({});
         const ctx = {req: {accessToken: {userId: 9}}};
@@ -14,7 +14,7 @@ describe('sale payBack()', () => {
         try {
             const options = {transaction: tx};
 
-            const response = await models.Sale.payBack(ctx, sales, ticketId, options);
+            const response = await models.Sale.refund(ctx, sales, ticketId, options);
             const [newTicketId] = await models.Sale.rawSql('SELECT MAX(t.id) id FROM vn.ticket t;', null, options);
 
             expect(response).toEqual(newTicketId.id);
@@ -26,7 +26,7 @@ describe('sale payBack()', () => {
         }
     });
 
-    it(`should throw an error if the user doesn't have privileges to create a pay back`, async() => {
+    it(`should throw an error if the user doesn't have privileges to create a refund`, async() => {
         const tx = await models.Sale.beginTransaction({});
         const ctx = {req: {accessToken: {userId: 1}}};
 
@@ -40,7 +40,7 @@ describe('sale payBack()', () => {
         try {
             const options = {transaction: tx};
 
-            await models.Sale.payBack(ctx, sales, ticketId, options);
+            await models.Sale.refund(ctx, sales, ticketId, options);
 
             await tx.rollback();
         } catch (e) {
@@ -49,6 +49,6 @@ describe('sale payBack()', () => {
         }
 
         expect(error).toBeDefined();
-        expect(error.message).toEqual(`You don't have privileges to create pay back`);
+        expect(error.message).toEqual(`You don't have privileges to create refund`);
     });
 });

modules/ticket/back/models/sale.js

@@ -6,7 +6,7 @@ module.exports = Self => {
     require('../methods/sale/updateQuantity')(Self);
     require('../methods/sale/updateConcept')(Self);
     require('../methods/sale/recalculatePrice')(Self);
-    require('../methods/sale/payBack')(Self);
+    require('../methods/sale/refund')(Self);
     require('../methods/sale/canEdit')(Self);
 
     Self.validatesPresenceOf('concept', {

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

@@ -139,6 +139,11 @@
             translate>
             Recalculate components
         </vn-item>
+        <vn-item
+            ng-click="refundAllConfirmation.show()"
+            translate>
+            Refund all
+        </vn-item>
     </vn-list>
 </vn-menu>
 
@@ -292,4 +297,12 @@
     on-accept="$ctrl.recalculateComponents()"
     question="Are you sure you want to recalculate the components?"
     message="Recalculate components">
+</vn-confirm>
+
+<!-- Refund all confirmation dialog -->
+<vn-confirm
+    vn-id="refundAllConfirmation"
+    on-accept="$ctrl.refundAll()"
+    question="Are you sure you want to refund all?"
+    message="Refund all">
 </vn-confirm>

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

@@ -272,6 +272,14 @@ class Controller extends Section {
             .then(() => this.reload())
             .then(() => this.vnApp.showSuccess(this.$t('Data saved!')));
     }
+
+    refundAll() {
+        const params = {ticketId: this.id};
+        const query = `Sales/refund`;
+        return this.$http.post(query, params).then(res => {
+            this.$state.go('ticket.card.sale', {id: res.data});
+        });
+    }
 }
 
 Controller.$inject = ['$element', '$scope', 'vnReport', 'vnEmail'];

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

@@ -7,4 +7,5 @@ Send PDF: Enviar PDF
 Send CSV: Enviar CSV
 Send CSV Delivery Note: Enviar albarán en CSV
 Send PDF Delivery Note: Enviar albarán en PDF
-Show Proforma: Ver proforma
+Show Proforma: Ver proforma
+Refund all: Abonar todo

modules/ticket/front/descriptor/locale/es.yml

@@ -29,4 +29,5 @@ SMS Minimum import: 'SMS Importe minimo'
 SMS Pending payment: 'SMS Pago pendiente'
 Restore ticket: Restaurar ticket
 You are going to restore this ticket: Vas a restaurar este ticket
-Are you sure you want to restore this ticket?: ¿Seguro que quieres restaurar el ticket?
+Are you sure you want to restore this ticket?: ¿Seguro que quieres restaurar el ticket?
+Are you sure you want to refund all?: ¿Seguro que quieres abonar todo?

modules/ticket/front/sale/index.html

@@ -491,10 +491,10 @@
         Unmark as reserved
     </vn-item>
     <vn-item translate
-        name="payBack"
-        ng-click="$ctrl.createPayBack()"
+        name="refund"
+        ng-click="$ctrl.createRefund()"
         vn-acl="claimManager, salesAssistant"
         vn-acl-action="remove">
-        Pay Back
+        Refund
     </vn-item>
 </vn-menu>

modules/ticket/front/sale/index.js

@@ -462,12 +462,12 @@ class Controller extends Section {
         });
     }
 
-    createPayBack() {
+    createRefund() {
         const sales = this.selectedValidSales();
         if (!sales) return;
 
         const params = {sales: sales, ticketId: this.ticket.id};
-        const query = `Sales/payBack`;
+        const query = `Sales/refund`;
         this.resetChanges();
         this.$http.post(query, params).then(res => {
             this.$state.go('ticket.card.sale', {id: res.data});

modules/ticket/front/sale/index.spec.js

@@ -701,15 +701,15 @@ describe('Ticket', () => {
             });
         });
 
-        describe('createPayBack()', () => {
+        describe('createRefund()', () => {
             it('should make an HTTP POST query and then call to the $state go() method', () => {
                 jest.spyOn(controller, 'selectedValidSales').mockReturnValue(controller.sales);
                 jest.spyOn(controller, 'resetChanges');
                 jest.spyOn(controller.$state, 'go');
 
                 const expectedId = 9999;
-                $httpBackend.expect('POST', `Sales/payBack`).respond(200, expectedId);
-                controller.createPayBack();
+                $httpBackend.expect('POST', `Sales/refund`).respond(200, expectedId);
+                controller.createRefund();
                 $httpBackend.flush();
 
                 expect(controller.resetChanges).toHaveBeenCalledWith();

modules/ticket/front/sale/locale/es.yml

@@ -36,4 +36,4 @@ Warehouse: Almacen
 Agency: Agencia
 Shipped: F. envio
 Packaging: Encajado
-Pay Back: Abono
+Refund: Abono