[Uml-devel] KDE/kdesdk/umbrello/umbrello

[Sharan Rao]

SVN commit 695283 by sharan:

If we have the checkbox for "Automatically add extension ..", then why do we check for the existence of the 
extension in the file name ? When the checkbox is in the "off" state, I guess, it should allow to save the file with 
any extension.

I guess this also solves BUG #146061. Please verify :)


 M  +8 -8      uml.cpp  


--- trunk/KDE/kdesdk/umbrello/umbrello/uml.cpp #695282:695283
@@ -917,14 +917,14 @@
             cont = false;
         else {
             // now check that we have a file extension; standard will be plain xmi
-            QString file = url.path( KUrl::RemoveTrailingSlash );
-            QFileInfo info(file);
-            ext = info.extension();
-            if (ext != "xmi" && ext != "xmi.tgz" && ext != "xmi.tar.bz2")
-            {
-                url.setFileName(url.fileName() + ".xmi");
-                ext = "xmi";
-            }
+            //QString file = url.path( KUrl::RemoveTrailingSlash );
+            //QFileInfo info(file);
+            //ext = info.completeSuffix();
+            //if (ext != "xmi" && ext != "xmi.tgz" && ext != "xmi.tar.bz2")
+            //{
+            //    url.setFileName(url.fileName() + ".xmi");
+            //    ext = "xmi";
+            //}
             QDir d = url.path( KUrl::RemoveTrailingSlash );
 
             if(QFile::exists(d.path())) {