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

Bug 338712

Summary: Default Table Names prefix should be null
Product: [WebTools] Dali JPA Tools Reporter: Shaun Smith <shaun.smith>
Component: Diagram EditorAssignee: Petya Sabeva <petya.sabeva>
Status: CLOSED FIXED QA Contact: Stefan Dimov <stefan.dimov>
Severity: normal    
Priority: P3 CC: neil.hauge
Version: 3.0Keywords: plan
Target Milestone: 3.3 M2Flags: stefan.dimov: review+
Hardware: PC   
OS: Windows XP   
Whiteboard:
Attachments:
Description Flags
patch none

Description Shaun Smith CLA 2011-03-02 13:04:01 EST 
Newly created classes are mapped to tables beginning with the default prefix "T_".  The default table name should use the JPA default which is the entity name therefore the prefix default should be "". 

The project default prefix can be specified but a workspace wide default should also be available.
Comment 1 Petya Sabeva CLA 2011-05-10 10:28:27 EDT 
Created attachment 195223 [details]
patch
Comment 2 Stefan Dimov CLA 2011-05-10 13:19:13 EDT 
The bug is trivial.

It has an easy workaround, because the default table prefix could be easily changed, but this bug stays open for a long time and perhaps it's annoying for the users.

The fix is tested manually. All the existing JUnit tests are passing successfully.

The default table name prefix in the editor is defined in a single hardcoded constant. That is the actual change - the constant is changed from "T_" to "".
I've reviewed the fix and it's ok.

The risk is low.
Comment 3 Neil Hauge CLA 2011-05-10 13:57:54 EDT 
While I agree with the fix, I don't think this quite meets the qualifications of an RC1 fix candidate.  Leaving targeted to 3.1.
Comment 4 Neil Hauge CLA 2011-10-27 11:16:43 EDT 
The 3.2 target now represents the Juno release.
Comment 5 Petya Sabeva CLA 2012-05-04 02:01:58 EDT 
Retargeting this bug to the Kepler realease (3.3).
Comment 6 Petya Sabeva CLA 2012-09-18 04:04:45 EDT 
Changes are committed in the master branch of git with the following git commit:

http://git.eclipse.org/c/dali/webtools.dali.git/commit/?id=474818faab56cf761430a89c7fb7e47be39bf93c

Change still needs to be released.
Comment 7 Petya Sabeva CLA 2012-10-03 02:39:45 EDT 
Changes are released...