Merge branch 'hotline#191394_Koha_Connection' into 'master'

hotline#191394 : Koha: fix problem with user connection

See merge request !4884

VERSIONS_HOTLINE/191394

+ - correctif #191394 : Koha : corrige un problème à la connexion quand une limite d'affichage du nombre du nombre de prêts par page est présente
\ No newline at end of file

library/Class/WebService/SIGB/Koha/Service.php

@@ -156,13 +156,13 @@ class Class_WebService_SIGB_Koha_Service extends Class_WebService_SIGB_AbstractR
 
 
   public function setLoansPerPage($loans_per_page) {
-    $this->loans_per_page = $loans_per_page;
+    $this->loans_per_page = (int) $loans_per_page;
     return $this;
   }
 
 
   public function getLoansPerPage() :int {
-    return $this->loans_per_page ?? 0;
+    return (int) $this->loans_per_page ?? 0;
   }
 
 

tests/library/Class/WebService/SIGB/KohaCommunityTest.php

@@ -703,3 +703,26 @@ class KohaCommunityEditUserTest extends KohaCommunityTestCase {
     $this->assertEquals([], $this->borrower->getErrors());
   }
 }
+
+
+
+
+/* @see https://forge.afi-sa.net/issues/191394 */
+class KohaCommunityLoansPerPageTest extends KohaCommunityTestCase {
+  public function setUp(){
+    parent::setUp();
+    $this->service->setLoansPerPage('50');
+  }
+
+
+  /** @test */
+  public function loansPerPageShouldBeInt() {
+    $this->assertEquals('integer', gettype($this->service->getLoansPerPage()));
+  }
+
+
+  /** @test */
+  public function loansPerPageServiceShouldBe50() {
+    $this->assertEquals(50, $this->service->getLoansPerPage());
+  }
+}