Newer
Older
# -*- 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.core.exceptions import PermissionDenied, SuspiciousOperation
Course,
Participation, participation_role, participation_status,
FlowVisit, FlowPageData, FlowPageVisit, flow_permission,
GradingOpportunity, GradeChange)
get_course_repo, get_course_desc, parse_date_spec,
def get_role_and_participation(request, course):
# "wake up" lazy object
# http://stackoverflow.com/questions/20534577/int-argument-must-be-a-string-or-a-number-not-simplelazyobject # noqa
user = (request.user._wrapped
if hasattr(request.user, '_wrapped')
else request.user)
if not user.is_authenticated():
return participation_role.unenrolled, None
participations = list(Participation.objects.filter(
user=user, course=course))
# The uniqueness constraint should have ensured that.
assert len(participations) <= 1
participation = participations[0]
if participation.status != participation_status.active:
if participation.temporary_role:
return participation.temporary_role, participation
else:
return participation.role, participation
def get_active_commit_sha(course, participation):
sha = course.active_git_commit_sha
if participation is not None and participation.preview_git_commit_sha:
sha = participation.preview_git_commit_sha
return sha.encode()
def get_flow_permissions(course_desc, participation, role, flow_id, flow_desc):
for exc in (
FlowAccessException.objects
.filter(participation=participation, flow_id=flow_id)
.order_by("expiration")):
if exc.expiration is not None and exc.expiration < now:
continue
stipulations = exc.stipulations
if not isinstance(stipulations, dict):
stipulations = {}
from course.content import dict_to_struct
stipulations = dict_to_struct(exc.stipulations)
return (
[entry.permission for entry in exc.entries.all()],
stipulations
)
# }}}
# {{{ interpret flow rules
if hasattr(rule, "roles"):
if role not in rule.roles:
continue
if hasattr(rule, "start"):
start_date = parse_date_spec(course_desc, rule.start)
if now < start_date:
continue
if hasattr(rule, "end"):
end_date = parse_date_spec(course_desc, rule.end)
if end_date < now:
continue
raise ValueError("Flow access rules of flow '%s' did not resolve "
"to access answer for '%s'" % (flow_id, participation))
courses_and_descs_and_invalid_flags = []
repo = get_course_repo(course)
desc = get_course_desc(repo, course.active_git_commit_sha.encode())
role, participation = get_role_and_participation(request, course)
show = True
if course.hidden:
if role not in [participation_role.teaching_assistant,
participation_role.instructor]:
show = False
if role != participation_role.instructor:
show = False
if show:
courses_and_descs_and_invalid_flags.append(
courses_and_descs_and_invalid_flags.sort(key=course_sort_key)
return render(request, "course/home.html", {
"courses_and_descs_and_invalid_flags": courses_and_descs_and_invalid_flags
def check_course_state(course, role):
if course.hidden:
if role not in [participation_role.teaching_assistant,
participation_role.instructor]:
raise PermissionDenied("only course staff have access")
if role != participation_role.instructor:
raise PermissionDenied("only the instructor has access")
def course_page(request, course_identifier):
course = get_object_or_404(Course, identifier=course_identifier)
role, participation = get_role_and_participation(request, course)
commit_sha = get_active_commit_sha(course, participation)
repo = get_course_repo(course)
course_desc = get_course_desc(repo, commit_sha)
from course.content import get_processed_course_chunks
chunks = get_processed_course_chunks(course, course_desc,
role)
return render(request, "course/course-page.html", {
"course": course,
"course_desc": course_desc,
"participation": participation,
class FlowContext(object):
def __init__(self, request, course_identifier, flow_identifier, flow_visit=None):
self.flow_visit = flow_visit
self.course_identifier = course_identifier
self.flow_identifier = flow_identifier
self.course = get_object_or_404(Course, identifier=course_identifier)
self.role, self.participation = get_role_and_participation(
request, self.course)
check_course_state(self.course, self.role)
if self.flow_visit is not None:
self.commit_sha = self.flow_visit.active_git_commit_sha.encode()
else:
self.commit_sha = get_active_commit_sha(self.course, self.participation)
self.repo = get_course_repo(self.course)
self.course_desc = get_course_desc(self.repo, self.commit_sha)
self.flow_desc = get_flow_desc(self.repo, self.course,
flow_identifier, self.commit_sha)
self.permissions, self.stipulations = get_flow_permissions(
self.course_desc, self.participation, self.role,
flow_identifier, self.flow_desc)
def will_receive_feedback(self):
from course.models import flow_permission
return (
flow_permission.see_correctness in self.permissions
or flow_permission.see_answer in self.permissions)
@property
def page_count(self):
return self.flow_visit.page_count
def instantiate_flow_page_with_ctx(fctx, page_data):
from course.content import get_flow_page_desc
page_desc = get_flow_page_desc(
fctx.flow_visit, fctx.flow_desc, page_data.group_id, page_data.page_id)
from course.content import instantiate_flow_page
return instantiate_flow_page(
"course '%s', flow '%s', page '%s/%s'"
% (fctx.course_identifier, fctx.flow_identifier,
page_data.group_id, page_data.page_id),
fctx.repo, page_desc, fctx.commit_sha)
def start_flow(request, course_identifier, flow_identifier):
fctx = FlowContext(request, course_identifier, flow_identifier)
from course.models import flow_permission
if flow_permission.view not in fctx.permissions:
raise PermissionDenied()
if request.method == "POST":
from course.content import set_up_flow_visit_page_data
if ("start_no_credit" in request.POST
or "start_credit" in request.POST):
# FIXME take into account max attempts
# FIXME resumption
# FIXME view past
visit.participation = fctx.participation
visit.active_git_commit_sha = fctx.commit_sha.decode()
visit.in_progress = True
visit.for_credit = "start_credit" in request.POST
visit.save()
request.session["flow_visit_id"] = visit.id
page_count = set_up_flow_visit_page_data(fctx.repo, visit,
fctx.flow_desc, fctx.commit_sha)
visit.page_count = page_count
return redirect("course.views.view_flow_page",
course_identifier,
flow_identifier,
0)
else:
raise SuspiciousOperation("unrecognized POST action")
can_start_credit = flow_permission.start_credit in fctx.permissions
can_start_no_credit = flow_permission.start_no_credit in fctx.permissions
past_visits = (FlowVisit.objects
.filter(
participation=fctx.participation,
flow_id=flow_identifier)
.order_by("start_time"))
for pv in past_visits:
pv.result()
# FIXME take into account max attempts
# FIXME resumption
# FIXME view past
return render(request, "course/flow-start.html", {
"participation": fctx.participation,
"course_desc": fctx.course_desc,
"course": fctx.course,
"flow_desc": fctx.flow_desc,
"can_start_credit": can_start_credit,
"can_start_no_credit": can_start_no_credit,
"""This object acts as a container for all the information that a flow page
may need to render itself or respond to a POST.
"""
def __init__(self, request, course_identifier, flow_identifier,
ordinal, flow_visit):
FlowContext.__init__(self, request, course_identifier, flow_identifier,
flow_visit=flow_visit)
from course.models import FlowPageData
page_data = self.page_data = get_object_or_404(
FlowPageData, flow_visit=flow_visit, ordinal=ordinal)
from course.content import get_flow_page_desc
self.page_desc = get_flow_page_desc(
flow_visit, self.flow_desc, page_data.group_id, page_data.page_id)
self.page = instantiate_flow_page_with_ctx(self, page_data)
from course.page import PageContext
self.page_context = PageContext(course=self.course)
# {{{ dig for previous answers
previous_answer_visits = (FlowPageVisit.objects
.filter(flow_visit=flow_visit)
.filter(page_data=page_data)
.filter(answer__isnull=False)
.order_by("-visit_time"))
self.prev_answer_is_final = False
self.prev_answer = None
for prev_visit in previous_answer_visits:
self.prev_answer = prev_visit.answer
self.prev_answer_is_final = prev_visit.answer_is_final
break
# }}}
@property
def ordinal(self):
return self.page_data.ordinal
@property
def percentage(self):
return int(100*(self.ordinal+1)/self.page_count)
def create_visit(self):
page_visit = FlowPageVisit()
page_visit.flow_visit = self.flow_visit
page_visit.page_data = self.page_data
page_visit.save()
def find_current_flow_visit(request, flow_identifier):
flow_visit = None
flow_visit_id = request.session.get("flow_visit_id")
if flow_visit_id is not None:
flow_visits = list(FlowVisit.objects.filter(id=flow_visit_id))
if flow_visits and flow_visits[0].flow_id == flow_identifier:
flow_visit, = flow_visits
def render_flow_page(request, fpctx, **kwargs):
args = {
"course": fpctx.course,
"course_desc": fpctx.course_desc,
"flow_identifier": fpctx.flow_identifier,
"flow_desc": fpctx.flow_desc,
"ordinal": fpctx.ordinal,
"page_data": fpctx.page_data,
"flow_visit": fpctx.flow_visit,
"participation": fpctx.participation,
}
args.update(kwargs)
return render(request, "course/flow-page.html", args)
def add_buttons_to_form(fpctx, form):
from crispy_forms.layout import Submit
form.helper.add_input(
Submit("save", "Save answer",
css_class="col-lg-offset-2"))
if fpctx.will_receive_feedback():
form.helper.add_input(Submit("submit", "Submit final answer"))
else:
# Only offer 'save and move on' if student will receive no feedback
if fpctx.page_data.ordinal + 1 < fpctx.flow_visit.page_count:
form.helper.add_input(
Submit("save_and_next", "Save answer and move on"))
form.helper.add_input(
Submit("save_and_finish", "Save answer and finish"))
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 view_flow_page(request, course_identifier, flow_identifier, ordinal):
flow_visit = find_current_flow_visit(request, flow_identifier)
if flow_visit is None:
messages.add_message(request, messages.WARNING,
"No in-progress visit record found for this flow. "
"Redirected to flow start page.")
return redirect("course.views.start_flow",
course_identifier,
flow_identifier)
fpctx = FlowPageContext(request, course_identifier, flow_identifier,
ordinal, flow_visit)
page_context = fpctx.page_context
page_data = fpctx.page_data
if flow_permission.view not in fpctx.permissions:
raise PermissionDenied("not allowed to view flow")
if request.method == "POST":
if "finish" in request.POST:
return redirect("course.views.finish_flow",
course_identifier, flow_identifier)
# reject if previous answer was final
if fpctx.prev_answer_is_final:
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.POST)
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_visit = fpctx.flow_visit
page_visit.page_data = fpctx.page_data
fpctx.page_context, fpctx.page_data.data,
form)
page_visit.answer_is_final = pressed_button == "submit"
page_visit.save()
answer_data = page_visit.answer
answer_is_final = page_visit.answer_is_final
if (pressed_button == "save_and_next"
and not fpctx.will_receive_feedback()):
return redirect("course.views.view_flow_page",
course_identifier,
flow_identifier,
fpctx.ordinal + 1)
elif (pressed_button == "save_and_finish"
and not fpctx.will_receive_feedback()):
return redirect("course.views.finish_flow",
course_identifier, flow_identifier)
# continue at common flow page generation below
form = fpctx.page.form_with_answer(page_context, page_data.data,
page_visit.answer, page_visit.answer_is_final)
# continue at common flow page generation below
fpctx.create_visit()
answer_data = None
answer_is_final = False
# continue at common flow page generation below
answer_data = fpctx.prev_answer
answer_is_final = fpctx.prev_answer_is_final
form = fpctx.page.form_with_answer(page_context, page_data.data,
else:
form = fpctx.page.fresh_form(page_context, page_data.data)
553
554
555
556
557
558
559
560
561
562
563
564
565
566
567
568
569
570
571
572
573
574
575
576
577
578
579
580
# start common flow page generation
# defined at this point: form, answer_data, answer_is_final
if form is not None and not answer_is_final:
form = add_buttons_to_form(fpctx, form)
show_correctness = None
show_answer = None
feedback = None
if (answer_data is not None
and answer_is_final):
show_correctness = flow_permission.see_correctness in fpctx.permissions
show_answer = flow_permission.see_answer in fpctx.permissions
if show_correctness or show_answer:
feedback = fpctx.page.grade(page_context, page_data.data, answer_data)
title = fpctx.page.title(page_context, page_data.data)
body = fpctx.page.body(page_context, page_data.data)
return render_flow_page(
request, fpctx, title=title, body=body, form=form,
feedback=feedback,
show_correctness=show_correctness,
show_answer=show_answer)
581
582
583
584
585
586
587
588
589
590
591
592
593
594
595
596
597
598
599
600
601
602
603
604
605
606
607
608
609
610
611
612
613
614
615
616
617
618
619
620
621
622
623
624
625
626
627
628
629
630
631
632
633
634
635
636
637
638
639
640
641
642
643
644
645
646
647
648
649
650
651
652
653
654
655
# }}}
# {{{ finish flow
def assemble_answer_visits(flow_visit):
answer_visits = [None] * flow_visit.page_count
from course.models import FlowPageVisit
answer_page_visits = (FlowPageVisit.objects
.filter(flow_visit=flow_visit)
.filter(answer__isnull=False)
.order_by("visit_time"))
for page_visit in answer_page_visits:
answer_visits[page_visit.page_data.ordinal] = page_visit
return answer_visits
def count_answered(fctx, answer_visits):
all_page_data = (FlowPageData.objects
.filter(flow_visit=fctx.flow_visit)
.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):
def __init__(self,
points, max_points,
fully_correct_count, partially_correct_count, incorrect_count):
self.points = points
self.max_points = max_points
self.fully_correct_count = fully_correct_count
self.partially_correct_count = partially_correct_count
self.incorrect_count = incorrect_count
def points_percent(self):
return 100*self.points/self.max_points
def missed_points_percent(self):
return 100 - self.points_percent()
def total_count(self):
return (self.fully_correct_count
+ self.partially_correct_count
+ self.incorrect_count)
def fully_correct_percent(self):
return 100*self.fully_correct_count/self.total_count()
def partially_correct_percent(self):
return 100*self.partially_correct_count/self.total_count()
def incorrect_percent(self):
return 100*self.incorrect_count/self.total_count()
657
658
659
660
661
662
663
664
665
666
667
668
669
670
671
672
673
674
675
676
677
678
679
680
681
682
683
684
685
686
687
688
689
690
691
692
693
694
695
696
697
698
699
700
701
702
703
704
def gather_grade_info(fctx, answer_visits):
all_page_data = (FlowPageData.objects
.filter(flow_visit=fctx.flow_visit)
.order_by("ordinal"))
points = 0
max_points = 0
fully_correct_count = 0
partially_correct_count = 0
incorrect_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 not page.expects_answer():
continue
from course.page import PageContext
page_context = PageContext(course=fctx.course)
feedback = page.grade(page_context, page_data.data, answer_data)
max_points += page.max_points(page_data.data)
points += page.max_points(page_data.data)*feedback.correctness
if feedback.correctness == 1:
fully_correct_count += 1
elif feedback.correctness == 0:
incorrect_count += 1
else:
partially_correct_count += 1
return GradeInfo(
points=points,
max_points=max_points,
fully_correct_count=fully_correct_count,
partially_correct_count=partially_correct_count,
incorrect_count=incorrect_count)
@transaction.atomic
def finish_flow(request, course_identifier, flow_identifier):
flow_visit = find_current_flow_visit(request, flow_identifier)
if flow_visit is None:
messages.add_message(request, messages.WARNING,
"Redirected to flow start page.")
return redirect("course.views.start_flow",
course_identifier,
flow_identifier)
fctx = FlowContext(request, course_identifier, flow_identifier,
flow_visit=flow_visit)
answer_visits = assemble_answer_visits(flow_visit)
if request.method == "POST":
if "submit" not in request.POST:
raise SuspiciousOperation("odd POST parameters")
if not flow_visit.in_progress:
raise PermissionDenied("Can't end a flow that's already ended")
request.session["flow_visit_id"] = None
from django.utils.timezone import now
flow_visit.completion_time = now()
flow_visit.in_progress = False
flow_visit.save()
# mark answers as final
for answer_visit in answer_visits:
if answer_visit is not None:
answer_visit.answer_is_final = True
answer_visit.save()
grade_info = gather_grade_info(fctx, answer_visits)
from course.models import get_flow_grading_opportunity
gopp = get_flow_grading_opportunity(
fctx.course, fctx.flow_identifier, fctx.flow_desc)
from course.models import grade_state_change_types
gchange = GradeChange()
gchange.opportunity = gopp
gchange.participation = fctx.participation
gchange.state = grade_state_change_types.graded
gchange.points = grade_info.points
gchange.max_points = grade_info.max_points
gchange.creator = request.user
gchange.flow_visit = flow_visit
gchange.save()
return render(request, "course/flow-completion-grade.html", {
"course": fctx.course,
"course_desc": fctx.course_desc,
"flow_identifier": fctx.flow_identifier,
"flow_desc": fctx.flow_desc,
"body": html_body(fctx.course, fctx.flow_desc.completion_text),
"participation": fctx.participation,
"grade_info": grade_info,
})
(answered_count, unanswered_count) = count_answered(fctx, answer_visits)
if answered_count + unanswered_count == 0:
# Not serious--no questions in flow. No need to end the flow visit.
from course.content import html_body
return render(request, "course/flow-completion.html", {
"course": fctx.course,
"course_desc": fctx.course_desc,
"flow_identifier": fctx.flow_identifier,
"flow_desc": fctx.flow_desc,
"last_page_nr": fctx.page_count-1,
"body": html_body(fctx.course, fctx.flow_desc.completion_text),
})
elif not flow_visit.in_progress:
grade_info = gather_grade_info(fctx, answer_visits)
return render(request, "course/flow-completion-grade.html", {
"course": fctx.course,
"course_desc": fctx.course_desc,
"flow_identifier": fctx.flow_identifier,
"flow_desc": fctx.flow_desc,
"body": html_body(fctx.course, fctx.flow_desc.completion_text),
"participation": fctx.participation,
"grade_info": grade_info,
})
else:
# confirm ending flow
return render(request, "course/flow-confirm-completion.html", {
"course": fctx.course,
"course_desc": fctx.course_desc,
"flow_identifier": fctx.flow_identifier,
"flow_desc": fctx.flow_desc,
"last_page_nr": fctx.page_count-1,
"answered_count": answered_count,
"unanswered_count": unanswered_count,
"total_count": answered_count+unanswered_count,
})
# }}}
# vim: foldmethod=marker