Description: ------------ There is an error on the table handling code that makes it impossible to generate a correct table with MediaWiki-syntax. You will either get a missing row, or a "|-" inside the table, but outside any cell. The simple patch below atleast allows me to render a nice table (although it is probably the regexp "$regexRows" that is really the problem. $ diff -u Table.php.orig Table.php --- Table.php.orig 2008-07-14 16:47:41.000000000 +0200 +++ Table.php 2008-07-14 17:28:50.000000000 +0200 @@ -158,6 +158,7 @@ array(&$this, 'processRows'), $expsub ); + $sub = trim(preg_replace("/\|-$/", "", $sub)); $param = array( 'type' => 'table_start', 'level' => $this->_level, Test script: --------------- $table = "{| |- | This is a cell || And another |- | And a new row || And yet another cell |- |}"; require("Text/Wiki/Mediawiki.php"); $wiki = new Text_Wiki_Mediawiki(); echo( $wiki->transform( $table, 'Xhtml' )); Expected result: ---------------- A table with two rows, each with two cells should be rendered. Actual result: -------------- A "|-" is added inside the table, but outside any cell, making it appear before the table content. If I try to remove the last "|-" from the code, the whole last line is written in the same way.

Has there been any activity on this bug? I have experienced the exact same issue, but unfortunately, Ola's patch did not work for me. Tables worked correctly until I upgraded from PHP 5.1 to 5.2.

i had the same issue and used this line instead to fix $sub = trim(preg_replace("/(\|-[^<]+)/", '', $sub)); where i search for a "|-" and remove anything after it, up to the next opening less-than

I have experienced similar problem. Here is the patch that worked for me. I was not able to attach the patch file itself via "Add a patch" form, so here it is embedded: Index: W:/home/localhost/www/oradyvan/sites/all/modules/pearwiki_filter/Text/Wiki/Parse/Mediawiki/Table.php =================================================================== --- W:/home/localhost/www/oradyvan/sites/all/modules/pearwiki_filter/Text/Wiki/Parse/Mediawiki/Table.php (revision 180) +++ W:/home/localhost/www/oradyvan/sites/all/modules/pearwiki_filter/Text/Wiki/Parse/Mediawiki/Table.php (revision 181) @@ -153,11 +153,16 @@ } $this->_countRows[$this->_level] = $this->_maxCells[$this->_level] = 0; $this->_countCells[$this->_level] = $this->_spanCells[$this->_level] = array(); + // fixing issue with \z PCRE assertion not working if the string does not end with newline + // by adding one more empty row to the end + $expsub = rtrim($expsub, "\n|-") . "\n|-"; $sub = preg_replace_callback( $this->regexRows, array(&$this, 'processRows'), $expsub ); + // and now removing that empty row from the end + $sub = rtrim($sub, "\n|-"); $param = array( 'type' => 'table_start', 'level' => $this->_level,

The following patch has been added/updated: Patch Name: fix-table-displaying Revision: 1239473426 URL: http://pear.php.net/bugs/patch-display.php?bug=14346&patch=fix-table-displaying&revision=1239473426&display=1

I have the same issue with php 5.2.9. oradyvan's fix has worked flawlessly, while the others didn't on more complex tables. I attached a patch.

I actually get the same result no matter if I apply any of the changes or not with the current CVS version - Could someone please verify if this is bound to this specific version?

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. Since there was no Feedback to my question I assume this bug does only exist in 0.1.0 and is fixed in CVS....