[PATCH] Fix installation path handling for CD-ROM and DVD.

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



If CD-ROM or DVD colum is an empty in installation procedure on virt-manager,
the installation stops and no error messages are shown.

This problem occurs because the method of getting an active text of the
ComboBox is wrong.

Without this fixes, follwoing messages are shown
(N.B. date has japanese code, please skip it)
==============
[月, 17  3月 2008 09:53:40 virt-manager 16808] ERROR (virt-manager:132) Traceback (most
recent call last):
  File "/usr/share/virt-manager/virtManager/create.py", line 294, in forward
    if(self.validate(notebook.get_current_page()) != True):
  File "/usr/share/virt-manager/virtManager/create.py", line 799, in validate
    src = self.get_config_install_source()
  File "/usr/share/virt-manager/virtManager/create.py", line 345, in get_config_install_source
    return model.get_value(cd.get_active_iter(), 0)
TypeError: iter must be a GtkTreeIter
None
==============

Thanks
Signed-off-by: Hiroyuki Kaguchi <fj7025cf@xxxxxxxxxxxxxxxxx>
diff -r 569fbd93ef88 src/virtManager/create.py
--- a/src/virtManager/create.py	Wed Mar 19 15:39:12 2008 -0400
+++ b/src/virtManager/create.py	Fri Mar 21 17:08:30 2008 +0900
@@ -365,9 +365,7 @@ class vmmCreate(gobject.GObject):
             if self.window.get_widget("media-iso-image").get_active():
                 return self.window.get_widget("fv-iso-location").get_text()
             else:
-                cd = self.window.get_widget("cd-path")
-                model = cd.get_model()
-                return model.get_value(cd.get_active_iter(), 0)
+                return self.window.get_widget("cd-path").get_active_text()
         else:
             return "PXE"
 
_______________________________________________
et-mgmt-tools mailing list
et-mgmt-tools@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/et-mgmt-tools

[Index of Archives]     [Fedora Users]     [Fedora Legacy List]     [Fedora Maintainers]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]

  Powered by Linux