Skip to content

Commit 2f7f163

Browse files
committed
Implementing the validation logic to all generators
1 parent 3ab4345 commit 2f7f163

File tree

7 files changed

+45
-12
lines changed

7 files changed

+45
-12
lines changed
Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,8 @@
11
validator.notEmpty={0} must not be empty
2-
validator.package.validPath=Please specify a valid Magento 2 installation path!
2+
validator.package.validPath=Please specify a valid Magento 2 installation path
33
validator.alphaNumericCharacters={0} must contain letters and numbers only
44
validator.startWithNumberOrCapitalLetter={0} must start from a number or a capital letter
55
validator.directory.isNotValid={0} is not valid
6-
validator.module.noSuchModule=No such module {0}.
6+
validator.module.noSuchModule=No such module {0}
7+
validator.file.alreadyExists={0} already exists
8+
validator.file.cantBeCreated={0} can't be created

src/com/magento/idea/magento2plugin/actions/generation/generator/ModuleBlockClassGenerator.java

Lines changed: 10 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -17,6 +17,7 @@
1717
import com.magento.idea.magento2plugin.magento.files.PhpPreference;
1818
import com.magento.idea.magento2plugin.magento.packages.Package;
1919
import com.magento.idea.magento2plugin.util.GetPhpClassByFQN;
20+
import com.magento.idea.magento2plugin.validators.ValidatorBundle;
2021
import org.jetbrains.annotations.NotNull;
2122
import javax.swing.*;
2223
import java.io.File;
@@ -37,14 +38,21 @@ public ModuleBlockClassGenerator(@NotNull BlockFileData blockFileData, Project p
3738
}
3839

