Index: lams_tool_forum/src/java/org/lamsfoundation/lams/tool/forum/web/actions/AuthoringAction.java
===================================================================
RCS file: /usr/local/cvsroot/lams_tool_forum/src/java/org/lamsfoundation/lams/tool/forum/web/actions/AuthoringAction.java,v
diff -u -r1.11 -r1.12
--- lams_tool_forum/src/java/org/lamsfoundation/lams/tool/forum/web/actions/AuthoringAction.java	2 Nov 2005 23:45:06 -0000	1.11
+++ lams_tool_forum/src/java/org/lamsfoundation/lams/tool/forum/web/actions/AuthoringAction.java	3 Nov 2005 03:23:16 -0000	1.12
@@ -130,7 +130,7 @@
 			HttpServletRequest request, HttpServletResponse response) {
 
 		
-		Long contentId = new Long(WebUtil.readLongParam(request,AttributeNames.TOOL_CONTENT_ID));
+		Long contentId = new Long(WebUtil.readLongParam(request,AttributeNames.PARAM_TOOL_CONTENT_ID));
 		ForumForm forumForm = (ForumForm)form;
 		//get back the topic list and display them on page
 		forumService = getForumManager();
@@ -327,6 +327,7 @@
 			forumService.createMessage(forumId,newMsg.getMessage());
 			
 		}
+		request.setAttribute(ForumConstants.SUCCESS_FLAG,"ATT_SUCCESS_FLAG");
 		request.setAttribute(ForumConstants.AUTHORING_TOPICS_INDEX,topicIndex);
     	return mapping.findForward("success");
     }
@@ -488,7 +489,7 @@
 	 * @return
 	 */
 	private ActionForward deleteFile(HttpServletRequest request, HttpServletResponse response, ActionForm form, String type) {
-		Long contentID = new Long(WebUtil.readLongParam(request,AttributeNames.TOOL_CONTENT_ID));
+		Long contentID = new Long(WebUtil.readLongParam(request,AttributeNames.PARAM_TOOL_CONTENT_ID));
 		Long versionID = new Long(WebUtil.readLongParam(request,"versionID"));
 		Long uuID = new Long(WebUtil.readLongParam(request,"uuID"));
 		
Index: lams_tool_forum/web/jsps/authoring/message/edit.jsp
===================================================================
RCS file: /usr/local/cvsroot/lams_tool_forum/web/jsps/authoring/message/edit.jsp,v
diff -u -r1.2 -r1.3
--- lams_tool_forum/web/jsps/authoring/message/edit.jsp	1 Nov 2005 05:30:54 -0000	1.2
+++ lams_tool_forum/web/jsps/authoring/message/edit.jsp	3 Nov 2005 03:23:16 -0000	1.3
@@ -1,5 +1,14 @@
 <%@ include file="/includes/taglibs.jsp" %>
 <script type="text/javascript" src="<html:rewrite page='/includes/scripts.jsp'/>"></script>
+<script type="text/javascript">
+	function success(){
+		var flag = "<c:out value="${SUCCESS_FLAG}"/>";
+		if(flag == "ATT_SUCCESS_FLAG"){
+			window.opener.parent.location.href = "<html:rewrite page='/authoring/finishTopic.do'/>";
+		}
+	}
+	success();
+</script>
 <html:errors property="error" />
 <div align="center">
 <html:form action="/authoring/updateTopic.do" focus="message.subject"
Index: lams_tool_forum/web/jsps/message/topiceditform.jsp
===================================================================
RCS file: /usr/local/cvsroot/lams_tool_forum/web/jsps/message/Attic/topiceditform.jsp,v
diff -u -r1.2 -r1.3
--- lams_tool_forum/web/jsps/message/topiceditform.jsp	1 Nov 2005 06:11:03 -0000	1.2
+++ lams_tool_forum/web/jsps/message/topiceditform.jsp	3 Nov 2005 03:23:16 -0000	1.3
@@ -17,7 +17,7 @@
 			<b><bean:message key="message.label.attachment" /></b>
 		</td>
 		<td valign="MIDDLE">&nbsp;
-			<c:if test="${not empty topic.message.attachments}">
+			<c:if test="${topic.hasAttachment}">
 				<c:forEach var="file" items="${topic.message.attachments}">
 					<c:set var="downloadURL">
 							<html:rewrite page="/download/?uuid=${file.fileUuid}&versionID=${file.fileVersionId}&preferDownload=true"/>
@@ -34,7 +34,7 @@
 					</a>					
 				</c:forEach>
 			</c:if>
-			<c:if test="${empty topic.message.attachments}">
+			<c:if test="${not topic.hasAttachment}">
 				<html:file tabindex="3"		property="attachmentFile" /> <html:errors	property="message.attachment" />
 			</c:if>
 		</td>