Index: lams_tool_forum/src/java/org/lamsfoundation/lams/tool/forum/web/actions/LearningAction.java
===================================================================
diff -u -r2b40643d409f3d7066e3e9505bc02994a4139542 -r0e9302b16c5c660442417c396fa5cf25f8a81a4f
--- lams_tool_forum/src/java/org/lamsfoundation/lams/tool/forum/web/actions/LearningAction.java	(.../LearningAction.java)	(revision 2b40643d409f3d7066e3e9505bc02994a4139542)
+++ lams_tool_forum/src/java/org/lamsfoundation/lams/tool/forum/web/actions/LearningAction.java	(.../LearningAction.java)	(revision 0e9302b16c5c660442417c396fa5cf25f8a81a4f)
@@ -930,15 +930,14 @@
      */
     private ActionForward deleteAttachment(ActionMapping mapping, ActionForm form, HttpServletRequest request,
 	    HttpServletResponse response) {
-
 	// only reset not attachment flag.
 	MessageDTO dto = new MessageDTO();
 	dto.setHasAttachment(false);
 	request.setAttribute(ForumConstants.AUTHORING_TOPIC, dto);
-	SessionMap sessionMap = (SessionMap) request.getSession()
-		.getAttribute(WebUtil.readStrParam(request, ForumConstants.ATTR_SESSION_MAP_ID));
+	String sessionMapId = WebUtil.readStrParam(request, ForumConstants.ATTR_SESSION_MAP_ID);
+	SessionMap sessionMap = (SessionMap) request.getSession().getAttribute(sessionMapId);
 	request.setAttribute(ForumConstants.ATTR_ALLOW_UPLOAD, sessionMap.get(ForumConstants.ATTR_ALLOW_UPLOAD));
-
+	request.setAttribute(ForumConstants.ATTR_SESSION_MAP_ID, sessionMapId);
 	return mapping.findForward("success");
     }
 
Fisheye: Tag d5fee901f5527e516add11b0ec872521de171133 refers to a dead (removed) revision in file `lams_tool_forum/web/jsps/learning/message/msgattachment.jsp'.
Fisheye: No comparison available.  Pass `N' to diff?
Index: lams_tool_forum/web/jsps/learning/message/topiceditform.jsp
===================================================================
diff -u -r0b0cbd0f4149520b777cd5f085c2c05c43f6f7e2 -r0e9302b16c5c660442417c396fa5cf25f8a81a4f
--- lams_tool_forum/web/jsps/learning/message/topiceditform.jsp	(.../topiceditform.jsp)	(revision 0b0cbd0f4149520b777cd5f085c2c05c43f6f7e2)
+++ lams_tool_forum/web/jsps/learning/message/topiceditform.jsp	(.../topiceditform.jsp)	(revision 0e9302b16c5c660442417c396fa5cf25f8a81a4f)
@@ -14,29 +14,9 @@
 <c:if test="${topic.hasAttachment || sessionMap.allowUpload}">
 	<c:set var="allowUpload" value="${sessionMap.allowUpload}" />
 
-	 <div class="form-group">
-    	<label><fmt:message key="message.label.attachment" />&nbsp;</label>
-		<div id="itemAttachmentArea">
-			<c:set var="ctxPath" value="${pageContext.request.contextPath}" scope="request"/>
-			<input type="hidden" name="hasAttachment" value="${topic.hasAttachment}"/>
-			<c:if test="${topic.hasAttachment}">
-					<c:forEach var="file" items="${topic.message.attachments}">
-					    <a id="removeAttachmentButton" href="#" onclick="removeAtt('${sessionMapID}')" class="btn btn-default btn-xs"><i class="fa fa-trash-o"></i></a>
-						<fmt:message key="message.label.attachment"/>: 
-						<c:set var="downloadURL">
-							<html:rewrite page="/download/?uuid=${file.fileUuid}&versionID=${file.fileVersionId}&preferDownload=true" /> 
-						</c:set>
-						<a href="<c:out value='${downloadURL}' escapeXml='false'/>"> <c:out value="${file.fileName}" /> </a>
-				  		<BR/>
-			  		</c:forEach>
-				</ul>
-			</c:if>
-			
-			<c:if test="${not topic.hasAttachment && allowUpload}">
-				<lams:FileUpload fileFieldname="attachmentFile" maxFileSize="${sessionMap.uploadMaxFileSize}"/>
-			</c:if>
-		</div>
-	</div>		
+	<div id="itemAttachmentArea">
+		<%@ include file="/jsps/learning/message/msgattachment.jsp"%>
+	</div>
 
 	<lams:WaitingSpinner id="itemAttachmentArea_Busy"/>
 </c:if>