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

Secret of the Ancients S1: Gaining Human Corpses on Ardonna's Death #4503

Closed
Wedge009 opened this issue Oct 22, 2019 · 5 comments
Closed

Secret of the Ancients S1: Gaining Human Corpses on Ardonna's Death #4503

Wedge009 opened this issue Oct 22, 2019 · 5 comments
Assignees
Labels

Comments

@Wedge009
Copy link
Member

@Wedge009 Wedge009 commented Oct 22, 2019

Game and System Information

  • 1.15.2+dev Windows

Describe the bug

Only occurs on master, presumably this will occur any time until Ardonna properly gains the ability to recruit human corpses, but it's easiest just to replicate this on S1.

To reproduce

  1. Start Secret of the Ancients, any difficulty.
  2. Debug-kill Ardonna (or play normally and let her die).
  3. She will give her death cry, and then the message 'You can now raise human corpses!' appears. Presumably because she is human and she died...

Expected behaviour

In contrast, doing this on 1.14 branch gives the expected behaviour of death cry then game over.

Additional context

It appears this is introduced with:

...which calls:
https://github.com/wesnoth/wesnoth/blob/master/data/campaigns/Secrets_of_the_Ancients/utils/zombie-utils.cfg

@Wedge009

This comment has been minimized.

Copy link
Member Author

@Wedge009 Wedge009 commented Oct 23, 2019

@sigurdfdragon Are you maintaining SotA, or do you know who is?

@Wedge009

This comment has been minimized.

Copy link
Member Author

@Wedge009 Wedge009 commented Oct 26, 2019

I should add that this bug is present in 1.15.1 release.

@sigurdfdragon sigurdfdragon self-assigned this Oct 26, 2019
@sigurdfdragon

This comment has been minimized.

Copy link
Contributor

@sigurdfdragon sigurdfdragon commented Oct 26, 2019

@Wedge009 Yes

@sigurdfdragon

This comment has been minimized.

Copy link
Contributor

@sigurdfdragon sigurdfdragon commented Oct 26, 2019

@Wedge009 Thanks for the report

@Wedge009

This comment has been minimized.

Copy link
Member Author

@Wedge009 Wedge009 commented Oct 26, 2019

Works for me, thank you!

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

Successfully merging a pull request may close this issue.

None yet
2 participants
You can’t perform that action at this time.