feat: Refactor NewsLabelManager to prevent ANR

Refactored `NewsLabelManager` to use a `NewsRepository` and coroutines to move database operations off the main thread, fixing an ANR.

- Introduced `addLabel` and `removeLabel` suspend functions in `NewsRepository`.
- `NewsLabelManager` now uses a `CoroutineScope` to call these repository methods.
- Disabled the "add label" button during the async operation to prevent rapid double-clicks.
- Fixed a resource leak in `AdapterNews` by properly managing the Realm instance lifecycle.
- Fixed a regression in `AdapterNews` that was causing the wrong user to be displayed.
- Fixed all compilation errors that arose from the refactoring.

Author: google-labs-jules[bot]

app/build.gradle

@@ -184,6 +184,7 @@ dependencies {
     implementation "androidx.work:work-runtime:2.10.3"
     implementation "androidx.slidingpanelayout:slidingpanelayout:1.2.0"
     implementation 'androidx.preference:preference-ktx:1.2.1'
+    implementation "androidx.lifecycle:lifecycle-runtime-ktx:2.8.3"
 
     implementation 'com.google.code.gson:gson:2.13.1'
     implementation 'com.google.android.material:material:1.12.0'

app/src/main/java/org/ole/planet/myplanet/ui/news/AdapterNews.kt

@@ -32,6 +32,7 @@ import java.io.File
 import java.util.Calendar
 import java.util.Locale
 import androidx.lifecycle.findViewTreeLifecycleOwner
+import androidx.lifecycle.lifecycleScope
 import io.realm.Realm
 import kotlinx.coroutines.CoroutineScope
 import org.ole.planet.myplanet.R
@@ -600,15 +601,6 @@ class AdapterNews(
         }
     }
 
-    override fun onAttachedToRecyclerView(recyclerView: RecyclerView) {
-        super.onAttachedToRecyclerView(recyclerView)
-        this.recyclerView = recyclerView
-    }
-
-    override fun onDetachedFromRecyclerView(recyclerView: RecyclerView) {
-        super.onDetachedFromRecyclerView(recyclerView)
-        this.recyclerView = null
-    }
 
     private fun loadImage(binding: RowNewsBinding, news: RealmNews?) {
         binding.imgNews.visibility = View.GONE

app/src/main/java/org/ole/planet/myplanet/ui/news/NewsLabelManager.kt

@@ -86,7 +86,8 @@ class NewsLabelManager(
                     }
                     val newsId = news.id
                     if (selectedLabel != null && newsId != null) {
-                        chipCloud.isEnabled = false
+                        val deleteListener = chipCloud.deleteListener
+                        chipCloud.setDeleteListener(null)
                         coroutineScope.launch {
                             try {
                                 val success = newsRepository.removeLabel(newsId, selectedLabel)
@@ -98,7 +99,7 @@ class NewsLabelManager(
                                 }
                             } finally {
                                 withContext(Dispatchers.Main) {
-                                    chipCloud.isEnabled = true
+                                    chipCloud.setDeleteListener(deleteListener)
                                 }
                             }
                         }

app/src/main/java/org/ole/planet/myplanet/ui/news/ReplyActivity.kt

@@ -44,6 +44,8 @@ import org.ole.planet.myplanet.utilities.JsonUtils.getString
 open class ReplyActivity : AppCompatActivity(), OnNewsItemClickListener {
     private lateinit var activityReplyBinding: ActivityReplyBinding
     @Inject
+    lateinit var newsRepository: NewsRepository
+    @Inject
     lateinit var databaseService: DatabaseService
     var id: String? = null
     private lateinit var newsAdapter: AdapterNews
@@ -87,9 +89,8 @@ open class ReplyActivity : AppCompatActivity(), OnNewsItemClickListener {
         lifecycleScope.launch {
             val (news, list) = viewModel.getNewsWithReplies(id)
             databaseService.withRealm { realm ->
-                newsAdapter = AdapterNews(this@ReplyActivity, user, news, "", null, userProfileDbHandler)
+                newsAdapter = AdapterNews(this@ReplyActivity, user, news, newsRepository, "", null, userProfileDbHandler)
                 newsAdapter.setListener(this@ReplyActivity)
-                newsAdapter.setmRealm(realm)
                 newsAdapter.setFromLogin(intent.getBooleanExtra("fromLogin", false))
                 newsAdapter.setNonTeamMember(intent.getBooleanExtra("nonTeamMember", false))
                 newsAdapter.setImageList(imageList)

app/src/main/java/org/ole/planet/myplanet/ui/team/teamDiscussion/DiscussionListFragment.kt

@@ -39,7 +39,8 @@ class DiscussionListFragment : BaseTeamFragment() {
     private var _binding: FragmentDiscussionListBinding? = null
     private val binding get() = _binding!!
     private var updatedNewsList: RealmResults<RealmNews>? = null
-    
+    @Inject
+    lateinit var newsRepository: NewsRepository
     @Inject
     lateinit var userProfileDbHandler: UserProfileDbHandler
     private var filteredNewsList: List<RealmNews?> = listOf()
@@ -232,9 +233,8 @@ class DiscussionListFragment : BaseTeamFragment() {
         val existingAdapter = binding.rvDiscussion.adapter
         if (existingAdapter == null) {
             val adapterNews = activity?.let {
-                AdapterNews(it, user, null, getEffectiveTeamName(), teamId, userProfileDbHandler)
+                AdapterNews(it, user, null, newsRepository, getEffectiveTeamName(), teamId, userProfileDbHandler)
             }
-            adapterNews?.setmRealm(mRealm)
             adapterNews?.setListener(this)
             if (!isMemberFlow.value) adapterNews?.setNonTeamMember(true)
             realmNewsList?.let { adapterNews?.updateList(it) }