[IO-348] Missing information in IllegalArgumentException thrown by org.apache.commons.io.FileUtils#validateListFilesParameters.

garydgregory · garydgregory · commit fe6e6f9fd25f · 2012-10-06T18:12:06.000Z
git-svn-id: https://svn.apache.org/repos/asf/commons/proper/io/trunk@1395139 13f79535-47bb-0310-9956-ffa450edef68
diff --git a/src/changes/changes.xml b/src/changes/changes.xml
@@ -47,6 +47,9 @@ The <action> type attribute can be add,update,fix,remove.
   <body>
     <!-- The release date is the date RC is cut -->
     <release version="2.5" date="201?-??-??" description="New features and bug fixes.">
+      <action issue="IO-348" dev="ggregory" type="add" due-to="plcstpierre">
+        Missing information in IllegalArgumentException thrown by org.apache.commons.io.FileUtils#validateListFilesParameters.
+      </action>            
       <action issue="IO-345" dev="ggregory" type="add" due-to="mkresse">
         Supply a hook method allowing Tailer actively determining stop condition.
       </action>            
diff --git a/src/main/java/org/apache/commons/io/FileUtils.java b/src/main/java/org/apache/commons/io/FileUtils.java
@@ -542,7 +542,7 @@ public static Collection<File> listFiles(
      */
     private static void validateListFilesParameters(File directory, IOFileFilter fileFilter) {
         if (!directory.isDirectory()) {
-            throw new IllegalArgumentException("Parameter 'directory' is not a directory");
+            throw new IllegalArgumentException("Parameter 'directory' is not a directory: " + directory);
         }
         if (fileFilter == null) {
             throw new NullPointerException("Parameter 'fileFilter' is null");

Original file line number	Diff line number	Diff line change
`@@ -542,7 +542,7 @@ public static Collection<File> listFiles(`
`542`	`542`	`*/`
`543`	`543`	`private static void validateListFilesParameters(File directory, IOFileFilter fileFilter) {`
`544`	`544`	`if (!directory.isDirectory()) {`
`545`		`- throw new IllegalArgumentException("Parameter 'directory' is not a directory");`
	`545`	`+ throw new IllegalArgumentException("Parameter 'directory' is not a directory: " + directory);`
`546`	`546`	`}`
`547`	`547`	`if (fileFilter == null) {`
`548`	`548`	`throw new NullPointerException("Parameter 'fileFilter' is null");`