Summary: | info:/topic is not the same as info:/topic/ | ||
---|---|---|---|
Product: | [Frameworks and Libraries] kio | Reporter: | Arne Schmitz <arne.schmitz> |
Component: | info | Assignee: | Matthias Hoelzer-Kluepfel <hoelzer> |
Status: | RESOLVED FIXED | ||
Severity: | wishlist | CC: | nicolasg |
Priority: | NOR | ||
Version: | unspecified | ||
Target Milestone: | --- | ||
Platform: | unspecified | ||
OS: | Linux | ||
Latest Commit: | Version Fixed In: |
Description
Arne Schmitz
2005-04-28 20:42:58 UTC
Still the case. 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 ); 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 ); |