fix: refs #7323 auto fill bic #642

Merged
jorgep merged 4 commits from 7323-hotfix-workerCreate into master 2024-08-26 11:57:11 +00:00
Member
No description provided.
jorgep added 1 commit 2024-08-26 10:49:14 +00:00
gitea/salix-front/pipeline/pr-master This commit looks good Details
6f9cd7fe15
fix: refs #7323 auto fill bic
jorgep reviewed 2024-08-26 10:50:01 +00:00
@ -157,2 +160,2 @@
? { [key]: { like: `%${val}%` } }
: { [key]: val };
let defaultWhere = {};
if ($props.filterOptions.length) {
Author
Member

Permite filtrar por varias opciones. Antes, si era un digito, solo filtraba por id.

Permite filtrar por varias opciones. Antes, si era un digito, solo filtraba por id.
jorgep reviewed 2024-08-26 10:51:17 +00:00
@ -225,7 +231,7 @@ function generateCodeUser(worker) {
<VnSelect
:label="t('worker.create.boss')"
v-model="data.bossFk"
:options="workersOptions"
Author
Member

Cargaba demasiados datos.

Cargaba demasiados datos.
jorgep requested review from jsegarra 2024-08-26 11:01:12 +00:00
jorgep added the
CR / Tests passed
label 2024-08-26 11:05:58 +00:00
jsegarra reviewed 2024-08-26 11:23:37 +00:00
@ -158,1 +160,3 @@
: { [key]: val };
let defaultWhere = {};
if ($props.filterOptions.length) {
defaultWhere = $props.filterOptions.reduce((obj, prop) => {
Member

Porque la logica de $props.useLike no se puede usar en el reduce?

Porque la logica de $props.useLike no se puede usar en el reduce?
jorgep marked this conversation as resolved
jsegarra requested changes 2024-08-26 11:32:09 +00:00
Dismissed
@ -59,0 +59,4 @@
async function autofillBic(worker) {
if (!worker || !worker.iban) return;
let bankEntityId = parseInt(worker.iban.substr(4, 4));
Member

Con esto obtenemos las posiciones 5,6, 7 y 8, no?

Con esto obtenemos las posiciones 5,6, 7 y 8, no?
Author
Member

Sí, lógica de Salix.

Sí, lógica de Salix.
jorgep marked this conversation as resolved
@ -59,0 +64,4 @@
const { data } = await axios.get(`BankEntities`, { params: { filter } });
const hasData = data && data[0];
if (hasData) worker.bankEntityFk = data[0].id;
Member

hasData solo puede tener 2 valores no?
Porque lo que es if...else

hasData solo puede tener 2 valores no? Porque lo que es if...else
Member

De todas maneras si vas a hacer una asignacion porque no quitas el if...else

De todas maneras si vas a hacer una asignacion porque no quitas el if...else
Author
Member

Está copiado de salix. De todas maneras, te lo he puesto como en el comentario de abajo.

Está copiado de salix. De todas maneras, te lo he puesto como en el comentario de abajo.
jorgep marked this conversation as resolved
@ -126,2 +125,4 @@
if (!worker.companyFk) worker.companyFk = user.companyFk;
}
async function autofillBic(worker) {
Member

Espera, la misma lógica 2 veces?
Mmm...podemos externalizar a un js dentro del módulo?

Espera, la misma lógica 2 veces? Mmm...podemos externalizar a un js dentro del módulo?
Author
Member

Ya hay una tarea para eso. Porque no es cosa de WorkerCreate, está en toda la web.

Ya hay una tarea para eso. Porque no es cosa de WorkerCreate, está en toda la web.
jorgep marked this conversation as resolved
@ -128,0 +131,4 @@
let bankEntityId = parseInt(worker.iban.substr(4, 4));
let filter = { where: { id: bankEntityId } };
const { data } = await axios.get(`BankEntities`, { params: { filter } });
Member

Lo dejaria asi
worker.bankEntityFk = data?.[0]?.id ?? undefined;

Lo dejaria asi `worker.bankEntityFk = data?.[0]?.id ?? undefined;`
jorgep marked this conversation as resolved
jorgep requested review from jsegarra 2024-08-26 11:44:40 +00:00
jorgep added 2 commits 2024-08-26 11:44:41 +00:00
jsegarra requested changes 2024-08-26 11:48:53 +00:00
Dismissed
@ -159,0 +161,4 @@
if ($props.filterOptions.length) {
defaultWhere = $props.filterOptions.reduce((obj, prop) => {
if (!obj.or) obj.or = [];
obj.or.push({ [prop]: $props.useLike ? { like: `%${val}%` } : val });
Member

Okey, pero ahora tenemos la misma logica 2 veces. La unica diferencia es como gestionas la variable key.
Porque no sacas el operador ternario a una funcion y luego lo asignas como valor a esa Key

Mas limpio, o que te parece?

Okey, pero ahora tenemos la misma logica 2 veces. La unica diferencia es como gestionas la variable key. Porque no sacas el operador ternario a una funcion y luego lo asignas como valor a esa Key Mas limpio, o que te parece?
jorgep marked this conversation as resolved
jorgep added 1 commit 2024-08-26 11:54:53 +00:00
gitea/salix-front/pipeline/pr-master This commit looks good Details
c418899fb3
chore: refs #7323 improve vnselect
jorgep requested review from jsegarra 2024-08-26 11:54:55 +00:00
jsegarra approved these changes 2024-08-26 11:56:28 +00:00
jorgep merged commit 8277b30033 into master 2024-08-26 11:57:11 +00:00
jorgep deleted branch 7323-hotfix-workerCreate 2024-08-26 11:57:11 +00:00
Sign in to join this conversation.
No reviewers
No Milestone
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: verdnatura/salix-front#642
No description provided.