Sorry for my bad English.

I'd used this module with CKEditor 3.2.1 without any problems.

Today I've updated to CKEditor 3.3 (released at May 28) and met strange behavior:
- CKEditor for new content is loaded properly (Add story, Add page). All functions work as expected.
- But if you try to edit existing content CKEditor is loaded but no any text is shown in editor. Just white field instead of node content. If I press "Show source" button - source is show. Returning back is not work, most editor buttons are disabled.
- I've tried CKEditor 3.2.2 - same strange behavior.
- Rolling back to CKEditor 3.2.1 work fine, editor works as expected.

Please help.

CommentFileSizeAuthor
#4 ckeditor bug.jpg169.05 KBTrips
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

gwaihx’s picture

I can at least reproduce this with the current dev-version and CKEditor 3.3
It does even not work with a clean install of both.

mcritchlow’s picture

could this be do to the following in the changelog for 3.3?

#3593 : The editing area is now enabled by contentEditable="true" instead of designMode="on" to allow creating uneditable content elements in all browsers.

enzipher’s picture

Title: Update to CKEditor 3.3 fails. » Teaser break not working with CKEditor 3.3

Hi, I was just going to file a report and notice this issue, which seems to be the same as I encountered. Here are my findings:

I have tested this on two different installations, and it's the same issue on both. When clicking the teaser break button in the editor nothing happens, and the Javascript errors below occur. Additionally, when loading a previous post that has a teaser break, the editors text area does not load up, and all following editors fail to load. When clicking the Source button it's possible to manually remove the current teaser break, and when clicking the Source again the text loads up fine, but the teaser button still doesn't work.

Module: CKEditor 6.x-1.1
Editor: CKEditor 3.3

These are the errors that shows up when clicking the teaser button:

Firefox 3.6.3

Error: l.getAttribute is not a function
Source File: /sites/all/libraries/ckeditor/ckeditor.js?I
Line: 114

IE8

Message: 'attributes.align' is null or not an object
Line: 115
Char: 269
Code: 0
URI: /sites/all/libraries/ckeditor/ckeditor.js?I

Possibly related
CKEditor 3.3 new features:
#4056 : Hidden fields will now be displayed as fake element just like in FCKeditor 2.

Hope this shines some light over this issue.

Cheers

Trips’s picture

FileSize
169.05 KB

I'm running across this same problem when trying desperately to get page break to work for my blogs.

You can easily recreate this problem by selecting "Switch to plain text editor" while using ckeditor. Once in there, click "Split summary at cursor" breaking the page into two. Save and then go back and edit the post. You will notice that ckeditor is now grayed out, only displaying the buttons.

Clicking source or switching to plain text editor shows the text and allows you to edit it, but not through ckeditor.

It would be amazing if someone could figure out a working page break in ckeditor. It is driving me crazy having to use the plain text editor to break the page up and then be unable to use ckeditor for editing posts.

mephir’s picture

Status: Active » Fixed

Changes commited into cvs. Bug fixed, please download the latest dev version, after 1:00PM CET or use cvs.
If you have any lack of clarity about issue, feel free to reopen it.

Georgii’s picture

Version: 6.x-1.1 » 6.x-1.x-dev
Status: Fixed » Active

dev version downloaded 5 minutes ago.
The bug still exits as described.
I tried to edit an existing node with teaser and main text. I see only empty text space in CKEditor (v. 3.3).

Trips’s picture

Second Edit:

I just installed the fix and it is indeed working for me. It has even fixed the pagebreak issue so I no longer need to work-around it. Thank you so much for the fix!

Be sure that you upload the fix to the modules folder, not the ckeditor folder.

Georgii’s picture

Of course :) But even complete re-installation of this module (6.x-1.x-dev 2010-Jun-14) with CKEditor 3.3.1 didn't work for me.
I go edit previously created text or I go create new one and see the same behavior.

"Insert Teaser Break" button doesn't do anything and if having two parts of the content (teaser & main part), editor does not show anything.

I'm using FF 3.6.3

Georgii’s picture

When installing on a Drupal with no previous installation of this module it seems to be working fine. What can be the reason?
I was trying module un-installation.
Removal of the module directory
Cache wipe and disabling.

