New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Historise les éditions de messages #4277

Merged
merged 17 commits into from May 8, 2017

Conversation

Projects
None yet
5 participants
@gcodeur
Member

gcodeur commented Apr 10, 2017

Q R
Type de modification nouvelle fonctionnalité
Ticket(s) (issue(s)) concerné(s) #4186

Cette pull request implémente l'historisation des éditions de messages. Chaque message (cela inclut les commentaires de contenus et les posts du forum) dispose d'un historique visible par son auteur et par le staff. Chaque entrée contient la date d'édition, l'éditeur et le contenu avant édition.

Il est possible de restaurer une version, ce qui entraîne la création d'une nouvelle entrée (comme une édition normale). Il est aussi possible de supprimer le contenu d'une version pour le staff (la ligne reste présente, mais il est indiqué qui a supprimé le contenu, et quand).

Afin d'éviter la duplication de code, une entrée d'historique contient une clé étrangère vers Comment (dont héritent Post et ContentReaction). Les pages d'historiques sont placée dans le module pages étant données qu'elles ne sont ni liées à forum, ni à tutorialv2.

À faire

  • Il reste les tests à faire, bien qu'il y en ait déjà un sur l'insertion.

QA

  • Reconstruire le front
  • Éditer un message, vérifier que la mention Édité est cliquable pour le staff et l'auteur.
  • Vérifier que l'historique ne s'affiche que pour le staff et l'auteur.
  • Vérifier que les informations de l'historique sont correctes.
  • Vérifier que l'affichage de la version avant édition marche correctement.
  • Vérifier que la restauration fonctionne (nouvelle entrée).
  • Vérifier la suppression d'une version, ainsi que le fait qu'elle ne puisse être réalisée que par le staff.
  • Ça fonctionne !
  • Code relu et approuvé !

@gcodeur gcodeur referenced this pull request Apr 10, 2017

Closed

Dernier appel pour la v23 #4278

3 of 3 tasks complete
@coveralls

This comment has been minimized.

Show comment
Hide comment
@coveralls

coveralls Apr 10, 2017

Coverage Status

Coverage decreased (-0.3%) to 88.369% when pulling d20c9f0 on GCodeur:fix-4186 into 659f4b7 on zestedesavoir:dev.

coveralls commented Apr 10, 2017

Coverage Status

Coverage decreased (-0.3%) to 88.369% when pulling d20c9f0 on GCodeur:fix-4186 into 659f4b7 on zestedesavoir:dev.

Show outdated Hide outdated templates/pages/comment_edits_history.html
Show outdated Hide outdated templates/pages/comment_edits_history.html
Show outdated Hide outdated templates/pages/edit_detail.html
Show outdated Hide outdated templates/pages/edit_detail.html
Show outdated Hide outdated templates/pages/edit_detail.html
def get_object(self):
return get_object_or_404(Comment, pk=self.kwargs['comment_pk'])
@method_decorator(login_required)

This comment has been minimized.

@gustavi

gustavi Apr 10, 2017

Member

