php.net |  support |  documentation |  report a bug |  advanced search |  search howto |  statistics |  random bug |  login

go to bug id or search bugs for

Request #63263 phpinfo() function generates old html/css
Submitted: 2012-10-11 14:17 UTC Modified: -
Votes:3
Avg. Score:3.0 ± 1.6
Reproduced:2 of 2 (100.0%)
Same Version:0 (0.0%)
Same OS:1 (50.0%)
From: pascal dot chevrel at free dot fr Assigned:
Status: Open Package: PHP options/info functions
PHP Version: 5.3Git-2012-10-11 (Git) OS: all
Private report: No CVE-ID: None
View Add Comment Developer Edit
Anyone can comment on a bug. Have a simpler test case? Does it work for you on a different platform? Let us know!
Just going to say 'Me too!'? Don't clutter the database with that please — but make sure to vote on the bug!
Your email address:
MUST BE VALID
Solve the problem:
43 - 28 = ?
Subscribe to this entry?

 
 [2012-10-11 14:17 UTC] pascal dot chevrel at free dot fr 
Description:
------------
The phpinfo() function does not use recent html and abuses classes attributes and css for semantics (like using <td class="h"> for a cell header instead of <th>

I am attaching a patch that simplifies the html generated, makes it an html5 compliant (and hence simplifies the code used to generate this html) while keeping the same layout and visual appearance. The html goes from 64KB to 40KB just by removing redundant html.

There is one change to php_info_print_table_row_internal() to remove a variable, that function is not used anywhere else in the code base. I didn't get any test failure with this patch.

Thanks

Patches

refactorphpinfo (last revision 2012-10-11 14:18 UTC by pascal dot chevrel at free dot fr)

Add a Patch

Pull Requests

Add a Pull Request
 
PHP Copyright © 2001-2024 The PHP Group
All rights reserved. 		Last updated: Wed Apr 24 15:01:30 2024 UTC