maildev@lists.thunderbird.net

Thunderbird email developers

View all threads

Help us! Major extension failure w 60.6.1+ - users using e-mail

CL
Christopher Leidigh
Thu, May 23, 2019 11:54 PM

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');

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');
GL
Geoff Lankow
Fri, May 24, 2019 12:12 AM

If it helps, I don't see anything likely that landed between 60.6.1 and
60.7.0:

https://hg.mozilla.org/releases/comm-esr60/pushloghtml?fromchange=THUNDERBIRD_60_6_1_RELEASE&tochange=THUNDERBIRD_60_7_0_RELEASE

GL

On 24/05/19 11:54, Christopher Leidigh wrote:

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

If it helps, I don't see anything likely that landed between 60.6.1 and 60.7.0: https://hg.mozilla.org/releases/comm-esr60/pushloghtml?fromchange=THUNDERBIRD_60_6_1_RELEASE&tochange=THUNDERBIRD_60_7_0_RELEASE GL On 24/05/19 11:54, Christopher Leidigh wrote: > > 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 >
BB
Ben Bucksch
Fri, May 24, 2019 2:25 PM

Did you try to compile a debug build and step through the C++ code?

stopped working after 60.6.1

Do you mean it still worked in 60.6.1, or do you mean it stopped working
after 60.6.1, i.e. 60.6.1 is already broken? Given the timing, I presume
the first, and Geoff also understood it like that, but just making sure.

(Regarding the reviews: reviews that are actually bug reports are unfair
to you, and they can be removed. Personally, I miss a possibility to
reply to a reviewer by email privately to ask for further information.)

Christopher Leidigh wrote on 24.05.19 01:54:

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

Did you try to compile a debug build and step through the C++ code? > stopped working after 60.6.1 Do you mean it still worked in 60.6.1, or do you mean it stopped working after 60.6.1, i.e. 60.6.1 is already broken? Given the timing, I presume the first, and Geoff also understood it like that, but just making sure. (Regarding the reviews: reviews that are actually bug reports are unfair to you, and they can be removed. Personally, I miss a possibility to reply to a reviewer by email privately to ask for further information.) Christopher Leidigh wrote on 24.05.19 01:54: > > 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 >