ylqjgm
/
canvas-lms
mirror of https://github.com/instructure/canvas-lms.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

fix group lti variables with non-group assignments 

closes #PLAT-3681

Change-Id: I0d55e46e52a592d83b005fb4fda5b4c8bd3fd148
Reviewed-on: https://gerrit.instructure.com/163056
Tested-by: Jenkins
Reviewed-by: Cody Cutrer <cody@instructure.com>
Product-Review: James Williams  <jamesw@instructure.com>
QA-Review: James Williams  <jamesw@instructure.com>
This commit is contained in:
James Williams 2018-09-04 09:02:03 -06:00
parent e27648e478
commit 1a82392bf3
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
lib/lti/variable_expander.rb
View File

@ -996,7 +996,7 @@ module Lti
# 481
# ```
register_expansion 'com.instructure.Group.id', [],
-> { (@assignment.group_category&.groups & @current_user.groups).first&.id },
-> { @assignment.group_category && (@assignment.group_category.groups & @current_user.groups).first&.id },
USER_GUARD,
ASSIGNMENT_GUARD,
default_name: 'vnd_canvas_group_id'
@ -1009,7 +1009,7 @@ module Lti
# Group One
# ```
register_expansion 'com.instructure.Group.name', [],
-> { (@assignment.group_category&.groups & @current_user.groups).first&.name },
-> { @assignment.group_category && (@assignment.group_category.groups & @current_user.groups).first&.name },
USER_GUARD,
ASSIGNMENT_GUARD,
default_name: 'vnd_canvas_group_name'