Skip to content

Commit 5c22c1a

Browse files
committed
Even less statics
1 parent 37bf0fb commit 5c22c1a

File tree

2 files changed

+12
-11
lines changed

2 files changed

+12
-11
lines changed

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

Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -338,7 +338,7 @@ protected void performNotificationScheduling(int id, Notification.Builder notifi
338338
fireTime += repeatInterval;
339339
}
340340

341-
Intent intent = buildNotificationIntent(mContext);
341+
Intent intent = buildNotificationIntent();
342342

343343
if (intent != null) {
344344
UnityNotificationManager.saveNotification(mContext, notificationBuilder.build());
@@ -370,19 +370,19 @@ static void scheduleAlarmWithNotification(Context context, Class activityClass,
370370
UnityNotificationManager.scheduleNotificationIntentAlarm(context, repeatInterval, fireTime, broadcast);
371371
}
372372

373-
static void scheduleAlarmWithNotification(Notification.Builder notificationBuilder, Context context) {
373+
void scheduleAlarmWithNotification(Notification.Builder notificationBuilder) {
374374
long fireTime = notificationBuilder.getExtras().getLong(KEY_FIRE_TIME, 0L);
375-
Intent intent = buildNotificationIntent(context);
375+
Intent intent = buildNotificationIntent();
376376

377377
Class openActivity;
378-
if (mUnityNotificationManager == null || mUnityNotificationManager.mOpenActivity == null) {
379-
openActivity = UnityNotificationUtilities.getOpenAppActivity(context, true);
378+
if (mOpenActivity == null) {
379+
openActivity = UnityNotificationUtilities.getOpenAppActivity(mContext, true);
380380
}
381381
else {
382382
openActivity = mUnityNotificationManager.mOpenActivity;
383383
}
384384

385-
scheduleAlarmWithNotification(context, openActivity, notificationBuilder, intent, fireTime);
385+
scheduleAlarmWithNotification(mContext, openActivity, notificationBuilder, intent, fireTime);
386386
}
387387

388388
private Notification buildNotificationForSending(Class openActivity, Notification.Builder builder) {
@@ -431,7 +431,7 @@ void performNotificationHousekeeping(Set<String> ids) {
431431
}
432432

433433
private Set<String> findInvalidNotificationIds(Set<String> ids) {
434-
Intent intent = buildNotificationIntent(mContext);
434+
Intent intent = buildNotificationIntent();
435435
HashSet<String> invalid = new HashSet<String>();
436436
for (String id : ids) {
437437
// Get the given broadcast PendingIntent by id as request code.
@@ -469,8 +469,8 @@ else synchronized (UnityNotificationManager.class) {
469469
return invalid;
470470
}
471471

472-
protected static Intent buildNotificationIntent(Context context) {
473-
Intent intent = new Intent(context, UnityNotificationManager.class);
472+
private Intent buildNotificationIntent() {
473+
Intent intent = new Intent(mContext, UnityNotificationManager.class);
474474
intent.setFlags(Intent.FLAG_ACTIVITY_NEW_TASK | Intent.FLAG_ACTIVITY_CLEAR_TASK);
475475
return intent;
476476
}

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

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,8 @@ public void onReceive(Context context, Intent received_intent) {
2323
}
2424

2525
private static void rescheduleSavedNotifications(Context context) {
26-
List<Notification.Builder> saved_notifications = UnityNotificationManager.getNotificationManagerImpl(context).loadSavedNotifications();
26+
UnityNotificationManager manager = UnityNotificationManager.getNotificationManagerImpl(context);
27+
List<Notification.Builder> saved_notifications = manager.loadSavedNotifications();
2728

2829
for (Notification.Builder notificationBuilder : saved_notifications) {
2930
Bundle extras = notificationBuilder.getExtras();
@@ -35,7 +36,7 @@ private static void rescheduleSavedNotifications(Context context) {
3536
boolean isRepeatable = repeatInterval > 0;
3637

3738
if (fireTimeDate.after(currentDate) || isRepeatable) {
38-
UnityNotificationManager.scheduleAlarmWithNotification(notificationBuilder, context);
39+
manager.scheduleAlarmWithNotification(notificationBuilder);
3940
}
4041
}
4142
}

0 commit comments

Comments
 (0)