• DDMsgReader: Inactivity disconnection more consistent with Synchronet

    From Rob Swindell@1:103/705 to GitLab note in main/sbbs on Sat Jan 25 21:31:06 2025
    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`. :-) --- SBBSecho 3.23-Linux
    * Origin: Vertrauen - [vert/cvs/bbs].synchro.net (1:103/705)
  • From Rob Swindell@1:103/705 to GitLab note in main/sbbs on Sat Jan 25 21:31:22 2025
    https://gitlab.synchro.net/main/sbbs/-/merge_requests/493#note_6299

    Same here. Seems like a no-op.
    --- SBBSecho 3.23-Linux
    * Origin: Vertrauen - [vert/cvs/bbs].synchro.net (1:103/705)
  • From Rob Swindell@1:103/705 to GitLab note in main/sbbs on Sat Jan 25 21:32:48 2025
    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. :-)
    --- SBBSecho 3.23-Linux
    * Origin: Vertrauen - [vert/cvs/bbs].synchro.net (1:103/705)
  • From Rob Swindell@1:103/705 to GitLab note in main/sbbs on Sat Jan 25 21:34:44 2025
    https://gitlab.synchro.net/main/sbbs/-/merge_requests/493#note_6301

    Looks like you tried this method but had an issue?
    --- SBBSecho 3.23-Linux
    * Origin: Vertrauen - [vert/cvs/bbs].synchro.net (1:103/705)
  • From Rob Swindell@1:103/705 to GitLab note in main/sbbs on Sat Jan 25 21:34:44 2025
    https://gitlab.synchro.net/main/sbbs/-/merge_requests/493#note_6302

    Some comments added
    --- SBBSecho 3.23-Linux
    * Origin: Vertrauen - [vert/cvs/bbs].synchro.net (1:103/705)
  • From Eric Oulashin@1:103/705 to GitLab note in main/sbbs on Sat Jan 25 21:49:16 2025
    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.
    --- SBBSecho 3.23-Linux
    * Origin: Vertrauen - [vert/cvs/bbs].synchro.net (1:103/705)
  • From Eric Oulashin@1:103/705 to GitLab note in main/sbbs on Sat Jan 25 21:49:44 2025
    https://gitlab.synchro.net/main/sbbs/-/merge_requests/493#note_6304

    Same as above, search & replace, and this stuff isn't actually needed anymore --- SBBSecho 3.23-Linux
    * Origin: Vertrauen - [vert/cvs/bbs].synchro.net (1:103/705)
  • From Eric Oulashin@1:103/705 to GitLab note in main/sbbs on Sat Jan 25 21:50:30 2025
    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?
    --- SBBSecho 3.23-Linux
    * Origin: Vertrauen - [vert/cvs/bbs].synchro.net (1:103/705)
  • From Rob Swindell@1:103/705 to GitLab note in main/sbbs on Sat Jan 25 21:53:12 2025
    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.
    --- SBBSecho 3.23-Linux
    * Origin: Vertrauen - [vert/cvs/bbs].synchro.net (1:103/705)
  • From Eric Oulashin@1:103/705 to GitLab note in main/sbbs on Sat Jan 25 22:08:22 2025
    https://gitlab.synchro.net/main/sbbs/-/merge_requests/493#note_6314

    I've made the change to use bbs.revert_text().
    --- SBBSecho 3.23-Linux
    * Origin: Vertrauen - [vert/cvs/bbs].synchro.net (1:103/705)