Rend générique en fonction du type la vue "WarnTypo" #6669
Conversation
Arnaud-D
force-pushed
the
retire-des-distinctions-entre-types-3
branch
from
5b05c04 to
d6e9bb7
Compare
* supprime la distinction de type de publication * préfère le terme "publication" à "contenu" * renomme les vues pour adopter la convention de Django * déplace le formulaire à côté de la vue
Arnaud-D
force-pushed
the
retire-des-distinctions-entre-types-3
branch
from
d6e9bb7 to
7526887
Compare
Rapport de QAQA OK 👍 Je n'ai pas pris le temps de faire la review, donc à review avant merge. |
|
|
||
| pm_title = _("J'ai trouvé une faute dans {} « {} ».").format(_type, form.content.title) | ||
|
|
||
| pm_title = _("J'ai trouvé une faute dans « {} ».").format(form.content.title) |
There was a problem hiding this comment.
pm_title est défini dans https://github.com/zestedesavoir/zds-site/pull/6669/files#diff-4b06fa578d536b8bff461bfdd2196ac3afa8179198ec0fe96319d859d9b947d3R103-R106, donc l'une des deux définitions est redondante.
| messages.error(self.request, _("Impossible d'envoyer la proposition de correction : vous êtes auteur.")) | ||
|
|
||
| else: # send correction | ||
| else: # Send correction |
There was a problem hiding this comment.
| else: # Send correction | |
| else: # Send PM |
| self._errors["text"] = self.error_class([_("Vous devez indiquer la faute commise.")]) | ||
| if "text" in cleaned_data: | ||
| del cleaned_data["text"] | ||
|
|
| version = content.sha_beta | ||
| if public: | ||
| version = content.sha_public |
There was a problem hiding this comment.
On peut sans doute factoriser cette portion du code avec la définition plus haut de self.previous_page_url (il y a le même if public).
| num_of_authors = content.authors.count() | ||
| for index, user in enumerate(content.authors.all()): |
There was a problem hiding this comment.
Il faudrait récupérer uniquement les membres contactables (cf https://github.com/zestedesavoir/zds-site/pull/6669/files#diff-4b06fa578d536b8bff461bfdd2196ac3afa8179198ec0fe96319d859d9b947d3R185). Ça me fait donc penser qu'on pourrait ajouter une méthode qui récupère les auteurs contactables dans la classe PublishableContent ?
Dans tous les cas, je pense qu'il est possible de factoriser ce code avec ce qu'il y a dans la vue.
| JsFiddleActivationForm, | ||
| PublicationForm, | ||
| PickOpinionForm, | ||
| UnpickOpinionForm, | ||
| PromoteOpinionToArticleForm, | ||
| ) | ||
| from zds.tutorialv2.views.misc import WarnTypoForm |
There was a problem hiding this comment.
En regardant dans ce fichier pourquoi on avait besoin de cet import, je me rends compte que cette ligne :
est dupliquée dix lignes plus loin sans raison apparente :On peut sans doute profiter de cette PR pour corriger ça.
Dans le cadre de la nouvelle organisation des contenus
Contrôle qualité
Tester le formulaire de signalement de faute sur les différents types de contenu, regarder les MP correspondants.