Skip to:

  1. Skip to Jira Navigation
  2. Skip to Side Navigation
  3. Skip to Main Content

CKEditor Media Embed should not put the user media in the movie param

Description

While testing CK Editor on [1] using Firefox 19.0.2, I was not able to insert most of the media types mentioned in the test script (.flv, .wma, .avi, & .mov).
See: media_errors.txt, ckeditor_media_errors.jpg
Test Script:
A3. Rich Text Editor / Inserting items
1. Insert the following:

  • Horizontal line

  • Smiley face

  • Special character

  • Image

  • .flv file (failed)

  • .wma file (failed)

  • .avi file (failed)

  • .mov file (failed)

Environment

[1] trunk.sakaiqa.org

Test Plan

None

Attachments

39
  • 11 Apr 2013, 01:54 PM
  • 11 Apr 2013, 01:54 PM
  • 11 Apr 2013, 01:54 PM
  • 11 Apr 2013, 01:54 PM
  • 11 Apr 2013, 01:54 PM
  • 11 Apr 2013, 01:54 PM
  • 11 Apr 2013, 01:53 PM
  • 11 Apr 2013, 01:53 PM
  • 11 Apr 2013, 01:53 PM
  • 11 Apr 2013, 01:53 PM
  • 11 Apr 2013, 01:53 PM
  • 11 Apr 2013, 01:53 PM

Activity

Show:

Neal Caidin May 16, 2013 at 8:41 AM

Notice that Andrea Schmidt cannot reproduce the problem I reported using Windows 7 and IE 9. The only difference is that I am using a VBox for Windows 7 on my Mac and she is using Windows 7 natively.

Neal Caidin May 15, 2013 at 8:56 AM

I tested this on the UvA QA server for CLE 2.9.2-rc02 and for a Flash movie uploaded to Resources, I can successfully include it in an Announcement with Mac OS X / Safari, but it fails on Windows 7 / IE 9.

Here is the error get 'Alert: The embed tag contained an attribute that we could not process. The movie attribute had a value of "http://qa1-nl.sakaiproject.org/access/content/group/835bda45-1c55-492d-9e97-8ae7865eb69e/barsandtone.flv". This value could not be accepted for security reasons. We have chosen to remove the entire embed tag in order to continue processing the input.'

I think I should open a new ticket for this, correct?

Thanks.

Anthony Whyte May 3, 2013 at 12:04 PM

Merged 2.9.x r123586.

Neal Caidin May 2, 2013 at 8:12 AM

Verified on trunk + experimental on HSQLDB - Built: 05/02/13 02:30 - Sakai Revision: 123505 (Kernel 1.4.0-SNAPSHOT)- Server sakai-nightly.uits.iupui.edu

Uploaded FLV file, Included in an Announcement.

Sam Ottenhoff April 29, 2013 at 12:06 PM

Yes, 2.9.x Merge flag is set. Needs verification and then it should be merged before 2.9.2 - rc02

Fixed

Details

Assignee

Reporter

Components

Fix versions

Affects versions

Priority

Created March 25, 2013 at 2:08 PM
Updated June 8, 2013 at 12:59 PM
Resolved April 12, 2013 at 12:36 PM

Flag notifications