The Wayback Machine - https://web.archive.org/web/20230615103145/https://github.com/doxygen/doxygen/issues/8649
Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

doxygen with php make ugly array parameter output #8649

Closed
paule32 opened this issue Jul 4, 2021 · 5 comments
Closed

doxygen with php make ugly array parameter output #8649

paule32 opened this issue Jul 4, 2021 · 5 comments

Comments

@paule32
Copy link

paule32 commented Jul 4, 2021

Hello,
I tried doxygen 1,9 2021 to document php files.
but i ran into trouble with array parameter.
for avoid cross posting, look here:

https://stackoverflow.com/questions/68241186/doxygen-for-php-dont-handle-default-parameter-array?noredirect=1#comment120609963_68241186

@paule32
Copy link
Author

paule32 commented Jul 4, 2021

new news:
I found a workaround:

public static function rgb2hex(
int $red = 0,
int $green = 0,
int $blue = 0,
array $default = array('r' => 0, 'g' => 0, 'b' => 0) # <-- here: array instead [ ] !
) : array {

it would be nice, when "array( ... )" is support with $arr = [ ];
Oh men, what a day ....

@albert-github
Copy link
Collaborator

The information at stackexchange might not be persistent, so:

  • Can you please attach a, small, self contained example (source+configuration file in a tar or zip) that allows us to reproduce the problem? Please don't add external links as they might not be persistent.
  • Please also specify the full doxygen version used (doxygen -v).

Note: You mention version doxygen 1.9 2021, which version is this ? I only know at the moment, for the 1.9 series, the doxygen versions 1.9.0 and 1.9.1 and further the current master (1.9.2 (7fa3837), with the master the git ID, as reported by doxygen -v, should always be specified).

albert-github added a commit to albert-github/doxygen that referenced this issue Jul 15, 2021
In php square brackets are also possible but were not handled. Added handling for square brackets.
@albert-github
Copy link
Collaborator

I've just pushed a proposed patch, pull request #8673

doxygen added a commit that referenced this issue Jul 15, 2021
issue #8649 doxygen with php make ugly array parameter output
@albert-github
Copy link
Collaborator

Code has been integrated in master on github (please don't close the issue as this will be done at the moment of an official release).

@albert-github albert-github added the fixed but not released Bug is fixed in github, but still needs to make its way to an official release label Jul 16, 2021
@doxygen
Copy link
Owner

doxygen commented Aug 18, 2021

This issue was previously marked 'fixed but not released',
which means it should be fixed in doxygen version 1.9.2.
Please verify if this is indeed the case. Reopen the
issue if you think it is not fixed and please include any additional information
that you think can be relevant (preferably in the form of a self-contained example).

@doxygen doxygen removed the fixed but not released Bug is fixed in github, but still needs to make its way to an official release label Aug 18, 2021
@doxygen doxygen closed this as completed Aug 18, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants