In D27181#676685, @meven wrote:

I am not sure we can use a nickname as author because of licensing.

Hello all,
Is there someone to commit this change?
Papoteur

In D27181#667237, @rthomsen wrote:

In D27181#666927, @papoteur wrote:

No, I don't have commit credentials. Go for myself.

Can you provide full name for commit message or should I use your nickname? And can you provide email-address?

In D27181#672643, @bugseforuns wrote:

In D27181#672572, @papoteur wrote:

In D27181#672563, @bugseforuns wrote:

My screenshot is showing preview of the compressed file (source) as "destination".

Hi Patrick,
Is it different with the original ark?

Ark 20.04.1 from Arch repos shows the thumbnail of source file twice.

In D27181#672643, @bugseforuns wrote:

Ark 20.04.1 from Arch repos shows the thumbnail of source file twice.

In D27181#672563, @bugseforuns wrote:

My screenshot is showing preview of the compressed file (source) as "destination".

No, I don't have commit credentials. Go for myself.

And now?

Should be OK now.

This version modifies the function signature of OverwriteQuery to pass directly data from source file.
Call to the function are adapted in the whole repo. I have tested it with extraction of one single file and a complete archive. Thus I have tested only the call from libarchiveplugin, line 306. I don't know how to test other calls.

This new patch provides data from source by calling setSourceData.
However, from my tests, it is never called.

In D27181#645642, @meven wrote:

ping @papoteur

In D27181#608097, @meven wrote:

Btw, You can mark the comment you treated as "Done" in phabricator, it it eases the review work.

Hello,
This patch has corrections for comments.

See D27181 which keep rename dialog but solve the problem it had.