Another piece of debug: When pushing the "Insert teaser break" button I see the following error:

Error:l.getAttribute is not a function
Source: http://igeorg.ru/sites/all/modules/ckeditor/ckeditor/ckeditor.js?I
Line: 115

Help is needed :)

mephir’s picture

@Georgii: which browser do you use? I'm trying to find any error in ff 3.6.3, ie8, safari 5, Opera and nothing worries happend.

mephir’s picture

Status: Active » Postponed (maintainer needs more info)
Georgii’s picture

Status: Postponed (maintainer needs more info) » Active

Dear mephir,
Thanks for checking this!

I'm using FF 3.6.3. I was trying installed IE 6.0. IE has also reported a script problem.
This is the problem I'm having on my main site where at first I have installed 6.x-1.1 version and then upgraded to 6.x-1.x-dev

However, I was trying installing this module to the installation of Drupal without previously installed CKEditor of earlier versions. So that the 6.x-1.x-dev 2010-Jun-14 was there first and the only version of CKEditor module. And there it seems working correctly.

On my main site I was trying to completely wipe this module out. Uncheck in modules list, uninstall, cleaned out from sites/all/modules, un-archiving there an installation tar-file and installing it in modules list. Does not work still.

I'd appreciate any help here.

mephir’s picture

Did you clear broswer cache ?

Georgii’s picture

Yes. All the caches were cleared out. Drupal caching was disabled IE and FF were forced to clear their cache before I navigate to the page for content editing.

I could provide more details if needed. Please let me know what can help.

mephir’s picture

It looks like cache problem, maybe your internet provider use some type of transparent proxy? Can you reproduce it on other computer or browser. I check plugin file on your website but, there is the latest version. Try to clear drupal cache, sometimes disabling them don't get expected results. Can you create test account with access to place where this bug exists and send me a PM with login and password?

Georgii’s picture

No I haven't tried another computer or browsers except of FF and IE, I will try so.
Also I have created an account for you and sent it to you.
Please let me know if there is anything else I can help you.

Thanks in advance.

mephir’s picture

Thanks for account, the problem must be into cache. When i last check drupalbreaks/plugin.js on your site there was proper file and still is it. I think it could depend on server cache configuration, which returns old file instead of properly. Quick fix:

  • add CKEDITOR.timestamp = CKEDITOR.timestamp + '1'; on the end of includes/ckeditor.utils.js, or
  • in ckeditor.config.js add third function parameter when loading drupal break plugin.
    About line 116 you have got code:
    CKEDITOR.plugins.addExternal('drupalbreaks', Drupal.settings.ckeditor.module_path + '/plugins/drupalbreaks/');
    

    please change them into

    CKEDITOR.plugins.addExternal('drupalbreaks', Drupal.settings.ckeditor.module_path + '/plugins/drupalbreaks/', 'plugin.js?1');
    

I'll make some protection instead of this cache issue into module 1.2 version.

Georgii’s picture

Status: Active » Fixed

Thank you so much!

The following modification DID WORK for me
CKEDITOR.timestamp = CKEDITOR.timestamp + '1';

Thank you again!

EWB’s picture

