Bug 422226 - Status bar says "Unknown size" when I open an archive (tar.gz, zip and 7z formats) as folder
Summary: Status bar says "Unknown size" when I open an archive (tar.gz, zip and 7z for...
Status: RESOLVED FIXED
Alias: None
Product: kio-extras
Classification: Frameworks and Libraries
Component: Archive (show other bugs)
Version: unspecified
Platform: Neon Linux
: NOR normal
Target Milestone: ---
Assignee: Plasma Bugs List
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2020-05-29 13:28 UTC by Patrick Silva
Modified: 2020-06-08 05:49 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In: 20.08


Attachments
screenshot (59.06 KB, image/png)
2020-05-29 13:28 UTC, Patrick Silva
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Patrick Silva 2020-05-29 13:28:11 UTC
Created attachment 128908 [details]
screenshot

SUMMARY
This issue was previously mentioned in bug 420027 comment 6.

STEPS TO REPRODUCE
1. enable "Open archives as folder" in "Navigation" section of Dolphin settings
2. open a tar.gz, zip or 7z archive with Dolphin
3. 

OBSERVED RESULT
As we can see in the attached screenshot, status bar says "Unknown size".

EXPECTED RESULT
I'm not sure what is expeced here (free space in current file system?
should free space info be omitted?), but certainly it's not "Unknown size".

SOFTWARE/OS VERSIONS
Operating System: KDE neon Unstable Edition
KDE Plasma Version: 5.19.80
KDE Frameworks Version: 5.71.0
Qt Version: 5.14.2
Comment 1 Méven Car 2020-05-29 13:53:18 UTC
archive ioslave should implement SlaveBase::GetFileSystemFreeSpace 
Similarly to desktop:/
https://invent.kde.org/plasma/plasma-workspace/-/merge_requests/19/
Comment 2 Méven Car 2020-06-08 05:49:49 UTC
Git commit bf4388c75525cffd150a51e6c4e0455f60db520f by Méven Car.
Committed on 08/06/2020 at 05:49.
Pushed by meven into branch 'master'.

Status bar: Hide the space free bar when size is unknown

Summary:
Related: bug 420027
FIXED-IN: 20.08

Test Plan: {F8312398}

Reviewers: #dolphin, ngraham, broulik, #vdg, elvisangelaccio

Reviewed By: #dolphin, ngraham, #vdg, elvisangelaccio

Subscribers: elvisangelaccio, kfm-devel

Tags: #dolphin

Differential Revision: https://phabricator.kde.org/D29213

M  +4    -2    src/statusbar/dolphinstatusbar.cpp
M  +3    -1    src/statusbar/spaceinfoobserver.cpp
M  +1    -0    src/statusbar/spaceinfoobserver.h
M  +45   -18   src/statusbar/statusbarspaceinfo.cpp
M  +6    -0    src/statusbar/statusbarspaceinfo.h

https://invent.kde.org/system/dolphin/commit/bf4388c75525cffd150a51e6c4e0455f60db520f