Ticket #4707 (closed Bug: fixed)

Opened 4 months ago

Last modified 3 months ago

CKEditor image Toolbar resubmits on click in IE 7 and FireFox 3.0.x

Reported by: shivakumar.kandasamy Owned by: wwalc
Priority: Normal Milestone: CKEditor 3.1
Component: General Version:
Keywords: Confirmed Review+ Cc:

Description

Hi,

When I click the image icon in CK Editor, the Image Properties

is displayed. The issue noted is:

1)The page url gets resubmitted in Firefox/3.0.15

2)An empty url gets submitted in IE7

On close of Image Properties an empty url gets submitted in IE7 and gets aborted.

This cause lot of issue as duplicate records get created and empty mails sent to the user.

I have searched google for the last two days but could not find any solution.

Any help in this regard is appreciated.

Attachments

image_properties_resubmits.JPG Download (137.1 KB) - added by shivakumar.kandasamy 4 months ago.
image_properties_resubmits snapshot
4707.patch Download (1.7 KB) - added by wwalc 4 months ago.

Change History

Changed 4 months ago by shivakumar.kandasamy

  • summary changed from CKEditor image Toolbar resubmits on click in IE 7 and FireFox 3.0.10 to CKEditor image Toolbar resubmits on click in IE 7 and FireFox 3.0.x

Changed 4 months ago by fredck

  • keywords WorksForMe added; Image Properties, resubmit removed

It works for me with no issues. Are you able to reproduce it at our online demo?

Changed 4 months ago by shivakumar.kandasamy

image_properties_resubmits snapshot

Changed 4 months ago by shivakumar.kandasamy

I was able to reproduce it in the demo using HttpWatchProfessional. I have attached the snapsot of the same.

Changed 4 months ago by fredck

  • keywords Confirmed added; WorksForMe removed
  • milestone set to CKEditor 3.2

Ok, now I understood the problem better. It happens with the image dialog only. It may be related to the preview image.

Changed 4 months ago by shivakumar.kandasamy

  • priority changed from Normal to High

Changed 4 months ago by fredck

#4715 has been marked as DUP.

Changed 4 months ago by fredck

  • priority changed from High to Normal

Changed 4 months ago by wwalc

Changed 4 months ago by wwalc

  • keywords Review? added

It seems that the problem is in empty "src" attribute.

An alternative to removing it would be setting "src" to something like CKEDITOR.basePath + 'images/spacer.gif', but this way we would have another HTTP request.

Changed 4 months ago by garry.yao

  • keywords Review+ added; Review? removed

The preview image is hidden by default, so no worry about the place holder image.

Changed 4 months ago by shivakumar.kandasamy

Hi,

Thanks for your prompt response. Now the image onclick problem seems to be resolved but the problem still persist in the image dialog close event and cancel event.

Any help in this regard is appreciated.

Thanks for your co-operation

Changed 4 months ago by wwalc

I cannot reproduce issues with closing image dialog (or canceling changes).

Could you check the patch using one of the samples provided with CKEditor (replacebyclass.html for example, just to be sure that issue is not caused by custom javascript on your web site)?

Please attach new screenshot with all HTTP requests made by IE7 after CKEditor is loaded to help us understand better the problem. When testing it on IE7/IE8 + Fiddler I didn't note any invalid http requests, only an image url is requested by the image dialog.

Changed 4 months ago by fredck

  • owner set to wwalc

Changed 3 months ago by wwalc

  • status changed from new to closed
  • resolution set to fixed

Fixed with [4715].

Changed 3 months ago by fredck

  • milestone changed from CKEditor 3.2 to CKEditor 3.1
Note: See TracTickets for help on using tickets.