Merge branch 'master' of dev.insomnia.fi:/data/bortal

code/LanBortalBeans/ejbModule/fi/insomnia/bortal/beans/AccountEventBean.java

@@ -25,7 +25,6 @@ import fi.insomnia.bortal.model.EventUser;
 import fi.insomnia.bortal.model.IUser;
 import fi.insomnia.bortal.model.Product;
 import fi.insomnia.bortal.model.Role;
-import fi.insomnia.bortal.model.User;
 
 /**
  * Session Bean implementation class AccountEventBean
@@ -95,7 +94,7 @@ public class AccountEventBean implements AccountEventBeanLocal {
 	public void shopCash(EventUser shoppingUser, Map<Product, BigDecimal> shopMap, boolean buyInstant) {
 		logger.debug("Shoping cash. buyinstant {}", buyInstant);
 		EventUser seller = permbean.getCurrentUser();
-		shoppingUser = userbean.findEventUserById(shoppingUser.getId());
+		shoppingUser = userbean.findByEventUserId(shoppingUser.getId());
 
 		BigDecimal tot = BigDecimal.ZERO;
 

code/LanBortalBeans/ejbModule/fi/insomnia/bortal/beans/OrgRoleBean.java

+package fi.insomnia.bortal.beans;
+
+import java.util.List;
+
+import javax.ejb.EJB;
+import javax.ejb.LocalBean;
+import javax.ejb.Stateless;
+
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
+
+import fi.insomnia.bortal.facade.OrgRoleFacade;
+import fi.insomnia.bortal.model.OrgRole;
+
+/**
+ * Session Bean implementation class OrgRoleBean
+ */
+@Stateless
+@LocalBean
+public class OrgRoleBean implements OrgRoleBeanLocal {
+
+	private static final Logger logger = LoggerFactory
+			.getLogger(OrgRoleBean.class);
+
+	@EJB
+	private EventBeanLocal eventBean;
+
+	@EJB
+	private OrgRoleFacade orgRoleFacade;
+
+	public OrgRoleBean() {
+	}
+
+	@Override
+	public List<OrgRole> listOrgRoles() {
+		return orgRoleFacade.findByOrganizer(eventBean.getCurrentEvent()
+				.getOrganiser());
+	}
+
+}

code/LanBortalBeans/ejbModule/fi/insomnia/bortal/beans/UserBean.java

@@ -298,11 +298,11 @@ public class UserBean implements UserBeanLocal {
 		return true;
 	}
 
-	@Override
-	@RolesAllowed(UserPermission.S_VIEW_ALL)
-	public User findById(Integer id) {
-		return userFacade.find(id);
-	}
+	// @Override
+	// @RolesAllowed(UserPermission.S_VIEW_ALL)
+	// public User findById(Integer id) {
+	// return userFacade.find(id);
+	// }
 
 	@Override
 	@RolesAllowed(UserPermission.S_VIEW_ALL)
@@ -407,7 +407,7 @@ public class UserBean implements UserBeanLocal {
 	}
 
 	@Override
-	public EventUser findEventUserById(Integer integer) {
+	public EventUser findByEventUserId(Integer integer) {
 		return eventUserFacade.find(integer);
 	}
 
@@ -435,4 +435,15 @@ public class UserBean implements UserBeanLocal {
 		}
 		return ret;
 	}
+
+	@Override
+	public EventUser findByUserId(Integer userid) {
+		User user = userFacade.find(userid);
+		EventUser ret = null;
+		if (user != null)
+		{
+			ret = this.getEventUser(user);
+		}
+		return ret;
+	}
 }

code/LanBortalBeans/ejbModule/fi/insomnia/bortal/facade/OrgRoleFacade.java

@@ -58,4 +58,13 @@ public class OrgRoleFacade extends IntegerPkGenericFacade<OrgRole> {
 		return getEm().createQuery(cq).getResultList();
 	}
 
+	public List<OrgRole> findByOrganizer(EventOrganiser organiser) {
+		CriteriaBuilder cb = getEm().getCriteriaBuilder();
+		CriteriaQuery<OrgRole> cq = cb.createQuery(OrgRole.class);
+		Root<OrgRole> root = cq.from(OrgRole.class);
+		cq.where(cb.equal(root.get(OrgRole_.eventOrganizer), organiser));
+		
+		return getEm().createQuery(cq).getResultList();
+	}
+
 }

