zestedesavoir / zds-site / 10257

89%

push

travis-ci

[Nettoyage] reprise de plusieurs PR front (#4993)

* Renomme les filtres top_categories et top_categories_content

Supprime `templates/tutorialv2/list_page_elements/side_bar.html` qui
était inutilisé.

* Ajout d'un minlength sur le formulaire de validation de contenu.

minlength=3
Dans zds/tutorialv2/forms.py

* Simplified handling of errors

In form zds.tutorialv2.forms.AcceptValidationForm
Using django.forms.Forms.add_error

* Trie les notifications de la plus récente à la plus ancienne

L’ordre a changé suite à un refactoring furtif dans 088f2cfa.

J’aurais bien rajouté un test là-dessus mais je sais pas trop faire.

* Affichage des erreurs propre !

Closes #4817

* Écriture des tests

- Tests du tag messages
- Optimisation des imports

* Déplacement du header pour alléger le base.html

* Transforme les templatetags des notifications en un context processor

Je ne pense pas que ce soit une bonne pratique de faire des queries vers
la DB dans des templatetags.

Ce commit limite le nombre maximal de notifications affichées dans la
liste déroulante en haut à droite du site. Il n’y en a pas plus de 10
désormais. Toutefois, le nombre de notifications affiché dans le badge
rouge correspond bien au nombre total des notifications.

Le code est quasi le même pour les messages privés, les alertes de
modération et les autres notifications.

J’ai essayé de soigner les queries, il y a d’abord un `SELECT COUNT(*) …`
puis un `SELECT * … LIMIT 10` et c’est tout.

* add unit test

* fix tests

* lint

* codacy, then

1706 of 3966 branches covered (43.02%)

60 of 60 new or added lines in 7 files covered. (100.0%)

11654 of 16905 relevant lines covered (68.94%)

1.71 hits per line