Closed (fixed)
Project:
Pathauto
Version:
7.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Reporter:
Created:
2 Mar 2011 at 01:19 UTC
Updated:
11 Nov 2011 at 19:21 UTC
Recent commits changed all the PHP files from:
<-- blank line here
/**
* @file
* Pathauto integration for core modules.
*
* @ingroup pathauto
*/
// Rest of file...
to
/**
* @file
* Pathauto integration for core modules.
*
* @ingroup pathauto
*/
// Rest of file...
I find this change unnecessary and sticks out like a sore thumb when first opening a file. We don't have a specific coding standard regarding this specific detail (blank lines after opening PHP tag), but I always think it is best to follow the example set by core, which always has blank line after it's opening PHP tag and the first line of code/docs. I find it is more readable this way too. I'd like to revert this change.
Comments
Comment #1
gregglesAgreed.
Comment #2
dave reidCommitted restoration of the blank lines in PHP files only:
http://drupalcode.org/project/pathauto.git/commit/adf5011
http://drupalcode.org/project/pathauto.git/commit/0642e13
http://drupalcode.org/project/pathauto.git/commit/27cfef7
Comment #3
gregglesThanks, Dave.
Comment #5
neclimdulActually, the change was technically correct. The space was sort of a bug in the way the git migration converted files and is a false standard generally kept around. Its not actually a coding standard documented anywhere.
Comment #6
dave reidIt is the standard set by Drupal core PHP files. This does not need to be discussed further.
Comment #7
neclimdulstarted a core discussion of the issue #1339022: Standardize yes/no on empty line before the @file docblock