From 7a75ab2102dafc99138cd1eea86c653c23871453 Mon Sep 17 00:00:00 2001
From: Lukasz Lenart <lukaszlenart@apache.org>
Date: Sun, 5 Apr 2026 20:01:09 +0200
Subject: [PATCH] refactor: remove experimental CreateBeanPropertyFixes
 dependency
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Drop usage of the experimental `CreateBeanPropertyFixes` API from
`BeanPropertyPathReference` and the delegating quick-fix aggregation
in `ResultParamNameCustomConverter.MergingBeanPropertyPathReference`.

This eliminates the plugin verifier warning about referencing an
`@ApiStatus.Experimental` class that may change in future IntelliJ
Platform releases. Bean-property resolution, completion, highlighting,
and rename/bind behavior are unaffected.

🤖 Generated by AI Assistant

Made-with: Cursor
---
 .../ResultParamNameCustomConverter.java       | 19 ++---------------
 .../common/BeanPropertyPathReference.java     | 21 +------------------
 2 files changed, 3 insertions(+), 37 deletions(-)

diff --git a/src/main/java/com/intellij/struts2/dom/params/custom/ResultParamNameCustomConverter.java b/src/main/java/com/intellij/struts2/dom/params/custom/ResultParamNameCustomConverter.java
index 898ea21..dbcc402 100644
--- a/src/main/java/com/intellij/struts2/dom/params/custom/ResultParamNameCustomConverter.java
+++ b/src/main/java/com/intellij/struts2/dom/params/custom/ResultParamNameCustomConverter.java
@@ -15,8 +15,6 @@
 package com.intellij.struts2.dom.params.custom;
 
 import com.intellij.codeInsight.daemon.EmptyResolveMessageProvider;
-import com.intellij.codeInspection.LocalQuickFix;
-import com.intellij.codeInspection.LocalQuickFixProvider;
 import com.intellij.psi.PsiElement;
 import com.intellij.psi.PsiMethod;
 import com.intellij.psi.PsiReference;
@@ -60,13 +58,13 @@ public class ResultParamNameCustomConverter extends ParamNameConverter.ParamName
 
 
   private static final class MergingBeanPropertyPathReference extends PsiReferenceBase<PsiElement>
-    implements EmptyResolveMessageProvider, LocalQuickFixProvider {
+    implements EmptyResolveMessageProvider {
 
     private final List<BeanPropertyPathReference[]> allReferences = new SmartList<>();
 
     /**
      * @param element        XML element.
-     * @param paramsElements First element will be used for quickfixes.
+     * @param paramsElements Elements providing bean property references.
      */
     private MergingBeanPropertyPathReference(@NotNull XmlAttributeValue element,
                                              ParamsElement @NotNull ... paramsElements) {
@@ -113,18 +111,5 @@ public String getUnresolvedMessagePattern() {
       return "Cannot resolve property '" + getValue() + "'";
     }
 
-    @Override
-    public @NotNull LocalQuickFix @Nullable [] getQuickFixes() {
-      List<LocalQuickFix> quickFixes = new SmartList<>();
-      for (BeanPropertyPathReference[] reference : allReferences) {
-        for (BeanPropertyPathReference pathReference : reference) {
-          final LocalQuickFix[] fixes = pathReference.getQuickFixes();
-          if (fixes != null) {
-            Collections.addAll(quickFixes, fixes);
-          }
-        }
-      }
-      return quickFixes.toArray(LocalQuickFix.EMPTY_ARRAY);
-    }
   }
 }
diff --git a/src/main/java/com/intellij/struts2/reference/common/BeanPropertyPathReference.java b/src/main/java/com/intellij/struts2/reference/common/BeanPropertyPathReference.java
index 6cc6376..9f45dc4 100644
--- a/src/main/java/com/intellij/struts2/reference/common/BeanPropertyPathReference.java
+++ b/src/main/java/com/intellij/struts2/reference/common/BeanPropertyPathReference.java
@@ -17,12 +17,8 @@
 
 import com.intellij.codeInsight.daemon.EmptyResolveMessageProvider;
 import com.intellij.codeInsight.lookup.LookupElementBuilder;
-import com.intellij.codeInspection.LocalQuickFix;
-import com.intellij.codeInspection.LocalQuickFixProvider;
 import com.intellij.openapi.util.TextRange;
-import com.intellij.openapi.util.text.StringUtil;
 import com.intellij.psi.*;
-import com.intellij.psi.impl.beanProperties.CreateBeanPropertyFixes;
 import com.intellij.psi.util.PropertyUtilBase;
 import com.intellij.util.ArrayUtilRt;
 import com.intellij.util.IncorrectOperationException;
@@ -39,7 +35,7 @@
  * @author Yann C&eacute;bron
  */
 public class BeanPropertyPathReference extends PsiReferenceBase<PsiElement>
-  implements EmptyResolveMessageProvider, LocalQuickFixProvider {
+  implements EmptyResolveMessageProvider {
 
   private final BeanPropertyPathReferenceSet referenceSet;
   private final int index;
@@ -119,21 +115,6 @@ public PsiElement bindToElement(@NotNull final PsiElement element) throws Incorr
     return getElement();
   }
 
-  @Override
-  public @NotNull LocalQuickFix @Nullable [] getQuickFixes() {
-    final String value = getValue();
-    if (StringUtil.isEmpty(value)) {
-      return LocalQuickFix.EMPTY_ARRAY;
-    }
-
-    final PsiClass psiClass = getPsiClass();
-    if (psiClass == null) {
-      return LocalQuickFix.EMPTY_ARRAY;
-    }
-
-    return CreateBeanPropertyFixes.createFixes(value, psiClass, null, true);
-  }
-
   @Nullable
   private PsiMethod resolveProperty(@NotNull final PsiClass psiClass, final String propertyName) {
     PsiMethod method = isLast() ?