Bug 268163

Summary:	Do not offer to compression service menu on an already compressed archive
Product:	[Applications] ark	Reporter:	whatmeurgent <whatmeurgent>
Component:	general	Assignee:	Elvis Angelaccio <elvis.angelaccio>
Status:	RESOLVED FIXED
Severity:	normal	CC:	g.batra24, peter.penz19
Priority:	NOR
Version:	unspecified
Target Milestone:	---
Platform:	Ubuntu
OS:	Linux
Latest Commit:	http://commits.kde.org/ark/c369264d6c9ba2763b338f025c359c1b4c86f66e	Version Fixed In:	16.08.0
Sentry Crash Report:

Description whatmeurgent 2011-03-10 21:44:07 UTC

Version:           unspecified (using KDE 4.6.1) 
OS:                Linux

Why Compress a compressed archive?

Dolphin context menu for compressed files includes an action to ... compress the file


Reproducible: Always

Steps to Reproduce:
01] open dolphin
02] right click on a tar.gz file


Actual Results:  
there is context menu option to compress what is an already compressed archive (tar.gz)

Expected Results:  
No such option present; alternatively I suppose there could be a more general option "convert compression type" if that was the purpose of this menu option and it was just named vaguely.

Comment 1 Peter Penz 2011-03-10 22:21:02 UTC

This is in the scope of Ark: If a MIME-type is not sufficient for deciding whether a service-menu can be used, then Ark should ideally provide a KFileItemActionPlugin instead (see http://ppenz.blogspot.com/2010/11/improved-service-menus.html)

Comment 2 Raphael Kubo da Costa 2011-03-11 01:12:21 UTC

Confirming, thanks for the report.

Comment 3 Gagan Batra 2014-01-30 08:23:04 UTC

hi,
i would like to contribute to this bug.

Comment 4 Elvis Angelaccio 2016-06-05 17:53:35 UTC

Git commit c369264d6c9ba2763b338f025c359c1b4c86f66e by Elvis Angelaccio.
Committed on 05/06/2016 at 17:50.
Pushed by elvisangelaccio into branch 'master'.

Port service menus to KFileItemAction

The Compress and Extract service menus shown in Dolphin's context menu are now
provided by two new KFileItemAction plugins. This allows us to do a lot of
useful checks before actually showing the actions in the context menu.

We can also stop hardcoding the supported mimetypes for the Extract
menu. This is actually required in order to support the Extract menu when more
than one archive is selected. In this case Dolphin sends to the kfileitemaction
plugins the
generic 'application/octet-stream' mimetype, so we have to manually check
anyway the mimetype of any given URL (to be sure that Ark supports it).

This also paves the way to dynamically generate the service menus (e.g. based
on some kind of user preset).
Related: bug 189177, bug 348582
FIXED-IN: 16.08.0

Differential Revision: D1761

M  +22   -6    app/CMakeLists.txt
D  +0    -179  app/ark_addtoservicemenu.desktop
D  +0    -167  app/ark_servicemenu.desktop.cmake
A  +99   -0    app/compressfileitemaction.cpp     [License: GPL (v2+)]
A  +54   -0    app/compressfileitemaction.h     [License: GPL (v2+)]
A  +15   -0    app/compressfileitemaction.json
A  +103  -0    app/extractfileitemaction.cpp     [License: GPL (v2+)]
A  +54   -0    app/extractfileitemaction.h     [License: GPL (v2+)]
A  +13   -0    app/extractfileitemaction.json

http://commits.kde.org/ark/c369264d6c9ba2763b338f025c359c1b4c86f66e