diff --git a/rm-server/config/alfresco/module/org_alfresco_module_rm/rm-service-context.xml b/rm-server/config/alfresco/module/org_alfresco_module_rm/rm-service-context.xml
index 504e276706..d6a0c1e57a 100644
--- a/rm-server/config/alfresco/module/org_alfresco_module_rm/rm-service-context.xml
+++ b/rm-server/config/alfresco/module/org_alfresco_module_rm/rm-service-context.xml
@@ -175,7 +175,6 @@
-
@@ -554,6 +553,7 @@
+
@@ -595,7 +595,7 @@
-
-
@@ -1197,9 +1197,9 @@
-
-
+
@@ -1221,7 +1221,6 @@
-
@@ -1276,7 +1275,6 @@
-
@@ -1538,4 +1536,4 @@
-
\ No newline at end of file
+
diff --git a/rm-server/config/alfresco/module/org_alfresco_module_rm/rm-webscript-context.xml b/rm-server/config/alfresco/module/org_alfresco_module_rm/rm-webscript-context.xml
index f605d8d10d..95e771046d 100644
--- a/rm-server/config/alfresco/module/org_alfresco_module_rm/rm-webscript-context.xml
+++ b/rm-server/config/alfresco/module/org_alfresco_module_rm/rm-webscript-context.xml
@@ -3,6 +3,14 @@
+
+
+
+
+
+
+
+
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
@@ -523,11 +512,22 @@
-
+ parent="webscript">
+
+
+
+
+
+
+
+
+
+
+
+
-
+
\ No newline at end of file
diff --git a/rm-server/source/java/org/alfresco/module/org_alfresco_module_rm/event/RecordsManagementEventService.java b/rm-server/source/java/org/alfresco/module/org_alfresco_module_rm/event/RecordsManagementEventService.java
index a32ff9f879..85e138c357 100644
--- a/rm-server/source/java/org/alfresco/module/org_alfresco_module_rm/event/RecordsManagementEventService.java
+++ b/rm-server/source/java/org/alfresco/module/org_alfresco_module_rm/event/RecordsManagementEventService.java
@@ -22,79 +22,90 @@ import java.util.List;
/**
* Records management event service interface
- *
+ *
* @author Roy Wetherall
*/
public interface RecordsManagementEventService
{
/**
* Register an event type
- *
+ *
* @param eventType event type
*/
void registerEventType(RecordsManagementEventType eventType);
-
+
/**
* Get a list of the event types
- *
+ *
* @return List list of the event types
*/
List getEventTypes();
-
+
/**
* Get the records management event type
- *
+ *
* @param eventType name
- * @return RecordsManagementEventType event type
+ * @return RecordsManagementEventType event type
*/
RecordsManagementEventType getEventType(String eventTypeName);
-
+
/**
* Get the list of available events
- *
+ *
* @return List list of events
*/
List getEvents();
-
+
/**
* Get a records management event given its name. Returns null if the event name is not
* recognised.
- *
+ *
* @param eventName event name
* @return RecordsManagementEvent event
*/
RecordsManagementEvent getEvent(String eventName);
-
+
/**
- * Indicates whether a perticular event exists. Returns true if it does, false otherwise.
- *
+ * Indicates whether a particular event exists. Returns true if it does, false otherwise.
+ *
* @param eventName event name
* @return boolean true if event exists, false otherwise
*/
boolean existsEvent(String eventName);
-
+
/**
- * Indicates whether a particular event display label exists. Returns true if it does, false otherwise.
- *
+ * Indicates whether a particular event can be created. Returns true if it is possible, false otherwise.
+ *
* @param eventDisplayLabel event display label
- * @return true if event display label exists, false otherwise
+ * @param eventName event name
+ * @return true if the event can be created, false otherwise
*/
- boolean existsEventDisplayLabel(String eventDisplayLabel);
-
+ boolean canCreateEvent(String eventDisplayLabel, String eventName);
+
+ /**
+ * Indicates whether a particular event can be edited. Returns true if it is possible, false otherwise.
+ *
+ * @param eventDisplayLabel event display label
+ * @param eventName event name
+ * @param eventType event type
+ * @return true if the event can be edited, false otherwise
+ */
+ boolean canEditEvent(String eventDisplayLabel, String eventName, String eventType);
+
/**
* Add an event
- *
+ *
* @param eventType event type
* @param eventName event name
* @param eventDisplayLabel event display label
*/
RecordsManagementEvent addEvent(String eventType, String eventName, String eventDisplayLabel);
-
+
/**
* Remove an event
- *
+ *
* @param eventName event name
*/
- void removeEvent(String eventName);
+ void removeEvent(String eventName);
}
diff --git a/rm-server/source/java/org/alfresco/module/org_alfresco_module_rm/event/RecordsManagementEventServiceImpl.java b/rm-server/source/java/org/alfresco/module/org_alfresco_module_rm/event/RecordsManagementEventServiceImpl.java
index e503cfbd34..69bc6a9b6d 100644
--- a/rm-server/source/java/org/alfresco/module/org_alfresco_module_rm/event/RecordsManagementEventServiceImpl.java
+++ b/rm-server/source/java/org/alfresco/module/org_alfresco_module_rm/event/RecordsManagementEventServiceImpl.java
@@ -20,6 +20,7 @@ package org.alfresco.module.org_alfresco_module_rm.event;
import java.util.ArrayList;
import java.util.HashMap;
+import java.util.Iterator;
import java.util.List;
import java.util.Map;
@@ -32,12 +33,13 @@ import org.alfresco.service.cmr.repository.ContentWriter;
import org.alfresco.service.cmr.repository.NodeRef;
import org.alfresco.service.cmr.repository.NodeService;
import org.alfresco.service.cmr.repository.StoreRef;
+import org.alfresco.util.ParameterCheck;
import org.json.JSONArray;
import org.json.JSONObject;
/**
* Records management event service implementation
- *
+ *
* @author Roy Wetherall
*/
public class RecordsManagementEventServiceImpl implements RecordsManagementEventService
@@ -61,7 +63,7 @@ public class RecordsManagementEventServiceImpl implements RecordsManagementEvent
/**
* Set the node service
- *
+ *
* @param nodeService node service
*/
public void setNodeService(NodeService nodeService)
@@ -71,7 +73,7 @@ public class RecordsManagementEventServiceImpl implements RecordsManagementEvent
/**
* Set the content service
- *
+ *
* @param contentService content service
*/
public void setContentService(ContentService contentService)
@@ -132,17 +134,99 @@ public class RecordsManagementEventServiceImpl implements RecordsManagementEvent
}
return false;
}
+
+ /**
+ * @see org.alfresco.module.org_alfresco_module_rm.event.RecordsManagementEventService#canCreateEvent(java.lang.String, java.lang.String)
+ */
+ @SuppressWarnings("unchecked")
+ public boolean canCreateEvent(String eventDisplayLabel, String eventName)
+ {
+ ParameterCheck.mandatoryString("eventDisplayLabel", eventDisplayLabel);
+ ParameterCheck.mandatoryString("eventName", eventName);
+
+ boolean canCreateEvent = true;
+
+ if (existsEvent(eventName) == true)
+ {
+ canCreateEvent = false;
+ }
+
+ if (canCreateEvent == true)
+ {
+ for (Iterator iterator = getEventMap().values().iterator(); iterator.hasNext();)
+ {
+ RecordsManagementEvent recordsManagementEvent = (RecordsManagementEvent) iterator.next();
+ if (recordsManagementEvent.getDisplayLabel().equalsIgnoreCase(eventDisplayLabel) == true)
+ {
+ canCreateEvent = false;
+ break;
+ }
+ }
+ }
+
+ return canCreateEvent;
+ }
/**
- * @see org.alfresco.module.org_alfresco_module_rm.event.RecordsManagementEventService#addEvent(java.lang.String,
- * java.lang.String, java.lang.String)
+ * @see org.alfresco.module.org_alfresco_module_rm.event.RecordsManagementEventService#canEditEvent(java.lang.String, java.lang.String, java.lang.String)
+ */
+ @SuppressWarnings("unchecked")
+ public boolean canEditEvent(String eventDisplayLabel, String eventName, String eventType)
+ {
+ ParameterCheck.mandatoryString("eventDisplayLabel", eventDisplayLabel);
+ ParameterCheck.mandatoryString("eventName", eventName);
+ ParameterCheck.mandatoryString("eventType", eventType);
+
+ boolean canEditEvent = true;
+
+ if (existsEvent(eventName) == false)
+ {
+ throw new AlfrescoRuntimeException("The event '" + eventName + "' does not exist.");
+ }
+
+ for (Iterator iterator = getEventMap().values().iterator(); iterator.hasNext();)
+ {
+ RecordsManagementEvent recordsManagementEvent = (RecordsManagementEvent) iterator.next();
+ if (recordsManagementEvent.getDisplayLabel().equalsIgnoreCase(eventDisplayLabel) == true)
+ {
+ if (recordsManagementEvent.getName().equalsIgnoreCase(eventName) == true)
+ {
+ if (recordsManagementEvent.getType().equalsIgnoreCase(eventType) == false)
+ {
+ canEditEvent = true;
+ }
+ else
+ {
+ canEditEvent = false;
+ }
+ }
+ else
+ {
+ throw new AlfrescoRuntimeException("Cannot edit event. An event with the display label '"
+ + eventDisplayLabel + "' already exist.");
+ }
+ break;
+ }
+ }
+
+ return canEditEvent;
+ }
+
+ /**
+ * @see org.alfresco.module.org_alfresco_module_rm.event.RecordsManagementEventService#addEvent(java.lang.String, java.lang.String, java.lang.String)
*/
public RecordsManagementEvent addEvent(String eventType, String eventName, String eventDisplayLabel)
{
// Check that the eventType is valid
- if (eventTypes.containsKey(eventType) == false) { throw new AlfrescoRuntimeException(
- "Can not add event because event " + eventName + " has an undefined eventType. (" + eventType + ")"); }
-
+ if (eventTypes.containsKey(eventType) == false)
+ {
+ throw new AlfrescoRuntimeException(
+ "Can not add event because event " +
+ eventName +
+ " has an undefined eventType. ("
+ + eventType + ")");
+ }
+
// Create event and add to map
RecordsManagementEvent event = new RecordsManagementEvent(eventType, eventName, eventDisplayLabel);
getEventMap().put(event.getName(), event);
@@ -168,7 +252,7 @@ public class RecordsManagementEventServiceImpl implements RecordsManagementEvent
/**
* Helper method to get the event map. Loads initial instance from persisted
* configuration file.
- *
+ *
* @return Map map of available events by
* event name
*/
@@ -191,8 +275,10 @@ public class RecordsManagementEventServiceImpl implements RecordsManagementEvent
public Object doWork() throws Exception
{
// Get the event config node
- if (nodeService.exists(CONFIG_NODE_REF) == false) { throw new AlfrescoRuntimeException(
- "Unable to find records management event configuration node."); }
+ if (nodeService.exists(CONFIG_NODE_REF) == false)
+ {
+ throw new AlfrescoRuntimeException("Unable to find records management event configuration node.");
+ }
// Read content from config node
ContentReader reader = contentService.getReader(CONFIG_NODE_REF, ContentModel.PROP_CONTENT);
@@ -214,9 +300,14 @@ public class RecordsManagementEventServiceImpl implements RecordsManagementEvent
String eventDisplayLabel = eventJSON.getString("eventDisplayLabel");
// Check that the eventType is valid
- if (eventTypes.containsKey(eventType) == false) { throw new AlfrescoRuntimeException(
- "Can not load rm event configuration because event " + eventName
- + " has an undefined eventType. (" + eventType + ")"); }
+ if (eventTypes.containsKey(eventType) == false)
+ {
+ throw new AlfrescoRuntimeException(
+ "Can not load rm event configuration because event " +
+ eventName +
+ " has an undefined eventType. ("
+ + eventType + ")");
+ }
// Create event and add to map
RecordsManagementEvent event = new RecordsManagementEvent(eventType, eventName, eventDisplayLabel);
@@ -238,8 +329,10 @@ public class RecordsManagementEventServiceImpl implements RecordsManagementEvent
public Object doWork() throws Exception
{
// Get the event config node
- if (nodeService.exists(CONFIG_NODE_REF) == false) { throw new AlfrescoRuntimeException(
- "Unable to find records management event configuration node."); }
+ if (nodeService.exists(CONFIG_NODE_REF) == false)
+ {
+ throw new AlfrescoRuntimeException("Unable to find records management event configuration node.");
+ }
JSONObject configJSON = new JSONObject();
JSONArray eventsJSON = new JSONArray();
diff --git a/rm-server/source/java/org/alfresco/module/org_alfresco_module_rm/script/admin/RMEventBase.java b/rm-server/source/java/org/alfresco/module/org_alfresco_module_rm/script/admin/RMEventBase.java
new file mode 100644
index 0000000000..7483eb7ef9
--- /dev/null
+++ b/rm-server/source/java/org/alfresco/module/org_alfresco_module_rm/script/admin/RMEventBase.java
@@ -0,0 +1,67 @@
+/*
+ * Copyright (C) 2005-2013 Alfresco Software Limited.
+ *
+ * This file is part of Alfresco
+ *
+ * Alfresco is free software: you can redistribute it and/or modify
+ * it under the terms of the GNU Lesser General Public License as published by
+ * the Free Software Foundation, either version 3 of the License, or
+ * (at your option) any later version.
+ *
+ * Alfresco is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+ * GNU Lesser General Public License for more details.
+ *
+ * You should have received a copy of the GNU Lesser General Public License
+ * along with Alfresco. If not, see .
+ */
+package org.alfresco.module.org_alfresco_module_rm.script.admin;
+
+import org.apache.commons.lang.StringUtils;
+import org.json.JSONException;
+import org.json.JSONObject;
+import org.springframework.extensions.webscripts.DeclarativeWebScript;
+import org.springframework.extensions.webscripts.Status;
+import org.springframework.extensions.webscripts.WebScriptException;
+
+/**
+ * Base class for Records management event web scripts
+ *
+ * @author Tuna Aksoy
+ * @since 2.1
+ */
+public class RMEventBase extends DeclarativeWebScript
+{
+ /**
+ * Helper method for getting the value for a key from a JSON object
+ *
+ * @param json The request content as JSON object
+ * @param key The key for which the value should be retrieved (e.g. "eventName")
+ * @return String The value for the provided key if the key exists, null otherwise
+ * @throws JSONException If there is no string value for the key
+ */
+ protected String getValue(JSONObject json, String key) throws JSONException
+ {
+ String result = null;
+ if (json.has(key) == true)
+ {
+ result = json.getString(key);
+ }
+ return result;
+ }
+
+ /**
+ * Helper method for checking the key (e.g. "eventName")
+ *
+ * @param key String The key which will be checked
+ * @param msg String The error message to throw if the key doesn't have a value
+ */
+ protected void doCheck(String key, String msg)
+ {
+ if (StringUtils.isBlank(key))
+ {
+ throw new WebScriptException(Status.STATUS_BAD_REQUEST, msg);
+ }
+ }
+}
diff --git a/rm-server/source/java/org/alfresco/module/org_alfresco_module_rm/script/admin/RmEventPut.java b/rm-server/source/java/org/alfresco/module/org_alfresco_module_rm/script/admin/RmEventPut.java
index 1af46bd6e6..b45daaf7f1 100644
--- a/rm-server/source/java/org/alfresco/module/org_alfresco_module_rm/script/admin/RmEventPut.java
+++ b/rm-server/source/java/org/alfresco/module/org_alfresco_module_rm/script/admin/RmEventPut.java
@@ -22,35 +22,31 @@ import java.io.IOException;
import java.util.HashMap;
import java.util.Map;
+import org.alfresco.error.AlfrescoRuntimeException;
import org.alfresco.module.org_alfresco_module_rm.event.RecordsManagementEvent;
import org.alfresco.module.org_alfresco_module_rm.event.RecordsManagementEventService;
-import org.springframework.extensions.webscripts.Cache;
-import org.springframework.extensions.webscripts.DeclarativeWebScript;
-import org.springframework.extensions.webscripts.Status;
-import org.springframework.extensions.webscripts.WebScriptException;
-import org.springframework.extensions.webscripts.WebScriptRequest;
-import org.apache.commons.logging.Log;
-import org.apache.commons.logging.LogFactory;
+import org.alfresco.util.ParameterCheck;
import org.json.JSONException;
import org.json.JSONObject;
import org.json.JSONTokener;
+import org.springframework.extensions.webscripts.Cache;
+import org.springframework.extensions.webscripts.Status;
+import org.springframework.extensions.webscripts.WebScriptException;
+import org.springframework.extensions.webscripts.WebScriptRequest;
/**
* Records management event PUT web script
- *
+ *
* @author Roy Wetherall
*/
-public class RmEventPut extends DeclarativeWebScript
+public class RmEventPut extends RMEventBase
{
- @SuppressWarnings("unused")
- private static Log logger = LogFactory.getLog(RmEventPut.class);
-
- /** Reccords management event service */
+ /** Records management event service */
private RecordsManagementEventService rmEventService;
-
+
/**
* Set the records management event service
- *
+ *
* @param rmEventService
*/
public void setRecordsManagementEventService(RecordsManagementEventService rmEventService)
@@ -58,59 +54,49 @@ public class RmEventPut extends DeclarativeWebScript
this.rmEventService = rmEventService;
}
+ /**
+ * @see org.springframework.extensions.webscripts.DeclarativeWebScript#executeImpl(org.springframework.extensions.webscripts.WebScriptRequest,
+ * org.springframework.extensions.webscripts.Status,
+ * org.springframework.extensions.webscripts.Cache)
+ */
@Override
public Map executeImpl(WebScriptRequest req, Status status, Cache cache)
{
- Map model = new HashMap();
+ ParameterCheck.mandatory("req", req);
+
+ Map model = new HashMap();
JSONObject json = null;
try
{
- // Event name
- Map templateVars = req.getServiceMatch().getTemplateVars();
- String eventName = templateVars.get("eventname");
- if (eventName == null)
- {
- throw new WebScriptException(Status.STATUS_NOT_FOUND, "No event name was provided on the URL.");
- }
-
- // Check the event exists
- if (rmEventService.existsEvent(eventName) == false)
- {
- throw new WebScriptException(Status.STATUS_NOT_FOUND, "The event " + eventName + " does not exist.");
- }
-
+ // Convert the request content to JSON
json = new JSONObject(new JSONTokener(req.getContent().getContent()));
-
- String eventDisplayLabel = null;
- if (json.has("eventDisplayLabel") == true)
+
+ // Check the event name
+ String eventName = getValue(json, "eventName");
+ doCheck(eventName, "No event name was provided.");
+
+ // Check the event display label
+ String eventDisplayLabel = getValue(json, "eventDisplayLabel");
+ doCheck(eventDisplayLabel, "No event display label was provided.");
+
+ // Check the event type
+ String eventType = getValue(json, "eventType");
+ doCheck(eventType, "No event type was provided.");
+
+ // Check if the event can be edited or not
+ RecordsManagementEvent event = null;
+ if (canEditEvent(eventDisplayLabel, eventName, eventType) == true)
{
- eventDisplayLabel = json.getString("eventDisplayLabel");
+ // Create event
+ event = rmEventService.addEvent(eventType, eventName, eventDisplayLabel);
}
- if (eventDisplayLabel == null || eventDisplayLabel.length() == 0)
+ else
{
- throw new WebScriptException(Status.STATUS_BAD_REQUEST, "No event display label provided.");
+ // Get event
+ event = rmEventService.getEvent(eventName);
}
- if (rmEventService.existsEventDisplayLabel(eventDisplayLabel))
- {
- throw new WebScriptException(Status.STATUS_BAD_REQUEST,
- "Cannot edit event. The event display label '"
- + eventDisplayLabel + "' already exists.");
- }
-
- String eventType = null;
- if (json.has("eventType") == true)
- {
- eventType = json.getString("eventType");
- }
- if (eventType == null || eventType.length() == 0)
- {
- throw new WebScriptException(Status.STATUS_BAD_REQUEST, "No event type provided.");
- }
-
-
- RecordsManagementEvent event = rmEventService.addEvent(eventType, eventName, eventDisplayLabel);
+
model.put("event", event);
-
}
catch (IOException iox)
{
@@ -122,7 +108,32 @@ public class RmEventPut extends DeclarativeWebScript
throw new WebScriptException(Status.STATUS_BAD_REQUEST,
"Could not parse JSON from req.", je);
}
-
+
return model;
}
+
+ /**
+ * Helper method for checking if an event can be edited or not. Throws an
+ * error if an event with the same display label already exists.
+ *
+ * @param eventDisplayLabel The display label of the event
+ * @param eventName The name of the event
+ * @param eventType The type of the event
+ * @return true if the event can be edited, false otherwise
+ */
+ private boolean canEditEvent(String eventDisplayLabel, String eventName, String eventType)
+ {
+ boolean canEditEvent;
+
+ try
+ {
+ canEditEvent = rmEventService.canEditEvent(eventDisplayLabel, eventName, eventType);
+ }
+ catch (AlfrescoRuntimeException are)
+ {
+ throw new WebScriptException(Status.STATUS_BAD_REQUEST, are.getLocalizedMessage());
+ }
+
+ return canEditEvent;
+ }
}
\ No newline at end of file
diff --git a/rm-server/source/java/org/alfresco/module/org_alfresco_module_rm/script/admin/RmEventsPost.java b/rm-server/source/java/org/alfresco/module/org_alfresco_module_rm/script/admin/RmEventsPost.java
index 5384764881..4d5f7b2f0b 100644
--- a/rm-server/source/java/org/alfresco/module/org_alfresco_module_rm/script/admin/RmEventsPost.java
+++ b/rm-server/source/java/org/alfresco/module/org_alfresco_module_rm/script/admin/RmEventsPost.java
@@ -25,33 +25,29 @@ import java.util.Map;
import org.alfresco.module.org_alfresco_module_rm.event.RecordsManagementEvent;
import org.alfresco.module.org_alfresco_module_rm.event.RecordsManagementEventService;
import org.alfresco.util.GUID;
-import org.springframework.extensions.webscripts.Cache;
-import org.springframework.extensions.webscripts.DeclarativeWebScript;
-import org.springframework.extensions.webscripts.Status;
-import org.springframework.extensions.webscripts.WebScriptException;
-import org.springframework.extensions.webscripts.WebScriptRequest;
-import org.apache.commons.logging.Log;
-import org.apache.commons.logging.LogFactory;
+import org.alfresco.util.ParameterCheck;
+import org.apache.commons.lang.StringUtils;
import org.json.JSONException;
import org.json.JSONObject;
import org.json.JSONTokener;
+import org.springframework.extensions.webscripts.Cache;
+import org.springframework.extensions.webscripts.Status;
+import org.springframework.extensions.webscripts.WebScriptException;
+import org.springframework.extensions.webscripts.WebScriptRequest;
/**
- *
- *
+ * Records management event POST web script
+ *
* @author Roy Wetherall
*/
-public class RmEventsPost extends DeclarativeWebScript
+public class RmEventsPost extends RMEventBase
{
- @SuppressWarnings("unused")
- private static Log logger = LogFactory.getLog(RmEventsPost.class);
-
- /** Reccords management event service */
+ /** Records management event service */
private RecordsManagementEventService rmEventService;
-
+
/**
* Set the records management event service
- *
+ *
* @param rmEventService
*/
public void setRecordsManagementEventService(RecordsManagementEventService rmEventService)
@@ -59,57 +55,41 @@ public class RmEventsPost extends DeclarativeWebScript
this.rmEventService = rmEventService;
}
-
+ /**
+ * @see org.springframework.extensions.webscripts.DeclarativeWebScript#executeImpl(org.springframework.extensions.webscripts.WebScriptRequest,
+ * org.springframework.extensions.webscripts.Status,
+ * org.springframework.extensions.webscripts.Cache)
+ */
@Override
public Map executeImpl(WebScriptRequest req, Status status, Cache cache)
{
- Map model = new HashMap();
+ ParameterCheck.mandatory("req", req);
+
+ Map model = new HashMap();
JSONObject json = null;
try
{
+ // Convert the request content to JSON
json = new JSONObject(new JSONTokener(req.getContent().getContent()));
-
- String eventName = null;
- if (json.has("eventName") == true)
- {
- eventName = json.getString("eventName");
- }
-
- if (eventName == null || eventName.length() == 0)
- {
- // Generate the event name
- eventName = GUID.generate();
- }
-
- String eventDisplayLabel = null;
- if (json.has("eventDisplayLabel") == true)
- {
- eventDisplayLabel = json.getString("eventDisplayLabel");
- }
- if (eventDisplayLabel == null || eventDisplayLabel.length() == 0)
- {
- throw new WebScriptException(Status.STATUS_BAD_REQUEST, "No event display label provided.");
- }
- if (rmEventService.existsEventDisplayLabel(eventDisplayLabel))
- {
- throw new WebScriptException(Status.STATUS_BAD_REQUEST,
- "Cannot create event. The event display label '"
- + eventDisplayLabel + "' already exists.");
- }
-
- String eventType = null;
- if (json.has("eventType") == true)
- {
- eventType = json.getString("eventType");
- }
- if (eventType == null || eventType.length() == 0)
- {
- throw new WebScriptException(Status.STATUS_BAD_REQUEST, "No event type provided.");
- }
-
+
+ // Get the event name
+ String eventName = getEventName(json);
+
+ // Check the event display label
+ String eventDisplayLabel = getValue(json, "eventDisplayLabel");
+ doCheck(eventDisplayLabel, "No event display label was provided.");
+
+ // Check if the event can be created
+ canCreateEvent(eventDisplayLabel, eventName);
+
+ // Check the event type
+ String eventType = getValue(json, "eventType");
+ doCheck(eventType, "No event type was provided.");
+
+ // Create event
RecordsManagementEvent event = rmEventService.addEvent(eventType, eventName, eventDisplayLabel);
+
model.put("event", event);
-
}
catch (IOException iox)
{
@@ -121,7 +101,48 @@ public class RmEventsPost extends DeclarativeWebScript
throw new WebScriptException(Status.STATUS_BAD_REQUEST,
"Could not parse JSON from req.", je);
}
-
+
return model;
}
+
+ /**
+ * Helper method for getting the event name
+ *
+ * @param json The request content as JSON object
+ * @return String The event name. A generated GUID if it doesn't exist
+ * @throws JSONException If there is no string value for the key
+ */
+ private String getEventName(JSONObject json) throws JSONException
+ {
+ String eventName = getValue(json, "eventName");
+
+ if (StringUtils.isBlank(eventName))
+ {
+ // Generate the event name
+ eventName = GUID.generate();
+ }
+
+ return eventName;
+ }
+
+ /**
+ * Helper method for checking if an event can be created or not. Throws an
+ * error if the event already exists.
+ *
+ * @param eventDisplayLabel The display label of the event
+ * @param eventName The name of the event
+ */
+ private boolean canCreateEvent(String eventDisplayLabel, String eventName)
+ {
+ boolean canCreateEvent = true;
+
+ if (rmEventService.canCreateEvent(eventDisplayLabel, eventName) == false)
+ {
+ throw new WebScriptException(Status.STATUS_BAD_REQUEST,
+ "Cannot create event. An event with the display label '"
+ + eventDisplayLabel + "' already exists.");
+ }
+
+ return canCreateEvent;
+ }
}
\ No newline at end of file