Add possibility to delete specific places from a map

code/moya-beans-client/ejbModule/fi/codecrew/moya/beans/EventMapBeanLocal.java

@@ -34,7 +34,7 @@ public interface EventMapBeanLocal {
 
 	EventMap find(Integer mapId);
 
-	EventMap clearPlaces(EventMap map);
+	EventMap clearPlaces(EventMap map, String placesToDelete);
 
 	void deletePlace(Place place);
 

code/moya-beans/ejbModule/fi/codecrew/moya/beans/EventMapBean.java

@@ -39,7 +39,7 @@ import fi.codecrew.moya.model.Place;
  * Session Bean implementation class EventMapBean
  */
 @Stateless
-@DeclareRoles({ MapPermission.S_MANAGE_MAPS })
+@DeclareRoles({MapPermission.S_MANAGE_MAPS})
 public class EventMapBean implements EventMapBeanLocal {
 
 	@EJB
@@ -97,10 +97,12 @@ public class EventMapBean implements EventMapBeanLocal {
 
 	@Override
 	@RolesAllowed(MapPermission.S_MANAGE_MAPS)
-	public EventMap clearPlaces(EventMap map) {
-		EventMap leMap = eventmapfacade.reload(map);
-		leMap.getPlaces().clear();
-		return leMap;
+	public EventMap clearPlaces(EventMap map, String placesToDelete) {
+		map = eventmapfacade.reload(map);
+		if (!eventbean.getCurrentEvent().equals(map.getEvent())) {
+			throw new EJBAccessException("Deleting places for wrong event!");
+		}
+		return placefacade.deletePlaces(map, placesToDelete);
 	}
 
 	@Override
@@ -109,7 +111,7 @@ public class EventMapBean implements EventMapBeanLocal {
 		place = placefacade.reload(place);
 		LanEvent currentEvent = eventbean.getCurrentEvent();
 		if (!currentEvent.equals(place.getMap().getEvent())) {
-			throw new EJBAccessException("Deleting placce for wrong event!");
+			throw new EJBAccessException("Deleting place for wrong event!");
 		}
 		placefacade.remove(place);
 	}

code/moya-beans/ejbModule/fi/codecrew/moya/beans/PlaceBean.java

@@ -646,7 +646,7 @@ public class PlaceBean implements PlaceBeanLocal {
 	@Override
 	public byte[] generatePlacesPdf(float width, float height, double font1, double font2) {
 		try {
-			List<Place> places = placeFacade.findAll();
+			List<Place> places = placeFacade.findAll(eventBean.getCurrentEvent());
 			Collections.sort(places);
 
 			logger.info("sorting places etc.");
@@ -821,7 +821,7 @@ public class PlaceBean implements PlaceBeanLocal {
 	@Override
 	public List<Place> findAllForEvent() {
 		LanEvent event = eventBean.getCurrentEvent();
-		List<Place> ret = placeFacade.findAllForEvent(event);
+		List<Place> ret = placeFacade.findAll(event);
 		return ret;
 	}
 

code/moya-beans/ejbModule/fi/codecrew/moya/facade/EventMapFacade.java

@@ -25,6 +25,7 @@ import javax.ejb.EJB;
 import javax.ejb.LocalBean;
 import javax.ejb.Stateless;
 import javax.persistence.criteria.CriteriaBuilder;
+import javax.persistence.criteria.CriteriaDelete;
 import javax.persistence.criteria.CriteriaQuery;
 import javax.persistence.criteria.Root;
 
@@ -49,8 +50,7 @@ public class EventMapFacade extends IntegerPkGenericFacade<EventMap> {
 	@EJB
 	private PermissionBeanLocal permbean;
 
-	public List<EventMap> getMaps()
-	{
+	public List<EventMap> getMaps() {
 		CriteriaBuilder cb = getEm().getCriteriaBuilder();
 		CriteriaQuery<EventMap> cq = cb.createQuery(EventMap.class);
 		Root<EventMap> root = cq.from(EventMap.class);

code/moya-beans/ejbModule/fi/codecrew/moya/facade/PlaceFacade.java

@@ -19,7 +19,6 @@
 package fi.codecrew.moya.facade;
 
 import java.util.Arrays;
-import java.util.Calendar;
 import java.util.List;
 
 import javax.ejb.EJB;
@@ -35,7 +34,6 @@ import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 
 import fi.codecrew.moya.beans.LoggingBeanLocal;
-import fi.codecrew.moya.utilities.moyamessage.MoyaEventType;
 
 @Stateless
 @LocalBean
@@ -57,11 +55,25 @@ public class PlaceFacade extends IntegerPkGenericFacade<Place> {
 		super(Place.class);
 	}
 
+	public EventMap deletePlaces(EventMap map, String nameLike) {
+		CriteriaBuilder cb = getEm().getCriteriaBuilder();
+		CriteriaDelete<Place> del = cb.createCriteriaDelete(Place.class);
+		Root<Place> root = del.getRoot();
+		del.where(
+			cb.equal(root.get(Place_.map), map),
+			cb.like(root.get(Place_.name), nameLike)
+		);
+		int i = getEm().createQuery(del).executeUpdate();
+
+		logger.info("Deleted {} places from {} with string '{}'", i, map, nameLike);
+		getEm().getEntityManagerFactory().getCache().evict(EventMap.class, map.getId());
+		getEm().refresh(map);
+		return map;
+	}
 
 	public List<Place> findUsersReservations(LanEvent event, EventUser user) {
 		CriteriaBuilder cb = getEm().getCriteriaBuilder();
 		CriteriaQuery<Place> cq = cb.createQuery(Place.class);
-
 		Root<Place> root = cq.from(Place.class);
 
 		cq.select(root);
@@ -142,11 +154,16 @@ public class PlaceFacade extends IntegerPkGenericFacade<Place> {
 		return getSingleNullableResult(getEm().createQuery(cq));
 	}
 
-	public List<Place> findAll() {
+	public List<Place> findAll(LanEvent event) {
 		CriteriaBuilder cb = getEm().getCriteriaBuilder();
 		CriteriaQuery<Place> cq = cb.createQuery(Place.class);
 		Root<Place> root = cq.from(Place.class);
-		cq.where(cb.equal(root.get(Place_.map).get(EventMap_.event), eventBean.getCurrentEvent()));
+
+		Path<EventMap> map = root.get(Place_.map);
+		cq.where(
+			cb.isTrue(map.get(EventMap_.active)),
+			cb.equal(map.get(EventMap_.event), event)
+		);
 		cq.orderBy(cb.asc(root.get(Place_.name)));
 		return getEm().createQuery(cq).getResultList();
 	}
@@ -162,7 +179,8 @@ public class PlaceFacade extends IntegerPkGenericFacade<Place> {
 			cb.isNull(root.get(Place_.reserveTime)),
 			cb.isNull(root.get(Place_.group)),
 			cb.isFalse(root.get(Place_.disabled)),
-				cb.isTrue(root.get(Place_.buyable))
+			cb.isTrue(root.get(Place_.buyable)),
+			cb.isTrue(root.get(Place_.map).get(EventMap_.active))
 		);
 		return super.getSingleNullableResult(getEm().createQuery(cq));
 	}
@@ -175,7 +193,8 @@ public class PlaceFacade extends IntegerPkGenericFacade<Place> {
 
 		cq.where(
 			cb.equal(root.get(Place_.product), product),
-				cb.isFalse(root.get(Place_.disabled))
+			cb.isFalse(root.get(Place_.disabled)),
+			cb.isTrue(root.get(Place_.map).get(EventMap_.active))
 		);
 		return super.getSingleNullableResult(getEm().createQuery(cq));
 	}
@@ -262,7 +281,7 @@ public class PlaceFacade extends IntegerPkGenericFacade<Place> {
 			cb.equal(placeJoin.get(Place_.map).get(EventMap_.event), eventBean.getCurrentEvent())
 		);
 
-		ProductFlag[] flags = { ProductFlag.CREATE_NEW_PLACE_WHEN_BOUGHT };
+		ProductFlag[] flags = {ProductFlag.CREATE_NEW_PLACE_WHEN_BOUGHT};
 
 		cq.where(
 			cb.or(
@@ -282,17 +301,12 @@ public class PlaceFacade extends IntegerPkGenericFacade<Place> {
 		CriteriaQuery<Place> cq = cb.createQuery(Place.class);
 		Root<Place> root = cq.from(Place.class);
 
-		cq.where(cb.equal(root.get(Place_.product).get(Product_.event), event), cb.equal(root.get(Place_.code), code));
+		cq.where(
+			cb.equal(root.get(Place_.product).get(Product_.event), event),
+			cb.equal(root.get(Place_.code), code)
+		);
 		return super.getSingleNullableResult(getEm().createQuery(cq));
 	}
 
-	public List<Place> findAllForEvent(LanEvent event) {
-		CriteriaBuilder cb = getEm().getCriteriaBuilder();
-		CriteriaQuery<Place> cq = cb.createQuery(Place.class);
-		Root<Place> root = cq.from(Place.class);
-
-		cq.where(cb.equal(root.get(Place_.product).get(Product_.event), event));
-		return getEm().createQuery(cq).getResultList();
 
-	}
 }

code/moya-web/WebContent/map/edit.xhtml

@@ -30,13 +30,17 @@
 			</h:form>
 			<map:genplaces />
 
+			<hr />
+
 			<h:form>
-				<h:commandButton action="#{mapManageView.removeAllPlaces}" value="#{i18n['mapEdit.removePlaces']}" onclick="return confirm('Are you sure you want to REMOVE ALL PLACES!\nThis action is final')" />
+				<p:inputText value="#{mapManageView.placesToDelete}" />
+				<p:commandButton action="#{mapManageView.removeSelectedPlaces}" value="#{i18n['mapEdit.removeSelectedPlaces']}" onclick="return confirm('Are you sure you want to remove these\nThis action is final')"  ajax="false"/>
 			</h:form>
 
+			<hr/>
 			<h:form enctype="multipart/form-data">
 				<p:fileUpload mode="simple" value="#{mapManageView.bgFile}" />
-				<h:commandButton action="#{mapManageView.submitBg}" value="#{i18n['map.submitMap']}" />
+				<p:commandButton action="#{mapManageView.submitBg}" value="#{i18n['map.submitMap']}" ajax="false"/>
 			</h:form>
 
 			<h:form>

code/moya-web/src/main/java/fi/codecrew/moya/web/cdiview/map/MapManageView.java

@@ -77,6 +77,7 @@ public class MapManageView extends GenericCDIView {
 	private String mapname;
 
 	private String buyableLike;
+	private String placesToDelete;
 
 	private boolean tablesHorizontal = false;
 	private boolean oneRowTable = false;
@@ -200,8 +201,10 @@ public class MapManageView extends GenericCDIView {
 		startY = new Integer(requestParams.get(clientId + ".y")).intValue();
 	}
 
-	public String removeAllPlaces() {
-		map = eventmapBean.clearPlaces(map);
+
+
+	public String removeSelectedPlaces() {
+		map = eventmapBean.clearPlaces(map, placesToDelete);
 		return null;
 	}
 
@@ -408,4 +411,12 @@ public class MapManageView extends GenericCDIView {
 	public void setSourceMap(EventMap sourceMap) {
 		this.sourceMap = sourceMap;
 	}
+
+	public String getPlacesToDelete() {
+		return placesToDelete;
+	}
+
+	public void setPlacesToDelete(String placesToDelete) {
+		this.placesToDelete = placesToDelete;
+	}
 }

code/moya-web/src/main/resources/fi/codecrew/moya/resources/i18n.properties

@@ -1609,3 +1609,4 @@ product.dependency.multiplier=Multiplier
 product.dependency.MAX_SUPPORTED_COUNT=Only up to same number of products can be bought
 
 submenu.neomap.moveplaces=Change places
+mapEdit.removeSelectedPlaces=Delete places

code/moya-web/src/main/resources/fi/codecrew/moya/resources/i18n_en.properties

@@ -1890,3 +1890,4 @@ placemove.alreadyTaken=Moving the places was cancelled because place {0} was alr
 
 placegroupview.moveUsersPlaces=Move users places
 submenu.neomap.moveplaces=Change places
+mapEdit.removeSelectedPlaces=Delete places

code/moya-web/src/main/resources/fi/codecrew/moya/resources/i18n_fi.properties

@@ -1876,3 +1876,4 @@ product.dependency.MAX_SUPPORTED_COUNT=Tuotteita voi ostaa enint\u00E4\u00E4n sa
 placemove.alreadyTaken=Paikkojen siirto peruuntui koska paikka {0} oli jo varattu.
 placegroupview.moveUsersPlaces=Siirr\u00E4 k\u00E4ytt\u00E4j\u00E4n paikkoja
 submenu.neomap.moveplaces=Vaihda paikkoja
+mapEdit.removeSelectedPlaces=Poista paikat