Select 'Convert to->Task/event' in the preview pane in mailmode causes an uncaught exception

VERIFIED FIXED in 0.9

Status

Calendar
Lightning Only
VERIFIED FIXED
9 years ago
9 years ago

People

(Reporter: Andreas Treumann, Assigned: Fallen)

Tracking

unspecified
Bug Flags:
blocking-calendar0.9 +

Details

Attachments

(1 attachment, 2 obsolete attachments)

(Reporter)

Description

9 years ago
STEPS TO REPRODUCE:
===================

- go to the mailmode
- don't select anything
- open the context menu and select 'Convert to -> Event/Task'

RESULT:
=======

- error console shows:

Error: uncaught exception: [Exception... "Component returned failure code: 0x80004003 (NS_ERROR_INVALID_POINTER) [nsIMessenger.msgHdrFromURI]"  nsresult: "0x80004003 (NS_ERROR_INVALID_POINTER)"  location: "JS frame :: chrome://messenger/content/messenger.xul :: oncommand :: line 1"  data: no] 

EXPECTED RESULT:
================

- no error console output
REPRODUCIBLE:
=============

- always
(Reporter)

Comment 1

9 years ago
for a better reproducibility:

step 3 of the STR should be:

- open the context menu in the preview pane and select 'Convert to -> Event/Task'

Summary: Select 'Convert to->Task/event' in the preview pain in mailmode causes an uncaught exception → Select 'Convert to->Task/event' in the preview pane in mailmode causes an uncaught exception
(Reporter)

Comment 2

9 years ago
for a better reproducibility:

step 3 of the STR should be:

- open the context menu in the preview pane and select 'Convert to -> Event/Task'

Updated

9 years ago
Flags: blocking-calendar0.9+
(Assignee)

Comment 3

9 years ago
Created attachment 333061 [details] [diff] [review]
Fix - v1
Assignee: Berend.Cornelius → philipp
Status: NEW → ASSIGNED
Attachment #333061 - Flags: review?(daniel.boelzle)
(Assignee)

Comment 4

9 years ago
Created attachment 333062 [details] [diff] [review]
Fix - v1.1
Attachment #333061 - Attachment is obsolete: true
Attachment #333062 - Flags: review?(daniel.boelzle)
Attachment #333061 - Flags: review?(daniel.boelzle)

Updated

9 years ago
Whiteboard: [patch in hand] [needs-review]
Comment on attachment 333062 [details] [diff] [review]
Fix - v1.1

This disables the "Convert To" menu item. I'd rather think we should remove the menu item in those cases like any other mail-content menu items disappear, too.
Attachment #333062 - Flags: review?(daniel.boelzle) → review-

Updated

9 years ago
Whiteboard: [patch in hand] [needs-review]
(Assignee)

Comment 6

9 years ago
Created attachment 333304 [details] [diff] [review]
Fix - v2

While I agree with the message pane, I don't with the thread pane. In the thread pane, the items are disabled, in the message pane they are gone.
Attachment #333062 - Attachment is obsolete: true
Attachment #333304 - Flags: review?(daniel.boelzle)
Comment on attachment 333304 [details] [diff] [review]
Fix - v2

works fine, looks good; r=dbo
Attachment #333304 - Flags: review?(daniel.boelzle) → review+

Updated

9 years ago
Whiteboard: [patch in hand]

Updated

9 years ago
Keywords: checkin-needed
Checked in on HEAD and MOZILLA_1_8_BRANCH => FIXED.
Status: ASSIGNED → RESOLVED
Last Resolved: 9 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Whiteboard: [patch in hand]
Target Milestone: --- → 0.9
(Reporter)

Comment 9

9 years ago
Checked in lightning 2008081803 -> VERIFIED
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.