Remove ()'s when rendering methodname in constructorsynopsis/destructorsynopsis (original) (raw)

| Bug #66644 | Remove ()'s when rendering methodname in constructorsynopsis/destructorsynopsis | | | | | ------------------------------ | ------------------------------------------------------------------------------- | --------- | -------------------------------------------------------------------------------------------------------------------- | | | Submitted: | 2014-02-04 21:15 UTC | Modified: | 2015-04-16 18:58 UTC | | | From: | salathe@php.net | Assigned: | salathe (profile) | | | Status: | Closed | Package: | Doc Build problem | | | PHP Version: | Irrelevant | OS: | | | | Private report: | No | CVE-ID: | None | |

[2014-02-04 21:15 UTC] salathe@php.net

Description:

Bug #51750 (Add ()'s when rendering methodname elements) was fixed back in 2010 (thanks bjori!).

The fix involved adding parentheses after methodname, when they occur outside of methodsysnopsis. Unfortunately, this applies the parentheses inside constructorsynopsis/destructorsynopsis elements, resulting in exactly the "silly" output mentioned in that bug.

Expected result:

Foo::__construct (string $string)

Actual result:

Foo::__construct() (string $string)

Patches

Pull Requests

History

AllCommentsChangesGit/SVN commitsRelated reports

[2014-02-04 21:21 UTC] salathe@php.net

-Summary: Remove ()'s when rending methodname in constructorsynopsis/destrutorsynopsis +Summary: Remove ()'s when rendering methodname in constructorsynopsis/destructorsynopsis

[2015-04-16 18:37 UTC] cmb@php.net

-Assigned To: +Assigned To: salathe

[2015-04-16 18:37 UTC] cmb@php.net

This issue appears to have been fixed in the meantime. Can you confirm, Peter?

[2015-04-16 18:58 UTC] salathe@php.net

Whoops. This was fixed the same day I reported it. Closing!