Check if a town can bypass the adjacent chunk check per-world - #8290
Check if a town can bypass the adjacent chunk check per-world#8290Veyronity wants to merge 1 commit into
Conversation
|
I think this looks correct, can you tell me which ways this has been tested? ie:
|
|
|
I think that claiming a outpost in a world you've already got claims in should be blocked, as those claims are not adjacent to the town in that world. Making a second outpost in that same world should also get blocked. I believe outposts should only be allowed in other worlds, and be limited to one per world, otherwise the adjacent claims feature is effectively bypassed. |
|
This PR doesn't interfere that behavior, even as of now players can claim multiple outposts in the same world. |
|
I think it ought to be blocked, when I made it possible to make an outpost I must have not taken that into account. Whether a town is allowed to make an outpost in a world they already have claims in could get added as a config option I guess. |
|
I can make a separate PR for that to help out if you'd like |
|
I'm happy with this one as it stands now. I think adjacent claiming rules should be per-world. You could make a new PR for the other thing or put it into this PR, either is fine. |
Description:
This updates the ProximityUtil check for if a town is large enough to enforce adjacent chunk rules, to keep in mind the TownyWorld the check is happening for
Attestations
By making this pull request, I represent that I have the right to waive copyright and related rights to my contribution, and agree that all copyright and related rights in my contributions are waived, and I acknowledge that the TownyAdvanced organization has the copyright to use and modify my contribution under the Towny License for perpetuity.
In making this pull request, I declare that I have not used an AI toolset to design or code this pull request, and that I am a human who coded this without any influence from an LLM.