FORGE-1339: idStrategy is no longer required and assumes default of AUTO

if not informed
forge · Nov 26, 2013 · 5929e8f · 5929e8f
1 parent 43f9aa5
commit 5929e8f
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/javaee/impl/src/main/java/org/jboss/forge/addon/javaee/jpa/ui/NewEntityCommand.java b/javaee/impl/src/main/java/org/jboss/forge/addon/javaee/jpa/ui/NewEntityCommand.java
@@ -50,7 +50,7 @@ public class NewEntityCommand extends AbstractJavaEECommand
    private UIInput<String> targetPackage;
 
    @Inject
-   @WithAttributes(label = "ID Column Generation Strategy", required = true)
+   @WithAttributes(label = "ID Column Generation Strategy")
    private UISelectOne<GenerationType> idStrategy;
 
    @Inject
@@ -137,6 +137,10 @@ public Result execute(UIExecutionContext context) throws Exception
       String entityName = named.getValue();
       String entityPackage = targetPackage.getValue();
       GenerationType idStrategyChosen = idStrategy.getValue();
+      if (idStrategyChosen == null)
+      {
+         idStrategyChosen = GenerationType.AUTO;
+      }
       DirectoryResource targetDir = targetLocation.getValue();
       Project project = getSelectedProject(context);
       JavaResource javaResource;