[edon1337]

Originally Posted by ^SmileY This m_flNextPrimaryAttack will not work since weapon is already fired. set_pdata_int(id, m_bCanShoot, false); Try with

Didn't work either

876 is the value for that off-set in Windows, right?

[Ghosted]

Originally Posted by edon1337 PHP Code: new bool:g_bVictoryBlockAttack[ 33 ]; #define g_iBitsum (1<<CSW_P228)|(1<<CSW_GLOCK)|(1<<CSW_C4)|(1<<CSW_ELITE)|(1<<CSW_FIVESEVEN)|(1<<CSW_UMP45)|(1<<CSW_GALIL)|(1<<CSW_FAMAS)|(1<<CSW_USP)|(1<<CSW_GLOCK18)|(1<<CSW_DEAGLE) public plugin_init( ) {     for( new i = CSW_P228; i <= CSW_P90; i++ )     {         if( g_iBitsum & (1<<i) )         continue;                  get_weaponname( i, szWeaponName, charsmax( szWeaponName ) );         RegisterHam( Ham_Weapon_PrimaryAttack, szWeaponName, "@HamPrimaryAttack_Pre" );     } } public @HamPrimaryAttack_Pre( iEnt ) {     new id;     id = pev( iEnt, pev_owner );          if( g_bVictoryBlockAttack[ id ] )     {         set_pdata_float( iEnt, m_flNextPrimaryAttack, 9999.9, 4 );         return HAM_SUPERCEDE;     }          return HAM_IGNORED; } Victory( iPlayer ) {     new szPlayers[ 32 ], iNum, iTempID;     get_players( szPlayers, iNum, "a" );          for( new i; i < iNum; i++ )     {         iTempID = szPlayers[ i ];         g_bVictoryBlockAttack[ iTempID ] = true;     } }

#1 Your code is not compilable cause there is not szWeaponName
#2 You should use new const instead of #define on bitsums, cause it wont work.

[Ghosted]

Try this, its working fiNe

new bool:g_bVictoryBlockAttack[ 33 ]; 

new const g_iBitsum = (1<<CSW_P228)|(1<<CSW_GLOCK)|(1<<CSW_C4)|(1<<CSW_ELITE)|(1<<CSW_FIVESEVEN)|(1<<CSW_UMP45)|(1<<CSW_GALIL)|(1<<CSW_FAMAS)|(1<<CSW_USP)|(1<<CSW_GLOCK18)|(1<<CSW_DEAGLE);

public plugin_init( ) 
{ 
new szWeaponName[32];
    for( new i = CSW_P228; i <= CSW_P90; i++ ) 
    { 
        if( g_iBitsum & (1<<i) ) 
        continue; 
         
        get_weaponname( i, szWeaponName, charsmax( szWeaponName ) ); 
        RegisterHam( Ham_Weapon_PrimaryAttack, szWeaponName, "@HamPrimaryAttack_Pre" ); 
    } 
} 

public @HamPrimaryAttack_Pre( iEnt ) 
{ 
    new id; 
    id = pev( iEnt, pev_owner ); 
     
    if( g_bVictoryBlockAttack[ id ] ) 
    { 
        return HAM_SUPERCEDE; 
    } 
     
    return HAM_IGNORED; 
} 

Victory( iPlayer ) 
{ 
    new szPlayers[ 32 ], iNum; 
    get_players( szPlayers, iNum, "a" ); 
     
    for( new i; i < iNum; i++ ) 
    { 
        g_bVictoryBlockAttack[szPlayers[ i ]] = true; 
    } 
}

[HamletEagle]

Originally Posted by ^SmileY This m_flNextPrimaryAttack will not work since weapon is already fired. set_pdata_int(id, m_bCanShoot, false); Try with

What? The code should work. If it doesn't then probably the if check is never true. There must be some logic error in the plugin.
edon, do some debugging to see if set_pdata_float is reached or not.

