improve: api.

Author: ItzNotABug

dfc/src/main/java/com/lazygeniouz/dfc/file/DocumentFileCompat.kt

@@ -9,7 +9,6 @@ import com.lazygeniouz.dfc.controller.DocumentController
 import com.lazygeniouz.dfc.file.internals.RawDocumentFileCompat
 import com.lazygeniouz.dfc.file.internals.SingleDocumentFileCompat
 import com.lazygeniouz.dfc.file.internals.TreeDocumentFileCompat
-import com.lazygeniouz.dfc.resolver.ResolverCompat
 import java.io.File
 
 /**
@@ -59,6 +58,14 @@ abstract class DocumentFileCompat(
      */
     abstract fun createDirectory(name: String): DocumentFileCompat?
 
+    /**
+     * This will return a list of [DocumentFileCompat] with all the defined fields
+     * only when the current document is a **Directory**.
+     *
+     * A [UnsupportedOperationException] is thrown if the uri is not a directory.
+     */
+    abstract fun listFiles(): List<DocumentFileCompat>
+
     /**
      * Same as [listFiles] but allows specifying a custom [projection] (columns to query).
      *
@@ -188,16 +195,6 @@ abstract class DocumentFileCompat(
         return fileController.isDirectory()
     }
 
-    /**
-     * This will return a list of [DocumentFileCompat] with all the defined fields
-     * only when the current document is a **Directory**.
-     *
-     * A [UnsupportedOperationException] is thrown if the uri is not a directory.
-     */
-    open fun listFiles(): List<DocumentFileCompat> {
-        return listFiles(ResolverCompat.fullProjection)
-    }
-
     /**
      * Converts a non serializable [DocumentFileCompat] to a serializable [SerializedFile].
      */

dfc/src/main/java/com/lazygeniouz/dfc/file/internals/RawDocumentFileCompat.kt

@@ -96,15 +96,20 @@ internal class RawDocumentFileCompat(context: Context, var file: File) :
         else null
     }
 
+    /**
+     * Returns list of files using File API.
+     */
+    override fun listFiles(): List<DocumentFileCompat> {
+        return file.listFiles()?.map { child -> fromFile(context, child) } ?: emptyList()
+    }
+
     /**
      * Returns list of files using File API.
      *
-     * The performance of File api is pretty great as compared to others.
-     *
-     * Note: [projection] parameter is ignored as File API doesn't support custom projections.
+     * Note: [projection] is ignored as the File API doesn't support it.
      */
     override fun listFiles(projection: Array<String>): List<DocumentFileCompat> {
-        return file.listFiles()?.map { child -> fromFile(context, child) } ?: emptyList()
+        return listFiles()
     }
 
     /**

dfc/src/main/java/com/lazygeniouz/dfc/file/internals/SingleDocumentFileCompat.kt

@@ -46,10 +46,19 @@ internal class SingleDocumentFileCompat(
      *
      * @throws UnsupportedOperationException
      */
-    override fun listFiles(projection: Array<String>): List<DocumentFileCompat> {
+    override fun listFiles(): List<DocumentFileCompat> {
         throw UnsupportedOperationException()
     }
 
+    /**
+     * Single document Uris don't have children, so projections are not applicable.
+     *
+     * @throws UnsupportedOperationException
+     */
+    override fun listFiles(projection: Array<String>): List<DocumentFileCompat> {
+        return listFiles()
+    }
+
     /**
      * No [listFiles], no children, no count.
      *

dfc/src/main/java/com/lazygeniouz/dfc/file/internals/TreeDocumentFileCompat.kt

@@ -57,6 +57,10 @@ internal class TreeDocumentFileCompat(
         return fileController.listFiles(projection)
     }
 
+    override fun listFiles(): List<DocumentFileCompat> {
+        return fileController.listFiles()
+    }
+
     /**
      * This will return the children count in the directory.
      *