Hi,
I found a typo in my patch which prevented remembering last used
directories being
remembered correctly. Also, 'is_media' variable have been changed to
'browse_reason'
which is the argument passed directly to util.browse_local() function so
please review.
Thanks,
Michal
On 06/23/2009 08:09 AM, Michal Novotny wrote:
Sorry, I attached wrong version, this is the right one...
Michal
On 06/23/2009 08:07 AM, Michal Novotny wrote:
Hi,
this is new version of my patch so please check.
Thanks,
Michal
...
------------------------------------------------------------------------
_______________________________________________
et-mgmt-tools mailing list
et-mgmt-tools@redhat.com
https://www.redhat.com/mailman/listinfo/et-mgmt-tools
# HG changeset patch
# User Michal Novotny <minov...@redhat.com>
# Date 1245935008 -7200
# Node ID 39b81ca60edc70066cc44cf760865c2d8f03ac8f
# Parent aa4f30fce78b6805c94c70729139bb326b023bcb
Fix typo in remember-paths and change 'is_media' to 'browse_reason'
This is the patch that fixes a bug introduced by my previous patch for
remembering last used paths in rev. aa4f30fce78b. Also, 'is_media'
variable is no longer used there and is replaced by 'browse_reason'
wherever applicable.
diff -r aa4f30fce78b -r 39b81ca60edc src/virtManager/addhardware.py
--- a/src/virtManager/addhardware.py Tue Jun 23 19:30:12 2009 -0400
+++ b/src/virtManager/addhardware.py Thu Jun 25 15:03:28 2009 +0200
@@ -678,9 +678,8 @@
def browse_storage_file_address(self, src, ignore=None):
textent = self.window.get_widget("storage-file-address")
- folder = self.config.get_default_image_dir(self.vm.get_connection())
filename = self._browse_file(_("Locate or Create New Storage File"),
- textent, folder=folder,
+ textent, folder=None,
confirm_overwrite=True)
if filename != None:
textent.set_text(filename)
@@ -699,17 +698,23 @@
if path:
textent.set_text(path)
+ if folder == None:
+ reason = self.config.CONFIG_DIR_IMAGE
+ else:
+ reason = None
+
conn = self.vm.get_connection()
if self.storage_browser == None:
- self.storage_browser = vmmStorageBrowser(self.config, conn, False)
+ self.storage_browser = vmmStorageBrowser(self.config, conn,
+ reason)
if self._browse_cb_id:
self.storage_browser.disconnect(self._browse_cb_id)
self._browse_cb_id = self.storage_browser.connect("storage-browse-finish", set_storage_cb)
self.storage_browser.local_args = { "dialog_name": dialog_name,
"confirm_func": confirm_func,
- "browse_reason":
- self.config.CONFIG_DIR_IMAGE }
+ "browse_reason": reason,
+ "start_folder": folder }
self.storage_browser.show(conn)
return None
diff -r aa4f30fce78b -r 39b81ca60edc src/virtManager/choosecd.py
--- a/src/virtManager/choosecd.py Tue Jun 23 19:30:12 2009 -0400
+++ b/src/virtManager/choosecd.py Thu Jun 25 15:03:28 2009 +0200
@@ -150,7 +150,7 @@
def _browse_file(self, dialog_name):
if self.storage_browser == None:
self.storage_browser = vmmStorageBrowser(self.config, self.conn,
- True)
+ self.config.CONFIG_DIR_MEDIA)
self.storage_browser.connect("storage-browse-finish",
self.set_storage_path)
self.storage_browser.local_args = { "dialog_name": dialog_name,
diff -r aa4f30fce78b -r 39b81ca60edc src/virtManager/create.py
--- a/src/virtManager/create.py Tue Jun 23 19:30:12 2009 -0400
+++ b/src/virtManager/create.py Thu Jun 25 15:03:28 2009 +0200
@@ -997,7 +997,7 @@
def browse_iso(self, ignore1=None, ignore2=None):
self._browse_file(_("Locate ISO Image"),
self.set_iso_storage_path,
- is_media=True)
+ browse_reason=self.config.CONFIG_DIR_MEDIA)
self.window.get_widget("install-local-box").activate()
def toggle_enable_storage(self, src):
@@ -1006,7 +1006,7 @@
def browse_storage(self, ignore1):
self._browse_file(_("Locate existing storage"),
self.set_disk_storage_path,
- is_media=False)
+ browse_reason=self.config.CONFIG_DIR_IMAGE)
def toggle_storage_select(self, src):
act = src.get_active()
@@ -1650,20 +1650,16 @@
logging.exception("Error detecting distro.")
self.detectedDistro = (None, None)
- def _browse_file(self, dialog_name, callback, folder=None, is_media=False):
+ def _browse_file(self, dialog_name, callback, folder=None, browse_reason=False):
if self.storage_browser == None:
self.storage_browser = vmmStorageBrowser(self.config, self.conn,
- is_media)
+ browse_reason)
self.storage_browser.connect("storage-browse-finish",
callback)
- if is_media:
- reason = self.config.CONFIG_DIR_MEDIA
- else:
- reason = self.config.CONFIG_DIR_IMAGE
self.storage_browser.local_args = { "dialog_name": dialog_name,
"start_folder": folder,
- "browse_reason": reason}
+ "browse_reason": browse_reason}
self.storage_browser.show(self.conn)
def show_help(self, ignore):
diff -r aa4f30fce78b -r 39b81ca60edc src/virtManager/storagebrowse.py
--- a/src/virtManager/storagebrowse.py Tue Jun 23 19:30:12 2009 -0400
+++ b/src/virtManager/storagebrowse.py Thu Jun 25 15:03:28 2009 +0200
@@ -38,7 +38,7 @@
gobject.TYPE_NONE, [str]),
}
- def __init__(self, config, conn, is_media=False):
+ def __init__(self, config, conn, browse_reason=False):
self.__gobject_init__()
self.window = gtk.glade.XML(config.get_glade_dir() + \
"/vmm-storage-browse.glade",
@@ -58,14 +58,9 @@
# Add Volume wizard
self.addvol = None
- if is_media:
- reason = self.config.CONFIG_DIR_MEDIA
- else:
- reason = self.config.CONFIG_DIR_IMAGE
-
# Arguments to pass to util.browse_local for local storage
self.local_args = {"dialog_name": _("Choose local storage"),
- "browse_reason": reason, }
+ "browse_reason": browse_reason }
self.window.signal_autoconnect({
"on_vmm_storage_browse_delete_event" : self.close,
diff -r aa4f30fce78b -r 39b81ca60edc src/virtManager/util.py
--- a/src/virtManager/util.py Tue Jun 23 19:30:12 2009 -0400
+++ b/src/virtManager/util.py Thu Jun 25 15:03:28 2009 +0200
@@ -158,7 +158,7 @@
fcdialog.destroy()
ret = None
- if ret and browse_reason and not ret.startwith("/dev"):
+ if ret and browse_reason and not ret.startswith("/dev"):
config.set_default_directory(os.path.dirname(ret), browse_reason)
return ret
_______________________________________________
et-mgmt-tools mailing list
et-mgmt-tools@redhat.com
https://www.redhat.com/mailman/listinfo/et-mgmt-tools
