Index: lams_gradebook/src/java/org/lamsfoundation/lams/gradebook/dto/GBActivityGridRowDTO.java =================================================================== diff -u -re074865bde063cd3ac9e531a90857d07de9d121c -rd800327041497bb1a053493a53c450512121b951 --- lams_gradebook/src/java/org/lamsfoundation/lams/gradebook/dto/GBActivityGridRowDTO.java (.../GBActivityGridRowDTO.java) (revision e074865bde063cd3ac9e531a90857d07de9d121c) +++ lams_gradebook/src/java/org/lamsfoundation/lams/gradebook/dto/GBActivityGridRowDTO.java (.../GBActivityGridRowDTO.java) (revision d800327041497bb1a053493a53c450512121b951) @@ -62,7 +62,7 @@ ret.add(rowName); } ret.add(status); - ret.add(output); + ret.add((output != null) ? output.toString() : CELL_EMPTY); ret.add(competences); ret.add((timeTaken != null) ? convertTimeToString(timeTaken) : CELL_EMPTY); ret.add(feedback); Index: lams_gradebook/src/java/org/lamsfoundation/lams/gradebook/dto/GBUserGridRowDTO.java =================================================================== diff -u -re074865bde063cd3ac9e531a90857d07de9d121c -rd800327041497bb1a053493a53c450512121b951 --- lams_gradebook/src/java/org/lamsfoundation/lams/gradebook/dto/GBUserGridRowDTO.java (.../GBUserGridRowDTO.java) (revision e074865bde063cd3ac9e531a90857d07de9d121c) +++ lams_gradebook/src/java/org/lamsfoundation/lams/gradebook/dto/GBUserGridRowDTO.java (.../GBUserGridRowDTO.java) (revision d800327041497bb1a053493a53c450512121b951) @@ -64,7 +64,7 @@ } ret.add(status); ret.add((timeTaken != null) ? convertTimeToString(timeTaken) : CELL_EMPTY); - ret.add(output); + ret.add((output != null) ? output.toString() : CELL_EMPTY); ret.add(feedback); ret.add((mark != null) ? mark.toString() : CELL_EMPTY); Index: lams_gradebook/src/java/org/lamsfoundation/lams/gradebook/service/GradebookService.java =================================================================== diff -u -rcd5cef591859ae82025af74f79cf6bf6f9151568 -rd800327041497bb1a053493a53c450512121b951 --- lams_gradebook/src/java/org/lamsfoundation/lams/gradebook/service/GradebookService.java (.../GradebookService.java) (revision cd5cef591859ae82025af74f79cf6bf6f9151568) +++ lams_gradebook/src/java/org/lamsfoundation/lams/gradebook/service/GradebookService.java (.../GradebookService.java) (revision d800327041497bb1a053493a53c450512121b951) @@ -200,14 +200,17 @@ // Get the tool outputs for this user if there are any ToolSession toolSession = toolService.getToolSessionByLearner(learner, activity); - if (toolSession != null) { + if (toolSession != null && learnerProgress != null) { // Set the activityLearner URL for this gradebook activity - gUserDTO.setActivityUrl(Configuration.get(ConfigurationKeys.SERVER_URL) - + activity.getTool().getLearnerProgressUrl() + "&userID=" + learner.getUserId() - + "&toolSessionID=" + toolSession.getToolSessionId().toString()); + byte activityState = learnerProgress.getProgressState(activity); + if (activityState == LearnerProgress.ACTIVITY_ATTEMPTED + || activityState == LearnerProgress.ACTIVITY_COMPLETED) { + gUserDTO.setActivityUrl(Configuration.get(ConfigurationKeys.SERVER_URL) + + activity.getTool().getLearnerProgressUrl() + "&userID=" + learner.getUserId() + + "&toolSessionID=" + toolSession.getToolSessionId().toString()); - gUserDTO.setOutput(this.getToolOutputsStr(activity, toolSession, learner)); - + gUserDTO.setOutput(this.getToolOutputsStr(activity, toolSession, learner)); + } } // Add marks and feedback @@ -545,13 +548,16 @@ // Get the tool outputs for this user if there are any ToolSession toolSession = toolService.getToolSessionByLearner(learner, activity); - if (toolSession != null) { - // Set the activityLearner URL for this gradebook activity - gactivityDTO.setActivityUrl(Configuration.get(ConfigurationKeys.SERVER_URL) - + activity.getTool().getLearnerProgressUrl() + "&userID=" + learner.getUserId() + "&toolSessionID=" - + toolSession.getToolSessionId().toString()); - - gactivityDTO.setOutput(this.getToolOutputsStr(activity, toolSession, learner)); + if (toolSession != null && learnerProgress != null) { + byte activityState = learnerProgress.getProgressState(activity); + if (activityState == LearnerProgress.ACTIVITY_ATTEMPTED + || activityState == LearnerProgress.ACTIVITY_COMPLETED) { + // Set the activityLearner URL for this gradebook activity + gactivityDTO.setActivityUrl(Configuration.get(ConfigurationKeys.SERVER_URL) + + activity.getTool().getLearnerProgressUrl() + "&userID=" + learner.getUserId() + + "&toolSessionID=" + toolSession.getToolSessionId().toString()); + gactivityDTO.setOutput(this.getToolOutputsStr(activity, toolSession, learner)); + } } return gactivityDTO; Index: lams_gradebook/web/gradebookCourseLearner.jsp =================================================================== diff -u -r824584ee2b82bea619fbe38b470de7cca15868ea -rd800327041497bb1a053493a53c450512121b951 --- lams_gradebook/web/gradebookCourseLearner.jsp (.../gradebookCourseLearner.jsp) (revision 824584ee2b82bea619fbe38b470de7cca15868ea) +++ lams_gradebook/web/gradebookCourseLearner.jsp (.../gradebookCourseLearner.jsp) (revision d800327041497bb1a053493a53c450512121b951) @@ -83,7 +83,7 @@ '', '', '', - '', + '', '' ], colModel: [