Description: ------------ If you add an array without all column numbers as a new row, addRow don't work as it should be. //This code: $row = array(1 => "Cell1", 2 => "Cell2", 5 => "Cell5"); $table->addRow($row); //only insert cell 1 and 2. Test script: --------------- //Lines 349-355 (addRow method): for ($counter = 0; $counter < count($contents); $counter++) { if ($type == 'TD') { $this->setCellContents($row, $counter, $contents[$counter]); } elseif ($type == 'TH') { $this->setHeaderContents($row, $counter, $contents[$counter]); } } //should be: foreach ($contents as $col => $content) { if ($type == 'TD') { $this->setCellContents($row, $col, $content); } elseif ($type == 'TH') { $this->setHeaderContents($row, $col, $content); } }

I can reproduce this problem with current CVS version and have tested the suggested solution - it works as expected. Here is a patch for Table/Storage.php: www.markwiesemann.de/php/HTML_Table/Bug_4944.patch I can commit it myself, if you want.

This bug has been fixed in CVS. If this was a documentation problem, the fix will appear on pear.php.net by the end of next Sunday (CET). If this was a problem with the pear.php.net website, the change should be live shortly. Otherwise, the fix will appear in the package's next release. Thank you for the report and for helping us make PEAR better.