• src/sbbs3/email.cpp

    From Rob Swindell@VERT to Git commit to main/sbbs/master on Monday, February 15, 2021 01:08:03
    https://gitlab.synchro.net/main/sbbs/-/commit/5771d524d0d0374a74eea8d7
    Modified Files:
    src/sbbs3/email.cpp
    Log Message:
    Fix typo in previous commit (missing semicolon)

    ---
    Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
  • From Rob Swindell@VERT to Git commit to main/sbbs/master on Monday, February 15, 2021 19:31:37
    https://gitlab.synchro.net/main/sbbs/-/commit/7299e0004eb697bf0fcef033
    Modified Files:
    src/sbbs3/email.cpp
    Log Message:
    Fix typo in previous commit (missing semicolon)

    ---
    Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
  • From Rob Swindell@VERT to Git commit to main/sbbs/master on Saturday, September 25, 2021 00:28:57
    https://gitlab.synchro.net/main/sbbs/-/commit/aaa6fa7e9c801734368bb1b4
    Modified Files:
    src/sbbs3/email.cpp
    Log Message:
    Call smb_updatethread() from sbbs_t::email() to perform Keyop "magic"

    As discovered by Keyop and reported via IRC, when replying *to* a local mail box and *not* using the hard-coded email menu (e.g. when when using msglist.js), the original message's "Replied" attribute flag was not set. When replying to a netmail address (at least, Internet or FidoNet, unsure about QWKnetmail), the attribute would be set.

    This discrepancy was because this email() function doesn't use the "modern" smb_addmsg() function (which also calls smb_updatethread()), but add the message body text the old/manual way and never called smb_updatethread(). smb_updatethread() does the thread linkage stuff (which is usually unnoticed in the mail base) *and* sets the "Replied" attribute of the original message, if it's not already set.

    This function should be overhauled to use smb_addmsg(), but this 2-line change at least addresses this issue for now. There's probably other places (e.g. importing mail replies via QWK/REP packets) where this thread-linkage and "Replied" attribute flag setting is still not happening.

    ---
    Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net