User: forder  
  Date: 00/09/09 16:23:46

  Modified:    src/main/org/jboss/ejb/plugins/jaws/jdbc
                        JDBCFindEntityCommand.java
  Log:
  Multiple matches in a Finder that should return a single entity now result in a 
FinderException, as recommended by Dan O'Connor.
  
  Revision  Changes    Path
  1.3       +5 -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.2
  retrieving revision 1.3
  diff -u -r1.2 -r1.3
  --- JDBCFindEntityCommand.java        2000/08/26 19:58:20     1.2
  +++ JDBCFindEntityCommand.java        2000/09/09 23:23:45     1.3
  @@ -29,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.2 $
  + * @version $Revision: 1.3 $
    */
   public class JDBCFindEntityCommand implements JPMFindEntityCommand
   {
  @@ -66,9 +66,12 @@
            if (result.size() == 0)
            {
               throw new ObjectNotFoundException("No such entity!");
  -         } else
  +         } else if (result.size() == 1)
            {
               return result.get(0);
  +         } else
  +         {
  +            throw new FinderException("More than one entity matches the finder 
criteria.");
            }
         }
      }
  
  
  

Reply via email to