User: ejort Date: 02/03/05 10:44:24 Modified: src/main/org/jboss/verifier/strategy AbstractVerifier.java EJBVerifier11.java EJBVerifier20.java VerificationContext.java Log: Add support for create<METHOD> and add a warning that the V2 verifier is missing Revision Changes Path 1.27 +35 -36 jboss/src/main/org/jboss/verifier/strategy/AbstractVerifier.java Index: AbstractVerifier.java =================================================================== RCS file: /cvsroot/jboss/jboss/src/main/org/jboss/verifier/strategy/AbstractVerifier.java,v retrieving revision 1.26 retrieving revision 1.27 diff -u -r1.26 -r1.27 --- AbstractVerifier.java 10 Feb 2002 17:19:50 -0000 1.26 +++ AbstractVerifier.java 5 Mar 2002 18:44:23 -0000 1.27 @@ -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: AbstractVerifier.java,v 1.26 2002/02/10 17:19:50 luke_t Exp $ + * $Id: AbstractVerifier.java,v 1.27 2002/03/05 18:44:23 ejort Exp $ */ // standard imports @@ -77,7 +77,7 @@ * </ul> * </p> * - * @version $Revision: 1.26 $ + * @version $Revision: 1.27 $ * @since JDK 1.3 */ public abstract class AbstractVerifier implements VerificationStrategy { @@ -137,7 +137,6 @@ ClassLoader parent = Thread.currentThread().getContextClassLoader(); this.classloader = new URLClassLoader(list, parent); } - } @@ -149,6 +148,11 @@ ************************************************************************* */ + public abstract boolean isCreateMethod(Method m); + + public abstract boolean isEjbCreateMethod(Method m); + + public boolean hasLegalRMIIIOPArguments(Method method) { Class[] params = method.getParameterTypes(); @@ -424,11 +428,6 @@ return (m.getName().startsWith("find")); } - public boolean isCreateMethod(Method m) { - return (m.getName().equals(CREATE_METHOD)); - } - - /** * Checks for at least one non-static field. */ @@ -451,11 +450,9 @@ Method[] method = c.getMethods(); - for (int i = 0; i < method.length; ++i) { - - String name = method[i].getName(); - - if (name.equals(CREATE_METHOD)) + for (int i = 0; i < method.length; ++i) + { + isCreateMethod(method[i]); return true; } @@ -469,11 +466,9 @@ Method[] method = c.getMethods(); - for (int i = 0; i < method.length; ++i) { - - String name = method[i].getName(); - - if (name.equals(EJB_CREATE_METHOD)) + for (int i = 0; i < method.length; ++i) + { + if (isEjbCreateMethod(method[i])) if (!isStatic(method[i]) && !isFinal(method[i]) && ((isSession && hasVoidReturnType(method[i])) @@ -498,9 +493,8 @@ for (int i = 0; i < method.length; ++i) { - String name = method[i].getName(); - - if (name.equals(CREATE_METHOD)) { + if (isCreateMethod(method[i])) + { Class[] params = method[i].getParameterTypes(); if (params.length == 0) @@ -623,7 +617,7 @@ Method[] method = c.getMethods(); for (int i = 0; i < method.length; ++i) - if (method[i].getName().equals(EJB_CREATE_METHOD)) + if (isEjbCreateMethod(method[i])) ejbCreates.add(method[i]); return ejbCreates.iterator(); @@ -650,9 +644,8 @@ for (int i = 0; i < method.length; ++i) { - String name = method[i].getName(); - - if (name.equals(CREATE_METHOD)) { + if (isCreateMethod(method[i])) + { ++count; } } @@ -707,9 +700,9 @@ return (a.getReturnType() == b.getReturnType()); } - public boolean hasMatchingEJBPostCreate(Class bean, Method ejbCreate) { + public boolean hasMatchingEJBPostCreate(Class bean, Method create) { try { - return (bean.getMethod(EJB_POST_CREATE_METHOD, ejbCreate.getParameterTypes()) != null); + return (bean.getMethod(getMatchingEJBPostCreateName(create.getName()), create.getParameterTypes()) != null); } catch (NoSuchMethodException e) { return false; @@ -719,17 +712,17 @@ public boolean hasMatchingEJBCreate(Class bean, Method create) { try { - return (bean.getMethod(EJB_CREATE_METHOD, create.getParameterTypes()) != null); + return (bean.getMethod(getMatchingEJBCreateName(create.getName()), create.getParameterTypes()) != null); } catch (NoSuchMethodException e) { return false; } } - public Method getMatchingEJBPostCreate(Class bean, Method ejbCreate) { + public Method getMatchingEJBPostCreate(Class bean, Method create) { try { - return bean.getMethod(EJB_POST_CREATE_METHOD, ejbCreate.getParameterTypes()); + return bean.getMethod(getMatchingEJBPostCreateName(create.getName()), create.getParameterTypes()); } catch (NoSuchMethodException e) { return null; @@ -739,7 +732,7 @@ public Method getMatchingEJBCreate(Class bean, Method create) { try { - return bean.getMethod(EJB_CREATE_METHOD, create.getParameterTypes()); + return bean.getMethod(getMatchingEJBCreateName(create.getName()), create.getParameterTypes()); } catch (NoSuchMethodException e) { return null; @@ -1032,6 +1025,15 @@ return true; } + private String getMatchingEJBCreateName(String createName) + { + return "ejb" + createName.substring(0,1).toUpperCase() + createName.substring(1); + } + private String getMatchingEJBPostCreateName(String createName) + { + int createIdx = createName.indexOf("Create"); + return "ejbPost" + createName.substring(createIdx>=0?createIdx:0); + } /* ************************************************************************* @@ -1063,16 +1065,13 @@ private final static String EJB_FIND_BY_PRIMARY_KEY = "ejbFindByPrimaryKey"; - private final static String EJB_CREATE_METHOD = + protected final static String EJB_CREATE_METHOD = "ejbCreate"; private final static String EJB_POST_CREATE_METHOD = "ejbPostCreate"; - private final static String EJB_POST_METHOD = - "ejbCreate"; - - private final static String CREATE_METHOD = + protected final static String CREATE_METHOD = "create"; private final static String FINALIZE_METHOD = 1.29 +11 -2 jboss/src/main/org/jboss/verifier/strategy/EJBVerifier11.java Index: EJBVerifier11.java =================================================================== RCS file: /cvsroot/jboss/jboss/src/main/org/jboss/verifier/strategy/EJBVerifier11.java,v retrieving revision 1.28 retrieving revision 1.29 diff -u -r1.28 -r1.29 --- EJBVerifier11.java 18 Jun 2001 20:01:29 -0000 1.28 +++ EJBVerifier11.java 5 Mar 2002 18:44:24 -0000 1.29 @@ -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.28 2001/06/18 20:01:29 mnf999 Exp $ + * $Id: EJBVerifier11.java,v 1.29 2002/03/05 18:44:24 ejort Exp $ */ @@ -55,7 +55,7 @@ * @author Aaron Mulder ([EMAIL PROTECTED]) * @author Vinay Menon ([EMAIL PROTECTED]) * - * @version $Revision: 1.28 $ + * @version $Revision: 1.29 $ * @since JDK 1.3 */ public class EJBVerifier11 extends AbstractVerifier { @@ -132,6 +132,15 @@ } } + public boolean isCreateMethod(Method m) + { + return m.getName().equals(CREATE_METHOD); + } + + public boolean isEjbCreateMethod(Method m) + { + return m.getName().equals(EJB_CREATE_METHOD); + } /* 1.10 +21 -13 jboss/src/main/org/jboss/verifier/strategy/EJBVerifier20.java Index: EJBVerifier20.java =================================================================== RCS file: /cvsroot/jboss/jboss/src/main/org/jboss/verifier/strategy/EJBVerifier20.java,v retrieving revision 1.9 retrieving revision 1.10 diff -u -r1.9 -r1.10 --- EJBVerifier20.java 3 Jan 2001 08:28:48 -0000 1.9 +++ EJBVerifier20.java 5 Mar 2002 18:44:24 -0000 1.10 @@ -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: EJBVerifier20.java,v 1.9 2001/01/03 08:28:48 tobias Exp $ + * $Id: EJBVerifier20.java,v 1.10 2002/03/05 18:44:24 ejort Exp $ */ @@ -43,7 +43,7 @@ * EJB 2.0 bean verifier. * * @author Juha Lindfors ([EMAIL PROTECTED]) - * @version $Revision: 1.9 $ + * @version $Revision: 1.10 $ * @since JDK 1.3 */ public class EJBVerifier20 extends AbstractVerifier { @@ -64,21 +64,29 @@ *********************************************************************** */ - public void checkSession(SessionMetaData session) { - - // NOT IMPLEMENTED YET + public void checkSession(SessionMetaData session) + { + System.out.println("WARNING: EJBVerifier2.0 Session verification not implemented"); } - public void checkEntity(EntityMetaData entity) { - - // NOT IMPLEMENTED YET + public void checkEntity(EntityMetaData entity) + { + System.out.println("WARNING: EJBVerifier2.0 Entity verification not implemented"); } - public void checkMessageBean(BeanMetaData bean) { - - // NOT IMPLEMENTED YET + public void checkMessageBean(BeanMetaData bean) + { + System.out.println("WARNING: EJBVerifier2.0 Message verification not implemented"); + } + + public boolean isCreateMethod(Method m) + { + return m.getName().startsWith(CREATE_METHOD); + } + + public boolean isEjbCreateMethod(Method m) + { + return m.getName().startsWith(EJB_CREATE_METHOD); } - - } 1.7 +3 -3 jboss/src/main/org/jboss/verifier/strategy/VerificationContext.java Index: VerificationContext.java =================================================================== RCS file: /cvsroot/jboss/jboss/src/main/org/jboss/verifier/strategy/VerificationContext.java,v retrieving revision 1.6 retrieving revision 1.7 diff -u -r1.6 -r1.7 --- VerificationContext.java 3 Jan 2001 08:28:48 -0000 1.6 +++ VerificationContext.java 5 Mar 2002 18:44:24 -0000 1.7 @@ -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: VerificationContext.java,v 1.6 2001/01/03 08:28:48 tobias Exp $ + * $Id: VerificationContext.java,v 1.7 2002/03/05 18:44:24 ejort Exp $ * * You can reach the author by sending email to [EMAIL PROTECTED] */ @@ -47,7 +47,7 @@ * @see << OTHER RELATED CLASSES >> * * @author Juha Lindfors - * @version $Revision: 1.6 $ + * @version $Revision: 1.7 $ * @since JDK 1.3 */ public interface VerificationContext extends StrategyContext, @@ -63,7 +63,7 @@ * Version identifier. */ public final static String VERSION_2_0 = - "No public release yet."; + "Enterprise JavaBeans V2.0, Final Release";
_______________________________________________ Jboss-development mailing list [EMAIL PROTECTED] https://lists.sourceforge.net/lists/listinfo/jboss-development