Bug 104728 - info:/topic is not the same as info:/topic/
Summary: info:/topic is not the same as info:/topic/
Status: RESOLVED FIXED
Alias: None
Product: kio
Classification: Frameworks and Libraries
Component: info (show other bugs)
Version: unspecified
Platform: unspecified Linux
: NOR wishlist
Target Milestone: ---
Assignee: Matthias Hoelzer-Kluepfel
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2005-04-28 20:42 UTC by Arne Schmitz
Modified: 2005-08-22 18:23 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Arne Schmitz 2005-04-28 20:42:58 UTC
Version:           Unbekannt (using KDE 3.4.0 Level "b" , SUSE 9.2 UNSUPPORTED)
Compiler:          gcc version 3.3.4 (pre 3.3.5 20040809)
OS:                Linux (i686) release 2.6.8-24.14-default

Normally one would use the URL info:/bison to look up the bison info page. But e.g. if your prior URL was info:/bison/Semantics and you hit the "Up" Icon in Konqueror, you get the URL info:/bison/, which will show nothing useful. So a trailing slash should be removed from info-URLs or be treated as optional.
Comment 1 Nicolas Goutte 2005-08-13 23:09:15 UTC
Still the case.
Comment 2 Nicolas Goutte 2005-08-22 18:21:37 UTC
SVN commit 452119 by goutte:

Re-direct if the URL has a trailing / character 
(as after using the "Up" button)
CCBUG:104728


 M  +12 -0     info.cc  


--- branches/KDE/3.5/kdebase/kioslave/info/info.cc #452118:452119
@@ -76,6 +76,18 @@
         return;
     }
 
+    if ( url.path().right(1) == "/" )
+    {
+        // Trailing / are not supported, so we need to remove them.
+        KURL newUrl( url );
+        QString newPath( url.path() );
+        newPath.truncate( newPath.length()-1 );
+        newUrl.setPath( newPath );
+        redirection( newUrl );
+        finished();
+        return;
+    }
+
     mimeType("text/html");
     // extract the path and node from url
     decodeURL( url );
Comment 3 Nicolas Goutte 2005-08-22 18:23:04 UTC
SVN commit 452121 by goutte:

Re-direct if the URL has a trailing / character
(Foreward port of revision 452119; might not compile)
BUG:104728


 M  +12 -0     info.cc  


--- trunk/KDE/kdebase/kioslave/info/info.cc #452120:452121
@@ -78,6 +78,18 @@
         return;
     }
 
+    if ( url.path().right(1) == "/" )
+    {
+        // Trailing / are not supported, so we need to remove them.
+        KURL newUrl( url );
+        QString newPath( url.path() );
+        newPath.chop( 1 );
+        newUrl.setPath( newPath );
+        redirection( newUrl );
+        finished();
+        return;
+    }
+
     mimeType("text/html");
     // extract the path and node from url
     decodeURL( url );