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: [