WordPress.org
Get WordPress

Make WordPress Core

Opened 16 years ago

Closed 16 years ago

#15509 closed defect (bug) (fixed)

Don't show the administration bar in plugin/theme install iframe

Reported by: ocean90 Owned by:
Priority: normal Milestone: 3.1
Component: Administration Version: 3.1
Severity: normal Keywords: has-patch
Cc: Focuses:

Description

iframe_header() contains show_admin_bar( false ) and wp-admin/plugin-install.php

if ( isset( $_GET['tab'] ) && ( 'plugin-information' == $_GET['tab'] ) )
	define('IFRAME_REQUEST' , true);

So, normally there shouldn't be one.
Screenshot: ​http://grab.by/7uls | ​http://grab.by/7up7

Attachments (2)

15509.patch​ (442 bytes ) - added by Utkarsh 16 years ago.
15509-network-admin.patch​ (2.1 KB ) - added by SergeyBiryukov 16 years ago.

Download all attachments as: .zip

Change History (10)

@Utkarsh
16 years ago

#1 @Utkarsh
16 years ago

Patch attached.

#2 @voyagerfan5761
16 years ago

  • Cc WordPress@… added

#3 @Utkarsh
16 years ago

  • Cc admin@… added
  • Keywords has-patch added; needs-patch removed

#4 @filosofo
16 years ago

  • Keywords reporter-feedback added

Does this error still occur? I cannot reproduce it on current trunk.

#5 @Utkarsh
16 years ago

  • Keywords reporter-feedback removed

Still present on a trunk MS install for me.

#6 @ocean90
16 years ago

Patch from Utkarsh is needed for the theme iframe. Plugin iframe works now.

#7 @ryan
16 years ago

This doesn't work for plugin-install.php and theme-install.php in the network admin because they load admin.php before the IFRAME_REQUEST constant is defined.

@SergeyBiryukov
16 years ago

#8 @PeteMall
16 years ago

  • Resolution → fixed
  • Status new → closed

(In [16658]) Set IFRAME_REQUEST for theme/plugin information iframe. Props SergeyBiryukov, fixes #15509

Note: See TracTickets for help on using tickets.