Add log message

Add some test that verify that we cant get in a enless loop and handle container_start corrently.

Author: Hellowlol

plexapi/library.py

@@ -381,7 +381,10 @@ def fetchItems(self, ekey, cls=None, container_start=None, container_size=None,
         data = self._server.query(ekey, params=url_kw)
 
         if '/all' in ekey:
-            self._total_size = utils.cast(int, data.attrib.get("totalSize"))
+            # totalSize is only included in the xml response
+            # if container size is used.
+            total_size = data.attrib.get("totalSize") or data.attrib.get("size")
+            self._total_size = utils.cast(int, total_size)
 
         items = self.findItems(data, cls, ekey, **kwargs)
 
@@ -585,25 +588,30 @@ def search(self, title=None, sort=None, maxresults=None,
 
         results = []
         subresults = []
+        offset = container_start
 
         if maxresults is not None:
             container_size = min(container_size, maxresults)
         while True:
             key = '/library/sections/%s/all%s' % (self.key, utils.joinArgs(args))
             subresults = self.fetchItems(key, container_start=container_start,
                                          container_size=container_size)
+            if not len(subresults):
+                if offset > self.totalSize:
+                    log.info("container_start is higher then the number of items in the library")
+                break
+
             results.extend(subresults)
 
             # self.totalSize is not used as a condition in the while loop as
             # this require a additional http request.
             # self.totalSize is updated from .fetchItems
+            wanted_number_of_items = self.totalSize - offset
             if maxresults is not None:
-                res = min(maxresults, self.totalSize)
+                wanted_number_of_items = min(maxresults, wanted_number_of_items)
                 container_size = min(container_size, maxresults - len(results))
-            else:
-                res = self.totalSize
 
-            if res <= len(results):
+            if wanted_number_of_items <= len(results):
                 break
 
             container_start += container_size

tests/test_library.py

@@ -263,3 +263,5 @@ def test_crazy_search(plex, movie):
     assert len(movies.search(container_start=2, maxresults=1)) == 1
     assert len(movies.search(container_size=None)) == 4
     assert len(movies.search(container_size=1)) == 4
+    assert len(movies.search(container_start=9999, container_size=1)) == 0
+    assert len(movies.search(container_start=2, container_size=1)) == 2