[firebase_messaging] undesired auto cleaning notifications when entering foreground (iOS)

[JasperShih]

Describe the bug
Hi, I found a undesired auto cleaning notifications scenario when application entering foreground. It is at line 159 in
https://github.com/FirebaseExtended/flutterfire/blob/master/packages/firebase_messaging/ios/Classes/FirebaseMessagingPlugin.m

- (void)applicationDidBecomeActive:(UIApplication *)application {
  _resumingFromBackground = NO;
  // Clears push notifications from the notification center, with the
  // side effect of resetting the badge count. We need to clear notifications
  // because otherwise the user could tap notifications in the notification
  // center while the app is in the foreground, and we wouldn't be able to
  // distinguish that case from the case where a message came in and the
  // user dismissed the notification center without tapping anything.
  // TODO(goderbauer): Revisit this behavior once we provide an API for managing
  // the badge number, or if we add support for running Dart in the background.
  // Setting badgeNumber to 0 is a no-op (= notifications will not be cleared)
  // if it is already 0,
  // therefore the next line is setting it to 1 first before clearing it again
  // to remove all
  // notifications.
  application.applicationIconBadgeNumber = 1;
  application.applicationIconBadgeNumber = 0;
}

To Reproduce

1. get >=2 remote notifications on iPhone
2. enter foreground
3. all notifications & badges be unexpectedly removed

Expected behavior
Just remove the notification we clicked. Thanks!

[izabekz]

same problem, I don't want notifications to disappear after launching app, even when launching app without clicking the notification makes it disappear and I lose info from notification

[kroikie]

@JasperShih and @izabekz thanks for pointing this out. As you have noted this is currently WAI for the plugin, I'm going to re-label this as a feature request and we will update the behaviour.

[izabekz]

@kroikie thanks, what does WAI mean? And another question, when the update will be available? This is very crucial for us.

[jigarfumakiya]

when this update will be available? this is important for us.

[kroikie]

@jigarfumakiya I agree that the current implementation of automatically clearing the notifications is something that does not work for all cases. I'm investigating the best way to provide the option to clear notifications by the developer.

[jigarfumakiya]

thank you so much @kroikie but still, there is any date when this will be solved.

[jlhour]

Thanks for the great work, but sorry to be insistent: why is this the expected/default behaviour?