I would like to confirm that the following (as suggested by Mephir in #5) fixed this issue for me :

1. Update module to 6.x-1.x-dev
2. Update to CKEditor 3.3.1
3. Clear browser cache (FF 3.6.3 in this case)

This appears to fix the issue for both existing nodes and newly created nodes.

peťo’s picture

I had the same problem as Georgii, but thanks to mephir post #17 it is fixed for me! thank you!

nicholasThompson’s picture

I can also confirm -dev fixed the issue (after clearing the browser cache).

DanilaD’s picture

Updating to the 6.x-1.x-dev version of module and clearing browser cache helped me. Thanks a lot!

lxm’s picture

Status: Fixed » Needs work

I've update to 6.x-1.x-dev and 3.3.1

The teaser break button does not show.

I like the ckeditor, but I've had to go back to fckeditor, because there is no teaser break in ckeditor.

I hope some one can fix this

Lloyd Standish’s picture

I would like to confirm EWB's information. After installing ckeditor 3.3.1 and updating the Drupal module to 6.x.1.x-dev and *clearing the browser cache* on Debian Iceweasel v 3.0.6 , the issue is fixed for both existing nodes and newly created ones. I found it was NOT necessary to make the module code changes mentioned by mephir in #17.

To Ixm (#23): did you clear your browser's cache after installing 6.x-1.x-dev and 3.3.1?

Thanks very much, mephir, for fixing this problem, and congratulations on an outstanding module.

abaddon’s picture

Status: Needs work » Reviewed & tested by the community

this fixed it for me as well, had to clear browser cache before, why isnt it in the stable release? :)

lxm’s picture

Yes, I cleared the browser cache and the drupal cache.

I also ran it in a browser I don't ordinarily use. I also ran it on a different machine entirely. Teaser break still is not appearing. I think I also tried running it under WYSIWIG.

lxm’s picture

I got it to work, finally.

First, I made a clean install of drupal 6.17 and made sure the teaser break appeared there. It did.
I uninstalled WYSIWYG and its associated libraries.

Then I reinstalled ckeditor (dev and 3.3.1).
Then I cleared the cache.
Then I disabled the cache.
Then I enabled the ckeditor.

Then the teaser break reappeared.

My guess is that disabling the cache was the critical item, but who knows.

imp_squared’s picture

add CKEDITOR.timestamp = CKEDITOR.timestamp + '1'; on the end of includes/ckeditor.utils.js

fixed it for me as well.

ArthurC’s picture

Just installed 6.x-1.1 & 3.3.2, in a clean Drupal 6.17 install.

CKEditor appears, with the Insert Teaser Break button.

- When I click the button, nothing gets inserted in the text I'm trying to edit.

- If I go into the source of the text I'm trying to edit, and explicitly type in

, then return from the source to the editor, the edit window is blank, and the only way I can see the text I have been editing is to switch to the plain text editor.

- Going into the plain text editor apparently strips out the

and I'm back where I started, without a teaser break.

LXM's fix (#27) includes disabling cache. Cache is disabled, but it still doesn't work. Besides, disabling cache to get around a bug in the editor code hardly seems like the right approach.

IMP_SQUARED (#28) added CKEDITOR.timestamp = CKEDITOR.timestamp + '1'; on the end of includes/ckeditor.utils.js and that worked. I tried it. It didn't do anything, but maybe I didn't understand. You just add that stand-alone line of code all by itself? Past the end of the last function?

I uninstalled CKEditor, couldn't locate any earlier versions, so installed the FCKeditor. It seems to be working okay, including the teaser break.

I'll come back to CKEditor when there's a clean version.

Thanks, though, to all the great people out there writing good software and making it available to us.

geerlingguy’s picture

I can also confirm updating to the -dev release fixes the issue. Just be sure your clear your browser cache!

abaddon’s picture

Status: Reviewed & tested by the community » Fixed

this should actually be marked fixed as per mephir's #5, its already committed in -dev

mtpultz’s picture

I'm not sure where I'm supposed to put this line of code you inserted to correct CKEditor teasers. I'm in the ckeditor.utils.js but where do I paste? Thanks :)

mtpultz’s picture

nvm, I upgraded to dev and grabbed the newest ckeditor and it all works.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

verta’s picture

Downloading the -dev and clearing cache fixed this for me as well. Just another data point if it might help to get this into the next release. Some admins are shy about dev releases.

michaelcchastain’s picture

Upgrading to dev and clearing cache did not work for me. I use boost and cleared that cache plus the default system cache...... then used drush to flush all the cache and still the

button is not working.

Jochus’s picture

I'm still getting a grey ckeditor when inserting the

line.

Drupal ckeditor: 1.2
Implementation of ckeditor: 3.4.2

Jochus’s picture

Nevermind, has been fixed by clearing ALL browser cache! Thanks!

Nikki Aiuto’s picture

Thank you for suggesting the -dev release (#5 Posted by mephir on June 14, 2010 at 3:25am ).
Yes, make sure you flush your cache. In IE, I hadn't seen an option under Delete Browsing History that preserves favorites website data. With this checked, the flush didn't seem to do the trick.