<div dir="ltr">Sorry, the subject should be "missing options"</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Tue, Dec 31, 2013 at 5:15 PM, Lincoln Baxter, III <span dir="ltr"><<a href="mailto:lincolnbaxter@gmail.com" target="_blank">lincolnbaxter@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">So I don't forget by the time we get back from new-years- The bug is in ShellWizard.java:<div><br></div>
<div><a href="https://issues.jboss.org/browse/FORGE-1407" target="_blank">https://issues.jboss.org/browse/FORGE-1407</a><br>
<div>
<p> <span>private</span> CommandLineParser populate(ShellContext shellContext, String line,</p>
<p> <span>final</span> Map<String, InputComponent<?, ?>> inputs)</p>
<p> <span>throws</span> Exception</p>
<p> {</p>
<p> inputs.putAll(getController().getInputs());</p>
<p> CommandLineParser parser = <span>commandLineUtil</span>.generateParser(getController(), shellContext, inputs);</p>
<p> CommandLine cmdLine = parser.parse(line, <span>true</span>);</p>
<p> Map<String, InputComponent<?, ?>> populatedInputs = <span>commandLineUtil</span>.populateUIInputs(cmdLine, inputs);</p>
<p> <span>if</span> (getController().isValid())</p>
<p> {</p>
<p> <span>if</span> (getController().canMoveToNextStep())</p>
<p> {</p>
<p><span> </span>// The bug is here. we are immediately moving to the next step and removing all inputs that are not populated.</p>
<p><span> </span>// this really needs to know if we have defaulted to next step or actually moved there willingly</p>
<p><span> </span>// it should not remove old choices until a new choice has been made</p>
<p> getController().next().initialize();</p>
<p> inputs.keySet().retainAll(populatedInputs.keySet());</p>
<p> parser = populate(shellContext, line, inputs);</p>
<p> }</p>
<p> }</p>
<p> <span>return</span> parser;</p>
<p> }</p><span class="HOEnZb"><font color="#888888"><div><div><br></div>-- <br>Lincoln Baxter, III<br><a href="http://ocpsoft.org" target="_blank">http://ocpsoft.org</a><br>"Simpler is better."
</div></font></span></div></div></div>
</blockquote></div><br><br clear="all"><div><br></div>-- <br>Lincoln Baxter, III<br><a href="http://ocpsoft.org" target="_blank">http://ocpsoft.org</a><br>"Simpler is better."
</div>