Skip to content

Fix #53: Sort refs by start when displaying by lemma #54

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Draft
johannesdewit wants to merge 1 commit into
base: main
Choose a base branch
from
Draft

Fix #53: Sort refs by start when displaying by lemma #54

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion heidegger_index/templates/components/ref_list/_group_by_lemma.html
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@
{% for lemma, page_ref_list in lemma_groups %}
<li>
<a href="{% url 'lemma-detail' lemma.slug %}">{{ lemma.display }}</a>:
{{ page_ref_list|join:', ' }}
{{ page_ref_list|dictsort:"start"|join:', ' }}
Copy link
Copy Markdown
Owner

@andredelft andredelft Oct 14, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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).

Copy link
Copy Markdown
Collaborator Author

@johannesdewit johannesdewit Mar 9, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ik denk dat je dit inderdaad zou moeten doen met database index.

Ik ga eens speuren.

</li>
{% endfor %}
</ul>