initialize grade columns in the LMS according to the grade passback m…

…ode and the nature of the LTI link
openwebwork · Sep 3, 2024 · b5d80a3 · b5d80a3
1 parent 3dc16db
commit b5d80a3
Show file tree

Hide file tree

Showing 2 changed files with 19 additions and 11 deletions.
diff --git a/lib/WeBWorK/ContentGenerator/LTIAdvanced.pm b/lib/WeBWorK/ContentGenerator/LTIAdvanced.pm
@@ -129,8 +129,11 @@ sub content_selection ($c) {
 						? {
 							'@type'   => 'LtiLinkItem',
 							mediaType => 'application/vnd.ims.lti.v1.ltilink',
-							title     => $c->maketext('Assignments'),
-							url       => $c->url_for('set_list', courseID => $c->stash->{courseID})->to_abs->to_string
+							title     => $c->maketext('WeBWorK Assignments'),
+							url       => $c->url_for('set_list', courseID => $c->stash->{courseID})->to_abs->to_string,
+							$c->ce->{LTIGradeMode} eq 'course'
+								? (lineItem => {scoreMaximum => 100})
+								: ()
 							}
 						: (),
 						map { {
@@ -141,12 +144,14 @@ sub content_selection ($c) {
 							url =>
 								$c->url_for('problem_list', courseID => $c->stash->{courseID}, setID => $_->set_id)
 								->to_abs->to_string,
-							lineItem => {
-								'@type'          => 'LineItem',
-								scoreConstraints => {
-									'@type'       => 'NumericLimits',
-									normalMaximum => $setMaxScores{ $_->set_id }
-								}
+							$c->ce->{LTIGradeMode} eq 'homework'
+								? (lineItem => {
+									'@type'          => 'LineItem',
+									scoreConstraints => {
+										'@type'       => 'NumericLimits',
+										normalMaximum => $setMaxScores{ $_->set_id })
+									}
+								: ()
 							}
 						} } @selectedSets
 					]

diff --git a/lib/WeBWorK/ContentGenerator/LTIAdvantage.pm b/lib/WeBWorK/ContentGenerator/LTIAdvantage.pm
@@ -266,8 +266,9 @@ sub content_selection ($c) {
 						$c->param('course_home_link')
 						? {
 							type  => 'ltiResourceLink',
-							title => $c->maketext('Assignments'),
-							url   => $c->url_for('set_list', courseID => $c->stash->{courseID})->to_abs->to_string
+							title => $c->maketext('WeBWorK Assignments'),
+							url   => $c->url_for('set_list', courseID => $c->stash->{courseID})->to_abs->to_string,
+							$c->ce->{LTIGradeMode} eq 'course' ? (lineItem => { scoreMaximum => 100 }) : ()
 							}
 						: (),
 						map { {
@@ -277,7 +278,9 @@ sub content_selection ($c) {
 							url =>
 								$c->url_for('problem_list', courseID => $c->stash->{courseID}, setID => $_->set_id)
 								->to_abs->to_string,
-							lineItem => { scoreMaximum => $setMaxScores{ $_->set_id } }
+							$c->ce->{LTIGradeMode} eq 'homework'
+							? (lineItem => { scoreMaximum => $setMaxScores{ $_->set_id } })
+							: ()
 						} } @selectedSets
 					]
 					)