1132336 - [Calendar] notifications aren't closed during click if it was triggered while the calendar app was closed

Status: VERIFIED FIXED

(Firefox OS Graveyard :: Gaia::Calendar, defect)

Description

[Miller Medeiros [:millermedeiros]]

steps to reproduce:

 1. create a new event with a reminder;
 2. close the calendar app before the notification is fired;
 3. tap the notification (can be on the lockscreen, tray or toast);

expected:

it should open the calendar app on the correct event and delete the notification.

actual:

notification remains on the tray.

Comment 1

[GitHub Autolander Bot]

Attachment: [gaia] millermedeiros:1132336-notification-click-close > mozilla-b2g:master

Comment 2

[Miller Medeiros [:millermedeiros]]

Comment on attachment 8563716 [details] [review]
[gaia] millermedeiros:1132336-notification-click-close > mozilla-b2g:master

added a "tag" to all the notifications ("tag" works like an ID) so if the app creates multiple notifications for same URL it will consider as a single notification (will override the previous one). It also uses the "tag" to find and close all the notifications during `launch`, that way we use the same logic if app is open or closed avoiding this kind of error.

also added some safeguards against the unlikely case of a busytime or event being null for some reason when the alarm is triggered (I think I saw it happen but was not able to reproduce).

Comment 3

[Gareth Aye [:gaye] (back from PTO)]

Comment on attachment 8563716 [details] [review]
[gaia] millermedeiros:1132336-notification-click-close > mozilla-b2g:master

LGTM!

Comment 4

[Stephany Wilkes]

Triage is blocking because this is a regression.

Comment 5

[GitHub Autolander Bot]

Pull request has landed in master: https://github.com/mozilla-b2g/gaia/commit/8df64a303797de9b678967d929d7a9215486eb8d

Comment 6

[Miller Medeiros [:millermedeiros]]

Comment on attachment 8563716 [details] [review]
[gaia] millermedeiros:1132336-notification-click-close > mozilla-b2g:master

[Approval Request Comment]
[Bug caused by] (feature/regressing bug #): 1042939
[User impact] if declined: user will need to delete the notification manually and might cause confusion (it's a regression and bad UX)
[Testing completed]: manual
[Risk to taking this patch] (and alternatives if risky): very low risk, changes are local and should not cause conflicts
[String changes made]: none

Comment 7

[(Out Sep 17-23) Ryan VanderMeulen]

v2.2: https://github.com/mozilla-b2g/gaia/commit/817f47167ffde2c6afb54fdb3fafbbca21296b07

Comment 8

[Jayme Mercado [:JMercado] (Inactive 3/4/2016)]

This issue is verified fixed on the latest Nightly 3.0 and 2.2 builds.
	
Actual results:  The notifications are dismissed when pressed on for calendar events even if the calendar is closed.

Environmental Variables:
Device: Flame 3.0
BuildID: 20150226010233
Gaia: 7894b929f1b0394f3c997f72a6482bc7813e758d
Gecko: dd6353d61993
Gonk: e7c90613521145db090dd24147afd5ceb5703190
Version: 39.0a1 (3.0) 
Firmware Version: v18D-1
User Agent: Mozilla/5.0 (Mobile; rv:39.0) Gecko/39.0 Firefox/39.0

Environmental Variables:
Device: Flame 2.2
BuildID: 20150226002503
Gaia: bf24aa57fa7760260ab05d1f53242c8d8ae59e83
Gecko: 363123044e61
Gonk: e7c90613521145db090dd24147afd5ceb5703190
Version: 37.0 (2.2) 
Firmware Version: v18D-1
User Agent: Mozilla/5.0 (Mobile; rv:37.0) Gecko/37.0 Firefox/37.0