diff --git a/common/djangoapps/course_modes/views.py b/common/djangoapps/course_modes/views.py index 0993467c1727d09bed82ac400454d40a6b03917e..7c4bb3a1f787a8330a604e182849919bc155abc1 100644 --- a/common/djangoapps/course_modes/views.py +++ b/common/djangoapps/course_modes/views.py @@ -82,9 +82,7 @@ class ChooseModeView(View): upgrade = request.GET.get('upgrade', False) - requested_mode = self.get_requested_mode(request.POST.get("mode")) - if requested_mode == "verified" and request.POST.get("honor-code"): - requested_mode = "honor" + requested_mode = self.get_requested_mode(request.POST) allowed_modes = CourseMode.modes_for_course_dict(course_id) if requested_mode not in allowed_modes: @@ -125,14 +123,14 @@ class ChooseModeView(View): reverse('verify_student_show_requirements', kwargs={'course_id': course_id}) + "?upgrade={}".format(upgrade)) - def get_requested_mode(self, user_choice): + def get_requested_mode(self, request_dict): """ - Given the text of `user_choice`, return the + Given the request object of `user_choice`, return the corresponding course mode slug """ - choices = { - "Select Audit": "audit", - "Select Certificate": "verified", - "Upgrade Your Registration": "verified" - } - return choices.get(user_choice) + if 'audit_mode' in request_dict: + return 'audit' + if 'certificate_mode' and request_dict.get("honor-code"): + return 'honor' + if 'certificate_mode' in request_dict: + return 'verified' diff --git a/common/templates/course_modes/choose.html b/common/templates/course_modes/choose.html index 67cc0a301d4bc305c0d4ddb108480ae72fc094b5..513dd9862102a5708f24417aef9640ade5c568c6 100644 --- a/common/templates/course_modes/choose.html +++ b/common/templates/course_modes/choose.html @@ -134,9 +134,9 @@ $(document).ready(function() { <ul class="list-actions"> <li class="action action-select"> %if upgrade: - <input type="submit" name="mode" value="Upgrade Your Registration" /> + <input type="submit" name="certificate_mode" value="${_('Upgrade Your Registration')}" /> %else: - <input type="submit" name="mode" value="Select Certificate" /> + <input type="submit" name="certificate_mode" value="${_('Select Certificate')}" /> %endif </li> @@ -179,7 +179,7 @@ $(document).ready(function() { <ul class="list-actions"> <li class="action action-select"> - <input type="submit" name="mode" value="Select Audit" /> + <input type="submit" name="audit_mode" value="${_('Select Audit')}" /> </li> </ul> </div>