Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

monster walkback pr causing issues #3615

Closed
2 tasks done
ghost opened this issue Aug 29, 2021 · 4 comments · Fixed by #3612
Closed
2 tasks done

monster walkback pr causing issues #3615

ghost opened this issue Aug 29, 2021 · 4 comments · Fixed by #3612

Comments

@ghost
Copy link

ghost commented Aug 29, 2021

Before creating an issue, please ensure:

  • This is a bug in the software that resides in this repository, and not a
    support matter (use https://otland.net/forums/support.16/ for support)
  • This issue is reproducible without changes to the C++ code in this repository

Steps to reproduce (include any configuration/script required to reproduce)

  1. go -1 with normal character, wait a second for mobs to notice you then rope yourself
  2. have gm wait on -1 to see what happens
  3. yes this was tested with both walkToSpawnRadius enabled / disabled

Actual behaviour

the monsters will keep walking towards the rope spot even tho normal character is not there (i thought the pr was about walking back and not surrounding the spot) regardless its a bug

Expected behaviour

before #3604 they stopped walking when player left -1

it was better before, but it was also not correct since Monster::canSee() needs an update

i don't know exactly how it should work on all floors in this version but so far i have seen on Rl that they walk if you are on floor 0 and they are on +1 and +2

aswell will walk if you are floor 0 and they are -1

@ghost
Copy link
Author

ghost commented Aug 29, 2021

#3612

@marmichalski
Copy link
Contributor

This should be resolved by #3612, since it marks monster as idle immediately now.

@ghost ghost linked a pull request Aug 29, 2021 that will close this issue
3 tasks
@ghost
Copy link

ghost commented Sep 2, 2021

hey @emil92b did you test it?

@ghost
Copy link
Author

ghost commented Sep 9, 2021

hey @emil92b did you test it?

been busy, can prob do it in a week or two

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant