[fenix] No Issue - Fixes the offset key to prevent endless history bug

5 years ago · 4019019ea3
parent 464a5f92cd
commit 4019019ea3
1 changed files with 3 additions and 1 deletions
--- a/app/src/main/java/org/mozilla/fenix/library/history/HistoryDataSource.kt
+++ b/app/src/main/java/org/mozilla/fenix/library/history/HistoryDataSource.kt
@ -43,7 +43,9 @@ class HistoryDataSource(
                    ?: visit.url.getHostFromUrl()
                    ?: visit.url

-                HistoryItem(offset + id, title, visit.url, visit.visitTime)
+                // We want IDs to start at 1, not 0 to be able to send the correct offset
+                // to the `historyProvider.getHistory` call.
+                HistoryItem(offset + id + 1, title, visit.url, visit.visitTime)
            }
        }
    }