Skip to content

Commit

Permalink
Expand existing sniffs to more folders.
Browse files Browse the repository at this point in the history
  • Loading branch information
realityking committed Sep 13, 2012
1 parent 0be531b commit a357942
Showing 1 changed file with 19 additions and 19 deletions.
38 changes: 19 additions & 19 deletions build/phpcs/Joomla/ruleset.xml
Expand Up @@ -66,24 +66,20 @@

<rule ref="Generic.Functions.FunctionCallArgumentSpacing"/>
<rule ref="PEAR.Functions.ValidDefaultValue">
<!-- We only want this for cli, language, includes and joomla-platform for now -->
<exclude-pattern>/administrator/*</exclude-pattern>
<!-- These exceptions are temporary for now -->
<exclude-pattern>/administrator/components/*</exclude-pattern>
<exclude-pattern>/components/*</exclude-pattern>
<exclude-pattern>/plugins/*</exclude-pattern>
<exclude-pattern>/modules/*</exclude-pattern>
<exclude-pattern>/installation/*</exclude-pattern>
<exclude-pattern>/libraries/cms/*</exclude-pattern>
<exclude-pattern>/templates/*</exclude-pattern>
</rule>

<rule ref="PEAR.NamingConventions.ValidClassName">
<!-- We only want this for libraries, cli and includes for now -->
<exclude-pattern>/administrator/*</exclude-pattern>
<!-- These exceptions are temporary -->
<exclude-pattern>/administrator/components/*</exclude-pattern>
<exclude-pattern>/administrator/modules/*</exclude-pattern>
<exclude-pattern>/components/*</exclude-pattern>
<exclude-pattern>/plugins/*</exclude-pattern>
<exclude-pattern>/modules/*</exclude-pattern>
<exclude-pattern>/installation/*</exclude-pattern>
<exclude-pattern>/templates/*</exclude-pattern>
<exclude-pattern>/language/*</exclude-pattern>
</rule>
<rule ref="Generic.NamingConventions.UpperCaseConstantName"/>
Expand All @@ -93,12 +89,9 @@
<rule ref="Generic.PHP.ForbiddenFunctions"/>

<rule ref="Squiz.Operators.IncrementDecrementUsage">
<!-- We only want this for libraries, language, cli and includes for now -->
<exclude-pattern>/administrator/*</exclude-pattern>
<exclude-pattern>/components/*</exclude-pattern>
<!-- These exceptions are temporary -->
<exclude-pattern>/administrator/components/*</exclude-pattern>
<exclude-pattern>/plugins/*</exclude-pattern>
<exclude-pattern>/modules/*</exclude-pattern>
<exclude-pattern>/installation/*</exclude-pattern>
</rule>

<rule ref="Squiz.Scope.StaticThisUsage"/>
Expand Down Expand Up @@ -190,18 +183,22 @@
<exclude-pattern>/plugins/*</exclude-pattern>
<exclude-pattern>/modules/*</exclude-pattern>
<exclude-pattern>/installation/*</exclude-pattern>
<exclude-pattern>/templates/*</exclude-pattern>
<exclude-pattern>/includes/*</exclude-pattern>
<!-- These exceptions are permanent -->
<exclude-pattern>*/tmpl/*</exclude-pattern>
<exclude-pattern>/templates/*</exclude-pattern>
<exclude-pattern>/installation/template/index.php</exclude-pattern>
</rule>

<rule ref="Joomla.ControlStructures.InlineControlStructure">
<!-- We only want this for cli, language and libraries for now -->
<exclude-pattern>/administrator/*</exclude-pattern>
<!-- These exceptions are temporary -->
<exclude-pattern>/administrator/components/*</exclude-pattern>
<exclude-pattern>/components/*</exclude-pattern>
<exclude-pattern>/plugins/*</exclude-pattern>
<exclude-pattern>/modules/*</exclude-pattern>
<exclude-pattern>/installation/*</exclude-pattern>
<!-- These exceptions are permanent -->
<exclude-pattern>*/tmpl/*</exclude-pattern>
<exclude-pattern>/templates/*</exclude-pattern>
<exclude-pattern>/installation/template/index.php</exclude-pattern>
</rule>

<rule ref="Joomla.ControlStructures.MultiLineCondition">
Expand All @@ -211,6 +208,9 @@
<exclude-pattern>/plugins/*</exclude-pattern>
<exclude-pattern>/modules/*</exclude-pattern>
<exclude-pattern>/installation/*</exclude-pattern>
<!-- These exceptions are permanent -->
<exclude-pattern>*/tmpl/*</exclude-pattern>
<exclude-pattern>/templates/*</exclude-pattern>
<exclude-pattern>/installation/template/index.php</exclude-pattern>
</rule>
</ruleset>

0 comments on commit a357942

Please sign in to comment.