c77fb3d0e5
Serves Demo Scene 6. mc.grading.scale + mc.grading.interval are pure
data (threshold/letter/point/description rows) - CBSE, ICSE, IB,
Cambridge, Ontario and a plain percentage scale all shipped as data
in data/mc_grading_scale_data.xml, noupdate="1" so a school's own
edits survive a module upgrade. mc.mark.grade resolves via
mc.program.grading_scale_id (new field on mc.program, since grading
board is a per-program concept already carrying "board" from O1) -
the moment this needed an `if board == "CBSE"` anywhere, per
shared/DOMAIN_MODEL.md sec 5, it would have been the wrong design;
verified instead that swapping a program's scale recomputes an
existing mark's grade with zero code involved
(test_changing_scale_recomputes_existing_marks).
Grid mark entry ("whole class on one screen, tab between fields,
keyboard only") is a plain editable list on mc.mark, filtered to one
exam - that native Odoo behavior already gives real Tab-key
navigation with no custom widget. mc.exam.action_enter_marks()
pre-creates a row per actively-enrolled student before opening it
(mirroring O4's bulk-attendance idempotency: creating it again never
duplicates or resets an already-entered mark).
The report card is a real rendered PDF, not just a template read by
inspection - rendered it for Aditya via odoo shell
(report._render_qweb_pdf), pulled the bytes out of the container, and
read the actual PDF: one A4 page, no clipped columns, correct grades
per subject matching the CBSE scale (92->A1, 85->A2, 78->B1, 67->B2,
58->C1), signature block, term correctly resolved. That last part
exposed a real logic bug before it shipped: the term-resolution
method originally preferred "today's date" over "the term with
marks", which would have shown an empty Term 1 when generating the
card *today* even though the demo's graded marks are all in Term 2 -
a report card is generated to review a term's results, typically
after that term's exams are done (often during the *next* term), so
marks-with-data now wins over the calendar, falling back to today's
date only for a student with nothing graded yet.
Attendance summary on the report degrades to None when
mc_education_attendance isn't installed (checked directly, not
assumed) - this module depends on mc_education_base only, matching
"each mc_education_* module installs independently" (CLAUDE.md sec
1.4), while still showing the real summary when both are installed
together for the actual demo.
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
46 lines
1.9 KiB
Python
46 lines
1.9 KiB
Python
from odoo import api, fields, models
|
|
from odoo.exceptions import ValidationError
|
|
|
|
|
|
class McMark(models.Model):
|
|
_name = "mc.mark"
|
|
_description = "Mark"
|
|
_order = "exam_id, student_id"
|
|
_rec_name = "display_name"
|
|
|
|
exam_id = fields.Many2one("mc.exam", string="Exam", required=True, ondelete="cascade")
|
|
student_id = fields.Many2one("mc.student", string="Student", required=True, ondelete="restrict")
|
|
marks_obtained = fields.Float(string="Marks Obtained", required=True)
|
|
grade = fields.Char(string="Grade", compute="_compute_grade", store=True)
|
|
|
|
_exam_student_uniq = models.Constraint(
|
|
"unique(exam_id, student_id)",
|
|
"This student already has marks recorded for this exam.",
|
|
)
|
|
|
|
@api.depends("student_id.name", "exam_id.display_name")
|
|
def _compute_display_name(self):
|
|
for mark in self:
|
|
mark.display_name = "%s - %s" % (mark.student_id.name or "?", mark.exam_id.display_name or "?")
|
|
|
|
@api.depends("marks_obtained", "exam_id.max_marks", "exam_id.batch_id.program_id.grading_scale_id.interval_ids.threshold")
|
|
def _compute_grade(self):
|
|
for mark in self:
|
|
scale = mark.exam_id.batch_id.program_id.grading_scale_id
|
|
if not scale or not mark.exam_id.max_marks:
|
|
mark.grade = False
|
|
continue
|
|
percentage = mark.marks_obtained / mark.exam_id.max_marks * 100
|
|
interval = scale.get_grade_interval(percentage)
|
|
mark.grade = interval.letter if interval else False
|
|
|
|
@api.constrains("marks_obtained", "exam_id")
|
|
def _check_marks_within_range(self):
|
|
for mark in self:
|
|
if mark.marks_obtained < 0 or mark.marks_obtained > mark.exam_id.max_marks:
|
|
raise ValidationError(
|
|
"Marks obtained (%.2f) must be between 0 and the exam's max marks (%.2f)." % (
|
|
mark.marks_obtained, mark.exam_id.max_marks,
|
|
)
|
|
)
|