#26848 [WFx]: Make highlight functions XHTML 1.0 Strict compliant
ID: 26848 User updated by: phpman at fosketts dot co dot uk Reported By: phpman at fosketts dot co dot uk Status: Wont fix Bug Type: Feature/Change Request Operating System: Win XP PHP Version: 4.3.4 New Comment: Surely standards compliance matters now. Any developer wishing to use these functions within a XHTML 1.0 Strict document will have to perform extra steps, even though a 'fix' could very easily be applied I guess I'll have to keep doing this: $str = preg_replace ( '#\#is', '', $str ); $str = str_replace ( '', '', $str ); Sean :) Previous Comments: [2004-01-10 10:21:01] [EMAIL PROTECTED] We'll do so when standard compliance really matters. (IIRC, the current output is valid XHTML 1.0 transitional.) -------------------- [2004-01-08 14:49:15] phpman at fosketts dot co dot uk Description: Will the highlight functions (highlight_string() and highlight_file()) be modified to produce valid XHTML 1.0 Strict output? This has already been done in other functions, e.g. nl2br(). A simple solution would be to edit zend_highlight.c as follows: 1) Replace with on lines 106 and 155 2) Replace with on lines 151, 191 and 193 Thanks Sean :) -- Edit this bug report at http://bugs.php.net/?id=26848&edit=1
#26848 [NEW]: Make highlight functions XHTML 1.0 Strict compliant
From: phpman at fosketts dot co dot uk Operating system: Win XP PHP version: 4.3.4 PHP Bug Type: Feature/Change Request Bug description: Make highlight functions XHTML 1.0 Strict compliant Description: Will the highlight functions (highlight_string() and highlight_file()) be modified to produce valid XHTML 1.0 Strict output? This has already been done in other functions, e.g. nl2br(). A simple solution would be to edit zend_highlight.c as follows: 1) Replace with on lines 106 and 155 2) Replace with on lines 151, 191 and 193 Thanks Sean :) -- Edit bug report at http://bugs.php.net/?id=26848&edit=1 -- Try a CVS snapshot (php4): http://bugs.php.net/fix.php?id=26848&r=trysnapshot4 Try a CVS snapshot (php5): http://bugs.php.net/fix.php?id=26848&r=trysnapshot5 Fixed in CVS: http://bugs.php.net/fix.php?id=26848&r=fixedcvs Fixed in release: http://bugs.php.net/fix.php?id=26848&r=alreadyfixed Need backtrace: http://bugs.php.net/fix.php?id=26848&r=needtrace Need Reproduce Script: http://bugs.php.net/fix.php?id=26848&r=needscript Try newer version: http://bugs.php.net/fix.php?id=26848&r=oldversion Not developer issue:http://bugs.php.net/fix.php?id=26848&r=support Expected behavior: http://bugs.php.net/fix.php?id=26848&r=notwrong Not enough info:http://bugs.php.net/fix.php?id=26848&r=notenoughinfo Submitted twice:http://bugs.php.net/fix.php?id=26848&r=submittedtwice register_globals: http://bugs.php.net/fix.php?id=26848&r=globals PHP 3 support discontinued: http://bugs.php.net/fix.php?id=26848&r=php3 Daylight Savings: http://bugs.php.net/fix.php?id=26848&r=dst IIS Stability: http://bugs.php.net/fix.php?id=26848&r=isapi Install GNU Sed:http://bugs.php.net/fix.php?id=26848&r=gnused Floating point limitations: http://bugs.php.net/fix.php?id=26848&r=float