Changed new controller dialog validator

Author: drpayyne

src/com/magento/idea/magento2plugin/actions/generation/dialog/NewControllerDialog.java

@@ -10,7 +10,9 @@
 import com.intellij.psi.PsiFile;
 import com.magento.idea.magento2plugin.actions.generation.NewControllerAction;
 import com.magento.idea.magento2plugin.actions.generation.data.ControllerFileData;
-import com.magento.idea.magento2plugin.actions.generation.dialog.validator.NewControllerValidator;
+import com.magento.idea.magento2plugin.actions.generation.dialog.validator.annotation.FieldValidation;
+import com.magento.idea.magento2plugin.actions.generation.dialog.validator.annotation.RuleRegistry;
+import com.magento.idea.magento2plugin.actions.generation.dialog.validator.rule.NotEmptyRule;
 import com.magento.idea.magento2plugin.actions.generation.generator.ModuleControllerClassGenerator;
 import com.magento.idea.magento2plugin.magento.files.ControllerBackendPhp;
 import com.magento.idea.magento2plugin.magento.files.ControllerFrontendPhp;
@@ -21,7 +23,6 @@
 import com.magento.idea.magento2plugin.ui.FilteredComboBox;
 import com.magento.idea.magento2plugin.util.magento.GetModuleNameByDirectoryUtil;
 import java.awt.event.ActionEvent;
-import java.awt.event.ActionListener;
 import java.awt.event.KeyEvent;
 import java.awt.event.WindowAdapter;
 import java.awt.event.WindowEvent;
@@ -40,19 +41,35 @@
         "PMD.ConstructorCallsOverridableMethod"
 })
 public class NewControllerDialog extends AbstractDialog {
-    private final NewControllerValidator validator;
     private final String moduleName;
     private final Project project;
     private JPanel contentPane;
     private JButton buttonOK;
     private JButton buttonCancel;
     private FilteredComboBox controllerAreaSelect;
     private FilteredComboBox httpMethodSelect;
-    private JTextField controllerName;
     private JTextField controllerParentDir;
     private JCheckBox inheritClass;
     private JPanel adminPanel;
     private JTextField acl;
+
+    private static final String CONTROLLER_NAME = "controller name";
+    private static final String ACTION_NAME = "action name";
+
+    @FieldValidation(rule = RuleRegistry.NOT_EMPTY,
+            message = {NotEmptyRule.MESSAGE, CONTROLLER_NAME})
+    @FieldValidation(rule = RuleRegistry.START_WITH_NUMBER_OR_CAPITAL_LETTER,
+            message = {NotEmptyRule.MESSAGE, CONTROLLER_NAME})
+    @FieldValidation(rule = RuleRegistry.ALPHANUMERIC,
+            message = {NotEmptyRule.MESSAGE, CONTROLLER_NAME})
+    @FieldValidation(rule = RuleRegistry.PHP_CLASS,
+            message = {NotEmptyRule.MESSAGE, CONTROLLER_NAME})
+    private JTextField controllerName;
+
+    @FieldValidation(rule = RuleRegistry.NOT_EMPTY,
+            message = {NotEmptyRule.MESSAGE, ACTION_NAME})
+    @FieldValidation(rule = RuleRegistry.PHP_NAMESPACE_NAME,
+            message = {NotEmptyRule.MESSAGE, ACTION_NAME})
     private JTextField actionName;
 
     /**
@@ -65,7 +82,6 @@ public NewControllerDialog(final Project project, final PsiDirectory directory)
         super();
         this.project = project;
         this.moduleName = GetModuleNameByDirectoryUtil.execute(directory, project);
-        this.validator = NewControllerValidator.getInstance(this);
 
         setContentPane(contentPane);
         setModal(true);
@@ -87,11 +103,7 @@ public void windowClosing(final WindowEvent event) {
 
         // call onCancel() on ESCAPE
         contentPane.registerKeyboardAction(
-                new ActionListener() {
-                    public void actionPerformed(final ActionEvent event) {
-                        onCancel();
-                    }
-                },
+                (final ActionEvent event) -> onCancel(),
                 KeyStroke.getKeyStroke(KeyEvent.VK_ESCAPE, 0),
                 JComponent.WHEN_ANCESTOR_OF_FOCUSED_COMPONENT
         );
@@ -178,7 +190,7 @@ public static void open(final Project project, final PsiDirectory directory) {
     }
 
     private void onOK() {
-        if (!validator.validate()) {
+        if (!validateFormFields()) {
             return;
         }
 
@@ -243,6 +255,7 @@ private Boolean getIsInheritClass() {
         return inheritClass.isSelected();
     }
 
+    @Override
     protected void onCancel() {
         dispose();
     }