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

Zephir stubs allow multiline @param #1968

Closed
Jurigag opened this issue Oct 13, 2019 · 1 comment
Closed

Zephir stubs allow multiline @param #1968

Jurigag opened this issue Oct 13, 2019 · 1 comment
Assignees
Labels
enhancement Hacktoberfest See https://hacktoberfest.digitalocean.com/ for participants details

Comments

@Jurigag
Copy link
Contributor

Jurigag commented Oct 13, 2019

When the parameter is provided like this:

     * @param array options = [
     *     'prefix' => 'phalcon'
     *     'lifetime' => 3600
     * ]

Which is totally correct phpdoc, it helps for plugin https://github.com/klesun/deep-assoc-completion to autocomplete code.

It's not detected correctly as parameter, it just add those lines as they are and additionally adds another @param from method parameters. I guess current regexp needs somehow to change https://github.com/phalcon/zephir/blob/development/Library/Stubs/MethodDocBlock.php#L110 here but im not regexp expert.

@ruudboon ruudboon added the Hacktoberfest See https://hacktoberfest.digitalocean.com/ for participants details label Oct 13, 2019
sergeyklay added a commit that referenced this issue Oct 17, 2019
…lock-params

Enchacement | #1968 multiline docblock params
@sergeyklay
Copy link
Contributor

Closing, this is implemented. If there are bugs, we can follow up in separate issues.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Hacktoberfest See https://hacktoberfest.digitalocean.com/ for participants details
Projects
None yet
Development

No branches or pull requests

4 participants