Ticket #1627 (closed Bug: fixed)

Opened 7 years ago

Last modified 7 years ago

Samples fail to load in IE7 in local filesystem

Reported by: martinkou Owned by: alfonsoml
Priority: Normal Milestone: FCKeditor 2.6
Component: General Version: SVN (FCKeditor) - Retired
Keywords: Review+ Cc:

Description

The bug was originally reported by alfonsoml in the FCKeditor IRC channel.

Steps to reproduce:

  1. Download the FCKeditor SVN trunk to your local filesystem on a Windows computer with IE7.
  2. Open sample01.html from local filesystem with IE7.
  3. IE7 gives out a security warning. Click allow.
  4. The FCKeditor iframe still does not work after you've clicked allow.

Attachments

1627.patch (10.9 KB) - added by alfonsoml 7 years ago.
Proposed SVN patch

Change History

comment:1 Changed 7 years ago by alfonsoml

  • Owner set to alfonsoml

The problem is that document.location.pathname returns something like "/C:\inetpub..." with a leading extra solidus. Changing the code to use document.location.href does fix it.

This also affects the <select> in sample03 and similar.

Changed 7 years ago by alfonsoml

Proposed SVN patch

comment:2 Changed 7 years ago by alfonsoml

  • Cc Confirmed IE removed
  • Keywords Confirmed IE7 Review? added

The proposed patch uses location.href instead of location.pathname and for the changes in <select>s it does strips out the query string and then adds the new one to the full href.

comment:3 Changed 7 years ago by alfonsoml

  • Keywords Confirmed IE7 removed

the report to show pending reviews seems that only shows items with exact matching of "Review?"

comment:4 Changed 7 years ago by martinkou

  • Keywords Review+ added; Review? removed

Sorry for the late review, I should have let it passed earlier. But I was concentrating on the floating dialog branch.

comment:5 Changed 7 years ago by alfonsoml

  • Status changed from new to closed
  • Resolution set to fixed
  • Milestone set to FCKeditor 2.6

Fixed with [1298]

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