Fix for mucConversationLog sender bug

I discovered a bug this morning in the mucConversationLog. Wildfire stores the room’‘s JID instead of the sender’'s raw JID with each post. Digging into the code it was just a few instances where message.getTo() was used instead of message.getFrom(). The following patch takes care of it and applies to src/java/org/jivesoftware/wildfire/muc/spi/MUCRoomImpl.java

cheers,

-john

@@ -845,8 +845,8 @@

if (isLogEnabled()) {

MUCRole senderRole = null;

JID senderAddress = null;

  •        if (message.getTo() != null && message.getTo().getResource() != null) {
    
  •            senderRole = occupants.get(message.getTo().getResource());
    
  •        if (message.getFrom() != null && message.getFrom().getResource() != null) {
    
  •            senderRole = occupants.get(message.getFrom().getResource());
    

}

if (senderRole == null) {

// The room itself is sending the message

/pre

Hey John,

Thanks for catching this bug and fixing it. Good job! The bug fix will be available in the next nightly build.

Regards,

– Gato