vgstation13 icon indicating copy to clipboard operation
vgstation13 copied to clipboard

Some on_hit fixes

Open boy2mantwicethefam opened this issue 1 year ago • 6 comments

Updates some on_hit effects so that they properly get blocked when they should be blocked, which means that martians carrying 5 or 6 riot shields can now menace the station. The most important part of this change is that it fixes on_hit so that it never happens to silicons, as it was intended and which was broken for about 5 years. Fixed Force Wall causing a projectile fired into it to not only hit everyone in the force wall but also apply on_hit effects regardless of protection. PROJECTILE_COLLISION_BLOCKED has a value of 2. Added a new projectile proc, robot_on_hit, which is called by silicons. By default it calls regular on_hit which will get blocked, but it can be overridden in order to allow projectiles to do unique robot effects, should the need arise. Tasers now slow down cyborgs this way.

:cl:

  • bugfix: Martians will now properly block projectile effects using their shields.
  • bugfix: Fixed a few instances in the code where having full protection against a projectile attack would still cause projectile effects to happen, even when it should be completely blocked.
  • bugfix: Fixed a 5-year-old bug where silicons were unintentionally subjected to various projectile effects such as syringes from syringe guns. However, they have retained their weakness to slowdown caused by tasers.
  • bugfix: Force Wall will now only pick one character on its turf to deal damage to if it is impacted by a projectile, as opposed to affecting every single character, and will now respect whether the user could block the attack or not.

boy2mantwicethefam avatar Jul 02 '24 00:07 boy2mantwicethefam

Bugfix? Good Buffing silicons against tasers? Bad Please make it so tasers into sillycones still work as expected, or just remove that chunk, cause slowed down borgs or drunk borgs is a fun and relatively harmless interaction in the large scheme of things.

Eneocho avatar Jul 02 '24 01:07 Eneocho

I'll fix the tasers but not the "drunkeness" which doesn't actually happen for borgs from what I recall.

boy2mantwicethefam avatar Jul 02 '24 01:07 boy2mantwicethefam

Tasers are fixed.

boy2mantwicethefam avatar Jul 02 '24 02:07 boy2mantwicethefam

Better still be able to tackle borgs too.

Dacendeth avatar Jul 02 '24 05:07 Dacendeth

Tackling should be intact.

boy2mantwicethefam avatar Jul 02 '24 10:07 boy2mantwicethefam

urrr how was this pr tested ?? ?

hacker-on-steroids avatar Jul 02 '24 19:07 hacker-on-steroids

@hacker-on-steroids I didn't, but I might.

boy2mantwicethefam avatar Jul 04 '24 15:07 boy2mantwicethefam