code/LanBortalBeansClient/ejbModule/fi/insomnia/bortal/beans/OrgRoleBeanLocal.java

+package fi.insomnia.bortal.beans;
+
+import java.util.List;
+
+import javax.ejb.Local;
+
+import fi.insomnia.bortal.model.OrgRole;
+
+@Local
+public interface OrgRoleBeanLocal {
+
+	List<OrgRole> listOrgRoles();
+
+}

code/LanBortalBeansClient/ejbModule/fi/insomnia/bortal/beans/UserBeanLocal.java

@@ -33,9 +33,9 @@ public interface UserBeanLocal {
 
 	boolean initPasswordReset(String user, String mailpath);
 
-	User findById(Integer integer);
+	EventUser findByUserId(Integer userid);
 
-	EventUser findEventUserById(Integer integer);
+	EventUser findByEventUserId(Integer integer);
 
 	GroupMembership findToken(String token);
 

code/LanBortalWeb/WebContent/orgrole/list.xhtml

+<!DOCTYPE html
+    PUBLIC "-//W3C//DTD XHTML 1.0 Strict//EN"
+    "http://www.w3.org/TR/xhtml1/DTD/xhtml1-strict.dtd">
+<html xmlns="http://www.w3.org/1999/xhtml"
+	xmlns:ui="http://java.sun.com/jsf/facelets"
+	xmlns:h="http://java.sun.com/jsf/html"
+	xmlns:f="http://java.sun.com/jsf/core"
+	xmlns:orgrole="http://java.sun.com/jsf/composite/cditools/orgrole"
+	xmlns:c="http://java.sun.com/jsp/jstl/core">
+<h:body>
+	<ui:composition
+		template="/layout/#{sessionHandler.layout}/template.xhtml">
+		<ui:param name="thispage" value="page.orgrole.list" />
+		<f:metadata>
+			<f:event type="preRenderView"
+				listener="#{orgRoleView.permissionRead()}" />
+		</f:metadata>
+		<ui:define name="content">
+			<orgrole:list />
+		</ui:define>
+	</ui:composition>
+</h:body>
+</html>
\ No newline at end of file

code/LanBortalWeb/WebContent/resources/cditools/orgrole/list.xhtml

+<?xml version='1.0' encoding='UTF-8' ?>
+<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN"
+    "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">
+<html xmlns="http://www.w3.org/1999/xhtml"
+	xmlns:h="http://java.sun.com/jsf/html"
+	xmlns:f="http://java.sun.com/jsf/core"
+	xmlns:composite="http://java.sun.com/jsf/composite"
+	xmlns:ui="http://java.sun.com/jsf/facelets"
+	xmlns:c="http://java.sun.com/jsp/jstl/core"
+	xmlns:tools="http://java.sun.com/jsf/composite/tools">
+
+
+<composite:interface>
+</composite:interface>
+
+<composite:implementation>
+
+	<h:form id="orgrolelist">
+		<h:dataTable border="1" id="user" value="#{orgRoleDataView.orgRoles}"
+			var="orgRole">
+			<h:column>
+				<f:facet name="header">
+					<h:outputText value="#" />
+				</f:facet>
+				<h:outputText value="#{orgRole.id}" />
+			</h:column>
+			<h:column>
+				<f:facet name="header">
+					<h:outputText value="#{i18n['orgrole.name']}" />
+				</f:facet>
+				<h:outputText value="#{orgRole.name}" />
+			</h:column>
+			<h:column>
+				<h:link outcome="/orgrole/edit" value="#{i18n['orgrole.edit']}">
+					<f:param name="orgroleid" value="#{orgRole.id}" />
+				</h:link>
+			</h:column>
+		</h:dataTable>
+	</h:form>
+
+
+</composite:implementation>
+</html>
\ No newline at end of file

code/LanBortalWeb/src/fi/insomnia/bortal/servlet/UploadServlet.java

@@ -110,7 +110,7 @@ public class UploadServlet extends HttpServlet {
 			String type = request.getParameter("type");
 			if (type.equals("userimage")) {
 				logger.debug("Uploading userimage for user {}");
-				UserImage userimage = userbean.uploadImage(userbean.getEventUser(userbean.findById(destId)), contenttype, imagedata, filename, description);
+				UserImage userimage = userbean.uploadImage(userbean.findByEventUserId(destId), contenttype, imagedata, filename, description);
 				utilbean.convertImage(userimage.getUser());
 			} else if (type.equals("mapimage")) {
 				eventmapbean.sendImage(destId, imagedata);

code/LanBortalWeb/src/fi/insomnia/bortal/web/cdiview/shop/ProductShopView.java

@@ -52,7 +52,7 @@ public class ProductShopView extends GenericCDIView {
 
 	public void initBillView()
 	{
-		if (requirePermissions(ShopPermission.LIST_USERPRODUCTS)) {
+		if (requirePermissions(ShopPermission.LIST_USERPRODUCTS) && shoppingcart == null) {
 			shoppingcart = new ListDataModel<ProductShopItem>(ProductShopItem.productList(productBean.listUserShoppableProducts()));
 			logger.debug("Initialized billing shoppingcart to {}", shoppingcart);
 			this.beginConversation();
@@ -62,7 +62,7 @@ public class ProductShopView extends GenericCDIView {
 
 	public void initShopView() {
 
-		if (requirePermissions(ShopPermission.SHOP_TO_OTHERS)) {
+		if (requirePermissions(ShopPermission.SHOP_TO_OTHERS) && shoppingcart == null) {
 			shoppingcart = new ListDataModel<ProductShopItem>(ProductShopItem.productList(productBean.listUserShoppableProducts()));
 			logger.debug("Initialized shoppingcart to {}", shoppingcart);
 			this.beginConversation();

code/LanBortalWeb/src/fi/insomnia/bortal/web/cdiview/user/OrgRoleDataView.java

+package fi.insomnia.bortal.web.cdiview.user;
+
+import javax.ejb.EJB;
+import javax.enterprise.context.RequestScoped;
+import javax.faces.model.ListDataModel;
+import javax.inject.Named;
+
+import fi.insomnia.bortal.beans.OrgRoleBeanLocal;
+import fi.insomnia.bortal.model.OrgRole;
+import fi.insomnia.bortal.web.cdiview.GenericCDIView;
+
+@Named
+@RequestScoped
+public class OrgRoleDataView extends GenericCDIView {
+
+	private static final long serialVersionUID = -607046631885684171L;
+
+	private transient ListDataModel<OrgRole> orgRoles;
+	
+	@EJB
+	OrgRoleBeanLocal orgRolesBean;
+
+	public ListDataModel<OrgRole> getOrgRoles() {
+
+		if (orgRoles == null) {
+			orgRoles = new ListDataModel<OrgRole>(orgRolesBean.listOrgRoles());
+		}
+
+		return orgRoles;
+	}
+
+}

code/LanBortalWeb/src/fi/insomnia/bortal/web/cdiview/user/UserView.java

@@ -88,7 +88,7 @@ public class UserView extends GenericCDIView {
 	public EventUser getSelectedUser() {
 		if (user == null) {
 			if (userid != null && permbean.hasPermission(UserPermission.VIEW_ALL)) {
-				user = userbean.findEventUserById(userid);
+				user = userbean.findByUserId(userid);
 			} else {
 				user = getCurrentUser();
 			}

code/LanBortalWeb/src/fi/insomnia/bortal/web/converter/EventUserConverter.java

@@ -17,7 +17,7 @@ public class EventUserConverter extends GenericIntegerEntityConverter<EventUser>
 
 	@Override
 	protected EventUser find(Integer id) {
-		return userbean.findEventUserById(id);
+		return userbean.findByEventUserId(id);
 	}
 
 }

code/LanBortalWeb/src/fi/insomnia/bortal/web/converter/UserConverter.java

-package fi.insomnia.bortal.web.converter;
-
-import javax.ejb.EJB;
-import javax.faces.bean.RequestScoped;
-import javax.inject.Named;
-
-import fi.insomnia.bortal.beans.UserBeanLocal;
-import fi.insomnia.bortal.model.User;
-import fi.insomnia.bortal.utilities.jsf.GenericIntegerEntityConverter;
-
-@Named
-@RequestScoped
-public class UserConverter extends GenericIntegerEntityConverter<User> {
-
-	@EJB
-	private UserBeanLocal userbean;
-
-	@Override
-	protected User find(Integer id) {
-		return userbean.findById(id);
-	}
-
-}