Skip to content

Commit 445727c

Browse files
author
Vitaliy Boyko
committed
Merge branch '4.0.0-develop' of github.com:magento/magento2-phpstorm-plugin into web-api-checks
2 parents 1fc0843 + 0889543 commit 445727c

29 files changed

+102
-28
lines changed

CHANGELOG.md

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,13 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0).
66

77
## 4.0.0
88

9+
## 3.2.1
10+
11+
### Fixed
12+
13+
- Directory validator
14+
- Entity data mapper file template
15+
916
## 3.2.0
1017

1118
### Added

build.gradle

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@
44
*/
55

66
plugins {
7-
id 'org.jetbrains.intellij' version '0.6.5'
7+
id 'org.jetbrains.intellij' version '0.7.2'
88
id 'checkstyle'
99
id 'pmd'
1010
id 'org.jetbrains.changelog' version '0.6.2'

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

0 commit comments

Comments
 (0)