Description: ------------ Class tree puts comma uncorrectly in interfaces list. Test script: --------------- <?php /** Test file for MyPackage2 class tree * @package MyPackage */ /** @package MyPackage */ interface MyInterface {} /** @package MyPackage */ interface MyInterface2 {} /** @package MyPackage */ interface MyInterface3 {} /** @package MyPackage */ class MyClass implements MyInterface, MyInterface2, MyInterface3 {} /** @package MyPackage */ class MyClass2 extends MyClass {} ?> Expected result: ---------------- Root class MyClass MyClass (implements MyInterface, MyInterface2, MyInterface3) MyClass2 Actual result: -------------- Root class MyClass MyClass (implements MyInterface, MyInterface2MyInterface3) MyClass2

Added patch classtree-fix.patch.

This patch works only for HTML:frames converter (you can fix another manually - just replace original getRootTree method). Also this patch fixes: http://pear.php.net/bugs/bug.php?id=18870 http://pear.php.net/bugs/bug.php?id=18871

Tried to verify this behavior using v1.4.3 against PHP 5.3.8 and 5.2.17, but tests are being affected by interface names failing to appear in the place where the commas are expected. Presumably this is due to some combination of bug 18870 and bug 18871, and the bug reporter likely discovered this bug when patching his code for the other two bugs. I'll have to get the other two bugs patched and tested before verifying this bug behavior, I think. As an aside, the issue is irrelevant when using PHP 4.4.9.

Nope, this tested with clear SVN trunk head revision (changed only "phpdoc" & "phpdoc.bat"). But only with HTML:frames:default converter. Review this with 318072 revision - the same result. /* https://svn.php.net/repository/pear/packages/PhpDocumentor/trunk/phpDocumentor/Converters/HTML/frames/HTMLframesConverter.inc:318062 lines:1361-1368 */ foreach ($implements as $i => $interface) { if ($i && $i != count($implements) - 1) $my_tree .= ', '; if ($link = $this->getLink('object ' . $interface)) { $my_tree .= $this->returnSee($link); } else { $my_tree .= $interface; } } // comma puts condition: if ($i && $i != count($implements) - 1) $my_tree .= ', '; So $i should be greater than 0 and not the last. And this is mistake because comma puts BEFORE interface. To fix this bug (not tested) just remove second condition. // comma puts fixed condition: if ($i > 0) $my_tree .= ', ';

Or maybe we had $implements arrays with different keys?

Followed the concept of the provided patch to pull interface handling outside of the child class handling logic. Tests good on PHP 5.3.8, 5.2.17, and 4.4.9 to solve the interface listing issues in the class list view. Committed to SVN.