TraktTools4: note collection limit, warn on missing page count header

Author: UweTrottmann

app/src/main/java/com/battlelancer/seriesguide/traktapi/TraktTools4.kt

@@ -16,6 +16,7 @@ import com.uwetrottmann.trakt5.services.Notes
 import com.uwetrottmann.trakt5.services.Sync
 import retrofit2.Call
 import retrofit2.awaitResponse
+import timber.log.Timber
 
 /**
  * Uses response classes inheriting from a Kotlin sealed interface.
@@ -74,6 +75,7 @@ object TraktTools4 {
             action = "get collected shows",
             reportIsNotVip = true // Should work even if not VIP
         ) { page ->
+            // 1000 is the maximum limit according to https://github.com/trakt/trakt-api/discussions/681
             traktSync.collectionShows(page, 1000, null)
         }
     }
@@ -113,6 +115,9 @@ object TraktTools4 {
                 is TraktNonNullResponse.Success -> {
                     allItems.addAll(response.data)
                     totalPageCount = response.pageCount
+                    if (totalPageCount == null) {
+                        Timber.w("Page count header not found for '$action'")
+                    }
                     currentPage++
                 }