In features.exsport.inc where the default code for the .module file is generated, the include_once is with parenthesis. But include_once dont require the parenthesis since it's a statement not a function.

So i have created a little patch, which changes this.

CommentFileSizeAuthor
#1 include_once_parenthesis_issue-1414304-1.patch1.58 KBramlev

Comments

ramlev’s picture

ramlev commented
StatusFileSize
new include_once_parenthesis_issue-1414304-1.patch1.58 KB

Heres the patch

hefox’s picture

hefox commented
Category: bug » feature
Priority: Normal » Minor

While it looks like core is standard around include_once, it doesn't suggest one is better than other, so what's the bug about this? http://drupal.org/coding-standards#includes

I have no big opinions on this, other than wanting it's issue category correct, and that changing how it looks may complicate the patches around the bugs of it being missing sometimes.

hefox’s picture

hefox commented
Status: Active » Closed (duplicate)

Looks like a reason commit changed this