From 76a375ca5ac2fcfb0595888acd2f9cb82239d915 Mon Sep 17 00:00:00 2001
From: Dmitry Avdeev <Dmitry.Avdeev@jetbrains.com>
Date: Wed, 22 Aug 2012 15:03:36 +0400
Subject: [PATCH] diagnostics for EA-38276 - NPE:
 XmlAttributeManipulator.getRangeInElement

---
 .../impl/manipulators/XmlAttributeManipulator.java    | 11 +++++++++--
 1 file changed, 9 insertions(+), 2 deletions(-)

diff --git a/xml/impl/src/com/intellij/psi/impl/source/resolve/reference/impl/manipulators/XmlAttributeManipulator.java b/xml/impl/src/com/intellij/psi/impl/source/resolve/reference/impl/manipulators/XmlAttributeManipulator.java
index 77074ed92cd7..0039c32e2a75 100644
--- a/xml/impl/src/com/intellij/psi/impl/source/resolve/reference/impl/manipulators/XmlAttributeManipulator.java
+++ b/xml/impl/src/com/intellij/psi/impl/source/resolve/reference/impl/manipulators/XmlAttributeManipulator.java
@@ -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());
   }
 }
\ No newline at end of file