Add CLI IgnoreViolationsOnExit option flag

README.rst

@@ -94,6 +94,9 @@ Command line options
 
   - ``--strict`` - Also report those nodes with a @SuppressWarnings annotation.
 
+  - ``--ignore-violations-on-exit`` - will exit with a zero code, even if any
+    violations are found.
+
   An example command line: ::
 
     phpmd PHP/Depend/DbusUI xml codesize --reportfile phpmd.xml --suffixes php,phtml
@@ -144,7 +147,9 @@ PHPMD's command line tool currently defines three different exit codes.
   interrupted PHPMD during execution.
 - *2*, This exit code means that PHPMD has processed the code under test
   without the occurence of an error/exception, but it has detected rule
-  violations in the analyzed source code.
+  violations in the analyzed source code. You can also prevent this behaviour
+  with the ``--ignore-violations-on-exit`` flag, which will result to a *0*
+  even if any violations are found.
 
 Renderers
 ---------

src/main/php/PHPMD/TextUI/Command.php

@@ -70,6 +70,9 @@ class Command
      * found in the analyzed code. Otherwise this method will return a value
      * equal to <b>EXIT_VIOLATION</b>.
      *
+     * The use of flag <b>--ignore-violations-on-exit</b> will result to a
+     * <b>EXIT_SUCCESS</b> even if any violation is found.
+     *
      * @param \PHPMD\TextUI\CommandLineOptions $opts
      * @param \PHPMD\RuleSetFactory $ruleSetFactory
      * @return integer
@@ -129,7 +132,7 @@ public function run(CommandLineOptions $opts, RuleSetFactory $ruleSetFactory)
             $ruleSetFactory
         );
 
-        if ($phpmd->hasViolations()) {
+        if ($phpmd->hasViolations() && !$opts->ignoreViolationsOnExit()) {
             return self::EXIT_VIOLATION;
         }
         return self::EXIT_SUCCESS;

src/main/php/PHPMD/TextUI/CommandLineOptions.php

@@ -139,6 +139,13 @@ class CommandLineOptions
      */
     protected $strict = false;
 
+    /**
+     * Should PHPMD exit without error code even if violation is found?
+     *
+     * @var boolean
+     */
+    protected $ignoreViolationsOnExit = false;
+
     /**
      * List of available rule-sets.
      *
@@ -193,6 +200,9 @@ public function __construct(array $args, array $availableRuleSets = array())
                 case '--strict':
                     $this->strict = true;
                     break;
+                case '--ignore-violations-on-exit':
+                    $this->ignoreViolationsOnExit = true;
+                    break;
                 case (preg_match('(^\-\-reportfile\-(xml|html|text)$)', $arg, $match) > 0):
                     $this->reportFiles[$match[1]] = array_shift($args);
                     break;
@@ -327,6 +337,16 @@ public function hasStrict()
         return $this->strict;
     }
 
+    /**
+     * Was the <b>--ignore-violations-on-exit</b> passed to PHPMD's command line interface?
+     *
+     * @return boolean
+     */
+    public function ignoreViolationsOnExit()
+    {
+        return $this->ignoreViolationsOnExit;
+    }
+
     /**
      * Creates a report renderer instance based on the user's command line
      * argument.
@@ -438,7 +458,9 @@ public function usage()
                '--exclude: comma-separated string of patterns that are used to ' .
                'ignore directories' . \PHP_EOL .
                 '--strict: also report those nodes with a @SuppressWarnings ' .
-               'annotation' . \PHP_EOL;
+               'annotation' . \PHP_EOL .
+                '--ignore-violations-on-exit: will exit with a zero code, ' .
+                'even if any violations are found' . \PHP_EOL;
     }
 
     /**

src/site/rst/documentation/index.rst

@@ -54,6 +54,9 @@ Command line options
 
   - ``--strict`` - Also report those nodes with a @SuppressWarnings annotation.
 
+  - ``--ignore-violations-on-exit`` - will exit with a zero code, even if any
+    violations are found.
+
   An example command line: ::
 
     phpmd PHP/Depend/DbusUI xml codesize --reportfile phpmd.xml --suffixes php,phtml
@@ -96,7 +99,9 @@ PHPMD's command line tool currently defines three different exit codes.
   interrupted PHPMD during execution.
 - *2*, This exit code means that PHPMD has processed the code under test
   without the occurrence of an error/exception, but it has detected rule
-  violations in the analyzed source code.
+  violations in the analyzed source code. You can also prevent this behaviour
+  with the ``--ignore-violations-on-exit`` flag, which will result to a *0*
+  even if any violations are found.
 
 Renderers
 =========

src/test/php/PHPMD/TextUI/CommandLineOptionsTest.php

@@ -174,7 +174,7 @@ public function testThrowsExpectedExceptionWhenInputFileNotExists()
     }
 
     /**
-     * testCliOptionsAcceptsVersionArgument
+     * testHasVersionReturnsFalseByDefault
      *
      * @return void
      */
@@ -199,6 +199,45 @@ public function testCliOptionsAcceptsVersionArgument()
         self::assertTrue($opts->hasVersion());
     }
 
