add middleware GradeCreateValidCheck

core/grading/controller.go

@@ -30,7 +30,7 @@ func (controller GradingController) CreateGradeRouter(route *mux.Router) {
 	route.Use(utils.DecodeBodyMiddleware(&models.Grade{}))
 	route.Use(controller.GradeCreatePermissionCheck())
 	// TODO: check if controllerade is valid (i.e., between min and max mark)
-
+	route.Use(controller.GradeCreateValidCheck())
 	route.HandleFunc("", utils.DBCreateHandleFunc(controller.DB, true)).Methods(http.MethodPost)
 }
 

core/grading/service.go

@@ -70,3 +70,22 @@ func (controller GradingController) DownloadPermissionCheck() mux.MiddlewareFunc
 		})
 	}
 }
+
+func (controller GradingController) GradeCreateValidCheck() mux.MiddlewareFunc {
+	return func(next http.Handler) http.Handler {
+		return http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
+			data := r.Context().Value(utils.DecodeBodyContextKey).(*models.Grade)
+
+			var rubric models.Rubric
+			controller.DB.Model(&models.Rubric{}).Where("id = ?", data.RubricID).Find(&rubric)
+
+			if data.Grade < rubric.MinMark {
+				utils.HandleResponse(w, "Grade cannot below minimum mark", http.StatusBadRequest)
+			} else if data.Grade > rubric.MaxMark {
+				utils.HandleResponse(w, "Grade cannot above maximum mark", http.StatusBadRequest)
+			} else {
+				next.ServeHTTP(w, r)
+			}
+		})
+	}
+}

frontend/src/actions/moduleActions.js

@@ -459,6 +459,11 @@ export const postGrade = async (moduleId, gradeData) => {
     rubricId: gradeData.RubricID,
     comment: gradeData.Comment,
     grade: gradeData.Grade,
+  }).catch((err) => {
+    let errMessage = err.response.data.message
+    if (errMessage) {
+      alert("Grade For Rubric " + gradeData.RubricID + " : " + errMessage)
+    }
   });
 };