From a45edbd1da723e4f58d7246a93c5250d15834273 Mon Sep 17 00:00:00 2001 From: Paul Hawke Date: Wed, 2 Jul 2025 16:00:20 -0500 Subject: [PATCH] Inline the creation of a page title object --- app/src/main/java/fr/free/nrw/commons/Media.kt | 6 ++++-- .../free/nrw/commons/category/CategoryDetailsActivity.kt | 6 +++++- app/src/main/java/fr/free/nrw/commons/utils/Utils.java | 9 --------- 3 files changed, 9 insertions(+), 12 deletions(-) diff --git a/app/src/main/java/fr/free/nrw/commons/Media.kt b/app/src/main/java/fr/free/nrw/commons/Media.kt index 028bcf7d3..dbe722e91 100644 --- a/app/src/main/java/fr/free/nrw/commons/Media.kt +++ b/app/src/main/java/fr/free/nrw/commons/Media.kt @@ -1,8 +1,9 @@ package fr.free.nrw.commons import android.os.Parcelable +import fr.free.nrw.commons.BuildConfig.COMMONS_URL import fr.free.nrw.commons.location.LatLng -import fr.free.nrw.commons.utils.Utils +import fr.free.nrw.commons.wikidata.model.WikiSite import fr.free.nrw.commons.wikidata.model.page.PageTitle import kotlinx.parcelize.IgnoredOnParcel import kotlinx.parcelize.Parcelize @@ -174,7 +175,8 @@ class Media constructor( * Gets file page title * @return New media page title */ - val pageTitle: PageTitle get() = Utils.getPageTitle(filename!!) + val pageTitle: PageTitle + get() = PageTitle(filename!!, WikiSite(COMMONS_URL)) /** * Returns wikicode to use the media file on a MediaWiki site diff --git a/app/src/main/java/fr/free/nrw/commons/category/CategoryDetailsActivity.kt b/app/src/main/java/fr/free/nrw/commons/category/CategoryDetailsActivity.kt index 714d650d5..c806e79e1 100644 --- a/app/src/main/java/fr/free/nrw/commons/category/CategoryDetailsActivity.kt +++ b/app/src/main/java/fr/free/nrw/commons/category/CategoryDetailsActivity.kt @@ -13,6 +13,7 @@ import androidx.fragment.app.FragmentManager import androidx.lifecycle.Lifecycle import androidx.lifecycle.lifecycleScope import androidx.lifecycle.repeatOnLifecycle +import fr.free.nrw.commons.BuildConfig.COMMONS_URL import fr.free.nrw.commons.Media import fr.free.nrw.commons.R import fr.free.nrw.commons.utils.Utils @@ -23,6 +24,8 @@ import fr.free.nrw.commons.explore.categories.parent.ParentCategoriesFragment import fr.free.nrw.commons.explore.categories.sub.SubCategoriesFragment import fr.free.nrw.commons.media.MediaDetailPagerFragment import fr.free.nrw.commons.theme.BaseActivity +import fr.free.nrw.commons.wikidata.model.WikiSite +import fr.free.nrw.commons.wikidata.model.page.PageTitle import kotlinx.coroutines.launch import javax.inject.Inject @@ -199,7 +202,8 @@ class CategoryDetailsActivity : BaseActivity(), override fun onOptionsItemSelected(item: MenuItem): Boolean { return when (item.itemId) { R.id.menu_browser_current_category -> { - val title = Utils.getPageTitle(CATEGORY_PREFIX + categoryName) + val title = PageTitle(CATEGORY_PREFIX + categoryName, WikiSite(COMMONS_URL)) + Utils.handleWebUrl(this, Uri.parse(title.canonicalUri)) true } diff --git a/app/src/main/java/fr/free/nrw/commons/utils/Utils.java b/app/src/main/java/fr/free/nrw/commons/utils/Utils.java index e7b2cf931..417b69cd8 100644 --- a/app/src/main/java/fr/free/nrw/commons/utils/Utils.java +++ b/app/src/main/java/fr/free/nrw/commons/utils/Utils.java @@ -16,13 +16,10 @@ import androidx.browser.customtabs.CustomTabColorSchemeParams; import androidx.browser.customtabs.CustomTabsIntent; import androidx.core.content.ContextCompat; -import fr.free.nrw.commons.BuildConfig; import fr.free.nrw.commons.R; import fr.free.nrw.commons.Urls; import java.util.Calendar; import java.util.Date; -import fr.free.nrw.commons.wikidata.model.WikiSite; -import fr.free.nrw.commons.wikidata.model.page.PageTitle; import java.util.Locale; import java.util.regex.Pattern; @@ -33,10 +30,6 @@ import timber.log.Timber; public class Utils { - public static PageTitle getPageTitle(@NonNull String title) { - return new PageTitle(title, new WikiSite(BuildConfig.COMMONS_URL)); - } - /** * Generates licence name with given ID * @param license License ID @@ -63,8 +56,6 @@ public class Utils { * @param license License ID * @return Url of license */ - - @NonNull public static String licenseUrlFor(String license) { switch (license) {