Download & Extend
Autosave » Issues

FCKEditor support for D6

Posted by Viybel on September 17, 2008 at 11:06am
Project:Autosave
Version:6.x-2.2
Component:Code
Category:feature request
Priority:normal
Assigned:Unassigned
Status:needs review

Issue Summary

Has anyone ported this patch?
#100335: FCKEditor Support

Vianney Stroebel
Likwid - Spécialistes Drupal - Paris

Login or register to post comments

Comments

#1

Posted by darren.ferguson on September 19, 2008 at 8:26pm
Status:active» needs review

The attached patch supports the FCKEditor functionality for the autosave module in D6.

The patch also includes the elements to ignore in form hash portion per http://drupal.org/node/308239 and they all seem to work well. Please test and let me know if any bugs are in these features.

Patches are against the autosave-6.x-1.1.tar.gz branch.

AttachmentSize
autosave.js_.patch 6.15 KB
autosave.module.patch 1.73 KB

#2

Posted by flexer on October 8, 2008 at 4:49pm

Hi,

I'll attach my patch that add this functionality, maybe merging the two could be better :)

I've done some little improvements to the js too, because it defines a lot of global variables (those which I added the "var" in front) and uses an obsolete form for the first parameter of setTimeout()

AttachmentSize
autosave_001.patch 4.19 KB

#3

Posted by noelbush on January 26, 2009 at 10:11am

The patch provided by flexer has an error -- it adds "var" before pos on line 84, causing the script to be unable to run.

Otherwise, though, I want to report that the patch applies to the 5.x-3.2 release just fine, and fixes the problem with FCKeditor. It is not necessary to apply the patch to the module file from comment #1 (and I am not sure why that was included here to begin with).

I also want to note that the current 5.x-3.x-dev release, as well as the 5.3 branch in Drupal, breaks compatibility with Drupal 5.

#4

Posted by designtist on January 29, 2009 at 7:17pm

noelbush, can you clarify your comment -- which patch applies to the 5.x-3.2 release? The one in comment #1 or the one in comment #2? If #2, doI have to remove "var" on line 84 to make it work?

thanks!

#5

Posted by pvhee on March 30, 2009 at 5:10pm

subscribing

#6

Posted by rjmackay on February 22, 2010 at 6:55am
Version:6.x-1.1» 6.x-2.2

I've just made a patch based on #1
But updated it to work with the fckeditor module 6.x-2.0-beta1 and the autosave 6.x-2.2 and removed the other fixes in the patch so now it just adds FCKEditor support.

I don't think this is the direction the module was aiming to head in (they want to support wysiwyg editors through the wysiwyg module instead) but its useful for me so hopefully it'll help someone else out too

AttachmentSize
fckeditor_autosave.patch 1.82 KB

#7

Posted by DamienMcKenna on September 9, 2010 at 9:41pm

FYI I marked #650768: Autosave with FCKEditor in WYSIWYG with patch not working as a duplicate.