From 7d79852c8403a8882061480cfb524724b84857ec Mon Sep 17 00:00:00 2001 From: Daniel Gultsch Date: Mon, 3 Feb 2014 16:04:27 +0100 Subject: fixed a bug in xml parsing --- src/de/gultsch/chat/services/XmppConnectionService.java | 6 ++---- src/de/gultsch/chat/ui/NewConversationActivity.java | 4 +++- src/de/gultsch/chat/xml/XmlReader.java | 6 ++++-- src/de/gultsch/chat/xmpp/XmppConnection.java | 2 +- 4 files changed, 10 insertions(+), 8 deletions(-) (limited to 'src/de') diff --git a/src/de/gultsch/chat/services/XmppConnectionService.java b/src/de/gultsch/chat/services/XmppConnectionService.java index d7ee76ee..fa2e6d7e 100644 --- a/src/de/gultsch/chat/services/XmppConnectionService.java +++ b/src/de/gultsch/chat/services/XmppConnectionService.java @@ -47,7 +47,6 @@ public class XmppConnectionService extends Service { @Override public void onMessagePacketReceived(Account account, MessagePacket packet) { if (packet.getType()==MessagePacket.TYPE_CHAT) { - Log.d(LOGTAG,account.getJid()+": message of type chat"); String fullJid = packet.getFrom(); String jid = fullJid.split("/")[0]; String name = jid.split("@")[0]; @@ -132,7 +131,6 @@ public class XmppConnectionService extends Service { @Override public void onIqPacketReceived(Account account, IqPacket packet) { Element roster = packet.findChild("query"); - Log.d(LOGTAG,roster.toString()); List contacts = new ArrayList(); for(Element item : roster.getChildren()) { String name = item.getAttribute("name"); @@ -182,10 +180,10 @@ public class XmppConnectionService extends Service { } public Conversation findOrCreateConversation(Account account, Contact contact) { - Log.d(LOGTAG,"was asked to find conversation for "+contact.getJid()); + //Log.d(LOGTAG,"was asked to find conversation for "+contact.getJid()); for(Conversation conv : this.getConversations()) { if ((conv.getAccount().equals(account))&&(conv.getContactJid().equals(contact.getJid()))) { - Log.d(LOGTAG,"found one in memory"); + //Log.d(LOGTAG,"found one in memory"); return conv; } } diff --git a/src/de/gultsch/chat/ui/NewConversationActivity.java b/src/de/gultsch/chat/ui/NewConversationActivity.java index b41ca761..bcd7fc9d 100644 --- a/src/de/gultsch/chat/ui/NewConversationActivity.java +++ b/src/de/gultsch/chat/ui/NewConversationActivity.java @@ -29,6 +29,7 @@ import android.widget.EditText; import android.widget.ListView; import android.widget.TextView; import android.widget.ImageView; +import android.annotation.SuppressLint; import android.app.Activity; import android.app.AlertDialog; import android.content.Context; @@ -67,9 +68,10 @@ public class NewConversationActivity extends XmppActivity { Collections.sort(aggregatedContacts, new Comparator() { + @SuppressLint("DefaultLocale") @Override public int compare(Contact lhs, Contact rhs) { - return lhs.getDisplayName().compareTo(rhs.getDisplayName()); + return lhs.getDisplayName().toLowerCase().compareTo(rhs.getDisplayName().toLowerCase()); } }); diff --git a/src/de/gultsch/chat/xml/XmlReader.java b/src/de/gultsch/chat/xml/XmlReader.java index 8963f8d7..e9429665 100644 --- a/src/de/gultsch/chat/xml/XmlReader.java +++ b/src/de/gultsch/chat/xml/XmlReader.java @@ -86,8 +86,10 @@ public class XmlReader { } //Log.d(LOGTAG,"reading till the end of "+element.getName()); while(!nextTag.isEnd(element.getName())) { - Element child = this.readElement(nextTag); - element.addChild(child); + if (!nextTag.isNo()) { + Element child = this.readElement(nextTag); + element.addChild(child); + } nextTag = this.readTag(); } //Log.d(LOGTAG,"return with element"+element); diff --git a/src/de/gultsch/chat/xmpp/XmppConnection.java b/src/de/gultsch/chat/xmpp/XmppConnection.java index 2303d239..1afe5b0c 100644 --- a/src/de/gultsch/chat/xmpp/XmppConnection.java +++ b/src/de/gultsch/chat/xmpp/XmppConnection.java @@ -299,7 +299,7 @@ public class XmppConnection implements Runnable { iqPacketCallbacks.put(id, callback); } tagWriter.flush(); - Log.d(LOGTAG,"sending: "+packet.toString()); + Log.d(LOGTAG,account.getJid()+": sending: "+packet.toString()); } public void sendMessagePacket(MessagePacket packet) throws IOException { -- cgit v1.2.3