Bug 398534 - Can't move a file from one desktop to another
Summary: Can't move a file from one desktop to another
Status: RESOLVED FIXED
Alias: None
Product: plasmashell
Classification: Plasma
Component: Desktop icons & Folder View widget (other bugs)
Version First Reported In: 5.12.6
Platform: Other Linux
: NOR normal
Target Milestone: 1.0
Assignee: Eike Hein
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2018-09-12 09:09 UTC by Oleg Solovyov
Modified: 2020-01-23 18:18 UTC (History)
3 users (show)

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


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Oleg Solovyov 2018-09-12 09:09:28 UTC
I have dual monitor set.
After I create a file on my left desktop I try to move it to the right desktop.

Result: KDE asks me what i want: copy file, move it or create a link
When I choose "move" I see a "file exists, rename it" prompt. When I leave filename unchanged and hit an "overwrite" button the prompt appears again
Comment 1 Kai Uwe Broulik 2018-09-12 09:39:15 UTC
Did you configure both of them to show "Desktop folder" (the top option in "Location") settings or did you set one of them to "Show location: Desktop" dropdown?
Comment 2 Oleg Solovyov 2018-09-12 10:30:27 UTC
(In reply to Kai Uwe Broulik from comment #1)
> Did you configure both of them to show "Desktop folder" (the top option in
> "Location") settings or did you set one of them to "Show location: Desktop"
> dropdown?

I "set" both of them to "Folder View: ~/Desktop"
afaik, it's default now
Comment 3 Oleg Solovyov 2018-09-12 10:34:59 UTC
I found an issue so far that can be relater to this problem:

KDE asks to rename a file because I'm trying to move it from file:///home/mcpain/Desktop/filename to file:///home/mcpain/Desktop/./filename

URLs are different but both paths are pointing to the same dir
Comment 4 Kai Uwe Broulik 2018-09-12 11:17:06 UTC
Does it work when you set both to "desktop:/" instead of "~/Desktop"?
Does it exhibit the same behavior (with the superfluous dot) when both are set to any other same path, e.g. set both to ~/Documents?
Comment 5 Oleg Solovyov 2018-09-12 11:26:03 UTC
(In reply to Kai Uwe Broulik from comment #4)
> Both are "Show the Desktop folder" in Location tab (I looked at the wrong place)

> Does it work when you set both to "desktop:/" instead of "~/Desktop"?
Not sure about difference between them and how to do it

> Does it exhibit the same behavior (with the superfluous dot) when both are
> set to any other same path, e.g. set both to ~/Documents?
I set both to file://home/mcpain/Documents and it's working fine now

I have no idea how to fix it so far
Comment 6 Kai Uwe Broulik 2018-09-12 11:27:44 UTC
> Not sure about difference between them and how to do it

desktop:/ is "Show desktop folder, top radio button"
~/Desktop is "Show place: [Desktop]" dropdown (removed in 5.14 to avoid this confusion)

I have seen this issue in the past but it might have been fixed in 5.13 or some newer KDE Frameworks version.
Comment 7 Oleg Solovyov 2018-09-12 12:29:07 UTC
(In reply to Kai Uwe Broulik from comment #6)
> I have seen this issue in the past but it might have been fixed in 5.13 or
> some newer KDE Frameworks version.

Using KDE Frameworks 5.49 now, but KDE Plasma 5.12.6

Tested it, seems fixed in 5.13. Can you help me to find out which commit fixes it?
Comment 8 Kai Uwe Broulik 2018-09-12 13:17:30 UTC
Thanks for reporting back. I'm not sure, actually, there have been quite some changes and improvements to the Screen Mapper in plasma-desktop, which is probably why it was deemed too invasive for backporting to LTS. If you could bisect plasma-desktop/containments/plugin/folder that would be very helpful.
Comment 9 Oleg Solovyov 2018-09-14 09:29:20 UTC
(In reply to Kai Uwe Broulik from comment #8)
> Thanks for reporting back. I'm not sure, actually, there have been quite
> some changes and improvements to the Screen Mapper in plasma-desktop, which
> is probably why it was deemed too invasive for backporting to LTS. If you
> could bisect plasma-desktop/containments/plugin/folder that would be very
> helpful.

it was commit cab9078eaf21f27c034c93ac94dbbfed8c228f7c when this issue has been fixed
Comment 10 Kai Uwe Broulik 2018-09-14 14:57:35 UTC
I just successfully applied that patch to 5.12 branch, can you try and see if that fixes the issue for you? To my surprise this commit doesn't have any dependencies on any of the screen mapper changes done in Plasma 5.13.
Comment 11 Oleg Solovyov 2018-09-14 15:03:33 UTC
(In reply to Kai Uwe Broulik from comment #10)
> I just successfully applied that patch to 5.12 branch, can you try and see
> if that fixes the issue for you? To my surprise this commit doesn't have any
> dependencies on any of the screen mapper changes done in Plasma 5.13.

Tried already, it works, but there are some other patches applied on my 5.12.6 version so I'm not sure whether it works on "clean" 5.12.6 or not, mine is "dirty"
Comment 12 Kai Uwe Broulik 2018-09-14 15:04:23 UTC
I applied it on clean 5.12 branch, ok, I'll suggest backporting this then
Comment 13 Oleg Solovyov 2018-10-08 08:17:51 UTC
Is it backported?
Comment 14 Christoph Feck 2018-10-30 12:15:26 UTC
What is the status of this ticket? Recent comments suggests there was a fix which can be backported to 5.12 branch.
Comment 15 Nate Graham 2020-01-23 18:18:53 UTC
Already fixed. Given the proximity of the next LTS release (next month) I'm not sure it's worth it to backport the fix for the last one.