diff --git a/app-common/src/main/java/im/angry/openeuicc/service/EuiccChannelManagerService.kt b/app-common/src/main/java/im/angry/openeuicc/service/EuiccChannelManagerService.kt index 788a251..fa22dc1 100644 --- a/app-common/src/main/java/im/angry/openeuicc/service/EuiccChannelManagerService.kt +++ b/app-common/src/main/java/im/angry/openeuicc/service/EuiccChannelManagerService.kt @@ -4,6 +4,7 @@ import android.content.Intent import android.content.pm.PackageManager import android.os.Binder import android.os.IBinder +import android.util.Log import androidx.core.app.NotificationChannelCompat import androidx.core.app.NotificationCompat import androidx.core.app.NotificationManagerCompat @@ -49,6 +50,7 @@ import net.typeblog.lpac_jni.ProfileDownloadCallback */ class EuiccChannelManagerService : LifecycleService(), OpenEuiccContextMarker { companion object { + private const val TAG = "EuiccChannelManagerService" private const val CHANNEL_ID = "tasks" private const val FOREGROUND_ID = 1000 } @@ -105,12 +107,18 @@ class EuiccChannelManagerService : LifecycleService(), OpenEuiccContextMarker { } private suspend fun updateForegroundNotification(title: String, iconRes: Int) { - val channel = - NotificationChannelCompat.Builder(CHANNEL_ID, NotificationManagerCompat.IMPORTANCE_LOW) - .setName(getString(R.string.task_notification)) - .setVibrationEnabled(false) - .build() - NotificationManagerCompat.from(this).createNotificationChannel(channel) + val nm = NotificationManagerCompat.from(this) + if (nm.getNotificationChannelCompat(CHANNEL_ID) == null) { + val channel = + NotificationChannelCompat.Builder( + CHANNEL_ID, + NotificationManagerCompat.IMPORTANCE_LOW + ) + .setName(getString(R.string.task_notification)) + .setVibrationEnabled(false) + .build() + nm.createNotificationChannel(channel) + } val state = foregroundTaskState.value @@ -127,7 +135,7 @@ class EuiccChannelManagerService : LifecycleService(), OpenEuiccContextMarker { if (state.progress == 0) { startForeground(FOREGROUND_ID, notification) } else if (checkSelfPermission(android.Manifest.permission.POST_NOTIFICATIONS) == PackageManager.PERMISSION_GRANTED) { - NotificationManagerCompat.from(this).notify(FOREGROUND_ID, notification) + nm.notify(FOREGROUND_ID, notification) } // Yield out so that the main looper can handle the notification event @@ -192,6 +200,8 @@ class EuiccChannelManagerService : LifecycleService(), OpenEuiccContextMarker { // This update will be sent by the subscriber (as shown below) foregroundTaskState.value = ForegroundTaskState.Done(null) } catch (t: Throwable) { + Log.e(TAG, "Foreground task encountered an error") + Log.e(TAG, Log.getStackTraceString(t)) foregroundTaskState.value = ForegroundTaskState.Done(t) } finally { stopSelf()