Il y a le mixin LoginRequiredMixin qui est plus simple/propre à utiliser (https://docs.djangoproject.com/fr/1.10/topics/auth/default/#the-loginrequired-mixin)

@gustavi

gustavi Apr 10, 2017

Member

Il y a le mixin LoginRequiredMixin qui est plus simple/propre à utiliser (https://docs.djangoproject.com/fr/1.10/topics/auth/default/#the-loginrequired-mixin)

This comment has been minimized.

@gcodeur

gcodeur Apr 10, 2017

Member

(Vu avec @gustavi sur IRC)

Ce mixin a été introduit en 1.9 et nous sommes en 1.8, on ne peut donc pas l'utiliser pour le moment. :/

@gcodeur

gcodeur Apr 10, 2017

Member

(Vu avec @gustavi sur IRC)

Ce mixin a été introduit en 1.9 et nous sommes en 1.8, on ne peut donc pas l'utiliser pour le moment. :/

This comment has been minimized.

@gustavi

gustavi Apr 12, 2017

Member

exact !

@gustavi

gustavi Apr 12, 2017

Member

exact !

This comment has been minimized.

@artragis

artragis Apr 12, 2017

Contributor

mais il me semble que zds a son propreLoginRequiredMixin.

@artragis

artragis Apr 12, 2017

Contributor

mais il me semble que zds a son propreLoginRequiredMixin.

verbose_name = "Édition d'un message"
verbose_name_plural = 'Éditions de messages'
comment = models.ForeignKey(Comment, on_delete=models.CASCADE, verbose_name='Message',

This comment has been minimized.

@gustavi

gustavi Apr 10, 2017

Member

i18n (pour tous les champs)

@gustavi

gustavi Apr 10, 2017

Member

i18n (pour tous les champs)

This comment has been minimized.

@gcodeur

gcodeur Apr 10, 2017

Member

Juste pour ça, je crois pas qu'on puisse faire d'i18n dans les champs de modèles, on ne l'a pas fait jusqu'à présent en tout cas.

(Par contre dans les templates, je m'y attelle !)

@gcodeur

gcodeur Apr 10, 2017

Member

Juste pour ça, je crois pas qu'on puisse faire d'i18n dans les champs de modèles, on ne l'a pas fait jusqu'à présent en tout cas.

(Par contre dans les templates, je m'y attelle !)

This comment has been minimized.

@gustavi

gustavi Apr 12, 2017

Member

exact

@gustavi

gustavi Apr 12, 2017

Member

exact

@coveralls

This comment has been minimized.

Show comment
Hide comment
@coveralls

coveralls Apr 18, 2017

Coverage Status

Coverage decreased (-0.3%) to 88.311% when pulling ef1c239 on GCodeur:fix-4186 into ffd9d0f on zestedesavoir:dev.

coveralls commented Apr 18, 2017

Coverage Status

Coverage decreased (-0.3%) to 88.311% when pulling ef1c239 on GCodeur:fix-4186 into ffd9d0f on zestedesavoir:dev.

@coveralls

This comment has been minimized.

Show comment
Hide comment
@coveralls

coveralls Apr 22, 2017

Coverage Status

Coverage decreased (-0.3%) to 88.311% when pulling bdda809 on GCodeur:fix-4186 into ffd9d0f on zestedesavoir:dev.

coveralls commented Apr 22, 2017

Coverage Status

Coverage decreased (-0.3%) to 88.311% when pulling bdda809 on GCodeur:fix-4186 into ffd9d0f on zestedesavoir:dev.

@coveralls

This comment has been minimized.

Show comment
Hide comment
@coveralls

coveralls Apr 22, 2017

Coverage Status

Coverage decreased (-0.09%) to 88.485% when pulling 8ab3404 on GCodeur:fix-4186 into ffd9d0f on zestedesavoir:dev.

coveralls commented Apr 22, 2017

Coverage Status

Coverage decreased (-0.09%) to 88.485% when pulling 8ab3404 on GCodeur:fix-4186 into ffd9d0f on zestedesavoir:dev.

@coveralls

This comment has been minimized.

Show comment
Hide comment
@coveralls

coveralls Apr 22, 2017

Coverage Status

Coverage decreased (-0.08%) to 88.496% when pulling 67d1582 on GCodeur:fix-4186 into ffd9d0f on zestedesavoir:dev.

coveralls commented Apr 22, 2017

Coverage Status

Coverage decreased (-0.08%) to 88.496% when pulling 67d1582 on GCodeur:fix-4186 into ffd9d0f on zestedesavoir:dev.

@coveralls

This comment has been minimized.

Show comment
Hide comment
@coveralls

coveralls Apr 22, 2017

Coverage Status

Coverage increased (+0.1%) to 88.687% when pulling 85d5ea2 on GCodeur:fix-4186 into ffd9d0f on zestedesavoir:dev.

coveralls commented Apr 22, 2017

Coverage Status

Coverage increased (+0.1%) to 88.687% when pulling 85d5ea2 on GCodeur:fix-4186 into ffd9d0f on zestedesavoir:dev.

@coveralls

This comment has been minimized.

Show comment
Hide comment
@coveralls

coveralls Apr 22, 2017

Coverage Status

Coverage increased (+0.1%) to 88.687% when pulling e19d85c on GCodeur:fix-4186 into ffd9d0f on zestedesavoir:dev.

coveralls commented Apr 22, 2017

Coverage Status

Coverage increased (+0.1%) to 88.687% when pulling e19d85c on GCodeur:fix-4186 into ffd9d0f on zestedesavoir:dev.

@coveralls

This comment has been minimized.

Show comment
Hide comment
@coveralls

coveralls Apr 22, 2017

Coverage Status

Coverage decreased (-33.4%) to 55.158% when pulling 4be4048 on GCodeur:fix-4186 into ffd9d0f on zestedesavoir:dev.

coveralls commented Apr 22, 2017

Coverage Status

Coverage decreased (-33.4%) to 55.158% when pulling 4be4048 on GCodeur:fix-4186 into ffd9d0f on zestedesavoir:dev.

@coveralls

This comment has been minimized.

Show comment
Hide comment
@coveralls

coveralls Apr 22, 2017

Coverage Status

Coverage increased (+0.1%) to 88.684% when pulling 98a80f4 on GCodeur:fix-4186 into ffd9d0f on zestedesavoir:dev.

coveralls commented Apr 22, 2017

Coverage Status

Coverage increased (+0.1%) to 88.684% when pulling 98a80f4 on GCodeur:fix-4186 into ffd9d0f on zestedesavoir:dev.

@gcodeur gcodeur changed the title from [WIP] Historise les éditions de messages to Historise les éditions de messages Apr 22, 2017

@gcodeur

This comment has been minimized.

Show comment
Hide comment
@gcodeur

gcodeur Apr 22, 2017

Member

Le code est terminé. Quelques infos :

  • L'attribut original_text_html n'était utile que pour la restauration et l'affichage des détails d'une édition. Pour éviter la sauvegarde de trop de données inutiles, je l'ai supprimé et n'ai gardé que original_text et le rendu HTML est calculé quand c'est nécessaire (comme on fait pour les biographies par exemple).
  • Je n'ai pas réussi à utiliser le LoginRequiredMixin de ZdS car il été ignoré à cause de ma surcharge de la méthode dispatch. Du coup, je suis resté sur le décorateur login_required.

Manque plus que la QA. :)

Member

gcodeur commented Apr 22, 2017

Le code est terminé. Quelques infos :

  • L'attribut original_text_html n'était utile que pour la restauration et l'affichage des détails d'une édition. Pour éviter la sauvegarde de trop de données inutiles, je l'ai supprimé et n'ai gardé que original_text et le rendu HTML est calculé quand c'est nécessaire (comme on fait pour les biographies par exemple).
  • Je n'ai pas réussi à utiliser le LoginRequiredMixin de ZdS car il été ignoré à cause de ma surcharge de la méthode dispatch. Du coup, je suis resté sur le décorateur login_required.

Manque plus que la QA. :)

@coveralls

This comment has been minimized.

Show comment
Hide comment
@coveralls

coveralls Apr 27, 2017

Coverage Status

Coverage increased (+0.1%) to 88.684% when pulling 8acd46c on GCodeur:fix-4186 into 05b4160 on zestedesavoir:dev.

coveralls commented Apr 27, 2017

Coverage Status

Coverage increased (+0.1%) to 88.684% when pulling 8acd46c on GCodeur:fix-4186 into 05b4160 on zestedesavoir:dev.

@coveralls

This comment has been minimized.

Show comment
Hide comment
@coveralls

coveralls May 6, 2017

Coverage Status

Coverage increased (+0.1%) to 88.747% when pulling 058a6a4 on GCodeur:fix-4186 into 9b12929 on zestedesavoir:dev.

coveralls commented May 6, 2017

Coverage Status

Coverage increased (+0.1%) to 88.747% when pulling 058a6a4 on GCodeur:fix-4186 into 9b12929 on zestedesavoir:dev.

@AmarOk1412

This comment has been minimized.

Show comment
Hide comment
@AmarOk1412

AmarOk1412 May 7, 2017

Member

QA: Ok !

Member

AmarOk1412 commented May 7, 2017

QA: Ok !

@coveralls

This comment has been minimized.

Show comment
Hide comment
@coveralls

coveralls May 7, 2017

Coverage Status

Coverage increased (+0.1%) to 88.747% when pulling 6293f72 on GCodeur:fix-4186 into a06b161 on zestedesavoir:dev.

coveralls commented May 7, 2017

Coverage Status

Coverage increased (+0.1%) to 88.747% when pulling 6293f72 on GCodeur:fix-4186 into a06b161 on zestedesavoir:dev.

@gcodeur

This comment has been minimized.

Show comment
Hide comment
@gcodeur

gcodeur May 7, 2017

Member

@AmarOk1412 : Merci ! Tu as fait le code review également ?

Member

gcodeur commented May 7, 2017

@AmarOk1412 : Merci ! Tu as fait le code review également ?

@AmarOk1412

This comment has been minimized.

Show comment
Hide comment
@AmarOk1412

AmarOk1412 May 7, 2017

Member

@gcodeur ouai j'ai pas vu de trucs qui me choquent :)

Member

AmarOk1412 commented May 7, 2017

@gcodeur ouai j'ai pas vu de trucs qui me choquent :)

@gcodeur

This comment has been minimized.

Show comment
Hide comment
@gcodeur

gcodeur May 8, 2017

Member

Parfait, allons-y :)

Member

gcodeur commented May 8, 2017

Parfait, allons-y :)

@gcodeur gcodeur removed the QA svp label May 8, 2017

@gcodeur gcodeur added this to the Version de développement milestone May 8, 2017

@gcodeur gcodeur merged commit 10da96f into zestedesavoir:dev May 8, 2017

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details

@gcodeur gcodeur deleted the gcodeur:fix-4186 branch Jul 17, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment