Merge pull request #79 from appbaseio/fix/patch-response

fix: consistent response for PATCH endpoints

Author: lakhansamani

plugins/permissions/handlers.go

@@ -230,9 +230,9 @@ func (p *permissions) patchPermission() http.HandlerFunc {
 			}
 		}
 
-		raw, err := p.es.patchPermission(req.Context(), username, patch)
-		if err == nil {
-			util.WriteBackRaw(w, raw, http.StatusOK)
+		_, err2 := p.es.patchPermission(req.Context(), username, patch)
+		if err2 == nil {
+			util.WriteBackMessage(w, "Permission is updated successfully", http.StatusOK)
 			return
 		}
 

plugins/users/handlers.go

@@ -210,9 +210,9 @@ func (u *Users) patchUser() http.HandlerFunc {
 			}
 		}
 
-		raw, err := u.es.patchUser(req.Context(), username, patch)
-		if err == nil {
-			util.WriteBackRaw(w, raw, http.StatusOK)
+		_, err2 := u.es.patchUser(req.Context(), username, patch)
+		if err2 == nil {
+			util.WriteBackMessage(w, "User is updated successfully", http.StatusOK)
 			return
 		}
 
@@ -281,9 +281,9 @@ func (u *Users) patchUserWithUsername() http.HandlerFunc {
 			}
 		}
 
-		raw, err := u.es.patchUser(req.Context(), username, patch)
-		if err == nil {
-			util.WriteBackRaw(w, raw, http.StatusOK)
+		_, err2 := u.es.patchUser(req.Context(), username, patch)
+		if err2 == nil {
+			util.WriteBackMessage(w, "User is updated successfully", http.StatusOK)
 			return
 		}