Merge pull request #2811 from dlglin/fixignoreMissingSourcedID

pstaabp · web-flow · commit de0407a1d1cf · 2025-09-09T15:27:12.000-04:00
Fix checkSet to honour the ignoreMissingSourcedID setting
diff --git a/lib/WeBWorK/Authz.pm b/lib/WeBWorK/Authz.pm
@@ -490,7 +490,7 @@ sub checkSet {
 				. 'Try logging in to the Learning Management System and visiting the set from there.',
 			$LMS
 		))
-			unless $set->lis_source_did;
+			unless $set->lis_source_did || ($ce->{LTIVersion} eq 'v1p3' && $ce->{LTI}{v1p3}{ignoreMissingSourcedID});
 	}
 
 	return 0;

Original file line number	Diff line number	Diff line change
`@@ -490,7 +490,7 @@ sub checkSet {`
`490`	`490`	`. 'Try logging in to the Learning Management System and visiting the set from there.',`
`491`	`491`	`$LMS`
`492`	`492`	`))`
`493`		`- unless $set->lis_source_did;`
	`493`	`+ unless $set->lis_source_did \|\| ($ce->{LTIVersion} eq 'v1p3' && $ce->{LTI}{v1p3}{ignoreMissingSourcedID});`
`494`	`494`	`}`
`495`	`495`
`496`	`496`	`return 0;`