From 7a64b201d8c09d9ca1a167e7b8bcd71d2b777701 Mon Sep 17 00:00:00 2001 From: Julia Beliaeva Date: Thu, 10 Mar 2016 18:14:16 +0300 Subject: [PATCH] [vcs-log] use getRequiredData instead of null checks --- .../vcs/log/ui/actions/GoToHashOrRefAction.java | 10 ++++------ 1 file changed, 4 insertions(+), 6 deletions(-) diff --git a/platform/vcs-log/impl/src/com/intellij/vcs/log/ui/actions/GoToHashOrRefAction.java b/platform/vcs-log/impl/src/com/intellij/vcs/log/ui/actions/GoToHashOrRefAction.java index c4b040444966..338dba40e373 100644 --- a/platform/vcs-log/impl/src/com/intellij/vcs/log/ui/actions/GoToHashOrRefAction.java +++ b/platform/vcs-log/impl/src/com/intellij/vcs/log/ui/actions/GoToHashOrRefAction.java @@ -16,6 +16,7 @@ package com.intellij.vcs.log.ui.actions; import com.intellij.openapi.actionSystem.AnActionEvent; +import com.intellij.openapi.actionSystem.CommonDataKeys; import com.intellij.openapi.project.DumbAwareAction; import com.intellij.openapi.project.Project; import com.intellij.openapi.vfs.VirtualFile; @@ -35,12 +36,9 @@ public class GoToHashOrRefAction extends DumbAwareAction { @Override public void actionPerformed(AnActionEvent e) { - Project project = e.getProject(); - final VcsLog log = e.getData(VcsLogDataKeys.VCS_LOG); - final VcsLogUiImpl logUi = (VcsLogUiImpl)e.getData(VcsLogDataKeys.VCS_LOG_UI); - if (project == null || log == null || logUi == null) { - return; - } + Project project = e.getRequiredData(CommonDataKeys.PROJECT); + final VcsLog log = e.getRequiredData(VcsLogDataKeys.VCS_LOG); + final VcsLogUiImpl logUi = (VcsLogUiImpl)e.getRequiredData(VcsLogDataKeys.VCS_LOG_UI); Set visibleRoots = VcsLogUtil.getVisibleRoots(logUi); Collection visibleBranches = VcsLogUtil.getVisibleBranches(log, visibleRoots);