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

The boat wipes out part of the whirlpool when the hero who was in it disembarks #9141

Closed
2 tasks done
oleg-derevenetz opened this issue Sep 24, 2024 · 1 comment · Fixed by #9217
Closed
2 tasks done
Assignees
Labels
bug Something doesn't work logic Things related to game logic
Milestone

Comments

@oleg-derevenetz
Copy link
Collaborator

oleg-derevenetz commented Sep 24, 2024

Preliminary checks

Platform

Windows

Describe the bug

Found during testing #9140. On the master branch:

fheroes2.engine.version_.1.1.2.2024-09-25.00-41-01.mp4

After the hero disembarks from a boat located on one of the tiles of the whirlpool, the OBJ_BOAT replaces the OBJ_WHIRLPOOL on that tile. On the master branch, another hero is still able to teleport to that tile with the OBJ_BOAT. If the hero gets back into the boat and sails away from the whirlpool, the whirlpool object on that tile is not being restored, but another hero still can teleport to it:

fheroes2.engine.version_.1.1.2.2024-09-25.00-41-36.mp4

With the fixes from #9140, the situation is a little different - OBJ_BOAT still replaces the OBJ_WHIRLPOOL upon disembarking, but another hero cannot teleport to that tile anymore (regardless of whether it's occupied with the boat or not):

fheroes2.engine.version_.1.1.2.2024-09-25.00-54-38.mp4

Save file

Test map:

!!TEST1.zip

Additional info

No response

@oleg-derevenetz
Copy link
Collaborator Author

Hi @ihhub none of my PRs are intended to fix this :) This is not a pathfinder issue, but rather an issue related to map internals, as I see it, since a map tile is not able to hold two objects at the same time (just like that issue with a pickupable object on a coast). I would like to suggest someone who understands the map internals to take a look.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something doesn't work logic Things related to game logic
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants