From dd34e1a087bbe2702cb22b52088376cd41f657bc Mon Sep 17 00:00:00 2001
From: Henk Kalkwater <chris+dev@netsoj.nl>
Date: Sat, 23 Jul 2022 00:38:15 +0200
Subject: [PATCH] setRequestStartIndex: actually set start index

It previously modified the limit for some template instances. That
obviously does not work.

Additionally, setRequestStartIndex and setRequestLimit have been
implemented for GetNextUpParams.

Fixes #19
---
 core/include/JellyfinQt/apimodel.h |  2 ++
 core/src/apimodel.cpp              | 14 ++++++++++++--
 2 files changed, 14 insertions(+), 2 deletions(-)

diff --git a/core/include/JellyfinQt/apimodel.h b/core/include/JellyfinQt/apimodel.h
index c72cf44..f1aa95e 100644
--- a/core/include/JellyfinQt/apimodel.h
+++ b/core/include/JellyfinQt/apimodel.h
@@ -251,6 +251,8 @@ extern template void setRequestLimit(Loader::GetResumeItemsParams &params, int l
 extern template bool setRequestStartIndex(Loader::GetResumeItemsParams &params, int offset);
 extern template void setRequestLimit(Loader::GetPublicUsersParams &params, int limit);
 extern template bool setRequestStartIndex(Loader::GetPublicUsersParams &params, int offset);
+extern template void setRequestLimit(Loader::GetNextUpParams &params, int limit);
+extern template bool setRequestStartIndex(Loader::GetNextUpParams &params, int offset);
 
 extern template QList<DTO::UserDto> extractRecords(const QList<DTO::UserDto> &result);
 extern template int extractTotalRecordCount(const QList<DTO::UserDto> &result);
diff --git a/core/src/apimodel.cpp b/core/src/apimodel.cpp
index c22ed66..739642c 100644
--- a/core/src/apimodel.cpp
+++ b/core/src/apimodel.cpp
@@ -150,7 +150,7 @@ void setRequestLimit(Loader::GetItemsByUserIdParams &params, int limit) {
 
 template<>
 bool setRequestStartIndex(Loader::GetItemsByUserIdParams &params, int index) {
-    params.setLimit(index);
+    params.setStartIndex(index);
     return true;
 }
 
@@ -161,7 +161,7 @@ void setRequestLimit(Loader::GetResumeItemsParams &params, int limit) {
 
 template<>
 bool setRequestStartIndex(Loader::GetResumeItemsParams &params, int index) {
-    params.setLimit(index);
+    params.setStartIndex(index);
     return true;
 }
 
@@ -174,6 +174,16 @@ bool setRequestStartIndex(Loader::GetPublicUsersParams &/*params*/, int /*offset
     return false;
 }
 
+template<>
+void setRequestLimit(Loader::GetNextUpParams &params, int limit) {
+    params.setLimit(limit);
+}
+template<>
+bool setRequestStartIndex(Loader::GetNextUpParams &params, int offset) {
+    params.setStartIndex(offset);
+    return true;
+}
+
 template<>
 QList<DTO::UserDto> extractRecords(const QList<DTO::UserDto> &result) {
     return result;