Details
Compile and install KIO with D23490 and D23780 applied.
(If Krusader is installed, change krarc to sevenz in the URL bar if necessary.)
Test using Dolphin:
Enable "Open archives as folder" in Dolphin, then open and browse a 7z archive using the sevenz protocol.
Test using Gwenview:
View a 7z archive containing photos. Browse and view images inside the archive.
Diff Detail
- Repository
- R320 KIO Extras
- Lint
Automatic diff as part of commit; lint not applicable. - Unit
Automatic diff as part of commit; unit tests not applicable.
Notes:
- I am not sure whether it is needed to include k7zip.h in archive/kio_archivebase.cpp.
- It can take some time to load the 7z archive using this new protocol.
Yes, I have tested it.
| archive/sevenz.protocol | ||
|---|---|---|
| 4 | Was it possible to use 7zip as protocol, it would probably be more user intuitive. | |
There is a weird bug if I name the protocol as "7z" or "7zip":
- If I double click on the 7z file, Dolphin gives the "invalid protocol" error.
- If I try to manually enter the URL in Dolphin, i.e. 7z:/tmp/bar.7z or 7zip:/tmp/bar.7z, it does not work too - it is somehow being treated as a host (hostname) instead of a protocol.
I suspect that a KIO protocol should not have a name that starts with a number, otherwise there will be bugs.
7zip:/ is in fact not correct because this would make the url scheme 7zip, but a url scheme must begin with a letter .
That's too bad URL aren't not more flexible.
Yeah, I agree that the name is not 100% ideal, but honestly that's an implementation detail that shouldn't block this. The code and functionality look sound!