[BUG]: RH_ExecuteServerStringCmd spams console

[that-is-not-great]

👉👈 Contact Details (optional)

No response

🤔 What happened?

RegisterHookChain(RH_ExecuteServerStringCmd, "RH_ExecuteServerStringCmd_Pre")

...

public RH_ExecuteServerStringCmd_Pre(const cmd[], source, id)
{
    static fullcmd[256];
 	read_argv(0, fullcmd, cx(fullcmd));

    log_amx("[OK] cmd [%s] [%s] [%s] [%i]", fullcmd, cmd, id, source)

    return HC_CONTINUE
}

the last used cmd will be spammed into the console forever, is that correct behavior?

⚠️ Meta-information

ReAPI            RUN   -    reapi_amxx_i386.so          v5.24.0.300-de

📄 Relevant log output

L 12/26/2024 - 13:57:38: [test.amxx] [OK] cmd [weapon_knife] [] [] [1]
L 12/26/2024 - 13:57:38: [test.amxx] [OK] cmd [weapon_knife] [] [] [1]
L 12/26/2024 - 13:57:38: [test.amxx] [OK] cmd [weapon_knife] [] [] [1]
L 12/26/2024 - 13:57:38: [test.amxx] [OK] cmd [weapon_knife] [] [] [1]
L 12/26/2024 - 13:57:38: [test.amxx] [OK] cmd [weapon_knife] [] [] [1]
L 12/26/2024 - 13:57:38: [test.amxx] [OK] cmd [weapon_knife] [] [] [1]
L 12/26/2024 - 13:57:38: [test.amxx] [OK] cmd [weapon_knife] [] [] [1]
L 12/26/2024 - 13:57:38: [test.amxx] [OK] cmd [weapon_knife] [] [] [1]
L 12/26/2024 - 13:57:38: [test.amxx] [OK] cmd [weapon_knife] [] [] [1]
L 12/26/2024 - 13:57:38: [test.amxx] [OK] cmd [weapon_knife] [] [] [1]
L 12/26/2024 - 13:57:38: [test.amxx] [OK] cmd [weapon_knife] [] [] [1]
L 12/26/2024 - 13:57:38: [test.amxx] [OK] cmd [weapon_knife] [] [] [1]
L 12/26/2024 - 13:57:38: [test.amxx] [OK] cmd [weapon_knife] [] [] [1]
L 12/26/2024 - 13:57:38: [test.amxx] [OK] cmd [weapon_knife] [] [] [1]
L 12/26/2024 - 13:57:38: [test.amxx] [OK] cmd [weapon_knife] [] [] [1]
L 12/26/2024 - 13:57:38: [test.amxx] [OK] cmd [weapon_knife] [] [] [1]
L 12/26/2024 - 13:57:38: [test.amxx] [OK] cmd [weapon_knife] [] [] [1]
L 12/26/2024 - 13:57:38: [test.amxx] [OK] cmd [weapon_knife] [] [] [1]
L 12/26/2024 - 13:57:38: [test.amxx] [OK] cmd [weapon_knife] [] [] [1]
L 12/26/2024 - 13:57:38: [test.amxx] [OK] cmd [weapon_knife] [] [] [1]
L 12/26/2024 - 13:57:38: [test.amxx] [OK] cmd [weapon_knife] [] [] [1]
L 12/26/2024 - 13:57:38: [test.amxx] [OK] cmd [weapon_knife] [] [] [1]
L 12/26/2024 - 13:57:38: [test.amxx] [OK] cmd [weapon_knife] [] [] [1]
L 12/26/2024 - 13:57:38: [test.amxx] [OK] cmd [weapon_ak47] [] [] [1]
L 12/26/2024 - 13:57:38: [test.amxx] [OK] cmd [weapon_ak47] [] [] [1]
L 12/26/2024 - 13:57:38: [test.amxx] [OK] cmd [weapon_ak47] [] [] [1]
L 12/26/2024 - 13:57:38: [test.amxx] [OK] cmd [weapon_ak47] [] [] [1]
L 12/26/2024 - 13:57:38: [test.amxx] [OK] cmd [weapon_ak47] [] [] [1]
L 12/26/2024 - 13:57:38: [test.amxx] [OK] cmd [weapon_ak47] [] [] [1]
L 12/26/2024 - 13:57:38: [test.amxx] [OK] cmd [weapon_ak47] [] [] [1]
L 12/26/2024 - 13:57:38: [test.amxx] [OK] cmd [weapon_ak47] [] [] [1]
L 12/26/2024 - 13:57:38: [test.amxx] [OK] cmd [weapon_ak47] [] [] [1]
L 12/26/2024 - 13:57:38: [test.amxx] [OK] cmd [weapon_ak47] [] [] [1]
L 12/26/2024 - 13:57:38: [test.amxx] [OK] cmd [weapon_ak47] [] [] [1]
L 12/26/2024 - 13:57:38: [test.amxx] [OK] cmd [weapon_ak47] [] [] [1]
L 12/26/2024 - 13:57:38: [test.amxx] [OK] cmd [weapon_ak47] [] [] [1]
L 12/26/2024 - 13:57:38: [test.amxx] [OK] cmd [weapon_ak47] [] [] [1]
L 12/26/2024 - 13:57:38: [test.amxx] [OK] cmd [weapon_ak47] [] [] [1]
L 12/26/2024 - 13:57:38: [test.amxx] [OK] cmd [weapon_ak47] [] [] [1]
L 12/26/2024 - 13:57:38: [test.amxx] [OK] cmd [weapon_ak47] [] [] [1]
L 12/26/2024 - 13:57:38: [test.amxx] [OK] cmd [weapon_ak47] [] [] [1]

[justgo97]

This is related to #300
both RH_ExecuteServerStringCmd & RH_SV_AllowPhysent should be swapped plugin side for now
It's already fixed in master but new version didn't get released.

@s1lentq @SergeyShorokhov Time for a new version?