-
Notifications
You must be signed in to change notification settings - Fork 93
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Handles regular element names and ones with prefixes Fixes #589 Refer to #589 for code to test Signed-off-by: Nikolas Komonen <[email protected]>
- Loading branch information
1 parent
bc9f68b
commit 655e17c
Showing
14 changed files
with
696 additions
and
28 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
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
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
106 changes: 106 additions & 0 deletions
106
...ml/extensions/contentmodel/participants/codeactions/cvc_complex_type_2_4_aCodeAction.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,106 @@ | ||
| package org.eclipse.lsp4xml.extensions.contentmodel.participants.codeactions; | ||
|
|
||
| import java.text.Collator; | ||
| import java.util.ArrayList; | ||
| import java.util.Collection; | ||
| import java.util.List; | ||
| import java.util.TreeSet; | ||
|
|
||
| import org.eclipse.lsp4j.CodeAction; | ||
| import org.eclipse.lsp4j.Diagnostic; | ||
| import org.eclipse.lsp4j.Range; | ||
| import org.eclipse.lsp4xml.commons.CodeActionFactory; | ||
| import org.eclipse.lsp4xml.dom.DOMDocument; | ||
| import org.eclipse.lsp4xml.dom.DOMElement; | ||
| import org.eclipse.lsp4xml.dom.DOMNode; | ||
| import org.eclipse.lsp4xml.extensions.contentmodel.model.CMElementDeclaration; | ||
| import org.eclipse.lsp4xml.extensions.contentmodel.model.ContentModelManager; | ||
| import org.eclipse.lsp4xml.services.extensions.ICodeActionParticipant; | ||
| import org.eclipse.lsp4xml.services.extensions.IComponentProvider; | ||
| import org.eclipse.lsp4xml.settings.XMLFormattingOptions; | ||
| import org.eclipse.lsp4xml.utils.LevenshteinDistance; | ||
| import org.eclipse.lsp4xml.utils.XMLPositionUtility; | ||
|
|
||
| /** | ||
| * cvc_complex_type_2_4_a | ||
| */ | ||
| public class cvc_complex_type_2_4_aCodeAction implements ICodeActionParticipant { | ||
|
|
||
| private static final float MAX_DISTANCE_DIFF_RATIO = 0.4f; | ||
|
|
||
| @Override | ||
| public void doCodeAction(Diagnostic diagnostic, Range range, DOMDocument document, List<CodeAction> codeActions, | ||
| XMLFormattingOptions formattingSettings, IComponentProvider componentProvider) { | ||
| try { | ||
| int offset = document.offsetAt(diagnostic.getRange().getStart()); | ||
| DOMNode node = document.findNodeAt(offset); | ||
| if (node != null && node.isElement()) { | ||
| // Get element from the diagnostic | ||
| DOMElement element = (DOMElement) node; | ||
| DOMElement parentElement = element.getParentElement(); | ||
| String tagName = element.getSuffix(); | ||
| boolean suffixOnly = element.getPrefix() != null; | ||
|
|
||
| // Get the XSD element declaration of the parent element. | ||
| ContentModelManager contentModelManager = componentProvider.getComponent(ContentModelManager.class); | ||
| CMElementDeclaration cmElement = contentModelManager.findCMElement(parentElement); | ||
| if (cmElement != null) { | ||
| // Collect all possible elements from the parent element upon the offset start | ||
| // of the element | ||
| Collection<CMElementDeclaration> possibleElements = cmElement.getPossibleElements(parentElement, | ||
| element.getStart()); | ||
|
|
||
| // When added to these collections, the names will be ordered alphabetically | ||
| Collection<String> otherElementNames = new TreeSet<String>(Collator.getInstance()); | ||
| Collection<String> similarElementNames = new TreeSet<String>(Collator.getInstance()); | ||
|
|
||
| // Try to collect similar names coming from tag name | ||
| for (CMElementDeclaration possibleElement : possibleElements) { | ||
| String possibleElementName = possibleElement.getName(); | ||
| if (isSimilar(possibleElementName, tagName)) { | ||
| similarElementNames.add(possibleElementName); | ||
| } | ||
| else { | ||
| otherElementNames.add(possibleElementName); | ||
| } | ||
| } | ||
|
|
||
| // Create ranges for the replace. | ||
| List<Range> ranges = new ArrayList<>(); | ||
| ranges.add(XMLPositionUtility.selectStartTagName(element, suffixOnly)); | ||
| Range end = XMLPositionUtility.selectEndTag(element, suffixOnly); | ||
| if (end != null) { | ||
| ranges.add(end); | ||
| } | ||
|
|
||
| if (!similarElementNames.isEmpty()) { | ||
| // // Add code actions for each similar elements | ||
| for (String elementName : similarElementNames) { | ||
| CodeAction similarCodeAction = CodeActionFactory.replaceAt( | ||
| "Did you mean '" + elementName + "'?", elementName, document.getTextDocument(), | ||
| diagnostic, ranges); | ||
| codeActions.add(similarCodeAction); | ||
| } | ||
| } else { | ||
| // Add code actions for each possible elements | ||
| for (String elementName : otherElementNames) { | ||
| CodeAction otherCodeAction = CodeActionFactory.replaceAt( | ||
| "Replace with '" + elementName + "'", elementName, document.getTextDocument(), | ||
| diagnostic, ranges); | ||
| codeActions.add(otherCodeAction); | ||
| } | ||
| } | ||
| } | ||
| } | ||
|
|
||
| } catch (Exception e) { | ||
| // Do nothing | ||
| } | ||
| } | ||
|
|
||
| private static boolean isSimilar(String reference, String current) { | ||
| int threshold = Math.round(MAX_DISTANCE_DIFF_RATIO * reference.length()); | ||
| LevenshteinDistance levenshteinDistance = new LevenshteinDistance(threshold); | ||
| return levenshteinDistance.apply(reference, current) != -1; | ||
| } | ||
| } |
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
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
Oops, something went wrong.