Ordering '/api/projects' with 'list_id' or 'list_name' doesn't work

Offen

#318 erstellt am 15.10.2019 1 Kommentar 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Bewertung

Schwierigkeit: 2/5
Geschätzter Aufwand: 1-3 Stunden
Anfängerfreundlichkeit: 45/100

Issue-Typ: Bug
Klarheit: Klar beschrieben
Aktivitätsstatus: Veraltet
Tech-Stack: python
Bereich: api, backend

Rechercherichtung

Beginne in patchwork/api/project.py bei ProjectList und prüfe die von /api/projects verwendeten ordering_fields. Implementiere die Sortierung nach list_id und list_name erneut und verifiziere anschließend, dass der Endpunkt bei aufsteigenden und absteigenden Anfragen den gemeldeten FieldError nicht mehr auslöst.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

api bug

If you set the ordering filter to list_id ascending/descending or list_name ascending/descending in /api/projects, the following exception will be thrown:

FieldError at /api/projects/

Cannot resolve keyword 'link_name' into field. Choices are: bundle, delegationrule, id, linkname, listemail, listid, maintainer_project, name, patch, scm_url, send_notifications, series, subject_match, submission, use_tags, web_url, webscm_url

Suggested fix:

diff --git a/patchwork/api/project.py b/patchwork/api/project.py
index 3609f73..c249f54 100644
--- a/patchwork/api/project.py
+++ b/patchwork/api/project.py
@@ -83,7 +83,7 @@ class ProjectList(ProjectMixin, ListAPIView):

     search_fields = ('link_name', 'list_id', 'list_email', 'web_url',
                      'scm_url', 'webscm_url')
-    ordering_fields = ('id', 'name', 'link_name', 'list_id')
+    ordering_fields = ('id', 'name', 'linkname', 'listid')
     ordering = 'id'

Does this fix look good enough? Should I send a patch?

Vorherrschende Sprache: Python
Sterne: 317
Forks: 91
PR-Merge-Kennzahlen: Keine gemergten PRs in 30 T.

Beitragsleitfaden

Beitragsleitfaden öffnen

Erste Schritte

Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
Forken Sie das Repository und arbeiten Sie in einem Branch.
Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.

Mehr aus getpatchwork/patchwork

Broken "documentation for this release" links in the release notes Offen

Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 68/100

getpatchwork/patchwork#668 · Erstellt am 03.8.2026
[v3.1.2] Submitter search doesn't autocomplete Offen

bug web-ui

Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 74/100

getpatchwork/patchwork#546 · 2 Kommentare · Erstellt am 30.7.2023
Patchwork mangling binary file deletion patch Offen

Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 52/100

getpatchwork/patchwork#664 · Erstellt am 28.7.2026
slight privacy inconsistency: /api/patches (HTML) lists people to anonymous users but /api/people doesn't Offen

Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 48/100

getpatchwork/patchwork#663 · Erstellt am 23.7.2026
Ignore whitespace before and after git tags Offen

Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 45/100

getpatchwork/patchwork#645 · Erstellt am 08.1.2026

Alle Issues in getpatchwork/patchwork