[HamletEagle]

Update to 1.4

[Eagle07]

Originally Posted by HamletEagle Update to 1.4

GJ ^^

[HamletEagle]

I think to add a new feature. If a player shot a hostage, the bullet will hit the player, and not the hostage. Is someone interest in this ? I will add it soon.

[zmd94]

If a player shot a hostage, the bullet will hit the player, and not the hostage.

Nice idea. ;)

[Eagle07]

Originally Posted by HamletEagle I think to add a new feature. If a player shot a hostage, the bullet will hit the player, and not the hostage. Is someone interest in this ? I will add it soon.

(y)

[deviss]

Nice plugin, thanks. I love the fact that it gets auto-paused and blocks he grenade

[HamletEagle]

Originally Posted by deviss Nice plugin, thanks. I love the fact that it gets auto-paused and blocks he grenade

Thx you. I updated the plugin to 0.5 version, it just bring some optimzation and needed checks added.

[Arkshine]

Why do you start second ifind_ent_by_class from iEnt[ 0 ], what if entity is defined before?
You should use equali ; it's possible someone changes manually map and inclufing uppercase letters.
server_cmd can format a string for your information.

[HamletEagle]

Originally Posted by Arkshine Why do you start second ifind_ent_by_class from iEnt[ 0 ], what if entity is defined before? You should use equali ; it's possible someone changes manually map and inclufing uppercase letters. server_cmd can format a string for your information.

Code fixed.

[Arkshine]

• Why the hell are you checking for cs_assault map? If it's because there are not such entities checked before, you can just check if there are hostages hostage_entity.
  Also you can take in count monster_scientist which is basically the same. You could do something like:
  

      new const hostageClassnames[][] = { "func_hostage_rescue", "info_hostage_rescue", "hostage_entity", "monster_entity" };
      
      for (new i = 0; i < sizeof hostageClassnames;  ++i)
      {
          if (find_ent_by_class(-1, hostageClassnames[i]))
          {
              return;
          }
      }
      
      // Pause, fail, whatever. 

  
  With AMXX 1.8.3
  

      if (!( get_map_objectives() & MapObjective_Hostage))
      [
          // Pause, fail, whatever.
      }

• iInflictor == iAttacker ; I don't think you should do that, an hostage could be hurt with something else, like a projectile.
• In TakeDamage, your return HAM_SUPERCEDE should be inside first check.
• hp_status doesn't exist, make sure you update your first post.
• Add support for pause/unpause plugin.
• Well, even if one sentence, multilingual could be welcomed.

Random feature suggestion: you could an option to make an hostage immunized against all kind of damage. This way, you would just need to hook hostage spawn, and altering setting pev->takedamage to DAMAGE_NO, then you would not need to hook TakeDamage/TraceAttack forwards.
Some people may want to deal damage but not enough to kill them, basically, letting the blood effect and such done in TraceAttack (not blocking this) but blocking damage in TakeDamage.
Well, just some random ideas.