Package home | Report new bug | New search | Development Roadmap Status: Open | Feedback | All | Closed Since Version 0.1.6

Bug #12838 XHTML compliance -- onClick handler must be all lowercase
Submitted: 2008-01-06 15:12 UTC
From: denny Assigned: wiesemann
Status: Closed Package: Structures_DataGrid_Renderer_HTMLTable (version 0.1.4)
PHP Version: Irrelevant OS: N/A
Roadmaps: (Not assigned)    
Subscription  
Comments Add Comment Add patch


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 !
Your email address:
MUST BE VALID
Solve the problem : 20 + 43 = ?

 
 [2008-01-06 15:12 UTC] denny (Denny Shimkoski)
Description: ------------ Setting the onMove option produces invalid XHTML. From the XHTML spec (http://www.w3.org/TR/xhtml1/) -- 4.2. Element and attribute names must be in lower case XHTML documents must use lower case for all HTML element and attribute names. This difference is necessary because XML is case-sensitive e.g. <li> and <LI> are different tags. Test script: --------------- require_once 'Structures/DataGrid.php'; $dg =& new Structures_DataGrid(); $dg->setRendererOption('onMove', 'updateGrid', true); $dg->bind(array(array('testval' => 123))); $dg->render(); Expected result: ---------------- <table> <thead> <tr> <th><a href="dgbug.php?orderBy=testval&direction=ASC&page=1" onclick="return updateGrid({ page: 1, sort: [{field: 'testval', direction:'ASC'}], data: '' });">testval</a></th> </tr> </thead> <tbody> <tr> <td align="right">123</td> </tr> </tbody> </table> Actual result: -------------- <table> <thead> <tr> <th><a href="dgbug.php?orderBy=testval&direction=ASC&page=1" onClick="return updateGrid({ page: 1, sort: [{field: 'testval', direction:'ASC'}], data: '' });">testval</a></th> </tr> </thead> <tbody> <tr> <td align="right">123</td> </tr> </tbody> </table>

Comments

 [2008-01-16 17:08 UTC] wiesemann (Mark Wiesemann)
Thank you for your bug report. This issue has been fixed in the latest released version of the package, which you can download at http://pear.php.net/get/Structures_DataGrid_Renderer_HTMLTable