486678 - Calendar summary dialog is borked

Status: RESOLVED FIXED

(Calendar :: General, defect)

Description

[Daniel Boelzle [:dbo]]

- does not show attendees
- no alarms selectable
- custom alarms does not raise reminder dialog

Comment 1

[Daniel Boelzle [:dbo]]

Attachment: fix - v1

Besides the fix, I've changed to allow directly choosing an alarm in case no DISPLAY alarms are supported (in read-write dialog).
for instance, WCAP currently only supports email alarms: choosing one of the direct list (non-custom) leads to adding a DISPLAY alarm which is filtered out, thus the alarm isn't saved. IMO it's OK to choose an email alarm in that list and save that. I'd prefer to show the email icon, too, but couldn't find out why it's not working.

Comment 2

[Philipp Kewisch [:Fallen] [:📆][:🧩]]

Attachment: Fix - v2

I've adapted your patch a bit to show an icon and also consolidated some code. This should fix the described issues.

Comment 3

[Daniel Boelzle [:dbo]]

The patch doesn't work:
- WCAP alarms could not be set
- my default alarm settings are not recognized ("custom" is set), but clicking it there's no alarm...

Comment 5

[Jay Rossiter]

Just wanted to make sure that this also references the missing Description field.  None of the other comments mention it.

Comment 6

[Philipp Kewisch [:Fallen] [:📆][:🧩]]

Attachment: Fix - v3

(In reply to comment #3)
> The patch doesn't work:
> - WCAP alarms could not be set
I could not reproduce, this worked for me.


> - my default alarm settings are not recognized ("custom" is set), but clicking
> it there's no alarm...
The fact that custom is selected is wrong, it should be "no reminders", since the default alarm should work for popup alarms only? We need real UI for default email alarms... or should we misuse the current UI for "an alarm for whatever the first alarm type is" ? For now I went with the misuse to not fully loose the default alarms for wcap.

Comment 7

[Daniel Boelzle [:dbo]]

Comment on attachment 373872 [details] [diff] [review]
Fix - v3

The patch works better now, default alarm settings are set and shown when creating a new event.

The only thing that does not yet work is storing an alarm in WCAP. I suspect that action type is still != EMAIL, thus filtered out in wcap:

    let alarms = item.getAlarms({}).filter(function(x) x.action == "EMAIL");

sorry, still r-

Comment 8

[Philipp Kewisch [:Fallen] [:📆][:🧩]]

Attachment: Fix - v4

Next try. I had the cache enabled, with which for some reason it worked. Disabling the cache exposed your error and this version fixes.

Comment 9

[Daniel Boelzle [:dbo]]

Comment on attachment 373910 [details] [diff] [review]
Fix - v4

patch works fine and looks good :)
r=dbo

Comment 10

[Philipp Kewisch [:Fallen] [:📆][:🧩]]

Pushed to comm-central <http://hg.mozilla.org/comm-central/rev/6799f9b73982>

-> FIXED