Fix #53: Sort refs by start when displaying by lemma#54
Conversation
| <li> | ||
| <a href="{% url 'lemma-detail' lemma.slug %}">{{ lemma.display }}</a>: | ||
| {{ page_ref_list|join:', ' }} | ||
| {{ page_ref_list|dictsort:"start"|join:', ' }} |
There was a problem hiding this comment.
Op het PageReference model staat een ordering als volgt:
# models.py, l. 222
ordering = ["whole", "lemma", "work", "start", "end", "suffix"]M.a.w.: er wordt in de database eerst op work gesorteerd, pas daarna op start. Dat is denk ik de oorzaak van #53, zou dat kunnen?
Ik zou wel willen kijken of we op dat niveau een oplossing kunnen bedenken, want het sorteren gaat op dat niveau wel een stuk efficiënter dan in de html templates. In het eerste geval sla je ze op een gesorteerde manier op, en heb je bij het ophalen altijd direct de sorting die je wil hebben, met deze oplossing worden elke keer bij het renderen alle dictionaries opnieuw gesorteerd (ook degene die al gesorteerd zijn).
There was a problem hiding this comment.
Ik denk dat je dit inderdaad zou moeten doen met database index.
Ik ga eens speuren.
2 participants
Fix #53