Opened 9 years ago

Last modified 6 years ago

#4376 confirmed Bug

Contents already be removed in <noembed></noembed> section.

Reported by: Jeff Chu Owned by:
Priority: Normal Milestone:
Component: Core : Output Data Version: 3.0
Keywords: IE8 IE7 Cc:

Description (last modified by Frederico Caldeira Knabben)

In FCKeditor 2.6.4, I insert a Flash object tag into editor with "Source" mode, and I switch to "WYSIWYG" editing mode, and switch to "Source" again, the content in <noembed></noembed> inside the EMBED tag will be removed automatically.

How to reproduce:

  1. In editor, switch to "Source" mode by click Source button on toolbar.
  2. Paste the following code:
<embed autostart="false" showstatusbar="1" wmode="transparent" type="application/x-mplayer2" height="310" width="365" src="FileDownLoad/VideoFile/Video/Test.wmv"><noembed>THIS CONTENT WILL BE REMOVED.</noembed></embed>
  1. Click "Source" button again to switch to normal mode.
  2. Click "Source" button to view source, you can notice <noembed></noembed>, the content already be removed automatically like this:
<embed autostart="false" showstatusbar="1" wmode="transparent" type="application/x-mplayer2" height="310" width="365" src="FileDownLoad/VideoFile/Video/Test.wmv"><noembed></noembed></embed>

Attachments (2)

FCKEditorProblemReproduce.rar (654.1 KB) - added by Jeff Chu 9 years ago.
ReproduceSteps.pdf (273.5 KB) - added by Jeff Chu 9 years ago.
Steps to reproduce problem

Download all attachments as: .zip

Change History (12)

comment:1 Changed 9 years ago by Frederico Caldeira Knabben

Component: AccessibilityCore : Output Data
Description: modified (diff)
Keywords: WorksForMe added; noembed accessibility removed
Milestone: FCKeditor 2.6.5
Priority: HighNormal

I've tested it with both the FCKeditor trunk and CKEditor and it works for me. I've precisely execute the described steps to reproduce and the noembed contents got preserved.

Do you have more information on this?

Changed 9 years ago by Jeff Chu

comment:2 Changed 9 years ago by Jeff Chu

I packed a rar file contains a HTML for reproduce this problem and entire FCKeditor files I used for this problem.

Please check it. Thanks.

comment:3 Changed 9 years ago by Frederico Caldeira Knabben

Keywords: Confirmed added; WorksForMe removed
Resolution: fixed
Status: newclosed

I'm not able to confirm this issue in the way you've described it with FCKeditor 2.6.4. I'm having the following instead, which is still not good:

<embed src="mydata.swf"></embed>
<p><noembed>NOEMBED DATA</noembed></p>

With CKEditor we instead have it working properly, so we can consider it fixed for our ticketing system purposes.

comment:4 Changed 9 years ago by Jeff Chu

Keywords: WorksForMe added; Confirmed removed
Resolution: fixed
Status: closedreopened

Thanks for your response. But, I tried your code, problem still exist.

I try to describe by some pictures included in PDF File in latest attachment file. Please confirm it. Thanks.

Changed 9 years ago by Jeff Chu

Attachment: ReproduceSteps.pdf added

Steps to reproduce problem

comment:5 Changed 9 years ago by Frederico Caldeira Knabben

Keywords: Confirmed added; WorksForMe removed
Resolution: fixed
Status: reopenedclosed

As said, I confirm that this is an issue with FCKeditor 2.6.4, but it has been fixed with CKEditor 3.0.

comment:6 Changed 9 years ago by Jeff Chu

Keywords: WorksForMe added; Confirmed removed
Resolution: fixed
Status: closedreopened
Version: FCKeditor 2.6.4CKEditor 3.0

I tried in CKEditor 3.0, but problem still exist. Reproduce step is this same of above.

If you need the screen capture, I can provide here later. Thanks.

comment:7 Changed 9 years ago by Frederico Caldeira Knabben

Keywords: Confirmed IE added; WorksForMe removed

Ok, there was an important information missing here. This is an IE only issue (with CKEditor). Works well with Firefox.

(PS: Please do not change the keywords. This is for internal use and you've been using it improperly)

comment:8 Changed 9 years ago by Alfonso Martínez de Lizarrondo

I would close this as wontfix.

First: <noembed> is deprecated, and CKEditor supports <object> instead of <embed>.

Second: it's not clear at all that nesting <noembed> within <embed> is valid at all, so moving it outside would be the correct behavior.

Check http://webdesign.about.com/od/htmltags/p/bltags_noembed.htm and http://www.blooberry.com/indexdot/html/tagpages/n/noembed.htm

comment:9 Changed 9 years ago by Jeff Chu

I tried with CKEditor 3.0 with FireFox, it works fine.

But it works with error in IE browser (I tried with IE8), include "Insert Flash" feature (<object> tag with <embed> tag).

Please test by the following code (generated by CKEditor 3.0) in IE:

<p>
<object classid="clsid:d27cdb6e-ae6d-11cf-96b8-444553540000" codebase="http://download.macromedia.com/pub/shockwave/cabs/flash/swflash.cab#version=6,0,40,0"><param name="quality" value="high" /><param name="movie" value="mydata.swf" /><embed pluginspage="http://www.macromedia.com/go/getflashplayer" quality="high" src="mydata.swf" type="application/x-shockwave-flash"></embed></object></p>
<noembed>DATA</noembed>

The error will be reproduced.

Thanks.

comment:10 Changed 6 years ago by Jakub Ś

Keywords: IE8 IE7 added; Confirmed IE removed
Status: reopenedconfirmed

I see that contents of noembed are removed in IE8 and IE7 only.

Note: See TracTickets for help on using tickets.
© 2003 – 2017 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy