[java-intentions] IDEA-304321 Fix "Create constructor" preview

GitOrigin-RevId: 3a473c492e709a280d8a06e7364a2af05babf37f

java/java-impl/src/com/intellij/lang/java/actions/CreateConstructorAction.kt

@@ -9,14 +9,13 @@ import com.intellij.codeInsight.daemon.impl.quickfix.CreateFromUsageUtils
 import com.intellij.codeInsight.daemon.impl.quickfix.GuessTypeParameters
 import com.intellij.codeInsight.generation.OverrideImplementUtil
 import com.intellij.codeInsight.intention.preview.IntentionPreviewInfo
+import com.intellij.codeInsight.intention.preview.IntentionPreviewUtils
 import com.intellij.codeInsight.template.Template
 import com.intellij.codeInsight.template.TemplateBuilder
 import com.intellij.codeInsight.template.TemplateBuilderImpl
 import com.intellij.codeInsight.template.TemplateEditingAdapter
-import com.intellij.ide.highlighter.JavaFileType
 import com.intellij.lang.java.request.CreateConstructorFromJavaUsageRequest
 import com.intellij.lang.jvm.actions.CreateConstructorRequest
-import com.intellij.openapi.command.WriteCommandAction
 import com.intellij.openapi.editor.Editor
 import com.intellij.openapi.project.Project
 import com.intellij.psi.*
