From ff1fb78d4a1308d91b4e571978e131590523c447 Mon Sep 17 00:00:00 2001
From: antonio <antoniocappiello.play@gmail.com>
Date: Sun, 4 Jun 2023 16:24:56 +0200
Subject: [PATCH] fix: limited the number of items to load

---
 .../play/ui/adapter/PodcastEpisodeAdapter.java         | 10 ++++++++++
 .../play/ui/fragment/HomeTabPodcastFragment.java       |  5 ++++-
 .../play/ui/fragment/PodcastChannelPageFragment.java   |  8 +++++---
 app/src/main/res/layout/fragment_home_tab_podcast.xml  |  7 ++++---
 .../main/res/layout/fragment_podcast_channel_page.xml  |  3 ++-
 5 files changed, 25 insertions(+), 8 deletions(-)

diff --git a/app/src/main/java/com/cappielloantonio/play/ui/adapter/PodcastEpisodeAdapter.java b/app/src/main/java/com/cappielloantonio/play/ui/adapter/PodcastEpisodeAdapter.java
index a166c251..6630c565 100644
--- a/app/src/main/java/com/cappielloantonio/play/ui/adapter/PodcastEpisodeAdapter.java
+++ b/app/src/main/java/com/cappielloantonio/play/ui/adapter/PodcastEpisodeAdapter.java
@@ -62,6 +62,16 @@ public class PodcastEpisodeAdapter extends RecyclerView.Adapter<PodcastEpisodeAd
         notifyDataSetChanged();
     }
 
+    @Override
+    public int getItemViewType(int position) {
+        return position;
+    }
+
+    @Override
+    public long getItemId(int position) {
+        return position;
+    }
+
     public class ViewHolder extends RecyclerView.ViewHolder {
         ItemHomePodcastEpisodeBinding item;
 
diff --git a/app/src/main/java/com/cappielloantonio/play/ui/fragment/HomeTabPodcastFragment.java b/app/src/main/java/com/cappielloantonio/play/ui/fragment/HomeTabPodcastFragment.java
index a14dddc0..b58e06e6 100644
--- a/app/src/main/java/com/cappielloantonio/play/ui/fragment/HomeTabPodcastFragment.java
+++ b/app/src/main/java/com/cappielloantonio/play/ui/fragment/HomeTabPodcastFragment.java
@@ -37,6 +37,9 @@ import com.cappielloantonio.play.util.UIUtil;
 import com.cappielloantonio.play.viewmodel.PodcastViewModel;
 import com.google.common.util.concurrent.ListenableFuture;
 
+import java.util.Objects;
+import java.util.stream.Collectors;
+
 @UnstableApi
 public class HomeTabPodcastFragment extends Fragment implements ClickCallback {
     private static final String TAG = "HomeTabPodcastFragment";
@@ -125,7 +128,7 @@ public class HomeTabPodcastFragment extends Fragment implements ClickCallback {
                 if (bind != null)
                     bind.homeNewestPodcastsSector.setVisibility(!podcastEpisodes.isEmpty() ? View.VISIBLE : View.GONE);
 
-                podcastEpisodeAdapter.setItems(podcastEpisodes);
+                podcastEpisodeAdapter.setItems(podcastEpisodes.stream().filter(podcastEpisode -> Objects.equals(podcastEpisode.getStatus(), "completed")).collect(Collectors.toList()));
             }
         });
     }
diff --git a/app/src/main/java/com/cappielloantonio/play/ui/fragment/PodcastChannelPageFragment.java b/app/src/main/java/com/cappielloantonio/play/ui/fragment/PodcastChannelPageFragment.java
index 177042b9..8f9947a3 100644
--- a/app/src/main/java/com/cappielloantonio/play/ui/fragment/PodcastChannelPageFragment.java
+++ b/app/src/main/java/com/cappielloantonio/play/ui/fragment/PodcastChannelPageFragment.java
@@ -31,6 +31,9 @@ import com.cappielloantonio.play.util.UIUtil;
 import com.cappielloantonio.play.viewmodel.PodcastChannelPageViewModel;
 import com.google.common.util.concurrent.ListenableFuture;
 
