Index: lams_common/src/java/org/lamsfoundation/lams/tool/ToolContentManager.java =================================================================== diff -u -r2b831e534444ad040546c3762a267a2f73e4199d -re48d40257f7cfd40e6cb04c76d4fcd23a16fc959 --- lams_common/src/java/org/lamsfoundation/lams/tool/ToolContentManager.java (.../ToolContentManager.java) (revision 2b831e534444ad040546c3762a267a2f73e4199d) +++ lams_common/src/java/org/lamsfoundation/lams/tool/ToolContentManager.java (.../ToolContentManager.java) (revision e48d40257f7cfd40e6cb04c76d4fcd23a16fc959) @@ -73,12 +73,21 @@ /** * Remove tool's content according specified the content id. It will be - * needed by lams to modify the learning design. Note: if session data - * for this toolContentId exists and removeSessionData = false, then - * the tool should throw SessionDataExists. + * needed by lams to modify the learning design. * - * If no matching data exists, return without throwing an exception. + * If the tool content includes files in the content repository (e.g. + * online and offline instructions) then the files should be removed + * from the repository. * + * If session data for this toolContentId exists and removeSessionData = true, + * then the tool should delete the session data as well as the content data. + * + * If session data for this toolContentId exists and removeSessionData = false, + * then the tool should throw SessionDataExists. + * + * If no matching data exists, the tool should return without throwing + * an exception. + * * @param toolContentId the requested tool content id. * @param removeSessionData should it remove any related session data? * @throws ToolException if any other error occurs