Fix HTML tags rendered as plain text in received notifications

src/org/kde/kdeconnect/Plugins/ReceiveNotificationsPlugin/ReceiveNotificationsPlugin.java

@@ -14,6 +14,7 @@
 import android.graphics.Bitmap;
 import android.graphics.BitmapFactory;
 import android.os.Build;
+import android.text.Html;
 import android.util.Log;
 
 import androidx.annotation.NonNull;
@@ -98,24 +99,40 @@ public boolean onPacketReceived(final NetworkPacket np) {
 
         NotificationManager notificationManager = ContextCompat.getSystemService(context, NotificationManager.class);
 
+        String rawTicker = np.getString("ticker");
+        CharSequence styledTicker = fromHtml(rawTicker);
+
         Notification noti = new NotificationCompat.Builder(context, NotificationHelper.Channels.RECEIVENOTIFICATION)
                 .setContentTitle(np.getString("appName"))
-                .setContentText(np.getString("ticker"))
+                .setContentText(styledTicker)
                 .setContentIntent(resultPendingIntent)
-                .setTicker(np.getString("ticker"))
+                .setTicker(styledTicker)
                 .setSmallIcon(R.drawable.ic_notification)
                 .setLargeIcon(largeIcon)
                 .setAutoCancel(true)
                 .setLocalOnly(true)  // to avoid bouncing the notification back to other kdeconnect nodes
                 .setDefaults(Notification.DEFAULT_ALL)
-                .setStyle(new NotificationCompat.BigTextStyle().bigText(np.getString("ticker")))
+                .setStyle(new NotificationCompat.BigTextStyle().bigText(styledTicker))
                 .build();
 
         NotificationHelper.notifyCompat(notificationManager, "kdeconnectId:" + np.getString("id", "0"), np.getInt("id", 0), noti);
 
         return true;
     }
 
+    @NonNull
+    static CharSequence fromHtml(@NonNull String text) {
+        if (!text.contains("<") && !text.contains("&")) {
+            return text;
+        }
+        if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.N) {
+            return Html.fromHtml(text, Html.FROM_HTML_MODE_COMPACT);
+        } else {
+            //noinspection deprecation
+            return Html.fromHtml(text);
+        }
+    }
+
     @Override
     public @NonNull String[] getSupportedPacketTypes() {
         return new String[]{PACKET_TYPE_NOTIFICATION};

tests/org/kde/kdeconnect/Plugins/ReceiveNotificationsPlugin/ReceiveNotificationsPluginTest.java

@@ -0,0 +1,112 @@
+/*
+ * SPDX-FileCopyrightText: 2025 Contributors
+ *
+ * SPDX-License-Identifier: GPL-2.0-only OR GPL-3.0-only OR LicenseRef-KDE-Accepted-GPL
+ */
+
+package org.kde.kdeconnect.Plugins.ReceiveNotificationsPlugin;
+
+import org.junit.Test;
+import org.junit.runner.RunWith;
+import org.robolectric.RobolectricTestRunner;
+import org.robolectric.annotation.Config;
+
+import static org.junit.Assert.assertEquals;
+import static org.junit.Assert.assertSame;
+
+@RunWith(RobolectricTestRunner.class)
+@Config(sdk = 28)
+public class ReceiveNotificationsPluginTest {
+
+    @Test
+    public void fromHtml_plainTextNoAngleBrackets_returnsSameString() {
+        String input = "Hello world";
+        CharSequence result = ReceiveNotificationsPlugin.fromHtml(input);
+        assertEquals("Hello world", result.toString());
+    }
+
+    @Test
+    public void fromHtml_plainTextWithNumbers_returnsSameString() {
+        String input = "Battery level 85%";
+        CharSequence result = ReceiveNotificationsPlugin.fromHtml(input);
+        assertEquals("Battery level 85%", result.toString());
+    }
+
+    @Test
+    public void fromHtml_boldTag_stripsTag() {
+        String input = "<b>Important</b> message";
+        CharSequence result = ReceiveNotificationsPlugin.fromHtml(input);
+        assertEquals("Important message", result.toString());
+    }
+
+    @Test
+    public void fromHtml_italicTag_stripsTag() {
+        String input = "<i>emphasis</i> here";
+        CharSequence result = ReceiveNotificationsPlugin.fromHtml(input);
+        assertEquals("emphasis here", result.toString());
+    }
+
+    @Test
+    public void fromHtml_brTag_convertsToNewline() {
+        String input = "Line1<br>Line2";
+        CharSequence result = ReceiveNotificationsPlugin.fromHtml(input);
+        assertEquals("Line1\nLine2", result.toString());
+    }
+
+    @Test
+    public void fromHtml_brSlashTag_convertsToNewline() {
+        String input = "Line1<br/>Line2";
+        CharSequence result = ReceiveNotificationsPlugin.fromHtml(input);
+        assertEquals("Line1\nLine2", result.toString());
+    }
+
+    @Test
+    public void fromHtml_multipleTags_stripsAll() {
+        String input = "<b>Bold</b> and <i>italic</i><br>new line";
+        CharSequence result = ReceiveNotificationsPlugin.fromHtml(input);
+        assertEquals("Bold and italic\nnew line", result.toString());
+    }
+
+    @Test
+    public void fromHtml_htmlEntities_decodesThem() {
+        String input = "5 &lt; 10 &amp; 10 &gt; 5";
+        CharSequence result = ReceiveNotificationsPlugin.fromHtml(input);
+        assertEquals("5 < 10 & 10 > 5", result.toString());
+    }
+
+    @Test
+    public void fromHtml_lessThanSymbol_treatedAsHtml() {
+        String input = "5 < 10";
+        CharSequence result = ReceiveNotificationsPlugin.fromHtml(input);
+        assertEquals("5 < 10", result.toString());
+    }
+
+    @Test
+    public void fromHtml_emptyString_returnsEmpty() {
+        String input = "";
+        CharSequence result = ReceiveNotificationsPlugin.fromHtml(input);
+        assertEquals("", result.toString());
+    }
+
+    @Test
+    public void fromHtml_pTags_stripsTags() {
+        String input = "<p>Paragraph</p>";
+        CharSequence result = ReceiveNotificationsPlugin.fromHtml(input);
+        assertEquals("Paragraph", result.toString().trim());
+    }
+
+    @Test
+    public void fromHtml_malformedHtml_handlesGracefully() {
+        String input = "<b>unclosed bold";
+        CharSequence result = ReceiveNotificationsPlugin.fromHtml(input);
+        assertEquals("unclosed bold", result.toString());
+    }
+
+    @Test
+    @Config(sdk = 23)
+    public void fromHtml_preApi24_handlesHtml() {
+        String input = "<b>Bold text</b>";
+        CharSequence result = ReceiveNotificationsPlugin.fromHtml(input);
+        assertEquals("Bold text", result.toString());
+    }
+}