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.

Sign up for GitHub

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

Jump to bottom

Bug with Mayors not being able to defend in a liberation siege #408

Closed
advex0123 opened this issue Nov 23, 2021 · 0 comments · Fixed by #409
Closed

Bug with Mayors not being able to defend in a liberation siege #408

advex0123 opened this issue Nov 23, 2021 · 0 comments · Fixed by #409
Assignees
@LlmDl
Labels
bug Something isn't working
Milestone
0.6.6

Comments

@advex0123
Copy link

Describe the bug
A clear and concise description of what the bug is.

To Reproduce
Steps to reproduce the behavior:

  1. A towny rank that has siegewar.town.siege.*
  2. The said town has to be sieged in a Liberation siege
  3. After this players will get war sickness in the town
  4. This error appears...

Expected behavior
After this players will get war sickness in the town

Screenshots
If applicable, add screenshots to help explain your problem.

Versions (please complete the following information):

  • Towny: 0.97.2.0
  • SiegeWar: 0.6.2
  • Spigot/Paper: Latest

Files
Startup log:
SiegeWar config: https://pastebin.com/XpDmrqjp
Townyperms.yml: https://pastebin.com/GEtMTwA7
@advex0123 advex0123 added the bug Something isn't working label Nov 23, 2021
LlmDl added a commit that referenced this issue Nov 23, 2021
@LlmDl
Fix Liberation siege incorrectly choosing SiegeSide.
0ea6eda 
Solves for siegesickness being given to the defending town's mayors,
assistants and guard.

They were only being tested for the nation battle points node.

Closes #408.
@LlmDl LlmDl mentioned this issue Nov 23, 2021
Merged
1 task
@LlmDl LlmDl added this to the 0.6.6 milestone Nov 23, 2021
@LlmDl LlmDl self-assigned this Nov 23, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@LlmDl LlmDl

Labels
bug Something isn't working
Projects
None yet
Milestone
0.6.6
Development

Successfully merging a pull request may close this issue.

Fix Liberation siege incorrectly choosing SiegeSide for Siege TownyAdvanced/SiegeWar
2 participants
@LlmDl @advex0123