From 3c2ea38f1e763ff18af9335fc6ec1d71dd41a96e Mon Sep 17 00:00:00 2001
From: CappielloAntonio <antoniocappiello.play@gmail.com>
Date: Fri, 30 Aug 2024 15:27:16 +0200
Subject: [PATCH] fix: further eliminate the use of HTML decode in titles and
 subtitles

---
 .../tempo/ui/adapter/ShareHorizontalAdapter.java |  2 +-
 .../tempo/ui/fragment/AlbumPageFragment.java     |  2 +-
 .../cappielloantonio/tempo/util/MusicUtil.java   | 16 ++++++++--------
 3 files changed, 10 insertions(+), 10 deletions(-)

diff --git a/app/src/main/java/com/cappielloantonio/tempo/ui/adapter/ShareHorizontalAdapter.java b/app/src/main/java/com/cappielloantonio/tempo/ui/adapter/ShareHorizontalAdapter.java
index e72774d7..213914fa 100644
--- a/app/src/main/java/com/cappielloantonio/tempo/ui/adapter/ShareHorizontalAdapter.java
+++ b/app/src/main/java/com/cappielloantonio/tempo/ui/adapter/ShareHorizontalAdapter.java
@@ -40,7 +40,7 @@ public class ShareHorizontalAdapter extends RecyclerView.Adapter<ShareHorizontal
     public void onBindViewHolder(ViewHolder holder, int position) {
         Share share = shares.get(position);
 
-        holder.item.shareTitleTextView.setText(MusicUtil.getReadableString(share.getDescription()));
+        holder.item.shareTitleTextView.setText(share.getDescription());
         holder.item.shareSubtitleTextView.setText(holder.itemView.getContext().getString(R.string.share_subtitle_item, UIUtil.getReadableDate(share.getExpires())));
 
         if (share.getEntries() != null && !share.getEntries().isEmpty()) CustomGlideRequest.Builder
diff --git a/app/src/main/java/com/cappielloantonio/tempo/ui/fragment/AlbumPageFragment.java b/app/src/main/java/com/cappielloantonio/tempo/ui/fragment/AlbumPageFragment.java
index bda30b79..8162b16b 100644
--- a/app/src/main/java/com/cappielloantonio/tempo/ui/fragment/AlbumPageFragment.java
+++ b/app/src/main/java/com/cappielloantonio/tempo/ui/fragment/AlbumPageFragment.java
@@ -182,7 +182,7 @@ public class AlbumPageFragment extends Fragment implements ClickCallback {
             if (albumInfo != null) {
                 if (bind != null) bind.albumNotesTextview.setVisibility(View.VISIBLE);
                 if (bind != null)
-                    bind.albumNotesTextview.setText(MusicUtil.getReadableString(albumInfo.getNotes()));
+                    bind.albumNotesTextview.setText(MusicUtil.forceReadableString(albumInfo.getNotes()));
 
                 if (bind != null && albumInfo.getLastFmUrl() != null && !albumInfo.getLastFmUrl().isEmpty()) {
                     bind.albumNotesTextview.setOnClickListener(v -> {
diff --git a/app/src/main/java/com/cappielloantonio/tempo/util/MusicUtil.java b/app/src/main/java/com/cappielloantonio/tempo/util/MusicUtil.java
index 1a212426..dfbf956a 100644
--- a/app/src/main/java/com/cappielloantonio/tempo/util/MusicUtil.java
+++ b/app/src/main/java/com/cappielloantonio/tempo/util/MusicUtil.java
@@ -179,14 +179,6 @@ public class MusicUtil {
         }
     }
 
-    public static String getReadableString(String string) {
-        if (string != null) {
-            return Html.fromHtml(string, Html.FROM_HTML_MODE_COMPACT).toString();
-        }
-
-        return "";
-    }
-
     public static String getReadableTrackNumber(Context context, Integer trackNumber) {
         if (trackNumber != null) {
             return String.valueOf(trackNumber);
@@ -195,6 +187,14 @@ public class MusicUtil {
         return context.getString(R.string.label_placeholder);
     }
 
+    public static String getReadableString(String string) {
+        if (string != null) {
+            return Html.fromHtml(string, Html.FROM_HTML_MODE_COMPACT).toString();
+        }
+
+        return "";
+    }
+
     public static String forceReadableString(String string) {
         if (string != null) {
             return getReadableString(string)