Index: lams_tool_laqa/src/java/org/lamsfoundation/lams/tool/qa/service/IQaService.java =================================================================== diff -u -r9310fe30da9974d1f174319efc054e3bde9eed84 -r154b350b57ebb41194f828c02d4a5f1a1b9d3242 --- lams_tool_laqa/src/java/org/lamsfoundation/lams/tool/qa/service/IQaService.java (.../IQaService.java) (revision 9310fe30da9974d1f174319efc054e3bde9eed84) +++ lams_tool_laqa/src/java/org/lamsfoundation/lams/tool/qa/service/IQaService.java (.../IQaService.java) (revision 154b350b57ebb41194f828c02d4a5f1a1b9d3242) @@ -179,15 +179,15 @@ public void createToolSession(Long toolSessionId, Long toolContentId) throws ToolException; /** - * leaveToolSession(Long toolSessionId, Long learnerId) should ideally should not be part this interface as it is + * leaveToolSession(Long toolSessionId, User learner) should ideally should not be part this interface as it is * already part of the interface ToolSessionManager. It is here for development purposes. * * It is also defined here since in development we want to be able call it directly from our web-layer * instead of it being called by the container. * @param toolSessionId * @param toolContentId */ - public String leaveToolSession(Long toolSessionId,Long learnerId) throws DataMissingException, ToolException; + public String leaveToolSession(Long toolSessionId,User learner) throws DataMissingException, ToolException; public BasicToolVO getToolBySignature(String toolSignature); Index: lams_tool_laqa/src/java/org/lamsfoundation/lams/tool/qa/service/QaServicePOJO.java =================================================================== diff -u -r9310fe30da9974d1f174319efc054e3bde9eed84 -r154b350b57ebb41194f828c02d4a5f1a1b9d3242 --- lams_tool_laqa/src/java/org/lamsfoundation/lams/tool/qa/service/QaServicePOJO.java (.../QaServicePOJO.java) (revision 9310fe30da9974d1f174319efc054e3bde9eed84) +++ lams_tool_laqa/src/java/org/lamsfoundation/lams/tool/qa/service/QaServicePOJO.java (.../QaServicePOJO.java) (revision 154b350b57ebb41194f828c02d4a5f1a1b9d3242) @@ -40,7 +40,6 @@ import org.lamsfoundation.lams.contentrepository.service.IRepositoryService; import org.lamsfoundation.lams.contentrepository.service.RepositoryProxy; import org.lamsfoundation.lams.contentrepository.service.SimpleCredentials; -import org.lamsfoundation.lams.learning.service.ILearnerService; import org.lamsfoundation.lams.lesson.Lesson; import org.lamsfoundation.lams.tool.BasicToolVO; import org.lamsfoundation.lams.tool.ToolContentManager; @@ -111,7 +110,6 @@ private IUserManagementService userManagementService; private ILamsToolService toolService; - private ILearnerService learnerService; public void configureContentRepository() throws QaApplicationException { logger.debug("retrieved repService: " + repositoryService); @@ -1321,28 +1319,31 @@ * Call controller service to complete the qa session * */ - public String leaveToolSession(Long toolSessionId,Long learnerId) throws DataMissingException, ToolException + public String leaveToolSession(Long toolSessionId,User learner) throws DataMissingException, ToolException { logger.debug("start of leaveToolSession with toolSessionId:" + toolSessionId); - logger.debug("start of leaveToolSession with learnerId:" + learnerId); + logger.debug("start of leaveToolSession with learner:" + learner); if (toolSessionId == null) { logger.debug("toolSessionId is null"); throw new DataMissingException("toolSessionId is missing"); } - if (learnerId == null) + if (learner == null) { - logger.debug("learnerId is null"); - throw new DataMissingException("learnerId is missing"); + logger.debug("learner is null"); + throw new DataMissingException("learner is missing"); } try { - String nextUrl = learnerService.completeToolSession(toolSessionId,learnerId); + /* + String nextUrl=learnerService.completeToolSession(toolSessionId,learner); logger.debug(logger + " " + this.getClass().getName() + " " + "nextUrl: " + nextUrl); return nextUrl; + */ + return "nextUrl"; } catch(DataAccessException e) { @@ -1612,14 +1613,4 @@ public IUserManagementService getUserManagementService() { return userManagementService; } - - - public ILearnerService getLearnerService() { - return learnerService; - } - - - public void setLearnerService(ILearnerService learnerService) { - this.learnerService = learnerService; - } }