Skip to content

Commit fccf403

Browse files
committed
Refactor housekeeping scheduling in manager
1 parent 1f4cafd commit fccf403

File tree

1 file changed

+5
-13
lines changed

1 file changed

+5
-13
lines changed

com.unity.mobile.notifications/Runtime/Android/Plugins/com/unity/androidnotifications/UnityNotificationManager.java

Lines changed: 5 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -378,21 +378,13 @@ private static synchronized void scheduleHousekeeping(Context context, Set<Strin
378378
++mSentSinceLastHousekeeping;
379379
if (mSentSinceLastHousekeeping > 50) {
380380
mSentSinceLastHousekeeping = 0;
381-
triggerHousekeeping(context, ids);
381+
triggerHousekeeping();
382382
}
383383
}
384384

385-
private static synchronized void triggerHousekeeping(Context context, Set<String> ids) {
386-
if (ids == null) {
387-
ids = getScheduledNotificationIDs(context);
388-
}
389-
390-
// needed for lamda
391-
final Set<String> notificationIds = ids;
385+
private static synchronized void triggerHousekeeping() {
392386
if (mUnityNotificationManager != null) {
393-
mUnityNotificationManager.mBackgroundThread.enqueueTask(() -> {
394-
performNotificationHousekeeping(context, notificationIds);
395-
});
387+
mUnityNotificationManager.mBackgroundThread.enqueueHousekeeping();
396388
}
397389
}
398390

@@ -592,7 +584,7 @@ public void cancelAllPendingNotificationIntents() {
592584
cancelPendingNotificationIntent(context, Integer.valueOf(id));
593585
deleteExpiredNotificationIntent(context, id);
594586
}
595-
triggerHousekeeping(context, null);
587+
triggerHousekeeping();
596588
});
597589
}
598590
}
@@ -613,7 +605,7 @@ private static synchronized void saveScheduledNotificationIDs(Context context, S
613605
public void cancelPendingNotification(int id) {
614606
synchronized (UnityNotificationManager.class) {
615607
UnityNotificationManager.cancelPendingNotificationIntent(mContext, id);
616-
triggerHousekeeping(mContext, null);
608+
triggerHousekeeping();
617609
}
618610
}
619611

0 commit comments

Comments
 (0)