3940
public PsiFile generate(String actionName) {
41+
String errorTitle = "Error";
4042
PhpClass block = GetPhpClassByFQN.getInstance(project).execute(getBlockFqn());
43+
4144
if (block != null) {
42-
JOptionPane.showMessageDialog(null, "Block Class already exist!", "Error", JOptionPane.ERROR_MESSAGE);
45+
String errorMessage = ValidatorBundle.message("validator.file.alreadyExists", "Block Class");
46+
JOptionPane.showMessageDialog(null, errorMessage, errorTitle, JOptionPane.ERROR_MESSAGE);
47+
4348
return null;
4449
}
50+
4551
PhpFile blockFile = createBlockClass(actionName);
4652
if (blockFile == null) {
47-
JOptionPane.showMessageDialog(null, "Block Class cant be created!", "Error", JOptionPane.ERROR_MESSAGE);
53+
String errorMessage = ValidatorBundle.message("validator.file.cantBeCreated", "Block Class");
54+
JOptionPane.showMessageDialog(null, errorMessage, errorTitle, JOptionPane.ERROR_MESSAGE);
55+
4856
return null;
4957
}
5058

src/com/magento/idea/magento2plugin/actions/generation/generator/ModuleGraphQlResolverClassGenerator.java

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -25,6 +25,7 @@
2525
import com.magento.idea.magento2plugin.magento.packages.MagentoPhpClass;
2626
import com.magento.idea.magento2plugin.util.GetFirstClassOfFile;
2727
import com.magento.idea.magento2plugin.util.GetPhpClassByFQN;
28+
import com.magento.idea.magento2plugin.validators.ValidatorBundle;
2829
import org.jetbrains.annotations.NotNull;
2930

3031
import javax.swing.*;
@@ -58,12 +59,14 @@ public PsiFile generate(String actionName) {
5859
}
5960

6061
if (graphQlResolverClass == null) {
62+
String errorMessage = ValidatorBundle.message("validator.file.cantBeCreated", "GraphQL Resolver Class");
6163
JOptionPane.showMessageDialog(
6264
null,
63-
"GraphQL Resolver Class can't be created!",
65+
errorMessage,
6466
"Error",
6567
JOptionPane.ERROR_MESSAGE
6668
);
69+
6770
return;
6871
}
6972

src/com/magento/idea/magento2plugin/actions/generation/generator/ModuleViewModelClassGenerator.java

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -16,6 +16,7 @@
1616
import com.magento.idea.magento2plugin.magento.files.ViewModelPhp;
1717
import com.magento.idea.magento2plugin.magento.packages.Package;
1818
import com.magento.idea.magento2plugin.util.GetPhpClassByFQN;
19+
import com.magento.idea.magento2plugin.validators.ValidatorBundle;
1920
import org.jetbrains.annotations.NotNull;
2021
import javax.swing.*;
2122
import java.io.File;
@@ -38,12 +39,17 @@ public ModuleViewModelClassGenerator(@NotNull ViewModelFileData viewModelFileDat
3839
public PsiFile generate(String actionName) {
3940
PhpClass block = GetPhpClassByFQN.getInstance(project).execute(getViewModelFqn());
4041
if (block != null) {
41-
JOptionPane.showMessageDialog(null, "View Model already exist!", "Error", JOptionPane.ERROR_MESSAGE);
42+
String errorMessage = ValidatorBundle.message("validator.file.alreadyExists", "View Model");
43+
JOptionPane.showMessageDialog(null, errorMessage, "Error", JOptionPane.ERROR_MESSAGE);
44+
4245
return null;
4346
}
47+
4448
PhpFile viewModelFile = createViewModelClass(actionName);
4549
if (viewModelFile == null) {
46-
JOptionPane.showMessageDialog(null, "View Model can't be created!", "Error", JOptionPane.ERROR_MESSAGE);
50+
String errorMessage = ValidatorBundle.message("validator.file.cantBeCreated", "View Model");
51+
JOptionPane.showMessageDialog(null, errorMessage, "Error", JOptionPane.ERROR_MESSAGE);
52+
4753
return null;
4854
}
4955

src/com/magento/idea/magento2plugin/actions/generation/generator/ObserverClassGenerator.java

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -21,6 +21,7 @@
2121
import com.magento.idea.magento2plugin.magento.packages.MagentoPhpClass;
2222
import com.magento.idea.magento2plugin.util.GetFirstClassOfFile;
2323
import com.magento.idea.magento2plugin.util.GetPhpClassByFQN;
24+
import com.magento.idea.magento2plugin.validators.ValidatorBundle;
2425
import javax.swing.*;
2526
import java.io.File;
2627
import java.util.Properties;
@@ -54,12 +55,14 @@ public PsiFile generate(String actionName) {
5455
}
5556

5657
if (observerClass == null) {
58+
String errorMessage = ValidatorBundle.message("validator.file.cantBeCreated", "Observer Class");
5759
JOptionPane.showMessageDialog(
5860
null,
59-
"Observer Class can't be created!",
61+
errorMessage,
6062
"Error",
6163
JOptionPane.ERROR_MESSAGE
6264
);
65+
6366
return;
6467
}
6568

src/com/magento/idea/magento2plugin/actions/generation/generator/PluginClassGenerator.java

Lines changed: 9 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -34,6 +34,7 @@
3434
import com.magento.idea.magento2plugin.magento.packages.MagentoPhpClass;
3535
import com.magento.idea.magento2plugin.util.GetFirstClassOfFile;
3636
import com.magento.idea.magento2plugin.util.GetPhpClassByFQN;
37+
import com.magento.idea.magento2plugin.validators.ValidatorBundle;
3738
import gnu.trove.THashSet;
3839
import org.jetbrains.annotations.NotNull;
3940
import javax.swing.*;
@@ -72,7 +73,9 @@ public PsiFile generate(String actionName)
7273
pluginClass = createPluginClass(actionName);
7374
}
7475
if (pluginClass == null) {
75-
JOptionPane.showMessageDialog(null, "Plugin Class cant be created!", "Error", JOptionPane.ERROR_MESSAGE);
76+
String errorMessage = ValidatorBundle.message("validator.file.cantBeCreated", "Plugin Class");
77+
JOptionPane.showMessageDialog(null, errorMessage, "Error", JOptionPane.ERROR_MESSAGE);
78+
7679
return;
7780
}
7881

@@ -82,8 +85,10 @@ public PsiFile generate(String actionName)
8285
PluginMethodsGenerator pluginGenerator = new PluginMethodsGenerator(pluginClass, targetMethod, targetClassKey);
8386

8487
PluginMethodData[] pluginMethodData = pluginGenerator.createPluginMethods(getPluginType());
85-
if (checkIfMethodExist(pluginClass, pluginMethodData)){
86-
JOptionPane.showMessageDialog(null, "Plugin method already exist!", "Error", JOptionPane.ERROR_MESSAGE);
88+
if (checkIfMethodExist(pluginClass, pluginMethodData)) {
89+
String errorMessage = ValidatorBundle.message("validator.file.alreadyExists", "Plugin Class");
90+
JOptionPane.showMessageDialog(null, errorMessage, "Error", JOptionPane.ERROR_MESSAGE);
91+
8792
return;
8893
}
8994

@@ -116,6 +121,7 @@ public PsiFile generate(String actionName)
116121
}
117122
codeStyleSettings.restore();
118123
});
124+
119125
return pluginFile[0];
120126
}
121127

src/com/magento/idea/magento2plugin/actions/generation/generator/PreferenceClassGenerator.java

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,6 +16,7 @@
1616
import com.magento.idea.magento2plugin.magento.files.PhpPreference;
1717
import com.magento.idea.magento2plugin.util.GetFirstClassOfFile;
1818
import com.magento.idea.magento2plugin.util.GetPhpClassByFQN;
19+
import com.magento.idea.magento2plugin.validators.ValidatorBundle;
1920
import org.jetbrains.annotations.NotNull;
2021
import javax.swing.*;
2122
import java.io.File;
@@ -39,11 +40,15 @@ public PreferenceClassGenerator(@NotNull PreferenceFileData preferenceFileData,
3940

4041
public PsiFile generate(String actionName) {
4142
PhpClass pluginClass = GetPhpClassByFQN.getInstance(project).execute(preferenceFileData.getPreferenceFqn());
43+
4244
if (pluginClass == null) {
4345
pluginClass = createPluginClass(actionName);
4446
}
47+
4548
if (pluginClass == null) {
46-
JOptionPane.showMessageDialog(null, "Preference Class cant be created!", "Error", JOptionPane.ERROR_MESSAGE);
49+
String errorMessage = ValidatorBundle.message("validator.file.cantBeCreated", "Preference Class");
50+
JOptionPane.showMessageDialog(null, errorMessage, "Error", JOptionPane.ERROR_MESSAGE);
51+
4752
return null;
4853
}
4954

0 commit comments

Comments
 (0)