+    /**
+     * Tests if ignoreViolationsOnExit returns false by default
+     *
+     * @return void
+     */
+    public function testIgnoreViolationsOnExitReturnsFalseByDefault()
+    {
+        $args = array(__FILE__, __FILE__, 'text', 'unusedcode');
+        $opts = new CommandLineOptions($args);
+
+        self::assertFalse($opts->ignoreViolationsOnExit());
+    }
+
+    /**
+     * Tests if CLI options accepts ignoreViolationsOnExit argument
+     *
+     * @return void
+     */
+    public function testCliOptionsAcceptsIgnoreViolationsOnExitArgument()
+    {
+        $args = array(__FILE__, __FILE__, 'text', 'unusedcode', '--ignore-violations-on-exit');
+        $opts = new CommandLineOptions($args);
+
+        self::assertTrue($opts->ignoreViolationsOnExit());
+    }
+
+    /**
+     * Tests if CLI usage contains ignoreViolationsOnExit option
+     *
+     * @return void
+     */
+    public function testCliUsageContainsIgnoreViolationsOnExitOption()
+    {
+        $args = array(__FILE__, __FILE__, 'text', 'codesize');
+        $opts = new CommandLineOptions($args);
+
+        $this->assertContains('--ignore-violations-on-exit:', $opts->usage());
+    }
+
     /**
      * testCliUsageContainsStrictOption
      * 

src/test/php/PHPMD/TextUI/CommandTest.php

@@ -77,7 +77,7 @@ public function testMainReturnsSuccessExitCodeForSourceWithoutViolations()
     }
 
     /**
-     * testMainReturnsViolationExitCodeForSourceWithNPathViolation
+     * Tests if main returns violation Exit Code for Source with NPath Violation
      *
      * @return void
      * @covers \PHPMD\TextUI\Command
@@ -99,4 +99,32 @@ public function testMainReturnsViolationExitCodeForSourceWithNPathViolation()
         );
         $this->assertEquals(Command::EXIT_VIOLATION, $exitCode);
     }
+
+    /**
+     * Tests if main returns success Exit Code for Source with NPath Violation and IgnoreViolationsOnExit Flag
+     *
+     * @see ::testMainReturnsViolationExitCodeForSourceWithNPathViolation
+     *      Same as the other test, but with '--ignore-violations-on-exit' set.
+     *
+     * @return void
+     * @covers \PHPMD\TextUI\Command
+     * @group phpmd
+     * @group phpmd::textui
+     * @group unittest
+     */
+    public function testMainReturnsSuccessExitCodeForSourceWithNPathViolationAndIgnoreViolationsOnExitFlag()
+    {
+        $exitCode = Command::main(
+            array(
+                __FILE__,
+                self::createFileUri('source/source_with_npath_violation.php'),
+                'text',
+                'codesize',
+                '--reportfile',
+                self::createTempFileUri(),
+                '--ignore-violations-on-exit',
+            )
+        );
+        $this->assertEquals(Command::EXIT_SUCCESS, $exitCode);
+    }
 }