The processing of the default options is no longer stopped by a flag with a false value (CLI-201)

git-svn-id: https://svn.apache.org/repos/asf/commons/proper/cli/trunk@955096 13f79535-47bb-0310-9956-ffa450edef68

Author: ebourg

src/java/org/apache/commons/cli/Parser.java

@@ -287,7 +287,7 @@ else if (!("yes".equalsIgnoreCase(value)
                 {
                     // if the value is not yes, true or 1 then don't add the
                     // option to the CommandLine
-                    break;
+                    continue;
                 }
 
                 cmd.addOption(opt);

src/test/org/apache/commons/cli/ValueTest.java

@@ -212,7 +212,7 @@ public void testPropertyOptionFlags() throws Exception
         cmd = parser.parse(opts, null, properties);
         assertTrue( !cmd.hasOption("a") );
         assertTrue( !cmd.hasOption("c") );
-        assertTrue( !cmd.hasOption("e") );
+        assertTrue( cmd.hasOption("e") ); // this option accepts as argument
 
 
         properties = new Properties();
@@ -233,7 +233,7 @@ public void testPropertyOptionFlags() throws Exception
         cmd = parser.parse(opts, null, properties);
         assertTrue( !cmd.hasOption("a") );
         assertTrue( !cmd.hasOption("c") );
-        assertTrue( !cmd.hasOption("e") );
+        assertTrue( cmd.hasOption("e") );
     } 
 
     public void testPropertyOptionMultipleValues() throws Exception