Duplicate usernames in combo-box when filtering patches by delegate

Offen

#383 erstellt am 24.10.2020 0 Kommentare 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: 52/100

Issue-Typ: Bug
Klarheit: Größtenteils klar
Aktivitätsstatus: Veraltet
Tech-Stack: python
Bereich: backend

Rechercherichtung

Beginne in patchwork/filters.py bei DelegateFilter ungefähr in Zeile 458 und untersuche _get_delegate_qs für die projektbezogene Delegate-Abfrage. Überprüfe, wie das Formular das aktuelle Projekt erhält. Erledigt ist dies, wenn die Combo-Box jeden Maintainer des ausgewählten Projekts genau einmal auflistet und Maintainer anderer Projekte ausschließt.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

bug

The combo-box with the list of delegates to filter by has duplicate usernames. It lists a maintainer more than once if he/she maintains multiple projects. It also lists maintainers of other projects who don't maintain the one I'm filtering patches in.

This happens because DelegateFilter's form doesn't filter by project:
https://github.com/getpatchwork/patchwork/blame/bdb049c7939b7cdea1eddf029a0bc47cb338da67/patchwork/filters.py#L458

        delegates = User.objects.filter(
            profile__maintainer_projects__isnull=False)

Eliminating duplicate rows by using distinct() works, but we still have the issue of different project maintainers being listed in the same list:

        delegates = User.objects.filter(
            profile__maintainer_projects__isnull=False).distinct()

Maybe we should use _get_delegate_qs instead?

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