refactor: split Utils.kt by function or receiver type

Author: WhiredPlanck

app/src/main/java/com/osfans/trime/ui/fragments/AboutFragment.kt

@@ -20,7 +20,6 @@ import com.osfans.trime.ui.components.PaddingPreferenceFragment
 import com.osfans.trime.ui.main.MainViewModel
 import com.osfans.trime.util.Const
 import com.osfans.trime.util.formatDateTime
-import com.osfans.trime.util.thirdPartySummary
 import com.osfans.trime.util.toast
 import splitties.systemservices.clipboardManager
 
@@ -58,10 +57,22 @@ class AboutFragment : PaddingPreferenceFragment() {
                     true
                 }
             }
-            get<Preference>("about__librime_version")
-                ?.thirdPartySummary(Rime.getLibrimeVersion())
-            get<Preference>("about__opencc_version")
-                ?.thirdPartySummary(OpenCCDictManager.getOpenCCVersion())
+            get<Preference>("about__librime_version")?.apply {
+                val code = Rime.getLibrimeVersion()
+                val hash = extractCommitHash(code)
+                summary = code
+                intent?.data?.also {
+                    intent!!.data = Uri.withAppendedPath(it, "commit/$hash")
+                }
+            }
+            get<Preference>("about__opencc_version")?.apply {
+                val code = OpenCCDictManager.getOpenCCVersion()
+                val hash = extractCommitHash(code)
+                summary = code
+                intent?.data?.also {
+                    intent!!.data = Uri.withAppendedPath(it, "commit/$hash")
+                }
+            }
             get<Preference>("about__open_source_licenses")?.apply {
                 setOnPreferenceClickListener {
                     findNavController().navigate(R.id.action_aboutFragment_to_licenseFragment)
@@ -75,4 +86,14 @@ class AboutFragment : PaddingPreferenceFragment() {
         super.onResume()
         viewModel.setToolbarTitle(getString(R.string.pref_about))
     }
+
+    companion object {
+        private val DASH_G_PATTERN = Regex("^(.*-g)([0-9a-f]+)(.*)$")
+        private val COMMON_PATTERN = Regex("^([^-]*)(-.*)$")
+
+        private fun extractCommitHash(versionCode: String): String =
+            DASH_G_PATTERN.find(versionCode)?.groupValues?.get(1)
+                ?: COMMON_PATTERN.find(versionCode)?.groupValues?.get(1)
+                ?: versionCode
+    }
 }

app/src/main/java/com/osfans/trime/util/AppContext.kt

@@ -0,0 +1,11 @@
+/*
+ * SPDX-FileCopyrightText: 2015 - 2024 Rime community
+ * SPDX-License-Identifier: GPL-3.0-or-later
+ */
+
+package com.osfans.trime.util
+
+import android.content.Context
+import com.osfans.trime.TrimeApplication
+
+val appContext: Context get() = TrimeApplication.getInstance().applicationContext

app/src/main/java/com/osfans/trime/util/Bundle.kt

@@ -0,0 +1,18 @@
+/*
+ * SPDX-FileCopyrightText: 2015 - 2024 Rime community
+ * SPDX-License-Identifier: GPL-3.0-or-later
+ */
+
+package com.osfans.trime.util
+
+import android.os.Build
+import android.os.Bundle
+import java.io.Serializable
+
+inline fun <reified T : Serializable> Bundle.serializable(key: String): T? =
+    if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.TIRAMISU) {
+        getSerializable(key, T::class.java)
+    } else {
+        @Suppress("DEPRECATION")
+        getSerializable(key) as? T
+    }

…n/java/com/osfans/trime/util/Datetime.kt …n/java/com/osfans/trime/util/DateTime.ktapp/src/main/java/com/osfans/trime/util/Datetime.kt renamed to app/src/main/java/com/osfans/trime/util/DateTime.kt app/src/main/java/com/osfans/trime/util/Datetime.kt renamed to app/src/main/java/com/osfans/trime/util/DateTime.kt

