Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 339560 - Invalid package does not trigger validation of mapped classes
Summary: Invalid package does not trigger validation of mapped classes
Status: RESOLVED FIXED
Alias: None
Product: Dali JPA Tools
Classification: WebTools
Component: General (show other bugs)
Version: unspecified   Edit
Hardware: PC Windows 7
: P3 normal (vote)
Target Milestone: 3.0.1   Edit
Assignee: Karen Butzke CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-03-10 12:38 EST by Nan Li CLA
Modified: 2012-02-01 09:38 EST (History)
2 users (show)

See Also:
neil.hauge: review+


Attachments
proposed patch against head (1.34 KB, patch)
2011-06-07 17:51 EDT, Karen Butzke CLA
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Nan Li CLA 2011-03-10 12:38:10 EST
Build Identifier: I-3.3.0-20110305235707

Giving the right setting first in the mapping file like below

<package>test.oracle.models</package>
<entity class="employee.Employee"></entity>

Then changing the package to an invalid one like below either by directly editing the mapping file source or thorugh the package chooser. Validation of the mapped class should be given against the invalid package, at least saying the mapped class cannot be resolved.

<package>BLAHBLAH</package>
<entity class="employee.Employee"></entity>

But if giving an invalid class name like below, some validation will show up. 

<package>BLAHBLAH</package>
<entity class="employee.EmployeeAAA"></entity>

Changing the class name back to the right one like below, the validaiton will remain with the invalid package name, which is what we expect.

<package>BLAHBLAH</package>
<entity class="employee.Employee"></entity>

Reproducible: Always
Comment 1 Karen Butzke CLA 2011-06-07 17:51:11 EDT
Created attachment 197548 [details]
proposed patch against head
Comment 2 Karen Butzke CLA 2011-06-24 07:57:26 EDT
checked in to HEAD for 3.0.1