php.net |  support |  documentation |  report a bug |  advanced search |  search howto |  statistics |  random bug |  login
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 Developer Edit
Welcome! If you don't have a Git account, you can't do anything here.
If you reported this bug, you can edit this bug over here.
(description)
Block user comment
Status: Assign to:
Package:
Bug Type:
Summary:
From: pascal dot chevrel at free dot fr
New email:
PHP Version: OS:

 

 [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)

Pull Requests

 
PHP Copyright © 2001-2024 The PHP Group
All rights reserved.
Last updated: Thu Nov 21 13:01:29 2024 UTC