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

#7793 - sortByWeight #763

Open
jsegarra wants to merge 10 commits from 7793_sortByWeight into dev
pull from: 7793_sortByWeight
merge into: verdnatura:dev
Conversation 1 Commits 10 Files Changed 1 +17
1 changed files with 17 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit 8f963ab78a - Show all commits

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

@ -139,6 +139,23 @@ function findKeyInOptions() {
}
function setOptions(data) {
data = data.sort((a, b) => {
alexm commented 2024-10-16 07:35:29 +00:00
Outdated
Review
Copy Link

Para esto no seria mejor ya pedir los datos ordenadors por optionLabel?

Para esto no seria mejor ya pedir los datos ordenadors por optionLabel?
jsegarra commented 2024-10-16 12:14:16 +00:00
Outdated
Review
Copy Link

De esta manera nos aseguramos que se ordenan por optionLabel, porque que pasa con esos VnSelect que no tienen definidos los sortBy? Tendríamos que ir uno a uno indicándoles la ordenación.
De esta manera nos aseguramos que, definido o no, se ordenan por label.
También nos aseguramos de aplicar el criterio de ordenar según la posición del texto buscado dentro del valor

De esta manera nos aseguramos que se ordenan por optionLabel, porque que pasa con esos VnSelect que no tienen definidos los sortBy? Tendríamos que ir uno a uno indicándoles la ordenación. De esta manera nos aseguramos que, definido o no, se ordenan por label. También nos aseguramos de aplicar el criterio de ordenar según la posición del texto buscado dentro del valor
const search = lastVal.value?.toString()?.toLowerCase();
const aValue = String(a[$props.optionLabel]).toLowerCase();
const bValue = String(b[$props.optionLabel]).toLowerCase();
const aIndex = aValue.indexOf(search);
const bIndex = bValue.indexOf(search);
const aPriority = aIndex === 0 ? 0 : aIndex > 0 ? 1 : 2;
const bPriority = bIndex === 0 ? 0 : bIndex > 0 ? 1 : 2;
if (aPriority !== bPriority) {
return aPriority - bPriority;
}
return aValue.localeCompare(bValue);
});
myOptions.value = JSON.parse(JSON.stringify(data));
myOptionsOriginal.value = JSON.parse(JSON.stringify(data));
}