Fixes #30

Author: rafael-pestano

.gitignore

@@ -45,3 +45,4 @@ test-output/
 
 #linux te files
 *~
+/.apt_generated/

src/main/java/com/github/adminfaces/persistence/service/CrudService.java

@@ -17,14 +17,14 @@
 import java.lang.reflect.ParameterizedType;
 import java.lang.reflect.Type;
 import java.util.ArrayList;
+import java.util.Arrays;
 import java.util.Collection;
 import java.util.List;
 import java.util.logging.Logger;
 
 /**
- * @author  <a href="http://github.com/rmpestano">Rafael Pestano</a>
- *
- * Template service for CRUD operations on top of a JPA entity
+ * @author rmpestano
+ * Utility service for crud operations
  */
 @Service
 public class CrudService<T extends PersistenceEntity, PK extends Serializable> extends CriteriaSupportHandler<T> implements CriteriaSupport<T>, Serializable {
@@ -385,5 +385,24 @@ public void beforeRemove(T entity) {
 
     public void afterRemove(T entity) {
     }
+    
+    /**
+     * Creates an array of Ids (pks) from a list of entities. 
+     * It is useful when working with `in clauses` on DeltaSpike criteria
+     * because the API only support primitive arrays.   
+     * 
+     * @param entities list of entities to create 
+     * @param idsType the type of the pk list, e.g new Long[0]
+     * 
+     * @return primitive array containing entities pks.  
+     */
+    @SuppressWarnings("unchecked")
+	protected <ID extends Serializable> ID[] toListOfIds(Collection<? extends PersistenceEntity> entities, ID[] idsType) {
+		List<ID> ids = new ArrayList<>();
+		for (PersistenceEntity entity : entities) {
+			ids.add((ID) entity.getId());
+		}
+		return (ID[]) ids.toArray(idsType);
+	}
 
 }

src/test/java/com/github/adminfaces/persistence/CrudServiceIt.java

@@ -13,6 +13,7 @@
 
 import javax.inject.Inject;
 import java.util.ArrayList;
+import java.util.Arrays;
 import java.util.List;
 
 import static org.assertj.core.api.AssertionsForInterfaceTypes.assertThat;
@@ -60,6 +61,19 @@ public void shouldFindCarByExample() {
                 .extracting("id")
                 .contains(new Integer(1));
     }
+    
+    @Test
+    @DataSet("cars.yml")
+    public void shouldFindCarsByListOfIds() {
+        Car ferrari = new Car(1);
+        Car mustang = new Car(2);
+        List<Car> carsToFind = Arrays.asList(ferrari,mustang);
+        List<Car> cars = carService.findCarsInList(carsToFind);
+        assertThat(cars).isNotNull()
+                .hasSize(2)
+                .extracting("id")
+                .contains(new Integer(1), new Integer(2));
+    }
 
     @Test
     public void shouldNotInsertCarWithoutName() {

src/test/java/com/github/adminfaces/persistence/service/CarService.java

@@ -107,6 +107,10 @@ public Double getTotalPriceByModel(Car car) {
         }
         return carRepository.getTotalPriceByModel(car.getModel().toUpperCase());
     }
+    
+    public List<Car> findCarsInList(List<Car> carsToFind) {
+    	return criteria().in(Car_.id, toListOfIds(carsToFind, new Integer[0])).getResultList();
+    }
 
     @Override
     @Transactional