Dotclear

Ticket #2042 (closed defect: fixed)

Opened 11 years ago

Last modified 10 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 11 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 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 ?)

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

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 ?

Version 0, edited 11 years ago by bruno (next)

comment:7 Changed 10 years ago by franck

  • Owner changed from team to franck

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

Note: See TracTickets for help on using tickets.

Sites map