Skip to content
This repository has been archived by the owner on Nov 11, 2020. It is now read-only.

Corrected fluent interface docblocks #251

Merged
merged 1 commit into from
Feb 29, 2016
Merged

Conversation

Bilge
Copy link
Contributor

@Bilge Bilge commented Feb 29, 2016

self means the base class will always be returned. This is not only false, but breaks code completion in some editors when attempting to access derived class methods. For more information see phpDocumentor types documentation.

`self` means the base class will always be returned. This is not only false, but breaks code completion in some editors when attempting to access derived class methods. For more information see [phpDocumentor types documentation](https://www.phpdoc.org/docs/latest/guides/types.html#keywords).
@malarzm malarzm added the task label Feb 29, 2016
@malarzm malarzm added this to the 1.3.0 milestone Feb 29, 2016
@malarzm
Copy link
Member

malarzm commented Feb 29, 2016

Keeping it open for a moment so we remember to change other places, thanks @Bilge for getting started!

@alcaeus alcaeus self-assigned this Feb 29, 2016
@alcaeus alcaeus merged commit 9848827 into doctrine:master Feb 29, 2016
@alcaeus
Copy link
Member

alcaeus commented Feb 29, 2016

Thanks for getting us started on this, I replaced the remaining occurrences of @return self with @return $this.

@Bilge
Copy link
Contributor Author

Bilge commented Feb 29, 2016

@alcaeus Oh, really? I was just in the middle of doing the same. Did you do it for ODM as well or shall I?

@Bilge Bilge deleted the patch-1 branch February 29, 2016 20:23
@alcaeus
Copy link
Member

alcaeus commented Feb 29, 2016

Apparently, I missed the announcement that you were doing the remaining changes as well - I apologize for that. Go ahead and finish up the ODM pull request. 👍

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants