Bug 274655 - Cannot open tar archive folder in new window
Summary: Cannot open tar archive folder in new window
Status: RESOLVED FIXED
Alias: None
Product: dolphin
Classification: Applications
Component: general (show other bugs)
Version: 16.12.2
Platform: Compiled Sources Linux
: NOR normal
Target Milestone: ---
Assignee: Peter Penz
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-06-01 03:23 UTC by Christoph Feck
Modified: 2012-07-18 18:41 UTC (History)
2 users (show)

See Also:
Latest Commit:
Version Fixed In: 4.9.0
Sentry Crash Report:


Attachments
Test archive (216 bytes, application/x-compressed-tar)
2011-06-01 03:23 UTC, Christoph Feck
Details
bugfix for bug 274655 (and bug 292822) (586 bytes, patch)
2012-07-15 18:54 UTC, Emmanuel Pescosta
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Christoph Feck 2011-06-01 03:23:09 UTC
Version:           unspecified (using Devel) 
OS:                Linux



Reproducible: Always

Steps to Reproduce:
1. open attached archive in Dolphin
2. enter Test directory, note the two folders A and B.
3. right-click on A and select "Open in New Window" in context menu


Actual Results:  
Error message appears, stating

    tar:/home/pepo/Test.tar.gz/Test/A is a folder, but a file was expected.

Expected Results:  
New dolphin window opens with contents of folder A.

Selecting "Open in New Tab" correctly shows the directory contents.
Comment 1 Christoph Feck 2011-06-01 03:23:43 UTC
Created attachment 60532 [details]
Test archive
Comment 2 Frank Reininghaus 2011-06-01 12:10:46 UTC
It's interesting though that right-clicking A and opening that folder in a new tab works.
Comment 3 Frank Reininghaus 2011-06-01 12:12:34 UTC
In Konqueror, both opening A in a new window and in a new tab work as expected.
Comment 4 Emmanuel Pescosta 2012-07-15 18:54:48 UTC
Created attachment 72540 [details]
bugfix for bug 274655 (and bug 292822)

This bugfix should also fix another bug https://bugs.kde.org/show_bug.cgi?id=292822
Comment 5 David Faure 2012-07-16 15:07:58 UTC
I believe the patch is not 100% correct, but that discussion has now moved to https://git.reviewboard.kde.org/r/105584/ :-)
Comment 6 David Faure 2012-07-18 18:36:48 UTC
Git commit bd963e1e9abd1f82b72824fcdf7118ecffdec756 by David Faure.
Committed on 18/07/2012 at 20:26.
Pushed by dfaure into branch 'KDE/4.8'.

Temp services don't have details, so assume they support KIO urls.

A KService created in memory from an exec line cannot possibly have
a Category or X-KDE-Protocols key.
Related: bug 292822

M  +3    -1    kio/kio/krun.cpp

http://commits.kde.org/kdelibs/bd963e1e9abd1f82b72824fcdf7118ecffdec756
Comment 7 David Faure 2012-07-18 18:41:40 UTC
Git commit bc8df3d5a1e3747a8971107bc17f593c506a6dc9 by David Faure.
Committed on 18/07/2012 at 20:40.
Pushed by dfaure into branch 'KDE/4.9'.

Fix open-in-new-window to work with remote URLs

FIXED-IN: 4.9.0
Related: bug 292822

M  +3    -3    dolphin/src/dolphinmainwindow.cpp

http://commits.kde.org/kde-baseapps/bc8df3d5a1e3747a8971107bc17f593c506a6dc9