New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Moving multiple messages via quickMove - fails the filter assistant #385
Comments
The reason this happened - the underlying copy / move functions by THunderbird failed if the message came from differing folders (and you can gather emails from different folders before you actually carry out the quickMove operation). So I broke up the mails into portions with equal source folders (rather than having one copy operation for each message), and then copied the processed messages to a "results" Array, like this: messageIdList.push(origIds); // <= successfully moved / copied
QuickFolders.CopyListener.OnStopCopy(status); I assumed the Array "oridId" would be concatenated to the messageIdList Array, instead it is nested as a member. I need to investigate if there is any unwanted fallout when I fix this (apart from fixing the message issue). To fix, I should do this instead: messageIdList.push(...origIds); |
Here are previews for both versions of Thunderbird: ESR102: QuickFolders-mx-5.17.1pre6.zip ESR115: QuickFolders-mx-6.1pre122.zip To test the versions above, download the zip file, drag the file into Thunderbird Add-ons Manager, do not extract contents or if won't install. |
Fixed in 6.1 (published 08/08/2023) and 5.17.1 (published 07/08/2023) |
It does not work here with neither the 122 pre-version nor the 6.1.1 version. When I move an email with the quickjump function (dropping it on the quickjump button and entering the name of a folder) the message is moved but the filter dialog does not pop up. Also, I still cannot create new folders with the quickjump function (old bug from me, I did not use it as it did never work here, but tried again and found it still does not work). |
Hmm it really looks like a recent change in 6.1 or Thunderbird must have broken it again... Possibly changes in #387. Let me investigate... |
This one works for me (here I am testing on an IMAP folder): Using Shift+M to trigger quickMove. Entering a new subfolder name in the search box (using the / syntax) and then clicking the create new subfolder menu item (1). This pops up the (2) New Folder dialog (is it shown on your system): then I can go to the folder and it takes a few seconds for my server to sync the new folder after which the emails are shown in the new folder: If there is a problem with this, let's file a new bug please. For the filter assistant, please see my following comment. |
So it seems like there is something different in the message database, I was using the target FOlder (the folder that the email was moved to) in order to retrieve the current messageHdr but that doesn't appear to work, so I fixed that on the quickFilters side. Can you check it with this version please? quickFilters-wx-6.1.2pre31.zip @PotatoCarl I might quickly release a maintenance version from that if it works for you. |
Great, works again. Thank you. |
thanks for the quick testing!! |
AS this is issue was closed on the quickFilters side, I am closing the issue here too. |
Apparently quickMove fails triggering the filter assistant when moving more than one email using the quickMove.
I found this after following up on issue #354 which prevents displaying the number of message moved. The problem is that the function
QuickFolders.Util.moveMessages()
returns an array that doesn't just contain a flat list of messages but is an Array of arrays. Needs to be fixed and tested ASAP.
The text was updated successfully, but these errors were encountered: