267: static fixes

Author: Vitaliy Boyko

src/com/magento/idea/magento2plugin/actions/generation/data/ModuleXmlData.java

@@ -7,6 +7,7 @@
 
 import com.intellij.psi.PsiDirectory;
 
+@SuppressWarnings({"PMD.DataClass"})
 public class ModuleXmlData {
     private final String packageName;
     private final String moduleName;
@@ -53,7 +54,7 @@ public String getSetupVersion() {
         return this.setupVersion;
     }
 
-    public boolean getCreateModuleDirs() {
+    public boolean isCreateModuleDirs() {
         return this.createModuleDirs;
     }
 }

src/com/magento/idea/magento2plugin/actions/generation/dialog/NewModuleDialog.java

@@ -22,7 +22,7 @@
 import com.magento.idea.magento2plugin.magento.packages.Package;
 import com.magento.idea.magento2plugin.project.Settings;
 import com.magento.idea.magento2plugin.util.CamelCaseToHyphen;
-import com.magento.idea.magento2plugin.util.magento.MagentoVersion;
+import com.magento.idea.magento2plugin.util.magento.MagentoVersionUtil;
 import java.awt.event.ActionEvent;
 import java.awt.event.ActionListener;
 import java.awt.event.KeyEvent;
@@ -244,7 +244,7 @@ public String getModuleVersion() {
      */
     public String getSetupVersion() {
         final String magentoVersion = getMagentoVersion();
-        if (!MagentoVersion.compare(magentoVersion, MAGENTO_BEFORE_DECLARATIVE_SCHEMA_VERSION)) {
+        if (!MagentoVersionUtil.compare(magentoVersion, MAGENTO_BEFORE_DECLARATIVE_SCHEMA_VERSION)) {
             return this.moduleVersion.getText().trim();
         }
         return null;

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

@@ -41,7 +41,7 @@ public ModuleXmlGenerator(
 
     @Override
     public PsiFile generate(final String actionName) {
-        if (moduleXmlData.getCreateModuleDirs()) {
+        if (moduleXmlData.isCreateModuleDirs()) {
             final ModuleDirectoriesData moduleDirectoriesData = directoryGenerator
                     .createOrFindModuleDirectories(
                             moduleXmlData.getPackageName(),

src/com/magento/idea/magento2plugin/generation/php/MagentoModuleGenerator.java

@@ -33,13 +33,13 @@
 
 
 public class MagentoModuleGenerator extends WebProjectTemplate<MagentoProjectGeneratorSettings> {
-    public static String ACTION_NAME = "Magento 2 Module";
+    public static String actionName = "Magento 2 Module";
 
     @Nls
     @NotNull
     @Override
     public String getName() {
-        return ACTION_NAME;
+        return actionName;
     }
 
     @Override
@@ -124,7 +124,7 @@ private PsiFile generateComposerJson(
                 settings.getModuleLicenses(),
                 new ArrayList<>(),
                 false
-        ), project).generate(ACTION_NAME);
+        ), project).generate(actionName);
     }
 
     /**
@@ -145,7 +145,7 @@ private PsiFile generateRegistrationPhp(
                 settings.getModuleName(),
                 baseDir,
                 false
-        ), project).generate(ACTION_NAME);
+        ), project).generate(actionName);
     }
 
     /**
@@ -169,7 +169,7 @@ private void generateModuleXml(
         );
         final ModuleXmlGenerator moduleXmlGenerator =
                 new ModuleXmlGenerator(moduleXmlData, project);
-        moduleXmlGenerator.generate(ACTION_NAME, true);
+        moduleXmlGenerator.generate(actionName, true);
     }
 
     @Nullable

src/com/magento/idea/magento2plugin/project/ProjectDetector.java

@@ -19,7 +19,7 @@
 import com.magento.idea.magento2plugin.indexes.IndexManager;
 import com.magento.idea.magento2plugin.magento.packages.MagentoComponentManager;
 import com.magento.idea.magento2plugin.util.magento.MagentoBasePathUtil;
-import com.magento.idea.magento2plugin.util.magento.MagentoVersion;
+import com.magento.idea.magento2plugin.util.magento.MagentoVersionUtil;
 import javax.swing.event.HyperlinkEvent;
 import org.jetbrains.annotations.NotNull;
 
@@ -48,7 +48,7 @@ public void hyperlinkActivated(
                                 settings.pluginEnabled = true;
                                 settings.mftfSupportEnabled = true;
                                 settings.magentoPath = project.getBasePath();
-                                settings.magentoVersion = MagentoVersion.get(
+                                settings.magentoVersion = MagentoVersionUtil.get(
                                         project,
                                         project.getBasePath()
                                 );

src/com/magento/idea/magento2plugin/project/SettingsForm.java

@@ -18,7 +18,7 @@
 import com.magento.idea.magento2plugin.init.ConfigurationManager;
 import com.magento.idea.magento2plugin.magento.packages.MagentoComponentManager;
 import com.magento.idea.magento2plugin.project.validator.SettingsFormValidator;
-import com.magento.idea.magento2plugin.util.magento.MagentoVersion;
+import com.magento.idea.magento2plugin.util.magento.MagentoVersionUtil;
 import java.awt.event.MouseAdapter;
 import java.awt.event.MouseEvent;
 import javax.swing.JButton;
@@ -217,7 +217,7 @@ public void changedUpdate(final DocumentEvent documentEvent) {
      */
     public void updateMagentoVersion() {
         final String magentoPathValue = this.magentoPath.getTextField().getText();
-        final String resolvedVersion = MagentoVersion.get(project, magentoPathValue);
+        final String resolvedVersion = MagentoVersionUtil.get(project, magentoPathValue);
         magentoVersion.setText(resolvedVersion);
     }
 

src/com/magento/idea/magento2plugin/project/validator/SettingsFormValidator.java

@@ -5,7 +5,7 @@
 package com.magento.idea.magento2plugin.project.validator;
 
 import com.intellij.openapi.options.ConfigurationException;
-import com.magento.idea.magento2plugin.util.magento.MagentoVersion;
+import com.magento.idea.magento2plugin.util.magento.MagentoVersionUtil;
 import com.magento.idea.magento2plugin.bundles.ValidatorBundle;
 import com.magento.idea.magento2plugin.project.SettingsForm;
 import com.magento.idea.magento2plugin.util.RegExUtil;
@@ -42,7 +42,7 @@ public void validate() throws ConfigurationException {
             throw new ConfigurationException(validatorBundle.message("validator.notEmpty", "Magento Version"));
         }
 
-        if (!magentoVersion.matches(RegExUtil.MAGENTO_VERSION) && !magentoVersion.equals(MagentoVersion.DEFAULT_VERSION)) {
+        if (!magentoVersion.matches(RegExUtil.MAGENTO_VERSION) && !magentoVersion.equals(MagentoVersionUtil.DEFAULT_VERSION)) {
             throw new ConfigurationException(validatorBundle.message("validator.magentoVersionInvalid"));
         }
     }

src/com/magento/idea/magento2plugin/util/magento/MagentoVersion.java renamed to src/com/magento/idea/magento2plugin/util/magento/MagentoVersionUtil.java

@@ -19,11 +19,11 @@
 import com.magento.idea.magento2plugin.magento.packages.File;
 import com.magento.idea.magento2plugin.magento.packages.Package;
 
-public final class MagentoVersion {
+public final class MagentoVersionUtil {
 
     public static final String DEFAULT_VERSION = "any";
 
-    private MagentoVersion() {}
+    private MagentoVersionUtil() {}
 
     /**
      * Parse composer.json to detect Magento 2 version