Index: lams_tool_larsrc/src/java/org/lamsfoundation/lams/tool/rsrc/util/ResourceWebUtils.java =================================================================== diff -u -r08950e1090443c3423a3d1c587416a2fccd8bbdf -re88d49b5c4d6b37bc23429c39cd6cb1c9905a089 --- lams_tool_larsrc/src/java/org/lamsfoundation/lams/tool/rsrc/util/ResourceWebUtils.java (.../ResourceWebUtils.java) (revision 08950e1090443c3423a3d1c587416a2fccd8bbdf) +++ lams_tool_larsrc/src/java/org/lamsfoundation/lams/tool/rsrc/util/ResourceWebUtils.java (.../ResourceWebUtils.java) (revision e88d49b5c4d6b37bc23429c39cd6cb1c9905a089) @@ -37,19 +37,5 @@ */ public class ResourceWebUtils { - public static boolean isResourceEditable(Resource resource) { - if ( (resource.isDefineLater() == true) && (resource.isContentInUse()==true) ) - { - // throw new ResourceApplicationException("An exception has occurred: There is a bug in this tool, conflicting flags are set"); - MonitoringAction.log.error("An exception has occurred: There is a bug in this tool, conflicting flags are set"); - return false; - } - else if ( (resource.isDefineLater() == true) && (resource.isContentInUse() == false)) - return true; - else if ( (resource.isDefineLater() == false) && (resource.isContentInUse() == false)) - return true; - else // (content.isContentInUse()==true && content.isDefineLater() == false) - return false; - } - +// We leave this blank for future use } Index: lams_tool_larsrc/src/java/org/lamsfoundation/lams/tool/rsrc/web/action/AuthoringAction.java =================================================================== diff -u -r88329eba8a9d648ae3907adfc06dcce370ea6880 -re88d49b5c4d6b37bc23429c39cd6cb1c9905a089 --- lams_tool_larsrc/src/java/org/lamsfoundation/lams/tool/rsrc/web/action/AuthoringAction.java (.../AuthoringAction.java) (revision 88329eba8a9d648ae3907adfc06dcce370ea6880) +++ lams_tool_larsrc/src/java/org/lamsfoundation/lams/tool/rsrc/web/action/AuthoringAction.java (.../AuthoringAction.java) (revision e88d49b5c4d6b37bc23429c39cd6cb1c9905a089) @@ -113,17 +113,9 @@ IResourceService service = getResourceService(); Resource resource = service.getResourceByContentId(contentId); - boolean isEditable = ResourceWebUtils.isResourceEditable(resource); - if(!isEditable){ - request.setAttribute(ResourceConstants.PAGE_EDITABLE, new Boolean(isEditable)); - return mapping.findForward("forbidden"); - } + resource.setDefineLater(true); + service.saveOrUpdateResource(resource); - if(!resource.isContentInUse()){ - resource.setDefineLater(true); - service.saveOrUpdateResource(resource); - } - request.setAttribute(AttributeNames.ATTR_MODE,ToolAccessMode.TEACHER.toString()); return start(mapping, form, request, response); } Index: lams_tool_larsrc/src/java/org/lamsfoundation/lams/tool/rsrc/web/action/MonitoringAction.java =================================================================== diff -u -rf50903db417c658ce7f49580001c058e012b7f13 -re88d49b5c4d6b37bc23429c39cd6cb1c9905a089 --- lams_tool_larsrc/src/java/org/lamsfoundation/lams/tool/rsrc/web/action/MonitoringAction.java (.../MonitoringAction.java) (revision f50903db417c658ce7f49580001c058e012b7f13) +++ lams_tool_larsrc/src/java/org/lamsfoundation/lams/tool/rsrc/web/action/MonitoringAction.java (.../MonitoringAction.java) (revision e88d49b5c4d6b37bc23429c39cd6cb1c9905a089) @@ -156,7 +156,7 @@ //cache into sessionMap sessionMap.put(ResourceConstants.ATTR_SUMMARY_LIST, groupList); - sessionMap.put(ResourceConstants.PAGE_EDITABLE, new Boolean(ResourceWebUtils.isResourceEditable(resource))); + sessionMap.put(ResourceConstants.PAGE_EDITABLE, resource.isContentInUse()); sessionMap.put(ResourceConstants.ATTR_RESOURCE, resource); sessionMap.put(ResourceConstants.ATTR_TOOL_CONTENT_ID, contentId); sessionMap.put(ResourceConstants.ATTR_REFLECT_LIST, relectList);