Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 358562 - Meeting Request with no e-mail address
Summary: Meeting Request with no e-mail address
Status: RESOLVED FIXED
Alias: None
Product: z_Archived
Classification: Eclipse Foundation
Component: Mylyn (show other bugs)
Version: unspecified   Edit
Hardware: PC Linux
: P3 normal (vote)
Target Milestone: 0.9   Edit
Assignee: Sebastien Dubois CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-09-22 07:48 EDT by Jacques Bouthillier CLA
Modified: 2012-01-24 15:35 EST (History)
1 user (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Jacques Bouthillier CLA 2011-09-22 07:48:35 EDT 
Meeting request: Do not allow to enter the e-mail of the participant not having an e-mail address, it displays (in the logs) the following error:

!ENTRY org.eclipse.mylyn.reviews.r4e.ui 2 0 2011-09-21 11:54:18.895
!MESSAGE Exception: org.eclipse.core.runtime.CoreException: ComponentObjectModelException: Null address
!STACK 1
org.eclipse.core.runtime.CoreException: ComponentObjectModelException: Null address
	at org.eclipse.mylyn.reviews.r4e.mail.smtp.mailVersion.Smtp.sendEmail(Smtp.java:119)
	at org.eclipse.mylyn.reviews.r4e.mail.smtp.mailVersion.Smtp.createMeetingRequest(Smtp.java:192)
	at org.eclipse.mylyn.reviews.r4e.ui.internal.utils.MailServicesProxy.sendMeetingRequest(MailServicesProxy.java:1017)
	at org.eclipse.mylyn.reviews.r4e.ui.internal.utils.MailServicesProxy.sendMeetingRequest(MailServicesProxy.java:987)
	at org.eclipse.mylyn.reviews.r4e.ui.internal.commands.SendNotificationHandler$1.runInUIThread(SendNotificationHandler.java:128
Comment 1 Sebastien Dubois CLA 2011-09-23 18:23:16 EDT 

Added a check to notify user (through an Error Dialog) when no valid destinations are defined to receive the meeting requests.  
Also, added a check to prevent empty email addresses from being used

This should prevent these errors from happening

Eventually, we will need a better way to validate email addresses.  This should be an enhancement.