pokerogue icon indicating copy to clipboard operation
pokerogue copied to clipboard

Implements cursed body

Open shayebeadling opened this issue 10 months ago • 6 comments

couldn't figure out a more elegant way of getting the attacker and move data into the getTriggerMessage function, open to suggestions about improvement. I imagine the args[] array could be used but it's not obvious to me how.

shayebeadling avatar Apr 21 '24 18:04 shayebeadling

Does it still disable struggle? That's the only concern I have since like you said, the other troublesome factors are either unimplemented or super edge cases

Tempo-anon avatar May 06 '24 17:05 Tempo-anon

From my testing, never disabled struggle. However, it also never disables a move if the pokemon with Cursed Body fainted. Have you checked against that?

https://bulbapedia.bulbagarden.net/wiki/Cursed_Body_(Ability)

bennybroseph avatar May 06 '24 17:05 bennybroseph

@bennybroseph good call, added the bypassFaint() tag that we use for similar abilities. Now working as intended. Tested by setting chance to 100%, and with these overrides. image

shayebeadling avatar May 06 '24 23:05 shayebeadling

Okay, looks to be working. Glad I checked again though. The message that plays when an enemy's move is no longer disabled doesn't specify who the message is for. Very confusing in this fight when we both had tackle. The one when someone is disabled plays properly

https://github.com/pagefaultgames/pokerogue/assets/13838608/d7ff9443-fad0-42d5-9d2b-10dd268af2a9

bennybroseph avatar May 08 '24 05:05 bennybroseph

It applies the same tag that disable does, so the messages are the same. But I just updated the messages to clarify which pokemon's move it is now. For both cursed body and disable.

shayebeadling avatar May 08 '24 13:05 shayebeadling

Still getting a confusing message. I see the code as updated to include the Pokemon's name, but it's not clear if it was you or the opponent

image

For reference, the image is of the enemy Houndoom no longer being disabled.

bennybroseph avatar May 08 '24 20:05 bennybroseph

Now sticking the prefix on the front, will be consistent with our other messages now

shayebeadling avatar May 08 '24 21:05 shayebeadling

Looks like there's an extra space in the message here:

image

Same here for trainer's pokemon:

image

For comparison:

image

bennybroseph avatar May 08 '24 22:05 bennybroseph

Looks good to me now

bennybroseph avatar May 08 '24 23:05 bennybroseph