Transmettre #ENV à propre() et aux modèles #2367

Closed
opened 11 years ago by davux · 4 comments
davux commented 11 years ago
Owner

Il est impossible d'utiliser <formulaire|ecrire_auteur> dans le corps d'un article. Le compilateur dit:

Champ FORMULAIRE_ECRIRE_AUTEUR hors d'un contexte AUTEURS/ARTICLES

Techniquement, il s'agit d'une demande d'évolution car ça n'a jamais marché ainsi, donc ça serait une fonctionnalité supplémentaire. D'un autre côté, d'un point de vue de rédacteur, il est logique de s'attendre à pouvoir utiliser cette écriture. Par ailleurs le contexte d'article est évident car on est dans le corps d'un article. Je le mets donc comme bug, mais avec priorité basse du fait que ça ne casse rien d'existant.

Il est impossible d'utiliser `<formulaire|ecrire_auteur>` dans le corps d'un article. Le compilateur dit: > Champ FORMULAIRE_ECRIRE_AUTEUR hors d'un contexte AUTEURS/ARTICLES Techniquement, il s'agit d'une demande d'évolution car ça n'a jamais marché ainsi, donc ça serait une fonctionnalité supplémentaire. D'un autre côté, d'un point de vue de rédacteur, il est logique de s'attendre à pouvoir utiliser cette écriture. Par ailleurs le contexte d'article est évident car on est dans le corps d'un article. Je le mets donc comme bug, mais avec priorité basse du fait que ça ne casse rien d'existant.
Owner

Ce ticket cache en fait le problème historique et récurrent que aucun contexte n'est transmis à propre() et donc aux modèles inclus dans un texte. J’étais persuadé qu'il y avait déja un ticket sur ce sujet, mais je ne le retrouve pas. Je change donc le titre de celui-ci.
Version cible mise à 3.1

Ce ticket cache en fait le problème historique et récurrent que aucun contexte n'est transmis à `propre()` et donc aux modèles inclus dans un texte. J’étais persuadé qu'il y avait déja un ticket sur ce sujet, mais je ne le retrouve pas. Je change donc le titre de celui-ci. **Version cible mise à 3.1**
Poster
Owner

Ah ok, merci des clarifications.

Par contre tu ne peux pas changer le type silencieusement. J'ai pris la peine d'expliquer dans le ticket pourquoi après hésitations j'avais choisi de le mettre comme bug + priorité basse, et non comme évolution: d'un point de vue utilisateur l'écriture <formulaire|xxx> doit marcher, et le contexte d'article existe bien, donc le fait qu'il ne soit pas pris en compte est bien un bug. Priorité basse car le bug a toujours existé donc ça ne casse rien, mais ça n'en reste pas moins un bug.

Après ça me va qu'on change le type mais il faudrait l'argumenter.

Ah ok, merci des clarifications. Par contre tu ne peux pas changer le type silencieusement. J'ai pris la peine d'expliquer dans le ticket pourquoi après hésitations j'avais choisi de le mettre comme bug + priorité basse, et non comme évolution: d'un point de vue utilisateur l'écriture `<formulaire|xxx>` doit marcher, et le contexte d'article existe bien, donc le fait qu'il ne soit pas pris en compte est bien un bug. Priorité basse car le bug a _toujours_ existé donc ça ne casse rien, mais ça n'en reste pas moins un bug. Après ça me va qu'on change le type mais il faudrait l'argumenter.
Owner

Je comprends bien que du point de vue <formulaire|xxx> c'est un bug. Il pourrait se résoudre en patchant le balise/formulaires_ecrire_auteur pour choisir un destinataire par défaut si pas de contexte.

Mais si on se ramène au problème général de l'environnement qui n'est pas transmis aux modèles insérés dans le texte, ce n'est pas un bug, mais bien une évolution fonctionnelle. Peut-être j'aurais mieux fait d'ouvrir un ticket différent...

Je comprends bien que du point de vue `<formulaire|xxx>` c'est un bug. Il pourrait se résoudre en patchant le balise/formulaires_ecrire_auteur pour choisir un destinataire par défaut si pas de contexte. Mais si on se ramène au problème général de l'environnement qui n'est pas transmis aux modèles insérés dans le texte, ce n'est pas un bug, mais bien une évolution fonctionnelle. Peut-être j'aurais mieux fait d'ouvrir un ticket différent...
Owner
Intégré par http://core.spip.org/projects/spip/repository/revisions/18654 http://core.spip.org/projects/spip/repository/revisions/18655 http://zone.spip.org/trac/spip-zone/changeset/53797 http://zone.spip.org/trac/spip-zone/changeset/53798 http://zone.spip.org/trac/spip-zone/changeset/53799 http://zone.spip.org/trac/spip-zone/changeset/53700 http://zone.spip.org/trac/spip-zone/changeset/53701 **Version cible mise à 3.0** **Statut changé à Fermé**
Sign in to join this conversation.
No Milestone
No project
No Assignees
2 Participants
Notifications
Due Date

No due date set.

Dependencies

This issue currently doesn't have any dependencies.

Loading…
There is no content yet.