Consolidate duplicate TagClickListener and fix conflict

google-labs-jules[bot] · google-labs-jules[bot] · commit f82481ef3012 · 2026-01-07T18:15:45.000Z
Merged `TagClickListener` into `OnTagClickListener` with default implementations. Updated `OnCourseItemSelected` and `CoursesFragment` to use non-nullable `RealmTag` in `onTagClicked` to resolve interface signature conflict. Deleted `TagClickListener`.
diff --git a/app/src/main/java/org/ole/planet/myplanet/callback/OnCourseItemSelected.kt b/app/src/main/java/org/ole/planet/myplanet/callback/OnCourseItemSelected.kt
@@ -6,5 +6,5 @@ import org.ole.planet.myplanet.model.RealmTag
 interface OnCourseItemSelected {
     @JvmSuppressWildcards
     fun onSelectedListChange(list: MutableList<RealmMyCourse?>)
-    fun onTagClicked(tag: RealmTag?)
+    fun onTagClicked(tag: RealmTag)
 }
diff --git a/app/src/main/java/org/ole/planet/myplanet/ui/courses/CoursesFragment.kt b/app/src/main/java/org/ole/planet/myplanet/ui/courses/CoursesFragment.kt
@@ -562,9 +562,9 @@ class CoursesFragment : BaseRecyclerFragment<RealmMyCourse?>(), OnCourseItemSele
         hideButtons()
     }
 
-    override fun onTagClicked(tag: RealmTag?) {
+    override fun onTagClicked(tag: RealmTag) {
         if (!searchTags.contains(tag)) {
-            tag?.let { searchTags.add(it) }
+            searchTags.add(tag)
         }
         filterCoursesAndUpdateUi()
         showTagText(searchTags, tvSelected)

Original file line number	Diff line number	Diff line change
`@@ -6,5 +6,5 @@ import org.ole.planet.myplanet.model.RealmTag`
`6`	`6`	`interface OnCourseItemSelected {`
`7`	`7`	`@JvmSuppressWildcards`
`8`	`8`	`fun onSelectedListChange(list: MutableList<RealmMyCourse?>)`
`9`		`- fun onTagClicked(tag: RealmTag?)`
	`9`	`+ fun onTagClicked(tag: RealmTag)`
`10`	`10`	`}`
Original file line number	Diff line number	Diff line change
`@@ -562,9 +562,9 @@ class CoursesFragment : BaseRecyclerFragment<RealmMyCourse?>(), OnCourseItemSele`
`562`	`562`	`hideButtons()`
`563`	`563`	`}`
`564`	`564`
`565`		`- override fun onTagClicked(tag: RealmTag?) {`
	`565`	`+ override fun onTagClicked(tag: RealmTag) {`
`566`	`566`	`if (!searchTags.contains(tag)) {`
`567`		`- tag?.let { searchTags.add(it) }`
	`567`	`+ searchTags.add(tag)`
`568`	`568`	`}`
`569`	`569`	`filterCoursesAndUpdateUi()`
`570`	`570`	`showTagText(searchTags, tvSelected)`