Keepers of the JBoss:
This change should have also been included in the "findByPrimaryKey"
method just below in the same file.
In other works, findByPrimaryKey should also through an
ObjectNotFoundException instead of a FinderException when it does not
find the row.
Pat McNerthney
Icicle Software, Inc.
jBoss CVS Development wrote:
>
> User: forder
> Date: 00/08/26 12:58:20
>
> Modified: src/main/org/jboss/ejb/plugins/jaws/jdbc
> JDBCFindEntityCommand.java
> Log:
> Failure to find now throws ObjectNotFoundException.
>
> Revision Changes Path
> 1.2 +3 -2
>jboss/src/main/org/jboss/ejb/plugins/jaws/jdbc/JDBCFindEntityCommand.java
>
> Index: JDBCFindEntityCommand.java
> ===================================================================
> RCS file:
>/products/cvs/ejboss/jboss/src/main/org/jboss/ejb/plugins/jaws/jdbc/JDBCFindEntityCommand.java,v
> retrieving revision 1.1
> retrieving revision 1.2
> diff -u -r1.1 -r1.2
> --- JDBCFindEntityCommand.java 2000/08/06 02:03:51 1.1
> +++ JDBCFindEntityCommand.java 2000/08/26 19:58:20 1.2
> @@ -15,6 +15,7 @@
> import java.rmi.RemoteException;
>
> import javax.ejb.FinderException;
> +import javax.ejb.ObjectNotFoundException;
>
> import org.jboss.ejb.EntityEnterpriseContext;
> import org.jboss.ejb.plugins.jaws.JPMFindEntityCommand;
> @@ -28,7 +29,7 @@
> * @author <a href="mailto:[EMAIL PROTECTED]">Marc Fleury</a>
> * @author <a href="mailto:[EMAIL PROTECTED]">Joe Shevland</a>
> * @author <a href="mailto:[EMAIL PROTECTED]">Justin Forder</a>
> - * @version $Revision: 1.1 $
> + * @version $Revision: 1.2 $
> */
> public class JDBCFindEntityCommand implements JPMFindEntityCommand
> {
> @@ -64,7 +65,7 @@
>
> if (result.size() == 0)
> {
> - throw new FinderException("No such entity!");
> + throw new ObjectNotFoundException("No such entity!");
> } else
> {
> return result.get(0);
>
>
>
