From 52bb6f9b764040fdd7190b33ce6cc4e419c8ba26 Mon Sep 17 00:00:00 2001 From: Sergey Ponomarev Date: Wed, 7 Jan 2026 22:05:41 +0200 Subject: [PATCH] ContactItem: make alias and nickname an empty string if null --- .../jivesoftware/spark/ui/ContactItem.java | 20 ++++++++++--------- 1 file changed, 11 insertions(+), 9 deletions(-) diff --git a/core/src/main/java/org/jivesoftware/spark/ui/ContactItem.java b/core/src/main/java/org/jivesoftware/spark/ui/ContactItem.java index 7fef8cc29..f11ddf0c2 100644 --- a/core/src/main/java/org/jivesoftware/spark/ui/ContactItem.java +++ b/core/src/main/java/org/jivesoftware/spark/ui/ContactItem.java @@ -53,6 +53,8 @@ import org.jivesoftware.sparkimpl.settings.local.SettingsManager; import org.jxmpp.jid.BareJid; import org.jxmpp.util.XmppStringUtils; +import static org.apache.commons.lang3.StringUtils.trimToEmpty; + /** * Represent a single contact within the ContactList. */ @@ -109,8 +111,8 @@ public class ContactItem extends JPanel { contactsDir = new File(SparkManager.getUserDirectory(), "contacts"); - this.alias = alias; - this.nickname = nickname; + this.alias = trimToEmpty(alias); + this.nickname = trimToEmpty(nickname); this.jid = jid; if (initUi) { @@ -163,11 +165,11 @@ public class ContactItem extends JPanel { */ public String getDisplayName() { String displayName = null; - if (alias != null && !alias.trim().isEmpty()) { - return alias.trim(); + if (!alias.isEmpty()) { + return alias; } - if (nickname != null && !nickname.trim().isEmpty()) { - return nickname.trim(); + if (!nickname.isEmpty()) { + return nickname; } BareJid bareJid = getJid(); if (bareJid != null) { @@ -195,8 +197,8 @@ public class ContactItem extends JPanel { * @param nickname the contact nickname. */ public void setNickname(String nickname) { - this.nickname = nickname; - if (alias == null) { + this.nickname = trimToEmpty(nickname); + if (alias.isEmpty()) { setDisplayName(); } } @@ -217,7 +219,7 @@ public class ContactItem extends JPanel { * @param alias the contact alias. */ public void setAlias(String alias) { - this.alias = alias; + this.alias = trimToEmpty(alias); setDisplayName(); }