zestedesavoir / zds-site / 10257 / 3
89%
dev: 89%

DEFAULT BRANCH: dev
Ran
Files 218
Run time 23s
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst
coverage: 57.374% (+0.07%) from 57.304%
ZDS_TEST_JOB="front zds.mp zds.gallery zds.pages zds.featured zds.notification zds.searchv2"

push

travis-ci

web-flow 
[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

1377 of 4026 branches covered (34.2%)

9699 of 16905 relevant lines covered (57.37%)

0.57 hits per line

Source Files on job 10257.3 (ZDS_TEST_JOB="front zds.mp zds.gallery zds.pages zds.featured zds.notification zds.searchv2")