Bug 183544 - [validation]The validation of the business method might not be invoked.
Summary: [validation]The validation of the business method might not be invoked.
Status: RESOLVED FIXED
Alias: None
Product: WTP Java EE Tools
Classification: WebTools
Component: jst.j2ee (show other bugs)
Version: 2.0   Edit
Hardware: PC Windows XP
: P3 normal (vote)
Target Milestone: 3.0.3   Edit
Assignee: Masashi Sato CLA
QA Contact: Chuck Bridgham CLA
URL:
Whiteboard:
Keywords: contributed, helpwanted
Depends on:
Blocks:
 
Reported: 2007-04-22 21:43 EDT by Masashi Sato CLA
Modified: 2008-10-01 12:24 EDT (History)
3 users (show)

See Also:
cbridgha: review+

Attachments
Patch (1.16 KB, patch)
2008-08-24 23:05 EDT, Masashi Sato CLA 		bjorn.freeman-benson: iplog+
Details | Diff
View All 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 Masashi Sato CLA 2007-04-22 21:43:40 EDT 
Build ID: I20070323-1616

Steps To Reproduce:
The validation of the business method is not invoked for the following condition. 
1. Create a CMP2.0 entity bean.
2. Define a method that throws RemoteException in the enterprise bean class.
3. Reflect the method definition only in a local interface.

In the CMP2.0 entity bean, because throwing RemoteException in the business method is not recommended, EJB Validator should display WARNING(CHKJ2503W).



More information:
Comment 1 Kaloyan Raev CLA 2008-07-18 07:35:23 EDT 
EJB 2.0 spec is quite old already to investigate team resources on this. 
However, any patch contributions by the community will be appreciated.
Comment 2 Masashi Sato CLA 2008-08-24 23:05:47 EDT 
Created attachment 110764 [details]
Patch

I investigated the bug.
The processing of a remote interface is executed two times.  One should be for a local interface. 
Please refer to the patch for details.
Comment 3 Kaloyan Raev CLA 2008-08-26 05:10:31 EDT 
Thanks for the patch, Masashi!

Chuck, the patch looks good for me, but please have a review since the
validation code is in the j2ee.core plugin.
Comment 4 Chuck Bridgham CLA 2008-09-29 14:31:29 EDT 
looks good - approved
Comment 5 Kaloyan Raev CLA 2008-10-01 12:24:37 EDT 
Committed to HEAD and R3_0_maintenance. Released to build.