1135: Fixed issues with incorrect generation of resolver class

Author: makzef

resources/magento2/inspection.properties

@@ -39,3 +39,6 @@ inspection.warning.class.does.not.exist=The class "{0}" does not exist
 inspection.warning.method.does.not.exist=The method "{0}" does not exist in the service class
 inspection.warning.method.should.have.public.access=The method "{0}" should have public access
 inspection.warning.method.should.have.public.access.fix=Change the method access
+inspection.warning.class.invalidFormat=The class "{0}" has invalid format
+inspection.error.class.tooShortFormat=The target resolver class format is incorrect.\nThe fully qualified name "{0}" should contain at least 3 parts.
+inspection.error.class.tooShortFormatTitle=Resolver class format is incorrect

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

@@ -25,7 +25,6 @@
 import com.magento.idea.magento2plugin.bundles.ValidatorBundle;
 import com.magento.idea.magento2plugin.indexes.ModuleIndex;
 import com.magento.idea.magento2plugin.magento.files.GraphQlResolverPhp;
-import com.magento.idea.magento2plugin.magento.packages.File;
 import com.magento.idea.magento2plugin.magento.packages.MagentoPhpClass;
 import com.magento.idea.magento2plugin.util.GetFirstClassOfFile;
 import com.magento.idea.magento2plugin.util.GetPhpClassByFQN;
@@ -132,10 +131,11 @@ private PhpClass createGraphQlResolverClass(final String actionName) {
         if (parentDirectory == null) {
             return null;
         }
-        final String[] graphQlResolverDirectories = graphQlResolverFileData
-                .getGraphQlResolverDirectory().split(File.separator);
-        for (final String graphQlResolverDirectory: graphQlResolverDirectories) {
-            parentDirectory = directoryGenerator.findOrCreateSubdirectory(
+
+        final String graphQlResolverDirectory = graphQlResolverFileData
+                .getGraphQlResolverDirectory();
+        if (!graphQlResolverDirectory.isBlank()) {
+            parentDirectory = directoryGenerator.findOrCreateSubdirectories(
                     parentDirectory,
                     graphQlResolverDirectory
             );

src/com/magento/idea/magento2plugin/inspections/graphqls/SchemaResolverInspection.java

@@ -8,12 +8,16 @@
 import com.intellij.codeInspection.LocalInspectionTool;
 import com.intellij.codeInspection.ProblemHighlightType;
 import com.intellij.codeInspection.ProblemsHolder;
+import com.intellij.lang.jsgraphql.psi.GraphQLNamedElement;
 import com.intellij.lang.jsgraphql.psi.GraphQLValue;
 import com.intellij.lang.jsgraphql.psi.GraphQLVisitor;
+import com.intellij.psi.PsiElement;
 import com.jetbrains.php.lang.psi.elements.PhpClass;
 import com.magento.idea.magento2plugin.bundles.InspectionBundle;
 import com.magento.idea.magento2plugin.inspections.graphqls.fix.CreateResolverClassQuickFix;
+import com.magento.idea.magento2plugin.magento.files.GraphQlResolver;
 import com.magento.idea.magento2plugin.util.GetPhpClassByFQN;
+import com.magento.idea.magento2plugin.util.RegExUtil;
 import com.magento.idea.magento2plugin.util.magento.graphql.GraphQlUtil;
 import org.jetbrains.annotations.NotNull;
 
@@ -31,20 +35,41 @@ public GraphQLVisitor buildVisitor(
             @Override
             public void visitValue(@NotNull final GraphQLValue element) {
                 final String getVisitedElementValue = element.getText();
-                if (getVisitedElementValue == null) {
+                final PsiElement parentElementValue = element.getParent();
+                if (getVisitedElementValue == null
+                        || !(parentElementValue instanceof GraphQLNamedElement)) {
+                    return;
+                }
+
+                final String attributeName = ((GraphQLNamedElement) parentElementValue).getName();
+                if (!GraphQlResolver.CLASS_ARGUMENT.equals(attributeName)) {
                     return;
                 }
 
                 final String resolverFQN
                         = GraphQlUtil.resolverStringToPhpFQN(getVisitedElementValue);
+                if (!resolverFQN.matches(RegExUtil.PhpRegex.FQN)) {
+                    holder.registerProblem(
+                            element,
+                            inspectionBundle.message(
+                                    "inspection.warning.class.invalidFormat",
+                                    resolverFQN
+                            ),
+                            ProblemHighlightType.WARNING
+                    );
+
+                    return;
+                }
+
                 final GetPhpClassByFQN getPhpClassByFQN
                         = GetPhpClassByFQN.getInstance(holder.getProject());
                 final PhpClass resolverClass = getPhpClassByFQN.execute(resolverFQN);
+
                 if (resolverClass == null) {
                     holder.registerProblem(
                             element,
                             inspectionBundle.message(
-                                "inspection.graphql.resolver.notExist"
+                                    "inspection.graphql.resolver.notExist"
                             ),
                             ProblemHighlightType.ERROR,
                             new CreateResolverClassQuickFix());

src/com/magento/idea/magento2plugin/inspections/graphqls/fix/CreateResolverClassQuickFix.java

@@ -14,12 +14,19 @@
 import com.magento.idea.magento2plugin.actions.generation.data.GraphQlResolverFileData;
 import com.magento.idea.magento2plugin.actions.generation.generator.ModuleGraphQlResolverClassGenerator;
 import com.magento.idea.magento2plugin.bundles.InspectionBundle;
+import com.magento.idea.magento2plugin.magento.packages.File;
 import java.util.ArrayList;
 import java.util.Arrays;
 import java.util.List;
+import javax.swing.JOptionPane;
 import org.jetbrains.annotations.NotNull;
 
 public class CreateResolverClassQuickFix implements LocalQuickFix {
+
+    private static final int CLASSPATH_MIN_DEPTH = 3;
+
+    private final InspectionBundle inspectionBundle = new InspectionBundle();
+
     @Override
     public @NotNull String getFamilyName() {
         return new InspectionBundle().message(
@@ -39,10 +46,21 @@ public void applyFix(
 
         fqnPartsList.removeIf(Strings::isNullOrEmpty);
 
+        if (fqnPartsList.size() < CLASSPATH_MIN_DEPTH) {
+            JOptionPane.showMessageDialog(
+                    null,
+                    inspectionBundle.message("inspection.error.class.tooShortFormat", resolverFqn),
+                    inspectionBundle.message("inspection.error.class.tooShortFormatTitle"),
+                    JOptionPane.ERROR_MESSAGE
+            );
+
+            return;
+        }
+
         final int endIndex = fqnPartsList.size() - 1;
         final String moduleName = String.join("_", fqnPartsList.subList(0, 2));
         final String resolverName = fqnPartsList.get(endIndex);
-        final String directory = fqnPartsList.get(2);
+        final String directory = String.join(File.separator, fqnPartsList.subList(2, endIndex));
         final String namespace = String.join("\\", fqnPartsList.subList(0, endIndex));
 
         final GraphQlResolverFileData graphQlResolverFileData = new GraphQlResolverFileData(