+ ${sessionMap.title} +
+ ++ ${sessionMap.reflectInstructions} +
+ +
+
+
Ticket
to login and
- * access the Content Repository.
- *
- * A valid ticket is needed in order to access the content from the repository. This method would be called evertime
- * the tool needs to upload/download files from the content repository.
- *
- * @return ITicket The ticket for repostory access
- * @throws SurveyApplicationException
- */
- private ITicket getRepositoryLoginTicket() throws SurveyApplicationException {
- ICredentials credentials = new SimpleCredentials(surveyToolContentHandler.getRepositoryUser(),
- surveyToolContentHandler.getRepositoryId());
- try {
- ITicket ticket = repositoryService
- .login(credentials, surveyToolContentHandler.getRepositoryWorkspaceName());
- return ticket;
- } catch (AccessDeniedException ae) {
- throw new SurveyApplicationException("Access Denied to repository." + ae.getMessage());
- } catch (WorkspaceNotFoundException we) {
- throw new SurveyApplicationException("Workspace not found." + we.getMessage());
- } catch (LoginException e) {
- throw new SurveyApplicationException("Login failed." + e.getMessage());
- }
- }
-
// *******************************************************************************
// ToolContentManager, ToolSessionManager methods
// *******************************************************************************
@@ -681,13 +583,8 @@
}
// set SurveyToolContentHandler as null to avoid copy file node in repository again.
- toolContentObj = Survey.newInstance(toolContentObj, toolContentId, null);
- toolContentObj.setToolContentHandler(null);
- toolContentObj.setOfflineFileList(null);
- toolContentObj.setOnlineFileList(null);
+ toolContentObj = Survey.newInstance(toolContentObj, toolContentId);
try {
- exportContentService.registerFileClassForExport(SurveyAttachment.class.getName(), "fileUuid",
- "fileVersionId");
exportContentService.exportToolContent(toolContentId, toolContentObj, surveyToolContentHandler, rootPath);
} catch (ExportToolContentException e) {
throw new ToolException(e);
@@ -698,9 +595,9 @@
String toVersion) throws ToolException {
try {
- exportContentService.registerFileClassForImport(SurveyAttachment.class.getName(), "fileUuid",
- "fileVersionId", "fileName", "fileType", null, null);
-
+ // register version filter class
+ exportContentService.registerImportVersionFilterClass(SurveyImportContentVersionFilter.class);
+
Object toolPOJO = exportContentService.importToolContent(toolContentPath, surveyToolContentHandler,
fromVersion, toVersion);
if (!(toolPOJO instanceof Survey)) {
@@ -773,7 +670,7 @@
}
}
- Survey toContent = Survey.newInstance(survey, toContentId, surveyToolContentHandler);
+ Survey toContent = Survey.newInstance(survey, toContentId);
surveyDao.saveObject(toContent);
// save survey items as well
@@ -790,23 +687,7 @@
public String getToolContentTitle(Long toolContentId) {
return getSurveyByContentId(toolContentId).getTitle();
}
-
- public void setAsDefineLater(Long toolContentId, boolean value) throws DataMissingException, ToolException {
- Survey survey = surveyDao.getByContentId(toolContentId);
- if (survey == null) {
- throw new ToolException("No found tool content by given content ID:" + toolContentId);
- }
- survey.setDefineLater(value);
- }
- public void setAsRunOffline(Long toolContentId, boolean value) throws DataMissingException, ToolException {
- Survey survey = surveyDao.getByContentId(toolContentId);
- if (survey == null) {
- throw new ToolException("No found tool content by given content ID:" + toolContentId);
- }
- survey.setRunOffline(value);
- }
-
public void removeToolContent(Long toolContentId, boolean removeSessionData) throws SessionDataExistsException,
ToolException {
Survey survey = surveyDao.getByContentId(toolContentId);
@@ -920,9 +801,6 @@
toolContentObj.setDefineLater(Boolean.FALSE);
toolContentObj.setInstructions(WebUtil.convertNewlines((String) importValues
.get(ToolContentImport102Manager.CONTENT_BODY)));
- toolContentObj.setOfflineInstructions(null);
- toolContentObj.setOnlineInstructions(null);
- toolContentObj.setRunOffline(Boolean.FALSE);
toolContentObj.setUpdated(now);
Boolean isReusable = WDDXProcessor.convertToBoolean(importValues,
@@ -1037,9 +915,6 @@
// *****************************************************************************
// set methods for Spring Bean
// *****************************************************************************
- public void setAuditService(IAuditService auditService) {
- this.auditService = auditService;
- }
public void setLearnerService(ILearnerService learnerService) {
this.learnerService = learnerService;
@@ -1049,14 +924,6 @@
this.messageService = messageService;
}
- public void setRepositoryService(IRepositoryService repositoryService) {
- this.repositoryService = repositoryService;
- }
-
- public void setSurveyAttachmentDao(SurveyAttachmentDAO surveyAttachmentDao) {
- this.surveyAttachmentDao = surveyAttachmentDao;
- }
-
public void setSurveyDao(SurveyDAO surveyDao) {
this.surveyDao = surveyDao;
}
Index: lams_tool_survey/src/java/org/lamsfoundation/lams/tool/survey/web/action/AuthoringAction.java
===================================================================
RCS file: /usr/local/cvsroot/lams_tool_survey/src/java/org/lamsfoundation/lams/tool/survey/web/action/AuthoringAction.java,v
diff -u -r1.26 -r1.27
--- lams_tool_survey/src/java/org/lamsfoundation/lams/tool/survey/web/action/AuthoringAction.java 25 Jul 2013 05:55:23 -0000 1.26
+++ lams_tool_survey/src/java/org/lamsfoundation/lams/tool/survey/web/action/AuthoringAction.java 17 Jan 2014 22:18:05 -0000 1.27
@@ -37,6 +37,7 @@
import java.util.Set;
import java.util.SortedSet;
import java.util.TreeSet;
+
import javax.servlet.ServletException;
import javax.servlet.http.HttpServletRequest;
import javax.servlet.http.HttpServletResponse;
@@ -60,7 +61,6 @@
import org.lamsfoundation.lams.tool.ToolAccessMode;
import org.lamsfoundation.lams.tool.survey.SurveyConstants;
import org.lamsfoundation.lams.tool.survey.model.Survey;
-import org.lamsfoundation.lams.tool.survey.model.SurveyAttachment;
import org.lamsfoundation.lams.tool.survey.model.SurveyCondition;
import org.lamsfoundation.lams.tool.survey.model.SurveyOption;
import org.lamsfoundation.lams.tool.survey.model.SurveyQuestion;
@@ -136,18 +136,6 @@
if (param.equals("updateContent")) {
return updateContent(mapping, form, request, response);
}
- if (param.equals("uploadOnlineFile")) {
- return uploadOnline(mapping, form, request, response);
- }
- if (param.equals("uploadOfflineFile")) {
- return uploadOffline(mapping, form, request, response);
- }
- if (param.equals("deleteOnlineFile")) {
- return deleteOnlineFile(mapping, form, request, response);
- }
- if (param.equals("deleteOfflineFile")) {
- return deleteOfflineFile(mapping, form, request, response);
- }
// ----------------------- Add survey item function ---------------------------
if (param.equals("newItemInit")) {
return newItemlInit(mapping, form, request, response);
@@ -461,11 +449,6 @@
surveyForm.setSurvey(survey);
- // initialize instruction attachment list
- List attachmentList = getAttachmentList(sessionMap);
- attachmentList.clear();
- attachmentList.addAll(survey.getAttachments());
-
// init it to avoid null exception in following handling
if (questions == null) {
questions = new ArrayList();
@@ -598,51 +581,11 @@
}
surveyPO.setCreatedBy(surveyUser);
-
- // **********************************Handle Authoring Instruction Attachement *********************
- // merge attachment info
- // so far, attPOSet will be empty if content is existed. because PropertyUtils.copyProperties() is executed
- Set attPOSet = surveyPO.getAttachments();
- if (attPOSet == null) {
- attPOSet = new HashSet();
- }
- List attachmentList = getAttachmentList(sessionMap);
- List deleteAttachmentList = getDeletedAttachmentList(sessionMap);
-
- // current attachemnt in authoring instruction tab.
- Iterator iter = attachmentList.iterator();
- while (iter.hasNext()) {
- SurveyAttachment newAtt = (SurveyAttachment) iter.next();
- attPOSet.add(newAtt);
- }
- attachmentList.clear();
-
- // deleted attachment. 2 possible types: one is persist another is non-persist before.
- iter = deleteAttachmentList.iterator();
- while (iter.hasNext()) {
- SurveyAttachment delAtt = (SurveyAttachment) iter.next();
- iter.remove();
- // it is an existed att, then delete it from current attachmentPO
- if (delAtt.getUid() != null) {
- Iterator attIter = attPOSet.iterator();
- while (attIter.hasNext()) {
- SurveyAttachment att = (SurveyAttachment) attIter.next();
- if (delAtt.getUid().equals(att.getUid())) {
- attIter.remove();
- break;
- }
- }
- service.deleteSurveyAttachment(delAtt.getUid());
- }// end remove from persist value
- }
-
- // copy back
- surveyPO.setAttachments(attPOSet);
// ************************* Handle survey questions *******************
Set questionList = new LinkedHashSet();
SortedSet topics = getSurveyItemList(sessionMap);
- iter = topics.iterator();
+ Iterator iter = topics.iterator();
while (iter.hasNext()) {
SurveyQuestion item = (SurveyQuestion) iter.next();
if (item != null) {
@@ -696,9 +639,6 @@
// finally persist surveyPO again
service.saveOrUpdateSurvey(surveyPO);
- // initialize attachmentList again
- attachmentList = getAttachmentList(sessionMap);
- attachmentList.addAll(survey.getAttachments());
surveyForm.setSurvey(surveyPO);
request.setAttribute(AuthoringConstants.LAMS_AUTHORING_SUCCESS_FLAG, Boolean.TRUE);
@@ -709,165 +649,6 @@
}
}
- /**
- * Handle upload online instruction files request.
- *
- * @param mapping
- * @param form
- * @param request
- * @param response
- * @return
- * @throws UploadSurveyFileException
- */
- public ActionForward uploadOnline(ActionMapping mapping, ActionForm form, HttpServletRequest request,
- HttpServletResponse response) throws UploadSurveyFileException {
- return uploadFile(mapping, form, IToolContentHandler.TYPE_ONLINE, request);
- }
-
- /**
- * Handle upload offline instruction files request.
- *
- * @param mapping
- * @param form
- * @param request
- * @param response
- * @return
- * @throws UploadSurveyFileException
- */
- public ActionForward uploadOffline(ActionMapping mapping, ActionForm form, HttpServletRequest request,
- HttpServletResponse response) throws UploadSurveyFileException {
- return uploadFile(mapping, form, IToolContentHandler.TYPE_OFFLINE, request);
- }
-
- /**
- * Common method to upload online or offline instruction files request.
- *
- * @param mapping
- * @param form
- * @param type
- * @param request
- * @return
- * @throws UploadSurveyFileException
- */
- private ActionForward uploadFile(ActionMapping mapping, ActionForm form, String type, HttpServletRequest request)
- throws UploadSurveyFileException {
-
- SurveyForm surveyForm = (SurveyForm) form;
- // get back sessionMAP
- SessionMap sessionMap = (SessionMap) request.getSession().getAttribute(surveyForm.getSessionMapID());
-
- FormFile file;
- if (StringUtils.equals(IToolContentHandler.TYPE_OFFLINE, type)) {
- file = surveyForm.getOfflineFile();
- } else {
- file = surveyForm.getOnlineFile();
- }
-
- if (file == null || StringUtils.isBlank(file.getFileName())) {
- return mapping.findForward(SurveyConstants.SUCCESS);
- }
-
- ActionMessages errors = new ActionMessages();
- FileValidatorUtil.validateFileSize(file, true, errors);
- if (!errors.isEmpty()) {
- this.saveErrors(request, errors);
- return mapping.findForward("success");
- }
-
- ISurveyService service = getSurveyService();
- // upload to repository
- SurveyAttachment att = service.uploadInstructionFile(file, type);
- // handle session value
- List attachmentList = getAttachmentList(sessionMap);
- List deleteAttachmentList = getDeletedAttachmentList(sessionMap);
- // first check exist attachment and delete old one (if exist) to deletedAttachmentList
- Iterator iter = attachmentList.iterator();
- SurveyAttachment existAtt;
- while (iter.hasNext()) {
- existAtt = (SurveyAttachment) iter.next();
- if (StringUtils.equals(existAtt.getFileName(), att.getFileName())
- && StringUtils.equals(existAtt.getFileType(), att.getFileType())) {
- // if there is same name attachment, delete old one
- deleteAttachmentList.add(existAtt);
- iter.remove();
- break;
- }
- }
- // add to attachmentList
- attachmentList.add(att);
-
- return mapping.findForward(SurveyConstants.SUCCESS);
-
- }
-
- /**
- * Delete offline instruction file from current Survey authoring page.
- *
- * @param mapping
- * @param form
- * @param request
- * @param response
- * @return
- */
- public ActionForward deleteOfflineFile(ActionMapping mapping, ActionForm form, HttpServletRequest request,
- HttpServletResponse response) {
- return deleteFile(mapping, request, response, form, IToolContentHandler.TYPE_OFFLINE);
- }
-
- /**
- * Delete online instruction file from current Survey authoring page.
- *
- * @param mapping
- * @param form
- * @param request
- * @param response
- * @return
- */
- public ActionForward deleteOnlineFile(ActionMapping mapping, ActionForm form, HttpServletRequest request,
- HttpServletResponse response) {
- return deleteFile(mapping, request, response, form, IToolContentHandler.TYPE_ONLINE);
- }
-
- /**
- * General method to delete file (online or offline)
- *
- * @param mapping
- * @param request
- * @param response
- * @param form
- * @param type
- * @return
- */
- private ActionForward deleteFile(ActionMapping mapping, HttpServletRequest request, HttpServletResponse response,
- ActionForm form, String type) {
- Long versionID = new Long(WebUtil.readLongParam(request, SurveyConstants.PARAM_FILE_VERSION_ID));
- Long uuID = new Long(WebUtil.readLongParam(request, SurveyConstants.PARAM_FILE_UUID));
-
- // get back sessionMAP
- String sessionMapID = WebUtil.readStrParam(request, SurveyConstants.ATTR_SESSION_MAP_ID);
- SessionMap sessionMap = (SessionMap) request.getSession().getAttribute(sessionMapID);
-
- // handle session value
- List attachmentList = getAttachmentList(sessionMap);
- List deleteAttachmentList = getDeletedAttachmentList(sessionMap);
- // first check exist attachment and delete old one (if exist) to deletedAttachmentList
- Iterator iter = attachmentList.iterator();
- SurveyAttachment existAtt;
- while (iter.hasNext()) {
- existAtt = (SurveyAttachment) iter.next();
- if (existAtt.getFileUuid().equals(uuID) && existAtt.getFileVersionId().equals(versionID)) {
- // if there is same name attachment, delete old one
- deleteAttachmentList.add(existAtt);
- iter.remove();
- }
- }
-
- request.setAttribute(SurveyConstants.ATTR_FILE_TYPE_FLAG, type);
- request.setAttribute(SurveyConstants.ATTR_SESSION_MAP_ID, sessionMapID);
- return mapping.findForward(SurveyConstants.SUCCESS);
-
- }
-
// *************************************************************************************
// Private method
// *************************************************************************************
@@ -881,22 +662,6 @@
}
/**
- * @param request
- * @return
- */
- private List getAttachmentList(SessionMap sessionMap) {
- return getListFromSession(sessionMap, SurveyConstants.ATT_ATTACHMENT_LIST);
- }
-
- /**
- * @param request
- * @return
- */
- private List getDeletedAttachmentList(SessionMap sessionMap) {
- return getListFromSession(sessionMap, SurveyConstants.ATTR_DELETED_ATTACHMENT_LIST);
- }
-
- /**
* List save current survey items.
*
* @param request
Index: lams_tool_survey/src/java/org/lamsfoundation/lams/tool/survey/web/action/LearningAction.java
===================================================================
RCS file: /usr/local/cvsroot/lams_tool_survey/src/java/org/lamsfoundation/lams/tool/survey/web/action/LearningAction.java,v
diff -u -r1.26 -r1.27
--- lams_tool_survey/src/java/org/lamsfoundation/lams/tool/survey/web/action/LearningAction.java 4 Jul 2013 11:35:09 -0000 1.26
+++ lams_tool_survey/src/java/org/lamsfoundation/lams/tool/survey/web/action/LearningAction.java 17 Jan 2014 22:18:05 -0000 1.27
@@ -144,7 +144,7 @@
// save toolContentID into HTTPSession
ToolAccessMode mode = WebUtil.readToolAccessModeParam(request, AttributeNames.PARAM_MODE, true);
Long sessionId = new Long(request.getParameter(AttributeNames.PARAM_TOOL_SESSION_ID));
- // it will be use when runOffline or lock on finish page.
+ // it will be use when submissionDeadline or lock on finish page.
request.setAttribute(SurveyConstants.ATTR_SESSION_MAP_ID, sessionMap.getSessionID());
// get back the survey and question list and display them on page
@@ -205,14 +205,6 @@
survey.setDefineLater(false);
service.saveOrUpdateSurvey(survey);
- // add run offline support
- if (survey.getRunOffline()) {
- sessionMap.put(SurveyConstants.PARAM_RUN_OFFLINE, true);
- return mapping.findForward(SurveyConstants.RUN_OFFLINE);
- } else {
- sessionMap.put(SurveyConstants.PARAM_RUN_OFFLINE, false);
- }
-
ActivityPositionDTO activityPosition = LearningWebUtil.putActivityPositionInRequestByToolSessionId(sessionId,
request, getServlet().getServletContext());
sessionMap.put(AttributeNames.ATTR_ACTIVITY_POSITION, activityPosition);
@@ -228,9 +220,9 @@
Date tzSubmissionDeadline = DateUtil.convertToTimeZoneFromDefault(learnerTimeZone, submissionDeadline);
Date currentLearnerDate = DateUtil.convertToTimeZoneFromDefault(learnerTimeZone, new Date());
- // calculate whether submission deadline has passed, and if so forward to "runOffline"
+ // calculate whether submission deadline has passed, and if so forward to "submissionDeadline"
if (currentLearnerDate.after(tzSubmissionDeadline)) {
- return mapping.findForward(SurveyConstants.RUN_OFFLINE);
+ return mapping.findForward("submissionDeadline");
}
}
Index: lams_tool_survey/src/java/org/lamsfoundation/lams/tool/survey/web/action/MonitoringAction.java
===================================================================
RCS file: /usr/local/cvsroot/lams_tool_survey/src/java/org/lamsfoundation/lams/tool/survey/web/action/MonitoringAction.java,v
diff -u -r1.22 -r1.23
--- lams_tool_survey/src/java/org/lamsfoundation/lams/tool/survey/web/action/MonitoringAction.java 12 Nov 2012 17:13:40 -0000 1.22
+++ lams_tool_survey/src/java/org/lamsfoundation/lams/tool/survey/web/action/MonitoringAction.java 17 Jan 2014 22:18:05 -0000 1.23
@@ -148,7 +148,6 @@
// get survey
Survey survey = service.getSurveyByContentId(contentId);
- survey.toDTO();
// get statistic
SortedMap
+
+
TaskListItemVisitDAO
.
Index: lams_tool_task/src/java/org/lamsfoundation/lams/tool/taskList/dbupdates/patch20140102.sql
===================================================================
RCS file: /usr/local/cvsroot/lams_tool_task/src/java/org/lamsfoundation/lams/tool/taskList/dbupdates/patch20140102.sql,v
diff -u
--- /dev/null 1 Jan 1970 00:00:00 -0000
+++ lams_tool_task/src/java/org/lamsfoundation/lams/tool/taskList/dbupdates/patch20140102.sql 17 Jan 2014 22:18:13 -0000 1.1
@@ -0,0 +1,19 @@
+-- Turn off autocommit, so nothing is committed if there is an error
+SET AUTOCOMMIT = 0;
+SET FOREIGN_KEY_CHECKS=0;
+----------------------Put all sql statements below here-------------------------
+
+-- LDEV-3147 Simplify tools: get rid of instructions tab, define in monitor and offline activity options
+ALTER TABLE tl_latask10_taskList DROP COLUMN online_instructions;
+ALTER TABLE tl_latask10_taskList DROP COLUMN offline_instructions;
+ALTER TABLE tl_latask10_taskList DROP COLUMN run_offline;
+DROP TABLE IF EXISTS tl_latask10_attachment;
+
+UPDATE lams_tool SET tool_version='20140102' WHERE tool_signature='latask10';
+
+----------------------Put all sql statements above here-------------------------
+
+-- If there were no errors, commit and restore autocommit to on
+COMMIT;
+SET AUTOCOMMIT = 1;
+SET FOREIGN_KEY_CHECKS=1;
\ No newline at end of file
Index: lams_tool_task/src/java/org/lamsfoundation/lams/tool/taskList/dto/GroupSummary.java
===================================================================
RCS file: /usr/local/cvsroot/lams_tool_task/src/java/org/lamsfoundation/lams/tool/taskList/dto/GroupSummary.java,v
diff -u -r1.2 -r1.3
--- lams_tool_task/src/java/org/lamsfoundation/lams/tool/taskList/dto/GroupSummary.java 26 Jun 2008 02:30:18 -0000 1.2
+++ lams_tool_task/src/java/org/lamsfoundation/lams/tool/taskList/dto/GroupSummary.java 17 Jan 2014 22:18:14 -0000 1.3
@@ -26,8 +26,6 @@
import java.util.ArrayList;
import java.util.List;
-import org.lamsfoundation.lams.tool.taskList.model.TaskListItem;
-
/**
* DTO object used in a monitoring. Contains information for group.
*
Index: lams_tool_task/src/java/org/lamsfoundation/lams/tool/taskList/dto/TasListItemDTO.java
===================================================================
RCS file: /usr/local/cvsroot/lams_tool_task/src/java/org/lamsfoundation/lams/tool/taskList/dto/TasListItemDTO.java,v
diff -u -r1.2 -r1.3
--- lams_tool_task/src/java/org/lamsfoundation/lams/tool/taskList/dto/TasListItemDTO.java 30 May 2008 09:13:33 -0000 1.2
+++ lams_tool_task/src/java/org/lamsfoundation/lams/tool/taskList/dto/TasListItemDTO.java 17 Jan 2014 22:18:14 -0000 1.3
@@ -23,16 +23,12 @@
/* $Id$ */
package org.lamsfoundation.lams.tool.taskList.dto;
-import java.util.ArrayList;
-import java.util.Date;
import java.util.HashSet;
-import java.util.List;
import java.util.Set;
import org.lamsfoundation.lams.tool.taskList.model.TaskListItem;
import org.lamsfoundation.lams.tool.taskList.model.TaskListItemAttachment;
import org.lamsfoundation.lams.tool.taskList.model.TaskListItemComment;
-import org.lamsfoundation.lams.tool.taskList.model.TaskListUser;
/**
* DTO object which contains data for displaying learning stuff.
Index: lams_tool_task/src/java/org/lamsfoundation/lams/tool/taskList/model/TaskList.java
===================================================================
RCS file: /usr/local/cvsroot/lams_tool_task/src/java/org/lamsfoundation/lams/tool/taskList/model/TaskList.java,v
diff -u -r1.14 -r1.15
--- lams_tool_task/src/java/org/lamsfoundation/lams/tool/taskList/model/TaskList.java 10 May 2011 23:45:57 -0000 1.14
+++ lams_tool_task/src/java/org/lamsfoundation/lams/tool/taskList/model/TaskList.java 17 Jan 2014 22:18:13 -0000 1.15
@@ -23,21 +23,15 @@
/* $Id$ */
package org.lamsfoundation.lams.tool.taskList.model;
-import java.util.ArrayList;
import java.util.Date;
import java.util.HashMap;
import java.util.HashSet;
import java.util.Iterator;
-import java.util.List;
-import java.util.Map;
import java.util.Set;
-import org.apache.commons.lang.StringUtils;
import org.apache.commons.lang.builder.EqualsBuilder;
import org.apache.commons.lang.builder.HashCodeBuilder;
import org.apache.log4j.Logger;
-import org.lamsfoundation.lams.contentrepository.client.IToolContentHandler;
-import org.lamsfoundation.lams.tool.taskList.util.TaskListToolContentHandler;
/**
* The main entity class of TaskList tool. Contains all the data related to the whole tool.
@@ -65,17 +59,10 @@
private boolean allowContributeTasks;
private boolean monitorVerificationRequired;
- private boolean runOffline;
private boolean defineLater;
private boolean contentInUse;
-
- // LDEV-2657
private Date submissionDeadline;
- //instructions
- private String onlineInstructions;
- private String offlineInstructions;
- private Set attachments;
//conditions
private Set conditions;
@@ -91,17 +78,12 @@
private String reflectInstructions;
//*************** NON Persist Fields ********************
- private IToolContentHandler toolContentHandler;
-
- private Listorg.lamsfoundation.lams.tool.taskList.service.ITaskListService
.
@@ -111,7 +97,6 @@
static Logger log = Logger.getLogger(TaskListServiceImpl.class.getName());
private TaskListDAO taskListDao;
private TaskListItemDAO taskListItemDao;
- private TaskListAttachmentDAO taskListAttachmentDao;
private TaskListConditionDAO taskListConditionDAO;
private TaskListUserDAO taskListUserDao;
private TaskListSessionDAO taskListSessionDao;
@@ -123,10 +108,8 @@
private MessageService messageService;
private TaskListOutputFactory taskListOutputFactory;
// system services
- private IRepositoryService repositoryService;
private ILamsToolService toolService;
private ILearnerService learnerService;
- private IAuditService auditService;
private IUserManagementService userManagementService;
private IExportToolContentService exportContentService;
private ICoreNotebookService coreNotebookService;
@@ -159,7 +142,7 @@
TaskList defaultContent = getDefaultTaskList();
// save default content by given ID.
TaskList content = new TaskList();
- content = TaskList.newInstance(defaultContent, contentId, taskListToolContentHandler);
+ content = TaskList.newInstance(defaultContent, contentId);
return content;
}
@@ -173,43 +156,18 @@
/**
* {@inheritDoc}
*/
- public TaskListAttachment uploadInstructionFile(FormFile uploadFile, String fileType)
+ public TaskListItemAttachment uploadTaskListItemFile(FormFile uploadFile, TaskListUser user)
throws UploadTaskListFileException {
if (uploadFile == null || StringUtils.isEmpty(uploadFile.getFileName())) {
throw new UploadTaskListFileException(messageService.getMessage("error.msg.upload.file.not.found",
new Object[] { uploadFile }));
}
// upload file to repository
- NodeKey nodeKey = processFile(uploadFile, fileType);
+ NodeKey nodeKey = processFile(uploadFile);
// create new attachement
- TaskListAttachment file = new TaskListAttachment();
- file.setFileType(fileType);
- file.setFileUuid(nodeKey.getUuid());
- file.setFileVersionId(nodeKey.getVersion());
- file.setFileName(uploadFile.getFileName());
- file.setCreated(new Date());
-
- return file;
- }
-
- /**
- * {@inheritDoc}
- */
- public TaskListItemAttachment uploadTaskListItemFile(FormFile uploadFile, String fileType, TaskListUser user)
- throws UploadTaskListFileException {
- if (uploadFile == null || StringUtils.isEmpty(uploadFile.getFileName())) {
- throw new UploadTaskListFileException(messageService.getMessage("error.msg.upload.file.not.found",
- new Object[] { uploadFile }));
- }
-
- // upload file to repository
- NodeKey nodeKey = processFile(uploadFile, fileType);
-
- // create new attachement
TaskListItemAttachment file = new TaskListItemAttachment();
- file.setFileType(fileType);
file.setFileUuid(nodeKey.getUuid());
file.setFileVersionId(nodeKey.getVersion());
file.setFileName(uploadFile.getFileName());
@@ -250,33 +208,13 @@
/**
* {@inheritDoc}
*/
- public void deleteFromRepository(Long fileUuid, Long fileVersionId) throws TaskListException {
- ITicket ticket = getRepositoryLoginTicket();
- try {
- repositoryService.deleteVersion(ticket, fileUuid, fileVersionId);
- } catch (Exception e) {
- throw new TaskListException("Exception occured while deleting files from" + " the repository "
- + e.getMessage());
- }
- }
-
- /**
- * {@inheritDoc}
- */
public void saveOrUpdateTaskList(TaskList taskList) {
taskListDao.saveObject(taskList);
}
/**
* {@inheritDoc}
*/
- public void deleteTaskListAttachment(Long attachmentUid) {
- taskListAttachmentDao.removeObject(TaskListAttachment.class, attachmentUid);
- }
-
- /**
- * {@inheritDoc}
- */
public void deleteTaskListCondition(Long conditionUid) {
taskListConditionDAO.removeObject(TaskListCondition.class, conditionUid);
}
@@ -710,10 +648,6 @@
// Set methods for Spring Bean
// *****************************************************************************
- public void setAuditService(IAuditService auditService) {
- this.auditService = auditService;
- }
-
public void setLearnerService(ILearnerService learnerService) {
this.learnerService = learnerService;
}
@@ -730,14 +664,10 @@
this.taskListOutputFactory = taskListOutputFactory;
}
- public void setRepositoryService(IRepositoryService repositoryService) {
- this.repositoryService = repositoryService;
+ public void setTaskListAttachmentDao(TaskListItemAttachmentDAO taskListItemAttachmentDao) {
+ this.taskListItemAttachmentDao = taskListItemAttachmentDao;
}
- public void setTaskListAttachmentDao(TaskListAttachmentDAO taskListAttachmentDao) {
- this.taskListAttachmentDao = taskListAttachmentDao;
- }
-
public void setTaskListConditionDao(TaskListConditionDAO taskListConditionDAO) {
this.taskListConditionDAO = taskListConditionDAO;
}
@@ -822,19 +752,14 @@
}
// set TaskListToolContentHandler as null to avoid copy file node in repository again.
- toolContentObj = TaskList.newInstance(toolContentObj, toolContentId, null);
- toolContentObj.setToolContentHandler(null);
- toolContentObj.setOfflineFileList(null);
- toolContentObj.setOnlineFileList(null);
+ toolContentObj = TaskList.newInstance(toolContentObj, toolContentId);
SetTicket
to login and
- * access the Content Repository.
- *
- * A valid ticket is needed in order to access the content from the repository. This method would be called evertime
- * the tool needs to upload/download files from the content repository.
- *
- * @return ITicket The ticket for repostory access
- * @throws TaskListException
- */
- private ITicket getRepositoryLoginTicket() throws TaskListException {
- ICredentials credentials = new SimpleCredentials(taskListToolContentHandler.getRepositoryUser(),
- taskListToolContentHandler.getRepositoryId());
- try {
- ITicket ticket = repositoryService.login(credentials, taskListToolContentHandler
- .getRepositoryWorkspaceName());
- return ticket;
- } catch (AccessDeniedException ae) {
- throw new TaskListException("Access Denied to repository." + ae.getMessage());
- } catch (WorkspaceNotFoundException we) {
- throw new TaskListException("Workspace not found." + we.getMessage());
- } catch (LoginException e) {
- throw new TaskListException("Login failed." + e.getMessage());
- }
- }
-
public Class[] getSupportedToolOutputDefinitionClasses(int definitionType) {
return getTaskListOutputFactory().getSupportedDefinitionClasses(definitionType);
}
Index: lams_tool_task/src/java/org/lamsfoundation/lams/tool/taskList/web/action/AuthoringAction.java
===================================================================
RCS file: /usr/local/cvsroot/lams_tool_task/src/java/org/lamsfoundation/lams/tool/taskList/web/action/AuthoringAction.java,v
diff -u -r1.19 -r1.20
--- lams_tool_task/src/java/org/lamsfoundation/lams/tool/taskList/web/action/AuthoringAction.java 25 Jul 2013 05:56:08 -0000 1.19
+++ lams_tool_task/src/java/org/lamsfoundation/lams/tool/taskList/web/action/AuthoringAction.java 17 Jan 2014 22:18:13 -0000 1.20
@@ -27,7 +27,6 @@
import java.sql.Timestamp;
import java.util.ArrayList;
import java.util.Date;
-import java.util.HashSet;
import java.util.Iterator;
import java.util.LinkedHashSet;
import java.util.LinkedList;
@@ -52,26 +51,21 @@
import org.apache.struts.action.ActionMapping;
import org.apache.struts.action.ActionMessage;
import org.apache.struts.action.ActionMessages;
-import org.apache.struts.upload.FormFile;
import org.lamsfoundation.lams.authoring.web.AuthoringConstants;
-import org.lamsfoundation.lams.contentrepository.client.IToolContentHandler;
import org.lamsfoundation.lams.tool.ToolAccessMode;
import org.lamsfoundation.lams.tool.taskList.TaskListConstants;
import org.lamsfoundation.lams.tool.taskList.model.TaskList;
-import org.lamsfoundation.lams.tool.taskList.model.TaskListAttachment;
import org.lamsfoundation.lams.tool.taskList.model.TaskListCondition;
import org.lamsfoundation.lams.tool.taskList.model.TaskListItem;
import org.lamsfoundation.lams.tool.taskList.model.TaskListUser;
import org.lamsfoundation.lams.tool.taskList.service.ITaskListService;
import org.lamsfoundation.lams.tool.taskList.service.TaskListException;
-import org.lamsfoundation.lams.tool.taskList.service.UploadTaskListFileException;
import org.lamsfoundation.lams.tool.taskList.util.TaskListConditionComparator;
import org.lamsfoundation.lams.tool.taskList.util.TaskListItemComparator;
import org.lamsfoundation.lams.tool.taskList.web.form.TaskListForm;
import org.lamsfoundation.lams.tool.taskList.web.form.TaskListItemForm;
import org.lamsfoundation.lams.tool.taskList.web.form.TaskListPedagogicalPlannerForm;
import org.lamsfoundation.lams.usermanagement.dto.UserDTO;
-import org.lamsfoundation.lams.util.FileValidatorUtil;
import org.lamsfoundation.lams.util.WebUtil;
import org.lamsfoundation.lams.web.session.SessionManager;
import org.lamsfoundation.lams.web.util.AttributeNames;
@@ -126,18 +120,6 @@
if (param.equals("updateContent")) {
return updateContent(mapping, form, request, response);
}
- if (param.equals("uploadOnlineFile")) {
- return uploadOnline(mapping, form, request, response);
- }
- if (param.equals("uploadOfflineFile")) {
- return uploadOffline(mapping, form, request, response);
- }
- if (param.equals("deleteOnlineFile")) {
- return deleteOnlineFile(mapping, form, request, response);
- }
- if (param.equals("deleteOfflineFile")) {
- return deleteOfflineFile(mapping, form, request, response);
- }
// ----------------------- Add taskListItem methods ---------------------------
if (param.equals("newItemInit")) {
@@ -220,11 +202,6 @@
}
taskListForm.setTaskList(taskList);
-
- // initialize instruction attachment list
- List attachmentList = getAttachmentList(sessionMap);
- attachmentList.clear();
- attachmentList.addAll(taskList.getAttachments());
} catch (Exception e) {
AuthoringAction.log.error(e);
throw new ServletException(e);
@@ -361,45 +338,6 @@
}
taskListPO.setCreatedBy(taskListUser);
-
- // **********************************Handle Authoring Instruction Attachement *********************
- // merge attachment info
- // so far, attPOSet will be empty if content is existed. because PropertyUtils.copyProperties() is executed
- Set attPOSet = taskListPO.getAttachments();
- if (attPOSet == null) {
- attPOSet = new HashSet();
- }
- List attachmentList = getAttachmentList(sessionMap);
- List deleteAttachmentList = getDeletedAttachmentList(sessionMap);
-
- // current attachemnt in authoring instruction tab.
- Iterator iter = attachmentList.iterator();
- while (iter.hasNext()) {
- TaskListAttachment newAtt = (TaskListAttachment) iter.next();
- attPOSet.add(newAtt);
- }
- attachmentList.clear();
-
- // deleted attachment. 2 possible types: one is persist another is non-persist before.
- iter = deleteAttachmentList.iterator();
- while (iter.hasNext()) {
- TaskListAttachment delAtt = (TaskListAttachment) iter.next();
- iter.remove();
- // it is an existed att, then delete it from current attachmentPO
- if (delAtt.getUid() != null) {
- Iterator attIter = attPOSet.iterator();
- while (attIter.hasNext()) {
- TaskListAttachment att = (TaskListAttachment) attIter.next();
- if (delAtt.getUid().equals(att.getUid())) {
- attIter.remove();
- break;
- }
- }
- service.deleteTaskListAttachment(delAtt.getUid());
- }// end remove from persist value
- }
- // copy back
- taskListPO.setAttachments(attPOSet);
// ************************* Handle taskList items *******************
Set itemList = new LinkedHashSet();
@@ -436,7 +374,7 @@
taskListPO.setConditions(conditions);
// delete TaskListConditions from database.
- iter = delConditions.iterator();
+ Iterator iter = delConditions.iterator();
while (iter.hasNext()) {
TaskListCondition condition = (TaskListCondition) iter.next();
iter.remove();
@@ -470,9 +408,6 @@
service.getTaskListByContentId(taskListPO.getContentId());
- // initialize attachmentList again
- attachmentList = getAttachmentList(sessionMap);
- attachmentList.addAll(taskList.getAttachments());
taskListForm.setTaskList(taskListPO);
request.setAttribute(AuthoringConstants.LAMS_AUTHORING_SUCCESS_FLAG, Boolean.TRUE);
@@ -483,166 +418,6 @@
}
}
- /**
- * Handle upload online instruction files request.
- *
- * @param mapping
- * @param form
- * @param request
- * @param response
- * @return
- * @throws UploadTaskListFileException
- */
- private ActionForward uploadOnline(ActionMapping mapping, ActionForm form, HttpServletRequest request,
- HttpServletResponse response) throws UploadTaskListFileException {
- return uploadFile(mapping, form, IToolContentHandler.TYPE_ONLINE, request);
- }
-
- /**
- * Handle upload offline instruction files request.
- *
- * @param mapping
- * @param form
- * @param request
- * @param response
- * @return
- * @throws UploadTaskListFileException
- */
- private ActionForward uploadOffline(ActionMapping mapping, ActionForm form, HttpServletRequest request,
- HttpServletResponse response) throws UploadTaskListFileException {
- return uploadFile(mapping, form, IToolContentHandler.TYPE_OFFLINE, request);
- }
-
- /**
- * Common method to upload online or offline instruction files request.
- *
- * @param mapping
- * @param form
- * @param type
- * @param request
- * @return
- * @throws UploadTaskListFileException
- */
- private ActionForward uploadFile(ActionMapping mapping, ActionForm form, String type, HttpServletRequest request)
- throws UploadTaskListFileException {
-
- TaskListForm taskListForm = (TaskListForm) form;
- // get back sessionMAP
- SessionMap sessionMap = (SessionMap) request.getSession().getAttribute(taskListForm.getSessionMapID());
-
- FormFile file;
- if (StringUtils.equals(IToolContentHandler.TYPE_OFFLINE, type)) {
- file = taskListForm.getOfflineFile();
- } else {
- file = taskListForm.getOnlineFile();
- }
-
- if (file == null || StringUtils.isBlank(file.getFileName())) {
- return mapping.findForward(TaskListConstants.SUCCESS);
- }
-
- // validate file size
- ActionMessages errors = new ActionMessages();
- FileValidatorUtil.validateFileSize(file, true, errors);
- if (!errors.isEmpty()) {
- this.saveErrors(request, errors);
- return mapping.findForward(TaskListConstants.SUCCESS);
- }
-
- ITaskListService service = getTaskListService();
- // upload to repository
- TaskListAttachment att = service.uploadInstructionFile(file, type);
- // handle session value
- List attachmentList = getAttachmentList(sessionMap);
- List deleteAttachmentList = getDeletedAttachmentList(sessionMap);
- // first check exist attachment and delete old one (if exist) to deletedAttachmentList
- Iterator iter = attachmentList.iterator();
- TaskListAttachment existAtt;
- while (iter.hasNext()) {
- existAtt = (TaskListAttachment) iter.next();
- if (StringUtils.equals(existAtt.getFileName(), att.getFileName())
- && StringUtils.equals(existAtt.getFileType(), att.getFileType())) {
- // if there is same name attachment, delete old one
- deleteAttachmentList.add(existAtt);
- iter.remove();
- break;
- }
- }
- // add to attachmentList
- attachmentList.add(att);
-
- return mapping.findForward(TaskListConstants.SUCCESS);
-
- }
-
- /**
- * Delete offline instruction file from current TaskList authoring page.
- *
- * @param mapping
- * @param form
- * @param request
- * @param response
- * @return
- */
- private ActionForward deleteOfflineFile(ActionMapping mapping, ActionForm form, HttpServletRequest request,
- HttpServletResponse response) {
- return deleteFile(mapping, request, response, form, IToolContentHandler.TYPE_OFFLINE);
- }
-
- /**
- * Delete online instruction file from current TaskList authoring page.
- *
- * @param mapping
- * @param form
- * @param request
- * @param response
- * @return
- */
- private ActionForward deleteOnlineFile(ActionMapping mapping, ActionForm form, HttpServletRequest request,
- HttpServletResponse response) {
- return deleteFile(mapping, request, response, form, IToolContentHandler.TYPE_ONLINE);
- }
-
- /**
- * General method to delete file (online or offline)
- *
- * @param mapping
- * @param request
- * @param response
- * @param form
- * @param type
- * @return
- */
- private ActionForward deleteFile(ActionMapping mapping, HttpServletRequest request, HttpServletResponse response,
- ActionForm form, String type) {
- Long versionID = new Long(WebUtil.readLongParam(request, TaskListConstants.PARAM_FILE_VERSION_ID));
- Long uuID = new Long(WebUtil.readLongParam(request, TaskListConstants.PARAM_FILE_UUID));
-
- // get back sessionMAP
- String sessionMapID = WebUtil.readStrParam(request, TaskListConstants.ATTR_SESSION_MAP_ID);
- SessionMap sessionMap = (SessionMap) request.getSession().getAttribute(sessionMapID);
-
- // handle session value
- List attachmentList = getAttachmentList(sessionMap);
- List deleteAttachmentList = getDeletedAttachmentList(sessionMap);
- // first check exist attachment and delete old one (if exist) to deletedAttachmentList
- Iterator iter = attachmentList.iterator();
- TaskListAttachment existAtt;
- while (iter.hasNext()) {
- existAtt = (TaskListAttachment) iter.next();
- if (existAtt.getFileUuid().equals(uuID) && existAtt.getFileVersionId().equals(versionID)) {
- // if there is same name attachment, delete old one
- deleteAttachmentList.add(existAtt);
- iter.remove();
- }
- }
-
- request.setAttribute(TaskListConstants.ATTR_FILE_TYPE_FLAG, type);
- request.setAttribute(TaskListConstants.ATTR_SESSION_MAP_ID, sessionMapID);
- return mapping.findForward(TaskListConstants.SUCCESS);
-
- }
-
// **********************************************************
// Add taskListItem methods
// **********************************************************
@@ -848,22 +623,6 @@
}
/**
- * @param request
- * @return
- */
- private List getAttachmentList(SessionMap sessionMap) {
- return getListFromSession(sessionMap, TaskListConstants.ATT_ATTACHMENT_LIST);
- }
-
- /**
- * @param request
- * @return
- */
- private List getDeletedAttachmentList(SessionMap sessionMap) {
- return getListFromSession(sessionMap, TaskListConstants.ATTR_DELETED_ATTACHMENT_LIST);
- }
-
- /**
* List save current taskList items.
*
* @param request
Index: lams_tool_task/src/java/org/lamsfoundation/lams/tool/taskList/web/action/LearningAction.java
===================================================================
RCS file: /usr/local/cvsroot/lams_tool_task/src/java/org/lamsfoundation/lams/tool/taskList/web/action/LearningAction.java,v
diff -u -r1.17 -r1.18
--- lams_tool_task/src/java/org/lamsfoundation/lams/tool/taskList/web/action/LearningAction.java 9 Apr 2013 12:45:55 -0000 1.17
+++ lams_tool_task/src/java/org/lamsfoundation/lams/tool/taskList/web/action/LearningAction.java 17 Jan 2014 22:18:13 -0000 1.18
@@ -26,14 +26,12 @@
import java.io.IOException;
import java.sql.Timestamp;
-import java.util.ArrayList;
import java.util.Comparator;
import java.util.Date;
-import java.util.HashSet;
import java.util.List;
import java.util.Set;
-import java.util.TreeSet;
import java.util.TimeZone;
+import java.util.TreeSet;
import javax.servlet.ServletException;
import javax.servlet.http.HttpServletRequest;
@@ -51,7 +49,6 @@
import org.apache.struts.action.ActionMessages;
import org.apache.struts.action.ActionRedirect;
import org.apache.struts.upload.FormFile;
-import org.lamsfoundation.lams.contentrepository.client.IToolContentHandler;
import org.lamsfoundation.lams.learning.web.bean.ActivityPositionDTO;
import org.lamsfoundation.lams.learning.web.util.LearningWebUtil;
import org.lamsfoundation.lams.notebook.model.NotebookEntry;
@@ -63,7 +60,6 @@
import org.lamsfoundation.lams.tool.taskList.model.TaskListItem;
import org.lamsfoundation.lams.tool.taskList.model.TaskListItemAttachment;
import org.lamsfoundation.lams.tool.taskList.model.TaskListItemComment;
-import org.lamsfoundation.lams.tool.taskList.model.TaskListItemVisitLog;
import org.lamsfoundation.lams.tool.taskList.model.TaskListSession;
import org.lamsfoundation.lams.tool.taskList.model.TaskListUser;
import org.lamsfoundation.lams.tool.taskList.service.ITaskListService;
@@ -328,15 +324,8 @@
ActivityPositionDTO activityPosition = LearningWebUtil.putActivityPositionInRequestByToolSessionId(sessionId,
request, getServlet().getServletContext());
sessionMap.put(AttributeNames.ATTR_ACTIVITY_POSITION, activityPosition);
-
- //add run offline support
- if(taskList.getRunOffline()){
- sessionMap.put(TaskListConstants.PARAM_RUN_OFFLINE, true);
- return mapping.findForward("runOffline");
- }else
- sessionMap.put(TaskListConstants.PARAM_RUN_OFFLINE, false);
- // check if there is submission deadline LDEV-2657
+ // check if there is submission deadline
Date submissionDeadline = taskList.getSubmissionDeadline();
if (submissionDeadline != null) {
HttpSession ss = SessionManager.getSession();
@@ -346,9 +335,9 @@
Date currentLearnerDate = DateUtil.convertToTimeZoneFromDefault(learnerTimeZone, new Date());
sessionMap.put(TaskListConstants.ATTR_SUBMISSION_DEADLINE, tzSubmissionDeadline);
- //calculate whether deadline has passed, and if so forward to "runOffline"
+ //calculate whether deadline has passed, and if so forward to "submissionDeadline"
if (currentLearnerDate.after(tzSubmissionDeadline)) {
- return mapping.findForward("runOffline");
+ return mapping.findForward("submissionDeadline");
}
}
@@ -574,7 +563,7 @@
ITaskListService service = getTaskListService();
UserDTO user = (UserDTO) SessionManager.getSession().getAttribute(AttributeNames.USER);
TaskListUser taskListUser = service.getUserByIDAndSession(new Long(user.getUserID().intValue()),sessionId);
- TaskListItemAttachment att = service.uploadTaskListItemFile(file, IToolContentHandler.TYPE_ONLINE, taskListUser);
+ TaskListItemAttachment att = service.uploadTaskListItemFile(file, taskListUser);
//persist TaskListItem changes in DB
Long itemUid = new Long(request.getParameter(TaskListConstants.PARAM_RESOURCE_ITEM_UID));
@@ -727,9 +716,7 @@
int numberCompletedTasks = service.getNumTasksCompletedByUser(sessionId, userID);
int minimumNumberTasks = service.getTaskListBySessionId(sessionId).getMinimumNumberTasks();
//if current user view less than reqired view count number, then just return error message.
- //if it is runOffline content, then need not check minimum view count
- Boolean runOffline = (Boolean) sessionMap.get(TaskListConstants.PARAM_RUN_OFFLINE);
- if((minimumNumberTasks - numberCompletedTasks) > 0 && !runOffline){
+ if((minimumNumberTasks - numberCompletedTasks) > 0){
ActionErrors errors = new ActionErrors();
errors.add(ActionMessages.GLOBAL_MESSAGE,new ActionMessage("lable.learning.minimum.view.number",minimumNumberTasks, numberCompletedTasks));
this.addErrors(request,errors);
Index: lams_tool_task/src/java/org/lamsfoundation/lams/tool/taskList/web/action/MonitoringAction.java
===================================================================
RCS file: /usr/local/cvsroot/lams_tool_task/src/java/org/lamsfoundation/lams/tool/taskList/web/action/MonitoringAction.java,v
diff -u -r1.15 -r1.16
--- lams_tool_task/src/java/org/lamsfoundation/lams/tool/taskList/web/action/MonitoringAction.java 17 May 2011 04:39:05 -0000 1.15
+++ lams_tool_task/src/java/org/lamsfoundation/lams/tool/taskList/web/action/MonitoringAction.java 17 Jan 2014 22:18:13 -0000 1.16
@@ -25,8 +25,8 @@
package org.lamsfoundation.lams.tool.taskList.web.action;
import java.io.IOException;
-import java.util.List;
import java.util.Date;
+import java.util.List;
import java.util.TimeZone;
import javax.servlet.ServletException;
@@ -43,21 +43,19 @@
import org.lamsfoundation.lams.notebook.model.NotebookEntry;
import org.lamsfoundation.lams.notebook.service.CoreNotebookConstants;
import org.lamsfoundation.lams.tool.taskList.TaskListConstants;
-import org.lamsfoundation.lams.tool.taskList.dto.GroupSummary;
import org.lamsfoundation.lams.tool.taskList.dto.ItemSummary;
import org.lamsfoundation.lams.tool.taskList.dto.ReflectDTO;
-import org.lamsfoundation.lams.tool.taskList.dto.TaskListItemVisitLogSummary;
import org.lamsfoundation.lams.tool.taskList.dto.Summary;
import org.lamsfoundation.lams.tool.taskList.model.TaskList;
import org.lamsfoundation.lams.tool.taskList.model.TaskListSession;
import org.lamsfoundation.lams.tool.taskList.model.TaskListUser;
import org.lamsfoundation.lams.tool.taskList.service.ITaskListService;
import org.lamsfoundation.lams.usermanagement.dto.UserDTO;
-import org.lamsfoundation.lams.util.WebUtil;
import org.lamsfoundation.lams.util.DateUtil;
+import org.lamsfoundation.lams.util.WebUtil;
+import org.lamsfoundation.lams.web.session.SessionManager;
import org.lamsfoundation.lams.web.util.AttributeNames;
import org.lamsfoundation.lams.web.util.SessionMap;
-import org.lamsfoundation.lams.web.session.SessionManager;
import org.springframework.web.context.WebApplicationContext;
import org.springframework.web.context.support.WebApplicationContextUtils;
@@ -103,7 +101,6 @@
ITaskListService service = getTaskListService();
TaskList taskList = service.getTaskListByContentId(contentId);
- taskList.toDTO();
List
+
+
Ticket
to login and
- * access the Content Repository.
- *
- * A valid ticket is needed in order to access the content from the repository. This method would be called evertime
- * the tool needs to upload/download files from the content repository.
- *
- * @return ITicket The ticket for repostory access
- * @throws SubmitFilesException
- */
- private ITicket getRepositoryLoginTicket() throws VideoRecorderException {
- ICredentials credentials = new SimpleCredentials(VideoRecorderToolContentHandler.repositoryUser,
- VideoRecorderToolContentHandler.repositoryId);
- try {
- ITicket ticket = repositoryService.login(credentials,
- VideoRecorderToolContentHandler.repositoryWorkspaceName);
- return ticket;
- } catch (AccessDeniedException ae) {
- throw new VideoRecorderException("Access Denied to repository." + ae.getMessage());
- } catch (WorkspaceNotFoundException we) {
- throw new VideoRecorderException("Workspace not found." + we.getMessage());
- } catch (LoginException e) {
- throw new VideoRecorderException("Login failed." + e.getMessage());
- }
- }
-
/* ===============Methods implemented from ToolContentImport102Manager =============== */
/**
@@ -649,15 +532,9 @@
videoRecorder.setInstructions(WebUtil.convertNewlines((String) importValues
.get(ToolContentImport102Manager.CONTENT_BODY)));
videoRecorder.setLockOnFinished(Boolean.TRUE);
- videoRecorder.setOfflineInstructions(null);
- videoRecorder.setOnlineInstructions(null);
- videoRecorder.setRunOffline(Boolean.FALSE);
videoRecorder.setTitle((String) importValues.get(ToolContentImport102Manager.CONTENT_TITLE));
videoRecorder.setToolContentId(toolContentId);
videoRecorder.setUpdateDate(now);
- // leave as empty, no need to set them to anything.
- // setVideoRecorderAttachments(Set videoRecorderAttachments);
- // setVideoRecorderSessions(Set videoRecorderSessions);
videoRecorderDAO.saveOrUpdate(videoRecorder);
}
@@ -679,14 +556,6 @@
// =========================================================================================
/* ********** Used by Spring to "inject" the linked objects ************* */
- public IVideoRecorderAttachmentDAO getVideoRecorderAttachmentDAO() {
- return videoRecorderAttachmentDAO;
- }
-
- public void setVideoRecorderAttachmentDAO(IVideoRecorderAttachmentDAO attachmentDAO) {
- videoRecorderAttachmentDAO = attachmentDAO;
- }
-
public IVideoRecorderDAO getVideoRecorderDAO() {
return videoRecorderDAO;
}
@@ -791,14 +660,6 @@
this.videoRecorderOutputFactory = videoRecorderOutputFactory;
}
- public IRepositoryService getRepositoryService() {
- return repositoryService;
- }
-
- public void setRepositoryService(IRepositoryService repositoryService) {
- this.repositoryService = repositoryService;
- }
-
public void releaseConditionsFromCache(VideoRecorder videoRecorder) {
if (videoRecorder.getConditions() != null) {
for (VideoRecorderCondition condition : videoRecorder.getConditions()) {
Index: lams_tool_videorecorder/src/java/org/lamsfoundation/lams/tool/videoRecorder/util/VideoRecorderConstants.java
===================================================================
RCS file: /usr/local/cvsroot/lams_tool_videorecorder/src/java/org/lamsfoundation/lams/tool/videoRecorder/util/VideoRecorderConstants.java,v
diff -u -r1.3 -r1.4
--- lams_tool_videorecorder/src/java/org/lamsfoundation/lams/tool/videoRecorder/util/VideoRecorderConstants.java 2 Jul 2009 08:18:29 -0000 1.3
+++ lams_tool_videorecorder/src/java/org/lamsfoundation/lams/tool/videoRecorder/util/VideoRecorderConstants.java 17 Jan 2014 22:18:17 -0000 1.4
@@ -33,8 +33,6 @@
public static final Integer SESSION_COMPLETED = new Integer(2);
public static final String AUTHORING_DEFAULT_TAB = "1";
- public static final String ATTACHMENT_LIST = "attachmentList";
- public static final String DELETED_ATTACHMENT_LIST = "deletedAttachmentList";
public static final String AUTH_SESSION_ID_COUNTER = "authoringSessionIdCounter";
public static final String AUTH_SESSION_ID = "authoringSessionId";
Index: lams_tool_videorecorder/src/java/org/lamsfoundation/lams/tool/videoRecorder/web/actions/AuthoringAction.java
===================================================================
RCS file: /usr/local/cvsroot/lams_tool_videorecorder/src/java/org/lamsfoundation/lams/tool/videoRecorder/web/actions/AuthoringAction.java,v
diff -u -r1.4 -r1.5
--- lams_tool_videorecorder/src/java/org/lamsfoundation/lams/tool/videoRecorder/web/actions/AuthoringAction.java 24 Apr 2009 02:43:31 -0000 1.4
+++ lams_tool_videorecorder/src/java/org/lamsfoundation/lams/tool/videoRecorder/web/actions/AuthoringAction.java 17 Jan 2014 22:18:18 -0000 1.5
@@ -52,14 +52,9 @@
import org.lamsfoundation.lams.tool.ToolSessionManager;
import org.lamsfoundation.lams.tool.videoRecorder.dto.VideoRecorderDTO;
import org.lamsfoundation.lams.tool.videoRecorder.dto.VideoRecorderRecordingDTO;
-import org.lamsfoundation.lams.tool.videoRecorder.dto.VideoRecorderSessionDTO;
import org.lamsfoundation.lams.tool.videoRecorder.model.VideoRecorder;
-import org.lamsfoundation.lams.tool.videoRecorder.model.VideoRecorderAttachment;
import org.lamsfoundation.lams.tool.videoRecorder.model.VideoRecorderCondition;
-import org.lamsfoundation.lams.tool.videoRecorder.model.VideoRecorderSession;
-import org.lamsfoundation.lams.tool.videoRecorder.model.VideoRecorderUser;
import org.lamsfoundation.lams.tool.videoRecorder.service.IVideoRecorderService;
-import org.lamsfoundation.lams.tool.videoRecorder.service.VideoRecorderService;
import org.lamsfoundation.lams.tool.videoRecorder.service.VideoRecorderServiceProxy;
import org.lamsfoundation.lams.tool.videoRecorder.util.VideoRecorderConstants;
import org.lamsfoundation.lams.tool.videoRecorder.web.forms.AuthoringForm;
@@ -90,11 +85,6 @@
private static final String KEY_TOOL_CONTENT_ID = "toolContentID";
private static final String KEY_CONTENT_FOLDER_ID = "contentFolderID";
private static final String KEY_MODE = "mode";
- private static final String KEY_ONLINE_FILES = "onlineFiles";
- private static final String KEY_OFFLINE_FILES = "offlineFiles";
- private static final String KEY_UNSAVED_ONLINE_FILES = "unsavedOnlineFiles";
- private static final String KEY_UNSAVED_OFFLINE_FILES = "unsavedOfflineFiles";
- private static final String KEY_DELETED_FILES = "deletedFiles";
/**
* Default method when no dispatch parameter is specified. It is expected that the parameter
@@ -201,24 +191,6 @@
ToolAccessMode mode = (ToolAccessMode) map.get(AuthoringAction.KEY_MODE);
updateVideoRecorder(videoRecorder, authForm, mode);
- // remove attachments marked for deletion.
- Set
+
+
+
+
-
-
-
- - | +|
+ + | -
- |
-
Ticket
to login and access
- * the Content Repository.
*
- * A valid ticket is needed in order to access the content from the repository. This method would be called evertime
- * the tool needs to upload/download files from the content repository.
- *
- * @return ITicket The ticket for repostory access
- * @throws SubmitFilesException
- */
- private ITicket getRepositoryLoginTicket() throws WikiException {
- ICredentials credentials = new SimpleCredentials(WikiToolContentHandler.repositoryUser,
- WikiToolContentHandler.repositoryId);
- try {
- ITicket ticket = repositoryService.login(credentials, WikiToolContentHandler.repositoryWorkspaceName);
- return ticket;
- } catch (AccessDeniedException ae) {
- throw new WikiException("Access Denied to repository." + ae.getMessage());
- } catch (WorkspaceNotFoundException we) {
- throw new WikiException("Workspace not found." + we.getMessage());
- } catch (LoginException e) {
- throw new WikiException("Login failed." + e.getMessage());
- }
- }
-
- /**
- *
* @param learnerId
* @param toolSessionId
* @return
@@ -934,9 +802,6 @@
wiki.setInstructions(WebUtil.convertNewlines((String) importValues
.get(ToolContentImport102Manager.CONTENT_BODY)));
wiki.setLockOnFinished(Boolean.TRUE);
- wiki.setOfflineInstructions(null);
- wiki.setOnlineInstructions(null);
- wiki.setRunOffline(Boolean.FALSE);
wiki.setTitle((String) importValues.get(ToolContentImport102Manager.CONTENT_TITLE));
wiki.setToolContentId(toolContentId);
wiki.setUpdateDate(now);
@@ -948,9 +813,6 @@
wiki.setMaximumEdits(0);
wiki.setMinimumEdits(0);
- // leave as empty, no need to set them to anything.
- // setWikiAttachments(Set wikiAttachments);
- // setWikiSessions(Set wikiSessions);
wikiDAO.saveOrUpdate(wiki);
}
@@ -974,14 +836,6 @@
// =========================================================================================
/* ********** Used by Spring to "inject" the linked objects ************* */
- public IWikiAttachmentDAO getWikiAttachmentDAO() {
- return wikiAttachmentDAO;
- }
-
- public void setWikiAttachmentDAO(IWikiAttachmentDAO attachmentDAO) {
- wikiAttachmentDAO = attachmentDAO;
- }
-
public IWikiDAO getWikiDAO() {
return wikiDAO;
}
@@ -1106,14 +960,6 @@
this.lessonService = lessonService;
}
- public IRepositoryService getRepositoryService() {
- return repositoryService;
- }
-
- public void setRepositoryService(IRepositoryService repositoryService) {
- this.repositoryService = repositoryService;
- }
-
public Class[] getSupportedToolOutputDefinitionClasses(int definitionType) {
return getWikiOutputFactory().getSupportedDefinitionClasses(definitionType);
}
Index: lams_tool_wiki/src/java/org/lamsfoundation/lams/tool/wiki/util/WikiConstants.java
===================================================================
RCS file: /usr/local/cvsroot/lams_tool_wiki/src/java/org/lamsfoundation/lams/tool/wiki/util/WikiConstants.java,v
diff -u -r1.8 -r1.9
--- lams_tool_wiki/src/java/org/lamsfoundation/lams/tool/wiki/util/WikiConstants.java 12 Mar 2012 20:34:38 -0000 1.8
+++ lams_tool_wiki/src/java/org/lamsfoundation/lams/tool/wiki/util/WikiConstants.java 17 Jan 2014 22:18:21 -0000 1.9
@@ -33,8 +33,6 @@
public static final Integer SESSION_COMPLETED = new Integer(2);
public static final String AUTHORING_DEFAULT_TAB = "1";
- public static final String ATTACHMENT_LIST = "attachmentList";
- public static final String DELETED_ATTACHMENT_LIST = "deletedAttachmentList";
public static final String AUTH_SESSION_ID_COUNTER = "authoringSessionIdCounter";
public static final String AUTH_SESSION_ID = "authoringSessionId";
Index: lams_tool_wiki/src/java/org/lamsfoundation/lams/tool/wiki/web/actions/AuthoringAction.java
===================================================================
RCS file: /usr/local/cvsroot/lams_tool_wiki/src/java/org/lamsfoundation/lams/tool/wiki/web/actions/AuthoringAction.java,v
diff -u -r1.6 -r1.7
--- lams_tool_wiki/src/java/org/lamsfoundation/lams/tool/wiki/web/actions/AuthoringAction.java 19 Mar 2012 14:39:35 -0000 1.6
+++ lams_tool_wiki/src/java/org/lamsfoundation/lams/tool/wiki/web/actions/AuthoringAction.java 17 Jan 2014 22:18:20 -0000 1.7
@@ -49,7 +49,6 @@
import org.lamsfoundation.lams.tool.wiki.dto.WikiPageContentDTO;
import org.lamsfoundation.lams.tool.wiki.dto.WikiPageDTO;
import org.lamsfoundation.lams.tool.wiki.model.Wiki;
-import org.lamsfoundation.lams.tool.wiki.model.WikiAttachment;
import org.lamsfoundation.lams.tool.wiki.model.WikiPage;
import org.lamsfoundation.lams.tool.wiki.model.WikiPageContent;
import org.lamsfoundation.lams.tool.wiki.model.WikiUser;
@@ -91,16 +90,6 @@
private static final String KEY_MODE = "mode";
- private static final String KEY_ONLINE_FILES = "onlineFiles";
-
- private static final String KEY_OFFLINE_FILES = "offlineFiles";
-
- private static final String KEY_UNSAVED_ONLINE_FILES = "unsavedOnlineFiles";
-
- private static final String KEY_UNSAVED_OFFLINE_FILES = "unsavedOfflineFiles";
-
- private static final String KEY_DELETED_FILES = "deletedFiles";
-
/**
* Default method when no dispatch parameter is specified. It is expected that the parameter
* toolContentID
will be passed in. This will be used to retrieve content for this tool.
@@ -264,24 +253,6 @@
ToolAccessMode mode = (ToolAccessMode) map.get(AuthoringAction.KEY_MODE);
updateWiki(wiki, authForm, mode);
- // remove attachments marked for deletion.
- Set