Merge pull request #104 from surevine/issue-103

Fixes issue 103

Author: lloydwatkin

src/main/java/org/buddycloud/channelserver/channel/node/configuration/field/Factory.java

@@ -13,8 +13,12 @@ public Field create(String type, String value)
     	if ((null == type) || (null == value)) {
     		throw new ConfigurationFieldException();
     	}
-    	if (type.equals("pubsub#owner") && (false == this.allowOwner)) {
+    	if (type.equals(Owner.FIELD_NAME) && (false == this.allowOwner)) {
     		throw new ConfigurationFieldException();
+    	} else if (type.equals(Owner.FIELD_NAME)) {
+    		Owner field = new Owner();
+    	    field.setValue(value);
+    	    return field;
     	} else if (type.equals(ChannelTitle.FIELD_NAME)) {
     	    ChannelTitle field = new ChannelTitle();
     	    field.setValue(value);

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;
+		}
 	}
 }