only perform a database look up for bare JIDs. Return direct to full JIDs or just domains.

Author: Lloyd Watkin

src/main/java/org/buddycloud/channelserver/utils/users/OnlineResourceManager.java

@@ -61,15 +61,15 @@ public void subscribeToNodeListeners(BlockingQueue<Packet> outQueue) {
 	}
 
 	public ArrayList<JID> getResources(JID jid) throws NodeStoreException {
-		if (true == useDatabaseStorage) {
-			return channelManager.onlineJids(jid);
-		}
 		if ((jid.getResource() != null)
 				|| ((jid.getResource() == null) && (jid.getNode() == null))) {
 			ArrayList<JID> user = new ArrayList<JID>();
 			user.add(jid);
 			return user;
 		}
+		if (true == useDatabaseStorage) {
+			ArrayList<JID> jids = channelManager.onlineJids(jid);
+		}
 		if (!users.containsKey(jid.toBareJID())) {
 			return new ArrayList<JID>();
 		}

src/test/java/org/buddycloud/channelserver/utils/users/OnlineResourceManagerTest.java

@@ -211,9 +211,9 @@ public void ifUsingDatabaseWeSeeRequestForOnlineUsersMethod()
 
 		channelManager = Mockito.mock(ChannelManager.class);
 		onlineUser = new OnlineResourceManager(configuration, channelManager);
-		onlineUser.getResources(localUserLaptop);
+		onlineUser.getResources(localUserNoResource);
 		Mockito.verify(channelManager, Mockito.times(1)).onlineJids(
-				Mockito.eq(localUserLaptop));
+				Mockito.eq(localUserNoResource));
 	}
 
 	@Test