# -*- coding: utf-8 -*- from __future__ import division __copyright__ = "Copyright (C) 2014 Andreas Kloeckner" __license__ = """ Permission is hereby granted, free of charge, to any person obtaining a copy of this software and associated documentation files (the "Software"), to deal in the Software without restriction, including without limitation the rights to use, copy, modify, merge, publish, distribute, sublicense, and/or sell copies of the Software, and to permit persons to whom the Software is furnished to do so, subject to the following conditions: The above copyright notice and this permission notice shall be included in all copies or substantial portions of the Software. THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE. """ from django.shortcuts import ( # noqa render, get_object_or_404, redirect) from django.contrib import messages from django.core.exceptions import ( PermissionDenied, SuspiciousOperation, ObjectDoesNotExist) from django.db import transaction from django.utils.safestring import mark_safe from django import forms from django import http from courseflow.utils import StyledForm from crispy_forms.layout import Submit import re from course.constants import ( flow_permission, participation_role, flow_session_expriration_mode, FLOW_SESSION_EXPIRATION_MODE_CHOICES, is_expiration_mode_allowed) from course.models import ( FlowSession, FlowPageData, FlowPageVisit, FlowPageVisitGrade, GradeChange) from course.utils import ( FlowContext, FlowPageContext, instantiate_flow_page_with_ctx, course_view, render_course_page) from course.views import get_now_or_fake_time def get_flow_session_id_map(request): result = request.session.setdefault("flow_session_id_map", {}) # Tell Django: This session has been modified, persist it. request.session["flow_session_id_map"] = result return result # {{{ grade page visit def grade_page_visit(visit, visit_grade_model=FlowPageVisitGrade, grade_data=None, graded_at_git_commit_sha=None): if not visit.is_graded_answer: raise RuntimeError("cannot grade ungraded answer") flow_session = visit.flow_session course = flow_session.course page_data = visit.page_data most_recent_grade = visit.get_most_recent_grade() if most_recent_grade is not None and grade_data is None: grade_data = most_recent_grade.grade_data from course.content import ( get_course_repo, get_course_commit_sha, get_flow_commit_sha, get_flow_desc, get_flow_page_desc, instantiate_flow_page) repo = get_course_repo(course) course_commit_sha = get_course_commit_sha( course, flow_session.participation) flow_desc_pre = get_flow_desc(repo, course, flow_session.flow_id, course_commit_sha) flow_commit_sha = get_flow_commit_sha( course, flow_session.participation, flow_desc_pre, visit.flow_session) flow_desc = get_flow_desc(repo, course, flow_session.flow_id, flow_commit_sha) page_desc = get_flow_page_desc( flow_session.flow_id, flow_desc, page_data.group_id, page_data.page_id) page = instantiate_flow_page( location="flow '%s', group, '%s', page '%s'" % (flow_session.flow_id, page_data.group_id, page_data.page_id), repo=repo, page_desc=page_desc, commit_sha=flow_commit_sha) from course.page import PageContext grading_page_context = PageContext( course=course, repo=repo, commit_sha=flow_commit_sha, flow_session=flow_session) answer_feedback = page.grade( grading_page_context, visit.page_data.data, visit.answer, grade_data=grade_data) grade = visit_grade_model() grade.visit = visit grade.grade_data = grade_data grade.max_points = page.max_points(visit.page_data) grade.graded_at_git_commit_sha = graded_at_git_commit_sha if answer_feedback is not None: grade.correctness = answer_feedback.correctness grade.feedback = answer_feedback.as_json() grade.save() # }}} # {{{ finish flow def get_flow_session_graded_answers_qset(flow_session): from django.db.models import Q qset = (FlowPageVisit.objects .filter(flow_session=flow_session) .filter(Q(answer__isnull=False) | Q(is_synthetic=True))) if not flow_session.in_progress: # Ungraded answers *can* show up in non-in-progress flows as a result # of a race between a 'save' and the 'end session'. If this happens, # we'll go ahead and ignore those. qset = qset.filter(is_graded_answer=True) return qset def get_prev_answer_visit(page_data): previous_answer_visits = ( get_flow_session_graded_answers_qset(page_data.flow_session) .filter(page_data=page_data) .order_by("-visit_time")) for prev_visit in previous_answer_visits[:1]: return prev_visit return None def assemble_answer_visits(flow_session): answer_visits = [None] * flow_session.page_count answer_page_visits = ( get_flow_session_graded_answers_qset(flow_session) .order_by("visit_time")) for page_visit in answer_page_visits: answer_visits[page_visit.page_data.ordinal] = page_visit if not flow_session.in_progress: # This is redundant with the answers being marked as # final at the end of a flow, but that's OK. # # Note that this change is generally not persisted. page_visit.is_graded_answer = True return answer_visits def count_answered(fctx, flow_session, answer_visits): all_page_data = (FlowPageData.objects .filter(flow_session=flow_session) .order_by("ordinal")) answered_count = 0 unanswered_count = 0 for i, page_data in enumerate(all_page_data): assert i == page_data.ordinal if answer_visits[i] is not None: answer_data = answer_visits[i].answer else: answer_data = None page = instantiate_flow_page_with_ctx(fctx, page_data) if page.expects_answer(): if answer_data is None: unanswered_count += 1 else: answered_count += 1 return (answered_count, unanswered_count) class GradeInfo(object): """ .. attribute:: points The final grade, in points. May be *None* if the grade is not yet final. """ def __init__(self, points, provisional_points, max_points, max_reachable_points, fully_correct_count, partially_correct_count, incorrect_count, unknown_count): self.points = points self.provisional_points = provisional_points self.max_points = max_points self.max_reachable_points = max_reachable_points self.fully_correct_count = fully_correct_count self.partially_correct_count = partially_correct_count self.incorrect_count = incorrect_count self.unknown_count = unknown_count # Rounding to larger than 100% will break the percent bars on the # flow results page. FULL_PERCENT = 99.99 def points_percent(self): """Only to be used for visualization purposes.""" if self.max_points is None or self.max_points == 0: if self.points == 0: return 100 else: return 0 else: return self.FULL_PERCENT*self.provisional_points/self.max_points def missed_points_percent(self): """Only to be used for visualization purposes.""" return (self.FULL_PERCENT - self.points_percent() - self.unreachable_points_percent()) def unreachable_points_percent(self): """Only to be used for visualization purposes.""" if (self.max_points is None or self.max_reachable_points is None or self.max_points == 0): return 0 else: return self.FULL_PERCENT*( self.max_points - self.max_reachable_points)/self.max_points def total_count(self): return (self.fully_correct_count + self.partially_correct_count + self.incorrect_count + self.unknown_count) def fully_correct_percent(self): """Only to be used for visualization purposes.""" return self.FULL_PERCENT*self.fully_correct_count/self.total_count() def partially_correct_percent(self): """Only to be used for visualization purposes.""" return self.FULL_PERCENT*self.partially_correct_count/self.total_count() def incorrect_percent(self): """Only to be used for visualization purposes.""" return self.FULL_PERCENT*self.incorrect_count/self.total_count() def unknown_percent(self): """Only to be used for visualization purposes.""" return self.FULL_PERCENT*self.unknown_count/self.total_count() def gather_grade_info(flow_session, answer_visits): all_page_data = (FlowPageData.objects .filter(flow_session=flow_session) .order_by("ordinal")) points = 0 provisional_points = 0 max_points = 0 max_reachable_points = 0 fully_correct_count = 0 partially_correct_count = 0 incorrect_count = 0 unknown_count = 0 for i, page_data in enumerate(all_page_data): assert i == page_data.ordinal if answer_visits[i] is None: # page did not expect an answer continue grade = answer_visits[i].get_most_recent_grade() assert grade is not None if grade.feedback is not None: from course.page import AnswerFeedback feedback = AnswerFeedback.from_json(grade.feedback) else: feedback = None max_points += grade.max_points if feedback is None or feedback.correctness is None: unknown_count += 1 points = None continue max_reachable_points += grade.max_points page_points = grade.max_points*feedback.correctness if points is not None: points += page_points provisional_points += page_points if grade.max_points > 0: if feedback.correctness == 1: fully_correct_count += 1 elif feedback.correctness == 0: incorrect_count += 1 else: partially_correct_count += 1 return GradeInfo( points=points, provisional_points=provisional_points, max_points=max_points, max_reachable_points=max_reachable_points, fully_correct_count=fully_correct_count, partially_correct_count=partially_correct_count, incorrect_count=incorrect_count, unknown_count=unknown_count) @transaction.atomic def grade_page_visits(fctx, flow_session, answer_visits, force_regrade=False): for i in range(len(answer_visits)): answer_visit = answer_visits[i] if answer_visit is not None: answer_visit.is_graded_answer = True answer_visit.save() else: page_data = flow_session.page_data.get(ordinal=i) page = instantiate_flow_page_with_ctx(fctx, page_data) if not page.expects_answer(): continue # Create a synthetic visit to attach a grade answer_visit = FlowPageVisit() answer_visit.flow_session = flow_session answer_visit.page_data = page_data answer_visit.is_synthetic = True answer_visit.answer = None answer_visit.is_graded_answer = True answer_visit.save() answer_visits[i] = answer_visit if (answer_visit is not None and (not answer_visit.grades.count() or force_regrade)): grade_page_visit(answer_visit, graded_at_git_commit_sha=fctx.flow_commit_sha) @transaction.atomic def finish_flow_session(fctx, flow_session, current_access_rule, force_regrade=False, now_datetime=None): if not flow_session.in_progress: raise RuntimeError("Can't end a session that's already ended") answer_visits = assemble_answer_visits(flow_session) (answered_count, unanswered_count) = count_answered( fctx, flow_session, answer_visits) is_graded_flow = bool(answered_count + unanswered_count) if is_graded_flow: grade_page_visits(fctx, flow_session, answer_visits, force_regrade=force_regrade) # ORDERING RESTRICTION: Must grade pages before gathering grade info if now_datetime is None: from django.utils.timezone import now now_datetime = now() flow_session.completion_time = now_datetime flow_session.in_progress = False flow_session.save() return grade_flow_session(fctx, flow_session, current_access_rule, answer_visits) @transaction.atomic def expire_flow_session(fctx, flow_session, current_access_rule, now_datetime): if not flow_session.in_progress: raise RuntimeError("Can't expire a session that's not in progress") if flow_session.participation is None: raise RuntimeError("Can't expire an anonymous flow session") if flow_session.expiration_mode == flow_session_expriration_mode.roll_over: from course.utils import get_current_flow_access_rule new_rules = get_current_flow_access_rule(flow_session.course, flow_session.participation, flow_session.participation.role, flow_session.flow_id, fctx.flow_desc, now_datetime, rule_id=None, use_exceptions=True) flow_session.access_rules_id = new_rules.id if not is_expiration_mode_allowed( flow_session.expiration_mode, new_rules.permissions): flow_session.expiration_mode = flow_session_expriration_mode.end flow_session.save() elif flow_session.expiration_mode == flow_session_expriration_mode.end: return finish_flow_session(fctx, flow_session, current_access_rule, now_datetime=now_datetime) else: raise ValueError("invalid expiration mode '%s' on flow session ID %d" % (flow_session.expiration_mode, flow_session.id)) def grade_flow_session(fctx, flow_session, current_access_rule, answer_visits=None): """Updates the grade on an existing flow session and logs a grade change with the grade records subsystem. """ if answer_visits is None: answer_visits = assemble_answer_visits(flow_session) (answered_count, unanswered_count) = count_answered( fctx, flow_session, answer_visits) is_graded_flow = bool(answered_count + unanswered_count) is_graded_flow = bool(answered_count + unanswered_count) grade_info = gather_grade_info(flow_session, answer_visits) assert grade_info is not None comment = None points = grade_info.points if points is not None and current_access_rule.credit_percent is not None: comment = "Counted at %.1f%% of %.1f points" % ( current_access_rule.credit_percent, points) points = points * current_access_rule.credit_percent / 100 flow_session.points = points flow_session.max_points = grade_info.max_points flow_session.append_comment(comment) flow_session.save() # Need to save grade record even if no grade is available yet, because # a grade record may *already* be saved, and that one might be mistaken # for the current one. if (is_graded_flow and flow_session.participation is not None and flow_session.for_credit): from course.models import get_flow_grading_opportunity gopp = get_flow_grading_opportunity( flow_session.course, flow_session.flow_id, fctx.flow_desc) from course.models import grade_state_change_types gchange = GradeChange() gchange.opportunity = gopp gchange.participation = flow_session.participation gchange.state = grade_state_change_types.graded gchange.attempt_id = "flow-session-%d" % flow_session.id gchange.points = points gchange.max_points = grade_info.max_points # creator left as NULL gchange.flow_session = flow_session gchange.comment = comment gchange.save() return grade_info def reopen_session(session, force=False, suppress_log=False): if session.in_progress: raise RuntimeError("Can't reopen a session that's already in progress") if session.participation is None: raise RuntimeError("Can't reopen anonymous sessions") if not force: other_in_progress_sessions = (FlowSession.objects .filter( participation=session.participation, flow_id=session.flow_id, in_progress=True, participation__isnull=False) .exclude(id=session.id)) if other_in_progress_sessions.count(): raise RuntimeError("Can't open multiple sessions at once") session.in_progress = True session.points = None session.max_points = None if not suppress_log: from django.utils.timezone import now session.append_comment( "Session reopened at %s, previous completion time was '%s'." % (now(), session.completion_time)) session.completion_time = None session.save() def finish_flow_session_standalone(repo, course, session, force_regrade=False, now_datetime=None, past_end_only=False): assert session.participation is not None from course.utils import FlowContext if now_datetime is None: from django.utils.timezone import now now_datetime = now() fctx = FlowContext(repo, course, session.flow_id, flow_session=session) current_access_rule = fctx.get_current_access_rule( session, session.participation.role, session.participation, now_datetime) if (past_end_only and current_access_rule.end is not None and now_datetime < current_access_rule.end): return False finish_flow_session(fctx, session, current_access_rule, force_regrade=force_regrade, now_datetime=now_datetime) return True def expire_flow_session_standalone(repo, course, session, now_datetime, past_end_only=False): assert session.participation is not None from course.utils import FlowContext fctx = FlowContext(repo, course, session.flow_id, flow_session=session) current_access_rule = fctx.get_current_access_rule( session, session.participation.role, session.participation, now_datetime) if (past_end_only and current_access_rule.end is not None and now_datetime < current_access_rule.end): return False expire_flow_session(fctx, session, current_access_rule, now_datetime) return True @transaction.atomic def regrade_session(repo, course, session): if session.in_progress: fctx = FlowContext(repo, course, session.flow_id, flow_session=session) answer_visits = assemble_answer_visits(session) for i in range(len(answer_visits)): answer_visit = answer_visits[i] if answer_visit is not None and answer_visit.get_most_recent_grade(): # Only make a new grade if there already is one. grade_page_visit(answer_visit, graded_at_git_commit_sha=fctx.flow_commit_sha) else: prev_completion_time = session.completion_time from django.utils.timezone import now session.append_comment("Session regraded at %s." % now()) session.save() reopen_session(session, force=True, suppress_log=True) finish_flow_session_standalone( repo, course, session, force_regrade=True, now_datetime=prev_completion_time) # }}} # {{{ view: start flow RESUME_RE = re.compile("^resume_([0-9]+)$") @transaction.atomic @course_view def start_flow(pctx, flow_identifier): request = pctx.request now_datetime = get_now_or_fake_time(request) fctx = FlowContext(pctx.repo, pctx.course, flow_identifier, participation=pctx.participation) current_access_rule = fctx.get_current_access_rule( None, pctx.role, pctx.participation, now_datetime) may_view = flow_permission.view in current_access_rule.permissions have_in_progress_session = (FlowSession.objects .filter( participation=pctx.participation, flow_id=fctx.flow_identifier, in_progress=True, participation__isnull=False, )).count() > 0 past_sessions = (FlowSession.objects .filter( participation=pctx.participation, flow_id=fctx.flow_identifier, participation__isnull=False) .order_by("start_time")) past_session_count = past_sessions.count() if current_access_rule.allowed_session_count is not None: allowed_another_session = ( past_session_count < current_access_rule.allowed_session_count) else: allowed_another_session = True if request.method == "POST": from course.content import set_up_flow_session_page_data resume_match = None for post_key in request.POST: resume_match = RESUME_RE.match(post_key) if resume_match is not None: break if resume_match is not None: resume_session_id = int(resume_match.group(1)) resume_session = get_object_or_404(FlowSession, pk=resume_session_id) if resume_session.participation != pctx.participation: raise PermissionDenied("not your session") if not may_view: raise PermissionDenied("may not resume session without " "'view' permission") if resume_session.participation is None: raise PermissionDenied("can't resume anonymous session") if resume_session.flow_id != fctx.flow_identifier: raise SuspiciousOperation("flow id mismatch on resume") if not (flow_permission.view_past in current_access_rule.permissions or resume_session.in_progress): raise PermissionDenied("not allowed to resume session") get_flow_session_id_map(request)[flow_identifier] = resume_session_id return redirect("course.flow.view_flow_page", pctx.course.identifier, flow_identifier, 0) elif ("start_no_credit" in request.POST or "start_credit" in request.POST): if not may_view: raise PermissionDenied("may not start session without " "'view' permission") if not allowed_another_session: raise PermissionDenied("new session would exceed " "allowed session count limit exceed") if have_in_progress_session: raise PermissionDenied("cannot start flow when other flow " "session is already in progress") session = FlowSession() session.course = fctx.course session.participation = pctx.participation session.active_git_commit_sha = fctx.flow_commit_sha.decode() session.flow_id = flow_identifier session.in_progress = True if (flow_permission.set_roll_over_expiration_mode in current_access_rule.permissions): session.expiration_mode = flow_session_expriration_mode.roll_over else: session.expiration_mode = flow_session_expriration_mode.end session.for_credit = "start_credit" in request.POST if current_access_rule.sticky is not None and current_access_rule.sticky: session.access_rules_id = current_access_rule.id session.save() get_flow_session_id_map(request)[flow_identifier] = session.id page_count = set_up_flow_session_page_data(fctx.repo, session, pctx.course.identifier, fctx.flow_desc, fctx.flow_commit_sha) session.page_count = page_count session.save() return redirect("course.flow.view_flow_page", pctx.course.identifier, flow_identifier, 0) else: raise SuspiciousOperation("unrecognized POST action") else: may_start_credit = ( may_view and not have_in_progress_session and allowed_another_session and flow_permission.start_credit in current_access_rule.permissions) may_start_no_credit = ( may_view and not have_in_progress_session and allowed_another_session and (flow_permission.start_no_credit in current_access_rule.permissions)) may_review = ( may_view and flow_permission.view_past in current_access_rule.permissions) if hasattr(fctx.flow_desc, "grade_aggregation_strategy"): from course.models import GRADE_AGGREGATION_STRATEGY_CHOICES grade_aggregation_strategy_text = ( dict(GRADE_AGGREGATION_STRATEGY_CHOICES) [fctx.flow_desc.grade_aggregation_strategy]) else: grade_aggregation_strategy_text = None # {{{ fish out relevant rules from course.utils import ( get_flow_access_rules, get_relevant_rules) rules = get_flow_access_rules(fctx.course, pctx.participation, flow_identifier, fctx.flow_desc) # }}} return render_course_page(pctx, "course/flow-start.html", { "flow_desc": fctx.flow_desc, "grade_aggregation_strategy": grade_aggregation_strategy_text, "flow_identifier": flow_identifier, "rules": get_relevant_rules(rules, pctx.role, now_datetime), "now": now_datetime, "may_start_credit": may_start_credit, "may_start_no_credit": may_start_no_credit, "may_review": may_review, "past_sessions": past_sessions, }, allow_instant_flow_requests=False) # }}} # {{{ view: flow page def find_current_flow_session(request, course, flow_identifier): flow_session = None flow_session_id = get_flow_session_id_map(request).get(flow_identifier) if flow_session_id is not None: try: flow_session = FlowSession.objects.get(id=flow_session_id) except ObjectDoesNotExist: return None if flow_session.course.pk != course.pk: return None if flow_session.flow_id != flow_identifier: return None return flow_session def will_receive_feedback(permissions): return ( flow_permission.see_correctness in permissions or flow_permission.see_answer in permissions) def add_buttons_to_form(form, fpctx, flow_session, permissions): from crispy_forms.layout import Submit form.helper.add_input( Submit("save", "Save answer", css_class="col-lg-offset-2 cf-save-button")) if will_receive_feedback(permissions): if flow_permission.change_answer in permissions: form.helper.add_input( Submit( "submit", "Submit answer for grading", accesskey="g", css_class="cf-save-button")) else: form.helper.add_input( Submit("submit", "Submit final answer", css_class="cf-save-button")) else: # Only offer 'save and move on' if student will receive no feedback if fpctx.page_data.ordinal + 1 < flow_session.page_count: form.helper.add_input( Submit("save_and_next", mark_safe("Save answer and move on »"), css_class="cf-save-button")) else: form.helper.add_input( Submit("save_and_finish", mark_safe("Save answer and finish »"), css_class="cf-save-button")) return form def get_pressed_button(form): buttons = ["save", "save_and_next", "save_and_finish", "submit"] for button in buttons: if button in form.data: return button raise SuspiciousOperation("could not find which button was pressed") def create_flow_page_visit(request, flow_session, page_data): page_visit = FlowPageVisit() page_visit.flow_session = flow_session page_visit.page_data = page_data page_visit.remote_address = request.META['REMOTE_ADDR'] page_visit.save() @course_view def view_flow_page(pctx, flow_identifier, ordinal): request = pctx.request flow_session = find_current_flow_session( request, pctx.course, flow_identifier) if flow_session is None: messages.add_message(request, messages.WARNING, "No in-progress session record found for this flow. " "Redirected to flow start page.") return redirect("course.flow.start_flow", pctx.course.identifier, flow_identifier) fpctx = FlowPageContext(pctx.repo, pctx.course, flow_identifier, ordinal, participation=pctx.participation, flow_session=flow_session) if fpctx.page_desc is None: messages.add_message(request, messages.ERROR, "Your session does not match the course content and needs " "to be reset. Course staff have been notified about this issue. " "Please get in touch with them to get help.") from django.template.loader import render_to_string message = render_to_string("course/session-mismatch.txt", { "page_data": fpctx.page_data, "course": pctx.course, "user": pctx.request.user, }) from django.core.mail import send_mail from django.conf import settings send_mail("[%s] session mismatch with course content" % pctx.course.identifier, message, settings.ROBOT_EMAIL_FROM, recipient_list=[pctx.course.email]) return redirect("course.flow.start_flow", pctx.course.identifier, flow_identifier) current_access_rule = fpctx.get_current_access_rule( flow_session, pctx.role, pctx.participation, get_now_or_fake_time(request)) permissions = fpctx.page.get_modified_permissions_for_page( current_access_rule.permissions) page_context = fpctx.page_context page_data = fpctx.page_data answer_data = None grade_data = None if flow_permission.view not in permissions: raise PermissionDenied("not allowed to view flow") if request.method == "POST": if "finish" in request.POST: return redirect("course.flow.finish_flow_session_view", pctx.course.identifier, flow_identifier) else: # reject answer update if flow is not in-progress if not flow_session.in_progress: raise PermissionDenied("session is not in progress") # reject if previous answer was final if (fpctx.prev_answer_visit is not None and fpctx.prev_answer_visit.is_graded_answer and flow_permission.change_answer not in permissions): raise PermissionDenied("already have final answer") form = fpctx.page.post_form( fpctx.page_context, fpctx.page_data.data, post_data=request.POST, files_data=request.FILES) pressed_button = get_pressed_button(form) if form.is_valid(): # {{{ form validated, process answer messages.add_message(request, messages.INFO, "Answer saved.") page_visit = FlowPageVisit() page_visit.flow_session = flow_session page_visit.page_data = fpctx.page_data page_visit.remote_address = request.META['REMOTE_ADDR'] answer_data = page_visit.answer = fpctx.page.answer_data( fpctx.page_context, fpctx.page_data.data, form, request.FILES) page_visit.is_graded_answer = pressed_button == "submit" page_visit.save() answer_was_graded = page_visit.is_graded_answer may_change_answer = ( not answer_was_graded or flow_permission.change_answer in permissions) feedback = fpctx.page.grade( page_context, page_data.data, page_visit.answer, grade_data=None) if page_visit.is_graded_answer: grade = FlowPageVisitGrade() grade.visit = page_visit grade.max_points = fpctx.page.max_points(page_data.data) grade.graded_at_git_commit_sha = fpctx.flow_commit_sha if feedback is not None: grade.correctness = feedback.correctness grade.feedback = feedback.as_json() grade.save() del grade if (pressed_button == "save_and_next" and not will_receive_feedback(permissions)): return redirect("course.flow.view_flow_page", pctx.course.identifier, flow_identifier, fpctx.ordinal + 1) elif (pressed_button == "save_and_finish" and not will_receive_feedback(permissions)): return redirect("course.flow.finish_flow_session_view", pctx.course.identifier, flow_identifier) else: form = fpctx.page.make_form( page_context, page_data.data, page_visit.answer, not may_change_answer) # continue at common flow page generation below # }}} del page_visit else: # form did not validate create_flow_page_visit(request, flow_session, fpctx.page_data) answer_was_graded = False may_change_answer = True # because we were allowed this far in by the check above feedback = None # continue at common flow page generation below else: create_flow_page_visit(request, flow_session, fpctx.page_data) if fpctx.prev_answer_visit is not None: answer_was_graded = fpctx.prev_answer_visit.is_graded_answer else: answer_was_graded = False may_change_answer = ( (not answer_was_graded or (flow_permission.change_answer in permissions)) # can happen if no answer was ever saved and flow_session.in_progress) if fpctx.page.expects_answer(): if fpctx.prev_answer_visit is not None: answer_data = fpctx.prev_answer_visit.answer most_recent_grade = fpctx.prev_answer_visit.get_most_recent_grade() if most_recent_grade is not None: if most_recent_grade.feedback is not None: from course.page import AnswerFeedback feedback = AnswerFeedback.from_json( most_recent_grade.feedback) else: feedback = None grade_data = most_recent_grade.grade_data else: feedback = None grade_data = None else: feedback = None form = fpctx.page.make_form( page_context, page_data.data, answer_data, not may_change_answer) else: form = None feedback = None # start common flow page generation # defined at this point: # form, form_html, may_change_answer, answer_was_graded, feedback if form is not None and may_change_answer: form = add_buttons_to_form(form, fpctx, flow_session, permissions) show_correctness = None show_answer = None shown_feedback = None if fpctx.page.expects_answer() and answer_was_graded: show_correctness = ( flow_permission.see_correctness in permissions or ( (flow_permission.see_correctness_after_completion in permissions) and not flow_session.in_progress)) show_answer = ( flow_permission.see_answer in permissions or ( (flow_permission.see_answer_after_completion in permissions) and not flow_session.in_progress)) if show_correctness or show_answer: shown_feedback = feedback elif fpctx.page.expects_answer() and not answer_was_graded: # Don't show answer yet pass else: show_answer = ( flow_permission.see_answer in permissions or ( (flow_permission.see_answer_after_completion in permissions) and not flow_session.in_progress)) title = fpctx.page.title(page_context, page_data.data) body = fpctx.page.body(page_context, page_data.data) if show_answer: correct_answer = fpctx.page.correct_answer( page_context, page_data.data, answer_data, grade_data) else: correct_answer = None # {{{ render flow page if form is not None: form_html = fpctx.page.form_to_html( pctx.request, page_context, form, answer_data) else: form_html = None expiration_mode_choices = [] for key, descr in FLOW_SESSION_EXPIRATION_MODE_CHOICES: if is_expiration_mode_allowed(key, current_access_rule.permissions): expiration_mode_choices.append((key, descr)) args = { "flow_identifier": fpctx.flow_identifier, "flow_desc": fpctx.flow_desc, "ordinal": fpctx.ordinal, "page_data": fpctx.page_data, "percentage": int(100*(fpctx.ordinal+1) / flow_session.page_count), "flow_session": flow_session, "page_numbers": zip( range(flow_session.page_count), range(1, flow_session.page_count+1)), "title": title, "body": body, "form": form, "form_html": form_html, "feedback": shown_feedback, "correct_answer": correct_answer, "show_correctness": show_correctness, "may_change_answer": may_change_answer, "may_change_graded_answer": ( (flow_permission.change_answer in permissions) and flow_session.in_progress), "will_receive_feedback": will_receive_feedback(permissions), "show_answer": show_answer, "expiration_mode_choices": expiration_mode_choices, "expiration_mode_choice_count": len(expiration_mode_choices), "expiration_mode": flow_session.expiration_mode, } if fpctx.page.expects_answer(): args["max_points"] = fpctx.page.max_points(fpctx.page_data) return render_course_page( pctx, "course/flow-page.html", args, allow_instant_flow_requests=False) # }}} @course_view def update_expiration_mode(pctx, flow_session_id): if pctx.request.method != "POST": raise SuspiciousOperation("only POST allowed") flow_session = get_object_or_404(FlowSession, id=flow_session_id) if flow_session.participation != pctx.participation: raise PermissionDenied("may only change your own flow sessions") if not flow_session.in_progress: raise PermissionDenied("may only change in-progress flow sessions") expmode = pctx.request.POST.get("expiration_mode") if not any(expmode == em_key for em_key, _ in FLOW_SESSION_EXPIRATION_MODE_CHOICES): raise SuspiciousOperation("invalid expiration mode") fctx = FlowContext(pctx.repo, pctx.course, flow_session.flow_id, participation=pctx.participation, flow_session=flow_session) current_access_rule = fctx.get_current_access_rule( flow_session, pctx.role, pctx.participation, get_now_or_fake_time(pctx.request)) if is_expiration_mode_allowed(expmode, current_access_rule.permissions): flow_session.expiration_mode = expmode flow_session.save() return http.HttpResponse("OK") else: raise PermissionDenied() # }}} # {{{ view: finish flow @transaction.atomic @course_view def finish_flow_session_view(pctx, flow_identifier): now_datetime = get_now_or_fake_time(pctx.request) request = pctx.request flow_session = find_current_flow_session( request, pctx.course, flow_identifier) if flow_session is None: messages.add_message(request, messages.WARNING, "No session record found for this flow. " "Redirected to flow start page.") return redirect("course.flow.start_flow", pctx.course.identifier, flow_identifier) fctx = FlowContext(pctx.repo, pctx.course, flow_identifier, participation=pctx.participation, flow_session=flow_session) current_access_rule = fctx.get_current_access_rule( flow_session, pctx.role, pctx.participation, get_now_or_fake_time(request)) answer_visits = assemble_answer_visits(flow_session) from course.content import markup_to_html completion_text = markup_to_html( fctx.course, fctx.repo, fctx.flow_commit_sha, fctx.flow_desc.completion_text) (answered_count, unanswered_count) = count_answered( fctx, flow_session, answer_visits) is_graded_flow = bool(answered_count + unanswered_count) if flow_permission.view not in current_access_rule.permissions: raise PermissionDenied() def render_finish_response(template, **kwargs): render_args = { "flow_identifier": fctx.flow_identifier, "flow_desc": fctx.flow_desc, } render_args.update(kwargs) return render_course_page( pctx, template, render_args, allow_instant_flow_requests=False) if request.method == "POST": if "submit" not in request.POST: raise SuspiciousOperation("odd POST parameters") if not flow_session.in_progress: raise PermissionDenied("Can't end a session that's already ended") # Actually end the flow session get_flow_session_id_map(request)[flow_identifier] = None grade_info = finish_flow_session( fctx, flow_session, current_access_rule, now_datetime=now_datetime) if is_graded_flow: return render_finish_response( "course/flow-completion-grade.html", completion_text=completion_text, grade_info=grade_info) else: return render_finish_response( "course/flow-completion.html", last_page_nr=None, completion_text=completion_text) if (not is_graded_flow and fctx.flow_commit_sha == fctx.course_commit_sha): # Not serious--no questions in flow, and no new version available. # No need to end the flow visit. return render_finish_response( "course/flow-completion.html", last_page_nr=flow_session.page_count-1, completion_text=completion_text) elif not flow_session.in_progress: # Just reviewing: re-show grades. grade_info = gather_grade_info(flow_session, answer_visits) return render_finish_response( "course/flow-completion-grade.html", completion_text=completion_text, grade_info=grade_info) else: # confirm ending flow return render_finish_response( "course/flow-confirm-completion.html", last_page_nr=flow_session.page_count-1, answered_count=answered_count, unanswered_count=unanswered_count, total_count=answered_count+unanswered_count) # }}} # {{{ view: regrade flow class RegradeFlowForm(StyledForm): def __init__(self, flow_ids, *args, **kwargs): super(RegradeFlowForm, self).__init__(*args, **kwargs) self.fields["flow_id"] = forms.ChoiceField( choices=[(fid, fid) for fid in flow_ids], initial=participation_role.student, required=True) self.fields["access_rules_id"] = forms.CharField( required=False, help_text="If non-empty, limit the regrading to sessions started " "under this access rules ID.") self.fields["regraded_session_in_progress"] = forms.ChoiceField( choices=( ("any", "Regrade in-progress and not-in-progress sessions"), ("yes", "Regrade in-progress sessions only"), ("no", "Regrade not-in-progress sessions only"), )) self.helper.add_input( Submit("regrade", "Regrade", css_class="col-lg-offset-2")) @transaction.atomic def _regrade_sessions(repo, course, sessions): count = 0 from course.flow import regrade_session for session in sessions: regrade_session(repo, course, session) count += 1 return count @course_view def regrade_not_for_credit_flows_view(pctx): if pctx.role != participation_role.instructor: raise PermissionDenied("must be instructor to regrade flows") from course.content import list_flow_ids flow_ids = list_flow_ids(pctx.repo, pctx.course_commit_sha) request = pctx.request if request.method == "POST": form = RegradeFlowForm(flow_ids, request.POST, request.FILES) if form.is_valid(): sessions = (FlowSession.objects .filter( course=pctx.course, flow_id=form.cleaned_data["flow_id"], for_credit=False)) if form.cleaned_data["access_rules_id"]: sessions = sessions.filter( access_rules_id=form.cleaned_data["access_rules_id"]) inprog_value = { "any": None, "yes": True, "no": False, }[form.cleaned_data["regraded_session_in_progress"]] if inprog_value is not None: sessions = sessions.filter( in_progress=inprog_value) count = _regrade_sessions(pctx.repo, pctx.course, sessions) messages.add_message(request, messages.SUCCESS, "%d sessions regraded." % count) else: form = RegradeFlowForm(flow_ids) return render_course_page(pctx, "course/generic-course-form.html", { "form": form, "form_text": "

This regrading process only considers not-for-credit flow " "sessions. If you would like to regrade for-credit flows, " "use the corresponding functionality in the grade book.

", "form_description": "Regrade not-for-credit Flow Sessions", }) # }}} # vim: foldmethod=marker