Sfoglia il codice sorgente

Remove toLowerCase. JID resources are case sensitive. updates #234

We should perhaps still make the node part lower case, but I'd like to double
check first.
JC Brand 10 anni fa
parent
commit
03c92b4487
1 ha cambiato i file con 2 aggiunte e 2 eliminazioni
  1. 2 2
      converse.js

+ 2 - 2
converse.js

@@ -2906,7 +2906,7 @@
                 var contact_jid, $forwarded, $received, $sent,
                 var contact_jid, $forwarded, $received, $sent,
                     msgid = $message.attr('id'),
                     msgid = $message.attr('id'),
                     chatbox, resource, roster_item,
                     chatbox, resource, roster_item,
-                    message_from = $message.attr('from').toLowerCase();
+                    message_from = $message.attr('from');
                 if (message_from === converse.connection.jid) {
                 if (message_from === converse.connection.jid) {
                     // FIXME: Forwarded messages should be sent to specific resources,
                     // FIXME: Forwarded messages should be sent to specific resources,
                     // not broadcasted
                     // not broadcasted
@@ -2933,7 +2933,7 @@
                     contact_jid = to;
                     contact_jid = to;
                     resource = Strophe.getResourceFromJid($message.attr('to'));
                     resource = Strophe.getResourceFromJid($message.attr('to'));
                 } else {
                 } else {
-                    contact_jid = from;
+                    contact_jid = from; // XXX: Should we add toLowerCase here? See ticket #234
                     resource = Strophe.getResourceFromJid(message_from);
                     resource = Strophe.getResourceFromJid(message_from);
                 }
                 }