BaseFragment: Drop unused app parameter

Signed-off-by: Aayush Gupta <aayushgupta219@gmail.com>
This commit is contained in:
Aayush Gupta
2025-06-30 12:50:56 +08:00
parent 2bb3bbec0c
commit cd5da2b83c
10 changed files with 10 additions and 13 deletions

View File

@@ -145,8 +145,7 @@ class AppsGamesFragment : BaseFragment<FragmentGenericWithSearchBinding>() {
.app(app) .app(app)
.click { _ -> .click { _ ->
openDetailsFragment( openDetailsFragment(
app.packageName, app.packageName
app
) )
} }
.longClick { _ -> .longClick { _ ->

View File

@@ -29,7 +29,6 @@ import androidx.navigation.fragment.findNavController
import androidx.viewbinding.ViewBinding import androidx.viewbinding.ViewBinding
import com.airbnb.epoxy.EpoxyRecyclerView import com.airbnb.epoxy.EpoxyRecyclerView
import com.aurora.extensions.navigate import com.aurora.extensions.navigate
import com.aurora.gplayapi.data.models.App
import com.aurora.gplayapi.data.models.Category import com.aurora.gplayapi.data.models.Category
import com.aurora.gplayapi.data.models.StreamCluster import com.aurora.gplayapi.data.models.StreamCluster
import com.aurora.store.MobileNavigationDirections import com.aurora.store.MobileNavigationDirections
@@ -82,7 +81,7 @@ abstract class BaseFragment<ViewBindingType : ViewBinding> : Fragment() {
super.onDestroyView() super.onDestroyView()
} }
fun openDetailsFragment(packageName: String, app: App? = null) { fun openDetailsFragment(packageName: String) {
requireContext().navigate( requireContext().navigate(
Screen.AppDetails(packageName) Screen.AppDetails(packageName)
) )

View File

@@ -91,7 +91,7 @@ class CategoryBrowseFragment : BaseFragment<FragmentGenericWithToolbarBinding>()
} }
override fun onAppClick(app: App) { override fun onAppClick(app: App) {
openDetailsFragment(app.packageName, app) openDetailsFragment(app.packageName)
} }
override fun onAppLongClick(app: App) { override fun onAppLongClick(app: App) {

View File

@@ -92,7 +92,7 @@ class ExpandedStreamBrowseFragment : BaseFragment<FragmentGenericWithToolbarBind
AppListViewModel_() AppListViewModel_()
.id(it.packageName.hashCode()) .id(it.packageName.hashCode())
.app(it) .app(it)
.click { _ -> openDetailsFragment(it.packageName, it) } .click { _ -> openDetailsFragment(it.packageName) }
) )
} }

View File

@@ -30,7 +30,6 @@ import com.aurora.Constants
import com.aurora.extensions.toast import com.aurora.extensions.toast
import com.aurora.store.R import com.aurora.store.R
import com.aurora.store.data.room.favourite.Favourite import com.aurora.store.data.room.favourite.Favourite
import com.aurora.store.data.room.favourite.Favourite.Companion.toApp
import com.aurora.store.databinding.FragmentFavouriteBinding import com.aurora.store.databinding.FragmentFavouriteBinding
import com.aurora.store.view.epoxy.views.FavouriteViewModel_ import com.aurora.store.view.epoxy.views.FavouriteViewModel_
import com.aurora.store.view.epoxy.views.app.NoAppViewModel_ import com.aurora.store.view.epoxy.views.app.NoAppViewModel_
@@ -104,7 +103,7 @@ class FavouriteFragment : BaseFragment<FragmentFavouriteBinding>() {
FavouriteViewModel_() FavouriteViewModel_()
.id(it.packageName.hashCode()) .id(it.packageName.hashCode())
.favourite(it) .favourite(it)
.onClick { _ -> openDetailsFragment(it.packageName, it.toApp()) } .onClick { _ -> openDetailsFragment(it.packageName) }
.onFavourite { _ -> viewModel.removeFavourite(it.packageName) } .onFavourite { _ -> viewModel.removeFavourite(it.packageName) }
) )
} }

View File

@@ -106,7 +106,7 @@ class ForYouFragment : BaseFragment<FragmentForYouBinding>(),
} }
override fun onAppClick(app: App) { override fun onAppClick(app: App) {
openDetailsFragment(app.packageName, app) openDetailsFragment(app.packageName)
} }
override fun onAppLongClick(app: App) { override fun onAppLongClick(app: App) {

View File

@@ -80,7 +80,7 @@ class StreamBrowseFragment : BaseFragment<FragmentGenericWithToolbarBinding>() {
AppListViewModel_() AppListViewModel_()
.id(it.packageName.hashCode()) .id(it.packageName.hashCode())
.app(it) .app(it)
.click { _ -> openDetailsFragment(it.packageName, it) } .click { _ -> openDetailsFragment(it.packageName) }
) )
} }

View File

@@ -123,7 +123,7 @@ class TopChartFragment : BaseFragment<FragmentTopContainerBinding>() {
AppListViewModel_() AppListViewModel_()
.id(app.id) .id(app.id)
.app(app) .app(app)
.click { _ -> openDetailsFragment(app.packageName, app) } .click { _ -> openDetailsFragment(app.packageName) }
) )
} }

View File

@@ -97,7 +97,7 @@ class DevProfileFragment : BaseFragment<FragmentDevProfileBinding>(),
} }
override fun onAppClick(app: App) { override fun onAppClick(app: App) {
openDetailsFragment(app.packageName, app) openDetailsFragment(app.packageName)
} }
override fun onAppLongClick(app: App) { override fun onAppLongClick(app: App) {

View File

@@ -153,7 +153,7 @@ class SearchResultsFragment : BaseFragment<FragmentSearchResultBinding>(),
} }
override fun onAppClick(app: App) { override fun onAppClick(app: App) {
openDetailsFragment(app.packageName, app) openDetailsFragment(app.packageName)
} }
override fun onAppLongClick(app: App) { override fun onAppLongClick(app: App) {