Merge pull request #122 from KamJiaYue/branch-add-chart

Update errors
AY2324S1-CS2103T-W13-2 · Nov 2, 2023 · 3f2a004 · 3f2a004
2 parents db80eab + 3703b03
commit 3f2a004
Show file tree

Hide file tree

Showing 2 changed files with 0 additions and 14 deletions.
diff --git a/src/main/java/seedu/address/logic/OrganizeData.java b/src/main/java/seedu/address/logic/OrganizeData.java
@@ -133,16 +133,6 @@ public static Map<String, Integer> bySubject(Model model) {
         return columnValueMapping;
     }
 
-    public static Map<String, Integer> byMonth(Model model) {
-        Map<String, Integer> columnValueMapping = new HashMap<>();
-
-        ObservableList<Student> studentList = model.getFilteredPersonList();
-
-        studentList.stream().forEach(student -> student.putSubjectsByMonth(columnValueMapping));
-
-        return columnValueMapping;
-    }
-
     /**
      * Maps Month with the number of students
      * @param model instance of Model subclass, e.g. ModelManager instance

diff --git a/src/main/java/seedu/address/model/person/Student.java b/src/main/java/seedu/address/model/person/Student.java
@@ -111,10 +111,6 @@ public boolean isSamePerson(Student otherStudent) {
     }
 
 
-    public void putSubjectsByMonth(Map<String, Integer> map) {
-        getSubjects().forEach(subject -> subject.putToMapByMonth(map));
-    }
-
     /**
      * Returns true if both persons have the same identity and data fields.
      * This defines a stronger notion of equality between two persons.