Cruze03 / discord-utilities

Utilities that can be used to integrate gameserver to discord server.
GNU General Public License v3.0
42 stars 15 forks source link

Bug with CallAdminReportTitle in discord #36

Closed MickaNoob closed 2 years ago

MickaNoob commented 2 years ago

Hi, I have detected a bug with the DU --> Calladmin .

The message for new calladmins is not displayed correctly, the title (CallAdminReportTitle) is missing.

CallAdminReportTitle is present in tranlastion, but not working

    "CallAdminReportTitle"
    {
        "en"        "New Calladmin"
        "fr"        "Nouveau CallAdmin"
        "hu"        "Egy jelentést éppen kezelnek!"
        "nl"        "De CallAdmin Report wordt behandelt!"
        "lv"        "CallAdmin Ziņojums tiek izskatīts!"
        "ru"        "CallAdmin Репорт обрабатывается!"
        "ro"        "CallAdmin Report se gestionează!"
    }

bug1

Check Images.

MickaNoob commented 2 years ago

Hi,

I have fixed this bug,

Juste in modules.sp modifie buffer by trans

char buffer[512], trans[64];
    Format( trans, sizeof( trans ), "%T", "CallAdminReportTitle", LANG_SERVER);
    embed.SetTitle( buffer );
char buffer[512], trans[64];
    Format( trans, sizeof( trans ), "%T", "CallAdminReportTitle", LANG_SERVER);
    embed.SetTitle( trans );
Cruze03 commented 2 years ago

Oh wow, good find. Will push this update with next update. Thank you!

MickaNoob commented 2 years ago

Hi,

I have found same bug with the "bug report".

Default:

    char buffer[512], trans[64];
    Format( trans, sizeof( trans ), "%T", "BugReportTitle", LANG_SERVER);
    embed.SetTitle( buffer );

Fixed version :

    char buffer[512], trans[64];
    Format( trans, sizeof( trans ), "%T", "BugReportTitle", LANG_SERVER);
    embed.SetTitle( trans );

image

image

Cruze03 commented 2 years ago

Wow, reasonable because they both are almost copy-pasta! Thanks for notifying though!