Merge pull request #289 from AxonFramework/fix/missing-class-renderers

CodeDrivenMitch · web-flow · commit bf15afa18c61 · 2024-10-01T10:29:45.000+02:00
Add renderers to class-level popups so they are rendered correctly
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -3,6 +3,7 @@
 # Axon Framework plugin Changelog
 
 ## [0.8.8]
+- Fix classes not rendering the correct value in popup of classes
 - Plugin is now compatible with IDEA 2024.3 (IDEA 243.*) with minimum version of 2024.2
 - Fix various deprecation warnings
 - Fix various javadoc issues
diff --git a/src/main/kotlin/org/axonframework/intellij/ide/plugin/markers/ClassLineMarkerProvider.kt b/src/main/kotlin/org/axonframework/intellij/ide/plugin/markers/ClassLineMarkerProvider.kt
@@ -55,6 +55,7 @@ class ClassLineMarkerProvider : LineMarkerProvider {
                         val publishers = element.creatorResolver().getCreatorsForPayload(qualifiedName)
                         handlers + publishers
                     })
+                    .setTargetRenderer { AxonNavigationTargetRenderer.INSTANCE }
                     .createLineMarkerInfo(element)
             }
         }
@@ -67,6 +68,7 @@ class ClassLineMarkerProvider : LineMarkerProvider {
                 .setTooltipText("Navigate to entities in the same command model hierarchy")
                 .setEmptyPopupText("No related entities were found")
                 .setTargets(NotNullLazyValue.lazy { items })
+                .setTargetRenderer { AxonNavigationTargetRenderer.INSTANCE }
                 .createLineMarkerInfo(element)
         }
 

Original file line number	Diff line number	Diff line change
`@@ -55,6 +55,7 @@ class ClassLineMarkerProvider : LineMarkerProvider {`
`55`	`55`	`val publishers = element.creatorResolver().getCreatorsForPayload(qualifiedName)`
`56`	`56`	`handlers + publishers`
`57`	`57`	`})`
	`58`	`+ .setTargetRenderer { AxonNavigationTargetRenderer.INSTANCE }`
`58`	`59`	`.createLineMarkerInfo(element)`
`59`	`60`	`}`
`60`	`61`	`}`
`@@ -67,6 +68,7 @@ class ClassLineMarkerProvider : LineMarkerProvider {`
`67`	`68`	`.setTooltipText("Navigate to entities in the same command model hierarchy")`
`68`	`69`	`.setEmptyPopupText("No related entities were found")`
`69`	`70`	`.setTargets(NotNullLazyValue.lazy { items })`
	`71`	`+ .setTargetRenderer { AxonNavigationTargetRenderer.INSTANCE }`
`70`	`72`	`.createLineMarkerInfo(element)`
`71`	`73`	`}`
`72`	`74`