From bbcb3dd3b9d3f0ebba683710d1b17272d201bd68 Mon Sep 17 00:00:00 2001
From: BenjaVR <benjamin.v.renterghem@gmail.com>
Date: Fri, 1 Mar 2019 00:09:16 +0100
Subject: [PATCH] Show capacity warning per education instead of per department
 + do not close modal with mask click

---
 .../departments/DepartmentsFormModal.tsx           |  1 +
 src/components/educations/EducationsFormModal.tsx  |  1 +
 src/components/planning/PlanningDetailsModal.tsx   |  2 +-
 src/components/planning/PlanningsFormModal.tsx     | 14 ++++++++++----
 src/components/planning/PlanningsPage.tsx          |  2 ++
 .../schools/SchoolInternshipSummaryModal.tsx       |  2 +-
 src/components/schools/SchoolsFormModal.tsx        |  1 +
 src/components/students/StudentsFormModal.tsx      |  1 +
 src/components/students/StudentsTable.tsx          |  2 +-
 src/models/Department.ts                           |  8 ++++++++
 10 files changed, 27 insertions(+), 7 deletions(-)

diff --git a/src/components/departments/DepartmentsFormModal.tsx b/src/components/departments/DepartmentsFormModal.tsx
index ed0f28c..5594c0c 100644
--- a/src/components/departments/DepartmentsFormModal.tsx
+++ b/src/components/departments/DepartmentsFormModal.tsx
@@ -159,6 +159,7 @@ class DepartmentFormModal extends React.Component<DepartmentFormModalProps, IDep
                     confirmLoading={this.state.isSubmitting}
                     destroyOnClose={true}
                     afterClose={this.handleAfterClose}
+                    maskClosable={false}
                 >
                     <Form>
                         <FormItem label="Naam">
diff --git a/src/components/educations/EducationsFormModal.tsx b/src/components/educations/EducationsFormModal.tsx
index deff5b5..72b5577 100644
--- a/src/components/educations/EducationsFormModal.tsx
+++ b/src/components/educations/EducationsFormModal.tsx
@@ -63,6 +63,7 @@ class EducationFormModal extends React.Component<EducationFormModalProps, IEduca
                     okText={this.props.okText}
                     confirmLoading={this.state.isSubmitting}
                     destroyOnClose={true}
+                    maskClosable={false}
                 >
                     <Form>
                         <FormItem label="Naam">
diff --git a/src/components/planning/PlanningDetailsModal.tsx b/src/components/planning/PlanningDetailsModal.tsx
index 6997df3..98d2402 100644
--- a/src/components/planning/PlanningDetailsModal.tsx
+++ b/src/components/planning/PlanningDetailsModal.tsx
@@ -50,7 +50,7 @@ class PlanningDetailsModal extends React.Component<IPlanningDetailsModalProps, I
                 onCancel={this.handleClose}
                 width={700}
                 footer={null}
-                maskClosable={true}
+                maskClosable={false}
             >
                 {this.renderTitle(this.props.selectedDate)}
                 {this.renderDepartments(studentsPlannedThisDay)}
diff --git a/src/components/planning/PlanningsFormModal.tsx b/src/components/planning/PlanningsFormModal.tsx
index f0afcf5..2e82aab 100644
--- a/src/components/planning/PlanningsFormModal.tsx
+++ b/src/components/planning/PlanningsFormModal.tsx
@@ -8,6 +8,7 @@ import { studentsPlannedFullyInRange, studentsPlannedInDay, studentsPlannedParti
 import { nameof } from "../../helpers/nameof";
 import { FormValidationTrigger } from "../../helpers/types";
 import { Department } from "../../models/Department";
+import { Education } from "../../models/Education";
 import { IStudentInternship, Student } from "../../models/Student";
 import { StudentsRepository } from "../../services/repositories/StudentsRepository";
 import styles from "./PlanningsFormModal.module.scss";
@@ -20,6 +21,7 @@ interface IPlanningsFormModalProps {
     departments: Department[];
     areDepartmentsLoading: boolean;
     isEdit: boolean;
+    educations: Education[];
     onCloseRequest: () => void;
     submitInternship(internship: IStudentInternship): Promise<void>;
 }
@@ -92,6 +94,7 @@ class PlanningsFormModal extends React.Component<PlanningsFormModalProps, IPlann
                     okText={this.props.okText}
                     confirmLoading={this.state.isSubmitting}
                     destroyOnClose={true}
+                    maskClosable={false}
                     afterClose={this.handleFormClosed}
                 >
                     <Form onKeyDown={this.handleKeyDown}>
@@ -257,12 +260,15 @@ class PlanningsFormModal extends React.Component<PlanningsFormModalProps, IPlann
                                 return student.id !== this.props.studentToPlan.id;
                             });
 
-                        const isCrossingTheCapacityLimits = department.totalCapacity <= department.getUsedCapacity(students);
+                        const education = this.props.educations.find((edu) => this.props.studentToPlan !== undefined && edu.id === this.props.studentToPlan.educationId);
+                        const isCrossingTheCapacityLimits = education === undefined
+                            ? false
+                            : department.getCapacityForEducation(education) <= department.getUsedCapacityForEducation(students, education);
 
