style: format code with black
diff --git a/erpnext/education/api.py b/erpnext/education/api.py
index 636b948..8fd1725 100644
--- a/erpnext/education/api.py
+++ b/erpnext/education/api.py
@@ -12,11 +12,12 @@
def get_course(program):
- '''Return list of courses for a particular program
+ """Return list of courses for a particular program
:param program: Program
- '''
- courses = frappe.db.sql('''select course, course_name from `tabProgram Course` where parent=%s''',
- (program), as_dict=1)
+ """
+ courses = frappe.db.sql(
+ """select course, course_name from `tabProgram Course` where parent=%s""", (program), as_dict=1
+ )
return courses
@@ -26,24 +27,24 @@
:param source_name: Student Applicant.
"""
- frappe.publish_realtime('enroll_student_progress', {"progress": [1, 4]}, user=frappe.session.user)
- student = get_mapped_doc("Student Applicant", source_name,
- {"Student Applicant": {
- "doctype": "Student",
- "field_map": {
- "name": "student_applicant"
- }
- }}, ignore_permissions=True)
+ frappe.publish_realtime("enroll_student_progress", {"progress": [1, 4]}, user=frappe.session.user)
+ student = get_mapped_doc(
+ "Student Applicant",
+ source_name,
+ {"Student Applicant": {"doctype": "Student", "field_map": {"name": "student_applicant"}}},
+ ignore_permissions=True,
+ )
student.save()
- student_applicant = frappe.db.get_value("Student Applicant", source_name,
- ["student_category", "program"], as_dict=True)
+ student_applicant = frappe.db.get_value(
+ "Student Applicant", source_name, ["student_category", "program"], as_dict=True
+ )
program_enrollment = frappe.new_doc("Program Enrollment")
program_enrollment.student = student.name
program_enrollment.student_category = student_applicant.student_category
program_enrollment.student_name = student.title
program_enrollment.program = student_applicant.program
- frappe.publish_realtime('enroll_student_progress', {"progress": [2, 4]}, user=frappe.session.user)
+ frappe.publish_realtime("enroll_student_progress", {"progress": [2, 4]}, user=frappe.session.user)
return program_enrollment
@@ -58,11 +59,15 @@
if course_schedule:
return frappe.get_list("Student Attendance", filters={"course_schedule": course_schedule})
else:
- return frappe.get_list("Student Attendance", filters={"student_group": student_group, "date": date})
+ return frappe.get_list(
+ "Student Attendance", filters={"student_group": student_group, "date": date}
+ )
@frappe.whitelist()
-def mark_attendance(students_present, students_absent, course_schedule=None, student_group=None, date=None):
+def mark_attendance(
+ students_present, students_absent, course_schedule=None, student_group=None, date=None
+):
"""Creates Multiple Attendance Records.
:param students_present: Students Present JSON.
@@ -73,26 +78,36 @@
"""
if student_group:
- academic_year = frappe.db.get_value('Student Group', student_group, 'academic_year')
+ academic_year = frappe.db.get_value("Student Group", student_group, "academic_year")
if academic_year:
- year_start_date, year_end_date = frappe.db.get_value('Academic Year', academic_year, ['year_start_date', 'year_end_date'])
+ year_start_date, year_end_date = frappe.db.get_value(
+ "Academic Year", academic_year, ["year_start_date", "year_end_date"]
+ )
if getdate(date) < getdate(year_start_date) or getdate(date) > getdate(year_end_date):
- frappe.throw(_('Attendance cannot be marked outside of Academic Year {0}').format(academic_year))
+ frappe.throw(
+ _("Attendance cannot be marked outside of Academic Year {0}").format(academic_year)
+ )
present = json.loads(students_present)
absent = json.loads(students_absent)
for d in present:
- make_attendance_records(d["student"], d["student_name"], "Present", course_schedule, student_group, date)
+ make_attendance_records(
+ d["student"], d["student_name"], "Present", course_schedule, student_group, date
+ )
for d in absent:
- make_attendance_records(d["student"], d["student_name"], "Absent", course_schedule, student_group, date)
+ make_attendance_records(
+ d["student"], d["student_name"], "Absent", course_schedule, student_group, date
+ )
frappe.db.commit()
frappe.msgprint(_("Attendance has been marked successfully."))
-def make_attendance_records(student, student_name, status, course_schedule=None, student_group=None, date=None):
+def make_attendance_records(
+ student, student_name, status, course_schedule=None, student_group=None, date=None
+):
"""Creates/Update Attendance Record.
:param student: Student.
@@ -100,13 +115,15 @@
:param course_schedule: Course Schedule.
:param status: Status (Present/Absent)
"""
- student_attendance = frappe.get_doc({
- "doctype": "Student Attendance",
- "student": student,
- "course_schedule": course_schedule,
- "student_group": student_group,
- "date": date
- })
+ student_attendance = frappe.get_doc(
+ {
+ "doctype": "Student Attendance",
+ "student": student,
+ "course_schedule": course_schedule,
+ "student_group": student_group,
+ "date": date,
+ }
+ )
if not student_attendance:
student_attendance = frappe.new_doc("Student Attendance")
student_attendance.student = student
@@ -125,8 +142,7 @@
:param student: Student.
"""
- guardians = frappe.get_all("Student Guardian", fields=["guardian"] ,
- filters={"parent": student})
+ guardians = frappe.get_all("Student Guardian", fields=["guardian"], filters={"parent": student})
return guardians
@@ -137,11 +153,19 @@
:param student_group: Student Group.
"""
if include_inactive:
- students = frappe.get_all("Student Group Student", fields=["student", "student_name"] ,
- filters={"parent": student_group}, order_by= "group_roll_number")
+ students = frappe.get_all(
+ "Student Group Student",
+ fields=["student", "student_name"],
+ filters={"parent": student_group},
+ order_by="group_roll_number",
+ )
else:
- students = frappe.get_all("Student Group Student", fields=["student", "student_name"] ,
- filters={"parent": student_group, "active": 1}, order_by= "group_roll_number")
+ students = frappe.get_all(
+ "Student Group Student",
+ fields=["student", "student_name"],
+ filters={"parent": student_group, "active": 1},
+ order_by="group_roll_number",
+ )
return students
@@ -152,8 +176,9 @@
:param program: Program.
:param academic_term: Academic Term.
"""
- fee_structure = frappe.db.get_values("Fee Structure", {"program": program,
- "academic_term": academic_term}, 'name', as_dict=True)
+ fee_structure = frappe.db.get_values(
+ "Fee Structure", {"program": program, "academic_term": academic_term}, "name", as_dict=True
+ )
return fee_structure[0].name if fee_structure else None
@@ -164,7 +189,12 @@
:param fee_structure: Fee Structure.
"""
if fee_structure:
- fs = frappe.get_all("Fee Component", fields=["fees_category", "description", "amount"] , filters={"parent": fee_structure}, order_by= "idx")
+ fs = frappe.get_all(
+ "Fee Component",
+ fields=["fees_category", "description", "amount"],
+ filters={"parent": fee_structure},
+ order_by="idx",
+ )
return fs
@@ -175,8 +205,12 @@
:param program: Program.
:param student_category: Student Category
"""
- fs = frappe.get_all("Program Fee", fields=["academic_term", "fee_structure", "due_date", "amount"] ,
- filters={"parent": program, "student_category": student_category }, order_by= "idx")
+ fs = frappe.get_all(
+ "Program Fee",
+ fields=["academic_term", "fee_structure", "due_date", "amount"],
+ filters={"parent": program, "student_category": student_category},
+ order_by="idx",
+ )
return fs
@@ -198,18 +232,23 @@
:param filters: Filters (JSON).
"""
from frappe.desk.calendar import get_event_conditions
+
conditions = get_event_conditions("Course Schedule", filters)
- data = frappe.db.sql("""select name, course, color,
+ data = frappe.db.sql(
+ """select name, course, color,
timestamp(schedule_date, from_time) as from_time,
timestamp(schedule_date, to_time) as to_time,
room, student_group, 0 as 'allDay'
from `tabCourse Schedule`
where ( schedule_date between %(start)s and %(end)s )
- {conditions}""".format(conditions=conditions), {
- "start": start,
- "end": end
- }, as_dict=True, update={"allDay": 0})
+ {conditions}""".format(
+ conditions=conditions
+ ),
+ {"start": start, "end": end},
+ as_dict=True,
+ update={"allDay": 0},
+ )
return data
@@ -220,8 +259,12 @@
:param Course: Course
"""
- return frappe.get_all("Course Assessment Criteria",
- fields=["assessment_criteria", "weightage"], filters={"parent": course}, order_by= "idx")
+ return frappe.get_all(
+ "Course Assessment Criteria",
+ fields=["assessment_criteria", "weightage"],
+ filters={"parent": course},
+ order_by="idx",
+ )
@frappe.whitelist()
@@ -233,17 +276,14 @@
student_result = {}
for d in result.details:
student_result.update({d.assessment_criteria: [cstr(d.score), d.grade]})
- student_result.update({
- "total_score": [cstr(result.total_score), result.grade],
- "comment": result.comment
- })
- student.update({
- "assessment_details": student_result,
- "docstatus": result.docstatus,
- "name": result.name
- })
+ student_result.update(
+ {"total_score": [cstr(result.total_score), result.grade], "comment": result.comment}
+ )
+ student.update(
+ {"assessment_details": student_result, "docstatus": result.docstatus, "name": result.name}
+ )
else:
- student.update({'assessment_details': None})
+ student.update({"assessment_details": None})
return student_list
@@ -253,8 +293,12 @@
:param Assessment Plan: Assessment Plan
"""
- return frappe.get_all("Assessment Plan Criteria",
- fields=["assessment_criteria", "maximum_score", "docstatus"], filters={"parent": assessment_plan}, order_by= "idx")
+ return frappe.get_all(
+ "Assessment Plan Criteria",
+ fields=["assessment_criteria", "maximum_score", "docstatus"],
+ filters={"parent": assessment_plan},
+ order_by="idx",
+ )
@frappe.whitelist()
@@ -264,8 +308,10 @@
:param Student: Student
:param Assessment Plan: Assessment Plan
"""
- results = frappe.get_all("Assessment Result", filters={"student": student,
- "assessment_plan": assessment_plan, "docstatus": ("!=", 2)})
+ results = frappe.get_all(
+ "Assessment Result",
+ filters={"student": student, "assessment_plan": assessment_plan, "docstatus": ("!=", 2)},
+ )
if results:
return frappe.get_doc("Assessment Result", results[0])
else:
@@ -280,11 +326,13 @@
:param Percentage: Score Percentage Percentage
"""
grading_scale_intervals = {}
- if not hasattr(frappe.local, 'grading_scale'):
- grading_scale = frappe.get_all("Grading Scale Interval", fields=["grade_code", "threshold"], filters={"parent": grading_scale})
+ if not hasattr(frappe.local, "grading_scale"):
+ grading_scale = frappe.get_all(
+ "Grading Scale Interval", fields=["grade_code", "threshold"], filters={"parent": grading_scale}
+ )
frappe.local.grading_scale = grading_scale
for d in frappe.local.grading_scale:
- grading_scale_intervals.update({d.threshold:d.grade_code})
+ grading_scale_intervals.update({d.threshold: d.grade_code})
intervals = sorted(grading_scale_intervals.keys(), key=float, reverse=True)
for interval in intervals:
if flt(percentage) >= interval:
@@ -297,21 +345,22 @@
@frappe.whitelist()
def mark_assessment_result(assessment_plan, scores):
- student_score = json.loads(scores);
+ student_score = json.loads(scores)
assessment_details = []
for criteria in student_score.get("assessment_details"):
- assessment_details.append({
- "assessment_criteria": criteria,
- "score": flt(student_score["assessment_details"][criteria])
- })
+ assessment_details.append(
+ {"assessment_criteria": criteria, "score": flt(student_score["assessment_details"][criteria])}
+ )
assessment_result = get_assessment_result_doc(student_score["student"], assessment_plan)
- assessment_result.update({
- "student": student_score.get("student"),
- "assessment_plan": assessment_plan,
- "comment": student_score.get("comment"),
- "total_score":student_score.get("total_score"),
- "details": assessment_details
- })
+ assessment_result.update(
+ {
+ "student": student_score.get("student"),
+ "assessment_plan": assessment_plan,
+ "comment": student_score.get("comment"),
+ "total_score": student_score.get("total_score"),
+ "details": assessment_details,
+ }
+ )
assessment_result.save()
details = {}
for d in assessment_result.details:
@@ -321,7 +370,7 @@
"student": assessment_result.student,
"total_score": assessment_result.total_score,
"grade": assessment_result.grade,
- "details": details
+ "details": details,
}
return assessment_result_dict
@@ -332,15 +381,17 @@
student_list = get_student_group_students(student_group)
for i, student in enumerate(student_list):
doc = get_result(student.student, assessment_plan)
- if doc and doc.docstatus==0:
+ if doc and doc.docstatus == 0:
total_result += 1
doc.submit()
return total_result
def get_assessment_result_doc(student, assessment_plan):
- assessment_result = frappe.get_all("Assessment Result", filters={"student": student,
- "assessment_plan": assessment_plan, "docstatus": ("!=", 2)})
+ assessment_result = frappe.get_all(
+ "Assessment Result",
+ filters={"student": student, "assessment_plan": assessment_plan, "docstatus": ("!=", 2)},
+ )
if assessment_result:
doc = frappe.get_doc("Assessment Result", assessment_result[0])
if doc.docstatus == 0:
@@ -369,10 +420,12 @@
email_list.append(email)
add_subscribers(name, email_list)
+
@frappe.whitelist()
def get_current_enrollment(student, academic_year=None):
current_academic_year = academic_year or frappe.defaults.get_defaults().academic_year
- program_enrollment_list = frappe.db.sql('''
+ program_enrollment_list = frappe.db.sql(
+ """
select
name as program_enrollment, student_name, program, student_batch_name as student_batch,
student_category, academic_term, academic_year
@@ -380,7 +433,10 @@
`tabProgram Enrollment`
where
student = %s and academic_year = %s
- order by creation''', (student, current_academic_year), as_dict=1)
+ order by creation""",
+ (student, current_academic_year),
+ as_dict=1,
+ )
if program_enrollment_list:
return program_enrollment_list[0]