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

Bug 344731

Summary: DS File wizard has confusing error
Product: [Eclipse Project] PDE Reporter: Curtis Windatt <curtis.windatt.public>
Component: UIAssignee: Curtis Windatt <curtis.windatt.public>
Status: VERIFIED FIXED QA Contact:
Severity: normal    
Priority: P3 CC: ankur_sharma
Version: 3.7Flags: ankur_sharma: review+
Target Milestone: 3.7 RC1   
Hardware: PC   
OS: Linux   
Whiteboard:
Attachments:
Description Flags
Fix
none
updated patch none

Description Curtis Windatt CLA 2011-05-04 13:02:23 EDT 
1) Open new component wizard
2) Make sure a folder is selected
3) Make sure the file has a name
4) Have no value for the class
Result:
The wizard says the component must have a name.
Expected result:
The wizard says that a class must be specified
Comment 1 Curtis Windatt CLA 2011-05-04 13:35:12 EDT 
Created attachment 194744 [details]
Fix
Comment 2 Curtis Windatt CLA 2011-05-04 13:36:55 EDT 
Fixed in HEAD.  When you open the wizard we fill in example values.  If the user deletes the text, a more accurate error is returned.

Ankur, please review for RC1.
Comment 3 Ankur Sharma CLA 2011-05-05 04:22:55 EDT 
Created attachment 194801 [details]
updated patch
Comment 4 Ankur Sharma CLA 2011-05-05 04:25:59 EDT 
The patch is good. 
However, I noticed that when the filename has not been provided (and other entries are) then the message reads "Name can not be empty". This gets ambiguous as it is not referring to the Component 'Name' but the 'File Name'. I am attaching an updated patch which does this additional validation. See if it looks better.
Comment 5 Curtis Windatt CLA 2011-05-05 12:06:08 EDT 
The updated patch is fine by me.  Please review and commit it.
Comment 6 Ankur Sharma CLA 2011-05-08 05:49:47 EDT 
+1

Applied to HEAD
Comment 7 Curtis Windatt CLA 2011-05-16 17:33:11 EDT 
Verified in I20110514-0800