Bug 339807 - Save in FTP Folder reports existing File although there isn't
Summary: Save in FTP Folder reports existing File although there isn't
Status: CONFIRMED
Alias: None
Product: okular
Classification: Applications
Component: general (show other bugs)
Version: 20.11.80
Platform: Chakra Linux
: NOR normal
Target Milestone: ---
Assignee: Okular developers
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2014-10-09 09:00 UTC by Richard Llom
Modified: 2020-11-27 01:55 UTC (History)
3 users (show)

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Richard Llom 2014-10-09 09:00:24 UTC
I noticed that when one wants to save (or save copy as) a PDF from Okular under an FTP-path, Okular *always* reports an already existing file, ie.:
"The file "my.pdf" already exists. Do you wish to overwrite it?"
although there is not.

Since this happens in the file dialog, I thought this might be KIO related. However I wasn't able to reproduce this in Kwrite, so it seems to be Okular (only) related.

Reproducible: Always

Steps to Reproduce:
1. Open random pdf-file in Okular
2. Choose File -> Save as... (or Save Copy as...)
3. Navigate to random ftp folder (ftp://mypath/)
4. Hit Save

Actual Results:  
Warning dialog with overwrite file pops-up ("The file "my.pdf" already exists. Do you wish to overwrite it?"), although no such file exists there.

Expected Results:  
Just saving.
Comment 1 Albert Astals Cid 2014-10-09 19:51:03 UTC
I'm pretty sure that this is a KIO problem, i've talked with David before about this and he said that it happens because the ftp ioslave defaults to "yes a file exists" when asked if it does because some servers are just plain stupid.

I'm CC'ing him in case we do something wrong.

David, what we do is just
KUrl saveUrl = KFileDialog::getSaveUrl( KUrl("kfiledialog:///okular/" + url().fileName()),
                                            QString(), widget(), QString(),
                                            KFileDialog::ConfirmOverwrite )

and then

KIO::file_copy( srcUrl, saveUrl, -1, KIO::Overwrite );
Comment 2 Andrew Crouthamel 2018-11-12 02:53:47 UTC
Dear Bug Submitter,

This bug has been stagnant for a long time. Could you help us out and re-test if the bug is valid in the latest version? I am setting the status to NEEDSINFO pending your response, please change the Status back to REPORTED when you respond.

Thank you for helping us make KDE software even better for everyone!
Comment 3 Andrew Crouthamel 2018-11-21 04:40:34 UTC
Dear Bug Submitter,

This is a reminder that this bug has been stagnant for a long time. Could you help us out and re-test if the bug is valid in the latest version? This bug will be moved back to REPORTED Status for manual review later, which may take a while. If you are able to, please lend us a hand.

Thank you for helping us make KDE software even better for everyone!
Comment 4 Justin Zobel 2020-11-27 01:55:48 UTC
(In reply to Albert Astals Cid from comment #1)
> I'm pretty sure that this is a KIO problem, i've talked with David before
> about this and he said that it happens because the ftp ioslave defaults to
> "yes a file exists" when asked if it does because some servers are just
> plain stupid.
> 
> I'm CC'ing him in case we do something wrong.
> 
> David, what we do is just
> KUrl saveUrl = KFileDialog::getSaveUrl( KUrl("kfiledialog:///okular/" +
> url().fileName()),
>                                             QString(), widget(), QString(),
>                                             KFileDialog::ConfirmOverwrite )
> 
> and then
> 
> KIO::file_copy( srcUrl, saveUrl, -1, KIO::Overwrite );

This is still occurring in git master of okular, etc.