Difuse/difuse-phone-android
2
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

Improved error log in case it is 'expected'

Browse source
This commit is contained in:
Sylvain Berfini 2023-05-22 09:41:23 +02:00
parent 666a4f1ad0
commit 41f6139137
2 changed files with 10 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
app/src/main/java/org/linphone/core/CoreService.kt
View file

@ -34,7 +34,8 @@ class CoreService : CoreService() {
} }
override fun onStartCommand(intent: Intent?, flags: Int, startId: Int): Int { override fun onStartCommand(intent: Intent?, flags: Int, startId: Int): Int {
Log.i("[Service] Ensuring Core exists") Log.i("[Service] Starting, ensuring Core exists")
if (corePreferences.keepServiceAlive) { if (corePreferences.keepServiceAlive) {
Log.i("[Service] Starting as foreground to keep app alive in background") Log.i("[Service] Starting as foreground to keep app alive in background")
if (!ensureCoreExists( if (!ensureCoreExists(

9
app/src/main/java/org/linphone/notifications/NotificationsManager.kt
View file

@ -320,7 +320,14 @@ class NotificationsManager(private val context: Context) {
try { try {
notificationManager.notify(tag, id, notification) notificationManager.notify(tag, id, notification)
} catch (iae: IllegalArgumentException) { } catch (iae: IllegalArgumentException) {
Log.e("[Notifications Manager] Exception occurred: $iae") if (service == null && tag == null) {
// We can't notify using CallStyle if there isn't a foreground service running
Log.w(
"[Notifications Manager] Foreground service hasn't started yet, can't display a CallStyle notification until then: $iae"
)
} else {
Log.e("[Notifications Manager] Exception occurred: $iae")
}
} }
} }