Opened 7 years ago

Closed 7 years ago

Last modified 7 years ago

#4707 closed Bug (fixed)

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 (2)

image_properties_resubmits.JPG (137.1 KB) - added by shivakumar.kandasamy 7 years ago.
image_properties_resubmits snapshot
4707.patch (1.7 KB) - added by wwalc 7 years ago.

Download all attachments as: .zip

Change History (16)

comment:1 Changed 7 years 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

comment:2 Changed 7 years 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 7 years ago by shivakumar.kandasamy

image_properties_resubmits snapshot

comment:3 Changed 7 years ago by shivakumar.kandasamy

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

comment:4 Changed 7 years 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.

comment:5 Changed 7 years ago by shivakumar.kandasamy

  • Priority changed from Normal to High

comment:6 Changed 7 years ago by fredck

#4715 has been marked as DUP.

comment:7 Changed 7 years ago by fredck

  • Priority changed from High to Normal

Changed 7 years ago by wwalc

comment:8 Changed 7 years 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.

comment:9 Changed 7 years ago by garry.yao

  • Keywords Review+ added; Review? removed

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

comment:10 Changed 7 years 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

comment:11 Changed 7 years 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.

comment:12 Changed 7 years ago by fredck

  • Owner set to wwalc

comment:13 Changed 7 years ago by wwalc

  • Resolution set to fixed
  • Status changed from new to closed

Fixed with [4715].

comment:14 Changed 7 years ago by fredck

  • Milestone changed from CKEditor 3.2 to CKEditor 3.1
Note: See TracTickets for help on using tickets.
© 2003 – 2016 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy