Ordering '/api/projects' with 'list_id' or 'list_name' doesn't work
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 2/5
- Temps estimé
- 1-3 heures
- Accessibilité débutants
- 45/100
Piste de recherche
Commencez dans patchwork/api/project.py, au niveau de ProjectList, et inspectez les ordering_fields utilisés par /api/projects. Reproduisez le tri par list_id et list_name, puis vérifiez que le endpoint ne lève plus le FieldError signalé pour les requêtes ascendantes et descendantes.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
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?
- Langage dominant
- Python
- Étoiles
- 317
- Forks
- 91
- Métriques de merge des PR
- Aucune PR mergée en 30 j
Guide de contribution
Ouvrir le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de getpatchwork/patchwork
-
Difficulté 2/5 1-3 heures Accessibilité débutants 68/100
getpatchwork/patchwork#668 ·
-
bug web-ui
Difficulté 2/5 1-3 heures Accessibilité débutants 74/100
getpatchwork/patchwork#546 · 2 commentaires ·
-
Difficulté 3/5 1-2 jours Accessibilité débutants 52/100
getpatchwork/patchwork#664 ·
-
Difficulté 4/5 3-5 jours Accessibilité débutants 48/100
getpatchwork/patchwork#663 ·
-
Difficulté 3/5 1-2 jours Accessibilité débutants 45/100
getpatchwork/patchwork#645 ·
Toutes les issues de getpatchwork/patchwork
Issues similaires
-
enhancement
Difficulté 2/5 1-3 heures Accessibilité débutants 70/100
canonical/paas-charm#368 · 1 commentaire ·
-
Difficulté 2/5 1-3 heures Accessibilité débutants 75/100
-
tech debt
Difficulté 2/5 1-3 heures Accessibilité débutants 75/100
-
addition to tracking list Ouverte
Difficulté 1/5 Moins d'une heure Accessibilité débutants 90/100
StevenBlack/hosts#3256 ·
-
Difficulté 1/5 Moins d'une heure Accessibilité débutants 90/100
qualcomm/qai-appbuilder#275 ·