Ticket #6443 (closed defect: fixed)

Opened 3 months ago

Last modified 3 months ago

flash uploader didn't show in IE7, showed script errors

Reported by: tsimmons Assigned to: anonymous
Priority: high Milestone: 2.5.1
Component: Administration Version: 2.5
Severity: critical Keywords: has-patch commit
Cc:

Description

When opening the "Add Media" dialog, the flash uploader was absent and instead replaced with the standard "file" input. The reason was due to two script errors that IE didn't like.

Deleting two extra commas in two files fixes the issue. I'm attaching a proposed patch.

Attachments

fileuploaderfix.patch (1.3 kB) - added by tsimmons on 03/29/08 04:01:38.
diff for patch

Change History

03/29/08 04:01:38 changed by tsimmons

  • attachment fileuploaderfix.patch added.

diff for patch

03/29/08 05:33:32 changed by Viper007Bond

  • keywords set to has-patch commit.
  • priority changed from normal to high.
  • severity changed from normal to blocker.

This is one of those bugs that drives me absolutely nuts.

03/29/08 11:13:52 changed by thee17

I too have tried this, and it works for me in IE7 on XP, also tested doesnt break Safari 3 Mac, FF2 Mac, FF2 XP.

03/29/08 17:49:10 changed by lloydbudd

  • severity changed from blocker to critical.
  • milestone changed from 2.6 to 2.5.1.

03/30/08 06:11:30 changed by markjaquith

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

(In [7572]) Remove errant commas to prevent file uploader script errors in IE7. props tsimmons. fixes #6443 for trunk

03/30/08 06:11:49 changed by markjaquith

(In [7573]) Remove errant commas to prevent file uploader script errors in IE7. props tsimmons. fixes #6443 for 2.5.1