Re: code check needed

Angelo Naselli <[email protected]>
Newsgroups gmane.comp.kde.gwenview
Message-ID <[email protected]>
> Hi trying to fix http://qa.mandriva.com/show_bug.cgi?id=31395

Aurelien can you check if the attached patch is ok for you, 
so that i can commit it?

TIA,
	Angelo

-------------------------------------------------------------------------
This SF.net email is sponsored by DB2 Express
Download DB2 Express C - the FREE version of DB2 express and take
control of your XML. No limits. Just data. Click to get it now.
http://sourceforge.net/powerbar/db2/

_______________________________________________
Gwenview-general mailing list
Gwenview-general-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org
https://lists.sourceforge.net/lists/listinfo/gwenview-general
rename.bug_m31395 (text/x-diff, 955 B)
Index: mainwindow.h
===================================================================
--- mainwindow.h	(revisione 680290)
+++ mainwindow.h	(copia locale)
@@ -148,6 +148,7 @@
 	void makeDir();
 	void goHome();
 	void renameFile();
+	void renamed(const QString&);
 	void copyFiles();
 	void moveFiles();
 	void linkFiles();
Index: mainwindow.cpp
===================================================================
--- mainwindow.cpp	(revisione 680290)
+++ mainwindow.cpp	(copia locale)
@@ -256,7 +256,12 @@
 	}
 }
 
-
+void MainWindow::renamed(const QString& fileName) {
+	KURL url = mDocument->url();
+	url.setFileName(fileName);
+	mDocument->setURL(url);
+}
+	
 void MainWindow::slotDirURLChanged(const KURL& dirURL) {
 	LOG(dirURL.prettyURL(0,KURL::StripFileProtocol));
 	
@@ -350,7 +355,7 @@
 	} else {
 		url = mDocument->url();
 	}
-	FileOperation::rename(url, this);
+	FileOperation::rename(url, this, this, SLOT(renamed(const QString &)));
 }
signature.asc (application/pgp-signature, 189 B)
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.7 (GNU/Linux)

iD8DBQBGgCouqEs9DA4DquARAiKBAJ4ppcJvP4MlTWpY9lM8+LrrLwqgKwCdHBhP
HPuuXCYYMy7bdAgZbom2ulI=
=VobD
-----END PGP SIGNATURE-----
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.