Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
I've clicked the "More information" link in admin/reports/status and the phpinfo() did not open in the overlay. Instead it closed the overlay and showed the phpinfo() in a normal display. Clicking « Back in my browser once, made the overlay background color (the darkish black background color of the overlay) appear, but no overlay appeared. Clicking back twice made me come back to admin/reports/status again, so that works.
Comment | File | Size | Author |
---|---|---|---|
#1 | overlay-phpinfo-941288-1.patch | 678 bytes | David_Rothstein |
woopsy.jpg | 111.52 KB | JayNL | |
step2.jpg | 172.98 KB | JayNL | |
step1.jpg | 40.5 KB | JayNL |
Comments
Comment #1
David_Rothstein CreditAttribution: David_Rothstein commentedThis should fix it, although it's a little bit ugly. I guess this page is really a special case though.
Comment #2
JayNL CreditAttribution: JayNL commentedWith a hardcoded URI it is indeed a bit ugly. It does work though, but there must be a better solution for it. Is the
phpinfo()
page the only page that jumps out the overlay? I'll investigate. Thanks for your feedback and actions so far.Comment #3
bleen CreditAttribution: bleen commentedThis may be just as ugly (if not, uglier) ... but we could create a page with the phpinfo page in an iframe and show that in the overlay.
Comment #4
JayNL CreditAttribution: JayNL commentedI've been clicking -every- link I could find in whole DP7.x.dev version, but none jumped out the overlay, so yes... I guess the phpinfo() is special case for now... We cannot look into the future though, so perhaps a nice and clean fix would be appropriate.
Comment #5
mo6I don't think it's ugly to have a special case for the phpinfo() page, especially as this is an exception because the content is generated outside of Drupal, by php itself. So I'd opt for the #1 solution.
Patch tested, is working as advertised.
Comment #6
webchickAgreed; I can't think of a better way either. The comment makes it clear why this was done.
Committed to HEAD. Thanks!