Closed (fixed)
Project:
Drupal core
Version:
8.0.x-dev
Component:
documentation
Priority:
Normal
Category:
Bug report
Assigned:
Issue tags:
Reporter:
Created:
5 Nov 2015 at 08:04 UTC
Updated:
9 Dec 2015 at 14:54 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #2
rakesh.gectcrComment #3
dawehnerGood catch
Comment #4
rakesh.gectcrComment #5
catchThanks for the patch. In order clean up these sorts of coding standards fairly and efficiently, many instances of identical fixes that can be reviewed with
git diff --color-wordsshould be combined into a single patch, ideally one that fixes all instances in core in preparation for enabling automated coding standards checking for them. This issue is postponed for now on #2571965: [meta] Fix PHP coding standards in core, stage 1.If the standard being fixed already has a coder rule, this issue should me marked as a duplicate of the issue that begins to fix that rule. If there is not yet a coder rule, we should add one. See for example #2606860: Docblock sections out of order in core/lib/Drupal.
Thanks for your ongoing efforts to clean up the codebase and fix these bugs!
Comment #8
catch@xjm pointed out this might not be catchable by coder, so..
Committed/pushed to 8.1.x and cherry-picked to 8.0.x. Thanks!