@@ -0,0 +1,51 @@
+/*
+ * SPDX-FileCopyrightText: 2015 - 2024 Rime community
+ * SPDX-License-Identifier: GPL-3.0-or-later
+ */
+
+package com.osfans.trime.util
+
+import android.content.Context
+import android.os.Environment
+import com.hjq.permissions.OnPermissionCallback
+import com.hjq.permissions.Permission
+import com.hjq.permissions.XXPermissions
+import com.osfans.trime.R
+
+@Suppress("NOTHING_TO_INLINE")
+inline fun Context.isStorageAvailable(): Boolean =
+    XXPermissions.isGranted(this, Permission.MANAGE_EXTERNAL_STORAGE) &&
+        Environment.getExternalStorageDirectory().absolutePath.isNotEmpty()
+
+fun Context.requestExternalStoragePermission() {
+    XXPermissions
+        .with(this)
+        .permission(Permission.MANAGE_EXTERNAL_STORAGE)
+        .request(
+            object : OnPermissionCallback {
+                override fun onGranted(
+                    permissions: List<String>,
+                    all: Boolean,
+                ) {
+                    if (all) {
+                        toast(R.string.external_storage_permission_granted)
+                    }
+                }
+
+                override fun onDenied(
+                    permissions: List<String>,
+                    never: Boolean,
+                ) {
+                    if (never) {
+                        toast(R.string.external_storage_permission_denied)
+                        XXPermissions.startPermissionActivity(
+                            this@requestExternalStoragePermission,
+                            permissions,
+                        )
+                    } else {
+                        toast(R.string.external_storage_permission_denied)
+                    }
+                }
+            },
+        )
+}

app/src/main/java/com/osfans/trime/util/Permissions.kt

@@ -0,0 +1,20 @@
+/*
+ * SPDX-FileCopyrightText: 2015 - 2024 Rime community
+ * SPDX-License-Identifier: GPL-3.0-or-later
+ */
+
+package com.osfans.trime.util
+
+import androidx.core.view.ViewCompat
+import androidx.core.view.WindowInsetsCompat
+import androidx.recyclerview.widget.RecyclerView
+
+fun RecyclerView.applyNavBarInsetsBottomPadding() {
+    clipToPadding = false
+    ViewCompat.setOnApplyWindowInsetsListener(this) { _, windowInsets ->
+        windowInsets.getInsets(WindowInsetsCompat.Type.navigationBars()).also {
+            setPadding(paddingLeft, paddingTop, paddingRight, it.bottom)
+        }
+        windowInsets
+    }
+}

app/src/main/java/com/osfans/trime/util/RecyclerView.kt

@@ -0,0 +1,34 @@
+/*
+ * SPDX-FileCopyrightText: 2015 - 2024 Rime community
+ * SPDX-License-Identifier: GPL-3.0-or-later
+ */
+
+package com.osfans.trime.util
+
+import android.content.Context
+import android.util.TypedValue
+import android.view.View
+import androidx.annotation.AttrRes
+import androidx.fragment.app.Fragment
+import splitties.experimental.InternalSplittiesApi
+import splitties.resources.withResolvedThemeAttribute
+
+@OptIn(InternalSplittiesApi::class)
+fun Context.styledFloat(
+    @AttrRes attrRes: Int,
+) = withResolvedThemeAttribute(attrRes) {
+    when (type) {
+        TypedValue.TYPE_FLOAT -> float
+        else -> throw IllegalArgumentException("float attribute expected")
+    }
+}
+
+@Suppress("NOTHING_TO_INLINE")
+inline fun View.styledFloat(
+    @AttrRes attrRes: Int,
+) = context.styledFloat(attrRes)
+
+@Suppress("NOTHING_TO_INLINE")
+inline fun Fragment.styledFloat(
+    @AttrRes attrRes: Int,
+) = context!!.styledFloat(attrRes)

app/src/main/java/com/osfans/trime/util/Splitties.kt

@@ -4,33 +4,8 @@
 
 package com.osfans.trime.util
 
+import java.util.Collections
 import java.util.WeakHashMap
 
-class WeakHashSet<T> : MutableSet<T> {
-    private val core = WeakHashMap<T, PlaceHolder>()
-
-    private object PlaceHolder
-
-    val view = object : Set<T> by core.keys {}
-    override val size get() = core.size
-
-    override fun iterator(): MutableIterator<T> = core.keys.iterator()
-
-    override fun add(element: T) = core.put(element, PlaceHolder) == null
-
-    override fun addAll(elements: Collection<T>) = elements.all(::add)
-
-    override fun remove(element: T) = core.remove(element) != null
-
-    override fun removeAll(elements: Collection<T>) = elements.all(::remove)
-
-    override fun clear() = core.clear()
-
-    override fun retainAll(elements: Collection<T>) = removeAll(core.keys.filter { it !in elements })
-
-    override operator fun contains(element: T) = core.containsKey(element)
-
-    override fun containsAll(elements: Collection<T>) = elements.all(::contains)
-
-    override fun isEmpty() = core.isEmpty()
-}
+@Suppress("FunctionName")
+fun <T> WeakHashSet(): MutableSet<T> = Collections.newSetFromMap(WeakHashMap<T, Boolean>())