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
Have you experienced this issue?
Rate the importance of this bug to you:

 [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: Tue Mar 19 14:01:30 2024 UTC