DoubleClick on the ToolWindow causes NullPointerException if no nodes are selected in the Tree

META-INF/plugin.xml

@@ -89,6 +89,5 @@
         <projectConfigurable instance="org.mvnsearch.intellij.plugin.zookeeper.ui.ZkProjectConfigurable"/>
         <projectService serviceInterface="org.mvnsearch.intellij.plugin.zookeeper.ZkConfigPersistence"
                         serviceImplementation="org.mvnsearch.intellij.plugin.zookeeper.ZkConfigPersistence"/>
-        <!--<psi.referenceContributor implementation="org.mvnsearch.intellij.plugin.zookeeper.ZkNodePathReferenceContributor"/>-->
     </extensions>
 </idea-plugin>

README.md

@@ -2,6 +2,12 @@ IntelliJ IDEA ZooKeeper Plugin
 =======================================
 ZooKeeper plugin for IntelliJ IDEA, and you can operate ZooKeeper directly in IDEA.
 
+### Install
+Add this to the Custom plugin repositories list:
+https://github.com/bonifaido/zookeeper-intellij/raw/master/plugins.xml
+
+Tested with IntelliJ 14.1.
+
 ### Features
 
 * ZooKeeper configuration for project: enable, host, port

plugins.xml

@@ -0,0 +1,3 @@
+<plugins>
+    <plugin id="me.bonifaido.zookeeper" url="https://github.com/bonifaido/zookeeper-intellij/raw/master/zookeeper-intellij.zip" version="0.1.0"/>
+</plugins>

src/main/java/org/mvnsearch/intellij/plugin/zookeeper/ZkConfigPersistence.java

@@ -1,15 +1,15 @@
 package org.mvnsearch.intellij.plugin.zookeeper;
 
-import com.intellij.openapi.components.*;
+import com.intellij.openapi.components.PersistentStateComponent;
+import com.intellij.openapi.components.ServiceManager;
+import com.intellij.openapi.components.State;
+import com.intellij.openapi.components.Storage;
+import com.intellij.openapi.components.StoragePathMacros;
 import com.intellij.openapi.project.Project;
 import com.intellij.openapi.util.text.StringUtil;
 import com.intellij.util.xmlb.XmlSerializerUtil;
 import org.jetbrains.annotations.Nullable;
 
