TypeError: MsgMoveMessageOrig1234 is undefined - error upon trying to (s)ave a message

[NewMexicoKid]

Issue type:

• Bug report

Version:

Version Information
Muttator: 1.4 (created: 2017/01/09 09:13:34)
Thunderbird: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:45.0) Gecko/20100101 Thunderbird/45.6.0

Description:

When executing a save ('s') on a mail message, the following error displays in the footer in white text with a red background:
TypeError: MsgMoveMessageOrig1234 is undefined

Prior to that, the normal muttator matching of folders shows as expected.

This occurred after upgrading from Thunderbird 38 to 45.6.

Expected behavior:

The mail message should have been saved to the folder selected.

Steps to reproduce:

1. Select mail message
2. s

[NewMexicoKid]

This problem has suddenly resolved itself! Suddenly the 's' key actually saved the message the way it used to do. Not sure if it's related, but this happened after I installed the Quick Folder Move extension (which, ironically, did not work; I think that shift-M was conflicting with some other keystroke definition).

[NewMexicoKid]

I spoke too soon. This morning the problem is back (how is this possible?).

[NewMexicoKid]

2017-03-20 update: bug still present in Thunderbird 45.8.0

Just an update: this bug is still present with Thunderbird 45.7.1

[NewMexicoKid]

2017-05-16 update: bug still present (grabbed the most current source and built it, no errors).
Interestingly, 'j' and 'k' keys now work. But 's' to save the message still fails with the 'TypeError: MsgMoveMessageOrig1234 is undefined' message.

Grepping for MsgMove, I found this code in muttator/content/mail.js (the only match in the whole code base):

_moveOrCopy: function (copy, destinationFolder, operateOnThread) {
    let folders = mail.getFolders(destinationFolder);
    if (folders.length > 1) {
        folders = mail.getExactFolder(destinationFolder);
    }
    if (folders.length == 0)
        return void liberator.echoerr("No matching folder for: " + destinationFolder);
    else if (folders.length > 1)
        return liberator.echoerr("More than one match for: " + destinationFolder);

    let count = gDBView.selection.count;
    if (!count)
        return void liberator.beep();

    (copy ? MsgCopyMessage : MsgMoveMessage)(folders[0]);
    setTimeout(function () {
        liberator.echomsg((copy ? "Copied " : "Moved ") + count + " message(s) " + " to: " + folders[0].prettyName);
    }, 100);
},

Not sure where the Orig1234 is coming from or how it relates to the above code snippet.

[NewMexicoKid]

2017-06-15 update:
With the latest Thunderbird versions (currently on 52.2.0), the jk keys don't work anymore for movement.
For now I have disabled muttator in my Thunderbird instance and have moved to using nostalgy for saving messages and to keyconfig for movement.