Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 331960 - plural converter annotations added in EclipseLink
Summary: plural converter annotations added in EclipseLink
Status: VERIFIED FIXED
Alias: None
Product: Dali JPA Tools
Classification: WebTools
Component: JPA (show other bugs)
Version: 3.0   Edit
Hardware: PC Windows XP
: P3 enhancement (vote)
Target Milestone: 3.2 M5   Edit
Assignee: Karen Butzke CLA
QA Contact:
URL:
Whiteboard: EclipseLink
Keywords: plan
Depends on:
Blocks:
 
Reported: 2010-12-06 17:55 EST by Karen Butzke CLA
Modified: 2012-01-17 07:10 EST (History)
3 users (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 Karen Butzke CLA 2010-12-06 17:55:19 EST 
In bug 282773  EclipseLink added @Converters, @TypeConverters, @ObjectTypeConverters, @StructConverters annotations. We need to add the corresponding support in Dali for the EclipseLink 2.2 platform
Comment 1 Stuart Schmukler CLA 2011-05-18 00:32:46 EDT 
I am not sure if my problem falls into this bug.

Generating an Entity from the table "DIM_DIAGNOSIS" creates a class "DimDiagnosi"
which is confusing.

...
@Entity
@Table(name="DIM_DIAGNOSIS")
public class DimDiagnosi 
...


Should I file this as a separate bug?
Comment 2 Neil Hauge CLA 2011-05-18 01:27:41 EDT 
(In reply to comment #1)
> I am not sure if my problem falls into this bug.
> 
> Generating an Entity from the table "DIM_DIAGNOSIS" creates a class
> "DimDiagnosi"
> which is confusing.
> 
> ...
> @Entity
> @Table(name="DIM_DIAGNOSIS")
> public class DimDiagnosi 
> ...
> 
> 
> Should I file this as a separate bug?

This would be a separate issue.  That said, this behavior is working as designed.  The idea is to create a proper java class name from a database table name.  This is why the underscore is removed and standard camel casing is applied to the 'D' in Diagnosis.  An underscore is not prohibited in a class name but it does not follow common Java practice/convention.
Comment 3 Neil Hauge CLA 2011-05-18 10:17:25 EDT 
(In reply to comment #2)
I see now that you were referring to the removal of the 's' at the end.  Apparently I shouldn't be commenting on bugs at 1:30 in the morning.
Comment 4 Neil Hauge CLA 2011-07-01 16:24:06 EDT 
Moving JPA specific bugs to new JPA component in bugzilla.
Comment 5 Karen Butzke CLA 2011-12-05 12:52:56 EST 
fixed in M5
Comment 6 Jolene Moffitt CLA 2012-01-17 07:10:10 EST 
Verified in Build I-3.4.0-2012010922312

Verified plural converter annotations were added in EL 2.2, EL 2.3 and 2.4
See the link to view test steps for verification.
http://wiki.eclipse.org/Dali_3.2_M5