WordPress.org

Make WordPress Core

Opened 3 years ago

Closed 3 years ago

Last modified 3 years ago

#15639 closed defect (bug) (fixed)

Message: 'isRtl' is undefined

Reported by: emiluzelac Owned by:
Milestone: 3.1 Priority: normal
Severity: normal Version: 3.1
Component: Warnings/Notices Keywords: has-patch
Focuses: Cc:

Description

Hi,

3.1-beta1-16590 and in
http://localhost/wp-admin/post-new.php I get this error in IE8:

Webpage error details

User Agent: Mozilla/4.0 (compatible; MSIE 8.0; Windows NT 6.1; Trident/4.0; GTB6.6; SLCC2; .NET CLR 2.0.50727; .NET CLR 3.5.30729; .NET CLR 3.0.30729; Media Center PC 6.0; .NET4.0C; Tablet PC 2.0)
Timestamp: Thu, 2 Dec 2010 02:36:51 UTC

Message: 'isRtl' is undefined
Line: 2
Char: 3843
Code: 0
URI: http://localhost/wp-admin/load-scripts.php?c=1&load=hoverIntent,common,jquery-color,imgareaselect,image-edit,set-post-thumbnail&ver=8557d196868028cc389265611c660a9a

Attachments (1)

rtl-for-iframe.15636.diff (967 bytes) - added by filosofo 3 years ago.

Download all attachments as: .zip

Change History (8)

comment:1 nacin3 years ago

  • Keywords reporter-feedback added
  • Milestone changed from Awaiting Review to 3.1

Not a clue why this is happening. isRtl is defined before we print any scripts.

Do you still see this with SCRIPT_DEBUG on?

comment:2 emiluzelac3 years ago

While the debug was on the IE8 crashed as well. This happens only when new post is open and you click on Upload/Insert i.e. image.

Breaking on JScript runtime - 'isRtl' is undefined

a=(isRtl?"left":"right")

comment:3 nacin3 years ago

Any plugins?

That's from common.dev.js, line 205: bgx = ( isRtl ? 'left' : 'right' ).

It's within a jQuery(document).ready, so I really don't have a clue.

comment:4 filosofo3 years ago

  • Keywords has-patch added; reporter-feedback removed

Try rtl-for-iframe.15636.diff.

I can reproduce, as the iframe used for media uploads does not get the isRtl definition in its header, unlike other admin pages.

comment:5 nacin3 years ago

Ah ha, missed that nugget in the follow-up comment.

comment:6 nacin3 years ago

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

(In [16687]) define isRtl in the media upload iframe. props filosofo, fixes #15639.

comment:7 emiluzelac3 years ago

Thank you nacin and filosofo, also please forgive me for not responding earlier to Nacin's question.

Note: See TracTickets for help on using tickets.