#6335 -ticketAdvance_splitOrCreate #1817
|
@ -8,8 +8,13 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
|
||||||
## [2348.01] - 2023-11-30
|
## [2348.01] - 2023-11-30
|
||||||
|
|
||||||
### Added
|
### Added
|
||||||
|
- (Ticket -> Adelantar) Permite mover lineas sin generar negativos
|
||||||
|
- (Ticket -> Adelantar) Permite modificar la fecha de los tickets
|
||||||
|
|
||||||
### Changed
|
### Changed
|
||||||
### Fixed
|
### Fixed
|
||||||
|
- (Ticket -> RocketChat) Arreglada detección de cambios
|
||||||
|
|
||||||
|
|
||||||
## [2346.01] - 2023-11-16
|
## [2346.01] - 2023-11-16
|
||||||
|
|
||||||
|
|
|
@ -0,0 +1,152 @@
|
||||||
|
DELIMITER $$
|
||||||
|
$$
|
||||||
|
CREATE OR REPLACE DEFINER=`root`@`localhost` PROCEDURE `vn`.`ticket_canAdvance`(vDateFuture DATE, vDateToAdvance DATE, vWarehouseFk INT)
|
||||||
alexm marked this conversation as resolved
|
|||||||
|
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 config;
|
||||||
|
|
||||||
|
CREATE OR REPLACE 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 itemTicketOut
|
||||||
|
WHERE shipped >= vDateInventory
|
||||||
|
AND shipped < vDateFuture
|
||||||
|
AND warehouseFk = vWarehouseFk
|
||||||
|
UNION ALL
|
||||||
|
SELECT itemFk, quantity
|
||||||
|
FROM itemEntryIn
|
||||||
|
WHERE landed >= vDateInventory
|
||||||
|
AND landed <= vDateToAdvance
|
||||||
alexm
commented
Aci he canviat que possava < vDateFuture Aci he canviat que possava < vDateFuture
|
|||||||
|
AND isVirtualStock = FALSE
|
||||||
|
AND warehouseInFk = vWarehouseFk
|
||||||
|
UNION ALL
|
||||||
|
SELECT itemFk, quantity
|
||||||
|
FROM itemEntryOut
|
||||||
|
WHERE shipped >= vDateInventory
|
||||||
|
AND shipped < vDateFuture
|
||||||
|
AND warehouseOutFk = vWarehouseFk
|
||||||
|
) t
|
||||||
|
GROUP BY itemFk HAVING amount != 0;
|
||||||
|
|
||||||
|
CREATE OR REPLACE TEMPORARY TABLE tmp.filter
|
||||||
|
(INDEX (id))
|
||||||
|
SELECT
|
||||||
|
origin.ticketFk futureId,
|
||||||
|
dest.ticketFk id,
|
||||||
|
dest.state,
|
||||||
|
origin.futureState,
|
||||||
|
origin.futureIpt,
|
||||||
|
dest.ipt,
|
||||||
|
origin.workerFk,
|
||||||
|
origin.futureLiters,
|
||||||
|
origin.futureLines,
|
||||||
|
dest.shipped,
|
||||||
|
origin.shipped futureShipped,
|
||||||
|
dest.totalWithVat,
|
||||||
|
origin.totalWithVat futureTotalWithVat,
|
||||||
|
dest.agency,
|
||||||
|
dest.agencyModeFk,
|
||||||
|
origin.futureAgency,
|
||||||
|
origin.agencyModeFk futureAgencyModeFk,
|
||||||
|
dest.lines,
|
||||||
|
dest.liters,
|
||||||
|
origin.futureLines - origin.hasStock AS notMovableLines,
|
||||||
|
(origin.futureLines = origin.hasStock) AS isFullMovable,
|
||||||
|
dest.zoneFk,
|
||||||
|
origin.futureZoneFk,
|
||||||
|
origin.futureZoneName,
|
||||||
|
origin.classColor futureClassColor,
|
||||||
|
dest.classColor,
|
||||||
|
origin.clientFk futureClientFk,
|
||||||
|
origin.addressFk futureAddressFk,
|
||||||
|
origin.warehouseFk futureWarehouseFk,
|
||||||
|
origin.companyFk futureCompanyFk,
|
||||||
|
IFNULL(dest.nickname, origin.nickname) nickname,
|
||||||
|
dest.landed
|
||||||
|
FROM (
|
||||||
|
SELECT
|
||||||
|
s.ticketFk,
|
||||||
|
c.salesPersonFk workerFk,
|
||||||
|
t.shipped,
|
||||||
|
t.totalWithVat,
|
||||||
|
st.name futureState,
|
||||||
|
am.name futureAgency,
|
||||||
|
count(s.id) futureLines,
|
||||||
|
GROUP_CONCAT(DISTINCT ipt.code ORDER BY ipt.code) futureIpt,
|
||||||
|
CAST(SUM(litros) AS DECIMAL(10,0)) futureLiters,
|
||||||
|
SUM((s.quantity <= IFNULL(st.amount,0))) hasStock,
|
||||||
|
z.id futureZoneFk,
|
||||||
alexm
commented
Añadidos campos necesarios para el front Añadidos campos necesarios para el front
|
|||||||
|
z.name futureZoneName,
|
||||||
|
st.classColor,
|
||||||
|
t.clientFk,
|
||||||
|
t.nickname,
|
||||||
|
t.addressFk,
|
||||||
|
t.warehouseFk,
|
||||||
|
t.companyFk,
|
||||||
|
t.agencyModeFk
|
||||||
|
FROM ticket t
|
||||||
|
JOIN client c ON c.id = t.clientFk
|
||||||
|
JOIN sale s ON s.ticketFk = t.id
|
||||||
|
JOIN saleVolume sv ON sv.saleFk = s.id
|
||||||
|
JOIN item i ON i.id = s.itemFk
|
||||||
|
JOIN ticketState ts ON ts.ticketFk = t.id
|
||||||
|
JOIN state st ON st.id = ts.stateFk
|
||||||
|
JOIN agencyMode am ON t.agencyModeFk = am.id
|
||||||
|
JOIN zone z ON t.zoneFk = z.id
|
||||||
|
LEFT JOIN itemPackingType ipt ON ipt.code = i.itemPackingTypeFk
|
||||||
|
LEFT JOIN tmp.stock st ON st.itemFk = i.id
|
||||||
|
WHERE t.shipped BETWEEN vDateFuture AND util.dayend(vDateFuture)
|
||||||
|
AND t.warehouseFk = vWarehouseFk
|
||||||
|
GROUP BY t.id
|
||||||
|
) origin
|
||||||
|
LEFT JOIN (
|
||||||
alexm
commented
Añadido LEFT JOIN Añadido LEFT JOIN
|
|||||||
|
SELECT
|
||||||
|
t.id ticketFk,
|
||||||
|
st.name state,
|
||||||
|
GROUP_CONCAT(DISTINCT ipt.code ORDER BY ipt.code) ipt,
|
||||||
|
t.shipped,
|
||||||
|
t.totalWithVat,
|
||||||
|
am.name agency,
|
||||||
|
CAST(SUM(litros) AS DECIMAL(10,0)) liters,
|
||||||
|
CAST(COUNT(*) AS DECIMAL(10,0)) `lines`,
|
||||||
|
st.classColor,
|
||||||
|
t.clientFk,
|
||||||
|
t.nickname,
|
||||||
|
t.addressFk,
|
||||||
|
t.zoneFk,
|
||||||
|
t.warehouseFk,
|
||||||
|
t.companyFk,
|
||||||
|
t.landed,
|
||||||
|
t.agencyModeFk
|
||||||
|
FROM ticket t
|
||||||
|
JOIN sale s ON s.ticketFk = t.id
|
||||||
|
JOIN saleVolume sv ON sv.saleFk = s.id
|
||||||
|
JOIN item i ON i.id = s.itemFk
|
||||||
|
JOIN ticketState ts ON ts.ticketFk = t.id
|
||||||
|
JOIN state st ON st.id = ts.stateFk
|
||||||
|
JOIN agencyMode am ON t.agencyModeFk = am.id
|
||||||
|
LEFT JOIN itemPackingType ipt ON ipt.code = i.itemPackingTypeFk
|
||||||
|
WHERE t.shipped BETWEEN vDateToAdvance AND util.dayend(vDateToAdvance)
|
||||||
|
AND t.warehouseFk = vWarehouseFk
|
||||||
|
AND st.order <= 5
|
||||||
|
GROUP BY t.id
|
||||||
|
) dest ON dest.addressFk = origin.addressFk
|
||||||
|
WHERE origin.hasStock;
|
||||||
|
|
||||||
|
DROP TEMPORARY TABLE tmp.stock;
|
||||||
|
END$$
|
||||||
|
DELIMITER ;
|
|
@ -722,7 +722,7 @@ export default {
|
||||||
isFullMovable: 'vn-check[ng-model="filter.isFullMovable"]',
|
isFullMovable: 'vn-check[ng-model="filter.isFullMovable"]',
|
||||||
warehouseFk: 'vn-autocomplete[label="Warehouse"]',
|
warehouseFk: 'vn-autocomplete[label="Warehouse"]',
|
||||||
tableButtonSearch: 'vn-button[vn-tooltip="Search"]',
|
tableButtonSearch: 'vn-button[vn-tooltip="Search"]',
|
||||||
moveButton: 'vn-button[vn-tooltip="Advance tickets"]',
|
moveButton: 'vn-button[vn-tooltip="Advance tickets with negatives"]',
|
||||||
acceptButton: '.vn-confirm.shown button[response="accept"]',
|
acceptButton: '.vn-confirm.shown button[response="accept"]',
|
||||||
firstCheck: 'tbody > tr:nth-child(2) > td > vn-check',
|
firstCheck: 'tbody > tr:nth-child(2) > td > vn-check',
|
||||||
tableId: 'vn-textfield[name="id"]',
|
tableId: 'vn-textfield[name="id"]',
|
||||||
|
|
|
@ -198,4 +198,4 @@
|
||||||
"The ticket is in preparation": "The ticket [{{ticketId}}]({{{ticketUrl}}}) of the sales person {{salesPersonId}} is in preparation",
|
"The ticket is in preparation": "The ticket [{{ticketId}}]({{{ticketUrl}}}) of the sales person {{salesPersonId}} is in preparation",
|
||||||
"You can only add negative amounts in refund tickets": "You can only add negative amounts in refund tickets",
|
"You can only add negative amounts in refund tickets": "You can only add negative amounts in refund tickets",
|
||||||
"Try again": "Try again"
|
"Try again": "Try again"
|
||||||
alexm marked this conversation as resolved
Outdated
jgallego
commented
d'on ix un text en camel case? d'on ix un text en camel case?
|
|||||||
}
|
}
|
||||||
|
|
|
@ -224,7 +224,7 @@
|
||||||
"date in the future": "Fecha en el futuro",
|
"date in the future": "Fecha en el futuro",
|
||||||
"reference duplicated": "Referencia duplicada",
|
"reference duplicated": "Referencia duplicada",
|
||||||
"This ticket is already a refund": "Este ticket ya es un abono",
|
"This ticket is already a refund": "Este ticket ya es un abono",
|
||||||
"isWithoutNegatives": "isWithoutNegatives",
|
"isWithoutNegatives": "Sin negativos",
|
||||||
"routeFk": "routeFk",
|
"routeFk": "routeFk",
|
||||||
"Can't change the password of another worker": "No se puede cambiar la contraseña de otro trabajador",
|
"Can't change the password of another worker": "No se puede cambiar la contraseña de otro trabajador",
|
||||||
"No hay un contrato en vigor": "No hay un contrato en vigor",
|
"No hay un contrato en vigor": "No hay un contrato en vigor",
|
||||||
|
|
|
@ -75,8 +75,7 @@ module.exports = Self => {
|
||||||
{
|
{
|
||||||
arg: 'option',
|
arg: 'option',
|
||||||
type: 'number',
|
type: 'number',
|
||||||
description: 'Action id',
|
description: 'Action id'
|
||||||
required: true
|
|
||||||
},
|
},
|
||||||
{
|
{
|
||||||
arg: 'isWithoutNegatives',
|
arg: 'isWithoutNegatives',
|
||||||
|
@ -88,7 +87,18 @@ module.exports = Self => {
|
||||||
arg: 'withWarningAccept',
|
arg: 'withWarningAccept',
|
||||||
type: 'boolean',
|
type: 'boolean',
|
||||||
description: 'Has pressed in confirm message',
|
description: 'Has pressed in confirm message',
|
||||||
}],
|
},
|
||||||
|
{
|
||||||
|
arg: 'newTicket',
|
||||||
|
type: 'number',
|
||||||
|
description: 'Ticket id you want to move the lines to, if applicable',
|
||||||
|
},
|
||||||
|
{
|
||||||
|
arg: 'keepPrice',
|
||||||
|
type: 'boolean',
|
||||||
|
description: 'If prices should be maintained',
|
||||||
|
}
|
||||||
|
],
|
||||||
returns: {
|
returns: {
|
||||||
type: ['object'],
|
type: ['object'],
|
||||||
root: true
|
root: true
|
||||||
|
@ -141,12 +151,18 @@ module.exports = Self => {
|
||||||
|
|
||||||
const salesNewTicketLength = salesNewTicket.length;
|
const salesNewTicketLength = salesNewTicket.length;
|
||||||
if (salesNewTicketLength && sales.length != salesNewTicketLength) {
|
if (salesNewTicketLength && sales.length != salesNewTicketLength) {
|
||||||
const newTicket = await models.Ticket.transferSales(ctx, args.id, null, salesNewTicket, myOptions);
|
const newTicket = await models.Ticket.transferSales(
|
||||||
|
ctx,
|
||||||
|
args.id,
|
||||||
|
args.newTicket,
|
||||||
|
salesNewTicket,
|
||||||
|
myOptions
|
||||||
|
);
|
||||||
args.id = newTicket.id;
|
args.id = newTicket.id;
|
||||||
alexm marked this conversation as resolved
alexm
commented
revisar revisar
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
const originalTicket = await models.Ticket.findOne({
|
const ticketToChange = await models.Ticket.findOne({
|
||||||
where: {id: args.id},
|
where: {id: args.id},
|
||||||
fields: [
|
fields: [
|
||||||
'id',
|
'id',
|
||||||
|
@ -179,33 +195,38 @@ module.exports = Self => {
|
||||||
},
|
},
|
||||||
]
|
]
|
||||||
}, myOptions);
|
}, myOptions);
|
||||||
|
const originalTicket = JSON.parse(JSON.stringify(ticketToChange));
|
||||||
|
const ticketChanges = {
|
||||||
alexm
commented
changes changes
|
|||||||
|
clientFk: args.clientFk,
|
||||||
|
nickname: args.nickname,
|
||||||
|
agencyModeFk: args.agencyModeFk,
|
||||||
|
addressFk: args.addressFk,
|
||||||
|
zoneFk: args.zoneFk,
|
||||||
|
warehouseFk: args.warehouseFk,
|
||||||
|
companyFk: args.companyFk,
|
||||||
|
shipped: args.shipped,
|
||||||
|
landed: args.landed,
|
||||||
|
isDeleted: args.isDeleted
|
||||||
|
};
|
||||||
|
|
||||||
args.routeFk = null;
|
let response;
|
||||||
|
if (args.keepPrice) {
|
||||||
|
ticketChanges.routeFk = null;
|
||||||
|
response = await ticketToChange.updateAttributes(ticketChanges, myOptions);
|
||||||
|
} else {
|
||||||
|
const hasToBeUnrouted = true;
|
||||||
|
response = await Self.rawSql(
|
||||||
|
'CALL vn.ticket_componentMakeUpdate(?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?)',
|
||||||
|
[args.id].concat(Object.values(ticketChanges), [hasToBeUnrouted, args.option]),
|
||||||
|
myOptions
|
||||||
|
);
|
||||||
|
}
|
||||||
if (args.isWithoutNegatives === false) delete args.isWithoutNegatives;
|
if (args.isWithoutNegatives === false) delete args.isWithoutNegatives;
|
||||||
const updatedTicket = Object.assign({}, args);
|
const updatedTicket = Object.assign({}, args);
|
||||||
delete updatedTicket.ctx;
|
delete updatedTicket.ctx;
|
||||||
delete updatedTicket.option;
|
delete updatedTicket.option;
|
||||||
|
|
||||||
// Force to unroute ticket
|
if (ticketToChange.addressFk != updatedTicket.addressFk && args.id) {
|
||||||
const hasToBeUnrouted = true;
|
|
||||||
const query = 'CALL vn.ticket_componentMakeUpdate(?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?)';
|
|
||||||
const res = await Self.rawSql(query, [
|
|
||||||
args.id,
|
|
||||||
args.clientFk,
|
|
||||||
args.nickname,
|
|
||||||
args.agencyModeFk,
|
|
||||||
args.addressFk,
|
|
||||||
args.zoneFk,
|
|
||||||
args.warehouseFk,
|
|
||||||
args.companyFk,
|
|
||||||
args.shipped,
|
|
||||||
args.landed,
|
|
||||||
args.isDeleted,
|
|
||||||
hasToBeUnrouted,
|
|
||||||
args.option
|
|
||||||
], myOptions);
|
|
||||||
|
|
||||||
if (originalTicket.addressFk != updatedTicket.addressFk && args.id) {
|
|
||||||
await models.TicketObservation.destroyAll({
|
await models.TicketObservation.destroyAll({
|
||||||
ticketFk: args.id
|
ticketFk: args.id
|
||||||
}, myOptions);
|
}, myOptions);
|
||||||
|
@ -232,10 +253,15 @@ module.exports = Self => {
|
||||||
const hasChanges = Object.keys(changes.old).length > 0 || Object.keys(changes.new).length > 0;
|
const hasChanges = Object.keys(changes.old).length > 0 || Object.keys(changes.new).length > 0;
|
||||||
|
|
||||||
if (hasChanges) {
|
if (hasChanges) {
|
||||||
|
delete changes.old.client;
|
||||||
|
delete changes.old.address;
|
||||||
|
delete changes.new.client;
|
||||||
|
delete changes.new.address;
|
||||||
|
|
||||||
const oldProperties = await loggable.translateValues(Self, changes.old);
|
const oldProperties = await loggable.translateValues(Self, changes.old);
|
||||||
const newProperties = await loggable.translateValues(Self, changes.new);
|
const newProperties = await loggable.translateValues(Self, changes.new);
|
||||||
|
|
||||||
const salesPersonId = originalTicket.client().salesPersonFk;
|
const salesPersonId = ticketToChange.client().salesPersonFk;
|
||||||
if (salesPersonId) {
|
if (salesPersonId) {
|
||||||
const url = await Self.app.models.Url.getUrl();
|
const url = await Self.app.models.Url.getUrl();
|
||||||
|
|
||||||
|
@ -256,10 +282,10 @@ module.exports = Self => {
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
res.id = args.id;
|
response.id = args.id;
|
||||||
if (tx) await tx.commit();
|
if (tx) await tx.commit();
|
||||||
|
|
||||||
return res;
|
return response;
|
||||||
} catch (e) {
|
} catch (e) {
|
||||||
if (tx) await tx.rollback();
|
if (tx) await tx.rollback();
|
||||||
throw e;
|
throw e;
|
||||||
|
|
|
@ -209,4 +209,112 @@ describe('ticket componentUpdate()', () => {
|
||||||
throw e;
|
throw e;
|
||||||
}
|
}
|
||||||
});
|
});
|
||||||
|
|
||||||
|
describe('componentUpdate() keepPrice', () => {
|
||||||
|
it('should change shipped and keep price', async() => {
|
||||||
|
const tx = await models.Ticket.beginTransaction({});
|
||||||
|
|
||||||
|
try {
|
||||||
|
const options = {transaction: tx};
|
||||||
|
|
||||||
|
const saleId = 27;
|
||||||
|
const originDate = today;
|
||||||
|
const newDate = tomorrow;
|
||||||
|
const ticketID = 14;
|
||||||
|
newDate.setHours(0, 0, 0, 0, 0);
|
||||||
|
originDate.setHours(0, 0, 0, 0, 0);
|
||||||
|
|
||||||
|
const args = {
|
||||||
|
id: ticketID,
|
||||||
|
clientFk: 1104,
|
||||||
|
agencyModeFk: 2,
|
||||||
|
addressFk: 4,
|
||||||
|
zoneFk: 9,
|
||||||
|
warehouseFk: 1,
|
||||||
|
companyFk: 442,
|
||||||
|
shipped: newDate,
|
||||||
|
landed: tomorrow,
|
||||||
|
isDeleted: false,
|
||||||
|
option: 1,
|
||||||
|
isWithoutNegatives: false,
|
||||||
|
keepPrice: true
|
||||||
|
};
|
||||||
|
|
||||||
|
const ctx = {
|
||||||
|
args: args,
|
||||||
|
req: {
|
||||||
|
accessToken: {userId: 9},
|
||||||
|
headers: {origin: 'http://localhost'},
|
||||||
|
__: value => {
|
||||||
|
return value;
|
||||||
|
}
|
||||||
|
}
|
||||||
|
};
|
||||||
|
|
||||||
|
const beforeSale = await models.Sale.findById(saleId, null, options);
|
||||||
|
await models.Ticket.componentUpdate(ctx, options);
|
||||||
|
const afterSale = await models.Sale.findById(saleId, null, options);
|
||||||
|
|
||||||
|
expect(beforeSale.price).toEqual(afterSale.price);
|
||||||
|
|
||||||
|
await tx.rollback();
|
||||||
|
} catch (e) {
|
||||||
|
await tx.rollback();
|
||||||
|
throw e;
|
||||||
|
}
|
||||||
|
});
|
||||||
|
|
||||||
|
it('should change shipped and not keep price', async() => {
|
||||||
|
const tx = await models.Ticket.beginTransaction({});
|
||||||
|
|
||||||
|
try {
|
||||||
|
const options = {transaction: tx};
|
||||||
|
|
||||||
|
const saleId = 27;
|
||||||
|
const originDate = today;
|
||||||
|
const newDate = tomorrow;
|
||||||
|
const ticketID = 14;
|
||||||
|
newDate.setHours(0, 0, 0, 0, 0);
|
||||||
|
originDate.setHours(0, 0, 0, 0, 0);
|
||||||
|
|
||||||
|
const args = {
|
||||||
|
id: ticketID,
|
||||||
|
clientFk: 1104,
|
||||||
|
agencyModeFk: 2,
|
||||||
|
addressFk: 4,
|
||||||
|
zoneFk: 9,
|
||||||
|
warehouseFk: 1,
|
||||||
|
companyFk: 442,
|
||||||
|
shipped: newDate,
|
||||||
|
landed: tomorrow,
|
||||||
|
isDeleted: false,
|
||||||
|
option: 1,
|
||||||
|
isWithoutNegatives: false,
|
||||||
|
keepPrice: false
|
||||||
|
};
|
||||||
|
|
||||||
|
const ctx = {
|
||||||
|
args: args,
|
||||||
|
req: {
|
||||||
|
accessToken: {userId: 9},
|
||||||
|
headers: {origin: 'http://localhost'},
|
||||||
|
__: value => {
|
||||||
|
return value;
|
||||||
|
}
|
||||||
|
}
|
||||||
|
};
|
||||||
|
|
||||||
|
const beforeSale = await models.Sale.findById(saleId, null, options);
|
||||||
|
await models.Ticket.componentUpdate(ctx, options);
|
||||||
|
const afterSale = await models.Sale.findById(saleId, null, options);
|
||||||
|
|
||||||
|
expect(beforeSale.price).not.toEqual(afterSale.price);
|
||||||
|
|
||||||
|
await tx.rollback();
|
||||||
|
} catch (e) {
|
||||||
|
await tx.rollback();
|
||||||
|
throw e;
|
||||||
|
}
|
||||||
|
});
|
||||||
|
});
|
||||||
});
|
});
|
||||||
|
|
|
@ -21,10 +21,17 @@
|
||||||
expr-builder="$ctrl.exprBuilder(param, value)"
|
expr-builder="$ctrl.exprBuilder(param, value)"
|
||||||
>
|
>
|
||||||
<slot-actions>
|
<slot-actions>
|
||||||
<vn-button disabled="$ctrl.checked.length === 0"
|
<vn-button
|
||||||
|
disabled="$ctrl.checked.length === 0"
|
||||||
icon="keyboard_double_arrow_left"
|
icon="keyboard_double_arrow_left"
|
||||||
ng-click="moveTicketsAdvance.show($event)"
|
ng-click="moveTicketsAdvance.show($event)"
|
||||||
vn-tooltip="Advance tickets">
|
vn-tooltip="Advance tickets with negatives">
|
||||||
|
</vn-button>
|
||||||
|
<vn-button
|
||||||
|
disabled="$ctrl.checked.length === 0"
|
||||||
|
icon="alt_route"
|
||||||
|
ng-click="splitTickets.show($event)"
|
||||||
|
vn-tooltip="Advance tickets without negatives">
|
||||||
</vn-button>
|
</vn-button>
|
||||||
</slot-actions>
|
</slot-actions>
|
||||||
<slot-table>
|
<slot-table>
|
||||||
|
@ -32,8 +39,14 @@
|
||||||
<thead>
|
<thead>
|
||||||
<tr second-header>
|
<tr second-header>
|
||||||
<td></td>
|
<td></td>
|
||||||
<th colspan="7" translate>Destination</th>
|
<th colspan="7">
|
||||||
<th colspan="9" translate>Origin</th>
|
<span translate>Destination</span>
|
||||||
|
{{model.userParams.dateToAdvance| date: 'dd/MM/yyyy'}}
|
||||||
|
</th>
|
||||||
|
<th colspan="11">
|
||||||
|
<span translate>Origin</span>
|
||||||
|
{{model.userParams.dateFuture | date: 'dd/MM/yyyy'}}
|
||||||
|
</th>
|
||||||
</tr>
|
</tr>
|
||||||
<tr>
|
<tr>
|
||||||
<th shrink>
|
<th shrink>
|
||||||
|
@ -48,9 +61,6 @@
|
||||||
<th field="id">
|
<th field="id">
|
||||||
<span translate>ID</span>
|
<span translate>ID</span>
|
||||||
</th>
|
</th>
|
||||||
<th field="shipped">
|
|
||||||
<span translate>Date</span>
|
|
||||||
</th>
|
|
||||||
<th field="ipt" title="{{'Item Packing Type' | translate}}">
|
<th field="ipt" title="{{'Item Packing Type' | translate}}">
|
||||||
<span>IPT</span>
|
<span>IPT</span>
|
||||||
</th>
|
</th>
|
||||||
|
@ -69,9 +79,6 @@
|
||||||
<th separator field="futureId">
|
<th separator field="futureId">
|
||||||
<span translate>ID</span>
|
<span translate>ID</span>
|
||||||
</th>
|
</th>
|
||||||
<th field="futureShipped">
|
|
||||||
<span translate>Date</span>
|
|
||||||
</th>
|
|
||||||
<th field="futureIpt" title="{{'Item Packing Type' | translate}}">
|
<th field="futureIpt" title="{{'Item Packing Type' | translate}}">
|
||||||
<span>IPT</span>
|
<span>IPT</span>
|
||||||
</th>
|
</th>
|
||||||
|
@ -105,7 +112,7 @@
|
||||||
</td>
|
</td>
|
||||||
<td>
|
<td>
|
||||||
<vn-icon
|
<vn-icon
|
||||||
ng-show="ticket.futureAgency !== ticket.agency"
|
ng-show="ticket.futureAgency !== ticket.agency && ticket.agency"
|
||||||
icon="icon-agency-term"
|
icon="icon-agency-term"
|
||||||
title="{{$ctrl.agencies(ticket.futureAgency, ticket.agency)}}">
|
title="{{$ctrl.agencies(ticket.futureAgency, ticket.agency)}}">
|
||||||
</vn-icon>
|
</vn-icon>
|
||||||
|
@ -114,12 +121,7 @@
|
||||||
<span
|
<span
|
||||||
ng-click="ticketDescriptor.show($event, ticket.id)"
|
ng-click="ticketDescriptor.show($event, ticket.id)"
|
||||||
class="link">
|
class="link">
|
||||||
{{::ticket.id | dashIfEmpty}}
|
{{::ticket.id}}
|
||||||
</span>
|
|
||||||
</td>
|
|
||||||
<td shrink-date>
|
|
||||||
<span class="chip {{$ctrl.compareDate(ticket.shipped)}}">
|
|
||||||
{{::ticket.shipped | date: 'dd/MM/yyyy'}}
|
|
||||||
</span>
|
</span>
|
||||||
</td>
|
</td>
|
||||||
<td>{{::ticket.ipt | dashIfEmpty}}</td>
|
<td>{{::ticket.ipt | dashIfEmpty}}</td>
|
||||||
|
@ -135,7 +137,7 @@
|
||||||
<span
|
<span
|
||||||
class="chip {{$ctrl.totalPriceColor(ticket.totalWithVat)}}"
|
class="chip {{$ctrl.totalPriceColor(ticket.totalWithVat)}}"
|
||||||
title="{{$ctrl.totalPriceTitle(ticket.totalWithVat) | translate}}">
|
title="{{$ctrl.totalPriceTitle(ticket.totalWithVat) | translate}}">
|
||||||
{{::(ticket.totalWithVat ? ticket.totalWithVat : 0) | currency: 'EUR': 2}}
|
{{::ticket.totalWithVat | currency: 'EUR': 2}}
|
||||||
</span>
|
</span>
|
||||||
</td>
|
</td>
|
||||||
<td separator>
|
<td separator>
|
||||||
|
@ -145,11 +147,6 @@
|
||||||
{{::ticket.futureId | dashIfEmpty}}
|
{{::ticket.futureId | dashIfEmpty}}
|
||||||
</span>
|
</span>
|
||||||
</td>
|
</td>
|
||||||
<td shrink-date>
|
|
||||||
<span class="chip {{$ctrl.compareDate(ticket.futureShipped)}}">
|
|
||||||
{{::ticket.futureShipped | date: 'dd/MM/yyyy'}}
|
|
||||||
</span>
|
|
||||||
</td>
|
|
||||||
<td>{{::ticket.futureIpt | dashIfEmpty}}</td>
|
<td>{{::ticket.futureIpt | dashIfEmpty}}</td>
|
||||||
<td>
|
<td>
|
||||||
<span
|
<span
|
||||||
|
@ -181,6 +178,38 @@
|
||||||
question="{{$ctrl.confirmationMessage}}"
|
question="{{$ctrl.confirmationMessage}}"
|
||||||
message="Advance tickets">
|
message="Advance tickets">
|
||||||
</vn-confirm>
|
</vn-confirm>
|
||||||
|
<vn-confirm
|
||||||
|
vn-id="splitTickets"
|
||||||
|
on-accept="$ctrl.splitTickets()"
|
||||||
|
question="{{$ctrl.confirmationMessage}}"
|
||||||
|
message="Advance tickets (without negatives)">
|
||||||
|
</vn-confirm>
|
||||||
|
<!-- Split progress -->
|
||||||
|
<vn-dialog
|
||||||
|
vn-id="splitProgress"
|
||||||
|
message="Progress">
|
||||||
|
<tpl-body class="split-progress">
|
||||||
|
<vn-vertical>
|
||||||
|
<vn-spinner
|
||||||
|
enable="splitProgress.enable">
|
||||||
|
</vn-spinner>
|
||||||
|
<vn-label-value
|
||||||
|
label="Total progress"
|
||||||
|
value="[{{$ctrl.progress.length}}/{{::$ctrl.checked.length}}]">
|
||||||
|
</vn-label-value>
|
||||||
|
<span class="vn-mt-md" ng-if="$ctrl.splitErrors.length" translate>Error list</span>
|
||||||
|
<vn-vertical ng-repeat="error in $ctrl.splitErrors">
|
||||||
|
<vn-label-value
|
||||||
|
label="{{error.id}}"
|
||||||
|
value="{{error.reason}}">
|
||||||
|
</vn-label-value>
|
||||||
|
</vn-vertical>
|
||||||
|
</vn-vertical>
|
||||||
|
</tpl-body>
|
||||||
|
<tpl-buttons>
|
||||||
|
<input type="button" response="cancel" translate-attr="{value: 'Cancel'}" ng-click="splitProgress.cancel = true"/>
|
||||||
|
</tpl-buttons>
|
||||||
|
</vn-dialog>
|
||||||
<vn-ticket-descriptor-popover
|
<vn-ticket-descriptor-popover
|
||||||
vn-id="ticketDescriptor">
|
vn-id="ticketDescriptor">
|
||||||
</vn-ticket-descriptor-popover>
|
</vn-ticket-descriptor-popover>
|
||||||
|
|
|
@ -75,20 +75,6 @@ export default class Controller extends Section {
|
||||||
});
|
});
|
||||||
}
|
}
|
||||||
|
|
||||||
compareDate(date) {
|
|
||||||
let today = Date.vnNew();
|
|
||||||
today.setHours(0, 0, 0, 0);
|
|
||||||
let timeTicket = new Date(date);
|
|
||||||
timeTicket.setHours(0, 0, 0, 0);
|
|
||||||
|
|
||||||
let comparation = today - timeTicket;
|
|
||||||
|
|
||||||
if (comparation == 0)
|
|
||||||
return 'warning';
|
|
||||||
if (comparation < 0)
|
|
||||||
return 'success';
|
|
||||||
}
|
|
||||||
|
|
||||||
get checked() {
|
get checked() {
|
||||||
const tickets = this.$.model.data || [];
|
const tickets = this.$.model.data || [];
|
||||||
const checkedLines = [];
|
const checkedLines = [];
|
||||||
|
@ -134,9 +120,16 @@ export default class Controller extends Section {
|
||||||
'\n' + this.$t(`Destination agency`, {agency: agency});
|
'\n' + this.$t(`Destination agency`, {agency: agency});
|
||||||
}
|
}
|
||||||
|
|
||||||
moveTicketsAdvance() {
|
async moveTicketsAdvance() {
|
||||||
let ticketsToMove = [];
|
let ticketsToMove = [];
|
||||||
this.checked.forEach(ticket => {
|
for (const ticket of this.checked) {
|
||||||
|
if (!ticket.id) {
|
||||||
|
try {
|
||||||
|
const {query, params} = await this.requestComponentUpdate(ticket, false);
|
||||||
|
this.$http.post(query, params);
|
||||||
|
} catch (e) {}
|
||||||
|
continue;
|
||||||
|
}
|
||||||
ticketsToMove.push({
|
ticketsToMove.push({
|
||||||
originId: ticket.futureId,
|
originId: ticket.futureId,
|
||||||
destinationId: ticket.id,
|
destinationId: ticket.id,
|
||||||
|
@ -144,15 +137,105 @@ export default class Controller extends Section {
|
||||||
destinationShipped: ticket.shipped,
|
destinationShipped: ticket.shipped,
|
||||||
workerFk: ticket.workerFk
|
workerFk: ticket.workerFk
|
||||||
});
|
});
|
||||||
});
|
}
|
||||||
|
|
||||||
const params = {tickets: ticketsToMove};
|
const params = {tickets: ticketsToMove};
|
||||||
return this.$http.post('Tickets/merge', params)
|
return this.$http.post('Tickets/merge', params)
|
||||||
.then(() => {
|
.then(() => {
|
||||||
this.$.model.refresh();
|
this.refresh();
|
||||||
this.vnApp.showSuccess(this.$t('Success'));
|
if (ticketsToMove.length)
|
||||||
|
this.vnApp.showSuccess(this.$t('Success', {tickets: ticketsToMove.length}));
|
||||||
});
|
});
|
||||||
}
|
}
|
||||||
|
|
||||||
|
async getLanded(params) {
|
||||||
|
const query = `Agencies/getLanded`;
|
||||||
|
return this.$http.get(query, {params}).then(res => {
|
||||||
|
if (res.data)
|
||||||
|
return res.data;
|
||||||
|
|
||||||
|
return this.vnApp.showError(
|
||||||
|
this.$t(`No delivery zone available for this landing date`)
|
||||||
|
);
|
||||||
|
});
|
||||||
|
}
|
||||||
|
|
||||||
|
async splitTickets() {
|
||||||
|
this.progress = [];
|
||||||
|
this.splitErrors = [];
|
||||||
|
this.$.splitTickets.hide();
|
||||||
|
this.$.splitProgress.enable = true;
|
||||||
|
this.$.splitProgress.cancel = false;
|
||||||
|
this.$.splitProgress.show();
|
||||||
|
|
||||||
|
for (const ticket of this.checked) {
|
||||||
|
if (this.$.splitProgress.cancel) break;
|
||||||
|
try {
|
||||||
|
const {query, params} = await this.requestComponentUpdate(ticket, true);
|
||||||
|
this.$http.post(query, params)
|
||||||
|
.catch(e => {
|
||||||
|
this.splitErrors.push({id: ticket.futureId, reason: e.message});
|
||||||
|
})
|
||||||
|
.finally(() => this.progressAdd(ticket.futureId));
|
||||||
|
} catch (e) {
|
||||||
|
this.splitErrors.push({id: ticket.futureId, reason: e.message});
|
||||||
|
this.progressAdd(ticket.futureId);
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
progressAdd(ticketId) {
|
||||||
|
this.progress.push(ticketId);
|
||||||
|
if (this.progress.length == this.checked.length) {
|
||||||
|
this.$.splitProgress.enable = false;
|
||||||
|
this.refresh();
|
||||||
|
if ((this.progress.length - this.splitErrors.length) > 0) {
|
||||||
|
this.vnApp.showSuccess(this.$t('Success', {
|
||||||
|
tickets: this.progress.length - this.splitErrors.length
|
||||||
|
}));
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
async requestComponentUpdate(ticket, isWithoutNegatives) {
|
||||||
|
const query = `tickets/${ticket.futureId}/componentUpdate`;
|
||||||
|
if (!ticket.landed) {
|
||||||
|
const newLanded = await this.getLanded({
|
||||||
|
shipped: this.$.model.userParams.dateToAdvance,
|
||||||
|
addressFk: ticket.addressFk,
|
||||||
|
agencyModeFk: ticket.agencyModeFk,
|
||||||
|
warehouseFk: ticket.warehouseFk
|
||||||
|
});
|
||||||
|
if (!newLanded)
|
||||||
|
throw new Error(this.$t(`No delivery zone available for this landing date`));
|
||||||
|
|
||||||
|
ticket.landed = newLanded.landed;
|
||||||
|
ticket.zoneFk = newLanded.zoneFk;
|
||||||
|
}
|
||||||
|
const params = {
|
||||||
|
clientFk: ticket.clientFk,
|
||||||
|
nickname: ticket.nickname,
|
||||||
|
agencyModeFk: ticket.agencyModeFk ?? ticket.futureAgencyModeFk,
|
||||||
alexm
commented
dest OR origin dest OR origin
|
|||||||
|
addressFk: ticket.addressFk,
|
||||||
|
zoneFk: ticket.zoneFk ?? ticket.futureZoneFk,
|
||||||
alexm
commented
dest OR origin dest OR origin
|
|||||||
|
warehouseFk: ticket.warehouseFk,
|
||||||
|
companyFk: ticket.companyFk,
|
||||||
|
shipped: this.$.model.userParams.dateToAdvance,
|
||||||
|
landed: ticket.landed,
|
||||||
|
isDeleted: false,
|
||||||
|
isWithoutNegatives,
|
||||||
|
newTicket: ticket.id ?? undefined,
|
||||||
|
keepPrice: true
|
||||||
|
};
|
||||||
|
|
||||||
|
return {query, params};
|
||||||
|
}
|
||||||
|
|
||||||
|
refresh() {
|
||||||
|
this.$.model.refresh();
|
||||||
|
this.$checkAll = null;
|
||||||
|
}
|
||||||
|
|
||||||
exprBuilder(param, value) {
|
exprBuilder(param, value) {
|
||||||
switch (param) {
|
switch (param) {
|
||||||
case 'id':
|
case 'id':
|
||||||
|
|
|
@ -24,31 +24,6 @@ describe('Component vnTicketAdvance', () => {
|
||||||
}];
|
}];
|
||||||
}));
|
}));
|
||||||
|
|
||||||
describe('compareDate()', () => {
|
|
||||||
it('should return warning when the date is the present', () => {
|
|
||||||
let today = Date.vnNew();
|
|
||||||
let result = controller.compareDate(today);
|
|
||||||
|
|
||||||
expect(result).toEqual('warning');
|
|
||||||
});
|
|
||||||
|
|
||||||
it('should return sucess when the date is in the future', () => {
|
|
||||||
let futureDate = Date.vnNew();
|
|
||||||
futureDate = futureDate.setDate(futureDate.getDate() + 10);
|
|
||||||
let result = controller.compareDate(futureDate);
|
|
||||||
|
|
||||||
expect(result).toEqual('success');
|
|
||||||
});
|
|
||||||
|
|
||||||
it('should return undefined when the date is in the past', () => {
|
|
||||||
let pastDate = Date.vnNew();
|
|
||||||
pastDate = pastDate.setDate(pastDate.getDate() - 10);
|
|
||||||
let result = controller.compareDate(pastDate);
|
|
||||||
|
|
||||||
expect(result).toEqual(undefined);
|
|
||||||
});
|
|
||||||
});
|
|
||||||
|
|
||||||
describe('checked()', () => {
|
describe('checked()', () => {
|
||||||
it('should return an array of checked tickets', () => {
|
it('should return an array of checked tickets', () => {
|
||||||
const result = controller.checked;
|
const result = controller.checked;
|
||||||
|
|
|
@ -1,7 +1,9 @@
|
||||||
Advance tickets: Adelantar tickets
|
Advance tickets: Adelantar tickets
|
||||||
|
Advance tickets with negatives: Adelantar tickets con negativos
|
||||||
|
Advance tickets without negatives: Adelantar tickets sin negativos
|
||||||
Search advance tickets by date: Busca tickets para adelantar por fecha
|
Search advance tickets by date: Busca tickets para adelantar por fecha
|
||||||
Advance confirmation: ¿Desea adelantar {{checked}} tickets?
|
Advance confirmation: ¿Desea adelantar {{checked}} tickets?
|
||||||
Success: Tickets movidos correctamente
|
Success: "{{tickets}} Tickets movidos correctamente"
|
||||||
Lines: Líneas
|
Lines: Líneas
|
||||||
Liters: Litros
|
Liters: Litros
|
||||||
Item Packing Type: Encajado
|
Item Packing Type: Encajado
|
||||||
|
@ -9,3 +11,7 @@ Origin agency: "Agencia origen: {{agency}}"
|
||||||
Destination agency: "Agencia destino: {{agency}}"
|
Destination agency: "Agencia destino: {{agency}}"
|
||||||
Less than 50€: Menor a 50€
|
Less than 50€: Menor a 50€
|
||||||
Not Movable: No movibles
|
Not Movable: No movibles
|
||||||
|
Error list: Lista de errores
|
||||||
|
Progress: Progreso
|
||||||
|
Total progress: Progreso total
|
||||||
|
Advance tickets (without negatives): Adelantar tickets (sin negativos)
|
||||||
|
|
|
@ -5,3 +5,12 @@ vn-ticket-advance{
|
||||||
color: #f7931e
|
color: #f7931e
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
.split-progress {
|
||||||
|
width: 40em;
|
||||||
|
}
|
||||||
|
|
||||||
|
@media screen and (max-width: 600px) {
|
||||||
|
.split-progress {
|
||||||
|
width: 100%;
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
|
@ -114,7 +114,8 @@ class Controller extends Component {
|
||||||
isDeleted: this.ticket.isDeleted,
|
isDeleted: this.ticket.isDeleted,
|
||||||
option: parseInt(this.ticket.option),
|
option: parseInt(this.ticket.option),
|
||||||
isWithoutNegatives: this.ticket.withoutNegatives,
|
isWithoutNegatives: this.ticket.withoutNegatives,
|
||||||
withWarningAccept: this.ticket.withWarningAccept
|
withWarningAccept: this.ticket.withWarningAccept,
|
||||||
|
keepPrice: false
|
||||||
};
|
};
|
||||||
|
|
||||||
this.$http.post(query, params)
|
this.$http.post(query, params)
|
||||||
|
|
Loading…
Reference in New Issue
posa un comentari en gitea amb que has modificat aci.