Skip to content

Commit d0780f2

Browse files
author
Vitaliy
authored
Merge pull request #557 from Iamwade/static-fixes-for-validation-namespace
Fixed copyrights, indentation and added @NotNull to methods
2 parents 3692f2e + b586c89 commit d0780f2

27 files changed

+93
-26
lines changed

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

Lines changed: 7 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -23,8 +23,10 @@
2323
import com.magento.idea.magento2plugin.util.xml.XmlPsiTreeUtil;
2424
import java.util.Collection;
2525
import javax.swing.JOptionPane;
26+
import org.jetbrains.annotations.NotNull;
2627

2728
public class NewEmailTemplateDialogValidator {
29+
2830
private final ValidatorBundle validatorBundle;
2931
private final CommonBundle commonBundle;
3032
private final Project project;
@@ -44,7 +46,7 @@ public NewEmailTemplateDialogValidator(final Project project) {
4446
* @param dialog NewEmailTemplateDialog
4547
* @return boolean
4648
*/
47-
public boolean validate(final NewEmailTemplateDialog dialog) {
49+
public boolean validate(final @NotNull NewEmailTemplateDialog dialog) {
4850
final EmailTemplatesXmlData emailTemplatesXmlData = dialog.getEmailTemplateData();
4951
final String errorTitle = commonBundle.message("common.error");
5052
final EmailTemplateHtmlData emailTemplateHtmlData = dialog.getEmailTemplateHtmlData();
@@ -92,7 +94,9 @@ public boolean validate(final NewEmailTemplateDialog dialog) {
9294
*
9395
* @return bool
9496
*/
95-
private boolean isTemplateTagAlreadyExists(final EmailTemplatesXmlData emailTemplatesXmlData) {
97+
private boolean isTemplateTagAlreadyExists(
98+
final @NotNull EmailTemplatesXmlData emailTemplatesXmlData
99+
) {
96100
final EmailTemplatesXml emailTemplatesXml = new EmailTemplatesXml();
97101
final XmlFile emailTemplatesFile = (XmlFile) FileBasedIndexUtil.findModuleConfigFile(
98102
emailTemplatesXml.getFileName(),
@@ -134,7 +138,7 @@ private boolean isTemplateTagAlreadyExists(final EmailTemplatesXmlData emailTemp
134138
* @return boolean
135139
*/
136140
private boolean isTemplateFileAlreadyExists(
137-
final EmailTemplateHtmlData emailTemplateData
141+
final @NotNull EmailTemplateHtmlData emailTemplateData
138142
) {
139143
final PsiFile templateFile = FileBasedIndexUtil.findModuleViewFile(
140144
emailTemplateData.getFileName(),

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

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,13 +6,15 @@
66
package com.magento.idea.magento2plugin.actions.generation.dialog.validator.rule;
77

88
import com.magento.idea.magento2plugin.util.RegExUtil;
9+
import org.jetbrains.annotations.NotNull;
910

1011
public class AclResourceIdRule implements ValidationRule {
12+
1113
public static final String MESSAGE = "validator.magentoAclResourceIdInvalid";
1214
private static final ValidationRule INSTANCE = new AclResourceIdRule();
1315

1416
@Override
15-
public boolean check(final String value) {
17+
public boolean check(final @NotNull String value) {
1618
return value.matches(RegExUtil.Magento.XML_IDENTIFIER);
1719
}
1820

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

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,13 +6,15 @@
66
package com.magento.idea.magento2plugin.actions.generation.dialog.validator.rule;
77

88
import com.magento.idea.magento2plugin.util.RegExUtil;
9+
import org.jetbrains.annotations.NotNull;
910

1011
public class AlphaWithDashRule implements ValidationRule {
12+
1113
public static final String MESSAGE = "validator.alphaAndDashCharacters";
1214
private static final ValidationRule INSTANCE = new AlphaWithDashRule();
1315

1416
@Override
15-
public boolean check(final String value) {
17+
public boolean check(final @NotNull String value) {
1618
return value.matches(RegExUtil.ALPHA_WITH_DASH);
1719
}
1820

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

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,13 +6,15 @@
66
package com.magento.idea.magento2plugin.actions.generation.dialog.validator.rule;
77

88
import com.magento.idea.magento2plugin.util.RegExUtil;
9+
import org.jetbrains.annotations.NotNull;
910

1011
public class AlphaWithPeriodRule implements ValidationRule {
12+
1113
public static final String MESSAGE = "validator.alphaAndPeriodCharacters";
1214
private static final ValidationRule INSTANCE = new AlphaWithPeriodRule();
1315

1416
@Override
15-
public boolean check(final String value) {
17+
public boolean check(final @NotNull String value) {
1618
return value.matches(RegExUtil.ALPHA_WITH_PERIOD);
1719
}
1820

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

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,13 +6,15 @@
66
package com.magento.idea.magento2plugin.actions.generation.dialog.validator.rule;
77

88
import com.magento.idea.magento2plugin.util.RegExUtil;
9+
import org.jetbrains.annotations.NotNull;
910

1011
public class AlphanumericRule implements ValidationRule {
12+
1113
public static final String MESSAGE = "validator.alphaNumericCharacters";
1214
private static final ValidationRule INSTANCE = new AlphanumericRule();
1315

1416
@Override
15-
public boolean check(final String value) {
17+
public boolean check(final @NotNull String value) {
1618
return value.matches(RegExUtil.ALPHANUMERIC);
1719
}
1820

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

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,13 +6,15 @@
66
package com.magento.idea.magento2plugin.actions.generation.dialog.validator.rule;
77

88
import com.magento.idea.magento2plugin.util.RegExUtil;
9+
import org.jetbrains.annotations.NotNull;
910

1011
public class AlphanumericWithUnderscoreRule implements ValidationRule {
12+
1113
public static final String MESSAGE = "validator.alphaNumericAndUnderscoreCharacters";
1214
private static final ValidationRule INSTANCE = new AlphanumericWithUnderscoreRule();
1315

1416
@Override
15-
public boolean check(final String value) {
17+
public boolean check(final @NotNull String value) {
1618
return value.matches(RegExUtil.ALPHANUMERIC_WITH_UNDERSCORE);
1719
}
1820

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

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,12 +5,15 @@
55

66
package com.magento.idea.magento2plugin.actions.generation.dialog.validator.rule;
77

8+
import org.jetbrains.annotations.NotNull;
9+
810
public class BoxNotEmptyRule implements ValidationRule {
11+
912
public static final String MESSAGE = "validator.box.notEmpty";
1013
private static final ValidationRule INSTANCE = new BoxNotEmptyRule();
1114

1215
@Override
13-
public boolean check(final String value) {
16+
public boolean check(final @NotNull String value) {
1417
return !value.isEmpty();
1518
}
1619

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

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,13 +6,15 @@
66
package com.magento.idea.magento2plugin.actions.generation.dialog.validator.rule;
77

88
import com.magento.idea.magento2plugin.util.RegExUtil;
9+
import org.jetbrains.annotations.NotNull;
910

1011
public class CliCommandRule implements ValidationRule {
12+
1113
public static final String MESSAGE = "validator.command.isNotValid";
1214
private static final ValidationRule INSTANCE = new CliCommandRule();
1315

1416
@Override
15-
public boolean check(final String value) {
17+
public boolean check(final @NotNull String value) {
1618
return value.matches(RegExUtil.CLI_COMMAND_NAME);
1719
}
1820

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

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,13 +6,15 @@
66
package com.magento.idea.magento2plugin.actions.generation.dialog.validator.rule;
77

88
import com.magento.idea.magento2plugin.util.RegExUtil;
9+
import org.jetbrains.annotations.NotNull;
910

1011
public class ConfigPathRule implements ValidationRule {
12+
1113
public static final String MESSAGE = "validator.configPath.invalidFormat";
1214
private static final ValidationRule INSTANCE = new ConfigPathRule();
1315

1416
@Override
15-
public boolean check(final String value) {
17+
public boolean check(final @NotNull String value) {
1618
return value.matches(RegExUtil.Magento.CONFIG_PATH);
1719
}
1820

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

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,13 +6,15 @@
66
package com.magento.idea.magento2plugin.actions.generation.dialog.validator.rule;
77

88
import com.magento.idea.magento2plugin.util.RegExUtil;
9+
import org.jetbrains.annotations.NotNull;
910

1011
public class CronScheduleRule implements ValidationRule {
12+
1113
public static final String MESSAGE = "validator.cronSchedule.invalidExpression";
1214
private static final ValidationRule INSTANCE = new CronScheduleRule();
1315

1416
@Override
15-
public boolean check(final String value) {
17+
public boolean check(final @NotNull String value) {
1618
return value.matches(RegExUtil.Magento.CRON_SCHEDULE);
1719
}
1820

0 commit comments

Comments
 (0)