Bug 231539 - "Null" table and join column messages
Summary: "Null" table and join column messages
Status: RESOLVED FIXED
Alias: None
Product: Dali JPA Tools
Classification: WebTools
Component: General (show other bugs)
Version: 2.0   Edit
Hardware: PC Windows XP
: P3 normal (vote)
Target Milestone: 2.1 M2   Edit
Assignee: Tran Le CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2008-05-12 09:03 EDT by Shaun Smith CLA
Modified: 2008-12-08 12:37 EST (History)
2 users (show)

See Also:


Attachments
Null table and column messages (85.59 KB, image/jpeg)
2008-05-12 09:03 EDT, Shaun Smith CLA
no flags Details
From my notes--this results from an unmapped List attribute (83.08 KB, image/jpeg)
2008-05-12 09:05 EDT, Shaun Smith CLA
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Shaun Smith CLA 2008-05-12 09:03:21 EDT
Created attachment 99696 [details]
Null table and column messages

When a OneToMany is defined as a simple List with no content type, e.g.

public class Publisher { 
  @OneToMany
  private List titles;
...

Dali produces error messages like (see screenshot):

  Column "null" cannot be resolved
  Join table "null" cannot be resolved

These messages do not help the developer resolve the problem and guide them to suppling the missing target entity information.  It would be better, when the join table name is to be derived and validated against the schema that, if the join table cannot be determined due to a missing target entity, Dali provide a message along the lines of:

  Target Entity for Publisher.titles not defined - join table cannot be determined

"null" anywhere in a message indicates that we are not providing useful diagnostics to the developer.
Comment 1 Shaun Smith CLA 2008-05-12 09:05:38 EDT
Created attachment 99697 [details]
From my notes--this results from an unmapped List attribute
Comment 2 Neil Hauge CLA 2008-05-13 23:26:44 EDT
This improvement will fit nicely with our focus on improved validation our next release.
Comment 3 Tran Le CLA 2008-09-30 15:05:40 EDT
Fixed in 2.1 M2
Comment 4 Karen Butzke CLA 2008-12-08 12:37:16 EST
The fix for this has been revised in bug 257629.  You can see an explanation of the new validation messages in that bug.  Shaun, if you have a chance, could you test this out?  We will release it later today if you want to use a build.  Otherwise I will check it into cvs soon.