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

Bug #786 Incorrect Handling of zero value integer
Submitted: 2004-02-18 15:30 UTC
From: b dot m dot d dot scott_NO_SPAM at herts dot ac dot uk Assigned: mansion
Status: Closed Package: HTML_Table
PHP Version: 4.3.4 OS: Windows XP
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 : 38 - 13 = ?

 
 [2004-02-18 15:30 UTC] b dot m dot d dot scott_NO_SPAM at herts dot ac dot uk
Description: ------------ If you attempt to add an integer with value 0 to a row it is replaced by the autofill value. I think that if you change: if (isset($this->_autoFill) && $contents == "") { $contents = $this->_autoFill; } To: if (isset($this->_autoFill) && $contents === "") { $contents = $this->_autoFill; } in the toHtml function that it will work. Reproduce code: --------------- require_once 'HTML/Table.php'; $table= new HTML_table(); $table->addRow(array(1,0,1)); print $table->toHTML(); Expected result: ---------------- <table> <tr> <td>1</td> <td>0</td> <td>1</td> </tr> </table> Actual result: -------------- <table> <tr> <td>1</td> <td> </td> <td>1</td> </tr> </table>

Comments

 [2004-07-07 16:22 UTC] bmansion at mamasam dot com
This bug has been fixed in CVS. In case this was a documentation problem, the fix will show up at the end of next Sunday (CET) on pear.php.net. In case this was a pear.php.net website problem, the change will show up on the website in short time. Thank you for the report, and for helping us make PEAR better.