-                        if (isCrossingTheCapacityLimits) {
+                        if (isCrossingTheCapacityLimits && education !== undefined) {
                             const intersectingDates: moment.Moment[] = [];
                             for (const m = moment(internship.startDate); m.diff(internship.endDate, "days") <= 0; m.add(1, "day")) {
-                                if (department.totalCapacity <= department.getUsedCapacity(studentsPlannedInDay(students, m))) {
+                                if (department.totalCapacity <= department.getUsedCapacityForEducation(studentsPlannedInDay(students, m), education)) {
                                     intersectingDates.push(moment(m));
                                 }
                             }
@@ -270,7 +276,7 @@ class PlanningsFormModal extends React.Component<PlanningsFormModalProps, IPlann
                                 title: "Capaciteit overschreden",
                                 content: (
                                     <React.Fragment>
-                                        <p>Er is minstens één geselecteerde dag waar de capaciteit voor <b>{department.name}</b> werd overschreden.</p>
+                                        <p>Er is minstens één geselecteerde dag waar de capaciteit voor <b>{department.name}</b> (opleiding <b>{education.name}</b>) werd overschreden.</p>
                                         <p>Bent u zeker dat u wilt verdergaan?</p>
                                         <ul>
                                             {intersectingDates.map((intersectingDate) => {
diff --git a/src/components/planning/PlanningsPage.tsx b/src/components/planning/PlanningsPage.tsx
index afff7ce..fb3e1f4 100644
--- a/src/components/planning/PlanningsPage.tsx
+++ b/src/components/planning/PlanningsPage.tsx
@@ -166,6 +166,7 @@ class PlanningsPage extends React.Component<PlanningsPageProps, IPlanningsPageSt
                     departments={this.state.departments}
                     areDepartmentsLoading={this.state.areDepartmentsLoading}
                     isEdit={false}
+                    educations={this.state.educations}
                 />
                 <PlanningsFormModal
                     title={this.state.plannedStudentToEdit === undefined
@@ -179,6 +180,7 @@ class PlanningsPage extends React.Component<PlanningsPageProps, IPlanningsPageSt
                     departments={this.state.departments}
                     areDepartmentsLoading={this.state.areDepartmentsLoading}
                     isEdit={true}
+                    educations={this.state.educations}
                 />
                 <PlanningDetailsModal
                     departments={this.state.departments}
diff --git a/src/components/schools/SchoolInternshipSummaryModal.tsx b/src/components/schools/SchoolInternshipSummaryModal.tsx
index 6db6161..ec90271 100644
--- a/src/components/schools/SchoolInternshipSummaryModal.tsx
+++ b/src/components/schools/SchoolInternshipSummaryModal.tsx
@@ -98,7 +98,7 @@ class SchoolInternshipSummaryModal extends React.Component<SchoolInternshipSumma
                 visible={this.props.isVisible}
                 footer={null}
                 closable={true}
-                maskClosable={true}
+                maskClosable={false}
                 onCancel={this.handleClose}
                 destroyOnClose={true}
                 width={700}
diff --git a/src/components/schools/SchoolsFormModal.tsx b/src/components/schools/SchoolsFormModal.tsx
index 51f498c..1eb5add 100644
--- a/src/components/schools/SchoolsFormModal.tsx
+++ b/src/components/schools/SchoolsFormModal.tsx
@@ -63,6 +63,7 @@ class SchoolsFormModal extends React.Component<SchoolFormModalProps, ISchoolsFor
                     okText={this.props.okText}
                     confirmLoading={this.state.isSubmitting}
                     destroyOnClose={true}
+                    maskClosable={false}
                 >
                     <Form>
                         <FormItem label="Naam">
diff --git a/src/components/students/StudentsFormModal.tsx b/src/components/students/StudentsFormModal.tsx
index b4e6935..0167607 100644
--- a/src/components/students/StudentsFormModal.tsx
+++ b/src/components/students/StudentsFormModal.tsx
@@ -90,6 +90,7 @@ class StudentsFormModal extends React.Component<StudentFormModalProps, IStudents
                     okText={this.props.okText}
                     confirmLoading={this.state.isSubmitting}
                     destroyOnClose={true}
+                    maskClosable={false}
                 >
                     <Form layout="horizontal">
                         <FormItem label="Voornaam">
diff --git a/src/components/students/StudentsTable.tsx b/src/components/students/StudentsTable.tsx
index faf0c30..302ae1c 100644
--- a/src/components/students/StudentsTable.tsx
+++ b/src/components/students/StudentsTable.tsx
@@ -81,7 +81,7 @@ class StudentsTable extends React.Component<IStudentsTableProps, IStudentTableSt
                     title={`Stage voor ${selectedStudent !== undefined ? selectedStudent.fullName : ""}`}
                     visible={this.state.isPlanningDetailsOpen}
                     closable={true}
-                    maskClosable={true}
+                    maskClosable={false}
                     onCancel={this.handleClosePlanningDetails}
                     footer={(
                         <div className={specificStyles.studentInternshipModalFooter}>
diff --git a/src/models/Department.ts b/src/models/Department.ts
index 444a9f1..03dc649 100644
--- a/src/models/Department.ts
+++ b/src/models/Department.ts
@@ -40,6 +40,14 @@ export class Department extends ModelBase<IDepartment> {
         }).length;
     }
 
+    public getUsedCapacityForEducation(students: Student[], education: Education): number {
+        return students.filter((student) => {
+            return student.internship !== undefined
+                && student.internship.departmentId === this.id
+                && student.educationId === education.id;
+        }).length;
+    }
+
     public getCapacityForEducation(education: Education): number {
         const educationCapacity = this.capacityPerEducation.find((capacity) => capacity.educationId === education.id);
         return educationCapacity === undefined