Better error logging.

Author: ItzNotABug

dfc/src/main/java/com/lazygeniouz/dfc/extension/Extension.kt

@@ -3,8 +3,8 @@ package com.lazygeniouz.dfc.file.internals
 import android.content.Context
 import android.net.Uri
 import android.webkit.MimeTypeMap
-import com.lazygeniouz.dfc.extension.logError
 import com.lazygeniouz.dfc.file.DocumentFileCompat
+import com.lazygeniouz.dfc.logger.ErrorLogger.logError
 import java.io.File
 
 /**
@@ -95,7 +95,7 @@ internal class RawDocumentFileCompat constructor(context: Context, var file: Fil
             if (target.createNewFile()) fromFile(context, target)
             else null
         } catch (exception: Exception) {
-            logError("Exception while creating a document: ${exception.message}")
+            logError("Exception while creating a document", exception)
             null
         }
     }

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

@@ -0,0 +1,16 @@
+package com.lazygeniouz.dfc.logger
+
+import android.util.Log
+
+/**
+ * A logging utility class.
+ */
+object ErrorLogger {
+
+    /**
+     * Log error to the logcat to let the developer know if something went wrong.
+     */
+    internal fun logError(message: String, throwable: Throwable?) {
+        Log.e("DocumentFileCompat", "$message: ${throwable?.message}")
+    }
+}

dfc/src/main/java/com/lazygeniouz/dfc/logger/ErrorLogger.kt

@@ -5,10 +5,10 @@ import android.content.Context
 import android.database.Cursor
 import android.net.Uri
 import android.provider.DocumentsContract
-import com.lazygeniouz.dfc.extension.logError
 import com.lazygeniouz.dfc.file.DocumentFileCompat
 import com.lazygeniouz.dfc.file.internals.SingleDocumentFileCompat
 import com.lazygeniouz.dfc.file.internals.TreeDocumentFileCompat
+import com.lazygeniouz.dfc.logger.ErrorLogger.logError
 
 /**
  * This class calls relevant queries on the [ContentResolver]
@@ -44,7 +44,7 @@ internal class ResolverCompat(
         return try {
             DocumentsContract.deleteDocument(contentResolver, uri)
         } catch (exception: Exception) {
-            logError("Exception while deleting document: ${exception.message}")
+            logError("Exception while deleting document", exception)
             false
         }
     }
@@ -58,7 +58,7 @@ internal class ResolverCompat(
         return try {
             (DocumentsContract.renameDocument(contentResolver, uri, name) != null)
         } catch (exception: Exception) {
-            logError("Exception while renaming document: ${exception.message}")
+            logError("Exception while renaming document", exception)
             false
         }
     }
@@ -75,7 +75,7 @@ internal class ResolverCompat(
         return try {
             DocumentsContract.createDocument(contentResolver, uri, mimeType, name)
         } catch (exception: Exception) {
-            logError("Exception while creating a document: ${exception.message}")
+            logError("Exception while creating a document", exception)
             null
         }
     }