@@ -41,8 +40,10 @@ internal class CreateConstructorAction(
   private fun constructorRenderer(project: Project) = JavaConstructorRenderer(project, target, request)
 
   override fun generatePreview(project: Project, editor: Editor, file: PsiFile): IntentionPreviewInfo {
-    val constructor = constructorRenderer(project).renderConstructor()
-    return IntentionPreviewInfo.CustomDiff(JavaFileType.INSTANCE, "", constructor.text)
+    val copyClass = PsiTreeUtil.findSameElementInCopy(target, file)
+    val javaFieldRenderer = JavaConstructorRenderer(project, copyClass, request)
+    javaFieldRenderer.doMagic()
+    return IntentionPreviewInfo.DIFF
   }
 
   override fun invoke(project: Project, editor: Editor?, file: PsiFile?) {
@@ -107,7 +108,7 @@ private class JavaConstructorRenderer(
 
       override fun templateFinished(template: Template, brokenOff: Boolean) {
         if (brokenOff) return
-        WriteCommandAction.runWriteCommandAction(project) { setupBody() }
+        IntentionPreviewUtils.write<Throwable> { setupBody() }
       }
 
       private fun setupBody() {

java/java-tests/testData/codeInsight/daemonCodeAnalyzer/quickFix/createConstructorFromCall/after1.java

@@ -1,4 +1,4 @@
-// "Create constructor" "true"
+// "Create constructor" "true-preview"
 public class Test {
     public static void main() {
         new MyCollection(10);

java/java-tests/testData/codeInsight/daemonCodeAnalyzer/quickFix/createConstructorFromCall/after2.java

@@ -1,4 +1,4 @@
-// "Create constructor" "true"
+// "Create constructor" "true-preview"
 public class Test {
     public void main2() {
         new MyCollection(this);

java/java-tests/testData/codeInsight/daemonCodeAnalyzer/quickFix/createConstructorFromCall/after3.java

@@ -1,4 +1,4 @@
-// "Create constructor" "true"
+// "Create constructor" "true-preview"
 class Outer {
     public Outer(String s) {
     }

java/java-tests/testData/codeInsight/daemonCodeAnalyzer/quickFix/createConstructorFromCall/afterAnon.java

@@ -1,4 +1,4 @@
-// "Create constructor" "true"
+// "Create constructor" "true-preview"
 class Test {
 
     public void t() {

java/java-tests/testData/codeInsight/daemonCodeAnalyzer/quickFix/createConstructorFromCall/afterDefaultSuperConstructor.java

@@ -1,4 +1,4 @@
-// "Create constructor" "true"
+// "Create constructor" "true-preview"
 class Test extends A{
 
     public Test(String a) {

java/java-tests/testData/codeInsight/daemonCodeAnalyzer/quickFix/createConstructorFromCall/afterDisjunction.java

@@ -1,4 +1,4 @@
-// "Create constructor" "true"
+// "Create constructor" "true-preview"
 class Test {
     void foo() throws Ex1 {}
     void bar() throws Ex2 {}

java/java-tests/testData/codeInsight/daemonCodeAnalyzer/quickFix/createConstructorFromCall/afterImplicitSuperConstructor.java

@@ -1,4 +1,4 @@
-// "Create constructor" "true"
+// "Create constructor" "true-preview"
 class Test extends A{
 
     public Test(String a) {

java/java-tests/testData/codeInsight/daemonCodeAnalyzer/quickFix/createConstructorFromCall/afterIncompleteEnum.java

@@ -1,4 +1,4 @@
-// "Create constructor" "true"
+// "Create constructor" "true-preview"
 public enum ReportType {
     AGING("Aging");
 

java/java-tests/testData/codeInsight/daemonCodeAnalyzer/quickFix/createConstructorFromCall/afterIncompleteEnum1.java

@@ -1,4 +1,4 @@
-// "Create constructor" "true"
+// "Create constructor" "true-preview"
 public enum ReportType {
     AGING("Aging"),
     ;

java/java-tests/testData/codeInsight/daemonCodeAnalyzer/quickFix/createConstructorFromCall/afterInferredType.java

@@ -1,4 +1,4 @@
-// "Create constructor" "true"
+// "Create constructor" "true-preview"
 class MyTest {
     <T> T id(T t) {
         return t;

java/java-tests/testData/codeInsight/daemonCodeAnalyzer/quickFix/createConstructorFromCall/afterInsertSuper.java

@@ -1,4 +1,4 @@
-// "Create constructor" "true"
+// "Create constructor" "true-preview"
 class Test extends A{
 
     public Test(String a) {

java/java-tests/testData/codeInsight/daemonCodeAnalyzer/quickFix/createConstructorFromCall/before1.java

@@ -1,4 +1,4 @@
-// "Create constructor" "true"
+// "Create constructor" "true-preview"
 public class Test {
     public static void main() {
         new My<caret>Collection(10);

java/java-tests/testData/codeInsight/daemonCodeAnalyzer/quickFix/createConstructorFromCall/before2.java

@@ -1,4 +1,4 @@
-// "Create constructor" "true"
+// "Create constructor" "true-preview"
 public class Test {
     public void main2() {
         new <caret>MyCollection(this);

java/java-tests/testData/codeInsight/daemonCodeAnalyzer/quickFix/createConstructorFromCall/before3.java

@@ -1,4 +1,4 @@
-// "Create constructor" "true"
+// "Create constructor" "true-preview"
 class Outer {
     public Outer(String s) {
     }

java/java-tests/testData/codeInsight/daemonCodeAnalyzer/quickFix/createConstructorFromCall/beforeAnon.java

@@ -1,4 +1,4 @@
-// "Create constructor" "true"
+// "Create constructor" "true-preview"
 class Test {
 
     public void t() {

java/java-tests/testData/codeInsight/daemonCodeAnalyzer/quickFix/createConstructorFromCall/beforeDefaultSuperConstructor.java

@@ -1,4 +1,4 @@
-// "Create constructor" "true"
+// "Create constructor" "true-preview"
 class Test extends A{
 
     public void t() {

java/java-tests/testData/codeInsight/daemonCodeAnalyzer/quickFix/createConstructorFromCall/beforeDisjunction.java

@@ -1,4 +1,4 @@
-// "Create constructor" "true"
+// "Create constructor" "true-preview"
 class Test {
     void foo() throws Ex1 {}
     void bar() throws Ex2 {}

java/java-tests/testData/codeInsight/daemonCodeAnalyzer/quickFix/createConstructorFromCall/beforeImplicitSuperConstructor.java

@@ -1,4 +1,4 @@
-// "Create constructor" "true"
+// "Create constructor" "true-preview"
 class Test extends A{
 
     public void t() {

java/java-tests/testData/codeInsight/daemonCodeAnalyzer/quickFix/createConstructorFromCall/beforeIncompleteEnum.java

@@ -1,4 +1,4 @@
-// "Create constructor" "true"
+// "Create constructor" "true-preview"
 public enum ReportType {
     AGING("Aging")<caret>
 }

java/java-tests/testData/codeInsight/daemonCodeAnalyzer/quickFix/createConstructorFromCall/beforeIncompleteEnum1.java

@@ -1,4 +1,4 @@
-// "Create constructor" "true"
+// "Create constructor" "true-preview"
 public enum ReportType {
     AGING("Aging")<caret>,
 }

java/java-tests/testData/codeInsight/daemonCodeAnalyzer/quickFix/createConstructorFromCall/beforeInferredType.java

@@ -1,4 +1,4 @@
-// "Create constructor" "true"
+// "Create constructor" "true-preview"
 class MyTest {
     <T> T id(T t) {
         return t;

java/java-tests/testData/codeInsight/daemonCodeAnalyzer/quickFix/createConstructorFromCall/beforeInsertSuper.java

@@ -1,4 +1,4 @@
-// "Create constructor" "true"
+// "Create constructor" "true-preview"
 class Test extends A{
 
     public void t() {