TinyMCE needs to be upgraded to the new Forms API. Going to start on a patch today hopefully have something posted for further review.

Comments

sangamreddi’s picture

I am waiting to see this.
However, i just included the legacy.inc in commom.inc using include once.
Now the Tinymce settings page is working fine, but tinymce isn't loading on node/add.
I still see the basic textarea, no enable/disable option too.

many people like me are waiting.

m3avrck’s picture

StatusFileSize
new17.01 KB

Here is a patch that upgrades TinyMCE to the new Forms API.

Still needs a bit of testing/tweaking but at least something to play around with. I had to comment out the part that reads the plugins near the top, it kept giving me all sorts of errors, I'll look more into this tomorrow. At least this gets the ball rolling :-)

Contact me on #drupal for questions.

m3avrck’s picture

Status: Active » Reviewed & tested by the community
StatusFileSize
new17.85 KB

Ok here is a new patch, Forms API is working 100% now, fully tested.

Also fixed two bugs: file('filename', 'r') is actually invalid, only need that on fopen() so I removed the 'r' there since that is how the file() works anyways.

On the INSERT at the bottom, because of the way checkboxes worked, if two were checked, it would insert them as RID=1 for both, when in fact it would be RID 1 and RID 2 and both have values=1 (both are checked). Put in a simple fix for that.

matt westgate’s picture

Hrm, patch doesn't want to apply. Care to upload a fresh one?

m3avrck’s picture

StatusFileSize
new17.85 KB

Ok how about this one.

m3avrck’s picture

StatusFileSize
new35.39 KB

Here is my full patched tinymce.module ... hopefull this works!

m3avrck’s picture

Status: Reviewed & tested by the community » Fixed

Patch has been committed, but there is another problem with the Forms API that needs to be fixed and then TinyMCE will be working: http://drupal.org/node/37586

Anonymous’s picture

Status: Fixed » Closed (fixed)