5914-transferInvoiceOut #1761
Labels
No Milestone
No Assignees
3 Participants
Notifications
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: verdnatura/salix#1761
Loading…
Reference in New Issue
No description provided.
Delete Branch "5914-transferInvoiceOut"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
@ -0,0 +3,4 @@
('CplusRectificationType', '*', 'READ', 'ALLOW', 'ROLE', 'employee'),
('CplusInvoiceType477', '*', 'READ', 'ALLOW', 'ROLE', 'employee'),
('InvoiceCorrectionType', '*', 'READ', 'ALLOW', 'ROLE', 'employee'),
('InvoiceOut', 'transferInvoice', 'WRITE', 'ALLOW', 'ROLE', 'employee');
solo debe de poder transferir una factura el rol administrative.
Por otra parte, recuerda que la carpeta del changes esta desfasada, lo tienes que mover a la 2340
@ -322,0 +319,4 @@
"Ticket without Route": "Ticket sin ruta",
"Select a different customer": "Seleccione un cliente distinto",
"Fill all the fields": "Rellene todos los campos"
}
"NIF/CIF invalido" podría llevar una tilde: "NIF/CIF inválido".
"Cuidad no puede estar en blanco" tiene un pequeño error tipográfico, debe ser "Ciudad no puede estar en blanco".
"Is invalid" y "is invalid" no se han traducido al español.
"This postal code is not valid" también está en inglés.
"Role already assigned", "Invalid role name", etc. están en inglés y no se han traducido.
"No puedes cambiar los datos basicos de una orden con artículos" podría llevar tilde en "básicos".
"El nombre de usuario solo debe contener letras minúsculas o, a partir del segundo carácter, números o subguiones, no esta permitido el uso de la letra ñ" podría llevar tilde en "está"
"La fecha de inicio debe ser menor que la fecha de fín" sin tilde en "fin".
"Debes especificar una fecha de inicio o de fín" sin tilde en "fin".
[chatgpt]
@ -0,0 +54,4 @@
const {clientFk} = await models.InvoiceOut.findById(args.id);
if (clientFk == args.newClientFk)
throw new UserError(`Select a different customer`);
mejor client
@ -43,6 +44,7 @@ module.exports = Self => {
Self.makePdf = async function(id, options) {
const fields = ['id', 'hasPdf', 'ref'];
const invoiceOut = await Self.findById(id, {fields}, options);
console.log('Recoge invoiceOut?');
olvido
@ -6,2 +22,4 @@
<vn-menu vn-id="menu">
<vn-list>
<vn-item
vn-acl="administrative"
fijate que aquí sí lo has puesto, pero en los acl no. En próximas ocasiones que vayan "juntos"
@ -25,0 +25,4 @@
}
@media screen and (min-width: 1000px) {
.transferInvoice {
min-width: 900px;
@alexm valorad si es estrictamente necesario y si lo es no poner un 1000px si no algo relativo
@ -0,0 +123,4 @@
]
};
const ticket = await models.Ticket.findById(ticketId, filter, myOptions);
const refundTicket = await models.Ticket.create({
conviene que uses /salix/modules/ticket/back/methods/ticket/new.js
ya que tiene y tendrá las verificaciones necesarias
@ -42,4 +43,2 @@
let refundTicket = null;
try {
const refundAgencyMode = await models.AgencyMode.findOne({
esta logica donde esta ahora?
Está dentro del método clone
@ -39,7 +39,6 @@ module.exports = Self => {
modules/ticket/back/methods/ticket/invoiceTicketsWithPdf.js este porque sale?
Se me olvidó borrarlo
@ -136,6 +136,11 @@
"type": "belongsTo",
"model": "Zone",
"foreignKey": "zoneFk"
},
esta repetido arriba con ticketSales
@ -0,0 +56,4 @@
if (clientFk == args.newClientFk)
throw new UserError(`Select a different customer`);
if (!args.newClientFk || !args.cplusRectificationId || !args.cplusInvoiceType477Id || !args.invoiceCorrectionTypeId)
Per a ficar esta restriccio pots ficar
required: true
en les validacions (dalt)@ -0,0 +28,4 @@
const refundTickets = [];
const mappedTickets = new Map();
const now = Date.vnNew();
const [firstTicketId] = ticketsIds;
y te ahorres fer if y else
@ -0,0 +44,4 @@
ticketId,
withWarehouse,
refundTickets,
mappedTickets,
El map de mappedTickets realment el pots omplir fer el bucle
WIP 5914-transferInvoiceOutto 5914-transferInvoiceOut@ -0,0 +7,4 @@
INSERT INTO `vn`.`invoiceCorrectionType` (description)
VALUES
('Error en el cálculo del IVA')
este registro ya existe en la tabla, es normal?
A mí en local no me aparece el registro
Entonces deberias poner una fixture (en ingles)
@ -24,1 +23,4 @@
}
@media screen and (min-width: 1000px) {
el 1000 no lo podemos quitar?
@ -0,0 +7,4 @@
INSERT INTO `vn`.`invoiceCorrectionType` (description)
VALUES
('Error en el cálculo del IVA')
En access solo tienen esa opcion?
Hay más, pero es para probar. ¿Quieres que las ponga todas?
@ -0,0 +1,123 @@
module.exports = Self => {
Si quieres usar refundTickets en varios sitios no seria mejor ponerlo fuera de las funciones? (No estoy seguro de que funcione)
Mirado juntos.
@ -0,0 +36,4 @@
ctx,
ticketsIds[ticketId],
withWarehouse,
refundTickets,
Aqui
@ -0,0 +62,4 @@
await models.SaleComponent.create(components, myOptions);
}
if (servicesIds && servicesIds.length > 0) {
if (servicesIds && servicesIds.length)
@ -0,0 +114,4 @@
refundTickets.push(refundTicket);
}
async function getTicketRefundId(group, ticketId, refundTickets, mappedTickets) {
Esta funcio te la pots ahorrar no?
Fent mappedTickets.get(ticketId) ja te torna lo que vols pq quan has fet el set esta ja agrupat o no
@ -2977,0 +2977,4 @@
INSERT INTO `vn`.`invoiceCorrectionType` (`id`, `description`)
VALUES
(1, 'Error en el cálculo del IVA'),
En ingles
cambiar a la carpeta changes actual
New commits pushed, approval review dismissed automatically according to repository settings
New commits pushed, approval review dismissed automatically according to repository settings
Carlos S. me ha dicho que ha encontrado un fallo, lo revisaré el lunes con él
New commits pushed, approval review dismissed automatically according to repository settings
Lo hemos comprobado y funciona correctamente
New commits pushed, approval review dismissed automatically according to repository settings
@ -0,0 +88,4 @@
throw e;
}
async function createTicketRefund(
Pero no quedarem en que clone.js no debia tindre res relacionat en refund
@ -126,1 +51,4 @@
true,
myOptions
);
Deuria ser per aci on fa que el ticket siga refund
New commits pushed, approval review dismissed automatically according to repository settings
@ -0,0 +109,4 @@
const newTicket = await models.Ticket.new(ctx, myOptions);
if (negative) {
En un futur se podria refactoritzar i extraure esta part a refund o un hipotetic ticket/clone