#335 - Remove deprecated method - delete(Collection<?> beans) ... migrate to deleteAll()

src/main/java/com/avaje/ebean/Ebean.java

@@ -750,14 +750,6 @@ public final class Ebean {
     serverMgr.getDefaultServer().deleteAll(beanType, ids);
   }
 
-  /**
-   * Delete all the beans from a Collection.
-   */
-  @Deprecated
-  public static int delete(Collection<?> beans) throws OptimisticLockException {
-    return serverMgr.getDefaultServer().deleteAll(beans);
-  }
-
   /**
    * Delete all the beans in the Collection.
    */

src/main/java/com/avaje/ebean/EbeanServer.java

@@ -1223,14 +1223,6 @@ public interface EbeanServer {
    */
   void delete(Object bean) throws OptimisticLockException;
 
-  /**
-   * Deprecated - please migrate to deleteAll().
-   *
-   * Delete all the beans in the collection.
-   */
-  @Deprecated
-  int delete(Collection<?> beans) throws OptimisticLockException;
-
   /**
    * Delete all the beans in the collection.
    */

src/main/java/com/avaje/ebeaninternal/server/core/DefaultServer.java

@@ -1795,14 +1795,6 @@ public final class DefaultServer implements SpiEbeanServer {
     persister.delete(checkEntityBean(bean), t);
   }
 
-  /**
-   * Delete all the beans in the collection.
-   */
-  @Override
-  public int delete(Collection<?> beans) {
-    return deleteAllInternal(beans.iterator(), null);
-  }
-
   @Override
   public int delete(Iterator<?> it, Transaction transaction) throws OptimisticLockException {
     return deleteAllInternal(it, transaction);

src/test/java/com/avaje/ebeaninternal/api/TDSpiEbeanServer.java

@@ -488,11 +488,6 @@ public class TDSpiEbeanServer implements SpiEbeanServer {
 
   }
 
-  @Override
-  public int delete(Collection<?> c) throws OptimisticLockException {
-    return 0;
-  }
-
   @Override
   public int delete(Class<?> beanType, Object id) {
     return 0;

src/test/java/com/avaje/tests/compositekeys/TestOnCascadeDeleteChildrenWithCompositeKeys.java

@@ -32,7 +32,7 @@ import com.avaje.ebeaninternal.server.deploy.BeanPropertyAssocMany;
 public class TestOnCascadeDeleteChildrenWithCompositeKeys extends BaseTestCase {
   @Before public void before() {
     // remove all the User records first
-    Ebean.delete(Ebean.find(User.class).findList());
+    Ebean.deleteAll(Ebean.find(User.class).findList());
 
     // insert 2 User records
     Ebean.save(new User(1L));

src/test/java/com/avaje/tests/m2m/TestM2MDeleteWithCascade.java

@@ -44,7 +44,7 @@ public class TestM2MDeleteWithCascade extends BaseTestCase {
 
     Assert.assertEquals("roles not deleted", 2, rc);
 
-    Ebean.delete(roles);
+    Ebean.deleteAll(roles);
 
     rc = Ebean.find(MRole.class).where().idIn(roleIds).findRowCount();