Index: lams_tool_nb/src/java/org/lamsfoundation/lams/tool/noticeboard/NoticeboardContent.java =================================================================== diff -u -r4aae0391a4892385de3df761a655e0ab8b1c0183 -r2a6d3a91cb4cf1c169518fe434154f645fabd602 --- lams_tool_nb/src/java/org/lamsfoundation/lams/tool/noticeboard/NoticeboardContent.java (.../NoticeboardContent.java) (revision 4aae0391a4892385de3df761a655e0ab8b1c0183) +++ lams_tool_nb/src/java/org/lamsfoundation/lams/tool/noticeboard/NoticeboardContent.java (.../NoticeboardContent.java) (revision 2a6d3a91cb4cf1c169518fe434154f645fabd602) @@ -430,16 +430,9 @@ while (iter.hasNext()) { NoticeboardAttachment element = (NoticeboardAttachment) iter.next(); NoticeboardAttachment newAttachment = new NoticeboardAttachment(newContent, element.getFilename(), element.isOnlineFile()); - if(toolContentHandler != null){ - //if it is not null, copy file node and refresh uuid and version - NodeKey keys = toolContentHandler.copyFile(element.getUuid()); - newAttachment.setUuid(keys.getUuid()); - newAttachment.setVersionId(keys.getVersion()); - }else{ - //keep old value - newAttachment.setUuid(element.getUuid()); - newAttachment.setVersionId(element.getVersionId()); - } + //keep old value do not duplicate file + newAttachment.setUuid(element.getUuid()); + newAttachment.setVersionId(element.getVersionId()); newAttachmentSet.add(newAttachment); } Index: lams_tool_nb/src/java/org/lamsfoundation/lams/tool/noticeboard/service/NoticeboardServicePOJO.java =================================================================== diff -u -r28b3045d39cf68d79a469dfa23e625b3822abfe0 -r2a6d3a91cb4cf1c169518fe434154f645fabd602 --- lams_tool_nb/src/java/org/lamsfoundation/lams/tool/noticeboard/service/NoticeboardServicePOJO.java (.../NoticeboardServicePOJO.java) (revision 28b3045d39cf68d79a469dfa23e625b3822abfe0) +++ lams_tool_nb/src/java/org/lamsfoundation/lams/tool/noticeboard/service/NoticeboardServicePOJO.java (.../NoticeboardServicePOJO.java) (revision 2a6d3a91cb4cf1c169518fe434154f645fabd602) @@ -765,7 +765,6 @@ { attachment.setNbContent(null); content.getNbAttachments().remove(attachment); - nbToolContentHandler.deleteFile(attachment.getUuid()); saveNoticeboard(content); } catch (DataAccessException e) Index: lams_tool_nb/src/java/org/lamsfoundation/lams/tool/noticeboard/web/NbAuthoringAction.java =================================================================== diff -u -r08950e1090443c3423a3d1c587416a2fccd8bbdf -r2a6d3a91cb4cf1c169518fe434154f645fabd602 --- lams_tool_nb/src/java/org/lamsfoundation/lams/tool/noticeboard/web/NbAuthoringAction.java (.../NbAuthoringAction.java) (revision 08950e1090443c3423a3d1c587416a2fccd8bbdf) +++ lams_tool_nb/src/java/org/lamsfoundation/lams/tool/noticeboard/web/NbAuthoringAction.java (.../NbAuthoringAction.java) (revision 2a6d3a91cb4cf1c169518fe434154f645fabd602) @@ -290,7 +290,7 @@ try { - // remove tool entry from db, includies removing entry from the content repository + // remove tool entry from db, does not removing entry from the content repository // deleting a non-existent entry shouldn't cause any errors. if ( attachment.getAttachmentId() != null ) { nbService.removeAttachment(nbContent, attachment);