Merge pull request #304 from aviraxp/patch-1

Avoid starting foreground service repeatedly

Author: xishang0128

service/src/main/java/com/github/kr328/clash/service/clash/module/DynamicNotificationModule.kt

@@ -5,6 +5,7 @@ import android.app.Service
 import android.content.Intent
 import android.os.PowerManager
 import androidx.core.app.NotificationCompat
+import androidx.core.app.NotificationManagerCompat
 import androidx.core.content.getSystemService
 import com.github.kr328.clash.common.compat.getColorCompat
 import com.github.kr328.clash.common.compat.pendingIntentFlags
@@ -40,6 +41,8 @@ class DynamicNotificationModule(service: Service) : Module<Unit>(service) {
             )
         )
 
+    private val notificationManager = NotificationManagerCompat.from(service)
+
     private fun update() {
         val now = Clash.queryTrafficNow()
         val total = Clash.queryTrafficTotal()
@@ -64,7 +67,7 @@ class DynamicNotificationModule(service: Service) : Module<Unit>(service) {
             )
             .build()
 
-        service.startForeground(R.id.nf_clash_status, notification)
+        notificationManager.notify(R.id.nf_clash_status, notification)
     }
 
     override suspend fun run() = coroutineScope {