X-Git-Url: https://git.pterodactylus.net/?a=blobdiff_plain;f=src%2Fmain%2Fjava%2Fnet%2Fpterodactylus%2Fsone%2Fweb%2FEditProfilePage.java;h=df5a35220070673f60246834e6e1e1b3b7cdb409;hb=2e03e9dddbea4b81aacaf1aa316f5c3ccffd4bf9;hp=62957c2ebc154418148267ec0d56b6aa5dd79b61;hpb=d5efb086bee8f103cbe90c7a953ffbb7ff27b689;p=Sone.git

diff --git a/src/main/java/net/pterodactylus/sone/web/EditProfilePage.java b/src/main/java/net/pterodactylus/sone/web/EditProfilePage.java
index 62957c2..df5a352 100644
--- a/src/main/java/net/pterodactylus/sone/web/EditProfilePage.java
+++ b/src/main/java/net/pterodactylus/sone/web/EditProfilePage.java
@@ -17,20 +17,23 @@
 
 package net.pterodactylus.sone.web;
 
+import static net.pterodactylus.sone.data.Identified.GET_ID;
+
 import java.util.List;
 
 import net.pterodactylus.sone.data.Profile;
 import net.pterodactylus.sone.data.Profile.Field;
 import net.pterodactylus.sone.data.Sone;
 import net.pterodactylus.sone.web.page.FreenetRequest;
+import net.pterodactylus.util.number.Numbers;
 import net.pterodactylus.util.template.Template;
 import net.pterodactylus.util.template.TemplateContext;
 import net.pterodactylus.util.web.Method;
 
-import com.google.common.primitives.Ints;
-
 import freenet.clients.http.ToadletContext;
 
+import com.google.common.base.Optional;
+
 /**
  * This page lets the user edit her profile.
  *
@@ -54,9 +57,6 @@ public class EditProfilePage extends SoneTemplatePage {
 	// TEMPLATEPAGE METHODS
 	//
 
-	/**
-	 * {@inheritDoc}
-	 */
 	@Override
 	protected void processTemplate(FreenetRequest request, TemplateContext templateContext) throws RedirectException {
 		super.processTemplate(request, templateContext);
@@ -76,18 +76,16 @@ public class EditProfilePage extends SoneTemplatePage {
 				firstName = request.getHttpRequest().getPartAsStringFailsafe("first-name", 256).trim();
 				middleName = request.getHttpRequest().getPartAsStringFailsafe("middle-name", 256).trim();
 				lastName = request.getHttpRequest().getPartAsStringFailsafe("last-name", 256).trim();
-				birthDay = Ints.tryParse(request.getHttpRequest().getPartAsStringFailsafe("birth-day", 256).trim());
-				birthMonth = Ints.tryParse(request.getHttpRequest().getPartAsStringFailsafe("birth-month", 256).trim());
-				birthYear = Ints.tryParse(request.getHttpRequest().getPartAsStringFailsafe("birth-year", 256).trim());
+				birthDay = Numbers.safeParseInteger(request.getHttpRequest().getPartAsStringFailsafe("birth-day", 256).trim());
+				birthMonth = Numbers.safeParseInteger(request.getHttpRequest().getPartAsStringFailsafe("birth-month", 256).trim());
+				birthYear = Numbers.safeParseInteger(request.getHttpRequest().getPartAsStringFailsafe("birth-year", 256).trim());
 				avatarId = request.getHttpRequest().getPartAsStringFailsafe("avatarId", 36);
-				profile.setFirstName(firstName.length() > 0 ? firstName : null);
-				profile.setMiddleName(middleName.length() > 0 ? middleName : null);
-				profile.setLastName(lastName.length() > 0 ? lastName : null);
-				profile.setBirthDay(birthDay).setBirthMonth(birthMonth).setBirthYear(birthYear);
-				profile.setAvatar(webInterface.getCore().getImage(avatarId, false));
+				profile.modify().setFirstName(getNameFromFormField(firstName)).setMiddleName(getNameFromFormField(middleName)).setLastName(getNameFromFormField(lastName)).update();
+				profile.modify().setBirthDay(birthDay).setBirthMonth(birthMonth).setBirthYear(birthYear).update();
+				profile.setAvatar(webInterface.getCore().getImage(avatarId).transform(GET_ID));
 				for (Field field : fields) {
 					String value = request.getHttpRequest().getPartAsStringFailsafe("field-" + field.getId(), 400);
-					field.setValue(value);
+					profile.setField(field, value);
 				}
 				currentSone.setProfile(profile);
 				webInterface.getCore().touchConfiguration();
@@ -111,21 +109,21 @@ public class EditProfilePage extends SoneTemplatePage {
 				}
 				id = getFieldId(request, "move-up-field-");
 				if (id != null) {
-					Field field = profile.getFieldById(id);
-					if (field == null) {
+					Optional<Field> field = profile.getFieldById(id);
+					if (!field.isPresent()) {
 						throw new RedirectException("invalid.html");
 					}
-					profile.moveFieldUp(field);
+					profile.moveFieldUp(field.get());
 					currentSone.setProfile(profile);
 					throw new RedirectException("editProfile.html#profile-fields");
 				}
 				id = getFieldId(request, "move-down-field-");
 				if (id != null) {
-					Field field = profile.getFieldById(id);
-					if (field == null) {
+					Optional<Field> field = profile.getFieldById(id);
+					if (!field.isPresent()) {
 						throw new RedirectException("invalid.html");
 					}
-					profile.moveFieldDown(field);
+					profile.moveFieldDown(field.get());
 					currentSone.setProfile(profile);
 					throw new RedirectException("editProfile.html#profile-fields");
 				}
@@ -145,6 +143,10 @@ public class EditProfilePage extends SoneTemplatePage {
 		templateContext.set("fields", fields);
 	}
 
+	private String getNameFromFormField(String name) {
+		return name.length() > 0 ? name : null;
+	}
+
 	//
 	// PRIVATE METHODS
 	//