Uploaded image for project: 'Forge'
  1. Forge
  2. FORGE-1407

Shell is missing options in wizards where default values allow immediately moving to next step

XMLWordPrintable

    • Icon: Feature Request Feature Request
    • Resolution: Done
    • Icon: Major Major
    • 2.0.0.CR1
    • 2.0.0.Beta4
    • UI - Shell
    • None

      Should have a lot more options:

      [jpa-demo]$ jpa-setup 
      --dataSourceName  --dbType  --persistenceUnitName 
      

      The problem is here in ShellWizard.java (in shell addon)

       private CommandLineParser populate(ShellContext shellContext, String line,
      
                  final Map<String, InputComponent<?, ?>> inputs)
      
                  throws Exception
      
         {
      
            inputs.putAll(getController().getInputs());
      
            CommandLineParser parser = commandLineUtil.generateParser(getController(), shellContext, inputs);
      
            CommandLine cmdLine = parser.parse(line, true);
      
            Map<String, InputComponent<?, ?>> populatedInputs = commandLineUtil.populateUIInputs(cmdLine, inputs);
      
            if (getController().isValid())
      
            {
      
               if (getController().canMoveToNextStep())
      
               {
      
                  // The bug is here. we are immediately moving to the next step and removing all inputs that are not populated.
      
                  // this really needs to know if we have defaulted to next step or actually moved there willingly
      
                  // it should not remove old choices until a new choice has been made
      
                  getController().next().initialize();
      
                  inputs.keySet().retainAll(populatedInputs.keySet());
      
                  parser = populate(shellContext, line, inputs);
      
               }
      
            }
      
            return parser;
      
         }
      

              rhn-support-ggastald George Gastaldi
              lincolnthree Lincoln Baxter III (Inactive)
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

                Created:
                Updated:
                Resolved: