diff options
Diffstat (limited to 'sb_builder/packages.py')
| -rw-r--r-- | sb_builder/packages.py | 16 |
1 files changed, 12 insertions, 4 deletions
diff --git a/sb_builder/packages.py b/sb_builder/packages.py index ab6f3ce..68de7c2 100644 --- a/sb_builder/packages.py +++ b/sb_builder/packages.py @@ -112,6 +112,16 @@ class Model(QAbstractTableModel): self.checked &= names self.endResetModel() + def boxes_changed(self): + """Only check states changed: dataChanged on the box column (count follows). + + Not layoutChanged: emitted without layoutAboutToBeChanged it leaves the + filtering proxy with stale persistent indexes, and Qt segfaults. + """ + if self.rows: + self.dataChanged.emit(self.index(0, 0), self.index(len(self.rows) - 1, 0), + [Qt.ItemDataRole.CheckStateRole]) + def rowCount(self, parent=QModelIndex()): return 0 if parent.isValid() else len(self.rows) @@ -337,13 +347,11 @@ class PackagesTab(QWidget): def select_visible(self): for i in range(self.proxy.rowCount()): self.model.checked.add(self.proxy.index(i, 0).data(Qt.ItemDataRole.UserRole)["name"]) - self.model.layoutChanged.emit() - self.update_count() + self.model.boxes_changed() def clear(self): self.model.checked.clear() - self.model.layoutChanged.emit() - self.update_count() + self.model.boxes_changed() def targets(self): """Ticked packages, sorted; the highlighted row alone is not a target.""" |