Remove non-archive mimetypes from krarc archive mime type list

Authored by sandsmark on Nov 19 2016, 12:14 PM.

Description

Remove non-archive mimetypes from krarc archive mime type list

Putting non-archive mimetypes in the list of archive mimetypes confuses
other applications using KIO, like Dolphin. Getting Krusader to open
normal files as archives should be done by linking them in the Krusader
Protocol settings.

BUG: 371765
REVIEW: 129434

Details

Thanks for fixing the krarc protocol!

I'd like to open a small discussion about adding the removed mimes in protocol settings of Krusader. I think individual users should not be required to manually add all the mimes to the protocol settings. So I feel we have 2 options:

  1. Add the mimes to default protocol settings.
  1. Hard code the mimes to e.g. KRarcHandler or KrServices static field and then check against it in code as well.

Personally I like 2) more because users with existing protocol settings won't notice. Also the mimes was hard-coded in protocol before anyway. If we agree on something, I'll gladly implement it:).