Merge lp:~jeremywootten/pantheon-files/fix-1456202-multiple-operations-one-view into lp:~elementary-apps/pantheon-files/trunk

[Jeremy Wootten]

remove the blocks on carrying out more than one copy/cut/paste operation or trash/delete operation on the same view at the same time

This branch removes the blocks on carrying out more than one copy/cut/paste operation or trash/delete operation on the same view at the same time.

While these blocks were presumably originally put in place in the interests of stability, removing has not immediately revealed any regressions.

TO TEST (E.G.):

1: Carry out a slow file operation e.g. copying a fairly large folder in place on a remote filesystem or USB stick.

2: Perform another similar copy paste operation on another (or the same) fairly large folder in the same view before the first operation completes. The destination must be the same open view.

In trunk this is not possible. In this branch, both operations run and complete successfully.

Note: With these blocks removed it is possible to do things like deleting the folder which is being copied while the copy is in progress. However, this does not cause a crash - it just causes the copy operation to fail.

This branch needs careful testing with a range of simultaneous operations on the same view to ensure stability is maintained.

[Dieter Debast (ddieter)]

I can't find any problems on this branch, unless I forgot to test something (but hopefully not). I tried a variety of copying, cutting, deleting and renaming and it doesn't crash or behave in a strange way.

review: Approve

[Adam Bieńkowski (donadigo)]

The code looks good to me.

review: Approve (code)

[Adam Bieńkowski (donadigo)]

I can confirm that this is working properly, both operations if long enough are showing in the copy dialog and they finish. I'm merging this.

=== modified file 'src/View/AbstractDirectoryView.vala'
--- src/View/AbstractDirectoryView.vala	2017-02-15 18:54:50 +0000
+++ src/View/AbstractDirectoryView.vala	2017-03-08 17:38:37 +0000
@@ -225,12 +225,8 @@
         private GLib.AppInfo default_app;
         private Gtk.TreePath? hover_path = null;
 
-        private bool can_trash_or_delete = true;
-
         /* Rapid keyboard paste support */
-        protected bool pasting_files = false;
         protected bool select_added_files = false;
-        private HashTable? pasted_files = null;
 
         public bool renaming {get; protected set; default = false;}
 
@@ -972,7 +968,6 @@
                 return;
             }
 
-            view.can_trash_or_delete = true;
             view.unblock_directory_monitor ();
         }
 
@@ -990,12 +985,8 @@
          * when using keybindings. So we remember if the current selection
          * was already removed (but the view doesn't know about it yet).
          */
-            if (!can_trash_or_delete)
-                return;
-
             unowned GLib.List<GOF.File> selection = get_selected_files_for_transfer ();
             if (selection != null) {
-                can_trash_or_delete = false;
                 trash_or_delete_files (selection, true, delete_immediately);
             }
         }
@@ -1222,16 +1213,14 @@
                 return;
             }
 
-            view.pasting_files = false;
-            if (uris == null || uris.size () == 0)
+            if (uris == null || uris.size () == 0) {
                 return;
-
-            view.pasted_files = uris;
+            }
 
             Idle.add (() => {
                 /* Select the most recently pasted files */
                 GLib.List<GLib.File> pasted_files_list = null;
-                view.pasted_files.foreach ((k, v) => {
+                uris.foreach ((k, v) => {
                     if (k is GLib.File)
                         pasted_files_list.prepend (k as File);
                 });
@@ -1242,9 +1231,6 @@
         }
 
         private void on_common_action_paste_into (GLib.SimpleAction action, GLib.Variant? param) {
-            if (pasting_files)
-                return;
-
             var file = get_files_for_action ().nth_data (0);
 
             if (file != null && clipboard.can_paste) {
@@ -1258,10 +1244,8 @@
 
                 if (target.has_uri_scheme ("trash")) {
                     /* Pasting files into trash is equivalent to trash or delete action */
-                    pasting_files = false;
                     call_back = (GLib.Callback)after_trash_or_delete;
                 } else {
-                    pasting_files = true;
                     /* callback takes care of selecting pasted files */
                     call_back = (GLib.Callback)after_pasting_files;
                 }