#41223 [NEW]: PHP should output better HTML for the .phps (highlight source) view
From: jeremy dot visser at gmail dot com Operating system: Ubuntu 7.04 PHP version: 5.2.1 PHP Bug Type: Output Control Bug description: PHP should output better HTML for the .phps (highlight source) view Description: When PHP outputs highlighted source, the HTML it outputs is ugly. It looks pretty when viewed in a browser, but the actual HTML is bad. For starters, it doesn't use a DOCTYPE, and includes no , , or elements like you'd expect. Even worse, it preformats the text not with a tag like should be used, but with several 's to space text out. What is wrong with using ? Expected result: Should be using proper HTML headers: http://www.w3.org/TR/xhtml1/DTD/xhtml1-strict.dtd";> http://www.w3.org/1999/xhtml"; xml:lang="en" lang="en"> test.phps <?php /* This is some PHP */ ?> You see, when you use a , you don't need all these 's to space things out. Actual result: -- Example of ugly code: <?php/*Test*/?> -- Edit bug report at http://bugs.php.net/?id=41223&edit=1 -- Try a CVS snapshot (PHP 4.4): http://bugs.php.net/fix.php?id=41223&r=trysnapshot44 Try a CVS snapshot (PHP 5.2): http://bugs.php.net/fix.php?id=41223&r=trysnapshot52 Try a CVS snapshot (PHP 6.0): http://bugs.php.net/fix.php?id=41223&r=trysnapshot60 Fixed in CVS: http://bugs.php.net/fix.php?id=41223&r=fixedcvs Fixed in release: http://bugs.php.net/fix.php?id=41223&r=alreadyfixed Need backtrace: http://bugs.php.net/fix.php?id=41223&r=needtrace Need Reproduce Script:http://bugs.php.net/fix.php?id=41223&r=needscript Try newer version:http://bugs.php.net/fix.php?id=41223&r=oldversion Not developer issue: http://bugs.php.net/fix.php?id=41223&r=support Expected behavior:http://bugs.php.net/fix.php?id=41223&r=notwrong Not enough info: http://bugs.php.net/fix.php?id=41223&r=notenoughinfo Submitted twice: http://bugs.php.net/fix.php?id=41223&r=submittedtwice register_globals: http://bugs.php.net/fix.php?id=41223&r=globals PHP 3 support discontinued: http://bugs.php.net/fix.php?id=41223&r=php3 Daylight Savings: http://bugs.php.net/fix.php?id=41223&r=dst IIS Stability:http://bugs.php.net/fix.php?id=41223&r=isapi Install GNU Sed: http://bugs.php.net/fix.php?id=41223&r=gnused Floating point limitations: http://bugs.php.net/fix.php?id=41223&r=float No Zend Extensions: http://bugs.php.net/fix.php?id=41223&r=nozend MySQL Configuration Error:http://bugs.php.net/fix.php?id=41223&r=mysqlcfg
#41223 [Bgs->Opn]: PHP should output better HTML for the .phps (highlight source) view
ID: 41223 User updated by: jeremy dot visser at gmail dot com Reported By: jeremy dot visser at gmail dot com -Status: Bogus +Status: Open Bug Type: Output Control Operating System: Ubuntu 7.04 PHP Version: 5.2.1 New Comment: OK, I see what you mean about the headers. However, I'm not talking about the highlight_file() or highlight_string() functions. Obviously, they should not return HTML or HTTP headers. The issue is when viewing a .phps file, where PHP does everything itself, without any script having to output the headers for it. When viewing a standalone .phps file, PHP should output the proper headers. Of course this is a bug. PHP is not returning standards-compliant or semantic markup. This has accessibility problems. That part of my report _is_ a bug. I'm reopening the bug, but if you close it again, I will not bother to try and argue. Previous Comments: [2007-04-29 14:39:43] [EMAIL PROTECTED] Thank you for taking the time to write to us, but this is not a bug. Please double-check the documentation available at http://www.php.net/manual/ and the instructions on how to report a bug at http://bugs.php.net/how-to-report.php The reason there are no headers is because the expectation is that the output will be embed in an existing page. Presence of the requested headers and elements would then cause the page to break. [2007-04-29 09:22:35] jeremy dot visser at gmail dot com Description: When PHP outputs highlighted source, the HTML it outputs is ugly. It looks pretty when viewed in a browser, but the actual HTML is bad. For starters, it doesn't use a DOCTYPE, and includes no , , or elements like you'd expect. Even worse, it preformats the text not with a tag like should be used, but with several 's to space text out. What is wrong with using ? Expected result: Should be using proper HTML headers: http://www.w3.org/TR/xhtml1/DTD/xhtml1-strict.dtd";> http://www.w3.org/1999/xhtml"; xml:lang="en" lang="en"> test.phps <?php /* This is some PHP */ ?> You see, when you use a , you don't need all these 's to space things out. Actual result: -- Example of ugly code: <?php/*Test*/?> -- Edit this bug report at http://bugs.php.net/?id=41223&edit=1