Fix moving folders to root, bug introduces when using row id's instead of folder names.

program/js/app.js

@@ -2476,41 +2476,36 @@ function rcube_webmail()
     {
     var row, folder;
     var reg = RegExp('['+RegExp.escape(this.env.delimiter)+']?[^'+RegExp.escape(this.env.delimiter)+']+$');
-    if (this.env.subscriptionrows[id] &&
+
-        (folder = this.env.subscriptionrows[id][0]) &&
+    if (this.drag_active && (row = document.getElementById(id)))
-        this.drag_active && this.check_droptarget(folder) &&
+      if (this.env.subscriptionrows[id] &&
-        (folder != this.env.folder.replace(reg, '')) &&
+          (folder = this.env.subscriptionrows[id][0]))
-        (!folder.match(new RegExp('^'+RegExp.escape(this.env.folder+this.env.delimiter)))) &&
-        (row = document.getElementById(id)))
-      if (find_in_array(this.env.defaultfolders, folder)>=0)
         {
-        if (this.env.folder.replace(reg, '')!='')
+        if (this.check_droptarget(folder) &&
+            (folder != this.env.folder.replace(reg, '')) &&
+            (!folder.match(new RegExp('^'+RegExp.escape(this.env.folder+this.env.delimiter)))))
           {
-          this.set_env('dstfolder', this.env.delimiter);
+          this.set_env('dstfolder', folder);
-          this.set_classname(this.subscription_list.frame, 'droptarget', true);
+          this.set_classname(row, 'droptarget', true);
           }
         }
-      else
+      else if (this.env.folder.match(new RegExp(RegExp.escape(this.env.delimiter))))
         {
-        this.set_env('dstfolder', folder);
+        this.set_env('dstfolder', this.env.delimiter);
-        this.set_classname(row, 'droptarget', true);
+        this.set_classname(this.subscription_list.frame, 'droptarget', true);
         }
     }
 
 
   this.unfocus_subscription = function(id)
     {
-    var row, folder;
+      var row;
-    if (this.env.subscriptionrows[id] &&
-        (folder = this.env.subscriptionrows[id][0]) &&
-        (row = document.getElementById(id)))
-      {
       this.set_env('dstfolder', null);
-      if (find_in_array(this.env.defaultfolders, folder)>=0)
+      if (this.env.subscriptionrows[id] &&
-        this.set_classname(this.subscription_list.frame, 'droptarget', false);
+          (row = document.getElementById(id)))
-      else
         this.set_classname(row, 'droptarget', false);
-      }
+      else
+        this.set_classname(this.subscription_list.frame, 'droptarget', false);
     }