From 4638acf2d31564815c07c0f09ad722b4975534ac Mon Sep 17 00:00:00 2001
From: Andreas Shimokawa <shimokawa@fsfe.org>
Date: Sun, 17 Mar 2019 19:02:18 +0100
Subject: [PATCH] Add setting to enable voip call support (default off)

There were too many bugs reported, so make that optional
---
 .../gadgetbridge/externalevents/NotificationListener.java | 8 +++-----
 app/src/main/res/values/strings.xml                       | 1 +
 app/src/main/res/xml/preferences.xml                      | 5 +++++
 3 files changed, 9 insertions(+), 5 deletions(-)

diff --git a/app/src/main/java/nodomain/freeyourgadget/gadgetbridge/externalevents/NotificationListener.java b/app/src/main/java/nodomain/freeyourgadget/gadgetbridge/externalevents/NotificationListener.java
index da6c2329f0..9481277f50 100644
--- a/app/src/main/java/nodomain/freeyourgadget/gadgetbridge/externalevents/NotificationListener.java
+++ b/app/src/main/java/nodomain/freeyourgadget/gadgetbridge/externalevents/NotificationListener.java
@@ -240,7 +240,9 @@ public class NotificationListener extends NotificationListenerService {
 
     @Override
     public void onNotificationPosted(StatusBarNotification sbn) {
-        if ("call".equals(sbn.getNotification().category)) {
+        Prefs prefs = GBApplication.getPrefs();
+
+        if ("call".equals(sbn.getNotification().category) && prefs.getBoolean("notification_support_voip_calls", false)) {
             handleCallNotification(sbn);
             return;
         }
@@ -249,10 +251,6 @@ public class NotificationListener extends NotificationListenerService {
             return;
         }
 
-
-
-        Prefs prefs = GBApplication.getPrefs();
-
         switch (GBApplication.getGrantedInterruptionFilter()) {
             case NotificationManager.INTERRUPTION_FILTER_ALL:
                 break;
diff --git a/app/src/main/res/values/strings.xml b/app/src/main/res/values/strings.xml
index caf9172cea..d9f73a9b73 100644
--- a/app/src/main/res/values/strings.xml
+++ b/app/src/main/res/values/strings.xml
@@ -83,6 +83,7 @@
     <string name="pref_header_notifications">Notifications</string>
     <string name="pref_title_notifications_repetitions">Repetitions</string>
     <string name="pref_title_notifications_call">Phone Calls</string>
+    <string name="pref_title_support_voip_calls">Enable VoIP app calls</string>
     <string name="pref_title_notifications_sms">SMS</string>
     <string name="pref_title_notifications_timeout">Minimum time between notifications</string>
     <string name="pref_title_notifications_pebblemsg">Pebble Messages</string>
diff --git a/app/src/main/res/xml/preferences.xml b/app/src/main/res/xml/preferences.xml
index 540d5d2e6c..68243a67e7 100644
--- a/app/src/main/res/xml/preferences.xml
+++ b/app/src/main/res/xml/preferences.xml
@@ -202,6 +202,11 @@
             android:key="notification_mode_calls"
             android:title="@string/pref_title_notifications_call"
             android:summary="%s" />
+        <CheckBoxPreference
+            android:layout="@layout/preference_checkbox"
+            android:defaultValue="false"
+            android:key="notification_support_voip_calls"
+            android:title="@string/pref_title_support_voip_calls" />
 
         <ListPreference
             android:defaultValue="when_screen_off"