• modopts.d may not be working as advertisied.

    From Nigel Reed@1:103/705 to GitLab issue in main/sbbs on Mon Feb 10 15:10:34 2025
    open https://gitlab.synchro.net/main/sbbs/-/issues/871

    I created /sbbs/ctrl/modopts.d/logonlist.ini

    ```
    [logonlist:lang=m7]
    first_caller_msg: "\x83You are the first caller of the day!"
    ```

    However when loginlist.js is run, it's not displaying the text. It seems that the [logonlist] section is modopts.ini is still being used.

    If I writeln(first_caller_msg) then it shows as "" and I have checked user.lang = m7.

    I'm pretty sure I've done everything right at this stage.
    --- 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 Mon Feb 10 17:11:36 2025
    https://gitlab.synchro.net/main/sbbs/-/issues/871#note_6369

    Does your ctrl/modopts.ini file contains this line:
    ```
    !include modopts.d/*.ini
    ```
    ?

    Do you have the latest `exec/load/modopts.js` file?
    --- SBBSecho 3.23-Linux
    * Origin: Vertrauen - [vert/cvs/bbs].synchro.net (1:103/705)
  • From Nigel Reed@1:103/705 to GitLab note in main/sbbs on Mon Feb 10 18:48:13 2025
    https://gitlab.synchro.net/main/sbbs/-/issues/871#note_6370

    Yes and yes.

    ```
    bbs@wibble:/sbbs/ctrl$ grep include modopts.ini
    !include modopts.d/*.ini
    ```

    Last commit logged for exec/load/modopts.js

    ```
    commit 72e3271b43a6ccf1e6ce953800b1984ff65dbb82
    Author: Rob Swindell (on Debian Linux) <rob@synchro.net>
    Date: Sat Feb 8 23:50:47 2025 -0800

    Load section or keys form [module:lang=<user-lang>] if it exists

    This allow language-specific module options
    ```
    --- 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 Mon Feb 10 19:10:40 2025
    https://gitlab.synchro.net/main/sbbs/-/issues/871#note_6371

    Examining the code further, this change won't work with the modopts.d/* files currently.

    You can, in the mean time, just add the [logonlist:lang=m7] section to the ctrl/modopts.ini file instead.

    I'm pondering a different solution using a separate .ini file per module (e.g. ctrl/logonlist.ini) that doesn't depend on the !include feature of the .ini parser (which doesn't support all use-cases).
    --- SBBSecho 3.23-Linux
    * Origin: Vertrauen - [vert/cvs/bbs].synchro.net (1:103/705)
  • From Rob Swindell@1:103/705 to GitLab issue in main/sbbs on Mon Feb 10 20:42:59 2025
    close https://gitlab.synchro.net/main/sbbs/-/issues/871
    --- SBBSecho 3.23-Linux
    * Origin: Vertrauen - [vert/cvs/bbs].synchro.net (1:103/705)
  • From Nigel Reed@1:103/705 to GitLab note in main/sbbs on Tue Feb 11 01:42:52 2025
    https://gitlab.synchro.net/main/sbbs/-/issues/871#note_6374

    I did an update and the commit did fix the issue.

    ![image](/uploads/a8edd9b5c9be946cf6ff016ea6ac1785/image.png){width=852 height=309}

    ```
    last_few_callers_msg: "\x82Last few callers:\r\n"
    last_few_callers_fmt: "\x85%1$-3s %2$-5s\x87%3$-22.22s\x85%6$-6.6s\x83%4$-26.26s\x86%5$s %7$4u"
    ```

    I was also able to use your suggestion to change which variables are printed where via format modifications so killed 2 issues I had with one stone there.

    Personally, I like the idea of having a .ini per .js file for configuration, that way it's easy to separate them out and you won't end up with a bulky modopts.ini file for all the modules.
    --- SBBSecho 3.23-Linux
    * Origin: Vertrauen - [vert/cvs/bbs].synchro.net (1:103/705)