a travaillé sur cet article ... #3576

Closed
opened 7 years ago by miros · 7 comments
miros commented 7 years ago

Vu sur une 3.1 le "a travaillé sur cet article" serait aussi déclenché juste en visualisant la page : en effet un AUTEUR a juste consulté l'article et apparaît ensuite comme "a modifié" cet article .

Vu sur une 3.1 le "a travaillé sur cet article" serait aussi déclenché juste en visualisant la page : en effet un AUTEUR a juste consulté l'article et apparaît ensuite comme "a modifié" cet article .
b_b commented 7 years ago
Owner

Sur signalement de Francky, et je confirme, cela ne semble se produire que si l'article comporte un document en embed avec la raccourci emb oO

Sur signalement de Francky, et je confirme, cela ne semble se produire que si l'article comporte un document en embed avec la raccourci emb oO
Poster

Quand on l'a vu c'était avec jluc (qui est rédacteur sur le blog) et nous on voyait qu'il avait modifié cet article http://blog.spip.net/ecrire/?exec=article&id_article=780 ( qui n'a pas d'embeded)

Quand on l'a vu c'était avec jluc (qui est rédacteur sur le blog) et nous on voyait qu'il avait modifié cet article http://blog.spip.net/ecrire/?exec=article&id_article=780 ( qui n'a pas d'embeded)
b_b commented 7 years ago
Owner

Ok, le problème est dans inc_marquer_doublons_doc_dist() qui appelle bien trois fois l'édition de liens, ce qui déclenche le pipeline revisions_post_edition_lien :

http://zone.spip.org/trac/spip-zone/browser/core/plugins/medias/inc/marquer_doublons_doc.php#L74

Deux solutions :

  • modifier marquer_doublons pour qu'elle ne modifie les liens que quand il y a modification
  • modifier le code du pipeline revisions_post_edition_lien

Avec la première on corrige le problème à la source...

Ok, le problème est dans inc_marquer_doublons_doc_dist() qui appelle bien trois fois l'édition de liens, ce qui déclenche le pipeline revisions_post_edition_lien : http://zone.spip.org/trac/spip-zone/browser/_core_/plugins/medias/inc/marquer_doublons_doc.php#L74 Deux solutions : - modifier marquer_doublons pour qu'elle ne modifie les liens que quand il y a modification - modifier le code du pipeline revisions_post_edition_lien Avec la première on corrige le problème à la source...
Owner

Il y a plusieurs soucis il me semble

A) inc_marquer_doublons_doc_dist() oubliait de prendre en compte certains champs parfois. Devrait être corrigé par http://zone.spip.org/trac/spip-zone/changeset/92751
B) inc_marquer_doublons_doc_dist() modifiait à chaque appel les liens de documents même s'ils n'avaient pas changés, ce qui exécutait souvent à tord les pipelines post_editions_liens sur lequel se base le plugin révisions, qui créait donc une révision à tord. Devrait être corrigé par http://zone.spip.org/trac/spip-zone/changeset/92753 et http://zone.spip.org/trac/spip-zone/changeset/92754

C) la fonction est systématiquement appelée à chaque visite d'une page de vue d'un objet dans l'espace -public- privé, par inc_documenter_objet_dist() depuis le pipeline medias_afficher_complement_objet(). Et ça ça semble tout de même inutile, vu que le marquage de doublons est déjà effectué à chaque modification du contenu d'un objet.

Il y a plusieurs soucis il me semble A) `inc_marquer_doublons_doc_dist()` oubliait de prendre en compte certains champs parfois. Devrait être corrigé par http://zone.spip.org/trac/spip-zone/changeset/92751 B) `inc_marquer_doublons_doc_dist()` modifiait à chaque appel les liens de documents même s'ils n'avaient pas changés, ce qui exécutait souvent à tord les pipelines post_editions_liens sur lequel se base le plugin révisions, qui créait donc une révision à tord. Devrait être corrigé par http://zone.spip.org/trac/spip-zone/changeset/92753 et http://zone.spip.org/trac/spip-zone/changeset/92754 C) la fonction est systématiquement appelée à chaque visite d'une page de vue d'un objet dans l'espace -public- privé, par inc_documenter_objet_dist() depuis le pipeline medias_afficher_complement_objet(). Et ça ça semble tout de même inutile, vu que le marquage de doublons est déjà effectué à chaque modification du contenu d'un objet.
b_b commented 7 years ago
Owner

Statut changé à En cours

**Statut changé à En cours**
Owner
There is no content yet.
b_b commented 7 years ago
Owner

On ferme, merci marcimat :)
Statut changé à Fermé

On ferme, merci marcimat :) **Statut changé à Fermé**
Sign in to join this conversation.
No Milestone
No project
No Assignees
3 Participants
Notifications
Due Date

No due date set.

Dependencies

This issue currently doesn't have any dependencies.

Loading…
There is no content yet.