[Sn!ff3r]

PHP Code:

#include <amxmodx>
#include <hamsandwich>
#include <fakemeta>

new hp; 

public plugin_init()
{
    register_plugin("HP Bug Fix", "1.0","Hattrick");
    RegisterHam(Ham_Spawn, "player", "AtSpawn", 1);
    
    hp = get_user_msgid("Health");
}

public AtSpawn(id)
{
    if(!is_user_alive(id))
        return;
    
    message_begin(MSG_ONE, hp, _, id);
    write_byte(pev(id, pev_health));
    message_end();
} 


Better a bit.

[ConnorMcLeod]

Approval requirements

• Don't register both pre and post Spawn forward, only post is needed.
• Don't send Health message, just set m_iClientHealth (359) private data to 0 or -1 and game gonna send the message itself.
• Remove [AMXX] from post title.

[claudiuhks]

That's it.

[ConnorMcLeod]

Check if player is alive.

PHP Code:

    new iHealth = 100;
    iHealth -= 1;
    set_pdata_int(id, 359, iHealth); 


why don't you do :
set_pdata_int(id, 359, 99)
??

If you want to be sure the message is sent, set some negative value.

[claudiuhks]

Alive check available, negative value is set.
Everything alright.

[ConnorMcLeod]

No return is needed in post forwards.

Would be better if you named the offset, or add some comment in plugin so if someone reads your code he gonna understand it.

Approved.