-import java.util.Arrays;
-import java.util.Collections;
-import java.util.List;
-
 /**
  * zookeeper configuration persistence
  *
@@ -29,10 +29,12 @@ public static ZkConfigPersistence getInstance(Project project) {
     }
 
     @Nullable
+    @Override
     public ZkConfigPersistence getState() {
         return this;
     }
 
+    @Override
     public void loadState(ZkConfigPersistence state) {
         XmlSerializerUtil.copyBean(state, this);
     }

src/main/java/org/mvnsearch/intellij/plugin/zookeeper/ZkProjectComponent.java

@@ -21,7 +21,6 @@
 import com.intellij.ui.content.Content;
 import com.intellij.ui.content.ContentManager;
 import com.intellij.ui.treeStructure.Tree;
-import com.intellij.util.IconUtil;
 import org.apache.curator.framework.CuratorFramework;
 import org.apache.curator.framework.CuratorFrameworkFactory;
 import org.apache.curator.retry.ExponentialBackoffRetry;
@@ -33,14 +32,18 @@
 import org.mvnsearch.intellij.plugin.zookeeper.ui.ZkTreeModel;
 import org.mvnsearch.intellij.plugin.zookeeper.vfs.ZkVirtualFileSystem;
 
-import javax.swing.*;
+import javax.swing.Icon;
+import javax.swing.JComponent;
+import javax.swing.JPanel;
+import javax.swing.JTree;
+import javax.swing.ToolTipManager;
 import javax.swing.tree.TreePath;
 import javax.swing.tree.TreeSelectionModel;
-import java.awt.*;
+import java.awt.Component;
+import java.awt.GridLayout;
 import java.awt.event.MouseEvent;
 import java.io.ByteArrayOutputStream;
 import java.net.Socket;
-import java.util.Arrays;
 
 /**
  * Zoo Keeper project component
@@ -62,19 +65,23 @@ public static ZkProjectComponent getInstance(Project project) {
         return project.getComponent(ZkProjectComponent.class);
     }
 
+    @Override
     public void initComponent() {
 
     }
 
+    @Override
     public void disposeComponent() {
 
     }
 
     @NotNull
+    @Override
     public String getComponentName() {
         return "ZkProjectComponent";
     }
 
+    @Override
     public void projectOpened() {
         initZk();
         if (this.curator != null) {
@@ -140,15 +147,20 @@ private JComponent createToolBar() {
         return toolBarPanel;
     }
 
+    @Override
     public void projectClosed() {
         if (curator != null) {
             this.curator.close();
         }
     }
 
+    @Override
     protected boolean onDoubleClick(MouseEvent mouseEvent) {
         Tree source = (Tree) mouseEvent.getSource();
         TreePath treePath = source.getSelectionPath();
+        if (treePath == null) {
+            return true;
+        }
         ZkNode selectedNode = (ZkNode) treePath.getLastPathComponent();
         if (selectedNode.isLeaf() && !selectedNode.isBinary()) {
             VirtualFile file = fileSystem.findFileByPath(selectedNode.getFilePath());

src/main/java/org/mvnsearch/intellij/plugin/zookeeper/actions/CopyNodePathAction.java

@@ -9,7 +9,7 @@
 import org.mvnsearch.intellij.plugin.zookeeper.ui.ZkNode;
 
 import javax.swing.tree.TreePath;
-import java.awt.*;
+import java.awt.Toolkit;
 import java.awt.datatransfer.Clipboard;
 import java.awt.datatransfer.ClipboardOwner;
 import java.awt.datatransfer.StringSelection;
@@ -21,6 +21,7 @@
  * @author linux_china
  */
 public class CopyNodePathAction extends AnAction implements ClipboardOwner {
+    @Override
     public void actionPerformed(final AnActionEvent anActionEvent) {
         Project project = anActionEvent.getProject();
         ZkProjectComponent zkProjectComponent = ZkProjectComponent.getInstance(project);
@@ -36,6 +37,7 @@ public void actionPerformed(final AnActionEvent anActionEvent) {
         }
     }
 
+    @Override
     public void lostOwnership(Clipboard clipboard, Transferable transferable) {
 
     }

src/main/java/org/mvnsearch/intellij/plugin/zookeeper/actions/CreateNodeAction.java

@@ -16,7 +16,7 @@
 import org.mvnsearch.intellij.plugin.zookeeper.ui.ZkNode;
 import org.mvnsearch.intellij.plugin.zookeeper.vfs.ZkNodeVirtualFile;
 
-import javax.swing.*;
+import javax.swing.JTextField;
 import javax.swing.tree.TreePath;
 
 /**
@@ -25,13 +25,15 @@
  * @author linux_china
  */
 public class CreateNodeAction extends AnAction {
+    @Override
     public void actionPerformed(final AnActionEvent anActionEvent) {
         final DialogBuilder builder = new DialogBuilder(anActionEvent.getProject());
         builder.setTitle("Create Node");
         final JTextField jTextField = new JTextField();
         builder.setPreferredFocusComponent(jTextField);
         builder.setCenterPanel(jTextField);
         builder.setOkOperation(new Runnable() {
+            @Override
             public void run() {
                 String nodeName = jTextField.getText();
                 if (StringUtil.isNotEmpty(nodeName)) {

src/main/java/org/mvnsearch/intellij/plugin/zookeeper/actions/DeleteNodeAction.java

@@ -11,7 +11,7 @@
 import org.mvnsearch.intellij.plugin.zookeeper.ZkProjectComponent;
 import org.mvnsearch.intellij.plugin.zookeeper.ui.ZkNode;
 
-import javax.swing.*;
+import javax.swing.JLabel;
 import javax.swing.tree.TreePath;
 
 /**
@@ -20,6 +20,7 @@
  * @author linux_china
  */
 public class DeleteNodeAction extends AnAction {
+    @Override
     public void actionPerformed(final AnActionEvent anActionEvent) {
         final Project project = anActionEvent.getProject();
         final ZkProjectComponent zkProjectComponent = ZkProjectComponent.getInstance(project);
@@ -31,6 +32,7 @@ public void actionPerformed(final AnActionEvent anActionEvent) {
         final JLabel jTextField = new JLabel("Path: " + currentNode.getFilePath());
         builder.setCenterPanel(jTextField);
         builder.setOkOperation(new Runnable() {
+            @Override
             public void run() {
                 CuratorFramework curator = zkProjectComponent.getCurator();
                 try {

src/main/java/org/mvnsearch/intellij/plugin/zookeeper/actions/OpenZkNodeInEditorAction.java

@@ -17,6 +17,7 @@
  */
 public class OpenZkNodeInEditorAction extends AnAction {
 
+    @Override
     public void actionPerformed(AnActionEvent actionEvent) {
         ZkProjectComponent zkProjectComponent = ZkProjectComponent.getInstance(actionEvent.getProject());
         Tree zkTree = zkProjectComponent.getZkTree();

src/main/java/org/mvnsearch/intellij/plugin/zookeeper/actions/UpdateZkNodeAction.java

@@ -1,7 +1,7 @@
 package org.mvnsearch.intellij.plugin.zookeeper.actions;
 
-import com.intellij.openapi.actionSystem.CommonDataKeys;
 import com.intellij.openapi.actionSystem.DataContext;
+import com.intellij.openapi.actionSystem.PlatformDataKeys;
 import com.intellij.openapi.editor.Editor;
 import com.intellij.openapi.editor.actionSystem.EditorAction;
 import com.intellij.openapi.editor.actionSystem.EditorActionHandler;
@@ -22,11 +22,11 @@ public UpdateZkNodeAction() {
         super(new EditorActionHandler() {
             @Override
             public void execute(Editor editor, DataContext dataContext) {
-                VirtualFile virtualFile = CommonDataKeys.VIRTUAL_FILE.getData(dataContext);
+                VirtualFile virtualFile = PlatformDataKeys.VIRTUAL_FILE.getData(dataContext);
                 if (virtualFile != null && virtualFile instanceof ZkNodeVirtualFile) {
                     ZkNodeVirtualFile nodeFile = (ZkNodeVirtualFile) virtualFile;
                     String nodeContent = editor.getDocument().getText();
-                    Project project = CommonDataKeys.PROJECT.getData(dataContext);
+                    Project project = PlatformDataKeys.PROJECT.getData(dataContext);
                     ZkProjectComponent zkProjectComponent = ZkProjectComponent.getInstance(project);
                     try {
                         if (nodeFile.isSingleFileZip()) {
@@ -44,7 +44,7 @@ public void execute(Editor editor, DataContext dataContext) {
 
             @Override
             public boolean isEnabled(Editor editor, DataContext dataContext) {
-                VirtualFile virtualFile = CommonDataKeys.VIRTUAL_FILE.getData(dataContext);
+                VirtualFile virtualFile = PlatformDataKeys.VIRTUAL_FILE.getData(dataContext);
                 return virtualFile instanceof ZkNodeVirtualFile;
             }
         });

src/main/java/org/mvnsearch/intellij/plugin/zookeeper/actions/ZkTreeRefreshAction.java

@@ -13,6 +13,7 @@
  * @author linux_china
  */
 public class ZkTreeRefreshAction extends AnAction {
+    @Override
     public void actionPerformed(AnActionEvent anActionEvent) {
         ZkProjectComponent zkProjectComponent = ZkProjectComponent.getInstance(anActionEvent.getProject());
         Tree zkTree = zkProjectComponent.getZkTree();
@@ -21,6 +22,5 @@ public void actionPerformed(AnActionEvent anActionEvent) {
         if (treePath != null) {
             zkTree.expandPath(treePath);
         }
-
     }
 }

src/main/java/org/mvnsearch/intellij/plugin/zookeeper/ui/ZkNode.java

@@ -1,6 +1,5 @@
 package org.mvnsearch.intellij.plugin.zookeeper.ui;
 
-import com.intellij.openapi.fileTypes.FileTypes;
 import org.apache.zookeeper.data.Stat;
 
 import java.util.Arrays;