MDS/openide
1
0
Fork 0
mirror of https://gitflic.ru/project/openide/openide.git synced 2026-09-27 10:03:11 +07:00
Code Issues Packages Projects Releases Wiki Activity

diagnostics for EA-38276 - NPE: XmlAttributeManipulator.getRangeInElement

Browse Source
This commit is contained in:
Dmitry Avdeev
2012-08-22 15:03:36 +04:00
parent 7377b61243
commit 76a375ca5a
1 changed files with 9 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files
xml/impl/src/com/intellij/psi/impl/source/resolve/reference/impl/manipulators/XmlAttributeManipulator.java
+9 -2
View File
@@ -16,6 +16,8 @@
package com.intellij.psi.impl.source.resolve.reference.impl.manipulators;
import com.intellij.lang.ASTNode;
import com.intellij.openapi.diagnostic.Logger;
import com.intellij.openapi.util.TextRange;
import com.intellij.psi.AbstractElementManipulator;
import com.intellij.psi.PsiElement;
@@ -23,13 +25,14 @@ import com.intellij.psi.xml.XmlAttribute;
import com.intellij.psi.xml.XmlAttributeValue;
import com.intellij.psi.xml.XmlChildRole;
import com.intellij.util.IncorrectOperationException;
import com.intellij.lang.ASTNode;
/**
* @author Gregory.Shrago
*/
public class XmlAttributeManipulator extends AbstractElementManipulator<XmlAttribute> {
private final static Logger LOG = Logger.getInstance(XmlAttributeManipulator.class);
public XmlAttribute handleContentChange(XmlAttribute attribute, TextRange range, String newContent) throws IncorrectOperationException {
String attr = attribute.getText();
ASTNode astNode = XmlChildRole.ATTRIBUTE_NAME_FINDER.findChild(attribute.getNode());
@@ -66,6 +69,10 @@ public class XmlAttributeManipulator extends AbstractElementManipulator<XmlAttri
public TextRange getRangeInElement(final XmlAttribute attribute) {
final XmlAttributeValue value = attribute.getValueElement();
if (value == null) return TextRange.from(0, 0);
return attribute.getValueTextRange().shiftRight(value.getStartOffsetInParent());
TextRange range = attribute.getValueTextRange();
if (range == null) {
LOG.error("Null range in " + attribute + " '" + attribute.getText() + "'");
}
return range.shiftRight(value.getStartOffsetInParent());
}
}