Un pipeline appelé une fois l'action de storage effectué. #4

Closed
maieul wants to merge 1 commits from pipeline_post_store into master
maieul commented 2 years ago
Collaborator

Utile dans les cas où celle ci n'appelle pas objet_modifier().

Par exemple pour le plugin formidable_participation.

Utile dans les cas où celle ci n'appelle pas objet_modifier(). Par exemple pour le plugin formidable_participation.
Poster
Collaborator

je m epermet de relancer sur ces 2 PR :) ?

je m epermet de relancer sur ces 2 PR :) ?
Poster
Collaborator

@cerdic je me permet de relancer sur ces 2 PR.

@cerdic je me permet de relancer sur ces 2 PR.
Owner

ça semble quand même bancal de reposer sur un pipeline spécifique de crayons non ?
Le code dérogatoire qui passe pas par objet_modifier() est du vieux code legacy qui normalement n'est utilisé que pour la compat avec les anciens codes.

Ça me parait ni sain ni souhaitable de venir y mettre des nouveaux plugins qui créent des chemins de mise à jour dérogatoire...

Il vaudrait mieux pas avoir des fonctions wrapper update pour ton objet, et tu y gères l'appel aux pipelines standards ?

Tu peux voir ici https://git.spip.net/spip-contrib-extensions/crayons/src/branch/master/action/crayons_store.php#L258 que crayons teste l'existence de fonctions xxx_revision, amha c'est dans une fonction de ce type qu'il faut que tu gères ton cas tordu

ça semble quand même bancal de reposer sur un pipeline spécifique de crayons non ? Le code dérogatoire qui passe pas par `objet_modifier()` est du vieux code legacy qui normalement n'est utilisé que pour la compat avec les anciens codes. Ça me parait ni sain ni souhaitable de venir y mettre des nouveaux plugins qui créent des chemins de mise à jour dérogatoire... Il vaudrait mieux pas avoir des fonctions wrapper update pour ton objet, et tu y gères l'appel aux pipelines standards ? Tu peux voir ici https://git.spip.net/spip-contrib-extensions/crayons/src/branch/master/action/crayons_store.php#L258 que crayons teste l'existence de fonctions `xxx_revision`, amha c'est dans une fonction de ce type qu'il faut que tu gères ton cas tordu
Owner

@maieul avait répondu sur une autre PR #2 et l'avait fermée par erreur.

Du coup je ferme ici, il y a tout ce qu'il faut dans le plugin crayons

@maieul avait répondu sur une autre PR https://git.spip.net/spip-contrib-extensions/crayons/pulls/2#issuecomment-1641 et l'avait fermée par erreur. Du coup je ferme ici, il y a tout ce qu'il faut dans le plugin crayons
cerdic closed this pull request 1 year ago
cerdic deleted branch pipeline_post_store 1 year ago
This pull request cannot be reopened because the branch was deleted.
Sign in to join this conversation.
No reviewers
No Label
No Milestone
No Assignees
2 Participants
Notifications
Due Date

No due date set.

Dependencies

This pull request currently doesn't have any dependencies.

Loading…
There is no content yet.