Update of /cvsroot/xdoclet/xdoclet/modules/ejb/src/xdoclet/modules/ejb/dd
In directory 
sc8-pr-cvs1.sourceforge.net:/tmp/cvs-serv15406/modules/ejb/src/xdoclet/modules/ejb/dd

Modified Files:
        EjbRefTagsHandler.java 
Log Message:
Proper error message if the ejb-name parameter for @ejb.ejb-ref is missing 
(XDT-911). Thanks to Edson Tirelli for the patch

Index: EjbRefTagsHandler.java
===================================================================
RCS file: 
/cvsroot/xdoclet/xdoclet/modules/ejb/src/xdoclet/modules/ejb/dd/EjbRefTagsHandler.java,v
retrieving revision 1.17
retrieving revision 1.18
diff -C2 -r1.17 -r1.18
*** EjbRefTagsHandler.java      4 Feb 2005 17:50:36 -0000       1.17
--- EjbRefTagsHandler.java      8 Feb 2005 21:46:21 -0000       1.18
***************
*** 85,89 ****
                  storeReferringClassId();
  
!                 XClass refedEJBClass = 
findEjb(tag.getAttributeValue("ejb-name"));
  
                  setCurrentClass(refedEJBClass);
--- 85,104 ----
                  storeReferringClassId();
  
!                 /*
!                  * Checking for null value to avoid NullPointerException when 
calling this.findEjb()
!                  * Author: Edson Tirelli - mailto:tirelli at post.com
!                  */
!                 String ejbNameAttribute = tag.getAttributeValue("ejb-name");
! 
!                 if (ejbNameAttribute == null) {
!                     log.error("Missing value for mandatory attribute 
\"ejb-name\" in tag @ejb.ejb-ref!");
!                     log.error("Current tag: @ejb.ejb-ref ref-name=\"" +
!                         tag.getAttributeValue("ref-name") + "\" ejb-name=\"" +
!                         tag.getAttributeValue("ejb-name") + "\" view-type=\"" 
+
!                         tag.getAttributeValue("view-type") + "\"");
!                     throw new XDocletException("Missing value for mandatory 
attribute \"ejb-name\" in tag @ejb.ejb-ref!");
!                 }
! 
!                 XClass refedEJBClass = findEjb(ejbNameAttribute);
  
                  setCurrentClass(refedEJBClass);



-------------------------------------------------------
SF email is sponsored by - The IT Product Guide
Read honest & candid reviews on hundreds of IT Products from real users.
Discover which products truly live up to the hype. Start reading now.
http://ads.osdn.com/?ad_id=6595&alloc_id=14396&op=click
_______________________________________________
xdoclet-devel mailing list
xdoclet-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/xdoclet-devel

Reply via email to