Merge pull request '5180-invoice-out_refund' (!1325) from 5180-invoice-out_refund into dev

Reviewed-on: #1325
Reviewed-by: Javi Gallego <jgallego@verdnatura.es>
Reviewed-by: Alex Moreno <alexm@verdnatura.es>

db/changes/230801/00-acl_itemConfig.sql

@@ -0,0 +1,3 @@
+INSERT INTO `salix`.`ACL` (`model`, `property`, `accessType`, `permission`, `principalType`, `principalId`)
+    VALUES
+        ('ItemConfig', '*', '*', 'ALLOW', 'ROLE', 'employee');

modules/invoiceOut/back/methods/invoiceOut/refund.js

@@ -35,8 +35,7 @@ module.exports = Self => {
             const tickets = await models.Ticket.find(filter, myOptions);
 
             const ticketsIds = tickets.map(ticket => ticket.id);
-
-            const refundedTickets = await models.Ticket.refund(ticketsIds, myOptions);
+            const refundedTickets = await models.Ticket.refund(ticketsIds, true, myOptions);
 
             if (tx) await tx.commit();
 

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

@@ -17,7 +17,7 @@ describe('InvoiceOut refund()', () => {
         try {
             const result = await models.InvoiceOut.refund('T1111111', options);
 
-            expect(result.length).toEqual(2);
+            expect(result.length).toEqual(1);
 
             await tx.rollback();
         } catch (e) {

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

@@ -12,6 +12,11 @@ module.exports = Self => {
                 arg: 'servicesIds',
                 type: ['number']
             },
+            {
+                arg: 'createSingleTicket',
+                type: 'boolean',
+                required: false
+            }
         ],
         returns: {
             type: ['number'],
@@ -23,7 +28,7 @@ module.exports = Self => {
         }
     });
 
-    Self.refund = async(salesIds, servicesIds, options) => {
+    Self.refund = async(salesIds, servicesIds, createSingleTicket = false, options) => {
         const models = Self.app.models;
         const myOptions = {};
         let tx;
@@ -63,38 +68,37 @@ module.exports = Self => {
             const ticketsIds = [...new Set(sales.map(sale => sale.ticketFk))];
 
             const refundTickets = [];
-
-            const now = Date.vnNew();
             const mappedTickets = new Map();
+            const now = Date.vnNew();
 
-            for (let ticketId of ticketsIds) {
-                const filter = {include: {relation: 'address'}};
-                const ticket = await models.Ticket.findById(ticketId, filter, myOptions);
-
-                const refundTicket = await models.Ticket.create({
-                    clientFk: ticket.clientFk,
-                    shipped: now,
-                    addressFk: ticket.address().id,
-                    agencyModeFk: refundAgencyMode.id,
-                    nickname: ticket.address().nickname,
-                    warehouseFk: ticket.warehouseFk,
-                    companyFk: ticket.companyFk,
-                    landed: now,
-                    zoneFk: refoundZoneId
-                }, myOptions);
-
-                refundTickets.push(refundTicket);
-
-                mappedTickets.set(ticketId, refundTicket.id);
-
-                await models.TicketRefund.create({
-                    refundTicketFk: refundTicket.id,
-                    originalTicketFk: ticket.id,
-                }, myOptions);
+            const [firstTicketId] = ticketsIds;
+            if (createSingleTicket) {
+                await createTicketRefund(
+                    firstTicketId,
+                    refundTickets,
+                    mappedTickets,
+                    now,
+                    refundAgencyMode,
+                    refoundZoneId,
+                    myOptions
+                );
+            } else {
+                for (let ticketId of ticketsIds) {
+                    await createTicketRefund(
+                        ticketId,
+                        refundTickets,
+                        mappedTickets,
+                        now,
+                        refundAgencyMode,
+                        refoundZoneId,
+                        myOptions
+                    );
+                }
             }
 
             for (const sale of sales) {
-                const refundTicketId = mappedTickets.get(sale.ticketFk);
+                const refundTicketId = await getTicketRefundId(createSingleTicket, sale.ticketFk, refundTickets, mappedTickets);
+
                 const createdSale = await models.Sale.create({
                     ticketFk: refundTicketId,
                     itemFk: sale.itemFk,
@@ -118,7 +122,7 @@ module.exports = Self => {
                 const services = await models.TicketService.find(servicesFilter, myOptions);
 
                 for (const service of services) {
-                    const refundTicketId = mappedTickets.get(service.ticketFk);
+                    const refundTicketId = await getTicketRefundId(createSingleTicket, service.ticketFk, refundTickets, mappedTickets);
 
                     await models.TicketService.create({
                         description: service.description,
@@ -139,4 +143,47 @@ module.exports = Self => {
             throw e;
         }
     };
+
+    async function createTicketRefund(
+        ticketId,
+        refundTickets,
+        mappedTickets,
+        now,
+        refundAgencyMode,
+        refoundZoneId,
+        myOptions
+    ) {
+        const models = Self.app.models;
+
+        const filter = {include: {relation: 'address'}};
+        const ticket = await models.Ticket.findById(ticketId, filter, myOptions);
+
+        const refundTicket = await models.Ticket.create({
+            clientFk: ticket.clientFk,
+            shipped: now,
+            addressFk: ticket.address().id,
+            agencyModeFk: refundAgencyMode.id,
+            nickname: ticket.address().nickname,
+            warehouseFk: ticket.warehouseFk,
+            companyFk: ticket.companyFk,
+            landed: now,
+            zoneFk: refoundZoneId
+        }, myOptions);
+
+        refundTickets.push(refundTicket);
+
+        mappedTickets.set(ticketId, refundTicket.id);
+
+        await models.TicketRefund.create({
+            refundTicketFk: refundTicket.id,
+            originalTicketFk: ticket.id,
+        }, myOptions);
+    }
+
+    async function getTicketRefundId(createSingleTicket, ticketId, refundTickets, mappedTickets) {
+        if (createSingleTicket) {
+            const [firstRefundTicket] = refundTickets;
+            return firstRefundTicket.id;
+        } else return mappedTickets.get(ticketId);
+    }
 };

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

@@ -1,7 +1,7 @@
 const models = require('vn-loopback/server/server').models;
 const LoopBackContext = require('loopback-context');
 
-describe('sale refund()', () => {
+describe('Sale refund()', () => {
     const userId = 5;
     const activeCtx = {
         accessToken: {userId: userId},
@@ -22,7 +22,7 @@ describe('sale refund()', () => {
         try {
             const options = {transaction: tx};
 
-            const response = await models.Sale.refund(salesIds, servicesIds, options);
+            const response = await models.Sale.refund(salesIds, servicesIds, false, options);
 
             expect(response.length).toBeGreaterThanOrEqual(1);
 
@@ -40,7 +40,8 @@ describe('sale refund()', () => {
         try {
             const options = {transaction: tx};
 
-            const tickets = await models.Sale.refund(salesIds, servicesIds, options);
+            const createSingleTicket = false;
+            const tickets = await models.Sale.refund(salesIds, servicesIds, createSingleTicket, options);
 
             const ticketsIds = tickets.map(ticket => ticket.id);
 

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

@@ -8,6 +8,11 @@ module.exports = Self => {
                 type: ['number'],
                 required: true
             },
+            {
+                arg: 'createSingleTicket',
+                type: 'boolean',
+                required: false
+            }
         ],
         returns: {
             type: ['number'],
@@ -19,7 +24,7 @@ module.exports = Self => {
         }
     });
 
-    Self.refund = async(ticketsIds, options) => {
+    Self.refund = async(ticketsIds, createSingleTicket = false, options) => {
         const models = Self.app.models;
         const myOptions = {};
         let tx;
@@ -41,7 +46,7 @@ module.exports = Self => {
             const services = await models.TicketService.find(filter, myOptions);
             const servicesIds = services.map(service => service.id);
 
-            const refundedTickets = await models.Sale.refund(salesIds, servicesIds, myOptions);
+            const refundedTickets = await models.Sale.refund(salesIds, servicesIds, createSingleTicket, myOptions);
 
             if (tx) await tx.commit();