Right, good catch. Also, do you think this method could belong to FieldOperations? That means it could be reused elsewhere.

> Em 12/04/2014, às 02:29, Robert Balent <notifications@github.com> escreveu:
> 
> In javaee/impl/src/main/java/org/jboss/forge/addon/javaee/jpa/ui/NewFieldWizard.java:
> 
> > @@ -214,9 +221,20 @@ public Boolean call() throws Exception
> >                       && (Date.class.getName().equals(typeValue) || Calendar.class.getName().equals(typeValue));
> >           }
> >        });
> > +
> > +      enumType.setEnabled(new Callable<Boolean>() {
> > +
> > +         @Override
> > +         public Boolean call() throws Exception
> > +         {
> > +            return !lob.getValue() && !transientField.getValue() && isFieldEnum(type.getValue(), project);
> There is default value for "type" set to "String" so in this case it should not return null. I don't think it's necessary but it would be really small change. Should I add something like this?
> 
> -      if (project != null)
> +      if (fieldType != null && project != null)
> —
> Reply to this email directly or view it on GitHub.