verdnatura/salix
verdnatura
/
salix
3
5
Fork 2
Code Issues Pull Requests 73 Releases Wiki Activity

refactor: refs #7950 Created cmr model #3180

Merged
guillermo merged 6 commits from 7950-cmrModelUnify into dev 2024-11-15 06:07:17 +00:00
Conversation 0 Commits 6 Files Changed 14 +2 -3
2 changed files with 2 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit 9c0af7f7c6 - Show all commits

2
loopback/common/methods/vn-model/printService.js
View File

@ -2,7 +2,7 @@ const {Report, Email} = require('vn-print');
module.exports = Self => {
Self.printReport = async function(ctx, id, reportName) {
const args = Object.assign({}, ctx.args);
const args = Object.assign({id}, ctx.args);
guillermo marked this conversation as resolved
jgallego commented 2024-11-14 07:55:46 +00:00
Review
Copy Link

bajar el id a params.

bajar el id a params.
guillermo commented 2024-11-14 08:50:06 +00:00
Review
Copy Link

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.

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.
const params = {lang: ctx.req.getLocale()};
delete args.ctx;

3
modules/ticket/back/methods/ticket/saveCmr.js
View File

@ -53,8 +53,7 @@ module.exports = Self => {
if (hasDmsCmr.length)
throw new UserError('This ticket already has a cmr saved');
Object.assign(ctx.args, {id: ticket.$cmrFk});
const response = await models.Cmr.print(ctx, myOptions);
const response = await models.Cmr.print(ctx, ticket.$cmrFk, myOptions);
jgallego marked this conversation as resolved Outdated
jgallego commented 2024-11-12 08:13:40 +00:00
Outdated
Review
Copy Link

el $ no el solem gastar, si no te un objectiu clar jo meu pensaria be antes de gastarlo, @alexm com ho veus ?

el $ no el solem gastar, si no te un objectiu clar jo meu pensaria be antes de gastarlo, @alexm com ho veus ?
guillermo commented 2024-11-12 08:38:45 +00:00
Outdated
Review
Copy Link

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

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
alexm commented 2024-11-12 09:01:57 +00:00
Outdated
Review
Copy Link

Mai havia vist ticket.$cmrFk si funciona i es natiu de loopback supose que abant

Mai havia vist ticket.$cmrFk si funciona i es natiu de loopback supose que abant
const pdfStream = Readable.from(Buffer.from(response[0]));
guillermo marked this conversation as resolved Outdated
jgallego commented 2024-11-12 12:18:58 +00:00
Outdated
Review
Copy Link

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.

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.
guillermo commented 2024-11-12 12:24:57 +00:00
Outdated
Review
Copy Link

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...

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...
alexm commented 2024-11-12 12:30:46 +00:00
Outdated
Review
Copy Link

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)

En este cas seria `await models.Cmr.print(ctx, ticket.$cmrFk); ` Son els parametres que te el back https://gitea.verdnatura.es/verdnatura/salix/src/commit/90e7ab1ec1d18366c7b1a394ddfaa03e90c01d52/modules/route/back/methods/cmr/print.js#L36
guillermo commented 2024-11-12 12:38:57 +00:00
Outdated
Review
Copy Link

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);

1) ticket saveCmr() should save cmr
  - Error: Required properties not found [id]
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); ``` 1) ticket saveCmr() should save cmr - Error: Required properties not found [id] ```
alexm commented 2024-11-12 13:17:43 +00:00
Outdated
Review
Copy Link

Se pot posar en 6ab431f8ef/loopback/common/methods/vn-model/printService.js (L5)

const args = Object.assign({id}, ctx.args);

Se pot posar en https://gitea.verdnatura.es/verdnatura/salix/src/commit/6ab431f8efec6e80495451332064c6eeb8b6684f/loopback/common/methods/vn-model/printService.js#L5 `const args = Object.assign({id}, ctx.args);`
jgallego commented 2024-11-14 06:45:17 +00:00
Outdated
Review
Copy Link

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.

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.
👍 1
const data = {
workerFk: ctx.req.accessToken.userId,