simplification of DataProviders: single-element NAVIGATABLE_ARRAY is automatically...
authornik <Nikolay.Chashnikov@jetbrains.com>
Fri, 6 Feb 2015 09:41:56 +0000 (12:41 +0300)
committernik <Nikolay.Chashnikov@jetbrains.com>
Mon, 9 Feb 2015 07:58:09 +0000 (10:58 +0300)
platform/testRunner/src/com/intellij/execution/testframework/TestsUIUtil.java
xml/dom-openapi/src/com/intellij/util/xml/tree/DomModelTreeView.java

index cbb0373f34c1dc1d2ade1d70ca477e5b26dad26e..59e30a064581a99ad4fcaae33b90e0abfb92965c 100644 (file)
@@ -57,10 +57,6 @@ public class TestsUIUtil {
     if (testProxy == null) return null;
     if (AbstractTestProxy.DATA_KEY.is(dataId)) return testProxy;
     if (CommonDataKeys.NAVIGATABLE.is(dataId)) return getOpenFileDescriptor(testProxy, model);
-    if (CommonDataKeys.NAVIGATABLE_ARRAY.is(dataId)) {
-      final Navigatable openFileDescriptor = getOpenFileDescriptor(testProxy, model);
-      return openFileDescriptor != null ? new Navigatable[]{openFileDescriptor} : null;
-    }
     if (CommonDataKeys.PSI_ELEMENT.is(dataId)) {
       final Location location = testProxy.getLocation(project, properties.getScope());
       if (location != null) {
index 15c3de313d436c4d1df3b634ec379c203ec81360..546cea563d1fbc3f2ca25546213ec902dcc71f1d 100644 (file)
@@ -190,17 +190,17 @@ public class DomModelTreeView extends Wrapper implements DataProvider, Disposabl
   @Override
   @Nullable
   public Object getData(String dataId) {
-    if (DomModelTreeView.DATA_KEY.is(dataId)) {
+    if (DATA_KEY.is(dataId)) {
       return this;
     }
     final SimpleNode simpleNode = getTree().getSelectedNode();
     if (simpleNode instanceof AbstractDomElementNode) {
       final DomElement domElement = ((AbstractDomElementNode)simpleNode).getDomElement();
       if (domElement != null && domElement.isValid()) {
-        if (CommonDataKeys.NAVIGATABLE_ARRAY.is(dataId)) {
+        if (CommonDataKeys.NAVIGATABLE.is(dataId)) {
           final XmlElement tag = domElement.getXmlElement();
           if (tag instanceof Navigatable) {
-            return new Navigatable[] { (Navigatable)tag };
+            return tag;
           }
         }
       }