[#8911] - Fixed the docbloc to match the code
- Closed
- 16 Jan 2016
- Medium
- Build: staging
- # 8911
- Diff
- roland-d:jlayout-docbloc
- Success Human Test Results: 2 Successful 0 Failed. Details
User tests: Successful: Unsuccessful:
The docblocks were inconsistent nor following the code. This pull request fixes the docblocks to match the code.
There are no testing instructions as only the docblocks have been changed.
| Status | New | ⇒ | Pending |
| Labels |
Added:
?
|
||
That's not grammatically correct. Better would be "Array of properties available for use inside the layout file to build the displayed output".
Agree with Chris. Its as important that Doc blocks are easy to understand
as it is that the code is correct.
On 16 Jan 2016 9:48 am, "Chris Davenport" notifications@github.com wrote:
That's not grammatically correct. Better would be "Array of properties
available for use inside the layout file to build the displayed output".—
Reply to this email directly or view it on GitHub
#8911 (comment).
This PR has received new commits.
CC: @infograf768
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/8911.
Thanks, replaced the description with the suggestion by @chrisdavenport
I have tested this item 
successfully on b8bc176
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/8911.
| Category | ⇒ | Code style Documentation |
I have tested this item successfully on b8bc176
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/8911.
| Status | Pending | ⇒ | Ready to Commit |
RTC. Thanks.
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/8911.
| Labels |
Added:
?
|
||
wilsonge
- | Status | Ready to Commit | ⇒ | Closed |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2016-01-16 23:06:13 |
| Closed_By | ⇒ | wilsonge |
| Labels |
Removed:
?
|
||
| Milestone |
Added: |
||
I have tested this item
successfully on cdf9b06
Can be merged on review
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/8911.