[virt-tools-list] [PATCH v2 3/6] details: Rename the variable 'dev' to 'controller' in refresh_controller_page
Lin Ma
lma at suse.com
Thu Nov 23 09:32:47 UTC 2017
Use the more appropriate name for more correctly identifies controllers.
Signed-off-by: Lin Ma <lma at suse.com>
---
virtManager/details.py | 18 +++++++++---------
1 file changed, 9 insertions(+), 9 deletions(-)
diff --git a/virtManager/details.py b/virtManager/details.py
index a7b17c2b..e77fa859 100644
--- a/virtManager/details.py
+++ b/virtManager/details.py
@@ -2991,28 +2991,28 @@ class vmmDetails(vmmGObjectUI):
uiutil.set_list_selection(self.widget("watchdog-action"), action)
def refresh_controller_page(self):
- dev = self.get_hw_selection(HW_LIST_COL_DEVICE)
- if not dev:
+ controller = self.get_hw_selection(HW_LIST_COL_DEVICE)
+ if not controller:
return
can_remove = True
- if self.vm.get_xmlobj().os.is_x86() and dev.type == "usb":
+ if self.vm.get_xmlobj().os.is_x86() and controller.type == "usb":
can_remove = False
- if dev.type == "pci":
+ if controller.type == "pci":
can_remove = False
self.widget("config-remove").set_sensitive(can_remove)
- type_label = dev.pretty_desc()
+ type_label = controller.pretty_desc()
self.widget("controller-type").set_text(type_label)
combo = self.widget("controller-model")
- vmmAddHardware.populate_controller_model_combo(combo, dev.type)
- show_model = (dev.model or len(combo.get_model()) > 1)
- if dev.type == "pci":
+ vmmAddHardware.populate_controller_model_combo(combo, controller.type)
+ show_model = (controller.model or len(combo.get_model()) > 1)
+ if controller.type == "pci":
show_model = False
uiutil.set_grid_row_visible(combo, show_model)
uiutil.set_list_selection(self.widget("controller-model"),
- dev.model or None)
+ controller.model or None)
def refresh_filesystem_page(self):
dev = self.get_hw_selection(HW_LIST_COL_DEVICE)
--
2.14.0
More information about the virt-tools-list
mailing list