Development

#3474 (Bug in sfRichTextEditorTinyMCE.class.php)

You must first sign up to be able to contribute.

Ticket #3474 (closed defect: fixed)

Opened 2 years ago

Last modified 2 years ago

Bug in sfRichTextEditorTinyMCE.class.php

Reported by: jamesmoey Assigned to: FabianLange
Priority: minor Milestone: 1.0.15
Component: helpers Version: 1.0.13
Keywords: sfRichTextEditorTinyMCE Cc:
Qualification: Accepted

Description

The id for textarea and elements id specify in tinyMCE.init function can become different. id for textarea is from get_id_from_name function, which convert "[]" to "_". Hence tinyMCE will not be able to find the element. editor will not be created if the element id contain "[]".

SOLUTION Assign the same id for both textarea element and tinyMCE.init( {elements} ) function.

Change History

05/05/08 09:22:28 changed by FabianLange

  • version changed from 1.1.0 DEV to 1.0.13.
  • qualification changed from Unreviewed to Design decision.
  • milestone deleted.

confirmed. idea would be to make 

protected function getID()
{
  return get_id_from_name(_get_option($this->options, 'id', $this->name), null);
}

in sfRichTextEditor.class and use this instead. However I haven't checked if the helper functions will always be available wehen invoking this. Needs to be verified.

(PS I think this applies for 1.0 as well)

05/06/08 23:43:47 changed by FabianLange

  • owner changed from fabien to FabianLange.
  • status changed from new to assigned.

05/07/08 00:06:40 changed by FabianLange

  • status changed from assigned to closed.
  • resolution set to fixed.
  • qualification changed from Design decision to Accepted.
  • milestone set to 1.0.15.

(in r8819) corrected ID generation for TinyMCE rich editor when no id was given. fixes #3474

The Sensio Labs Network

Since 1998, Sensio Labs has been promoting the Open-Source software movement by providing quality web application development, training, consulting.
Sensio Labs also supports several large Open-Source projects.

Open-Source Products

Services

> More