-
Notifications
You must be signed in to change notification settings - Fork 4
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Fixed #120 Open declaration on services should show all services with…
… more and less specific types.
- Loading branch information
Showing
7 changed files
with
397 additions
and
8 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
10 changes: 7 additions & 3 deletions
10
plugins/org.eclipse.acceleo.aql.ide.ui/.settings/org.eclipse.jdt.core.prefs
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
163 changes: 163 additions & 0 deletions
163
...eo.aql.ide.ui/src/org/eclipse/acceleo/aql/ide/ui/declaration/AcceleoLSBasedHyperlink.java
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,163 @@ | ||
| /******************************************************************************* | ||
| * Copyright (c) 2016, 2023 Red Hat Inc. and others. | ||
| * This program and the accompanying materials are made | ||
| * available under the terms of the Eclipse Public License 2.0 | ||
| * which is available at https://www.eclipse.org/legal/epl-2.0/ | ||
| * | ||
| * SPDX-License-Identifier: EPL-2.0 | ||
| * | ||
| * Contributors: | ||
| * Mickael Istria (Red Hat Inc.) - initial implementation | ||
| * Michał Niewrzał (Rogue Wave Software Inc.) - hyperlink range detection | ||
| * Lucas Bullen (Red Hat Inc.) - [Bug 517428] Requests sent before initialization | ||
| * Martin Lippert (Pivotal Inc.) - [Bug 561270] labels include more details now | ||
| *******************************************************************************/ | ||
| package org.eclipse.acceleo.aql.ide.ui.declaration; | ||
|
|
||
| import java.io.InputStream; | ||
| import java.net.URI; | ||
| import java.nio.charset.Charset; | ||
| import java.nio.file.Path; | ||
| import java.nio.file.Paths; | ||
|
|
||
| import org.eclipse.core.resources.IFile; | ||
| import org.eclipse.core.resources.IWorkspaceRoot; | ||
| import org.eclipse.core.resources.ResourcesPlugin; | ||
| import org.eclipse.core.runtime.IPath; | ||
| import org.eclipse.jface.text.Document; | ||
| import org.eclipse.jface.text.IDocument; | ||
| import org.eclipse.jface.text.IRegion; | ||
| import org.eclipse.jface.text.hyperlink.IHyperlink; | ||
| import org.eclipse.lsp4e.LSPEclipseUtils; | ||
| import org.eclipse.lsp4j.Location; | ||
| import org.eclipse.lsp4j.LocationLink; | ||
| import org.eclipse.lsp4j.Range; | ||
| import org.eclipse.lsp4j.jsonrpc.messages.Either; | ||
| import org.eclipse.lsp4j.jsonrpc.validation.NonNull; | ||
|
|
||
| // copied from org.eclipse.lsp4e.operations.declaration.LSBasedHyperlink | ||
| public class AcceleoLSBasedHyperlink implements IHyperlink { | ||
|
|
||
| private static final String DASH_SEPARATOR = " - "; //$NON-NLS-1$ | ||
|
|
||
| private final Either<Location, LocationLink> location; | ||
|
|
||
| private final IRegion highlightRegion; | ||
|
|
||
| public AcceleoLSBasedHyperlink(Either<Location, LocationLink> location, IRegion highlightRegion) { | ||
| this.location = location; | ||
| this.highlightRegion = highlightRegion; | ||
| } | ||
|
|
||
| public AcceleoLSBasedHyperlink(Location location, IRegion linkRegion) { | ||
| this(Either.forLeft(location), linkRegion); | ||
| } | ||
|
|
||
| public AcceleoLSBasedHyperlink(LocationLink locationLink, IRegion linkRegion) { | ||
| this(Either.forRight(locationLink), linkRegion); | ||
| } | ||
|
|
||
| @Override | ||
| public IRegion getHyperlinkRegion() { | ||
| return this.highlightRegion; | ||
| } | ||
|
|
||
| @Override | ||
| public String getTypeLabel() { | ||
| return getLabel(); | ||
| } | ||
|
|
||
| @Override | ||
| public String getHyperlinkText() { | ||
| return getLabel(); | ||
| } | ||
|
|
||
| /** | ||
| * @return | ||
| * @noreference test only | ||
| */ | ||
| public Either<Location, LocationLink> getLocation() { | ||
| return location; | ||
| } | ||
|
|
||
| @Override | ||
| public void open() { | ||
| if (location.isLeft()) { | ||
| LSPEclipseUtils.openInEditor(location.getLeft()); | ||
| } else { | ||
| LSPEclipseUtils.openInEditor(location.getRight()); | ||
| } | ||
| } | ||
|
|
||
| private String getLabel() { | ||
| if (this.location != null) { | ||
| String uri = this.location.isLeft() ? this.location.getLeft().getUri() | ||
| : this.location.getRight().getTargetUri(); | ||
| if (uri != null) { | ||
| if (uri.startsWith(LSPEclipseUtils.FILE_URI) && uri.length() > LSPEclipseUtils.FILE_URI | ||
| .length()) { | ||
| Range range = this.location.isLeft() ? this.location.getLeft().getRange() | ||
| : this.location.getRight().getTargetSelectionRange(); | ||
| int line = -1; | ||
| if (range != null && range.getStart() != null) { | ||
| line = range.getStart().getLine(); | ||
| } | ||
| return getFileBasedLabel(uri, line); | ||
| } | ||
| return getGenericUriBasedLabel(uri); | ||
| } | ||
| } | ||
|
|
||
| return ""; | ||
| } | ||
|
|
||
| private String getGenericUriBasedLabel(String uri) { | ||
| return uri; | ||
| } | ||
|
|
||
| private String getFileBasedLabel(String uriStr, int line) { | ||
| URI uri = URI.create(uriStr); | ||
| IWorkspaceRoot workspaceRoot = ResourcesPlugin.getWorkspace().getRoot(); | ||
| IFile[] files = workspaceRoot.findFilesForLocationURI(uri); | ||
| if (files != null && files.length == 1 && files[0].getProject() != null) { | ||
| IFile file = files[0]; | ||
|
|
||
| @SuppressWarnings("null") | ||
| String lineContent = getLineContent(line, file); | ||
| if (lineContent != null) { | ||
| return lineContent; | ||
| } else { | ||
| IPath containerPath = file.getParent().getProjectRelativePath(); | ||
| return file.getName() + DASH_SEPARATOR + file.getProject().getName() + (containerPath | ||
| .isEmpty() ? "" : IPath.SEPARATOR + containerPath.toString()); //$NON-NLS-1$ | ||
| } | ||
| } | ||
| Path path = Paths.get(uri); | ||
| return path.getFileName() + (path.getParent() == null || path.getParent().getParent() == null ? "" //$NON-NLS-1$ | ||
| : DASH_SEPARATOR + path.toString()); | ||
| } | ||
|
|
||
| /** | ||
| * Gets the text content of the given line in the given {@link IFile}. | ||
| * | ||
| * @param line | ||
| * the line index | ||
| * @param file | ||
| * the {@link IFile} | ||
| * @return the text content of the given line in the given {@link IFile} | ||
| */ | ||
| public static String getLineContent(int line, final @NonNull IFile file) { | ||
| try (InputStream is = file.getContents()) { | ||
| String content = new String(is.readNBytes((int)file.getLocation().toFile().length()), Charset | ||
| .forName(file.getCharset())); | ||
| IDocument document = new Document(content); | ||
|
|
||
| int offset = document.getLineOffset(line); | ||
| int length = document.getLineLength(line); | ||
|
|
||
| return document.get(offset, length).trim(); | ||
| } catch (Exception e) { | ||
| return null; | ||
| } | ||
| } | ||
| } |
Oops, something went wrong.