[java-inspections] UseBulkOperation: Fix missing warning when super is a qualifier

GitOrigin-RevId: 3e482302d12de0997964e9826c396456b86ec921

java/java-impl/src/com/intellij/codeInspection/bulkOperation/UseBulkOperationInspection.java

@@ -251,7 +251,7 @@ public class UseBulkOperationInspection extends AbstractBaseJavaLocalInspectionT
                             @NotNull BulkMethodInfo info,
                             @NotNull PsiReferenceExpression methodExpression) {
         PsiExpression qualifier = PsiUtil.skipParenthesizedExprDown(ExpressionUtils.getEffectiveQualifier(methodExpression));
-        if (qualifier instanceof PsiThisExpression) {
+        if (qualifier instanceof PsiThisExpression || qualifier instanceof PsiSuperExpression) {
           PsiMethod method = PsiTreeUtil.getParentOfType(iterable, PsiMethod.class);
           // Likely we are inside of the bulk method implementation
           if (method != null && method.getName().equals(info.getBulkName())) return;
@@ -265,7 +265,7 @@ public class UseBulkOperationInspection extends AbstractBaseJavaLocalInspectionT
 
       @Contract("null -> false")
       private boolean isSupportedQualifier(PsiExpression qualifier) {
-        if (qualifier instanceof PsiThisExpression) return true;
+        if (qualifier instanceof PsiThisExpression || qualifier instanceof PsiSuperExpression) return true;
         if (qualifier instanceof PsiReferenceExpression) {
           PsiExpression subQualifier = ((PsiReferenceExpression)qualifier).getQualifierExpression();
           return subQualifier == null || isSupportedQualifier(subQualifier);

java/java-tests/testData/codeInsight/daemonCodeAnalyzer/quickFix/useBulkOperation/afterAddAllInsideOther2.java

@@ -0,0 +1,22 @@
+// "Replace iteration with bulk 'Collection.addAll' call" "true"
+
+import java.util.*;
+
+public class Main {
+  static class MyList extends AbstractCollection<String> {
+    @Override
+    public Iterator<String> iterator() {
+      return Collections.emptyIterator();
+    }
+
+    @Override
+    public int size() {
+      return 0;
+    }
+
+    public boolean myAdd(Collection<? extends String> c) {
+        super.addAll(c);
+      return true;
+    }
+  }
+}

java/java-tests/testData/codeInsight/daemonCodeAnalyzer/quickFix/useBulkOperation/beforeAddAllInsideAddAll2.java

@@ -0,0 +1,25 @@
+// "Replace iteration with bulk 'Collection.addAll' call" "false"
+
+import java.util.*;
+
+public class Main {
+  static class MyList extends AbstractCollection<String> {
+    @Override
+    public Iterator<String> iterator() {
+      return Collections.emptyIterator();
+    }
+
+    @Override
+    public int size() {
+      return 0;
+    }
+
+    @Override
+    public boolean addAll(Collection<? extends String> c) {
+      for (String e : c) {
+        super.a<caret>dd(e);
+      }
+      return true;
+    }
+  }
+}

java/java-tests/testData/codeInsight/daemonCodeAnalyzer/quickFix/useBulkOperation/beforeAddAllInsideOther2.java

@@ -0,0 +1,24 @@
+// "Replace iteration with bulk 'Collection.addAll' call" "true"
+
+import java.util.*;
+
+public class Main {
+  static class MyList extends AbstractCollection<String> {
+    @Override
+    public Iterator<String> iterator() {
+      return Collections.emptyIterator();
+    }
+
+    @Override
+    public int size() {
+      return 0;
+    }
+
+    public boolean myAdd(Collection<? extends String> c) {
+      for (String e : c) {
+        super.<caret>add(e);
+      }
+      return true;
+    }
+  }
+}