Merge pull request #316 from Khenus/Khenus

Fix Bug in storage check
AY2021S1-CS2113T-F12-1 · Nov 9, 2020 · 7a3f65a · 7a3f65a
2 parents 47ba12f + 4075344
commit 7a3f65a
Showing 1 changed file with 2 additions and 6 deletions.
diff --git a/src/main/java/seedu/duke/storage/commons/FieldValidator.java b/src/main/java/seedu/duke/storage/commons/FieldValidator.java
@@ -99,12 +99,8 @@ public boolean validate() {
             if (moduleWithLargestCap != null) {
                 ModuleValidator validator = new ModuleValidator();
 
-                if (!validator.isRetakeGrade(moduleWithLargestCap.getGrade())) {
-                    isAllFieldValid = false;
-                    details[INVALID_GRADE]++;
-                }
-
-                if (isAllFieldValid && moduleWithLargestCap.getCap() >= testModule.getCap()) {
+                if (!validator.isRetakeGrade(moduleWithLargestCap.getGrade())
+                        && moduleWithLargestCap.getCap() >= testModule.getCap()) {
                     isAllFieldValid = false;
                     details[INVALID_GRADE]++;
                 }