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

Bug 363535

Summary: DBWSBuilder should set javaClassName for Object/Varray/ObjectTable types to NOT be in Java's default package
Product: z_Archived Reporter: Mike Norman <michael.norman>
Component: EclipselinkAssignee: Nobody - feel free to take it <nobody>
Status: RESOLVED FIXED QA Contact:
Severity: normal    
Priority: P3 CC: david.twelves, eclipselink.dbws-inbox
Version: unspecified   
Target Milestone: ---   
Hardware: All   
OS: All   
Whiteboard:
Attachments:
Description Flags
Proposed fix.
none
Supporting test cases.
none
Proposed fix v2.
none
Supporting test cases.
none
Proposed fix v3. none

Description Mike Norman CLA 2011-11-10 15:39:58 EST

    
Comment 1 David McCann CLA 2012-01-11 13:12:45 EST
Created attachment 209334 [details]
Proposed fix.

We now prepend the package (project name + '.') to the Java class name for Object, ObjectTable and VArray types.
Comment 2 David McCann CLA 2012-01-11 13:14:26 EST
Created attachment 209335 [details]
Supporting test cases.

The existing Object, ObjectTable and VArray test suites were modified to also check the Java class names of the processed types to verify that the package was prepended properly.
Comment 3 David McCann CLA 2012-01-12 13:15:02 EST
Created attachment 209397 [details]
Proposed fix v2.

Now uses an existing method on Util class that returns the correct java class name.
Comment 4 David McCann CLA 2012-01-12 13:15:20 EST
Created attachment 209398 [details]
Supporting test cases.
Comment 5 David McCann CLA 2012-01-12 14:27:35 EST
Created attachment 209402 [details]
Proposed fix v3.

Added default project name (myProject) to handle null/empty string project names.
Comment 6 David McCann CLA 2012-01-12 15:50:50 EST
Reviewed by:  michael.norman@oracle.com
Tests:  All unit tests pass as expected; ObjectTypeTestSuite, ObjectTableTypeTestSuite and VArrayTestSuite modded

Revision: 10679
Comment 7 Eclipse Webmaster CLA 2022-06-09 10:29:34 EDT
The Eclipselink project has moved to Github: https://github.com/eclipse-ee4j/eclipselink