#155 - set constant for notification id to avoid multiple notifications

thuryn · Nov 21, 2020 · a2bd7b5 · a2bd7b5
1 parent 74b1eb0
commit a2bd7b5
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/app/src/main/java/org/thosp/yourlocalweather/service/SensorLocationUpdateService.java b/app/src/main/java/org/thosp/yourlocalweather/service/SensorLocationUpdateService.java
@@ -142,7 +142,7 @@ private void registerSensorListener() {
             return;
         }
 
-        startForeground(android.os.Process.myPid(), notification);
+        startForeground(NotificationUtils.NOTIFICATION_ID, notification);
 
         sensorResolutionMultiplayer = 1 / senAccelerometer.getResolution();
         int maxDelay = 10000;

diff --git a/app/src/main/java/org/thosp/yourlocalweather/utils/NotificationUtils.java b/app/src/main/java/org/thosp/yourlocalweather/utils/NotificationUtils.java
@@ -29,6 +29,8 @@ public class NotificationUtils {
 
     private static final String TAG = "NotificationUtils";
 
+    public static final int NOTIFICATION_ID = 2109876543;
+
     public static void weatherNotification(Context context, Long locationId) {
         /*String updateAutoPeriodStr = AppPreference.getLocationAutoUpdatePeriod(context);
         boolean updateBySensor = "0".equals(updateAutoPeriodStr);
@@ -136,7 +138,7 @@ public static Notification getNotification(Context context, Location location, C
     public static void showNotification(Context context, Notification notification) {
         NotificationManager notificationManager =
                 (NotificationManager) context.getSystemService(Context.NOTIFICATION_SERVICE);
-        notificationManager.notify(android.os.Process.myPid(), notification);
+        notificationManager.notify(NOTIFICATION_ID, notification);
     }
 
     private static Notification regularNotification(Context context,