Skip to content

Conversation

@fprochazka
Copy link
Contributor

Without this, the aucompletion doesn't work.

@fprochazka fprochazka mentioned this pull request Jun 8, 2014
Closed
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

But that's still not true, if someone doesn't use nette/reflection in his project (and it is not a dependency of nette/utils). :/

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

But it's consistent with @property-read

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

So maybe fix property-read as well. :)

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't wanna.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You do, this will not make code completion work properly.

@dg
Copy link
Member

dg commented Jun 8, 2014

This is the only one case in framework?

@fprochazka
Copy link
Contributor Author

@dg Not sure, I wasn't searching for more broken typehints.

@dg
Copy link
Member

dg commented Jun 9, 2014

@fprochazka but have one issue for each wrong typehint is the same evil as issues for simple typos in phpdoc…

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants