[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-bugs-dist
Subject:    [Bug 64244] kioslave settings:// does not work correctly in tree
From:       Luke Sandell <sandell () freeshell ! org>
Date:       2004-05-18 1:11:33
Message-ID: 20040518011133.7636.qmail () ktown ! kde ! org
[Download RAW message or body]

------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.
      
http://bugs.kde.org/show_bug.cgi?id=64244      




------- Additional Comments From sandell freeshell org  2004-05-18 03:11 -------
One comment on the patch:

+    
+    // If the first entry is a directory, we have to go
+    // one dir up, to get the parent dir, because
+    // setFileName("") will not delete the last part
+    // of the path in that case.
+    if (dir.path().endsWith("/"))
+        dir = dir.upURL();
+    else        
     dir.setFileName( "" );

This is madness! All you need is

+ dir = dir.upURL();
- dir.setFileName( "" )
[prev in list] [next in list] [prev in thread] [next in thread]