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

Fix antennae handling for auto attacking #37686

Merged
merged 4 commits into from
Feb 4, 2020

Conversation

Ramza13
Copy link
Contributor

@Ramza13 Ramza13 commented Feb 4, 2020

Summary

SUMMARY: Bugfixes "Fix antennae handling for auto attacking"

Purpose of change

Fixes #37234
Antennae allowed attempting to auto attack targets across z levels and through walls.

Describe the solution

Added check to make sure targetable creatures can not only be seen by player but are reachable from current location

Describe alternatives you've considered

Testing

Additional context

src/player.cpp Outdated Show resolved Hide resolved
@ZhilkinSerg ZhilkinSerg added <Bugfix> This is a fix for a bug (or closes open issue) [C++] Changes (can be) made in C++. Previously named `Code` Mutations / Traits / Professions/ Hobbies Mutations / Traits / Professions/ Hobbies labels Feb 4, 2020
@ZhilkinSerg ZhilkinSerg merged commit 7b46ee2 into CleverRaven:master Feb 4, 2020
@Ramza13 Ramza13 deleted the auto_attack_z branch February 4, 2020 14:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
<Bugfix> This is a fix for a bug (or closes open issue) [C++] Changes (can be) made in C++. Previously named `Code` Mutations / Traits / Professions/ Hobbies Mutations / Traits / Professions/ Hobbies
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Antennae Multilevel attacks
3 participants