Cleaned up PEPService and fixed a bug in PEPService.isServiceAdmin()

git-svn-id: http://svn.igniterealtime.org/svn/repos/openfire/branches/pep@8823 b35dd754-fafc-0310-a699-88a17e54d16e

src/java/org/jivesoftware/openfire/pep/PEPService.java

@@ -25,7 +25,6 @@ import org.jivesoftware.openfire.pubsub.models.AccessModel;
 import org.jivesoftware.openfire.pubsub.models.PublisherModel;
 import org.jivesoftware.openfire.PacketRouter;
 import org.jivesoftware.openfire.XMPPServer;
-import org.jivesoftware.util.JiveGlobals;
 import org.jivesoftware.util.StringUtils;
 import org.xmpp.packet.JID;
 import org.xmpp.packet.Message;
@@ -61,9 +60,38 @@ public class PEPService implements PubSubService {
      */
     private Map<String, Node> nodes = new ConcurrentHashMap<String, Node>();
 
+    /**
+     * The packet router for the server.
+     */
+    private PacketRouter router = null;
+
+    /**
+     * Default configuration to use for newly created leaf nodes.
+     */
+    private DefaultNodeConfiguration leafDefaultConfiguration;
+
+    /**
+     * Default configuration to use for newly created collection nodes.
+     */
+    private DefaultNodeConfiguration collectionDefaultConfiguration;
+
+    /**
+     * Returns the permission policy for creating nodes. A true value means that
+     * not anyone can create a node, only the service admin.
+     */
+    private boolean nodeCreationRestricted = true;
+
+    /**
+     * Flag that indicates if a user may have more than one subscription with
+     * the node. When multiple subscriptions is enabled each subscription
+     * request, event notification, and unsubscription request should include a
+     * subid attribute.
+     */
+    private boolean multipleSubscriptionsEnabled = false;
+
     /**
      * Keep a registry of the presence's show value of users that subscribed to
-     * a node of the pubsub service and for which the node only delivers
+     * a node of the pep service and for which the node only delivers
      * notifications for online users or node subscriptions deliver events based
      * on the user presence show value. Offline users will not have an entry in
      * the map. Note: Key-> bare JID and Value-> Map whose key is full JID of
@@ -104,36 +132,6 @@ public class PEPService implements PubSubService {
      */
     private Timer timer = new Timer("PEP service maintenance");
 
-    /**
-     * Returns the permission policy for creating nodes. A true value means that
-     * not anyone can create a node, only the JIDs listed in
-     * <code>allowedToCreate</code> are allowed to create nodes.
-     */
-    private boolean nodeCreationRestricted = false;
-
-    /**
-     * Flag that indicates if a user may have more than one subscription with
-     * the node. When multiple subscriptions is enabled each subscription
-     * request, event notification and unsubscription request should include a
-     * subid attribute.
-     */
-    private boolean multipleSubscriptionsEnabled = true;
-
-    /**
-     * The packet router for the server.
-     */
-    private PacketRouter router = null;
-
-    /**
-     * Default configuration to use for newly created leaf nodes.
-     */
-    private DefaultNodeConfiguration leafDefaultConfiguration;
-
-    /**
-     * Default configuration to use for newly created collection nodes.
-     */
-    private DefaultNodeConfiguration collectionDefaultConfiguration;
-
     /**
      * Constructs a PEPService.
      * 
@@ -144,18 +142,15 @@ public class PEPService implements PubSubService {
         this.bareJID = bareJID;
         router = server.getPacketRouter();
 
-        // Initialize the ad-hoc commands manager to use for this pubsub service
+        // Initialize the ad-hoc commands manager to use for this pep service
         manager = new AdHocCommandManager();
         manager.addCommand(new PendingSubscriptionsCommand(this));
 
         // Save or delete published items from the database every 2 minutes
-        // starting in
-        // 2 minutes (default values)
+        // starting in 2 minutes (default values)
         publishedItemTask = new PublishedItemTask(this);
         timer.schedule(publishedItemTask, items_task_timeout, items_task_timeout);
 
-        multipleSubscriptionsEnabled = JiveGlobals.getBooleanProperty("xmpp.pubsub.multiple-subscriptions", true);
-
         // Load default configuration for leaf nodes
         leafDefaultConfiguration = PubSubPersistenceManager.loadDefaultConfiguration(this, true);
         if (leafDefaultConfiguration == null) {
@@ -200,19 +195,17 @@ public class PEPService implements PubSubService {
         // Load nodes to memory
         PubSubPersistenceManager.loadNodes(this);
         // Ensure that we have a root collection node
-        String rootNodeID = JiveGlobals.getProperty("xmpp.pubsub.root.nodeID", bareJID);
         if (nodes.isEmpty()) {
             // Create root collection node
-            String creator = JiveGlobals.getProperty("xmpp.pubsub.root.creator");
-            JID creatorJID = creator != null ? new JID(creator) : server.getAdmins().iterator().next();
-            rootCollectionNode = new CollectionNode(this, null, rootNodeID, creatorJID);
+            JID creatorJID = new JID(bareJID);
+            rootCollectionNode = new CollectionNode(this, null, bareJID, creatorJID);
             // Add the creator as the node owner
             rootCollectionNode.addOwner(creatorJID);
             // Save new root node
             rootCollectionNode.saveToDB();
         }
         else {
-            rootCollectionNode = (CollectionNode) getNode(rootNodeID);
+            rootCollectionNode = (CollectionNode) getNode(bareJID);
         }
     }
 
@@ -262,7 +255,8 @@ public class PEPService implements PubSubService {
     }
 
     public String getServiceID() {
-        return bareJID;     // The bare JID of the user is the service ID
+        // The bare JID of the user is the service ID for PEP
+        return bareJID;
     }
 
     public Collection<String> getShowPresences(JID subscriber) {
@@ -284,7 +278,7 @@ public class PEPService implements PubSubService {
     public boolean isServiceAdmin(JID user) {
         // Here we consider a 'service admin' to be the user that this PEPService
         // is associated with.
-        if (bareJID == user.toBareJID()) {
+        if (bareJID.equals(user.toBareJID())) {
             return true;
         }
         else {

src/java/org/jivesoftware/openfire/pubsub/NodeAffiliate.java

@@ -95,6 +95,7 @@ public class NodeAffiliate {
                     if (getNode().getNodeID().indexOf("@") >= 0) {
                         items.addAttribute("node", publishedItem.getNode().getNodeID());                        
                     }
+
                     // Add item information to the event notification
                     Element item = items.addElement("item");
                     if (leafNode.isItemRequired()) {