-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
NullPointerException in Battle.doDestroyImprovementsAbility #12524
Comments
I need a save file to debug this, how can getTileImprovement() return null if attackedTile.improvement is not null? |
Right? It's a strange one. Start Auto-Play after loading, and it occurs.... Show Saved Game
|
Apparently we would add city ruins even where they don't exist in the ruleset |
Wow, great debugging. I was looking through for a while and couldn't find it. Thanks so much, I'll test this out later today. |
This is only in Github, not in the latest release yet :) |
Tested, and worked well! Have Unciv building locally 😉 |
Is there an existing issue for this?
Game Version
4.19.9
Describe the bug
Platform: Desktop
Version: 4.14.9 (Build 1,074)
Rulesets: [Civ IV Beyond the Sword, Civ V - Vanilla, Civ IV Audio, Civ V - Gods & Kings, Civ IV Icons, Civ IV Warlords, Fairline, Civ IV]
Last Screen:
com.unciv.ui.screens.worldscreen.WorldScreen
OS: Linux (amd64, 5.15.0-126-generic)
Freedesktop SDK 23.08 (Flatpak runtime)
Java: Flathub 21.9
Max Memory: 8008 MB
System default encoding: UTF-8
Message:
Save Mods:
Steps to Reproduce
N/A
Screenshots
No response
Link to save file
No response
Operating System
Linux
Additional Information
The text was updated successfully, but these errors were encountered: