Index: lams_central/src/java/org/lamsfoundation/lams/authoring/service/AuthoringService.java
===================================================================
diff -u -r79c858a92c17ab3ca404cab4a0cf6094254e68d5 -rfca7f5f4bc72061fa82b651377d299a76ed9d090
--- lams_central/src/java/org/lamsfoundation/lams/authoring/service/AuthoringService.java (.../AuthoringService.java) (revision 79c858a92c17ab3ca404cab4a0cf6094254e68d5)
+++ lams_central/src/java/org/lamsfoundation/lams/authoring/service/AuthoringService.java (.../AuthoringService.java) (revision fca7f5f4bc72061fa82b651377d299a76ed9d090)
@@ -234,17 +234,6 @@
}
return flashMessage.serializeMessage();
}
- /**
- * Checks whether the WDDX packet contains any invalid
- * "". It returns true if there exists any such null
- */
- private boolean containsNulls(String packet)
- {
- if (packet.indexOf("") != -1)
- return true;
- else
- return false;
- }
public String copyLearningDesign(Long originalDesignID,Integer copyType,
Integer userID, Integer workspaceFolderID)throws UserException, LearningDesignException,
WorkspaceFolderException, IOException{
@@ -420,12 +409,6 @@
public String storeLearningDesignDetails(String wddxPacket) throws Exception{
LearningDesignDTO learningDesignDTO = null;
- if(containsNulls(wddxPacket)){
- flashMessage = new FlashMessage("storeLearningDesignDetails",
- "WDDXPacket contains null",
- FlashMessage.ERROR);
- return flashMessage.serializeMessage();
- }
Hashtable table = (Hashtable)WDDXProcessor.deserialize(wddxPacket);
ObjectExtractor extractor = new ObjectExtractor(userDAO,learningDesignDAO,
activityDAO,workspaceFolderDAO,
@@ -514,12 +497,6 @@
*/
public String storeTheme(String wddxPacket) throws Exception {
- if(containsNulls(wddxPacket)){
- flashMessage = new FlashMessage("storeTheme",
- "WDDXPacket contains null",
- FlashMessage.ERROR);
- return flashMessage.serializeMessage();
- }
Hashtable table = (Hashtable)WDDXProcessor.deserialize(wddxPacket);
CSSThemeDTO themeDTO = new CSSThemeDTO(table);
Index: lams_common/src/java/org/lamsfoundation/lams/web/servlet/AbstractStoreWDDXPacketServlet.java
===================================================================
diff -u -r21a2b87e69eac5b2bf9b38863e4637e25d35b773 -rfca7f5f4bc72061fa82b651377d299a76ed9d090
--- lams_common/src/java/org/lamsfoundation/lams/web/servlet/AbstractStoreWDDXPacketServlet.java (.../AbstractStoreWDDXPacketServlet.java) (revision 21a2b87e69eac5b2bf9b38863e4637e25d35b773)
+++ lams_common/src/java/org/lamsfoundation/lams/web/servlet/AbstractStoreWDDXPacketServlet.java (.../AbstractStoreWDDXPacketServlet.java) (revision fca7f5f4bc72061fa82b651377d299a76ed9d090)
@@ -91,6 +91,13 @@
log.debug("Request "+request.getRequestURI()+" received packet "+packet);
}
+ if(containsNulls(packet)){
+ FlashMessage flashMessage = new FlashMessage(getMessageKey(packet,request),
+ "WDDXPacket contains null",
+ FlashMessage.ERROR);
+ writer.write(flashMessage.serializeMessage());
+ }
+
replyPacket = process(packet, request);
if ( log.isDebugEnabled() ) {
@@ -103,7 +110,7 @@
log.error(uri+" request triggered exception ",e);
FlashMessage flashMessage = FlashMessage.getExceptionOccured(getMessageKey(packet,request),
- e.getMessage()!=null?e.getMessage():"");
+ e.getMessage()!=null?e.getMessage():e.getClass().getName());
writer.write(flashMessage.serializeMessage());
return;
}
@@ -115,6 +122,18 @@
/**
+ * Checks whether the WDDX packet contains any invalid
+ * "". It returns true if there exists any such null
+ */
+ private boolean containsNulls(String packet)
+ {
+ if (packet.indexOf("") != -1)
+ return true;
+ else
+ return false;
+ }
+
+ /**
* Method to get the post body.
* Retrieves the body of the post as binary data, and then converts back to a character stream.
* The reason why we didn't use getReader() which retrieves the body of the post as character