Author: ngn
Date: Sun Aug 16 20:41:17 2009
New Revision: 804770

URL: http://svn.apache.org/viewvc?rev=804770&view=rev
Log:
Add null checks in constructor and toString

Modified:
    
mina/sandbox/vysper/trunk/server/extensions/xep0045-muc/src/main/java/org/apache/vysper/xmpp/modules/extension/xep0045_muc/model/Occupant.java

Modified: 
mina/sandbox/vysper/trunk/server/extensions/xep0045-muc/src/main/java/org/apache/vysper/xmpp/modules/extension/xep0045_muc/model/Occupant.java
URL: 
http://svn.apache.org/viewvc/mina/sandbox/vysper/trunk/server/extensions/xep0045-muc/src/main/java/org/apache/vysper/xmpp/modules/extension/xep0045_muc/model/Occupant.java?rev=804770&r1=804769&r2=804770&view=diff
==============================================================================
--- 
mina/sandbox/vysper/trunk/server/extensions/xep0045-muc/src/main/java/org/apache/vysper/xmpp/modules/extension/xep0045_muc/model/Occupant.java
 (original)
+++ 
mina/sandbox/vysper/trunk/server/extensions/xep0045-muc/src/main/java/org/apache/vysper/xmpp/modules/extension/xep0045_muc/model/Occupant.java
 Sun Aug 16 20:41:17 2009
@@ -35,6 +35,11 @@
     private String name;
     
     public Occupant(Entity jid, String name, Affiliation affiliation, Role 
role) {
+        if(jid == null) throw new IllegalArgumentException("JID can not be 
null");
+        if(name == null) throw new IllegalArgumentException("Name can not be 
null");
+        if(affiliation == null) throw new 
IllegalArgumentException("Affiliation can not be null");
+        if(role == null) throw new IllegalArgumentException("Role can not be 
null");
+        
         this.jid = jid;
         this.name = name;
         this.affiliation = affiliation;
@@ -73,5 +78,9 @@
         return role == Role.Moderator || role == Role.Participant;
     }
     
+    @Override
+    public String toString() {
+        return jid.getFullQualifiedName();
+    }
     
 }


Reply via email to