Include scheduled toast in the fix

andrewleader · andrewleader · commit f8135ceac420 · 2021-03-12T13:34:54.000-08:00
diff --git a/Microsoft.Toolkit.Uwp.Notifications/Toasts/Compat/ToastNotifierCompat.cs b/Microsoft.Toolkit.Uwp.Notifications/Toasts/Compat/ToastNotifierCompat.cs
@@ -157,10 +157,10 @@ private void PreprocessScheduledToast(ScheduledToastNotification notification)
             // For apps that don't have identity...
             if (!DesktopBridgeHelpers.HasIdentity())
             {
-                // If tag is specified
-                if (!string.IsNullOrEmpty(notification.Tag))
+                // If tag is specified and group isn't specified
+                if (!string.IsNullOrEmpty(notification.Tag) && string.IsNullOrEmpty(notification.Group))
                 {
-                    // If group isn't specified, we have to add a group since otherwise can't remove without a group
+                    // We have to add a group since otherwise can't remove without a group
                     notification.Group = ToastNotificationManagerCompat.DEFAULT_GROUP;
                 }
             }

Original file line number	Diff line number	Diff line change
`@@ -157,10 +157,10 @@ private void PreprocessScheduledToast(ScheduledToastNotification notification)`
`157`	`157`	`// For apps that don't have identity...`
`158`	`158`	`if (!DesktopBridgeHelpers.HasIdentity())`
`159`	`159`	`{`
`160`		`- // If tag is specified`
`161`		`- if (!string.IsNullOrEmpty(notification.Tag))`
	`160`	`+ // If tag is specified and group isn't specified`
	`161`	`+ if (!string.IsNullOrEmpty(notification.Tag) && string.IsNullOrEmpty(notification.Group))`
`162`	`162`	`{`
`163`		`- // If group isn't specified, we have to add a group since otherwise can't remove without a group`
	`163`	`+ // We have to add a group since otherwise can't remove without a group`
`164`	`164`	`notification.Group = ToastNotificationManagerCompat.DEFAULT_GROUP;`
`165`	`165`	`}`
`166`	`166`	`}`