diff --git a/app/src/main/java/fr/free/nrw/commons/AboutActivity.kt b/app/src/main/java/fr/free/nrw/commons/AboutActivity.kt index e24522c33..f6e8714ab 100644 --- a/app/src/main/java/fr/free/nrw/commons/AboutActivity.kt +++ b/app/src/main/java/fr/free/nrw/commons/AboutActivity.kt @@ -18,7 +18,7 @@ import fr.free.nrw.commons.databinding.ActivityAboutBinding import fr.free.nrw.commons.theme.BaseActivity import fr.free.nrw.commons.utils.ConfigUtils.getVersionNameWithSha import fr.free.nrw.commons.utils.DialogUtil.showAlertDialog -import fr.free.nrw.commons.utils.Utils +import fr.free.nrw.commons.utils.UnderlineUtils import fr.free.nrw.commons.utils.UrlUtils.handleWebUrl import java.util.Collections import androidx.core.net.toUri @@ -65,27 +65,27 @@ class AboutActivity : BaseActivity() { binding!!.aboutImprove.setHtmlText(improveText) binding!!.aboutVersion.text = applicationContext.getVersionNameWithSha() - Utils.setUnderlinedText( + UnderlineUtils.setUnderlinedText( binding!!.aboutFaq, R.string.about_faq, applicationContext ) - Utils.setUnderlinedText( + UnderlineUtils.setUnderlinedText( binding!!.aboutRateUs, R.string.about_rate_us, applicationContext ) - Utils.setUnderlinedText( + UnderlineUtils.setUnderlinedText( binding!!.aboutUserGuide, R.string.user_guide, applicationContext ) - Utils.setUnderlinedText( + UnderlineUtils.setUnderlinedText( binding!!.aboutPrivacyPolicy, R.string.about_privacy_policy, applicationContext ) - Utils.setUnderlinedText( + UnderlineUtils.setUnderlinedText( binding!!.aboutTranslate, R.string.about_translate, applicationContext ) - Utils.setUnderlinedText( + UnderlineUtils.setUnderlinedText( binding!!.aboutCredits, R.string.about_credits, applicationContext ) diff --git a/app/src/main/java/fr/free/nrw/commons/WelcomePagerAdapter.java b/app/src/main/java/fr/free/nrw/commons/WelcomePagerAdapter.java index c5eac863e..512e42fbf 100644 --- a/app/src/main/java/fr/free/nrw/commons/WelcomePagerAdapter.java +++ b/app/src/main/java/fr/free/nrw/commons/WelcomePagerAdapter.java @@ -8,7 +8,7 @@ import android.widget.TextView; import androidx.viewpager.widget.PagerAdapter; import fr.free.nrw.commons.utils.UrlUtils; -import fr.free.nrw.commons.utils.Utils; +import fr.free.nrw.commons.utils.UnderlineUtils; public class WelcomePagerAdapter extends PagerAdapter { private static final int[] PAGE_LAYOUTS = new int[]{ @@ -48,7 +48,7 @@ public class WelcomePagerAdapter extends PagerAdapter { if (position == PAGE_LAYOUTS.length - 1) { // Add link to more information TextView moreInfo = layout.findViewById(R.id.welcomeInfo); - Utils.setUnderlinedText(moreInfo, R.string.welcome_help_button_text, container.getContext()); + UnderlineUtils.setUnderlinedText(moreInfo, R.string.welcome_help_button_text, container.getContext()); moreInfo.setOnClickListener(view -> UrlUtils.handleWebUrl( container.getContext(), Uri.parse("https://commons.wikimedia.org/wiki/Help:Contents") diff --git a/app/src/main/java/fr/free/nrw/commons/media/MediaDetailFragment.kt b/app/src/main/java/fr/free/nrw/commons/media/MediaDetailFragment.kt index 883817017..5bfbacc28 100644 --- a/app/src/main/java/fr/free/nrw/commons/media/MediaDetailFragment.kt +++ b/app/src/main/java/fr/free/nrw/commons/media/MediaDetailFragment.kt @@ -77,7 +77,7 @@ import fr.free.nrw.commons.CommonsApplication.Companion.instance import fr.free.nrw.commons.Media import fr.free.nrw.commons.MediaDataExtractor import fr.free.nrw.commons.R -import fr.free.nrw.commons.utils.Utils +import fr.free.nrw.commons.utils.UnderlineUtils import fr.free.nrw.commons.actions.ThanksClient import fr.free.nrw.commons.auth.SessionManager import fr.free.nrw.commons.auth.csrf.InvalidLoginTokenException @@ -320,7 +320,7 @@ class MediaDetailFragment : CommonsDaggerSupportFragment(), CategoryEditHelper.C val view: View = binding.root - Utils.setUnderlinedText(binding.seeMore, R.string.nominated_see_more, requireContext()) + UnderlineUtils.setUnderlinedText(binding.seeMore, R.string.nominated_see_more, requireContext()) if (isCategoryImage) { binding.authorLinearLayout.visibility = View.VISIBLE diff --git a/app/src/main/java/fr/free/nrw/commons/utils/Utils.java b/app/src/main/java/fr/free/nrw/commons/utils/UnderlineUtils.java similarity index 66% rename from app/src/main/java/fr/free/nrw/commons/utils/Utils.java rename to app/src/main/java/fr/free/nrw/commons/utils/UnderlineUtils.java index e12e44c41..255c60c5e 100644 --- a/app/src/main/java/fr/free/nrw/commons/utils/Utils.java +++ b/app/src/main/java/fr/free/nrw/commons/utils/UnderlineUtils.java @@ -5,7 +5,7 @@ import android.text.SpannableString; import android.text.style.UnderlineSpan; import android.widget.TextView; -public class Utils { +public final class UnderlineUtils { /** * This method sets underlined string text to a TextView @@ -14,8 +14,8 @@ public class Utils { * @param stringResourceName string resource name * @param context */ - public static void setUnderlinedText(TextView textView, int stringResourceName, Context context) { - SpannableString content = new SpannableString(context.getString(stringResourceName)); + public static void setUnderlinedText(final TextView textView, final int stringResourceName, final Context context) { + final SpannableString content = new SpannableString(context.getString(stringResourceName)); content.setSpan(new UnderlineSpan(), 0, content.length(), 0); textView.setText(content); } diff --git a/app/src/test/kotlin/fr/free/nrw/commons/upload/MediaLicensePresenterTest.kt b/app/src/test/kotlin/fr/free/nrw/commons/upload/MediaLicensePresenterTest.kt index c118c6f9c..3aa7fb619 100644 --- a/app/src/test/kotlin/fr/free/nrw/commons/upload/MediaLicensePresenterTest.kt +++ b/app/src/test/kotlin/fr/free/nrw/commons/upload/MediaLicensePresenterTest.kt @@ -1,7 +1,7 @@ package fr.free.nrw.commons.upload import com.nhaarman.mockitokotlin2.verify -import fr.free.nrw.commons.utils.Utils +import fr.free.nrw.commons.utils.UnderlineUtils import fr.free.nrw.commons.kvstore.JsonKvStore import fr.free.nrw.commons.repository.UploadRepository import fr.free.nrw.commons.upload.license.MediaLicenseContract @@ -26,7 +26,7 @@ import org.robolectric.RobolectricTestRunner */ @RunWith(RobolectricTestRunner::class) -@PrepareForTest(Utils::class) +@PrepareForTest(UnderlineUtils::class) class MediaLicensePresenterTest { @Mock internal lateinit var repository: UploadRepository @@ -40,7 +40,7 @@ class MediaLicensePresenterTest { @InjectMocks lateinit var mediaLicensePresenter: MediaLicensePresenter - private lateinit var mockedUtil: MockedStatic + private lateinit var mockedUtil: MockedStatic /** * initial setup test environemnt @@ -50,7 +50,7 @@ class MediaLicensePresenterTest { fun setUp() { MockitoAnnotations.openMocks(this) mediaLicensePresenter.onAttachView(view) - mockedUtil = Mockito.mockStatic(Utils::class.java) + mockedUtil = Mockito.mockStatic(UnderlineUtils::class.java) `when`(Licenses.licenseNameFor(ArgumentMatchers.anyString())).thenReturn(1) }