Skip to content
Snippets Groups Projects
Commit 06cc7805 authored by Tor-Einar Skog's avatar Tor-Einar Skog
Browse files

Bugfix: Now Organization Admin can edit and create POIs again

parent c4371205
No related branches found
No related tags found
1 merge request!17Develop
...@@ -544,6 +544,13 @@ public class PointOfInterestController extends HttpServlet { ...@@ -544,6 +544,13 @@ public class PointOfInterestController extends HttpServlet {
{ {
request.getSession().setAttribute("users", em.createNamedQuery("VipsLogicUser.findAll", VipsLogicUser.class).getResultList()); request.getSession().setAttribute("users", em.createNamedQuery("VipsLogicUser.findAll", VipsLogicUser.class).getResultList());
} }
else if(user.isOrganizationAdmin())
{
request.getSession().setAttribute("users",
em.createNamedQuery("VipsLogicUser.findByOrganizationId", VipsLogicUser.class)
.setParameter("organizationId", user.getOrganizationId()).getResultList()
);
}
Integer organizationId = Integer.valueOf(request.getParameter("organizationId")); Integer organizationId = Integer.valueOf(request.getParameter("organizationId"));
request.setAttribute("returnURL","poi?organizationId=" + organizationId); request.setAttribute("returnURL","poi?organizationId=" + organizationId);
request.getRequestDispatcher("/poiForm.ftl").forward(request, response); request.getRequestDispatcher("/poiForm.ftl").forward(request, response);
...@@ -584,6 +591,13 @@ public class PointOfInterestController extends HttpServlet { ...@@ -584,6 +591,13 @@ public class PointOfInterestController extends HttpServlet {
{ {
request.getSession().setAttribute("users", em.createNamedQuery("VipsLogicUser.findAll", VipsLogicUser.class).getResultList()); request.getSession().setAttribute("users", em.createNamedQuery("VipsLogicUser.findAll", VipsLogicUser.class).getResultList());
} }
else if(user.isOrganizationAdmin())
{
request.getSession().setAttribute("users",
em.createNamedQuery("VipsLogicUser.findByOrganizationId", VipsLogicUser.class)
.setParameter("organizationId", user.getOrganizationId()).getResultList()
);
}
request.setAttribute("returnURL","poi?organizationId=" + poi.getUserId().getOrganizationId().getOrganizationId()); request.setAttribute("returnURL","poi?organizationId=" + poi.getUserId().getOrganizationId().getOrganizationId());
request.getRequestDispatcher("/poiForm.ftl").forward(request, response); request.getRequestDispatcher("/poiForm.ftl").forward(request, response);
} }
...@@ -656,7 +670,7 @@ public class PointOfInterestController extends HttpServlet { ...@@ -656,7 +670,7 @@ public class PointOfInterestController extends HttpServlet {
// If userId is set from form, always update // If userId is set from form, always update
if(user.isSuperUser() && !formValidation.getFormField("userId").isEmpty()) if((user.isSuperUser() || user.isOrganizationAdmin()) && !formValidation.getFormField("userId").isEmpty())
{ {
poi.setUserId(em.find(VipsLogicUser.class, formValidation.getFormField("userId").getValueAsInteger())); poi.setUserId(em.find(VipsLogicUser.class, formValidation.getFormField("userId").getValueAsInteger()));
} }
......
...@@ -176,7 +176,7 @@ ...@@ -176,7 +176,7 @@
</fieldset> </fieldset>
</#if> </#if>
<#if user.isSuperUser()> <#if user.isSuperUser() || user.isOrganizationAdmin()>
<div class="form-group"> <div class="form-group">
<label for="userId">${i18nBundle.vipsLogicUserId}</label> <label for="userId">${i18nBundle.vipsLogicUserId}</label>
<select class="form-control" name="userId" onblur="validateField(this);"> <select class="form-control" name="userId" onblur="validateField(this);">
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment