Skip to content

Commit ace0d42

Browse files
authored
Merge pull request #110 from cedricziel/route-marker
Fix RouteLineMarker by calling setTargets
2 parents 084c1ff + 982fefd commit ace0d42

File tree

1 file changed

+9
-2
lines changed

1 file changed

+9
-2
lines changed

src/main/java/com/cedricziel/idea/typo3/codeInsight/RouteLineMarkerProvider.java

Lines changed: 9 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,7 @@
33
import com.cedricziel.idea.typo3.TYPO3CMSIcons;
44
import com.cedricziel.idea.typo3.index.RouteIndex;
55
import com.cedricziel.idea.typo3.psi.PhpElementsUtil;
6+
import com.cedricziel.idea.typo3.routing.RouteHelper;
67
import com.cedricziel.idea.typo3.routing.RouteStub;
78
import com.intellij.codeInsight.daemon.RelatedItemLineMarkerInfo;
89
import com.intellij.codeInsight.daemon.RelatedItemLineMarkerProvider;
@@ -17,7 +18,7 @@
1718

1819
public class RouteLineMarkerProvider extends RelatedItemLineMarkerProvider {
1920
@Override
20-
protected void collectNavigationMarkers(@NotNull PsiElement element, Collection<? super RelatedItemLineMarkerInfo> result) {
21+
protected void collectNavigationMarkers(@NotNull PsiElement element, @NotNull Collection<? super RelatedItemLineMarkerInfo> result) {
2122

2223
if (!(element instanceof StringLiteralExpression)) {
2324
return;
@@ -36,8 +37,14 @@ protected void collectNavigationMarkers(@NotNull PsiElement element, Collection<
3637
if (RouteIndex.hasRoute(element.getProject(), value)) {
3738
Collection<RouteStub> routes = RouteIndex.getRoute(element.getProject(), value);
3839
routes.forEach(def -> {
40+
PsiElement[] routeDefinitionElements = RouteHelper.getRouteDefinitionElements(element.getProject(), value);
3941
NavigationGutterIconBuilder<PsiElement> builder = NavigationGutterIconBuilder
40-
.create(TYPO3CMSIcons.ROUTE_ICON);
42+
.create(TYPO3CMSIcons.ROUTE_ICON)
43+
.setTargets(routeDefinitionElements);
44+
45+
if (def.getPath() != null) {
46+
builder.setTooltipTitle("Path: " + def.getPath());
47+
}
4148

4249
result.add(builder.createLineMarkerInfo(element));
4350
});

0 commit comments

Comments
 (0)