Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 358562

Summary:	Meeting Request with no e-mail address
Product:	z_Archived	Reporter:	Jacques Bouthillier <lmcbout>
Component:	Mylyn	Assignee:	Sebastien Dubois <sebastien.dubois>
Status:	RESOLVED FIXED	QA Contact:
Severity:	normal
Priority:	P3	CC:	alvaro.sanchez-leon
Version:	unspecified
Target Milestone:	0.9
Hardware:	PC
OS:	Linux
Whiteboard:

Description Jacques Bouthillier

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

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.