Validate owner details

Lloyd Watkin · Lloyd Watkin · commit 174f3d4e157d · 2013-11-29T11:28:08.000Z
diff --git a/src/main/java/org/buddycloud/channelserver/channel/node/configuration/field/Owner.java b/src/main/java/org/buddycloud/channelserver/channel/node/configuration/field/Owner.java
@@ -1,5 +1,7 @@
 package org.buddycloud.channelserver.channel.node.configuration.field;
 
+import org.xmpp.packet.JID;
+
 public class Owner extends Field
 {
 	public static final String FIELD_NAME    = "pubsub#owner";
@@ -12,6 +14,12 @@ public Owner()
 	
 	public boolean isValid()
 	{
-		return true;
+		try {
+			JID owner = new JID(getValue());
+			owner.toBareJID();
+			return true;
+		} catch (IllegalArgumentException e) {
+			return false;
+		}
 	}
 }

Original file line number	Diff line number	Diff line change
`@@ -1,5 +1,7 @@`
`1`	`1`	`package org.buddycloud.channelserver.channel.node.configuration.field;`
`2`	`2`
	`3`	`+import org.xmpp.packet.JID;`
	`4`	`+`
`3`	`5`	`public class Owner extends Field`
`4`	`6`	`{`
`5`	`7`	`public static final String FIELD_NAME = "pubsub#owner";`
`@@ -12,6 +14,12 @@ public Owner()`
`12`	`14`
`13`	`15`	`public boolean isValid()`
`14`	`16`	`{`
`15`		`- return true;`
	`17`	`+ try {`
	`18`	`+ JID owner = new JID(getValue());`
	`19`	`+ owner.toBareJID();`
	`20`	`+ return true;`
	`21`	`+ } catch (IllegalArgumentException e) {`
	`22`	`+ return false;`
	`23`	`+ }`
`16`	`24`	`}`
`17`	`25`	`}`