svergeylen / collector

Collector : site de gestion des BD, films, DVD, jeux de société, ...
2 stars 0 forks source link

Requete pour afficher "derniers ajourts" par.. catégorie #78

Closed svergeylen closed 6 years ago

svergeylen commented 6 years ago

Auparavant, une requete simple permettait d'afficher les derniers items ajoutés (ou modifiés, càd pour lesque un attachement avait été ajouté) par catégorie... Le retrait des catégorie et la profondeur infinie de l'arbre des folders (=ex-tags) ne permet plus de faire cela facilement. il faudrait faire un requete générique dans model > Folder pour afficher les derniers items de ce folder (et potentiellement sou folders, donc !) Attention aux performances, ca cette requete sera lancée très souvent (folder > show)

dvergeylen commented 6 years ago

Normalement (je n'ai pas encore de quoi tester), tout Item doit avoir l'ensemble des folders dont il fait partie dans l'association ownertag.

Ceci permet de trier les Items ayant "BD" comme folder, même s'ils sont bien plus profond dans la hiérarchie. 👍

svergeylen commented 6 years ago

Ha oui, ce serait parfait, en effet... (à condition que toutes les bd possède bien le folder "bd"... ce qui va d'office etre oublié par Luc à la création de bd... mais c'est une autre question j'avoue)

Du coup, en une seule requete ce serait bien... Item.includes(:ownerfolders).where(folder_id: "id du fodler bd").... et du coup le fodler bd devrait devient une fixture (à cocher dans folder > edit )... ou encore mieux, on fait un code défensif qui tente de trouver le folder "bd" et s'il le trouve, fait le rendu des derniers ajouts ...

On pourrait alors imaginer de créer un option supplémentaire de folder qui serait : "afficher les derniers ajouts pour ce folder" et ainsi adapter la vue en fonction de ce paramètre (afficher ou non le bloc)... Ca, ce serait très plaisant et générique non ? :-)

dvergeylen commented 6 years ago

Ce serait très plaisant et générique en effet!

Je ne retrouve plus l'endroit où j'avais écrit que chaque Item devait avoir l'ensemble des folders dont il fait partie, ça aurait été utile de relire la spec que j'avais écrite... 🤔

Pour cette vue d'ajout de Folder à un nouvel, à mon avis il faut:

svergeylen commented 6 years ago

C'est fait :-)

(pour les root tags pour le moment, on verra s'il faut généraliser à tous les tags, mais cela n'a pas forcément de sens de présenter des news pour des sous-sous-sous-tags....)