openjpa-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Albert Lee (JIRA)" <>
Subject [jira] Commented: (OPENJPA-430) Automatically remove hungarian notation from column names.
Date Tue, 18 Mar 2008 18:01:25 GMT


Albert Lee commented on OPENJPA-430:

This feature intended to remove/truncate fields using the Hungarian notations to the form
as described in the original append. However the following changes seems to do more than just
transforming a field, i.e. " name = dict.getValidColumnName(name, local);"

--- openjpa/trunk/openjpa-persistence-jdbc/src/main/java/org/apache/openjpa/persistence/jdbc/
2007/11/06 07:33:44	592318
+++ openjpa/trunk/openjpa-persistence-jdbc/src/main/java/org/apache/openjpa/persistence/jdbc/
2007/11/06 07:36:55	592319
@@ -181,6 +181,12 @@
         if (target instanceof Column) {
             if (elem)
                 name = vm.getFieldMapping().getName();
+            if (isRemoveHungarianNotation())
+                name = removeHungarianNotation(name);
+            name = dict.getValidColumnName(name, local);
             col.setName(name + "_" + ((Column) target).getName());

This change has some undesired consequence that broke some previous application.  Specifically,
if the field is a keyword (e.g. identity), a digit is appended to the name so a foreign key
column is changed from, e.g.   identity_id to identity0_id.  If an application already has
a table defined and no automatic synchronize mapping is specified, the application will fail.
This name augmentation is not warranted because even though identity is a keyword but the
combined foreign key column identity_id is NOT a keyword.

My question is what is the intended purpose of this name transformation and how does it play
into the "remove Hungarian Notation" feature? 

As Patrick mentioned in his  last note, some function in Kodo test also failed.  Can this
name transformation be removed ?

Albert Lee.

> Automatically remove hungarian notation from column names.
> ----------------------------------------------------------
>                 Key: OPENJPA-430
>                 URL:
>             Project: OpenJPA
>          Issue Type: New Feature
>    Affects Versions: 1.1.0
>            Reporter: Ben Short
>            Priority: Minor
>             Fix For: 1.1.0
>         Attachments: OPENJPA-430-DOCS.patch, OPENJPA-430.patch
> Add a flag to switch on/off automatic truncation of hungarian fields when automatically
generating column names.
> Fields such as mFoobar and strBarFoo would become columns named* foobar and barfoo respectivly.
> *Current naming DBDictionary rules would still apply.

This message is automatically generated by JIRA.
You can reply to this email to add a comment to the issue online.

View raw message