From cfd627398ef372ec573c8405249089dd7082a5f6 Mon Sep 17 00:00:00 2001
From: bohdan-harniuk <b.harniuk@atwix.com>
Date: Mon, 20 Dec 2021 23:21:16 +0200
Subject: [PATCH 01/10] 864: Code refactoring

---
 .../actions/generation/NewBlockAction.java               | 9 +++++----
 1 file changed, 5 insertions(+), 4 deletions(-)

diff --git a/src/com/magento/idea/magento2plugin/actions/generation/NewBlockAction.java b/src/com/magento/idea/magento2plugin/actions/generation/NewBlockAction.java
index cefba0e7c..1fe6a3db4 100644
--- a/src/com/magento/idea/magento2plugin/actions/generation/NewBlockAction.java
+++ b/src/com/magento/idea/magento2plugin/actions/generation/NewBlockAction.java
@@ -18,6 +18,7 @@
 import org.jetbrains.annotations.NotNull;
 
 public class NewBlockAction extends AnAction {
+
     public static final String ACTION_NAME = "Magento 2 Block";
     public static final String ACTION_DESCRIPTION = "Create a new Magento 2 Block";
 
@@ -26,23 +27,24 @@ public NewBlockAction() {
     }
 
     @Override
-    public void actionPerformed(@NotNull final AnActionEvent event) {
+    public void actionPerformed(final @NotNull AnActionEvent event) {
         final DataContext dataContext = event.getDataContext();
         final IdeView view = LangDataKeys.IDE_VIEW.getData(dataContext);
+
         if (view == null) {
             return;
         }
 
         final Project project = CommonDataKeys.PROJECT.getData(dataContext);
+
         if (project == null) {
             return;
         }
-
         final PsiDirectory directory = view.getOrChooseDirectory();
+
         if (directory == null) {
             return;
         }
-
         NewBlockDialog.open(project, directory);
     }
 
@@ -51,4 +53,3 @@ public boolean isDumbAware() {
         return false;
     }
 }
-

From 6b1ff0eb3cea2fe869b47af6491e47123c535660 Mon Sep 17 00:00:00 2001
From: bohdan-harniuk <b.harniuk@atwix.com>
Date: Mon, 20 Dec 2021 23:24:18 +0200
Subject: [PATCH 02/10] 864: Code refactoring

---
 .../generation/dialog/NewBlockDialog.java        | 16 ++++++++--------
 1 file changed, 8 insertions(+), 8 deletions(-)

diff --git a/src/com/magento/idea/magento2plugin/actions/generation/dialog/NewBlockDialog.java b/src/com/magento/idea/magento2plugin/actions/generation/dialog/NewBlockDialog.java
index 6bc982cc5..30d417b77 100644
--- a/src/com/magento/idea/magento2plugin/actions/generation/dialog/NewBlockDialog.java
+++ b/src/com/magento/idea/magento2plugin/actions/generation/dialog/NewBlockDialog.java
@@ -117,8 +117,8 @@ protected void onOK() {
         exit();
     }
 
-    private PsiFile generateFile() {
-        return new ModuleBlockClassGenerator(new BlockFileData(
+    private void generateFile() {
+        new ModuleBlockClassGenerator(new BlockFileData(
                 getBlockDirectory(),
                 getBlockName(),
                 getModuleName(),
@@ -140,6 +140,7 @@ public String getBlockDirectory() {
 
     private void suggestBlockDirectory() {
         final String moduleIdentifierPath = getModuleIdentifierPath();
+
         if (moduleIdentifierPath == null) {
             blockParentDir.setText(BlockPhp.DEFAULT_DIR);
             return;
@@ -147,6 +148,7 @@ private void suggestBlockDirectory() {
         final String path = baseDir.getVirtualFile().getPath();
         final String[] pathParts = path.split(moduleIdentifierPath);
         final int minimumPathParts = 2;
+
         if (pathParts.length != minimumPathParts) {
             blockParentDir.setText(BlockPhp.DEFAULT_DIR);
             return;
@@ -161,14 +163,17 @@ private void suggestBlockDirectory() {
 
     private String getModuleIdentifierPath() {
         final String[]parts = moduleName.split(Package.vendorModuleNameSeparator);
+
         if (parts[0] == null || parts[1] == null || parts.length > 2) {
             return null;
         }
+
         return parts[0] + File.separator + parts[1];
     }
 
     private String getNamespace() {
         final String[]parts = moduleName.split(Package.vendorModuleNameSeparator);
+
         if (parts[0] == null || parts[1] == null || parts.length > 2) {
             return null;
         }
@@ -176,12 +181,7 @@ private String getNamespace() {
                 File.separator,
                 Package.fqnSeparator
         );
-        return parts[0] + Package.fqnSeparator + parts[1] + Package.fqnSeparator + directoryPart;
-    }
 
-    @Override
-    public void onCancel() {
-        // add your code here if necessary
-        dispose();
+        return parts[0] + Package.fqnSeparator + parts[1] + Package.fqnSeparator + directoryPart;
     }
 }

From a741849bbe47f60c5b9ac23745f250465a3e52f5 Mon Sep 17 00:00:00 2001
From: bohdan-harniuk <b.harniuk@atwix.com>
Date: Mon, 20 Dec 2021 23:34:55 +0200
Subject: [PATCH 03/10] 864: Enhanced error outputting for new Magento 2 Block
 generation

---
 .../generation/dialog/NewBlockDialog.form     | 22 ++++++++++++++++---
 .../generation/dialog/NewBlockDialog.java     | 17 +++++++-------
 2 files changed, 28 insertions(+), 11 deletions(-)

diff --git a/src/com/magento/idea/magento2plugin/actions/generation/dialog/NewBlockDialog.form b/src/com/magento/idea/magento2plugin/actions/generation/dialog/NewBlockDialog.form
index 9689cc6a0..c460ac6fa 100644
--- a/src/com/magento/idea/magento2plugin/actions/generation/dialog/NewBlockDialog.form
+++ b/src/com/magento/idea/magento2plugin/actions/generation/dialog/NewBlockDialog.form
@@ -53,7 +53,7 @@
           </grid>
         </children>
       </grid>
-      <grid id="e3588" layout-manager="GridLayoutManager" row-count="2" column-count="2" same-size-horizontally="false" same-size-vertically="false" hgap="-1" vgap="-1">
+      <grid id="e3588" layout-manager="GridLayoutManager" row-count="4" column-count="2" same-size-horizontally="false" same-size-vertically="false" hgap="-1" vgap="-1">
         <margin top="0" left="0" bottom="0" right="0"/>
         <constraints>
           <grid row="0" column="0" row-span="1" col-span="1" vsize-policy="3" hsize-policy="3" anchor="0" fill="3" indent="0" use-parent-layout="false"/>
@@ -85,7 +85,7 @@
           </component>
           <component id="fdc52" class="javax.swing.JTextField" binding="blockParentDir">
             <constraints>
-              <grid row="1" column="1" row-span="1" col-span="1" vsize-policy="0" hsize-policy="6" anchor="8" fill="1" indent="0" use-parent-layout="false">
+              <grid row="2" column="1" row-span="1" col-span="1" vsize-policy="0" hsize-policy="6" anchor="8" fill="1" indent="0" use-parent-layout="false">
                 <preferred-size width="150" height="-1"/>
               </grid>
             </constraints>
@@ -96,7 +96,7 @@
           </component>
           <component id="be25f" class="javax.swing.JLabel">
             <constraints>
-              <grid row="1" column="0" row-span="1" col-span="1" vsize-policy="0" hsize-policy="0" anchor="8" fill="0" indent="0" use-parent-layout="false"/>
+              <grid row="2" column="0" row-span="1" col-span="1" vsize-policy="0" hsize-policy="0" anchor="8" fill="0" indent="0" use-parent-layout="false"/>
             </constraints>
             <properties>
               <labelFor value="fdc52"/>
@@ -106,6 +106,22 @@
               <html.disable class="java.lang.Boolean" value="true"/>
             </clientProperties>
           </component>
+          <component id="48e8" class="javax.swing.JLabel" binding="blockNameErrorMessage">
+            <constraints>
+              <grid row="1" column="1" row-span="1" col-span="1" vsize-policy="0" hsize-policy="0" anchor="8" fill="0" indent="0" use-parent-layout="false"/>
+            </constraints>
+            <properties>
+              <text value=""/>
+            </properties>
+          </component>
+          <component id="340c3" class="javax.swing.JLabel" binding="blockParentDirErrorMessage">
+            <constraints>
+              <grid row="3" column="1" row-span="1" col-span="1" vsize-policy="0" hsize-policy="0" anchor="8" fill="0" indent="0" use-parent-layout="false"/>
+            </constraints>
+            <properties>
+              <text value=""/>
+            </properties>
+          </component>
         </children>
       </grid>
       <grid id="ddd4f" layout-manager="GridLayoutManager" row-count="2" column-count="1" same-size-horizontally="false" same-size-vertically="false" hgap="-1" vgap="-1">
diff --git a/src/com/magento/idea/magento2plugin/actions/generation/dialog/NewBlockDialog.java b/src/com/magento/idea/magento2plugin/actions/generation/dialog/NewBlockDialog.java
index 30d417b77..44342993a 100644
--- a/src/com/magento/idea/magento2plugin/actions/generation/dialog/NewBlockDialog.java
+++ b/src/com/magento/idea/magento2plugin/actions/generation/dialog/NewBlockDialog.java
@@ -7,7 +7,6 @@
 
 import com.intellij.openapi.project.Project;
 import com.intellij.psi.PsiDirectory;
-import com.intellij.psi.PsiFile;
 import com.magento.idea.magento2plugin.actions.generation.NewBlockAction;
 import com.magento.idea.magento2plugin.actions.generation.data.BlockFileData;
 import com.magento.idea.magento2plugin.actions.generation.dialog.validator.annotation.FieldValidation;
@@ -26,6 +25,7 @@
 import java.awt.event.WindowEvent;
 import javax.swing.JButton;
 import javax.swing.JComponent;
+import javax.swing.JLabel;
 import javax.swing.JPanel;
 import javax.swing.JRadioButton;
 import javax.swing.JTextField;
@@ -33,6 +33,7 @@
 import javax.swing.KeyStroke;
 
 public class NewBlockDialog extends AbstractDialog {
+
     private final PsiDirectory baseDir;
     private final String moduleName;
     private JPanel contentPanel;
@@ -44,18 +45,18 @@ public class NewBlockDialog extends AbstractDialog {
     private static final String NAME = "name";
     private static final String DIRECTORY = "directory";
 
-    @FieldValidation(rule = RuleRegistry.NOT_EMPTY,
-            message = {NotEmptyRule.MESSAGE, NAME})
-    @FieldValidation(rule = RuleRegistry.PHP_CLASS,
-            message = {PhpClassRule.MESSAGE, NAME})
+    @FieldValidation(rule = RuleRegistry.NOT_EMPTY, message = {NotEmptyRule.MESSAGE, NAME})
+    @FieldValidation(rule = RuleRegistry.PHP_CLASS, message = {PhpClassRule.MESSAGE, NAME})
     private JTextField blockName;
 
-    @FieldValidation(rule = RuleRegistry.NOT_EMPTY,
-            message = {NotEmptyRule.MESSAGE, DIRECTORY})
+    @FieldValidation(rule = RuleRegistry.NOT_EMPTY, message = {NotEmptyRule.MESSAGE, DIRECTORY})
     @FieldValidation(rule = RuleRegistry.PHP_DIRECTORY,
             message = {PhpDirectoryRule.MESSAGE, DIRECTORY})
     private JTextField blockParentDir;
 
+    private JLabel blockNameErrorMessage;//NOPMD
+    private JLabel blockParentDirErrorMessage;//NOPMD
+
     /**
      * Constructor.
      *
@@ -113,8 +114,8 @@ public static void open(final Project project, final PsiDirectory directory) {
     protected void onOK() {
         if (validateFormFields()) {
             generateFile();
+            exit();
         }
-        exit();
     }
 
     private void generateFile() {

From 15d99746827e71a6a211bdd2425f4b17dbc984dd Mon Sep 17 00:00:00 2001
From: bohdan-harniuk <b.harniuk@atwix.com>
Date: Tue, 21 Dec 2021 01:16:27 +0200
Subject: [PATCH 04/10] 867: Code refactoring

---
 .../actions/generation/NewControllerAction.java             | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/src/com/magento/idea/magento2plugin/actions/generation/NewControllerAction.java b/src/com/magento/idea/magento2plugin/actions/generation/NewControllerAction.java
index 277636830..9460114bd 100644
--- a/src/com/magento/idea/magento2plugin/actions/generation/NewControllerAction.java
+++ b/src/com/magento/idea/magento2plugin/actions/generation/NewControllerAction.java
@@ -16,8 +16,8 @@
 import com.magento.idea.magento2plugin.MagentoIcons;
 import com.magento.idea.magento2plugin.actions.generation.dialog.NewControllerDialog;
 
-@SuppressWarnings({"PMD.OnlyOneReturn", "PMD.FieldNamingConventions"})
 public class NewControllerAction extends AnAction {
+
     public static final String ACTION_NAME = "Magento 2 Controller";
     public static final String ACTION_DESCRIPTION = "Create a new Magento 2 Controller";
 
@@ -36,13 +36,13 @@ public void actionPerformed(final AnActionEvent event) {
         if (view == null) {
             return;
         }
-
         final Project project = CommonDataKeys.PROJECT.getData(dataContext);
+
         if (project == null) {
             return;
         }
-
         final PsiDirectory directory = view.getOrChooseDirectory();
+
         if (directory == null) {
             return;
         }

From 0d859165dd76049377df5492a403c4e44d93c7f9 Mon Sep 17 00:00:00 2001
From: bohdan-harniuk <b.harniuk@atwix.com>
Date: Tue, 21 Dec 2021 01:28:43 +0200
Subject: [PATCH 05/10] 867: Enhanced error outputting for new Magento 2
 Controller generation

---
 .../dialog/NewControllerDialog.form           | 38 ++++++++++++++++---
 .../dialog/NewControllerDialog.java           | 22 +++++------
 2 files changed, 43 insertions(+), 17 deletions(-)

diff --git a/src/com/magento/idea/magento2plugin/actions/generation/dialog/NewControllerDialog.form b/src/com/magento/idea/magento2plugin/actions/generation/dialog/NewControllerDialog.form
index 59cbf84c0..3d6b85fc7 100644
--- a/src/com/magento/idea/magento2plugin/actions/generation/dialog/NewControllerDialog.form
+++ b/src/com/magento/idea/magento2plugin/actions/generation/dialog/NewControllerDialog.form
@@ -3,7 +3,7 @@
   <grid id="cbd77" binding="contentPane" layout-manager="GridLayoutManager" row-count="8" column-count="1" same-size-horizontally="false" same-size-vertically="false" hgap="-1" vgap="-1">
     <margin top="10" left="10" bottom="10" right="10"/>
     <constraints>
-      <xy x="48" y="54" width="454" height="423"/>
+      <xy x="48" y="54" width="454" height="431"/>
     </constraints>
     <properties>
       <preferredSize width="450" height="300"/>
@@ -51,7 +51,7 @@
           </grid>
         </children>
       </grid>
-      <grid id="e3588" layout-manager="GridLayoutManager" row-count="3" column-count="2" same-size-horizontally="false" same-size-vertically="false" hgap="-1" vgap="-1">
+      <grid id="e3588" layout-manager="GridLayoutManager" row-count="6" column-count="2" same-size-horizontally="false" same-size-vertically="false" hgap="-1" vgap="-1">
         <margin top="0" left="0" bottom="0" right="0"/>
         <constraints>
           <grid row="5" column="0" row-span="1" col-span="1" vsize-policy="3" hsize-policy="3" anchor="0" fill="3" indent="0" use-parent-layout="false"/>
@@ -61,7 +61,7 @@
         <children>
           <component id="5a121" class="javax.swing.JLabel">
             <constraints>
-              <grid row="1" column="0" row-span="1" col-span="1" vsize-policy="0" hsize-policy="0" anchor="8" fill="0" indent="0" use-parent-layout="false"/>
+              <grid row="2" column="0" row-span="1" col-span="1" vsize-policy="0" hsize-policy="0" anchor="8" fill="0" indent="0" use-parent-layout="false"/>
             </constraints>
             <properties>
               <text resource-bundle="magento2/common" key="common.controller.name"/>
@@ -69,7 +69,7 @@
           </component>
           <component id="4e6cb" class="javax.swing.JTextField" binding="controllerName">
             <constraints>
-              <grid row="1" column="1" row-span="1" col-span="1" vsize-policy="0" hsize-policy="6" anchor="8" fill="1" indent="0" use-parent-layout="false">
+              <grid row="2" column="1" row-span="1" col-span="1" vsize-policy="0" hsize-policy="6" anchor="8" fill="1" indent="0" use-parent-layout="false">
                 <preferred-size width="150" height="-1"/>
               </grid>
             </constraints>
@@ -79,7 +79,7 @@
           </component>
           <component id="d25a" class="javax.swing.JLabel">
             <constraints>
-              <grid row="2" column="0" row-span="1" col-span="1" vsize-policy="0" hsize-policy="0" anchor="8" fill="0" indent="0" use-parent-layout="false"/>
+              <grid row="4" column="0" row-span="1" col-span="1" vsize-policy="0" hsize-policy="0" anchor="8" fill="0" indent="0" use-parent-layout="false"/>
             </constraints>
             <properties>
               <text resource-bundle="magento2/common" key="common.parentDirectory"/>
@@ -87,7 +87,7 @@
           </component>
           <component id="7df01" class="javax.swing.JTextField" binding="controllerParentDir">
             <constraints>
-              <grid row="2" column="1" row-span="1" col-span="1" vsize-policy="0" hsize-policy="6" anchor="8" fill="1" indent="0" use-parent-layout="false">
+              <grid row="4" column="1" row-span="1" col-span="1" vsize-policy="0" hsize-policy="6" anchor="8" fill="1" indent="0" use-parent-layout="false">
                 <preferred-size width="150" height="-1"/>
               </grid>
             </constraints>
@@ -111,6 +111,30 @@
             </constraints>
             <properties/>
           </component>
+          <component id="77374" class="javax.swing.JLabel" binding="actionNameErrorMessage">
+            <constraints>
+              <grid row="1" column="1" row-span="1" col-span="1" vsize-policy="0" hsize-policy="0" anchor="8" fill="0" indent="0" use-parent-layout="false"/>
+            </constraints>
+            <properties>
+              <text value=""/>
+            </properties>
+          </component>
+          <component id="d71e2" class="javax.swing.JLabel" binding="controllerNameErrorMessage">
+            <constraints>
+              <grid row="3" column="1" row-span="1" col-span="1" vsize-policy="0" hsize-policy="0" anchor="8" fill="0" indent="0" use-parent-layout="false"/>
+            </constraints>
+            <properties>
+              <text value=""/>
+            </properties>
+          </component>
+          <component id="f1afe" class="javax.swing.JLabel" binding="controllerParentDirErrorMessage">
+            <constraints>
+              <grid row="5" column="1" row-span="1" col-span="1" vsize-policy="0" hsize-policy="0" anchor="8" fill="0" indent="0" use-parent-layout="false"/>
+            </constraints>
+            <properties>
+              <text value=""/>
+            </properties>
+          </component>
         </children>
       </grid>
       <component id="b8894" class="com.magento.idea.magento2plugin.ui.FilteredComboBox" binding="controllerAreaSelect" custom-create="true">
@@ -118,6 +142,7 @@
           <grid row="1" column="0" row-span="1" col-span="1" vsize-policy="0" hsize-policy="0" anchor="8" fill="0" indent="0" use-parent-layout="false"/>
         </constraints>
         <properties>
+          <editable value="true"/>
           <toolTipText value=""/>
         </properties>
         <clientProperties>
@@ -145,6 +170,7 @@
           <grid row="3" column="0" row-span="1" col-span="1" vsize-policy="0" hsize-policy="0" anchor="8" fill="0" indent="0" use-parent-layout="false"/>
         </constraints>
         <properties>
+          <editable value="true"/>
           <toolTipText value=""/>
         </properties>
         <clientProperties>
diff --git a/src/com/magento/idea/magento2plugin/actions/generation/dialog/NewControllerDialog.java b/src/com/magento/idea/magento2plugin/actions/generation/dialog/NewControllerDialog.java
index 818915cab..402cedb96 100644
--- a/src/com/magento/idea/magento2plugin/actions/generation/dialog/NewControllerDialog.java
+++ b/src/com/magento/idea/magento2plugin/actions/generation/dialog/NewControllerDialog.java
@@ -33,6 +33,7 @@
 import javax.swing.JButton;
 import javax.swing.JCheckBox;
 import javax.swing.JComponent;
+import javax.swing.JLabel;
 import javax.swing.JPanel;
 import javax.swing.JTextField;
 import javax.swing.KeyStroke;
@@ -43,6 +44,7 @@
         "PMD.ExcessiveImports"
 })
 public class NewControllerDialog extends AbstractDialog {
+
     private final String moduleName;
     private final Project project;
     private JPanel contentPane;
@@ -50,7 +52,6 @@ public class NewControllerDialog extends AbstractDialog {
     private JButton buttonCancel;
     private FilteredComboBox controllerAreaSelect;
     private FilteredComboBox httpMethodSelect;
-    private JTextField controllerParentDir;
     private JCheckBox inheritClass;
     private JPanel adminPanel;
     private JTextField acl;
@@ -64,12 +65,16 @@ public class NewControllerDialog extends AbstractDialog {
             message = {PhpDirectoryRule.MESSAGE, CONTROLLER_NAME})
     private JTextField controllerName;
 
-    @FieldValidation(rule = RuleRegistry.NOT_EMPTY,
-            message = {NotEmptyRule.MESSAGE, ACTION_NAME})
-    @FieldValidation(rule = RuleRegistry.PHP_CLASS,
-            message = {PhpClassRule.MESSAGE, ACTION_NAME})
+    @FieldValidation(rule = RuleRegistry.NOT_EMPTY, message = {NotEmptyRule.MESSAGE, ACTION_NAME})
+    @FieldValidation(rule = RuleRegistry.PHP_CLASS, message = {PhpClassRule.MESSAGE, ACTION_NAME})
     private JTextField actionName;
 
+    private JTextField controllerParentDir;
+
+    private JLabel actionNameErrorMessage;//NOPMD
+    private JLabel controllerNameErrorMessage;//NOPMD
+    private JLabel controllerParentDirErrorMessage;//NOPMD
+
     /**
      * Open new dialog for adding new controller.
      *
@@ -196,8 +201,8 @@ public static void open(final Project project, final PsiDirectory directory) {
     private void onOK() {
         if (validateFormFields()) {
             generateFile();
+            exit();
         }
-        exit();
     }
 
     /**
@@ -264,11 +269,6 @@ private Boolean getIsInheritClass() {
         return inheritClass.isSelected();
     }
 
-    @Override
-    protected void onCancel() {
-        dispose();
-    }
-
     private List<String> getAreaList() {
         return new ArrayList<>(
                 Arrays.asList(

From a6f19bc89d15cf765dde1504871901f4224aa045 Mon Sep 17 00:00:00 2001
From: bohdan-harniuk <b.harniuk@atwix.com>
Date: Tue, 21 Dec 2021 01:33:45 +0200
Subject: [PATCH 06/10] 868: Code refactoring

---
 .../actions/generation/NewCronjobAction.java             | 7 +++----
 .../actions/generation/dialog/NewCronjobDialog.java      | 9 +++------
 2 files changed, 6 insertions(+), 10 deletions(-)

diff --git a/src/com/magento/idea/magento2plugin/actions/generation/NewCronjobAction.java b/src/com/magento/idea/magento2plugin/actions/generation/NewCronjobAction.java
index 538df7a1e..d734549d4 100644
--- a/src/com/magento/idea/magento2plugin/actions/generation/NewCronjobAction.java
+++ b/src/com/magento/idea/magento2plugin/actions/generation/NewCronjobAction.java
@@ -17,8 +17,8 @@
 import com.magento.idea.magento2plugin.actions.generation.dialog.NewCronjobDialog;
 import org.jetbrains.annotations.NotNull;
 
-@SuppressWarnings({"PMD.FieldNamingConventions", "PMD.OnlyOneReturn"})
 public class NewCronjobAction extends AnAction {
+
     public static String ACTION_NAME = "Magento 2 Cron Job";
     public static String ACTION_DESCRIPTION = "Create a new Magento 2 Cron Job";
 
@@ -34,17 +34,16 @@ public void actionPerformed(final @NotNull AnActionEvent event) {
         if (view == null) {
             return;
         }
-
         final Project project = CommonDataKeys.PROJECT.getData(dataContext);
+
         if (project == null) {
             return;
         }
-
         final PsiDirectory directory = view.getOrChooseDirectory();
+
         if (directory == null) {
             return;
         }
-
         NewCronjobDialog.open(project, directory);
     }
 
diff --git a/src/com/magento/idea/magento2plugin/actions/generation/dialog/NewCronjobDialog.java b/src/com/magento/idea/magento2plugin/actions/generation/dialog/NewCronjobDialog.java
index dee2accf1..677c5371f 100644
--- a/src/com/magento/idea/magento2plugin/actions/generation/dialog/NewCronjobDialog.java
+++ b/src/com/magento/idea/magento2plugin/actions/generation/dialog/NewCronjobDialog.java
@@ -26,6 +26,8 @@
 import com.magento.idea.magento2plugin.ui.FilteredComboBox;
 import com.magento.idea.magento2plugin.util.CamelCaseToSnakeCase;
 import com.magento.idea.magento2plugin.util.magento.GetModuleNameByDirectoryUtil;
+import org.jetbrains.annotations.NotNull;
+
 import java.awt.event.ActionEvent;
 import java.awt.event.FocusEvent;
 import java.awt.event.FocusListener;
@@ -117,7 +119,7 @@ public class NewCronjobDialog extends AbstractDialog {
      * @param project Project
      * @param directory Directory
      */
-    public NewCronjobDialog(final Project project, final PsiDirectory directory) {
+    public NewCronjobDialog(final @NotNull Project project, final @NotNull PsiDirectory directory) {
         super();
         this.project = project;
         this.moduleName = GetModuleNameByDirectoryUtil.execute(directory, project);
@@ -246,11 +248,6 @@ public String getCronjobScheduleConfigPath() {
         return this.configPathField.getText().trim();
     }
 
-    @Override
-    protected void onCancel() {
-        dispose();
-    }
-
     private void createUIComponents() {
         final List<String> cronGroups = CronGroupIndex.getInstance(project).getGroups();
 

From 36fc25706cf7ba76a8a51d1bc0e2ccef887e11ed Mon Sep 17 00:00:00 2001
From: bohdan-harniuk <b.harniuk@atwix.com>
Date: Tue, 21 Dec 2021 01:34:34 +0200
Subject: [PATCH 07/10] 868: Code refactoring

---
 .../magento2plugin/actions/generation/NewCronjobAction.java   | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/com/magento/idea/magento2plugin/actions/generation/NewCronjobAction.java b/src/com/magento/idea/magento2plugin/actions/generation/NewCronjobAction.java
index d734549d4..948174758 100644
--- a/src/com/magento/idea/magento2plugin/actions/generation/NewCronjobAction.java
+++ b/src/com/magento/idea/magento2plugin/actions/generation/NewCronjobAction.java
@@ -19,8 +19,8 @@
 
 public class NewCronjobAction extends AnAction {
 
-    public static String ACTION_NAME = "Magento 2 Cron Job";
-    public static String ACTION_DESCRIPTION = "Create a new Magento 2 Cron Job";
+    public static final String ACTION_NAME = "Magento 2 Cron Job";
+    public static final String ACTION_DESCRIPTION = "Create a new Magento 2 Cron Job";
 
     public NewCronjobAction() {
         super(ACTION_NAME, ACTION_DESCRIPTION, MagentoIcons.MODULE);

From 715575e76ed160ddeea1dae22801ca685909b070 Mon Sep 17 00:00:00 2001
From: bohdan-harniuk <b.harniuk@atwix.com>
Date: Tue, 21 Dec 2021 01:45:41 +0200
Subject: [PATCH 08/10] 868: Enhanced error outputting for new Magento 2 Cron
 Job generation

---
 .../generation/dialog/NewCronjobDialog.form   | 70 +++++++++++++++----
 .../generation/dialog/NewCronjobDialog.java   | 20 ++++--
 2 files changed, 68 insertions(+), 22 deletions(-)

diff --git a/src/com/magento/idea/magento2plugin/actions/generation/dialog/NewCronjobDialog.form b/src/com/magento/idea/magento2plugin/actions/generation/dialog/NewCronjobDialog.form
index bfb797c3d..0b3535abe 100644
--- a/src/com/magento/idea/magento2plugin/actions/generation/dialog/NewCronjobDialog.form
+++ b/src/com/magento/idea/magento2plugin/actions/generation/dialog/NewCronjobDialog.form
@@ -3,7 +3,7 @@
   <grid id="cbd77" binding="contentPane" layout-manager="GridLayoutManager" row-count="2" column-count="1" same-size-horizontally="false" same-size-vertically="false" hgap="-1" vgap="-1">
     <margin top="10" left="10" bottom="10" right="10"/>
     <constraints>
-      <xy x="48" y="54" width="941" height="420"/>
+      <xy x="48" y="54" width="941" height="479"/>
     </constraints>
     <properties/>
     <border type="none"/>
@@ -49,7 +49,7 @@
           </grid>
         </children>
       </grid>
-      <grid id="e3588" layout-manager="GridLayoutManager" row-count="8" column-count="3" same-size-horizontally="false" same-size-vertically="false" hgap="-1" vgap="-1">
+      <grid id="e3588" layout-manager="GridLayoutManager" row-count="12" column-count="3" same-size-horizontally="false" same-size-vertically="false" hgap="-1" vgap="-1">
         <margin top="0" left="0" bottom="0" right="0"/>
         <constraints>
           <grid row="0" column="0" row-span="1" col-span="1" vsize-policy="3" hsize-policy="3" anchor="0" fill="3" indent="0" use-parent-layout="false"/>
@@ -78,7 +78,7 @@
           </component>
           <component id="e23f7" class="javax.swing.JLabel">
             <constraints>
-              <grid row="1" column="0" row-span="1" col-span="1" vsize-policy="0" hsize-policy="0" anchor="8" fill="0" indent="0" use-parent-layout="false"/>
+              <grid row="2" column="0" row-span="1" col-span="1" vsize-policy="0" hsize-policy="0" anchor="8" fill="0" indent="0" use-parent-layout="false"/>
             </constraints>
             <properties>
               <labelFor value="b2d70"/>
@@ -87,7 +87,7 @@
           </component>
           <component id="462a3" class="javax.swing.JLabel">
             <constraints>
-              <grid row="4" column="0" row-span="1" col-span="1" vsize-policy="0" hsize-policy="0" anchor="8" fill="0" indent="0" use-parent-layout="false"/>
+              <grid row="8" column="0" row-span="1" col-span="1" vsize-policy="0" hsize-policy="0" anchor="8" fill="0" indent="0" use-parent-layout="false"/>
             </constraints>
             <properties>
               <text resource-bundle="magento2/common" key="common.schedule"/>
@@ -95,12 +95,12 @@
           </component>
           <vspacer id="efb9e">
             <constraints>
-              <grid row="7" column="1" row-span="1" col-span="2" vsize-policy="6" hsize-policy="1" anchor="0" fill="2" indent="0" use-parent-layout="false"/>
+              <grid row="11" column="1" row-span="1" col-span="2" vsize-policy="6" hsize-policy="1" anchor="0" fill="2" indent="0" use-parent-layout="false"/>
             </constraints>
           </vspacer>
           <component id="93bb3" class="javax.swing.JRadioButton" binding="fixedScheduleRadioButton">
             <constraints>
-              <grid row="4" column="1" row-span="1" col-span="1" vsize-policy="0" hsize-policy="3" anchor="8" fill="0" indent="0" use-parent-layout="false"/>
+              <grid row="8" column="1" row-span="1" col-span="1" vsize-policy="0" hsize-policy="3" anchor="8" fill="0" indent="0" use-parent-layout="false"/>
             </constraints>
             <properties>
               <selected value="true"/>
@@ -109,16 +109,16 @@
           </component>
           <component id="649c" class="javax.swing.JRadioButton" binding="configurableScheduleRadioButton">
             <constraints>
-              <grid row="4" column="2" row-span="1" col-span="1" vsize-policy="0" hsize-policy="3" anchor="8" fill="0" indent="0" use-parent-layout="false"/>
+              <grid row="8" column="2" row-span="1" col-span="1" vsize-policy="0" hsize-policy="3" anchor="8" fill="0" indent="0" use-parent-layout="false"/>
             </constraints>
             <properties>
               <text resource-bundle="magento2/common" key="common.schedule.configurable"/>
             </properties>
           </component>
-          <grid id="832a2" binding="fixedSchedulePanel" layout-manager="GridLayoutManager" row-count="2" column-count="3" same-size-horizontally="false" same-size-vertically="false" hgap="-1" vgap="-1">
+          <grid id="832a2" binding="fixedSchedulePanel" layout-manager="GridLayoutManager" row-count="3" column-count="3" same-size-horizontally="false" same-size-vertically="false" hgap="-1" vgap="-1">
             <margin top="0" left="0" bottom="0" right="0"/>
             <constraints>
-              <grid row="5" column="1" row-span="1" col-span="2" vsize-policy="3" hsize-policy="3" anchor="0" fill="3" indent="0" use-parent-layout="false"/>
+              <grid row="9" column="1" row-span="1" col-span="2" vsize-policy="3" hsize-policy="3" anchor="0" fill="3" indent="0" use-parent-layout="false"/>
             </constraints>
             <properties/>
             <border type="none"/>
@@ -162,12 +162,20 @@
                   <text resource-bundle="magento2/common" key="common.schedule.atMidnight"/>
                 </properties>
               </component>
+              <component id="a207e" class="javax.swing.JLabel" binding="cronjobScheduleFieldErrorMessage">
+                <constraints>
+                  <grid row="2" column="0" row-span="1" col-span="1" vsize-policy="0" hsize-policy="0" anchor="8" fill="0" indent="0" use-parent-layout="false"/>
+                </constraints>
+                <properties>
+                  <text value=""/>
+                </properties>
+              </component>
             </children>
           </grid>
           <grid id="f84c7" binding="configurableSchedulePanel" layout-manager="GridLayoutManager" row-count="1" column-count="2" same-size-horizontally="false" same-size-vertically="false" hgap="-1" vgap="-1">
             <margin top="0" left="0" bottom="0" right="0"/>
             <constraints>
-              <grid row="6" column="1" row-span="1" col-span="2" vsize-policy="3" hsize-policy="3" anchor="0" fill="3" indent="0" use-parent-layout="false"/>
+              <grid row="10" column="1" row-span="1" col-span="2" vsize-policy="3" hsize-policy="3" anchor="0" fill="3" indent="0" use-parent-layout="false"/>
             </constraints>
             <properties>
               <visible value="false"/>
@@ -194,7 +202,7 @@
           </grid>
           <component id="122dc" class="javax.swing.JLabel">
             <constraints>
-              <grid row="3" column="0" row-span="1" col-span="1" vsize-policy="0" hsize-policy="0" anchor="8" fill="0" indent="0" use-parent-layout="false"/>
+              <grid row="6" column="0" row-span="1" col-span="1" vsize-policy="0" hsize-policy="0" anchor="8" fill="0" indent="0" use-parent-layout="false"/>
             </constraints>
             <properties>
               <text resource-bundle="magento2/common" key="common.cronGroup"/>
@@ -202,7 +210,7 @@
           </component>
           <component id="77d9d" class="com.magento.idea.magento2plugin.ui.FilteredComboBox" binding="cronGroupComboBox" custom-create="true">
             <constraints>
-              <grid row="3" column="1" row-span="1" col-span="2" vsize-policy="0" hsize-policy="2" anchor="0" fill="1" indent="0" use-parent-layout="false"/>
+              <grid row="6" column="1" row-span="1" col-span="2" vsize-policy="0" hsize-policy="2" anchor="0" fill="1" indent="0" use-parent-layout="false"/>
             </constraints>
             <properties>
               <toolTipText value=""/>
@@ -210,7 +218,7 @@
           </component>
           <component id="d14bf" class="javax.swing.JLabel">
             <constraints>
-              <grid row="2" column="0" row-span="1" col-span="1" vsize-policy="0" hsize-policy="0" anchor="8" fill="0" indent="0" use-parent-layout="false"/>
+              <grid row="4" column="0" row-span="1" col-span="1" vsize-policy="0" hsize-policy="0" anchor="8" fill="0" indent="0" use-parent-layout="false"/>
             </constraints>
             <properties>
               <text resource-bundle="magento2/common" key="common.name"/>
@@ -218,7 +226,7 @@
           </component>
           <component id="555fd" class="javax.swing.JTextField" binding="cronjobNameField">
             <constraints>
-              <grid row="2" column="1" row-span="1" col-span="2" vsize-policy="0" hsize-policy="6" anchor="8" fill="1" indent="0" use-parent-layout="false">
+              <grid row="4" column="1" row-span="1" col-span="2" vsize-policy="0" hsize-policy="6" anchor="8" fill="1" indent="0" use-parent-layout="false">
                 <preferred-size width="150" height="-1"/>
               </grid>
             </constraints>
@@ -226,7 +234,7 @@
           </component>
           <component id="b2d70" class="javax.swing.JTextField" binding="cronjobDirectoryField">
             <constraints>
-              <grid row="1" column="1" row-span="1" col-span="2" vsize-policy="0" hsize-policy="6" anchor="8" fill="1" indent="0" use-parent-layout="false">
+              <grid row="2" column="1" row-span="1" col-span="2" vsize-policy="0" hsize-policy="6" anchor="8" fill="1" indent="0" use-parent-layout="false">
                 <preferred-size width="150" height="-1"/>
               </grid>
             </constraints>
@@ -235,6 +243,38 @@
               <text value="Cron"/>
             </properties>
           </component>
+          <component id="d2f0c" class="javax.swing.JLabel" binding="cronjobClassNameFieldErrorMessage">
+            <constraints>
+              <grid row="1" column="1" row-span="1" col-span="1" vsize-policy="0" hsize-policy="0" anchor="8" fill="0" indent="0" use-parent-layout="false"/>
+            </constraints>
+            <properties>
+              <text value=""/>
+            </properties>
+          </component>
+          <component id="8a62f" class="javax.swing.JLabel" binding="cronjobDirectoryFieldErrorMessage">
+            <constraints>
+              <grid row="3" column="1" row-span="1" col-span="1" vsize-policy="0" hsize-policy="0" anchor="8" fill="0" indent="0" use-parent-layout="false"/>
+            </constraints>
+            <properties>
+              <text value=""/>
+            </properties>
+          </component>
+          <component id="d9cce" class="javax.swing.JLabel" binding="cronjobNameFieldErrorMessage">
+            <constraints>
+              <grid row="5" column="1" row-span="1" col-span="1" vsize-policy="0" hsize-policy="0" anchor="8" fill="0" indent="0" use-parent-layout="false"/>
+            </constraints>
+            <properties>
+              <text value=""/>
+            </properties>
+          </component>
+          <component id="7c57c" class="javax.swing.JLabel" binding="cronGroupComboBoxErrorMessage">
+            <constraints>
+              <grid row="7" column="1" row-span="1" col-span="1" vsize-policy="0" hsize-policy="0" anchor="8" fill="0" indent="0" use-parent-layout="false"/>
+            </constraints>
+            <properties>
+              <text value=""/>
+            </properties>
+          </component>
         </children>
       </grid>
     </children>
diff --git a/src/com/magento/idea/magento2plugin/actions/generation/dialog/NewCronjobDialog.java b/src/com/magento/idea/magento2plugin/actions/generation/dialog/NewCronjobDialog.java
index 677c5371f..c70402b1a 100644
--- a/src/com/magento/idea/magento2plugin/actions/generation/dialog/NewCronjobDialog.java
+++ b/src/com/magento/idea/magento2plugin/actions/generation/dialog/NewCronjobDialog.java
@@ -26,8 +26,6 @@
 import com.magento.idea.magento2plugin.ui.FilteredComboBox;
 import com.magento.idea.magento2plugin.util.CamelCaseToSnakeCase;
 import com.magento.idea.magento2plugin.util.magento.GetModuleNameByDirectoryUtil;
-import org.jetbrains.annotations.NotNull;
-
 import java.awt.event.ActionEvent;
 import java.awt.event.FocusEvent;
 import java.awt.event.FocusListener;
@@ -37,11 +35,13 @@
 import java.util.List;
 import javax.swing.JButton;
 import javax.swing.JComponent;
+import javax.swing.JLabel;
 import javax.swing.JOptionPane;
 import javax.swing.JPanel;
 import javax.swing.JRadioButton;
 import javax.swing.JTextField;
 import javax.swing.KeyStroke;
+import org.jetbrains.annotations.NotNull;
 
 @SuppressWarnings({
         "PMD.UncommentedEmptyMethodBody",
@@ -56,6 +56,11 @@
         "PMD.ExcessiveImports",
 })
 public class NewCronjobDialog extends AbstractDialog {
+
+    private final Project project;
+    private final String moduleName;
+    private final CamelCaseToSnakeCase camelCaseToSnakeCase;
+
     private JPanel contentPane;
     private JButton buttonOK;
     private JButton buttonCancel;
@@ -66,6 +71,7 @@ public class NewCronjobDialog extends AbstractDialog {
     private JRadioButton atMidnightRadioButton;
     private JPanel fixedSchedulePanel;
     private JPanel configurableSchedulePanel;
+
     private static final String CLASS_NAME = "class name";
     private static final String DIRECTORY = "directory";
     private static final String CRON_NAME = "name";
@@ -109,9 +115,11 @@ public class NewCronjobDialog extends AbstractDialog {
             message = {NotEmptyRule.MESSAGE, CRON_GROUP})
     private FilteredComboBox cronGroupComboBox;
 
-    private Project project;
-    private String moduleName;
-    private CamelCaseToSnakeCase camelCaseToSnakeCase;
+    private JLabel cronjobClassNameFieldErrorMessage;//NOPMD
+    private JLabel cronjobDirectoryFieldErrorMessage;//NOPMD
+    private JLabel cronjobNameFieldErrorMessage;//NOPMD
+    private JLabel cronGroupComboBoxErrorMessage;//NOPMD
+    private JLabel cronjobScheduleFieldErrorMessage;//NOPMD
 
     /**
      * Open a new cronjob generation dialog form.
@@ -282,10 +290,8 @@ private String suggestCronjobName(final String cronjobClassname) {
      */
     private void onOK() {
         if (!validateFormFields()) {
-            exit();
             return;
         }
-
         final NamespaceBuilder namespaceBuilder = new NamespaceBuilder(
                 this.getCronjobModule(),
                 this.getCronjobClassName(),

From abb9b107a62cf3a52c18d4f0edc35e27e3a7d6f7 Mon Sep 17 00:00:00 2001
From: bohdan-harniuk <b.harniuk@atwix.com>
Date: Tue, 21 Dec 2021 02:01:10 +0200
Subject: [PATCH 09/10] 868: Enhanced error outputting for new Magento 2 Cron
 Job generation

---
 .../generation/dialog/NewControllerDialog.form |  4 ++--
 .../generation/dialog/NewCronjobDialog.form    | 18 ++++++++++++++----
 .../generation/dialog/NewCronjobDialog.java    |  1 +
 3 files changed, 17 insertions(+), 6 deletions(-)

diff --git a/src/com/magento/idea/magento2plugin/actions/generation/dialog/NewControllerDialog.form b/src/com/magento/idea/magento2plugin/actions/generation/dialog/NewControllerDialog.form
index 3d6b85fc7..5abf1a5be 100644
--- a/src/com/magento/idea/magento2plugin/actions/generation/dialog/NewControllerDialog.form
+++ b/src/com/magento/idea/magento2plugin/actions/generation/dialog/NewControllerDialog.form
@@ -142,7 +142,7 @@
           <grid row="1" column="0" row-span="1" col-span="1" vsize-policy="0" hsize-policy="0" anchor="8" fill="0" indent="0" use-parent-layout="false"/>
         </constraints>
         <properties>
-          <editable value="true"/>
+          <editable value="false"/>
           <toolTipText value=""/>
         </properties>
         <clientProperties>
@@ -170,7 +170,7 @@
           <grid row="3" column="0" row-span="1" col-span="1" vsize-policy="0" hsize-policy="0" anchor="8" fill="0" indent="0" use-parent-layout="false"/>
         </constraints>
         <properties>
-          <editable value="true"/>
+          <editable value="false"/>
           <toolTipText value=""/>
         </properties>
         <clientProperties>
diff --git a/src/com/magento/idea/magento2plugin/actions/generation/dialog/NewCronjobDialog.form b/src/com/magento/idea/magento2plugin/actions/generation/dialog/NewCronjobDialog.form
index 0b3535abe..1da079a05 100644
--- a/src/com/magento/idea/magento2plugin/actions/generation/dialog/NewCronjobDialog.form
+++ b/src/com/magento/idea/magento2plugin/actions/generation/dialog/NewCronjobDialog.form
@@ -3,9 +3,11 @@
   <grid id="cbd77" binding="contentPane" layout-manager="GridLayoutManager" row-count="2" column-count="1" same-size-horizontally="false" same-size-vertically="false" hgap="-1" vgap="-1">
     <margin top="10" left="10" bottom="10" right="10"/>
     <constraints>
-      <xy x="48" y="54" width="941" height="479"/>
+      <xy x="48" y="54" width="941" height="504"/>
     </constraints>
-    <properties/>
+    <properties>
+      <preferredSize width="550" height="350"/>
+    </properties>
     <border type="none"/>
     <children>
       <grid id="94766" layout-manager="GridLayoutManager" row-count="1" column-count="2" same-size-horizontally="false" same-size-vertically="false" hgap="-1" vgap="-1">
@@ -49,7 +51,7 @@
           </grid>
         </children>
       </grid>
-      <grid id="e3588" layout-manager="GridLayoutManager" row-count="12" column-count="3" same-size-horizontally="false" same-size-vertically="false" hgap="-1" vgap="-1">
+      <grid id="e3588" layout-manager="GridLayoutManager" row-count="13" column-count="3" same-size-horizontally="false" same-size-vertically="false" hgap="-1" vgap="-1">
         <margin top="0" left="0" bottom="0" right="0"/>
         <constraints>
           <grid row="0" column="0" row-span="1" col-span="1" vsize-policy="3" hsize-policy="3" anchor="0" fill="3" indent="0" use-parent-layout="false"/>
@@ -95,7 +97,7 @@
           </component>
           <vspacer id="efb9e">
             <constraints>
-              <grid row="11" column="1" row-span="1" col-span="2" vsize-policy="6" hsize-policy="1" anchor="0" fill="2" indent="0" use-parent-layout="false"/>
+              <grid row="12" column="1" row-span="1" col-span="2" vsize-policy="6" hsize-policy="1" anchor="0" fill="2" indent="0" use-parent-layout="false"/>
             </constraints>
           </vspacer>
           <component id="93bb3" class="javax.swing.JRadioButton" binding="fixedScheduleRadioButton">
@@ -275,6 +277,14 @@
               <text value=""/>
             </properties>
           </component>
+          <component id="38172" class="javax.swing.JLabel" binding="configPathFieldErrorMessage">
+            <constraints>
+              <grid row="11" column="2" row-span="1" col-span="1" vsize-policy="0" hsize-policy="0" anchor="8" fill="0" indent="0" use-parent-layout="false"/>
+            </constraints>
+            <properties>
+              <text value=""/>
+            </properties>
+          </component>
         </children>
       </grid>
     </children>
diff --git a/src/com/magento/idea/magento2plugin/actions/generation/dialog/NewCronjobDialog.java b/src/com/magento/idea/magento2plugin/actions/generation/dialog/NewCronjobDialog.java
index c70402b1a..5d072ae1b 100644
--- a/src/com/magento/idea/magento2plugin/actions/generation/dialog/NewCronjobDialog.java
+++ b/src/com/magento/idea/magento2plugin/actions/generation/dialog/NewCronjobDialog.java
@@ -120,6 +120,7 @@ public class NewCronjobDialog extends AbstractDialog {
     private JLabel cronjobNameFieldErrorMessage;//NOPMD
     private JLabel cronGroupComboBoxErrorMessage;//NOPMD
     private JLabel cronjobScheduleFieldErrorMessage;//NOPMD
+    private JLabel configPathFieldErrorMessage;
 
     /**
      * Open a new cronjob generation dialog form.

From 3ca91dc94d580feae771c6e4337374a52ec24607 Mon Sep 17 00:00:00 2001
From: bohdan-harniuk <b.harniuk@atwix.com>
Date: Tue, 21 Dec 2021 09:32:38 +0200
Subject: [PATCH 10/10] 868: Fixed code style check

---
 .../actions/generation/dialog/NewCronjobDialog.java             | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/com/magento/idea/magento2plugin/actions/generation/dialog/NewCronjobDialog.java b/src/com/magento/idea/magento2plugin/actions/generation/dialog/NewCronjobDialog.java
index 5d072ae1b..a8f71b0f1 100644
--- a/src/com/magento/idea/magento2plugin/actions/generation/dialog/NewCronjobDialog.java
+++ b/src/com/magento/idea/magento2plugin/actions/generation/dialog/NewCronjobDialog.java
@@ -120,7 +120,7 @@ public class NewCronjobDialog extends AbstractDialog {
     private JLabel cronjobNameFieldErrorMessage;//NOPMD
     private JLabel cronGroupComboBoxErrorMessage;//NOPMD
     private JLabel cronjobScheduleFieldErrorMessage;//NOPMD
-    private JLabel configPathFieldErrorMessage;
+    private JLabel configPathFieldErrorMessage;//NOPMD
 
     /**
      * Open a new cronjob generation dialog form.