Media plugin inserting broken images in IE8 [#2024149]

Hi,
I have an issue with media plugin, and inserting styled images. I am hitting the "503 Service Unavailable - Image generation in progress. Try again shortly.". The problem is that when the <img> tag is passed to the jquery constructor $(), jquery preloads the src and the image derivative starts building. But then the html is taken from the jquery element and passed to CKEDITOR. CKEDITOR inserts the tag in the DOM, and the browser requests the src a second time, and when the browsers get the "503 Service Unavailable - Image generation in progress. Try again shortly.", they usually retry, but not IE8. With IE8, I see a broken image instead.

I am using:
ckeditor module 7.x-1.x revision 493649f5bc9f494f16975949b38ca946369c7207
media module 7.x=2.x revision ecb93923272336edcbe83db2cb6871ead8188912 (Tried using a newer version of media, but I am getting a type error.)
CKEDITOR library 3.6.6.1

My fix is the attached file, but I am interested if anyone has experienced the issue, or has an alternative setup that does not require a fix.

Comment	File	Size	Author
#2	2024149-ckeditor-media-broken-image-ie8-2.patch	1.46 KB	MiroslavBanov
#2	7.x-1.x: PHP 5.3 & MySQL 5.5, D7 Patch failed to apply
#1	2024149-ckeditor-media-broken-image-ie8-1.diff	1.46 KB	MiroslavBanov
#1	7.x-1.x: PHP 5.3 & MySQL 5.5, D7 Patch failed to apply

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

MiroslavBanov CreditAttribution: MiroslavBanov commented 20 June 2013 at 12:15

File	Size
2024149-ckeditor-media-broken-image-ie8-1.diff	1.46 KB
7.x-1.x: PHP 5.3 & MySQL 5.5, D7 Patch failed to apply

Attaching my fix

Comment #2

MiroslavBanov CreditAttribution: MiroslavBanov commented 20 June 2013 at 12:51

File	Size
2024149-ckeditor-media-broken-image-ie8-2.patch	1.46 KB
7.x-1.x: PHP 5.3 & MySQL 5.5, D7 Patch failed to apply

Adding it again because my drush make doesn't like ".diff" extension apparently

Comment #3

iSoLate CreditAttribution: iSoLate at Randstad Digital for Government of Flanders commented 29 February 2016 at 14:42

Issue summary:	View changes
Status:	Active	» Needs review

This seems to work for me.

Comment #4

29 February 2016 at 14:44

The last submitted patch, 1: 2024149-ckeditor-media-broken-image-ie8-1.diff, failed testing.

Comment #5

29 February 2016 at 14:44

Status:

Needs review

» Needs work

The last submitted patch, 2: 2024149-ckeditor-media-broken-image-ie8-2.patch, failed testing.

Comment #6

angel.h

English

Plovdiv

CreditAttribution: angel.h as a volunteer and commented 3 June 2016 at 16:28

Status:

Needs work

» Closed (outdated)

This issue is no longer relevant since https://www.drupal.org/node/2542566 was done and media integration was removed. The plugins/media/library.js file which it patches no longer exists. Closing it.