From 944f130ce2f109bb300b1714e7df0cdd7d2cbb60 Mon Sep 17 00:00:00 2001 From: Tom Copeland Date: Mon, 18 Nov 2002 14:02:04 +0000 Subject: [PATCH] added some error checking git-svn-id: https://pmd.svn.sourceforge.net/svnroot/pmd/trunk@1250 51baf565-9d33-0410-a72c-fc3788e3496d --- .../src/net/sourceforge/pmd/jedit/PMDJEditPlugin.java | 8 ++++++++ 1 file changed, 8 insertions(+) diff --git a/pmd-jedit/src/net/sourceforge/pmd/jedit/PMDJEditPlugin.java b/pmd-jedit/src/net/sourceforge/pmd/jedit/PMDJEditPlugin.java index 046d9d898e..cae7191733 100644 --- a/pmd-jedit/src/net/sourceforge/pmd/jedit/PMDJEditPlugin.java +++ b/pmd-jedit/src/net/sourceforge/pmd/jedit/PMDJEditPlugin.java @@ -62,6 +62,10 @@ public class PMDJEditPlugin extends EditPlugin { public void instanceCheckDirectory(View view) { if (jEdit.getBooleanProperty(PMDJEditPlugin.OPTION_UI_DIRECTORY_POPUP)) { final String dir = JOptionPane.showInputDialog(jEdit.getFirstView(), "Please type in a directory to scan", NAME, JOptionPane.QUESTION_MESSAGE); + if (dir == null || !(new File(dir)).exists() || !(new File(dir)).isDirectory() ) { + JOptionPane.showMessageDialog(jEdit.getFirstView(), dir + " is not a valid directory name", NAME, JOptionPane.ERROR_MESSAGE); + return; + } process(findFilesInDirectory(dir)); } else { final VFSBrowser browser = (VFSBrowser)view.getDockableWindowManager().getDockable("vfs.browser"); @@ -81,6 +85,10 @@ public class PMDJEditPlugin extends EditPlugin { public void instanceCheckDirectoryRecursively(View view) { if (jEdit.getBooleanProperty(PMDJEditPlugin.OPTION_UI_DIRECTORY_POPUP)) { final String dir = JOptionPane.showInputDialog(jEdit.getFirstView(), "Please type in a directory to scan recursively", NAME, JOptionPane.QUESTION_MESSAGE); + if (dir == null || !(new File(dir)).exists() || !(new File(dir)).isDirectory() ) { + JOptionPane.showMessageDialog(jEdit.getFirstView(), dir + " is not a valid directory name", NAME, JOptionPane.ERROR_MESSAGE); + return; + } process(findFilesRecursively(dir)); } else { final VFSBrowser browser = (VFSBrowser)view.getDockableWindowManager().getDockable("vfs.browser");