From c9755b193a04be003fd466ef092e53f30137f2ce Mon Sep 17 00:00:00 2001
From: ribeaudc <ribeaudc>
Date: Sat, 20 Oct 2007 00:38:34 +0000
Subject: [PATCH] change: - listFiles moved from 'Utilities' to 'FileUtilities'

SVN: 2218
---
 .../java/ch/systemsx/cisd/bds/Utilities.java  | 22 -------------------
 .../bds/storage/filesystem/Directory.java     |  5 ++---
 .../cisd/common/utilities/FileUtilities.java  | 20 +++++++++++++++++
 3 files changed, 22 insertions(+), 25 deletions(-)

diff --git a/bds/source/java/ch/systemsx/cisd/bds/Utilities.java b/bds/source/java/ch/systemsx/cisd/bds/Utilities.java
index 948a0623a39..5b2ccfcdf62 100644
--- a/bds/source/java/ch/systemsx/cisd/bds/Utilities.java
+++ b/bds/source/java/ch/systemsx/cisd/bds/Utilities.java
@@ -16,12 +16,10 @@
 
 package ch.systemsx.cisd.bds;
 
-import java.io.File;
 
 import ch.systemsx.cisd.bds.storage.IDirectory;
 import ch.systemsx.cisd.bds.storage.IFile;
 import ch.systemsx.cisd.bds.storage.INode;
-import ch.systemsx.cisd.common.exceptions.EnvironmentFailureException;
 import ch.systemsx.cisd.common.exceptions.UserFailureException;
 
 /**
@@ -101,26 +99,6 @@ public class Utilities
         return file.getStringContent();
     }
 
-    /**
-     * Lists files of given <var>directory</var>.
-     * <p>
-     * Throws an <code>EnvironmentFailureException</code> if {@link File#listFiles()} returns <code>null</code>.
-     * </p>
-     * 
-     * @param directory must be a directory.
-     */
-    public final static File[] listFiles(final File directory) throws EnvironmentFailureException
-    {
-        assert directory.isDirectory();
-        final File[] fileList = directory.listFiles();
-        if (fileList == null)
-        {
-            throw EnvironmentFailureException.fromTemplate("Was not able to list files of directory '%s'", directory
-                    .getAbsolutePath());
-        }
-        return fileList;
-    }
-
     private Utilities()
     {
     }
diff --git a/bds/source/java/ch/systemsx/cisd/bds/storage/filesystem/Directory.java b/bds/source/java/ch/systemsx/cisd/bds/storage/filesystem/Directory.java
index 2614c054ffa..ca9c668c57d 100644
--- a/bds/source/java/ch/systemsx/cisd/bds/storage/filesystem/Directory.java
+++ b/bds/source/java/ch/systemsx/cisd/bds/storage/filesystem/Directory.java
@@ -21,7 +21,6 @@ import java.util.Iterator;
 
 import org.apache.commons.io.FileUtils;
 
-import ch.systemsx.cisd.bds.Utilities;
 import ch.systemsx.cisd.bds.storage.IDirectory;
 import ch.systemsx.cisd.bds.storage.IFile;
 import ch.systemsx.cisd.bds.storage.ILink;
@@ -46,7 +45,7 @@ class Directory extends AbstractNode implements IDirectory
 
     public INode tryToGetNode(String name)
     {
-        final java.io.File[] files = Utilities.listFiles(nodeFile);
+        final java.io.File[] files = FileUtilities.listFiles(nodeFile);
         for (java.io.File file : files)
         {
             if (file.getName().equals(name))
@@ -101,7 +100,7 @@ class Directory extends AbstractNode implements IDirectory
     {
         return new Iterator<INode>()
             {
-                private java.io.File[] files = Utilities.listFiles(nodeFile);
+                private java.io.File[] files = FileUtilities.listFiles(nodeFile);
 
                 private int index;
 
diff --git a/common/source/java/ch/systemsx/cisd/common/utilities/FileUtilities.java b/common/source/java/ch/systemsx/cisd/common/utilities/FileUtilities.java
index 8432fc111eb..a3e15a2c008 100644
--- a/common/source/java/ch/systemsx/cisd/common/utilities/FileUtilities.java
+++ b/common/source/java/ch/systemsx/cisd/common/utilities/FileUtilities.java
@@ -717,4 +717,24 @@ public final class FileUtilities
             return null;
         }
     }
+
+    /**
+     * Lists files of given <var>directory</var>.
+     * <p>
+     * Throws an <code>EnvironmentFailureException</code> if {@link File#listFiles()} returns <code>null</code>.
+     * </p>
+     * 
+     * @param directory must be a directory.
+     */
+    public final static File[] listFiles(final File directory) throws EnvironmentFailureException
+    {
+        assert directory.isDirectory();
+        final File[] fileList = directory.listFiles();
+        if (fileList == null)
+        {
+            throw EnvironmentFailureException.fromTemplate("Failed to get listing of directory '%s'", directory
+                    .getAbsolutePath());
+        }
+        return fileList;
+    }
 }
-- 
GitLab