diff --git a/platform/lang-api/src/com/intellij/ide/actions/searcheverywhere/SearchEverywhereContributor.java b/platform/lang-api/src/com/intellij/ide/actions/searcheverywhere/SearchEverywhereContributor.java index 42bbf25db5ce..8302a47d183c 100644 --- a/platform/lang-api/src/com/intellij/ide/actions/searcheverywhere/SearchEverywhereContributor.java +++ b/platform/lang-api/src/com/intellij/ide/actions/searcheverywhere/SearchEverywhereContributor.java @@ -3,7 +3,6 @@ */ package com.intellij.ide.actions.searcheverywhere; -import com.intellij.openapi.actionSystem.DataContext; import com.intellij.openapi.extensions.ExtensionPointName; import com.intellij.openapi.progress.ProgressIndicator; import org.jetbrains.annotations.NotNull; @@ -44,8 +43,7 @@ public interface SearchEverywhereContributor { ListCellRenderer getElementsRenderer(); - @NotNull - DataContext getDataContextForItem(Object element); + Object getDataForItem(Object element, String dataId); static List getProviders() { return Arrays.asList(EP_NAME.getExtensions()); diff --git a/platform/lang-impl/src/com/intellij/ide/actions/searcheverywhere/AbstractGotoSEContributor.java b/platform/lang-impl/src/com/intellij/ide/actions/searcheverywhere/AbstractGotoSEContributor.java index 55eda86b63b4..3db88096ceb4 100644 --- a/platform/lang-impl/src/com/intellij/ide/actions/searcheverywhere/AbstractGotoSEContributor.java +++ b/platform/lang-impl/src/com/intellij/ide/actions/searcheverywhere/AbstractGotoSEContributor.java @@ -5,7 +5,6 @@ import com.intellij.codeInsight.navigation.NavigationUtil; import com.intellij.ide.util.gotoByName.ChooseByNameModel; import com.intellij.ide.util.gotoByName.ChooseByNamePopup; import com.intellij.openapi.actionSystem.CommonDataKeys; -import com.intellij.openapi.actionSystem.DataContext; import com.intellij.openapi.diagnostic.Logger; import com.intellij.openapi.progress.ProgressIndicator; import com.intellij.openapi.project.DumbService; @@ -83,11 +82,7 @@ public abstract class AbstractGotoSEContributor implements SearchEverywhereContr } @Override - public DataContext getDataContextForItem(Object element) { - return (dataId) -> getItemData(dataId, element); - } - - protected Object getItemData(String dataId, Object element) { + public Object getDataForItem(Object element, String dataId) { if (CommonDataKeys.PSI_ELEMENT.is(dataId)) { return element; } diff --git a/platform/lang-impl/src/com/intellij/ide/actions/searcheverywhere/ActionSearchEverywhereContributor.java b/platform/lang-impl/src/com/intellij/ide/actions/searcheverywhere/ActionSearchEverywhereContributor.java index 344aed9959e0..1c6bdf0f758f 100644 --- a/platform/lang-impl/src/com/intellij/ide/actions/searcheverywhere/ActionSearchEverywhereContributor.java +++ b/platform/lang-impl/src/com/intellij/ide/actions/searcheverywhere/ActionSearchEverywhereContributor.java @@ -98,10 +98,9 @@ public class ActionSearchEverywhereContributor implements SearchEverywhereContri return ActionSearchEverywhereContributor.class.getSimpleName(); } - @NotNull @Override - public DataContext getDataContextForItem(Object element) { - return DataContext.EMPTY_CONTEXT; + public Object getDataForItem(Object element, String dataId) { + return null; } @Override diff --git a/platform/lang-impl/src/com/intellij/ide/actions/searcheverywhere/FileSearchEverywhereContributor.java b/platform/lang-impl/src/com/intellij/ide/actions/searcheverywhere/FileSearchEverywhereContributor.java index 00777fd93597..dd80f76e0cf3 100644 --- a/platform/lang-impl/src/com/intellij/ide/actions/searcheverywhere/FileSearchEverywhereContributor.java +++ b/platform/lang-impl/src/com/intellij/ide/actions/searcheverywhere/FileSearchEverywhereContributor.java @@ -66,12 +66,12 @@ public class FileSearchEverywhereContributor extends AbstractGotoSEContributor { } @Override - protected Object getItemData(String dataId, Object element) { + public Object getDataForItem(Object element, String dataId) { if (CommonDataKeys.PSI_FILE.is(dataId)) { return element; } - return super.getItemData(dataId, element); + return super.getDataForItem(element, dataId); } @Override diff --git a/platform/lang-impl/src/com/intellij/ide/actions/searcheverywhere/SearchEverywhereUI.java b/platform/lang-impl/src/com/intellij/ide/actions/searcheverywhere/SearchEverywhereUI.java index e97100efa369..ca752ad34d38 100644 --- a/platform/lang-impl/src/com/intellij/ide/actions/searcheverywhere/SearchEverywhereUI.java +++ b/platform/lang-impl/src/com/intellij/ide/actions/searcheverywhere/SearchEverywhereUI.java @@ -229,8 +229,7 @@ public class SearchEverywhereUI extends BorderLayoutPanel implements Disposable, } SearchEverywhereContributor contributor = myListModel.getContributorForIndex(index); - DataContext context = contributor.getDataContextForItem(myListModel.getElementAt(index)); - return context.getData(dataId); + return contributor.getDataForItem(myListModel.getElementAt(index), dataId); } @Override diff --git a/platform/lang-impl/src/com/intellij/ide/actions/searcheverywhere/TopHitSEContributor.java b/platform/lang-impl/src/com/intellij/ide/actions/searcheverywhere/TopHitSEContributor.java index c0b03b4ab79f..3193a749be44 100644 --- a/platform/lang-impl/src/com/intellij/ide/actions/searcheverywhere/TopHitSEContributor.java +++ b/platform/lang-impl/src/com/intellij/ide/actions/searcheverywhere/TopHitSEContributor.java @@ -9,7 +9,10 @@ import com.intellij.ide.ui.search.BooleanOptionDescription; import com.intellij.ide.ui.search.OptionDescription; import com.intellij.navigation.ItemPresentation; import com.intellij.navigation.NavigationItem; -import com.intellij.openapi.actionSystem.*; +import com.intellij.openapi.actionSystem.AbbreviationManager; +import com.intellij.openapi.actionSystem.ActionManager; +import com.intellij.openapi.actionSystem.AnAction; +import com.intellij.openapi.actionSystem.Presentation; import com.intellij.openapi.actionSystem.ex.ActionUtil; import com.intellij.openapi.application.ApplicationManager; import com.intellij.openapi.progress.ProgressIndicator; @@ -93,10 +96,9 @@ public class TopHitSEContributor implements SearchEverywhereContributor { return new ContributorSearchResult<>(new ArrayList<>(res), interrupted); } - @NotNull @Override - public DataContext getDataContextForItem(Object element) { - return DataContext.EMPTY_CONTEXT; + public Object getDataForItem(Object element, String dataId) { + return null; } @Override