refactor: refs #7950 Created cmr model #3180
No reviewers
jgallego
Labels
No Milestone
No Assignees
3 Participants
Notifications
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: verdnatura/salix#3180
Loading…
Reference in New Issue
No description provided.
Delete Branch "7950-cmrModelUnify"
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?
@ -55,3 +55,2 @@ctx.args.id = ticket.cmrFk;const response = await models.Route.cmr(ctx, myOptions);ctx.args.id = ticket.$cmrFk;el $ no el solem gastar, si no te un objectiu clar jo meu pensaria be antes de gastarlo, @alexm com ho veus ?
El problema es que ara com es un model, te que accedir de una altra forma, ticket.cmrFk ja no val.
Es pot gastar ticket.cmrFk().id, pero ticket.$cmrFk es mes curt
Mai havia vist ticket.$cmrFk si funciona i es natiu de loopback supose que abant
@ -56,2 +56,2 @@ctx.args.id = ticket.cmrFk;const response = await models.Route.cmr(ctx, myOptions);ctx.args.id = ticket.$cmrFk;const response = await models.Cmr.print(ctx, myOptions);ho he comentat en Juan i m'ha dit lo mateix, que si es natiu de loopback avant.
Pero que l'objecte ctx no hi ha que alterarlo.
Entonces com li pase el id? Es que es un param que va per URL, crec que en el seu moment no vaig trovar altra forma...
En este cas seria
await models.Cmr.print(ctx, ticket.$cmrFk);Son els parametres que te el back
90e7ab1ec1/modules/route/back/methods/cmr/print.js (L36)Ya lo comprobé en su momento y no pude, lo he vuelto a comprobar por si acaso y lo mismo:
const response = await models.Cmr.print(ctx, ticket.$cmrFk, myOptions);
Se pot posar en
6ab431f8ef/loopback/common/methods/vn-model/printService.js (L5)const args = Object.assign({id}, ctx.args);Cuidao @guillermo el assign tal com l'has posat actualiza l'objecte ctx, que es el que no volem.
Amb la proposata d'Alex es crea uno nou, gasta ixa opcio.
hi ha un comentari nou
@ -3,3 +3,3 @@module.exports = Self => {Self.printReport = async function(ctx, id, reportName) {const args = Object.assign({}, ctx.args);const args = Object.assign({id}, ctx.args);bajar el id a params.
He revisado el código, y habiamos visto printEmail, en printReport no se hace.
Por lo que el cambio es correcto.
Además, he probado a poner
params.id = id;y da error, ya que no es lo mismo.