UCT-723: Added ImportedNonApiInterface inspection

Iamwade · Iamwade · commit ddef264f2d4f · 2021-11-17T19:42:01.000+02:00
diff --git a/resources/META-INF/plugin.xml b/resources/META-INF/plugin.xml
@@ -410,6 +410,13 @@
                          enabledByDefault="false"
                          level="WARNING"
                          implementationClass="com.magento.idea.magento2uct.inspections.php.api.ImportedNonApiClass"/>
+        <localInspection language="PHP" groupPath="UCT"
+                         shortName="ImportedNonApiInterface"
+                         bundle="uct.bundle.inspection" key="inspection.displayName.ImportedNonApiInterface"
+                         groupBundle="uct.bundle.inspection" groupKey="inspection.api.group.name"
+                         enabledByDefault="false"
+                         level="WARNING"
+                         implementationClass="com.magento.idea.magento2uct.inspections.php.api.ImportedNonApiInterface"/>
         <!-- \UCT inspection -->
 
         <internalFileTemplate name="Magento Composer JSON"/>
diff --git a/resources/inspectionDescriptions/ImportedNonApiInterface.html b/resources/inspectionDescriptions/ImportedNonApiInterface.html
@@ -0,0 +1,6 @@
+<html>
+<body>
+<p>[1322] The imported interface is not marked as an API.</p>
+<!-- tooltip end -->
+</body>
+</html>
diff --git a/resources/uct/bundle/inspection.properties b/resources/uct/bundle/inspection.properties
@@ -24,6 +24,7 @@ inspection.displayName.OverriddenNonExistentProperty=Overridden non-existent Ado
 inspection.displayName.CalledNonExistentMethod=Call non-existent Adobe Commerce method
 inspection.displayName.UsedNonExistentType=Used non-existent Adobe Commerce type
 inspection.displayName.ImportedNonApiClass=Imported non Adobe Commerce API class
+inspection.displayName.ImportedNonApiInterface=Imported non Adobe Commerce API interface
 customCode.warnings.deprecated.1131=[1131] Extending from @deprecated class ''{0}''
 customCode.warnings.deprecated.1132=[1132] Importing @deprecated class ''{0}''
 customCode.warnings.deprecated.1134=[1134] Using @deprecated class ''{0}''
@@ -46,3 +47,4 @@ customCode.critical.existence.1515=[1515] Overridden property ''{0}'' that is re
 customCode.critical.existence.1410=[1410] Called method ''{0}'' that is removed in the ''{1}''
 customCode.critical.existence.1110=[1110] Used type ''{0}'' that is removed in the ''{1}''
 customCode.critical.api.1122=[1122] Imported class ''{0}'' is not marked as an API
+customCode.critical.api.1322=[1322] Imported interface ''{0}'' is not marked as an API
diff --git a/src/com/magento/idea/magento2uct/inspections/php/api/ImportedNonApiInterface.java b/src/com/magento/idea/magento2uct/inspections/php/api/ImportedNonApiInterface.java
@@ -0,0 +1,45 @@
+/*
+ * Copyright © Magento, Inc. All rights reserved.
+ * See COPYING.txt for license details.
+ */
+
+package com.magento.idea.magento2uct.inspections.php.api;
+
+import com.intellij.codeInspection.ProblemHighlightType;
+import com.intellij.codeInspection.ProblemsHolder;
+import com.intellij.openapi.project.Project;
+import com.jetbrains.php.lang.psi.elements.PhpUse;
+import com.magento.idea.magento2uct.inspections.UctProblemsHolder;
+import com.magento.idea.magento2uct.inspections.php.ImportInspection;
+import com.magento.idea.magento2uct.packages.IssueSeverityLevel;
+import com.magento.idea.magento2uct.packages.SupportedIssue;
+import com.magento.idea.magento2uct.versioning.VersionStateManager;
+import org.jetbrains.annotations.NotNull;
+
+public class ImportedNonApiInterface extends ImportInspection {
+
+    @Override
+    protected void execute(
+            final Project project,
+            final @NotNull ProblemsHolder problemsHolder,
+            final PhpUse use,
+            final boolean isInterface
+    ) {
+        if (!isInterface || VersionStateManager.getInstance(project).isApi(use.getFQN())) {
+            return;
+        }
+        final String message = SupportedIssue.IMPORTED_NON_API_INTERFACE.getMessage(use.getFQN());
+
+        if (problemsHolder instanceof UctProblemsHolder) {
+            ((UctProblemsHolder) problemsHolder).setReservedErrorCode(
+                    SupportedIssue.IMPORTED_NON_API_INTERFACE.getCode()
+            );
+        }
+        problemsHolder.registerProblem(use, message, ProblemHighlightType.WARNING);
+    }
+
+    @Override
+    protected IssueSeverityLevel getSeverityLevel() {
+        return SupportedIssue.IMPORTED_NON_API_INTERFACE.getLevel();
+    }
+}
diff --git a/src/com/magento/idea/magento2uct/packages/SupportedIssue.java b/src/com/magento/idea/magento2uct/packages/SupportedIssue.java
@@ -10,6 +10,7 @@
 import com.magento.idea.magento2uct.bundles.UctInspectionBundle;
 import com.magento.idea.magento2uct.inspections.UctProblemsHolder;
 import com.magento.idea.magento2uct.inspections.php.api.ImportedNonApiClass;
+import com.magento.idea.magento2uct.inspections.php.api.ImportedNonApiInterface;
 import com.magento.idea.magento2uct.inspections.php.deprecation.CallingDeprecatedMethod;
 import com.magento.idea.magento2uct.inspections.php.deprecation.ExtendingDeprecatedClass;
 import com.magento.idea.magento2uct.inspections.php.deprecation.ImplementedDeprecatedInterface;
@@ -170,6 +171,12 @@ public enum SupportedIssue {
             IssueSeverityLevel.ERROR,
             "customCode.critical.api.1122",
             ImportedNonApiClass.class
+    ),
+    IMPORTED_NON_API_INTERFACE(
+            1322,
+            IssueSeverityLevel.ERROR,
+            "customCode.critical.api.1322",
+            ImportedNonApiInterface.class
     );
 
     private final int code;

-Original file line number
+Diff line change
@@ @@ -0,0 +1,6 @@ @@
 +<html>
 +<body>
 +<p>[1322] The imported interface is not marked as an API.</p>
 +<!-- tooltip end -->
 +</body>
 +</html>