Issue Details (XML | Word | Printable)

Key: MDL-11315
Type: Bug Bug
Status: Open Open
Priority: Major Major
Assignee: Mathieu Petit-Clair
Reporter: Mark Nielsen
Votes: 0
Watchers: 0
Operations

Add/Edit UI Mockup to this issue
If you were logged in you would be able to see more operations.
Moodle

New theme setting $THEME->htmleditorpostprocess

Created: 18/Sep/07 04:16 AM   Updated: 21/Apr/08 03:20 PM
Return to search
Component/s: HTML Editor, Lib, Themes
Affects Version/s: 1.8
Fix Version/s: None

Participants: Mark Nielsen and Mathieu Petit-Clair
Security Level: None
Affected Branches: MOODLE_18_STABLE


 Description  « Hide
I noticed the theme setting $THEME->htmleditorpostprocess in lib/weblib.php. This is great since it is needed and we currently have our own version of the HTMLEditor post process, but we would like to use the standard code.

From what I can tell, the current code sticks the HTMLEditor name into a global JavaScript variable, but what happens if there is more than one HTMLEditor on the page? The last one would over write all the previous ones.

In our code, we have a global JavaScript array declared in the theme and then lib/weblib.php prints JavaScript that looks to see if the global array exists, if yes, then save the necessary JavaScript to be evaled or if not, then do the default implementation (meaning, no post rendering).

The HTMLEditor is a beast of its own, so perhaps the current code does work but I don't know how it works. If this is the case, then please provide some example code or some sort of explanation ;)

Cheers and thanks!

 All   Comments   Change History   Version Control      Sort Order: Ascending order - Click to sort in descending order
Martin Dougiamas made changes - 21/Apr/08 03:20 PM
Field Original Value New Value
Assignee Janne Mikkonen [ julmis ] Mathieu Petit-Clair [ scyrma ]