• DDMsgReader: Inactivity disconnection more consistent with Synchronet

    From Rob Swindell@VERT to GitLab note in main/sbbs on Saturday, January 25, 2025 21:31:06
    https://gitlab.synchro.net/main/sbbs/-/merge_requests/493#note_6298

    I don't think this actually achieves anything. It's like writing `x = x`. :-)

    ---
    þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net
  • From Rob Swindell@VERT to GitLab note in main/sbbs on Saturday, January 25, 2025 21:31:22
    https://gitlab.synchro.net/main/sbbs/-/merge_requests/493#note_6299

    Same here. Seems like a no-op.

    ---
    þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net
  • From Rob Swindell@VERT to GitLab note in main/sbbs on Saturday, January 25, 2025 21:32:48
    https://gitlab.synchro.net/main/sbbs/-/merge_requests/493#note_6300

    The more correct way to do this is to use `bbs.revert_text()`. And then you don't have to save the original string yourself. :-)

    ---
    þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net
  • From Rob Swindell@VERT to GitLab note in main/sbbs on Saturday, January 25, 2025 21:34:44
    https://gitlab.synchro.net/main/sbbs/-/merge_requests/493#note_6301

    Looks like you tried this method but had an issue?

    ---
    þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net
  • From Rob Swindell@VERT to GitLab note in main/sbbs on Saturday, January 25, 2025 21:34:45
    https://gitlab.synchro.net/main/sbbs/-/merge_requests/493#note_6302

    Some comments added

    ---
    þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net
  • From Eric Oulashin@VERT to GitLab note in main/sbbs on Saturday, January 25, 2025 21:49:17
    https://gitlab.synchro.net/main/sbbs/-/merge_requests/493#note_6303

    This was probably a result of a search & replace, and I forgot to remove this stuff. I had defined my own KEY_PAGE_UP and KEY_PAGE_DOWN a long time ago.

    ---
    þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net
  • From Eric Oulashin@VERT to GitLab note in main/sbbs on Saturday, January 25, 2025 21:49:44
    https://gitlab.synchro.net/main/sbbs/-/merge_requests/493#note_6304

    Same as above, search & replace, and this stuff isn't actually needed anymore

    ---
    þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net
  • From Eric Oulashin@VERT to GitLab note in main/sbbs on Saturday, January 25, 2025 21:50:30
    https://gitlab.synchro.net/main/sbbs/-/merge_requests/493#note_6305

    My concern was whether the sysop is using their own customized text for that string - Would bbs.revert_text() revert to the sysop's customized string, or back to the stock string?

    ---
    þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net
  • From Rob Swindell@VERT to GitLab note in main/sbbs on Saturday, January 25, 2025 21:53:12
    https://gitlab.synchro.net/main/sbbs/-/merge_requests/493#note_6310

    `bbs.revert_text()` reverts to whatever the sysop has set in your text.dat or text.ini files. It's intended to just undo whatever dynamic changes are made via Baja or JS, so it's the right thing to use in this instance.

    ---
    þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net
  • From Eric Oulashin@VERT to GitLab note in main/sbbs on Saturday, January 25, 2025 22:08:23
    https://gitlab.synchro.net/main/sbbs/-/merge_requests/493#note_6314

    I've made the change to use bbs.revert_text().

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