Merge pull request #10 from mediact/feature/phpmd-loosen-overstrict-checks

3.0.0 Feature: loosened a few PHPMD checks which were too strict

Author: marcelmediact

composer.json

@@ -32,5 +32,14 @@
         "grumphp": {
             "config-default-path": "vendor/mediact/testing-suite/config/default/grumphp.yml"
         }
+    },
+    "archive": {
+        "exclude": [
+            "/.gitignore",
+            "/phpunit.xml",
+            "/phpmd.xml",
+            "/phpstan.neon",
+            "/phpcs.xml"
+        ]
     }
 }

phpunit.xml

@@ -1,6 +1,7 @@
 <phpunit xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
       xsi:noNamespaceSchemaLocation="vendor/phpunit/phpunit/phpunit.xsd"
-          bootstrap="vendor/autoload.php">
+          bootstrap="vendor/autoload.php"
+          cacheResult="false">
     <testsuites>
         <testsuite name="default">
             <directory>tests</directory>

src/MediaCT/phpmd.xml

@@ -14,11 +14,25 @@
         <exclude name="ElseExpression"/>
         <exclude name="StaticAccess"/>
         <exclude name="UndefinedVariable"/>
+        <exclude name="BooleanArgumentFlag"/>
     </rule>
     <rule ref="rulesets/codesize.xml"/>
-    <rule ref="rulesets/design.xml"/>
+    <rule ref="rulesets/design.xml">
+        <exclude name="CouplingBetweenObjects"/>
+    </rule>
+    <rule ref="rulesets/design.xml/CouplingBetweenObjects">
+        <properties>
+            <property name="maximum" value="20"/>
+        </properties>
+    </rule>
     <rule ref="rulesets/naming.xml">
         <exclude name="ShortVariable"/>
+        <exclude name="LongVariable"/>
+    </rule>
+    <rule ref="rulesets/naming.xml/LongVariable">
+        <properties>
+            <property name="maximum" value="30"/>
+        </properties>
     </rule>
     <rule ref="rulesets/unusedcode.xml">
         <exclude name="UnusedFormalParameter"/>

src/MediactCommon/ruleset.xml

@@ -8,8 +8,8 @@
 <ruleset name="Mediact/Common">
     <description>MediaCT its extension to PSR2.</description>
 
-    <!-- Base rules on PSR2 -->
-    <rule ref="PSR2">
+    <!-- Base rules on PSR12 -->
+    <rule ref="PSR12">
         <exclude name="Squiz.Functions.MultiLineFunctionDeclaration.UseOneParamPerLine" />
     </rule>