From 1906ccb420a099b66c998a54e00cbf59f8c882f8 Mon Sep 17 00:00:00 2001
From: Daniel Gultsch <daniel@gultsch.de>
Date: Thu, 16 Jun 2016 20:38:35 +0200
Subject: don't fail on missing jid in bookmarks

---
 src/main/java/eu/siacs/conversations/entities/Bookmark.java | 8 +++++---
 1 file changed, 5 insertions(+), 3 deletions(-)

(limited to 'src/main')

diff --git a/src/main/java/eu/siacs/conversations/entities/Bookmark.java b/src/main/java/eu/siacs/conversations/entities/Bookmark.java
index 3f3707d01..e6aeadf4a 100644
--- a/src/main/java/eu/siacs/conversations/entities/Bookmark.java
+++ b/src/main/java/eu/siacs/conversations/entities/Bookmark.java
@@ -55,7 +55,9 @@ public class Bookmark extends Element implements ListItem {
 				&& !getBookmarkName().trim().isEmpty()) {
 			return getBookmarkName().trim();
 		} else {
-			return this.getJid().getLocalpart();
+			Jid jid = this.getJid();
+			String name = jid != null ? jid.getLocalpart() : getAttribute("jid");
+			return name != null ? name : "";
 		}
 	}
 
@@ -65,7 +67,7 @@ public class Bookmark extends Element implements ListItem {
 		if (jid != null) {
 			return jid.toString();
 		} else {
-			return null;
+			return getAttribute("jid"); //fallback if jid wasn't parsable
 		}
 	}
 
@@ -76,7 +78,7 @@ public class Bookmark extends Element implements ListItem {
 
 	@Override
 	public List<Tag> getTags(Context context) {
-		ArrayList<Tag> tags = new ArrayList<Tag>();
+		ArrayList<Tag> tags = new ArrayList<>();
 		for (Element element : getChildren()) {
 			if (element.getName().equals("group") && element.getContent() != null) {
 				String group = element.getContent();
-- 
cgit v1.2.3