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

Bug 349031

Summary: [EclipseLink] descriptor customizer validation incorrect when default zero arg constructor
Product: [WebTools] Dali JPA Tools Reporter: Karen Butzke <karenfbutzke>
Component: GeneralAssignee: Nan Li <nan.n.li>
Status: VERIFIED FIXED QA Contact:
Severity: normal    
Priority: P3 CC: douglas.clarke, jolene.moffitt, neil.hauge
Version: 3.0Flags: neil.hauge: review+
karenfbutzke: review+
Target Milestone: 3.0.1   
Hardware: PC   
OS: Windows XP   
Whiteboard:
Attachments:
Description Flags
Proposed Patch
none
Proposed Patch karenfbutzke: iplog+, karenfbutzke: review+

Description Karen Butzke CLA 2011-06-10 09:45:50 EDT
If you set the MyCustomizer class as a descriptor customizer on an eclipseLink entity you get a validation error: The entity customizer class "model.MyCustomizer" must have a public, zero-argument constructor. This class has a default zero-arg constructor so there should be no error.


public class MyCustomizer implements DescriptorCustomizer {

	public void customize(ClassDescriptor descriptor) throws Exception {
		
	}
}
Comment 1 Karen Butzke CLA 2011-06-10 10:28:58 EDT
This problem would apply to all the validation errors added in bug https://bugs.eclipse.org/bugs/show_bug.cgi?id=330619
Comment 2 Nan Li CLA 2011-06-10 14:37:27 EDT
Created attachment 197807 [details]
Proposed Patch

Handled default constructor case
Comment 3 Nan Li CLA 2011-06-24 11:27:16 EDT
Created attachment 198545 [details]
Proposed Patch
Comment 4 Karen Butzke CLA 2011-06-24 11:43:36 EDT
I have reviewed and applied Nan's latest patch to HEAD for the 3.0.1 release
Comment 5 Jolene Moffitt CLA 2011-08-24 14:43:54 EDT
Verified in Build I-3.4.0-20110822071621 

Verified if you set the MyCustomizer class as a descriptor customizer on an eclipseLink entity you do not get a validation error.  See the link to view test steps for verification.  http://wiki.eclipse.org/Dali_3.0.1
Comment 6 Neil Hauge CLA 2011-10-18 09:58:07 EDT
*** Bug 361196 has been marked as a duplicate of this bug. ***