User: juha
Date: 00/08/26 13:14:14
Modified: src/main/org/jboss/verifier/strategy EJBVerifier11.java
Log:
A tad too paranoid on the ejbFindByPrimaryKey() methods.
For CMP they're provided by the container.
Revision Changes Path
1.14 +8 -4 jboss/src/main/org/jboss/verifier/strategy/EJBVerifier11.java
Index: EJBVerifier11.java
===================================================================
RCS file:
/products/cvs/ejboss/jboss/src/main/org/jboss/verifier/strategy/EJBVerifier11.java,v
retrieving revision 1.13
retrieving revision 1.14
diff -u -r1.13 -r1.14
--- EJBVerifier11.java 2000/08/20 20:29:16 1.13
+++ EJBVerifier11.java 2000/08/26 20:14:14 1.14
@@ -19,7 +19,7 @@
* Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA 02111-1307, USA.
*
* This package and its source code is available at www.jboss.org
- * $Id: EJBVerifier11.java,v 1.13 2000/08/20 20:29:16 juha Exp $
+ * $Id: EJBVerifier11.java,v 1.14 2000/08/26 20:14:14 juha Exp $
*/
@@ -58,7 +58,7 @@
* @see org.jboss.verifier.strategy.AbstractVerifier
*
* @author Juha Lindfors ([EMAIL PROTECTED])
- * @version $Revision: 1.13 $
+ * @version $Revision: 1.14 $
* @since JDK 1.3
*/
public class EJBVerifier11 extends AbstractVerifier {
@@ -946,8 +946,12 @@
*
* Spec 9.2.5
*/
- if (!hasEJBFindByPrimaryKey(bean)) {
-
+ if (entity.isBMP() && (!hasEJBFindByPrimaryKey(bean))) {
+ /* Even though the spec states that all entities must have the
+ * ejbFindByPrimaryKey() implementation, we only check BMP.
+ * For CMP it is the responsibility of the container to
+ * provide the implementation. */
+
fireSpecViolationEvent(entity, new Section("9.2.5.a"));
status = false;
