From 46660b11efd2726e97299357f2da725e2d6f739c Mon Sep 17 00:00:00 2001
From: Florian Bruhin <git@the-compiler.org>
Date: Wed, 29 Jan 2014 09:14:37 +0100
Subject: [PATCH] Refactor CompletionView

---
 qutebrowser/utils/completion.py   | 30 +++++++++++++++++++++
 qutebrowser/widgets/completion.py | 44 ++++++-------------------------
 2 files changed, 38 insertions(+), 36 deletions(-)

diff --git a/qutebrowser/utils/completion.py b/qutebrowser/utils/completion.py
index 624516e0e..79835935d 100644
--- a/qutebrowser/utils/completion.py
+++ b/qutebrowser/utils/completion.py
@@ -111,6 +111,28 @@ class CompletionModel(QAbstractItemModel):
                 newitem = CompletionItem(item, newcat)
                 newcat.children.append(newitem)
 
+    def mark_all_items(self, needle):
+        for i in range(self.rowCount()):
+            cat = self.index(i, 0)
+            for k in range(self.rowCount(cat)):
+                idx = self.index(k, 0, cat)
+                old = self.data(idx).value()
+                marks = self._get_marks(needle, old)
+                self.setData(idx, marks, Qt.UserRole)
+
+    def _get_marks(self, needle, haystack):
+        pos1 = pos2 = 0
+        marks = []
+        if not needle:
+            return marks
+        while True:
+            pos1 = haystack.find(needle, pos2)
+            if pos1 == -1:
+                break
+            pos2 = pos1 + len(needle)
+            marks.append((pos1, pos2))
+        return marks
+
 
 class CompletionItem():
     parent = None
@@ -192,3 +214,11 @@ class CompletionFilterModel(QSortFilterProxyModel):
             return False
         else:
             return left < right
+
+    def first_item(self):
+        cat = self.index(0, 0)
+        return self.index(0, 0, cat)
+
+    def last_item(self):
+        cat = self.index(self.rowCount() - 1, 0)
+        return self.index(self.rowCount(cat) - 1, 0, cat)
diff --git a/qutebrowser/widgets/completion.py b/qutebrowser/widgets/completion.py
index de2ba7ea5..e4e91fc3a 100644
--- a/qutebrowser/widgets/completion.py
+++ b/qutebrowser/widgets/completion.py
@@ -152,40 +152,10 @@ class CompletionView(QTreeView):
         self.setmodel('command')
         text = text.lstrip(':')
         self.model.pattern = text
-        self.mark_all_items(text)
+        self.model.sourceModel().mark_all_items(text)
         if self.enabled:
             self.show()
 
-    def first_item(self):
-        cat = self.model.index(0, 0)
-        return self.model.index(0, 0, cat)
-
-    def last_item(self):
-        cat = self.model.index(self.model.rowCount() - 1, 0)
-        return self.model.index(self.model.rowCount(cat) - 1, 0, cat)
-
-    def mark_all_items(self, needle):
-        for i in range(self.model.rowCount()):
-            cat = self.model.index(i, 0)
-            for k in range(self.model.rowCount(cat)):
-                idx = self.model.index(k, 0, cat)
-                old = self.model.data(idx)
-                marks = self.get_marks(needle, old)
-                self.model.setData(idx, marks, Qt.UserRole)
-
-    def get_marks(self, needle, haystack):
-        pos1 = pos2 = 0
-        marks = []
-        if not needle:
-            return marks
-        while True:
-            pos1 = haystack.find(needle, pos2)
-            if pos1 == -1:
-                break
-            pos2 = pos1 + len(needle)
-            marks.append((pos1, pos2))
-        return marks
-
     def tab_handler(self, shift):
         if not self.completing:
             return
@@ -201,13 +171,15 @@ class CompletionView(QTreeView):
         idx = self.selectionModel().currentIndex()
         if not idx.isValid():
             # No item selected yet
-            return self.first_item()
+            return self.model.first_item()
         while True:
             idx = self.indexAbove(idx) if shift else self.indexBelow(idx)
-            if not idx.isValid():
-                # wrap around if we arrived at beginning/end
-                return self.last_item() if shift else self.first_item()
-            if idx.parent().isValid():
+            # wrap around if we arrived at beginning/end
+            if not idx.isValid() and shift:
+                return self.model.last_item()
+            elif not idx.isValid() and not shift:
+                return self.model.first_item()
+            elif idx.parent().isValid():
                 # Item is a real item, not a category header -> success
                 return idx