From 58381fa3e18936bf21db19f60ee8a408291f0c2a Mon Sep 17 00:00:00 2001
From: Sola <loli@sola.love>
Date: Fri, 18 Dec 2015 03:12:20 +0800
Subject: [PATCH] add more information to profile edit

---
 .../sola/netsupport/api/ProfileModify.java    | 23 ++++++++++---------
 1 file changed, 12 insertions(+), 11 deletions(-)

diff --git a/src/main/java/love/sola/netsupport/api/ProfileModify.java b/src/main/java/love/sola/netsupport/api/ProfileModify.java
index 1b6c2e2..083b082 100644
--- a/src/main/java/love/sola/netsupport/api/ProfileModify.java
+++ b/src/main/java/love/sola/netsupport/api/ProfileModify.java
@@ -60,17 +60,18 @@ public class ProfileModify extends HttpServlet {
 		int block = checkBlock(request.getParameter("block"));
 		int room = checkRoom(request.getParameter("room"), block);
 		long phone = checkPhoneNumber(request.getParameter("phone"));
-		if (netAccount != null) {
-			u.setIsp(isp);
-			u.setNetAccount(netAccount);
-		}
-		if (room != -1) {
-			u.setBlock(block);
-			u.setRoom(room);
-		}
-		if (phone != -1) {
-			u.setPhone(phone);
-		}
+		if (room == -1)
+			return new Response(Response.ResponseCode.REQUEST_FAILED, "Invalid_Room");
+		if (phone == -1)
+			return new Response(Response.ResponseCode.REQUEST_FAILED, "Invalid_Phone_Number");
+		if (netAccount == null)
+			return new Response(Response.ResponseCode.REQUEST_FAILED, "Invalid_Account");
+
+		u.setIsp(isp);
+		u.setNetAccount(netAccount);
+		u.setBlock(block);
+		u.setRoom(room);
+		u.setPhone(phone);
 		try {
 			TableUser.update(u);
 		} catch (ConstraintViolationException e) {