offset the speaker ids to not overlaps the event ones

Author: quiro91

app/src/main/java/net/squanchy/search/view/SearchAdapter.kt

@@ -51,7 +51,7 @@ internal class SearchAdapter(activity: AppCompatActivity) : RecyclerView.Adapter
         val item = searchResult.elements[position]
         return when (item) {
             is EventElement -> item.event.numericId
-            is SpeakerElement -> item.speaker.numericId
+            is SpeakerElement -> speakerIdOffsetter(item.speaker.numericId)
             is SearchListElement.EventHeader -> ITEM_ID_EVENTS_HEADER
             is SearchListElement.SpeakerHeader -> ITEM_ID_SPEAKERS_HEADER
             is SearchListElement.AlgoliaLogo -> ITEM_ID_ALGOLIA_LOGO
@@ -115,8 +115,13 @@ internal class SearchAdapter(activity: AppCompatActivity) : RecyclerView.Adapter
         // random values that are used for non-hardcoded numeric IDs (for events
         // and speakers). This is a reasonable assumption in the Long range.
         // In addition, the CRC32 values we use as numeric IDs are always positive.
-        private const val ITEM_ID_EVENTS_HEADER: Long = -100
-        private const val ITEM_ID_SPEAKERS_HEADER: Long = -101
-        private const val ITEM_ID_ALGOLIA_LOGO: Long = -102
+        private const val ITEM_ID_EVENTS_HEADER: Long = -1
+        private const val ITEM_ID_SPEAKERS_HEADER: Long = -2
+        private const val ITEM_ID_ALGOLIA_LOGO: Long = -3
+
+        // As we cannot guarantee that the speaker ids are going to be unique compared
+        // to the ones for the events, we are making them negatives and offsetting them by 4,
+        // to be sure they do not overlap the ids for the two headers and the logo
+        private val speakerIdOffsetter: (Long) -> Long = { (it * -1) - 4 }
     }
 }