I wanted to use TinyMCE in a 6.x installation and so started creating a patch to do so.

The patch is attached and gets us almost all of the way there--the module installs and you can add and edit profiles. You may have to 'touch tinymce.schema' in your modules/tinymce directory to get the patch to apply cleanly as this file did not exist in the previous version. This is a patch against 5.x-1.9.

It does not, however, currently show up on textareas on the node/user/etc. edit pages yet. I may have time in the next week to investigate further.

I used the "Coder" module to ensure that the code was in compliance with the Drupal spec.

CommentFileSizeAuthor
#40 tinymce.patch9.72 KBalaa
#6 tinymce.tar_.gz12.93 KBdomasj
tinymce.10.2.patch15.27 KBBeau Gunderson

Comments

murz’s picture

murz
Primary language English
Location Yerevan, Armenia
commented

subscribed

rkendall’s picture

rkendall commented

subscribe

kreynen’s picture

kreynen commented

With Drupal 6 RC1 coming this week, getting TinyMCE updated for 6 is a top priority... unfortunately one of many. I'm starting a thread about this at http://groups.drupal.org/tinymce-development

techczech’s picture

techczech commented

subscribe

IcyAndy’s picture

IcyAndy commented

subscribe

domasj’s picture

domasj commented
StatusFileSize
new tinymce.tar_.gz12.93 KB

I also tried to do some porting. The results:

  • The admnistration interface seems to be working well and it is separated from the rest of the module.
  • E_ALL errors are handled the ugliest way, but it works.

However, there are some problems.

  • The first one is that the actual textarea transformation doesn't work but it is at least initiated.
  • The second issue is about new menu system in Drupal 6 which should be used throught the module to reduce overall complexity.
Jamie Holly’s picture

Jamie Holly commented

subscribe

charles_elwood’s picture

charles_elwood commented

subscribe

kpaul’s picture

kpaul commented

bumping with the recent release of D6...

I have a little time and could do testing...

Steve Dondley’s picture

Steve Dondley commented

So we have two patches. Do we apply the first one and then the second? Someone needs to coordinate these efforts.

Steve Dondley’s picture

Steve Dondley commented

Also, I just found a bug in the tarball that was uploaded. So where is the best way to submit this patch?

I think whoever is in charge of this module needs to start a drupal6 branch and post it here so it can be worked on collaboratively.

modoq’s picture

modoq commented

subscribe

kckern’s picture

kckern commented

How do I apply the patch without command line access?

keesje’s picture

keesje commented

subscribing

Anonymous’s picture

Anonymous (not verified) commented
Priority: Normal » Critical
simontol’s picture

simontol commented

That's the reason stopping me from upgrade to 6.x

ekendra’s picture

ekendra commented

subscribing

fm’s picture

fm commented

subscribed.

ilbeppe’s picture

ilbeppe commented

subscribed

gellpak’s picture

gellpak commented

subscribe

Wutimer’s picture

Wutimer commented

Subscribed.

Devis’s picture

Devis commented

subscribe

sdub’s picture

sdub commented

subscribe

Anonymous’s picture

Anonymous (not verified) commented

is there already a view about the date that the module will be ported to drupal 6.1?

boris mann’s picture

boris mann commented
Priority: Critical » Normal

Aaargh on all the subscribes :P

Please do no set to critical -- this is not a bug, it's a feature. We have a couple of patches, please test and provide feedback.

I see multiple maintainers listed but looks like folks are out of time. I will have my team do some testing and see if we have resources to work on a port.

wildtang3nt’s picture

wildtang3nt commented

subscribe

dropcube’s picture

dropcube commented

subscribing

anuradha-1’s picture

anuradha-1 commented

subscribe

Thanks Boris for looking into the further development of the patches.

mikeryan’s picture

mikeryan
he/him
Primary language English
Location Pittsfield, MA, USA
commented

Subscribe

-hgn-’s picture

-hgn- commented

Very important update

bump

piero.giobbi’s picture

piero.giobbi commented

Hi.

how do you apply the patch?

thx

dropcube’s picture

dropcube commented

how do you apply the patch?

Read about on the handbook pages http://drupal.org/patch/apply

janeri’s picture

janeri commented

Subscribe

akie’s picture

akie commented

Hi. I've ported TinyMCE to Drupal 6. It took me over two days to fix it, but it does work now.

You can download it at http://akie.nl/2008/03/16/tinymce-for-drupal-6/

Basic functionality works for both TinyMCE 2 and TinyMCE 3. TinyMCE 2 is preferred though, since the DrupalBreak-button doesn't work on TinyMCE 3. Can someone please take a look at that? If we fix that, we'll finally have a decent editor for Drupal 6.

Anyway, good luck with this. Please leave a comment at the link above if you like it or dislike it.

Hope this helps,
Akie

ekendra’s picture

ekendra commented

bump-a-luff-a-gus-!!

Anonymous’s picture

Anonymous (not verified) commented

Here are my results after a quick test of Akie's code (I tested this in Drupal 6.1 using TinyMCE 2.1.3):

- Getting this warning message on the TinyMCE error page when creating a new profile (doesn't occur when editing a profile).
warning: Invalid argument supplied for foreach() in /var/www/local.sandbox.com/htdocs/drupal-6.1/includes/form.inc on line 1181.

- The default state option doesn't seem to have any weight on whether TinyMCE is run. Everytime I create a new page, TinyMCE is shown without an option to disable it.

- Other than those two issues, everything I was able to test in the matter of 15 minutes has worked fine and I will most likely use this for one of my clients (they won't need to disable the WYSIWYG).

Thanks for putting this together and hope those issues can get figured out soon!

owen barton’s picture

owen barton commented

Subscribe

ogi’s picture

ogi commented

subscribe

physiotek’s picture

physiotek commented

subscribing

alaa’s picture

alaa commented
StatusFileSize
new tinymce.patch9.72 KB

here is another patch this one works fine but requires some cleanup to split code into multiple files and either remove or work with drupal's teaser splitter.

IMO it needs a rewrite, the code is a mess.

xcorex’s picture

xcorex commented

subscribing

izmeez’s picture

izmeez commented

I have installed TinyMCE for Drupal 6.x as described by Akie.

After enabling the module and going to Administer>Site configuration>TinyMCE>Add new profile, I get the message:

warning: Invalid argument supplied for foreach() in \drupal\includes\form.inc on line 1181.

which has been described before by klamping.

Is this new patch file from alaa for that release or is this another patch file?

Should I apply this patch or just go ahead and use the release as it is?

Thanks for the help.

Izzy

alaa’s picture

alaa commented

izmeez the patch I posted on #40 is a separate patch and has nothing to do with Akie's module.

yogi_sydbma’s picture

yogi_sydbma commented
Status: Needs work » Closed (won't fix)

it wont work for me.

Including the project notes etc...

its there under profiles but it wont show.. it tells me that the tiectory is missing.. but its there.

John_0’s picture

John_0 commented
Assigned: Unassigned » John_0

subscribe

kreynen’s picture

kreynen commented
Version: master » 6.x-1.1-dev
Assigned: John_0 » Unassigned
Status: Closed (won't fix) » Closed (fixed)

I've created a release from Akie's updates and installed the release as a demo at http://tinymce-dev.sidewalktheory.com/. Please submit new issues and patches for the 6.x version. Closing this thread.

izmeez’s picture

izmeez commented

alaa thanks for the clarification. I came to that realization my self as I saw the link for the akie release also has discussion further down on that web page.

Izzy