assemblee-virtuelle / archipelago

Fostering interconnections between communities by creating synergies between their platforms
Apache License 2.0
14 stars 6 forks source link

ReferenceFilter: pouvoir n'afficher que les ressources avec au moins un élément #92

Open srosset81 opened 1 year ago

srosset81 commented 1 year ago

C'est chouette de pouvoir maintenant voir dans les filtres le nombre d'éléments correspondants (cf https://github.com/assemblee-virtuelle/semapps/pull/1022 faite par @VincentFarcy)

https://archipel.assemblee-virtuelle.org/Organization

Par contre à mon avis ce serait intéressant de pouvoir cacher les ressources qui n'ont aucune correspondance, car de toute façon si on clique dessus, on aura 0 résultat (donc peu d'intérêt pour l'utilisateur).

Et peut-être même que cette option soit par défaut ?

srosset81 commented 1 year ago

En fait je réalise en écrivant la doc que inverseSource était censé faire ça, mais ça ne semble pas marcher. :/

Ligne concernée:

ids.filter(id => !inverseSource || data[id][inverseSource])
VincentFarcy commented 1 year ago

j'ai repéré 2 bugs sur les compteurs dans les filtres, et je suis confronté à un problème que j'ai déjà rencontré et sur lequel j'aimerais avoir ton avis.

le 1er est fixé (Pb de test d'une valeur unique au lieu d'un tableau de valeurs)

le 2ème est lié au fait que inverseSource peut exister dans les données pour des ressources autres que celle consultée.

Ex sur les cdlt : je consulte la liste des organisations il y a un filtre sur sector or pair:sectorOf (relation inverse) peut contenir des orga mais aussi des events pour masquer l'item s'il n'est pas utilisé par une orga, il faudrait tester en plus de la présence d'une relation inverse, qu'au moins une relation inverse est en lien avec une ressource de type orga.

J'en arrive au point intéressant ! Pour faire ce test j'ai à ma disposition :

useResourceContext() qui renvoit par exemple Organization et le type de ressource dans les data. ex : "type": "pair:Organization"

=> Comment comparer ces 2 valeurs proprement ?

VincentFarcy commented 1 year ago

La nuit porte conseil ! useDataModel

srosset81 commented 1 year ago

Oui !! :)

VincentFarcy commented 1 year ago

https://github.com/assemblee-virtuelle/semapps/pull/1055