Need help with a bug about roundcube-contextmenu

[johndoh]

I think this is related to roundcube/roundcubemail#6145. The contextmenu plugin enables commands when its opened and then returns them to their original state so as not to mess up the main UI.

The move and copy functions are unique in that they have a 2 stage execution and the actual command execution isn't until stage 2 which is after contextmenu is closed. The state of an command is checked before any JS events are triggered so I think the solution can only be found inside app.js

I'm not sure what is best though as its really only useful for plugins rather than anything in the core. The command could be temp enabled like it is inside contextmenu:

       uids = this.env.uid ? [this.env.uid] : this.message_list.get_selection();
-      return this.folder_selector(event, function(folder) {
-        ref.move_messages(folder, null, uids);
+      return this.folder_selector(event, function(folder, obj) {
+        var prev_command = ref.commands['move'];
+        rcmail.enable_command('move', true);
+        ref.command('move', {id: folder, uids: uids}, obj, event);
+        rcmail.enable_command('move', prev_command);
       });

Or may be something added to rcmail.command() to make it pass the enabled check.

[alecpl]

Fixed with roundcube/roundcubemail@cb8c07843bd63f4c98. Thanks.