From e1d51c881a96cce5e649899e4013a37336823c2d Mon Sep 17 00:00:00 2001
From: Akinwale Ariwodola <akinwale@gmail.com>
Date: Tue, 12 Jan 2021 09:07:33 +0100
Subject: [PATCH] Fix crash bug for anonymous repost claim results. Better
 handle channel repost navigation.

---
 .../main/java/io/lbry/browser/MainActivity.java   |  5 +++++
 .../io/lbry/browser/adapter/ClaimListAdapter.java |  2 +-
 .../browser/ui/findcontent/FileViewFragment.java  | 15 ++++++++++-----
 .../browser/ui/findcontent/ShuffleFragment.java   |  3 +--
 4 files changed, 17 insertions(+), 8 deletions(-)

diff --git a/app/src/main/java/io/lbry/browser/MainActivity.java b/app/src/main/java/io/lbry/browser/MainActivity.java
index c0b84f3e..5816b9bf 100644
--- a/app/src/main/java/io/lbry/browser/MainActivity.java
+++ b/app/src/main/java/io/lbry/browser/MainActivity.java
@@ -3402,6 +3402,11 @@ public class MainActivity extends AppCompatActivity implements SdkStatusListener
         return String.format("%s-%d", fragmentClass.getName(), navItemId);
     }
 
+    public void popFragment(Class fragmentClass, int navItemId, Map<String, Object> params) {
+        String key = buildNavFragmentKey(fragmentClass, navItemId, params);
+        openNavFragments.remove(key);
+    }
+
     public void openFragment(Class fragmentClass, boolean allowNavigateBack, int navItemId, Map<String, Object> params) {
         try {
             String key = buildNavFragmentKey(fragmentClass, navItemId, params);
diff --git a/app/src/main/java/io/lbry/browser/adapter/ClaimListAdapter.java b/app/src/main/java/io/lbry/browser/adapter/ClaimListAdapter.java
index 616432c6..9ff60c4a 100644
--- a/app/src/main/java/io/lbry/browser/adapter/ClaimListAdapter.java
+++ b/app/src/main/java/io/lbry/browser/adapter/ClaimListAdapter.java
@@ -404,7 +404,7 @@ public class ClaimListAdapter extends RecyclerView.Adapter<ClaimListAdapter.View
         vh.publishTimeView.setVisibility(!isPending ? View.VISIBLE : View.GONE);
         vh.pendingTextView.setVisibility(isPending && !item.isLoadingPlaceholder() ? View.VISIBLE : View.GONE);
         vh.repostInfoView.setVisibility(isRepost && type != VIEW_TYPE_FEATURED ? View.VISIBLE : View.GONE);
-        vh.repostChannelView.setText(isRepost ? original.getSigningChannel().getName() : null);
+        vh.repostChannelView.setText(isRepost && original.getSigningChannel() != null ? original.getSigningChannel().getName() : null);
         vh.repostChannelView.setOnClickListener(new View.OnClickListener() {
             @Override
             public void onClick(View view) {
diff --git a/app/src/main/java/io/lbry/browser/ui/findcontent/FileViewFragment.java b/app/src/main/java/io/lbry/browser/ui/findcontent/FileViewFragment.java
index 028bce17..76b75038 100644
--- a/app/src/main/java/io/lbry/browser/ui/findcontent/FileViewFragment.java
+++ b/app/src/main/java/io/lbry/browser/ui/findcontent/FileViewFragment.java
@@ -12,6 +12,7 @@ import android.graphics.Color;
 import android.net.Uri;
 import android.os.AsyncTask;
 import android.os.Bundle;
+import android.os.Handler;
 import android.text.Editable;
 import android.text.TextWatcher;
 import android.text.format.DateUtils;
@@ -310,8 +311,7 @@ public class FileViewFragment extends BaseFragment implements
                         String mediaSourceUrl = getStreamingUrl();
                         long duration = MainActivity.appPlayer.getDuration();
                         long position = MainActivity.appPlayer.getCurrentPosition();
-                        // TODO: Determine a hash for the userId
-                        String userIdHash = Helper.SHA256(Lbryio.currentUser != null ? String.valueOf(Lbryio.currentUser.getId()) : "0");
+                        String userIdHash = Lbryio.currentUser != null ? String.valueOf(Lbryio.currentUser.getId()) : "0";
                         if (mediaSourceUrl.startsWith(CDN_PREFIX)) {
                             BufferEventTask bufferEvent = new BufferEventTask(claim.getPermanentUrl(), duration, position, 1, userIdHash);
                             bufferEvent.executeOnExecutor(AsyncTask.SERIAL_EXECUTOR);
@@ -429,8 +429,11 @@ public class FileViewFragment extends BaseFragment implements
                             renderNothingAtLocation();
                         } else if (claim.getName().startsWith("@")) {
                             // this is a reposted channel, so launch the channel url
-                            if (context instanceof  MainActivity) {
-                                ((MainActivity) context).openChannelUrl(!Helper.isNullOrEmpty(claim.getShortUrl()) ? claim.getShortUrl() : claim.getPermanentUrl());
+                            if (context instanceof MainActivity) {
+                                MainActivity activity = (MainActivity) context;
+                                //activity.onBackPressed(); // remove the reposted url page from the back stack
+                                activity.getSupportFragmentManager().popBackStack();
+                                activity.openChannelUrl(!Helper.isNullOrEmpty(claim.getShortUrl()) ? claim.getShortUrl() : claim.getPermanentUrl());
                             }
                             return;
                         }
@@ -899,7 +902,9 @@ public class FileViewFragment extends BaseFragment implements
                             // this is a reposted channel, so finish this activity and launch the channel url
                             Context context = getContext();
                             if (context instanceof  MainActivity) {
-                                ((MainActivity) context).openChannelUrl(!Helper.isNullOrEmpty(claim.getShortUrl()) ? claim.getShortUrl() : claim.getPermanentUrl());
+                                MainActivity activity = (MainActivity) context;
+                                activity.getSupportFragmentManager().popBackStack();
+                                activity.openChannelUrl(!Helper.isNullOrEmpty(claim.getShortUrl()) ? claim.getShortUrl() : claim.getPermanentUrl());
                             }
                             return;
                         }
diff --git a/app/src/main/java/io/lbry/browser/ui/findcontent/ShuffleFragment.java b/app/src/main/java/io/lbry/browser/ui/findcontent/ShuffleFragment.java
index efce66c8..f94044b6 100644
--- a/app/src/main/java/io/lbry/browser/ui/findcontent/ShuffleFragment.java
+++ b/app/src/main/java/io/lbry/browser/ui/findcontent/ShuffleFragment.java
@@ -131,8 +131,7 @@ public class ShuffleFragment extends BaseFragment {
                         String mediaSourceUrl = getStreamingUrl();
                         long duration = MainActivity.appPlayer.getDuration();
                         long position = MainActivity.appPlayer.getCurrentPosition();
-                        // TODO: Determine a hash for the userId
-                        String userIdHash = Helper.SHA256(Lbryio.currentUser != null ? String.valueOf(Lbryio.currentUser.getId()) : "0");
+                        String userIdHash = Lbryio.currentUser != null ? String.valueOf(Lbryio.currentUser.getId()) : "0";
                         if (mediaSourceUrl.startsWith(FileViewFragment.CDN_PREFIX)) {
                             BufferEventTask bufferEvent = new BufferEventTask(current.getPermanentUrl(), duration, position, 1, userIdHash);
                             bufferEvent.executeOnExecutor(AsyncTask.SERIAL_EXECUTOR);