verdnatura/salix
verdnatura
/
salix
3
4
Fork 0
Code Issues Pull Requests 59 Releases Wiki Activity

refs #6291 validateTin #1836

Merged
carlossa merged 12 commits from 6291-comprobarDNI into dev 2024-01-09 14:03:21 +00:00
Conversation 0 Commits 12 Files Changed 2 +4 -9
2 changed files with 4 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit c77715d865 - Show all commits

2
db/dump/fixtures.sql
View File

@ -3014,7 +3014,7 @@ UPDATE `vn`.`client`
SET fi='65004204V'
WHERE id=1;
UPDATE `vn`.`client`
UPDATE `vn`.`worker`
SET fi='59328808D'
WHERE id=1106;

9
modules/worker/back/models/worker.js
View File

@ -34,17 +34,12 @@ module.exports = Self => {
fields: ['code'],
where: {id: this.countryFk}
};
const country = await Self.app.models.Country.findOne(filter);
const code = country ? country.code.toLowerCase() : null;
const client = (await Self.findById(this.id, {
include: {relation: 'client'}}))?.client();
if (client) {
const countryCode = client.fi?.toLowerCase().substring(0, 2);
const countryCode = this.fi?.toLowerCase().substring(0, 2);
carlossa marked this conversation as resolved Outdated
jsegarra commented 2023-12-22 07:32:57 +00:00
Outdated
Review
Copy Link

Creo que se puede simplificar
const code = country?.code?.toLowerCase() ?? null;
Y después de esto, quizás emitiría un error

Creo que se puede simplificar const code = country?.code?.toLowerCase() ?? null; Y después de esto, quizás emitiría un error
if (!client.fi || !validateTin(client.fi, code) || countryCode == code)
if (!this.fi || !validateTin(this.fi, code) || countryCode == code)
err();
}
done();
jsegarra marked this conversation as resolved Outdated
jsegarra commented 2023-12-22 07:18:57 +00:00
Outdated
Review
Copy Link

El nombre de la variable es correcto?
Porque lo llamas countryCode pero no usas referencias a country

El nombre de la variable es correcto? Porque lo llamas countryCode pero no usas referencias a country
carlossa commented 2023-12-26 12:54:07 +00:00
Outdated
Review
Copy Link

Es como se hacia también en client, por continuar la misma dinámica

Es como se hacia también en client, por continuar la misma dinámica
jsegarra commented 2024-01-02 06:22:47 +00:00
Outdated
Review
Copy Link

Entiendo, pero cuando creas un worker(pej), crea también el cliente y es en ese modelo donde se guardan los datos que usas en el worker, por eso digo, que siendo otro model a l mejor necesita otro nombre.

Entiendo, pero cuando creas un worker(pej), crea también el cliente y es en ese modelo donde se guardan los datos que usas en el worker, por eso digo, que siendo otro model a l mejor necesita otro nombre.
}
};