+import java.util.Objects;
+import java.util.stream.Collectors;
+
 @UnstableApi
 public class PodcastChannelPageFragment extends Fragment implements ClickCallback {
     private FragmentPodcastChannelPageBinding bind;
@@ -121,9 +124,8 @@ public class PodcastChannelPageFragment extends Fragment implements ClickCallbac
                     bind.podcastChannelPageEpisodesPlaceholder.placeholder.setVisibility(View.GONE);
 
                 if (channels.get(0) != null && channels.get(0).getEpisodes() != null) {
-                    if (bind != null)
-                        bind.podcastChannelPageEpisodesSector.setVisibility(!channels.get(0).getEpisodes().isEmpty() ? View.VISIBLE : View.GONE);
-                    podcastEpisodeAdapter.setItems(channels.get(0).getEpisodes());
+                    if (bind != null) bind.podcastChannelPageEpisodesSector.setVisibility(!channels.get(0).getEpisodes().isEmpty() ? View.VISIBLE : View.GONE);
+                    podcastEpisodeAdapter.setItems(channels.get(0).getEpisodes().stream().filter(podcastEpisode -> Objects.equals(podcastEpisode.getStatus(), "completed")).collect(Collectors.toList()));
                 }
             }
         });
diff --git a/app/src/main/res/layout/fragment_home_tab_podcast.xml b/app/src/main/res/layout/fragment_home_tab_podcast.xml
index c2757016..45d757a9 100644
--- a/app/src/main/res/layout/fragment_home_tab_podcast.xml
+++ b/app/src/main/res/layout/fragment_home_tab_podcast.xml
@@ -1,6 +1,5 @@
 <?xml version="1.0" encoding="utf-8"?>
-<androidx.core.widget.NestedScrollView
-    xmlns:android="http://schemas.android.com/apk/res/android"
+<androidx.core.widget.NestedScrollView xmlns:android="http://schemas.android.com/apk/res/android"
     android:layout_width="match_parent"
     android:layout_height="wrap_content">
 
@@ -51,6 +50,7 @@
                 android:layout_marginTop="8dp"
                 android:layout_marginBottom="8dp"
                 android:clipToPadding="false"
+                android:nestedScrollingEnabled="false"
                 android:paddingTop="8dp"
                 android:paddingBottom="8dp" />
         </LinearLayout>
@@ -71,8 +71,8 @@
                 android:layout_width="match_parent"
                 android:layout_height="wrap_content"
                 android:layout_marginBottom="8dp"
-                android:paddingTop="16dp"
                 android:paddingStart="16dp"
+                android:paddingTop="16dp"
                 android:paddingEnd="16dp"
                 android:text="@string/home_title_newest_podcasts" />
 
@@ -83,6 +83,7 @@
                 android:layout_marginTop="8dp"
                 android:layout_marginBottom="8dp"
                 android:clipToPadding="false"
+                android:nestedScrollingEnabled="false"
                 android:paddingTop="8dp" />
         </LinearLayout>
 
diff --git a/app/src/main/res/layout/fragment_podcast_channel_page.xml b/app/src/main/res/layout/fragment_podcast_channel_page.xml
index d46e4202..18bfb462 100644
--- a/app/src/main/res/layout/fragment_podcast_channel_page.xml
+++ b/app/src/main/res/layout/fragment_podcast_channel_page.xml
@@ -14,8 +14,8 @@
             android:id="@+id/collapsing_toolbar"
             android:layout_width="match_parent"
             android:layout_height="match_parent"
-            app:expandedTitleMarginStart="@dimen/activity_margin_content"
             app:contentScrim="?attr/colorSurface"
+            app:expandedTitleMarginStart="@dimen/activity_margin_content"
             app:layout_scrollFlags="scroll|exitUntilCollapsed|snap">
 
             <ImageView
@@ -114,6 +114,7 @@
                     android:layout_marginTop="8dp"
                     android:layout_marginBottom="8dp"
                     android:clipToPadding="false"
+                    android:nestedScrollingEnabled="false"
                     android:paddingTop="8dp" />
             </LinearLayout>