verdnatura/salix
verdnatura
/
salix
3
5
Fork 0
Code Issues Pull Requests 65 Releases Wiki Activity

DEP: #7004 - HOTFIX Create new Supplier #2148

Closed
jsegarra wants to merge 9 commits from 7004_hotFix_newSupplier into master
pull from: 7004_hotFix_newSupplier
merge into: verdnatura:master
Conversation 1 Commits 9 Files Changed 8 +2 -2
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit e6b2424f3e - Show all commits

4
modules/supplier/back/methods/supplier/updateFiscalData.js
View File

@ -87,11 +87,11 @@ module.exports = Self => {
}
}
if (city === null) {
if (city === null || (city === undefined && !supplier.city)) {
alexm commented 2024-03-08 07:13:32 +00:00
Outdated
Review
Copy Link

Y pq no poner los argumentos como requeridos?

Y pq no poner los argumentos como requeridos?
jsegarra commented 2024-03-08 07:20:19 +00:00
Outdated
Review
Copy Link

No puedes hacer eso porque la segunda vez que guardas, si esos campos no los has modificado te dará error porque en el body de la request no aparece

No puedes hacer eso porque la segunda vez que guardas, si esos campos no los has modificado te dará error porque en el body de la request no aparece
alexm commented 2024-03-08 07:24:21 +00:00
Outdated
Review
Copy Link

Entonces con if pasaria lo mismo

Entonces con if pasaria lo mismo
jsegarra commented 2024-03-08 07:49:07 +00:00
Outdated
Review
Copy Link

He modificado la condición porque lo que no queremos que sea null y antes aceptaba undefined y null.
Ahora es mas restrictivo.

He modificado la condición porque lo que no queremos que sea null y antes aceptaba undefined y null. Ahora es mas restrictivo.
throw new UserError('The city cannot be empty'
);
}
if (nif === null) {
if (nif === null || (nif === undefined && !supplier.nif)) {
throw new UserError('The nif cannot be empty'
);
}