Closed (fixed)
Project:
Drupal core
Version:
8.0.x-dev
Component:
documentation
Priority:
Minor
Category:
Task
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
14 Apr 2015 at 15:13 UTC
Updated:
1 May 2015 at 13:14 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
meramo commentedSending patch for review
Comment #2
Reno Greenleaf commentedReviewed. Looks good.
Comment #3
googletorp commentedAlso reviewed, think this is RTBC.
Comment #4
alexpottThis returns
$thisAfaics this returns the file path.
Comment #5
meramo commentedComments amended. There are still a lot of inconsistencies in the documentation, but it least we'll have the returns documented everywhere.
Comment #6
xanoIncorrect description. Also, when using
@return $this, the description may be left out completely.Comment #7
googletorp commentedRerolled the patch with proper documentation.
Comment #8
googletorp commentedAdd missing \ in previous patch
Comment #9
googletorp commentedComment #10
xanoThis must be
@return $this.Comment #11
googletorp commentedWith $this it is :)
Comment #12
xanoNo description is needed, because
@return $thiscan only mean one thing (the object on which the method was called). See the handbook for more information (and we copied it from PhpDoc again).Comment #13
googletorp commented@Xano Thanks for clearing this up and the review, I wasn't aware of the special @return $this.
Updated in the patch, removed the description after @return $this.
Comment #14
xanoLooks good. Thanks!
Comment #15
meramo commentedI wasn't aware of the $this trick as well, thanks everyone for the help!
Looks good!
Comment #16
webchickI think that's right, but sending to jhodgdon to confirm.
Comment #17
jhodgdonI can confirm that the formatting of the documentation added lines is correct. I have not reviewed them for accuracy, but others have done that. +1.
Comment #19
xjmThanks @jhodgdon for confirming. This issue only improves documentation, so per https://www.drupal.org/core/beta-changes, this can be completed any time during the Drupal 8 beta phase.
I checked over both methods' return values and the documentation appears accurate. (Though, yowza at
CssOptimizer::loadNestedFile()generally, but improving the inline docs isn't in scope here.)Committed and pushed to 8.0.x. Thanks everyone!