Closed
Bug 1362422
Opened 8 years ago
Closed 8 years ago
Lack of partstat yields misleading appearance and empty selection box
Categories
(Calendar :: Lightning Only, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
5.4.3
People
(Reporter: jens.poenninghaus, Assigned: MakeMyDay)
Details
Attachments
(2 files)
18.89 KB,
image/png
|
Details | |
2.48 KB,
patch
|
Fallen
:
review+
Fallen
:
approval-calendar-beta+
Fallen
:
approval-calendar-esr+
|
Details | Diff | Splinter Review |
User Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:31.0) Gecko/20100101 Firefox/31.0
Build ID: 20140825060242
Steps to reproduce:
- Invite lightning user to an event (import event with lightning user being an attendee)
- Ensure partstat=needs-action is missing,
e.g. copy event to editor remove parameter property insert modified event
- check derived event for x-lic errors (there should be none)
BEGIN:VEVENT
CREATED:20170505T143419Z
LAST-MODIFIED:20170505T143717Z
DTSTAMP:20170505T143717Z
UID:a8693d56-90db-401b-a6f4-965f20e9305a
SUMMARY:partstat-no-needs-action
ORGANIZER;CN=The organizer:mailto:org@some-testdomain.example.com
ATTENDEE;RSVP=TRUE;CN=the attendee:mailto:att@some-testdomain.example.com
DTSTART;TZID=Europe/Warsaw:20170505T100000
DTEND;TZID=Europe/Warsaw:20170505T110000
CLASS:PUBLIC
DESCRIPTION:partstat-no-needs-action
X-APPLE-NEEDS-REPLY:TRUE
END:VEVENT
Actual results:
- as lightning user
* observe optical appearance of modified event resembling an already accepted event
* check invitation list (modified event not present)
* open event details: Observe reply drop down list populated with '' / an empty label.
Expected results:
Missing partstat should have been treated as partstat=needs-action
-> visual appearance of open invitation
-> list of open invitations
-> reply: 'i will confirm later'
Reporter | ||
Comment 1•8 years ago
|
||
Snapshot taken on lightning 5.4.1. Mark that both events still need confirmation. Mark that event on right hand side has empty label in 'reply' selection. Mark the difference in appearance of those two events.
Assignee | ||
Comment 2•8 years ago
|
||
This patch takes care to display the 'I will confirm later' label in the summary dialog if the param is completely missing. The partstat itself is not added until the user will actively choose another partstat.
Assignee: nobody → makemyday
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
Attachment #8869852 -
Flags: review?(philipp)
Updated•8 years ago
|
Attachment #8869852 -
Flags: review?(philipp)
Attachment #8869852 -
Flags: review+
Attachment #8869852 -
Flags: approval-calendar-esr+
Attachment #8869852 -
Flags: approval-calendar-beta+
Updated•8 years ago
|
Keywords: checkin-needed
Pushed by mozilla@jorgk.com:
https://hg.mozilla.org/comm-central/rev/ed2940d5db2f
Display default partstat value in event summary dialog if the parameter is missing in attendee property. r=philipp
Updated•8 years ago
|
Target Milestone: --- → 5.9
Comment 4•8 years ago
|
||
Beta (TB 56, Calendar 5.8):
https://hg.mozilla.org/releases/comm-beta/rev/f25750d3bb82cc305dffa01dc888a52667c97eba
Target Milestone: 5.9 → 5.8
Comment 5•8 years ago
|
||
ESR 52.3, Calendar 5.4.3:
https://hg.mozilla.org/releases/comm-esr52/rev/4265c16f92d3
Target Milestone: 5.8 → 5.4.3
You need to log in
before you can comment on or make changes to this bug.
Description
•