Corrige les pages de validation pour montrer les actions de validation

[Arnaud-D]

Fix #6573.

Je corrige ce bug en créant une page dédiée au visionnage de la version en validation (jusqu'ici, c'était un affichage de version générique). Avoir une page dédiée permet alors d'avoir un affichage customisé, avec notamment les actions de validation dessus.

J'en ai profité au passage pour que cette page soit celle à laquelle on accède par défaut quand on vient de la liste des publications en cours de validation, et qu'on retourne sur cette version quand on réserve le contenu en tant que validateur.

Reste à faire

Les tests unitaire pour cette nouvelle route.

Contrôle qualité

Vérifier que cette nouvelle page fonctionne bien et tout le workflow de validation

[coveralls]

coverage: 88.664% (-0.05%) from 88.71%
when pulling f72bf4c on Arnaud-D:fix-6573
into 7275ad0 on zestedesavoir:dev.

[philippemilink]

Je viens de tester, ça a l'air bien, mais sur la vue ContainerValidationView, il n'y a pas la section Validation (en validation quand tu regardes un chapitre d'un big tuto). D'un point fonctionnel, je ne sais pas si c'est normal ou pas, mais si je regarde le code, j'ai l'impression que le code des deux vues ContainerValidationView et ContentValidationView est quasiment identique, donc ça me paraît bizarre.

Tu comptes rajouter le test dans cette PR, ou bien je peux fusionner quand le point ci-dessus sera éclairci ?

Et puis il faudrait un rebase sur la branche dev.

[Arnaud-D]

J'aimerais ajouter quelques tests avant de fusionner.

Pour ce qui est d'afficher les actions de validation quand on est dans un conteneur, c'est faisable. Si on décide de le faire, je préfèrerais que ça soit dans une autre PR. Il y a des problèmes similaires pour l'encart de bêta et l'encart "version publique" et il y a quelques autres éléments ailleurs sur la page dans le même cas.

La grosse PR qui a chamboulé ces vues a permis de retirer la redondance entre les différents templates qui géraient la version publique, le brouillon, la beta, etc. Par contre, la dupluication entre le rendu d'un Content et celui d'un Container est encore-là. J'aimerais que la correction des bugs d'affichage Content/Container se fasse d'une manière qui facilite la maintenabilité (et la justesse du code). Je pense que j'ai pas attaqué ça dans la grosse refacto précédente parce que c'était déjà beaucoup ce que j'avais fait.

T'en penses quoi ?

[philippemilink]

J'aimerais ajouter quelques tests avant de fusionner.

Ok, j'attends pour fusionner.

Pour ce qui est d'afficher les actions de validation quand on est dans un conteneur, c'est faisable. Si on décide de le faire, je préfèrerais que ça soit dans une autre PR. Il y a des problèmes similaires pour l'encart de bêta et l'encart "version publique" et il y a quelques autres éléments ailleurs sur la page dans le même cas.

La grosse PR qui a chamboulé ces vues a permis de retirer la redondance entre les différents templates qui géraient la version publique, le brouillon, la beta, etc. Par contre, la dupluication entre le rendu d'un Content et celui d'un Container est encore-là. J'aimerais que la correction des bugs d'affichage Content/Container se fasse d'une manière qui facilite la maintenabilité (et la justesse du code). Je pense que j'ai pas attaqué ça dans la grosse refacto précédente parce que c'était déjà beaucoup ce que j'avais fait.

T'en penses quoi ?

Si je comprends bien, tu dis qu'il faudrait factoriser l'affichage Content et Container et que ça facilitera la résolution du problème des menus pas affichés partout où ce serait logique qu'ils le soient ? Je suis d'accord avec ça (de toute façon dans le libre, c'est celui qui fait qui a raison...), mais ce serait de faire au moins un ticket pour retenir l'idée et savoir qu'on a "bug" connu.

[Arnaud-D]

J'ai ajouté les tests.

J'ai créé le ticket : #6584.