Revert the loan pagination bump

Turns out this is limited to 100 and the API just returns 100
and ignores the offset then.

Add some extra error handling so this doesn't happen again.

src/Service/AlmaApi.php

@@ -1183,7 +1183,7 @@ public function getBookLoansJsonDataByPerson(Person $person): ?array
         try {
             $resultList = [];
             $loopCount = 0;
-            $limit = 1000;
+            $limit = 100;
             $offset = 0;
 
             // do as many requests as necessary to get all loans by the user

src/Service/AlmaUrlApi.php

@@ -150,8 +150,12 @@ public function getBookOfferLoansUrl(string $identifier): string
     /**
      * @throws UriException
      */
-    public function getLoansByUserIdUrl(string $userId, int $limit = 1000, int $offset = 0): string
+    public function getLoansByUserIdUrl(string $userId, int $limit = 100, int $offset = 0): string
     {
+        if ($limit < 0 || $limit > 100) {
+            throw new \RuntimeException('Valid values are 0-100');
+        }
+
         // see: https://developers.exlibrisgroup.com/alma/apis/docs/users/R0VUIC9hbG1hd3MvdjEvdXNlcnMve3VzZXJfaWR9L2xvYW5z/
         $uriTemplate = new UriTemplate('users/{userId}/loans{?limit,offset}');
 

tests/AlmaApiUrlTest.php

@@ -71,7 +71,7 @@ public function testGetBookOfferLoansUrl()
     public function testGetLoansByUserIdUrl()
     {
         $this->assertEquals(
-            'users/bla%3F/loans?limit=1000&offset=0',
+            'users/bla%3F/loans?limit=100&offset=0',
             $this->urls->getLoansByUserIdUrl('bla?'));
         $this->assertEquals(
             'users/bla%3F/loans?limit=20&offset=0',