Bug 104728

Summary:	info:/topic is not the same as info:/topic/
Product:	[Unmaintained] kio	Reporter:	Arne Schmitz <arne.schmitz>
Component:	info	Assignee:	Matthias Hoelzer-Kluepfel <hoelzer>
Status:	RESOLVED FIXED
Severity:	wishlist	CC:	nicolasg
Priority:	NOR
Version First Reported In:	unspecified
Target Milestone:	---
Platform:	unspecified
OS:	Linux
Latest Commit:		Version Fixed/Implemented In:
Sentry Crash Report:

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 );