Dotclear

Ticket #2042 (closed defect: fixed)

Opened 9 years ago

Last modified 9 years ago

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:1 Changed 9 years ago by franck

  • Milestone changed from 2.7.3 to 2.8

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

comment:2 Changed 9 years ago by bruno

Si on ajoute un paramètre facultatif à la méthode, ça ne casse rien...

comment:3 Changed 9 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 ?)

Last edited 9 years ago by franck (previous) (diff)

comment:4 follow-up: ↓ 5 Changed 9 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 9 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 9 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 ?

En ce qui concerne dcLegacyEditor, c'est en fonction du mode actif (côté JS), côté dcCKEditor il n'y a qu'une seule syntaxe (xhtml), donc le besoin ne s'est pas fait sentir, jusqu'à maintenant.

Je suis pour ce nouvel argument, mais si j'intègre ça dans la 2.7.3, on casse les deux plugins que je mentionnais et peut-être d'autres…

Last edited 9 years ago by franck (previous) (diff)

comment:7 Changed 9 years ago by franck

  • Owner changed from team to franck

comment:8 Changed 9 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

Note: See TracTickets for help on using tickets.

Sites map