Since all notebook pages share the same store, a device being visible on one
page means it'll be counted as being visible on all pages. That means
flipping to the correct initial page doesn't work. Instead, we need to count
using the visible methods on callback objects to only count the devices that
are members of each page as visible.
---
iw/filter_gui.py | 11 ++++++++++-
1 files changed, 10 insertions(+), 1 deletions(-)

diff --git a/iw/filter_gui.py b/iw/filter_gui.py
index 4247503..efdb5a4 100644
--- a/iw/filter_gui.py
+++ b/iw/filter_gui.py
@@ -357,7 +357,16 @@ class NotebookPage(object):
self.filteredModel.refilter()

def getNVisible(self):
- return self.ds.getNVisible()
+ retval = 0
+ iter = self.filteredModel.get_iter_first()
+
+ while iter:
+ if self.cb.visible(self.filteredModel, iter, self.treeView):
+ retval += 1
+
+ iter = self.filteredModel.iter_next(iter)
+
+ return retval

class FilterWindow(InstallWindow):
windowTitle = N_("Device Filter")
--
1.6.5.1

_______________________________________________
Anaconda-devel-list mailing list
Anaconda-devel-list@redhat.com
https://www.redhat.com/mailman/listinfo/anaconda-devel-list

@@ -357,7 +357,16 @@ class NotebookPage(object):
self.filteredModel.refilter()

def getNVisible(self):
- return self.ds.getNVisible()
+ retval = 0
+ iter = self.filteredModel.get_iter_first()
+
+ while iter:
+ if self.cb.visible(self.filteredModel, iter, self.treeView):
+ retval += 1
+
+ iter = self.filteredModel.iter_next(iter)
+
+ return retval

class FilterWindow(InstallWindow):
windowTitle = N_("Device Filter")


I can't really see the details, but it looks good to me.

_______________________________________________
Anaconda-devel-list mailing list
Anaconda-devel-list@redhat.com
https://www.redhat.com/mailman/listinfo/anaconda-devel-list