Merge branch 'npeFix' into 'develop'

Add logging for potential NPE

See merge request ultrasonic/ultrasonic!933

gradle_scripts/code_quality.gradle

@@ -31,5 +31,4 @@ if (isCodeQualityEnabled) {
         }
         tasks.detekt.jvmTarget = "11"
     }
-
 }

ultrasonic/src/main/kotlin/org/moire/ultrasonic/fragment/TrackCollectionFragment.kt

@@ -600,14 +600,17 @@ open class TrackCollectionFragment(
                 listModel.getRandom(size, append)
             } else {
                 setTitle(name)
+                requireNotNull(id) {
+                    "ID must be set. NavArgs: ${navArgs.toBundle()}"
+                }
                 if (ActiveServerProvider.isID3Enabled()) {
                     if (isAlbum) {
-                        listModel.getAlbum(refresh2, id!!, name)
+                        listModel.getAlbum(refresh2, id, name)
                     } else {
                         throw IllegalAccessException("Use AlbumFragment instead!")
                     }
                 } else {
-                    listModel.getMusicDirectory(refresh2, id!!, name)
+                    listModel.getMusicDirectory(refresh2, id, name)
                 }
             }
 

ultrasonic/src/main/kotlin/org/moire/ultrasonic/util/CommunicationError.kt

@@ -39,14 +39,14 @@ object CommunicationError {
     }
 
     @JvmStatic
-    fun handleError(error: Throwable?, context: Context?) {
+    fun handleError(error: Throwable, context: Context?) {
         Timber.w(error)
 
         if (context == null) return
 
         ErrorDialog(
             context = context,
-            message = getErrorMessage(error!!, context)
+            message = getErrorMessage(error, context)
         ).show()
     }