if (dstFolder != srcFolder) {
   console.debug('check folder');
   // If the source folder doesn't support deleting messages, we
   // make archive a copy, not a move.
   MailServices.copy.CopyMessages(
     srcFolder, moveArray, dstFolder,
     srcFolder.canDeleteMessages, this, this.msgWindow, true
   );
 console.debug('after');

To sum up our findings from yesterday test on IRC: If you had a localfolder created with TB52 (with your addon) it continued to work in TB60. However if you create a new folder (with your addon) with TB60, it does not work until you flip him to maildir and back to mbox. There is a pref you can set in TB60.7 to allow flipping: mail.store_conversion_enabled You now need to find out, what that flipping does to see if any other value is flipped/set. Also, from my tests I did not observe lost emails. The emails where simply not moved, they remained in the original folder. Best regards, John > Am 24.05.2019 um 21:04 schrieb Christopher Leidigh <cleidigh@gmail.com>: > > John > > The latest version has been working fine since it's introduction for 60.0 and was also tested and working with TB67beta. > > Something introduced with 60.6.1 (I believe this was the point) changed how the local folder accounts are handled. > > It's very consistent in that any attempt to place messages into the additional local folder fails . There is one exception > > in that when archiving the archiving code can and does create the appropriate archive folder . This further makes me convinced > > that there is some new flag or property that has been introduced preventing messages to be moved to the folder. > > I have been trying to find the code but I'm currently unsuccessful. I am not set up to debug into the C++ code. > > Thanks > > Christopher > > >> On 5/24/2019 5:57 AM, John Bieling wrote: >> Hi Christopher, >> >> Does the older version 0.8.12 also show that behaviour? Could it be some5hing that you introduced in newer versions? >> >> Best regards, >> John >> >>> Am 24.05.2019 um 01:54 schrieb Christopher Leidigh <cleidigh@gmail.com>: >>> >>> >>> I have somewhat urgent problem with my adopted extension "LocalFolders". It's urgent >>> because it looks like it completely stopped working after 60.6.1 . It was working fine before. >>> the urgency is because I just got my second 1 star review but, the really bad thing is the second user >>> is talking about a scenario where he may have Lost a couple years of e-mails he was placing the local folder. >>> >>> I have tried debugging using daily as I cannot build old revisions. 68 exhibits the same behavior as >>> 60.6.1 and 60.7 >>> >>> The extension can successfully create new local folders as before. The problem is that one cannot place >>> any messages within any part of the folder. Copying or moving messages fail silently, no debug information. >>> I do get a failure message if I try to archive to one of the new local folders. It is also not possible >>> to save a draft. All this points to me like some new required flag has been added that makes the new local folder >>> unable to receive any messages. When debugging archiving I can see it must fail within copy messages. >>> I cannot debug into that. >>> >>> I think I will quickly have a lot of angry users especially if they're losing e-mail.escape >>> >>> I would really love help to avoid that! >>> >>> Thanks to anyone in advance... >>> Christopher >>> >>> feels here within processBatch after calling CopyMessages - this is when archiving >>> >>> if (dstFolder != srcFolder) { >>> console.debug('check folder'); >>> // If the source folder doesn't support deleting messages, we >>> // make archive a copy, not a move. >>> MailServices.copy.CopyMessages( >>> srcFolder, moveArray, dstFolder, >>> srcFolder.canDeleteMessages, this, this.msgWindow, true >>> ); >>> console.debug('after'); >>> >>> >>> _______________________________________________ >>> Maildev mailing list >>> Maildev@lists.thunderbird.net >>> http://lists.thunderbird.net/mailman/listinfo/maildev_lists.thunderbird.net