# HG changeset patch
# User Mads Kiilerich <mads@kiilerich.com>
# Date 2019-11-07 03:12:41
# Node ID 7433775cc53b4fe04f491f2caa91486c9ecc8812
# Parent  2323e2bb27976c09301b9a618bdfa028a41f87a4

page: minimal change to move from webhelpers.paginate to paginate

webhelpers is dead and doesn't work with py3. paginate is not very actively
maintained, but it is the natural successor to webhelpers.paginate, it seems
stable, and it works with py3.

This is a minimal change that seems to work. It preserves existing tech debt
... and adds a little bit more. It will be cleaned up next.

webhelpers.paginate had built-in SqlAlchemy support - now we have to handle it
explicitly.

diff --git a/kallithea/lib/page.py b/kallithea/lib/page.py
--- a/kallithea/lib/page.py
+++ b/kallithea/lib/page.py
@@ -17,8 +17,10 @@ Custom paging classes
 import logging
 import re
 
+import paginate
+import paginate_sqlalchemy
+import sqlalchemy.orm
 from webhelpers2.html import HTML, literal
-from webhelpers.paginate import Page as _Page
 
 from kallithea.config.routing import url
 
@@ -26,15 +28,19 @@ from kallithea.config.routing import url
 log = logging.getLogger(__name__)
 
 
-class Page(_Page):
-    """
-    Custom pager emitting Bootstrap paginators
-    """
+class Page(paginate.Page):
+
     def __init__(self, collection,
                  page=1, items_per_page=20, item_count=None,
                  **kwargs):
-        _Page.__init__(self, collection, page=page, items_per_page=items_per_page, item_count=item_count,
-                       url=url.current, **kwargs)
+        if isinstance(collection, sqlalchemy.orm.query.Query):
+            collection = paginate_sqlalchemy.SqlalchemyOrmWrapper(collection)
+        paginate.Page.__init__(self, collection, page=page, items_per_page=items_per_page, item_count=item_count,
+                               url_maker=lambda page: url.current(page=page, **kwargs))
+
+    def _pagerlink(self, page, text):
+        """hack to mimic old webhelpers.paginate internals"""
+        return literal('''<li><a class="pager_link" href="%s">%s</a></li>''') % (self.url_maker(page), text)
 
     def _get_pos(self, cur_page, max_page, items):
         edge = (items / 2) + 1
diff --git a/setup.py b/setup.py
--- a/setup.py
+++ b/setup.py
@@ -69,6 +69,8 @@ requirements = [
     "bleach >= 3.0, < 3.2",
     "Click >= 7.0, < 8",
     "ipaddr >= 2.1.10, < 2.3",
+    "paginate >= 0.5, < 0.6",
+    "paginate_sqlalchemy >= 0.3.0, < 0.4",
 ]
 
 if not is_windows: