fix: refs #7323 auto fill bic #642
No reviewers
Labels
No Milestone
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: verdnatura/salix-front#642
Loading…
Reference in New Issue
No description provided.
Delete Branch "7323-hotfix-workerCreate"
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?
@ -157,2 +160,2 @@
? { [key]: { like: `%${val}%` } }
: { [key]: val };
let defaultWhere = {};
if ($props.filterOptions.length) {
Permite filtrar por varias opciones. Antes, si era un digito, solo filtraba por id.
@ -225,7 +231,7 @@ function generateCodeUser(worker) {
<VnSelect
:label="t('worker.create.boss')"
v-model="data.bossFk"
:options="workersOptions"
Cargaba demasiados datos.
@ -158,1 +160,3 @@
: { [key]: val };
let defaultWhere = {};
if ($props.filterOptions.length) {
defaultWhere = $props.filterOptions.reduce((obj, prop) => {
Porque la logica de $props.useLike no se puede usar en el reduce?
@ -59,0 +59,4 @@
async function autofillBic(worker) {
if (!worker || !worker.iban) return;
let bankEntityId = parseInt(worker.iban.substr(4, 4));
Con esto obtenemos las posiciones 5,6, 7 y 8, no?
Sí, lógica de Salix.
@ -59,0 +64,4 @@
const { data } = await axios.get(`BankEntities`, { params: { filter } });
const hasData = data && data[0];
if (hasData) worker.bankEntityFk = data[0].id;
hasData solo puede tener 2 valores no?
Porque lo que es if...else
De todas maneras si vas a hacer una asignacion porque no quitas el if...else
Está copiado de salix. De todas maneras, te lo he puesto como en el comentario de abajo.
@ -126,2 +125,4 @@
if (!worker.companyFk) worker.companyFk = user.companyFk;
}
async function autofillBic(worker) {
Espera, la misma lógica 2 veces?
Mmm...podemos externalizar a un js dentro del módulo?
Ya hay una tarea para eso. Porque no es cosa de WorkerCreate, está en toda la web.
@ -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 } });
Lo dejaria asi
worker.bankEntityFk = data?.[0]?.id ?? undefined;
@ -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 });
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?