From e78c68e90a0bb0f35b4f3c1e40107ff422951205 Mon Sep 17 00:00:00 2001 From: OmegaPhil Date: Wed, 12 Jul 2017 20:10:20 +0100 Subject: [PATCH 4/5] filename is not optional in TreeView._get_launcher_instances --- menulibre/MenulibreTreeview.py | 11 +++++------ 1 file changed, 5 insertions(+), 6 deletions(-) diff --git a/menulibre/MenulibreTreeview.py b/menulibre/MenulibreTreeview.py index cea7a2b..602a2c6 100644 --- a/menulibre/MenulibreTreeview.py +++ b/menulibre/MenulibreTreeview.py @@ -276,8 +276,7 @@ class Treeview(GObject.GObject): def update_launcher_instances(self, filename, row_data): """Update all same launchers with the new information.""" model, treeiter = self._get_selected_iter() - for instance in self._get_launcher_instances(model=model, - filename=filename): + for instance in self._get_launcher_instances(filename, model): for i in range(len(row_data)): model[instance][i] = row_data[i] @@ -452,7 +451,7 @@ class Treeview(GObject.GObject): row = model[treeiter] return treeview.row_expanded(row.path) - def _get_launcher_instances(self, model=None, parent=None, filename=None): + def _get_launcher_instances(self, filename, model=None, parent=None): """Return a list of all treeiters referencing this filename.""" if model is None: model, treeiter = self._get_selected_iter() @@ -463,12 +462,12 @@ class Treeview(GObject.GObject): if iter_filename == filename: treeiters.append(treeiter) if model.iter_has_child(treeiter): - treeiters += self._get_launcher_instances(model, treeiter, - filename) + treeiters += self._get_launcher_instances(filename, model, + treeiter) return treeiters def _get_n_launcher_instances(self, filename): - return len(self._get_launcher_instances(filename=filename)) + return len(self._get_launcher_instances(filename)) def _is_menu_locked(self): """Return True if menu editing is currently locked.""" -- 2.13.2