Launchpad has imported 7 comments from the remote bug at https://bugzilla.xfce.org/show_bug.cgi?id=13535.
If you reply to an imported comment from within Launchpad, your comment will be sent to the remote bug automatically. Read more about Launchpad's inter-bugtracker facilities at https://help.launchpad.net/InterBugTracking. ------------------------------------------------------------------------ On 2017-04-28T11:19:05+00:00 Kev Bowring wrote: Created attachment 7095 warning screenshot Empty folders after deleting and restoring from trash works without warning. Folders with contents are restored AND left in trash, with a warning about modifying. Have 1.6.11 with patch from 13481 installed locally. Reply at: https://bugs.launchpad.net/ubuntu/+source/thunar/+bug/1687075/comments/0 ------------------------------------------------------------------------ On 2017-04-28T19:54:22+00:00 Kev Bowring wrote: Retry - this option restores the folder to it's source, but respawns the warning dialogue Cancel - restores the folder to source Skip all/Skip - restores the folder to source Reply at: https://bugs.launchpad.net/ubuntu/+source/thunar/+bug/1687075/comments/6 ------------------------------------------------------------------------ On 2017-05-01T17:53:56+00:00 Vlado-chren wrote: Created attachment 7096 [PATCH] Allow GIO copy/delete fallback for file restore from Trash Error message showed on attached the picture is very probably set in gvfsbackendtrash.c:trash_backend_delete() if (!is_toplevel) g_set_error_literal (&error, G_IO_ERROR, G_IO_ERROR_PERMISSION_DENIED, _("Items in the trash may not be modified")); *Analysis* This happens when thunar restores file using schemes (e.g. trash:///trasheddir -> file:///restoredir), sets flag G_FILE_COPY_NO_FALLBACK_FOR_MOVE to g_file_move(), then g_file_move() "wants" to fallback to copy/delete because move between different mount points is not supported, and then thunar fallbacks to internal manual copy/delete file by file. Finally trying delete file which is not a top level file in trash:// is not permitted by gvfs (see above). *Solution* I don't know why this flag is set globally here for every file move a then thunar internally implements this copy/delete fallback feature. Hence in this patch, I removed flag G_FILE_COPY_NO_FALLBACK_FOR_MOVE for trash restore operation only. I would also think about adding G_FILE_COPY_ALL_METADATA later. It looks reasonable to me. In 'pcmanfm' (file manager), flags used in this operations are G_FILE_COPY_NOFOLLOW_SYMLINKS | G_FILE_COPY_ALL_METADATA. In thunar flags are (were) G_FILE_COPY_NOFOLLOW_SYMLINKS | G_FILE_COPY_NO_FALLBACK_FOR_MOVE. Reply at: https://bugs.launchpad.net/ubuntu/+source/thunar/+bug/1687075/comments/8 ------------------------------------------------------------------------ On 2018-02-22T20:41:13+00:00 Gitbot wrote: vc-01 referenced this bugreport in commit 92b29110322e27489709ce293b58455884576ca5 Restoring non-empty folders spawns a warning about modification/restores folder and leaves copy in Trash (bug #13535) https://git.xfce.org/xfce/thunar/commit?id=92b29110322e27489709ce293b58455884576ca5 Reply at: https://bugs.launchpad.net/ubuntu/+source/thunar/+bug/1687075/comments/9 ------------------------------------------------------------------------ On 2018-02-22T20:48:26+00:00 Alexxcons-x wrote: Thanks alot for report and patch! And sorry for the late reply ! @vc-01 As you suggested, I added G_FILE_COPY_ALL_METADATA. I as well think it is reasonable to keep all metadata when a file is moved ( AFAIK this flag anyhow only make sense if the copy + delete fallback is used, so before the patch it would have no effect ) Fixed now for thunar master, to be released as 1.8.0 Reply at: https://bugs.launchpad.net/ubuntu/+source/thunar/+bug/1687075/comments/10 ------------------------------------------------------------------------ On 2018-02-22T21:03:18+00:00 Gitbot wrote: vc-01 referenced this bugreport in commit ccc1ea458273eec2e238c34118e4460631349434 Restoring non-empty folders spawns a warning about modification/restores folder and leaves copy in Trash (bug #13535) https://git.xfce.org/xfce/thunar/commit?id=ccc1ea458273eec2e238c34118e4460631349434 Reply at: https://bugs.launchpad.net/ubuntu/+source/thunar/+bug/1687075/comments/11 ------------------------------------------------------------------------ On 2018-02-22T21:05:37+00:00 Alexxcons-x wrote: As well fixed for xfce-4.12 branch ( will be released in thunar 1.6.15 ) Reply at: https://bugs.launchpad.net/ubuntu/+source/thunar/+bug/1687075/comments/12 -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/1687075 Title: Restoring non-empty folders spawns warning - leaves copy behind To manage notifications about this bug go to: https://bugs.launchpad.net/thunar/+bug/1687075/+subscriptions -- ubuntu-bugs mailing list [email protected] https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
