Ticket #2042 (closed defect: fixed)
Add post format to behavior adminPostEditor
Reported by: | bruno | Owned by: | franck |
---|---|---|---|
Priority: | normal | Milestone: | 2.8 |
Component: | module:core | Version: | 2.7.2 |
Severity: | normal | Keywords: | |
Cc: |
Description
A new editor plugin cannot easily know the format of the contents it is asked to beautify.
Adding the post format to adminPostEditor would be appreciated.
Change History
comment:2 Changed 11 years ago by bruno
Si on ajoute un paramètre facultatif à la méthode, ça ne casse rien...
comment:3 Changed 11 years ago by franck
Si on ajoute un paramètre au moment de l'appel du behaviour ça va couiner chez ceux qui ne l'ont pas prévu, non ?
A priori, externalMedia et noembedMedia, et peut-être d'autres, devront être repris si on ajoute ce paramètre à l'appel.
Ex : dans _admin.php de externalMedia, le callback du behaviour a comme signature :
public static function adminPostEditor($editor='',$context='',array $tags=array())
Si on appelle ça avec ('dcCKEditor','post',array('#post_content'),'xhtml'), ça va couiner sur le 4e argument (ou alors j'ai pas compris un truc ?)
comment:4 follow-up: ↓ 5 Changed 11 years ago by bruno
Ce behavior fonctionne à l'envers des autres, c'est les dc*Editor qui implémentent les behaviors via addBehavior(), les autres plugins ne font que des callBehaviors.
Donc les seuls réellement impactés sont les 3 plugins dc*Editor (et le core), il leur suffit d'implémenter un paramètre $post_format par défaut à null.
comment:5 in reply to: ↑ 4 ; follow-up: ↓ 6 Changed 11 years ago by franck
Replying to bruno:
Ce behavior fonctionne à l'envers des autres, c'est les dc*Editor qui implémentent les behaviors via addBehavior(), les autres plugins ne font que des callBehaviors.
Pas tout à fait exact, les plugins qui s'installent sur les barres d'outils des éditeurs répondent à ce behaviour.
comment:6 in reply to: ↑ 5 Changed 11 years ago by bruno
Replying to franck:
Pas tout à fait exact, les plugins qui s'installent sur les barres d'outils des éditeurs répondent à ce behaviour.
Et ils font comment, ces plugins, pour savoir si on est en wiki, en markdown ou en xhtml ?
comment:8 Changed 10 years ago by franck <carnet.franck.paul@…>
- Status changed from new to closed
- Resolution set to fixed
(In [b4724a397193]) Add syntax arg to adminPostEditor behaviour (may be 'wiki', 'xhtml', 'markdown', …), fixes #2042
On va casser les plugins qui ont déjà été repris pour gérer ce behaviour, vaut mieux prévoir ça pour la 2.8