pr14768 - modified so it can only be added to the required option list once

John Keyes · John Keyes · commit 500e99fe4596 · 2003-01-16T21:35:30.000Z
git-svn-id: https://svn.apache.org/repos/asf/jakarta/commons/proper/cli/trunk@129870 13f79535-47bb-0310-9956-ffa450edef68
diff --git a/src/java/org/apache/commons/cli/Options.java b/src/java/org/apache/commons/cli/Options.java
@@ -184,8 +184,11 @@ public Options addOption(Option opt)
         }
 
         // if the option is required add it to the required list
-        if (opt.isRequired() && !requiredOpts.contains(key))
+        if (opt.isRequired() ) 
         {
+            if( requiredOpts.contains(key) ) {
+                requiredOpts.remove( requiredOpts.indexOf(key) );
+            }
             requiredOpts.add(key);
         }
 

Original file line number	Diff line number	Diff line change
`@@ -184,8 +184,11 @@ public Options addOption(Option opt)`
`184`	`184`	`}`
`185`	`185`
`186`	`186`	`// if the option is required add it to the required list`
`187`		`- if (opt.isRequired() && !requiredOpts.contains(key))`
	`187`	`+ if (opt.isRequired() )`
`188`	`188`	`{`
	`189`	`+ if( requiredOpts.contains(key) ) {`
	`190`	`+ requiredOpts.remove( requiredOpts.indexOf(key) );`
	`191`	`+ }`
`189`	`192`	`requiredOpts.add(key);`
`190`	`193`	`}`
`191`	`194`