Added mutuallyExclusive logic

Author: Rejoice472

src/main/java/io/openliberty/tools/intellij/lsp4jakarta/lsp4ij/cdi/ManagedBeanConstants.java

@@ -36,9 +36,12 @@ public class ManagedBeanConstants {
     public static final String DIAGNOSTIC_CODE_INVALID_INJECT_PARAM = "RemoveInjectOrConflictedAnnotations";
     public static final String DIAGNOSTIC_CODE_INVALID_PRODUCES_PARAM = "RemoveProducesOrConflictedAnnotations";
     public static final String DIAGNOSTIC_CODE_INVALID_DISPOSES_PARAM = "RemoveDisposesOrConflictedAnnotations";
+    public static final String DIAGNOSTIC_INJECT_MULTIPLE_METHOD_PARAM = "InvalidInjectAnnotationOnMultipleMethodParams";
 
     public static final String DIAGNOSTIC_CODE_REDUNDANT_DISPOSES = "RemoveExtraDisposes";
-
+    //Added as part of fix that adds two quick fixes which are mutually exclusive issue #540
+    public static final String[] INVALID_DISPOSER_FQ_PARAMS = { DISPOSES_FQ_NAME };
+    public static final String[] INVALID_DISPOSER_FQ_CONFLICTED_PARAMS = { OBSERVES_FQ_NAME, OBSERVES_ASYNC_FQ_NAME };
     public static final String[] INVALID_INJECT_PARAMS_FQ = { DISPOSES_FQ_NAME, OBSERVES_FQ_NAME,
             OBSERVES_ASYNC_FQ_NAME };
 

src/main/java/io/openliberty/tools/intellij/lsp4jakarta/lsp4ij/cdi/ManagedBeanDiagnosticsCollector.java

@@ -13,10 +13,7 @@
 
 package io.openliberty.tools.intellij.lsp4jakarta.lsp4ij.cdi;
 
-import java.util.ArrayList;
-import java.util.List;
-import java.util.Set;
-import java.util.TreeSet;
+import java.util.*;
 import java.util.stream.Stream;
 
 import com.intellij.psi.*;
@@ -30,6 +27,7 @@
 
 import static io.openliberty.tools.intellij.lsp4jakarta.lsp4ij.JDTUtils.getSimpleName;
 import static io.openliberty.tools.intellij.lsp4jakarta.lsp4ij.cdi.ManagedBeanConstants.*;
+import static org.eclipse.lsp4jakarta.jdt.internal.cdi.Constants.INVALID_INITIALIZER_PARAMS_FQ;
 
 public class ManagedBeanDiagnosticsCollector extends AbstractDiagnosticsCollector {
 
@@ -234,7 +232,7 @@ else if (INJECT_FQ_NAME.equals(annotation))
             if (isManagedBean) {
                 boolean isClassGeneric = type.getTypeParameters().length != 0;
                 boolean isDependent = managedBeanAnnotations.stream()
-                        .anyMatch(annotation -> DEPENDENT_FQ_NAME.equals(annotation));
+                            .anyMatch(annotation -> DEPENDENT_FQ_NAME.equals(annotation));
 
                 if (isClassGeneric && !isDependent) {
                     diagnostics.add(createDiagnostic(type, unit, Messages.getMessage("ManagedBeanGenericType"),
@@ -313,6 +311,8 @@ else if (INJECT_FQ_NAME.equals(annotation))
 
     private void invalidParamsCheck(PsiJavaFile unit, List<Diagnostic> diagnostics, PsiClass type, String target,
                                     String diagnosticCode) {
+        Set<String> paramScopesSet;
+        boolean mutuallyExclusive = false;
         for (PsiMethod method : type.getMethods()) {
             PsiAnnotation targetAnnotation = null;
 
@@ -335,20 +335,29 @@ private void invalidParamsCheck(PsiJavaFile unit, List<Diagnostic> diagnostics,
                 for (String annotation : paramScopes) {
                     invalidAnnotations.add("@" + getSimpleName(annotation));
                 }
+                paramScopesSet = new LinkedHashSet<>(paramScopes);
+                if (paramScopesSet.size() == INVALID_INITIALIZER_PARAMS_FQ.length && paramScopesSet.equals(Set.of(INVALID_INITIALIZER_PARAMS_FQ))) {
+                    mutuallyExclusive = true;
+                    diagnosticCode = DIAGNOSTIC_INJECT_MULTIPLE_METHOD_PARAM;
+                }
             }
 
             if (!invalidAnnotations.isEmpty()) {
                 String label = PRODUCES_FQ_NAME.equals(target) ?
                         createInvalidProducesLabel(invalidAnnotations) :
-                        createInvalidInjectLabel(invalidAnnotations);
+                        createInvalidInjectLabel(invalidAnnotations, mutuallyExclusive);
                 diagnostics.add(createDiagnostic(method, unit, label, diagnosticCode, null, DiagnosticSeverity.Error));
             }
 
         }
     }
 
-    private String createInvalidInjectLabel(Set<String> invalidAnnotations) {
-        return Messages.getMessage("ManagedBeanInvalidInject", String.join(", ", invalidAnnotations)); // assuming comma delimited list is ok
+    private String createInvalidInjectLabel(Set<String> invalidAnnotations, boolean mutuallyExclusive) {
+        if (mutuallyExclusive) {
+            return Messages.getMessage("ManagedBeanInvalidInject", String.join(", ", invalidAnnotations)); // assuming comma delimited list is ok
+        } else {
+            return Messages.getMessage("ManagedBeanInvalidInject", String.join(", ", invalidAnnotations)); // assuming comma delimited list is ok
+        }
     }
 
     private String createInvalidProducesLabel(Set<String> invalidAnnotations) {