Andal.LND/edx-platform
6
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

PR to fix issue loading dashboard when there is a version mismatch in

Browse Source 
CourseOverview

    This line of code will cause a failure in the retrieval of the
    CourseOverview get_from_id method because it attempts to load with a
    string rather than a CourseKey.
This commit is contained in:
Albert (AJ) St. Aubin
2020-05-04 21:52:20 -04:00
parent d04f2735a1
commit bf3aaffe4d
2 changed files with 8 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
lms/templates/dashboard.html
View File

@@ -183,7 +183,10 @@ from student.models import CourseEnrollment
pseudo_session = unfulfilled_entitlement_pseudo_sessions[str(entitlement.uuid)]
if not pseudo_session:
continue
enrollment = CourseEnrollment(user=user, course=CourseOverview.get_from_id(pseudo_session['key']), mode=pseudo_session['type'])
pseudo_key = pseudo_session['key']
if not isinstance(pseudo_key, CourseKey):
pseudo_key = CourseKey.from_string(pseudo_session['key'])
enrollment = CourseEnrollment(user=user, course=CourseOverview.get_from_id(pseudo_key), mode=pseudo_session['type'])
# We only show email settings for entitlement cards if the entitlement has an associated enrollment
show_email_settings = is_fulfilled_entitlement and (entitlement_session.course_id in show_email_settings_for)
else:

5
themes/edx.org/lms/templates/dashboard.html
View File

@@ -179,7 +179,10 @@ from student.models import CourseEnrollment
pseudo_session = unfulfilled_entitlement_pseudo_sessions[str(entitlement.uuid)]
if not pseudo_session:
continue
enrollment = CourseEnrollment(user=user, course=CourseOverview.get_from_id(pseudo_session['key']), mode=pseudo_session['type'])
pseudo_key = pseudo_session['key']
if not isinstance(pseudo_key, CourseKey):
pseudo_key = CourseKey.from_string(pseudo_session['key'])
enrollment = CourseEnrollment(user=user, course=CourseOverview.get_from_id(pseudo_key), mode=pseudo_session['type'])
# We only show email settings for entitlement cards if the entitlement has an associated enrollment
show_email_settings = is_fulfilled_entitlement and (entitlement_session.course_id in show_email_settings_for)
else: