Feeling Lucky
?
[#642] - CodeSniffer - Each PHP statement must be ona line by itself
- Closed
- 8 Jan 2013
- Medium
- Build: master
- # 642
I get some messages when use in the same line more than 1 php code.
To solve that I add in ruleset.xml this code:
< rule ref="Generic.Formatting.DisallowMultipleStatements.SameLine">
< severity>0
< /rule>
I use PHP_CodeSniffer version 1.4.3.
I could make a request to update the file ruleset.xml, or it would not be necessary?
javigomez
- comment
- 8 Jan 2013
Hey Bruno, thanks for your input!!! ^_^
Just mention that the code sniffers are in this specific repository: https://github.com/joomla/coding-standards/tree/master/Sniffs
If you do a pull request I guess that is better to do it there first.
I recommend you to share this issue in the Platform mail list: https://groups.google.com/forum/?fromgroups#!forum/joomla-dev-platform
zero-24
-
brunobatista
- comment
- 8 Jan 2013
Thanks @javigomez!
| Labels |
Added:
?
|
||
| Build | ⇒ | staging | |
I get another messager: Closing brace must be on a line by itself
I believe that "exclude-pattern" is not working ...
< rule ref="Squiz.WhiteSpace.ScopeClosingBrace">
<!-- These exceptions are temporary -->
< exclude-pattern>/administrator/components/
< exclude-pattern>/components/
< exclude-pattern>/plugins/
< exclude-pattern>/modules/
< exclude-pattern>/templates/
< exclude-pattern>/layouts/
< /rule>
I had to comment out the code to remove this message.