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

#7136 - Enable paginate event in VnSelectFilter #255

Closed
jsegarra wants to merge 86 commits from 7136_vnselectFilter_paginate into dev
pull from: 7136_vnselectFilter_paginate
merge into: verdnatura:dev
Conversation 8 Commits 86 Files Changed 6 +5 -7
1 changed files with 5 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit e775d922e8 - Show all commits

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

@ -103,14 +103,13 @@ const arrayData = useURL.value
: ref(null);
onMounted(async () => {
setOptions(options.value);
if (!$props.options) fetchFilter($props.modelValue);
jsegarra marked this conversation as resolved Outdated
alexm commented 2024-06-07 05:15:50 +00:00
Outdated
Review
Copy Link

De primeras diria que esta linea no hace falta, dado que la primera ja lo hace

De primeras diria que esta linea no hace falta, dado que la primera ja lo hace
jsegarra commented 2024-06-07 10:15:12 +00:00
Outdated
Review
Copy Link

Pero quiero evitar que haga fetch
Un caso de uso sería: se lo declaro en el componente o una variable

Pero quiero evitar que haga fetch Un caso de uso sería: se lo declaro en el componente o una variable
alexm commented 2024-06-25 09:56:58 +00:00
Outdated
Review
Copy Link

updateRouter: false sino podrá cosas en la url

`updateRouter: false` sino podrá cosas en la url
if (useURL.value) {
alexm commented 2024-06-07 05:19:32 +00:00
Outdated
Review
Copy Link

Esta parte solo debe hacerla si hay url. Y por como esta ahora ya habria hecho la peticion al entrar en if (useURL.value)

Esta parte solo debe hacerla si hay url. Y por como esta ahora ya habria hecho la peticion al entrar en `if (useURL.value)`
jsegarra commented 2024-06-07 10:21:02 +00:00
Outdated
Review
Copy Link

Lo cambio para dejarlo asi
if (!$props.options) fetchFilter($props.modelValue);

Lo cambio para dejarlo asi if (!$props.options) fetchFilter($props.modelValue);
arrayData.store.userFilter = $props.where;
arrayData.store.filter.where = $props.where;
const { data } = await arrayData.fetch({ append: true });
setOptions(data);
return;
}
if (!$props.options) fetchFilter($props.modelValue);
});
watch(modelValue, (newValue) => {
@ -125,7 +124,7 @@ function filter(val, options) {
alexm commented 2024-07-08 05:29:12 +00:00
Outdated
Review
Copy Link

El useLike no me lo quites pls

El useLike no me lo quites pls
jsegarra commented 2024-07-12 10:50:03 +00:00
Outdated
Review
Copy Link

hecho

hecho
if (!search) return options;
const optionsFiltered = options.filter((row) => {
return options.filter((row) => {
if ($props.filterOptions.length) {
return $props.filterOptions.some((prop) => {
const propValue = String(row[prop]).toLowerCase();
@ -138,16 +137,15 @@ function filter(val, options) {
return id == search || optionLabel.includes(search);
});
return optionsFiltered;
}
async function fetchFilter(val) {
if (!useURL.value || !dataRef.value) return;
const { fields, sortBy, limit } = $props;
let key = null;
if (new RegExp(/\d/g).test(val)) key = optionFilter.value ?? optionValue.value;
let key = new RegExp(/\d/g).test(val)
? optionFilter.value ?? optionValue.value
jsegarra marked this conversation as resolved Outdated
alexm commented 2024-06-25 09:59:04 +00:00
Outdated
Review
Copy Link

Por como esta useArrayData, fetch no acepta los datos por paramtros. Solo { append = false, updateRouter = true }

Por como esta useArrayData, fetch no acepta los datos por paramtros. Solo `{ append = false, updateRouter = true }`
😕 1
: optionValue.value;
const where = { ...{ [key]: { like: `%${val}%` } }, ...$props.where };
return dataRef.value.fetch({ fields, where, order: sortBy, limit });
alexm commented 2024-06-07 05:22:18 +00:00
Outdated
Review
Copy Link

Si de por si solo entra en fetchFilter si hay url, no hace falta separar la construccion del where en una funcion apart, que solo se usa en un sitio

Si de por si solo entra en fetchFilter si hay url, no hace falta separar la construccion del where en una funcion apart, que solo se usa en un sitio
jsegarra commented 2024-06-07 10:24:36 +00:00
Outdated
Review
Copy Link

Vale, lo debí separar para otra situación que acabé borrando y esto se quedó así
Lo limipio

Vale, lo debí separar para otra situación que acabé borrando y esto se quedó así Lo limipio