Tickets :: [#4975] Horde_Editor::TinyMCE JS syntax error

6.0.0-beta1

9/24/25

Summary	Horde_Editor::TinyMCE JS syntax error
Queue	Horde Framework Packages
Queue Version	HEAD
Type	Bug
State	Resolved
Priority	1. Low
Owners	slusarz (at) horde (dot) org
Requester	mclion (at) obala (dot) net
Created	02/06/2007 (6805 days ago)
Due
Updated	02/12/2007 (6799 days ago)
Assigned	02/06/2007 (6805 days ago)
Resolved	02/09/2007 (6802 days ago)
Github Issue Link
Github Pull Request
Milestone
Patch	No

02/12/2007 11:32:13 PM	Michael Slusarz	Comment #4	Reply to this comment
Hm... in the fix you override the $params variable with $params = array() before the loop and then try to do something with this empty variable... maybe you didn't notice that you used the same name for a temp variable as the input one... Fixed.

02/12/2007 11:06:46 AM	mclion (at) obala (dot) net	Comment #3	Reply to this comment
Fixed, albeit in a slightly different way than suggested. Hm... in the fix you override the $params variable with $params = array() before the loop and then try to do something with this empty variable... maybe you didn't notice that you used the same name for a temp variable as the input one...

02/09/2007 06:03:06 AM	Michael Slusarz	Comment #2 State ⇒ Resolved	Reply to this comment
Fixed, albeit in a slightly different way than suggested.

02/06/2007 04:12:20 PM	Chuck Hagenbuch	State ⇒ Assigned Assigned to Michael Slusarz

02/06/2007 03:22:23 PM	mclion (at) obala (dot) net	Comment #1 Priority ⇒ 1. Low Type ⇒ Bug Summary ⇒ Horde_Editor::TinyMCE JS syntax error Queue ⇒ Horde Framework Packages State ⇒ Unconfirmed	Reply to this comment
/Horde/Editor/tinymce.php foreach ($params['config'] as $config => $value) { $js .= 'tinyMCE.init({' . $config . ' : ' . '"' . $value . '"})'; } this should be like this: foreach ($params['config'] as $config => $value) { $js .= 'tinyMCE.init({' . $config . ' : ' . '"' . $value . '"});'."\n"; } So it's missing the ; or a new line or both...if someone specifies more than one config value...