verdnatura/salix-front
verdnatura
/
salix-front
2
3
Fork 1
Code Issues Pull Requests 87 Releases Wiki Activity

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
Conversation 0 Commits 4 Files Changed 3 +4 -3
1 changed files with 4 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit c418899fb3 - Show all commits

7
src/components/common/VnSelect.vue
View File

@ -161,11 +161,10 @@ async function fetchFilter(val) {
if ($props.filterOptions.length) {
jorgep commented 2024-08-26 10:50:01 +00:00
Review
Copy Link

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.
👍 1
defaultWhere = $props.filterOptions.reduce((obj, prop) => {
jorgep marked this conversation as resolved
jsegarra commented 2024-08-26 11:23:37 +00:00
Review
Copy Link

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?
if (!obj.or) obj.or = [];
obj.or.push({ [prop]: $props.useLike ? { like: `%${val}%` } : val });
obj.or.push({ [prop]: getVal(val) });
jorgep marked this conversation as resolved Outdated
jsegarra commented 2024-08-26 11:48:52 +00:00
Outdated
Review
Copy Link

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?
return obj;
}, {});
} else
defaultWhere = $props.useLike ? { [key]: { like: `%${val}%` } } : { [key]: val };
} else defaultWhere = { [key]: getVal(val) };
const where = { ...(val ? defaultWhere : {}), ...$props.where };
const fetchOptions = { where, include, limit };
if (fields) fetchOptions.fields = fields;
@ -204,6 +203,8 @@ async function filterHandler(val, update) {
function nullishToTrue(value) {
return value ?? true;
}
const getVal = (val) => ($props.useLike ? { like: `%${val}%` } : val);
</script>
<template>