Removing unnecessary validations

Author: eduard13

src/com/magento/idea/magento2plugin/actions/generation/dialog/validator/NewCronGroupValidator.java

@@ -49,13 +49,7 @@ public NewCronGroupValidator() {
     public boolean validate(final NewCronGroupDialog dialog) {
         final CronGroupXmlData cronGroupXmlData = dialog.getCronGroupXmlData();
 
-        return this.isGroupNameValid(cronGroupXmlData)
-                && isScheduleGenerateEveryValid(cronGroupXmlData)
-                && isScheduleAheadForValid(cronGroupXmlData)
-                && isScheduleLifetimeValid(cronGroupXmlData)
-                && isHistoryCleanupEveryValid(cronGroupXmlData)
-                && isHistorySuccessLifetimeValid(cronGroupXmlData)
-                && isHistoryFailureLifetimeValid(cronGroupXmlData);
+        return this.isGroupNameValid(cronGroupXmlData);
     }
 
     private boolean isGroupNameValid(final CronGroupXmlData cronGroupXmlData) {
@@ -94,144 +88,6 @@ private boolean isGroupNameValid(final CronGroupXmlData cronGroupXmlData) {
         return true;
     }
 
-    private boolean isScheduleGenerateEveryValid(final CronGroupXmlData cronGroupXmlData) {
-        if (cronGroupXmlData.getScheduleGenerateEvery() == null) {
-            return true;
-        }
-
-        if (cronGroupXmlData.getScheduleGenerateEvery() <= 0) {
-            final String errorMessage = validatorBundle.message(
-                    MUST_NOT_BE_NEGATIVE,
-                    "schedule_generate_every option"
-            );
-            JOptionPane.showMessageDialog(
-                    null,
-                    errorMessage,
-                    getErrorTitle(),
-                    JOptionPane.ERROR_MESSAGE
-            );
-
-            return false;
-        }
-
-        return true;
-    }
-
-    private boolean isScheduleAheadForValid(final CronGroupXmlData cronGroupXmlData) {
-        if (cronGroupXmlData.getScheduleAheadFor() == null) {
-            return true;
-        }
-
-        if (cronGroupXmlData.getScheduleAheadFor() <= 0) {
-            final String errorMessage = validatorBundle.message(
-                    MUST_NOT_BE_NEGATIVE,
-                    "schedule_ahead_for option"
-            );
-            JOptionPane.showMessageDialog(
-                    null,
-                    errorMessage,
-                    getErrorTitle(),
-                    JOptionPane.ERROR_MESSAGE
-            );
-
-            return false;
-        }
-
-        return true;
-    }
-
-    private boolean isScheduleLifetimeValid(final CronGroupXmlData cronGroupXmlData) {
-        if (cronGroupXmlData.getScheduleLifetime() == null) {
-            return true;
-        }
-
-        if (cronGroupXmlData.getScheduleLifetime() <= 0) {
-            final String errorMessage = validatorBundle.message(
-                    MUST_NOT_BE_NEGATIVE,
-                    "schedule_lifetime option"
-            );
-            JOptionPane.showMessageDialog(
-                    null,
-                    errorMessage,
-                    getErrorTitle(),
-                    JOptionPane.ERROR_MESSAGE
-            );
-
-            return false;
-        }
-
-        return true;
-    }
-
-    private boolean isHistoryCleanupEveryValid(final CronGroupXmlData cronGroupXmlData) {
-        if (cronGroupXmlData.getHistoryCleanupEvery() == null) {
-            return true;
-        }
-
-        if (cronGroupXmlData.getHistoryCleanupEvery() <= 0) {
-            final String errorMessage = validatorBundle.message(
-                    MUST_NOT_BE_NEGATIVE,
-                    "history_cleanup_every option"
-            );
-            JOptionPane.showMessageDialog(
-                    null,
-                    errorMessage,
-                    getErrorTitle(),
-                    JOptionPane.ERROR_MESSAGE
-            );
-
-            return false;
-        }
-
-        return true;
-    }
-
-    private boolean isHistorySuccessLifetimeValid(final CronGroupXmlData cronGroupXmlData) {
-        if (cronGroupXmlData.getHistorySuccessLifetime() == null) {
-            return true;
-        }
-
-        if (cronGroupXmlData.getHistorySuccessLifetime() <= 0) {
-            final String errorMessage = validatorBundle.message(
-                    MUST_NOT_BE_NEGATIVE,
-                    "history_success_lifetime option"
-            );
-            JOptionPane.showMessageDialog(
-                    null,
-                    errorMessage,
-                    getErrorTitle(),
-                    JOptionPane.ERROR_MESSAGE
-            );
-
-            return false;
-        }
-
-        return true;
-    }
-
-    private boolean isHistoryFailureLifetimeValid(final CronGroupXmlData cronGroupXmlData) {
-        if (cronGroupXmlData.getHistoryFailureLifetime() == null) {
-            return true;
-        }
-
-        if (cronGroupXmlData.getHistoryFailureLifetime() <= 0) {
-            final String errorMessage = validatorBundle.message(
-                    MUST_NOT_BE_NEGATIVE,
-                    "history_failure_lifetime option"
-            );
-            JOptionPane.showMessageDialog(
-                    null,
-                    errorMessage,
-                    getErrorTitle(),
-                    JOptionPane.ERROR_MESSAGE
-            );
-
-            return false;
-        }
-
-        return true;
-    }
-
     private String getErrorTitle() {
         return commonBundle.message("common.error");
     }