LandSandBoat / server

:sailboat: LandSandBoat - a server emulator for Final Fantasy XI
https://landsandboat.github.io/server/
GNU General Public License v3.0
303 stars 614 forks source link

🐛 Mount command does not respect gained enmity #6130

Open HaxPrime opened 3 months ago

HaxPrime commented 3 months ago

I affirm:

OS / platform the server is running (if known)

Windows 11

Branch affected by issue

base

----== SystemMessage ==----
base-be17f7f3c3
Merge pull request #6121 from TiberonKalkaz/youre_a_wzard_galkas
Mon Aug 19 03:12:19 2024

client version
version 30240806_0

Steps to reproduce

Video https://cdn.discordapp.com/attachments/1095358627520458832/1275201865424048290/2024-08-19_15-52-37.mkv?ex=66c507c8&is=66c3b648&hm=20a9270f0702272bdb6c0d7f862c49256d9756ced07dc4505be6d8463dd96c59&

Expected behavior

Players should not be able to use mounts with gained enmity.

You are unable to mount while on a monster’s enmity list. https://www.bg-wiki.com/ffxi/Category:Mounts

Xaver-DaRed commented 3 months ago

What command are you refering to?

HaxPrime commented 3 months ago

What command are you refering to?

/mount crab example. Same results when using the abilities / mount player menu.

zach2good commented 3 months ago

You'll need to provide more detailed instructions, because it works fine on my end.

  1. !addallmounts
  2. Punch a mob in the face
  3. Disengage the mob (you can't call a mount while you're engaged)
  4. /mount crab
  5. You are rejected
image

I debugged it, and the check of NotorietyContainer (the reverse enmity container) works as expected

image image
HaxPrime commented 3 months ago

You'll need to provide more detailed instructions, because it works fine on my end.

Thanks for the quick reply and debug information. To further elaborate on this issue it might have to do with how the enmity is gained, to perform the bug you must not take action vs the monster, passively gain enmity / detection. I'll list more detailed steps to recreate;

  1. !changejob sam 20
  2. !pos 483.43 25.00 484.36 109 -- Pashhow Marshlands
  3. Passively gain aggression from the goblins here, take no action vs them.
  4. /mount crab

Thanks for the time guys.

cocosolos commented 3 months ago

This is probably due to the supertanking enmity changes in #5632. Passive aggro and linking doesn't add to the enmity list any more, just causes the mob to engage a target.

zach2good commented 3 months ago

Yup, that'd do it