Originally Posted by edon1337 Didn't work either 876 is the value for that off-set in Windows, right?

No. 876 is the full offset. In a plugin you have to divide by the size, so 876/4 to use set_pdata_int or use set_pdata_bool(amxx 1.8.3 or Connor's stock).

[Ghosted]

Originally Posted by HamletEagle What? The code should work. If it doesn't then probably the if check is never true. There must be some logic error in the plugin. edon, do some debugging to see if set_pdata_float is reached or not. No. 876 is the full offset. In a plugin you have to divide by the size, so 876/4.

Yes, error is in the plugin i posted where (in bitsums)

[edon1337]

Originally Posted by HamletEagle What? The code should work. If it doesn't then probably the if check is never true. There must be some logic error in the plugin. edon, do some debugging to see if set_pdata_float is reached or not. No. 876 is the full offset. In a plugin you have to divide by the size, so 876/4 to use set_pdata_int or use set_pdata_bool(amxx 1.8.3 or Connor's stock).

Basically the problem is somewhere else

PHP Code:

    new szWeaponName[ 32 ];
    
    for( new i = CSW_P228; i <= CSW_P90; i++ )
    {        
        get_weaponname( i, szWeaponName, charsmax( szWeaponName ) );
        RegisterHam( Ham_Weapon_PrimaryAttack, szWeaponName, "@HamPrimaryAttack_Pre" );
        
        log_to_file( "test.txt", "Weapon name: %s", szWeaponName );
    } 


It only loops once and stops..

L 07/13/2018 - 12: 38: 06: Weapon name: weapon_p228

And I get this error

L 07/13/2018 - 12:44:12: [HAMSANDWICH] Failed to retrieve classtype for "", hook for "@HamPrimaryAttack_Pre" not active.

Originally Posted by Ghosted #1 Your code is not compilable cause there is not szWeaponName #2 You should use new const instead of #define on bitsums, cause it wont work.

I never said it wasn't compilable? szWeaponName is already in my code I just forgot to add it in the code I provided here, not a big deal.
Yes, that's true that the #define on bitsums doesn't work, but I recently noticed I don't even need the bitsums, so I removed that part.

[HamletEagle]

If you debugged properly(adding the debug message before RegisterHam) you would see it actually loops twice:

PHP Code:

Weapon name = weapon_p228 | i = 1
Weapon name =  | i = 2 


And if you look at the CSW_ indexes from here: http://amxmodx.org/api/cstrike_const the issue should be more than obvious.

[edon1337]

Originally Posted by HamletEagle If you debugged properly(adding the debug message before RegisterHam) you would see it actually loops twice: PHP Code: Weapon name = weapon_p228 | i = 1 Weapon name =  | i = 2  And if you look at the CSW_ indexes from here: http://amxmodx.org/api/cstrike_const the issue should be more than obvious.

So should I just skip CSW_GLOCK ?

EDIT: Yea it works, someone should even release a plugin including this and a code that fixes the glitch that Crazy mentioned with the console.

[klippy]

Regarding the cvar bug - can you post "amxx version"?

EDIT: You are clearly using a revision of 1.8.3, I see the behavior has changed and I'll see if this is really an issue, but it isn't important for your case. What's important is that you should put all cvars in server.cfg - amxx.cfg is used only for AMXX core cvars and people shouldn't add more cvars to it. It wasn't consistent even before - if you had a plugin that would load before admin.sma your cvars wouldn't be up to date in plugins_init.

[edon1337]

Originally Posted by KliPPy Regarding the cvar bug - can you post "amxx version"? EDIT: You are clearly using a revision of 1.8.3, I see the behavior has changed and I'll see if this is really an issue, but it isn't important for your case. What's important is that you should put all cvars in server.cfg - amxx.cfg is used only for AMXX core cvars.

I'm actually using AMXX 1.8.2 but setting them in server.cfg seems to be working. Thanks.