"Requires a [buildingFilter] in at least [amount] cities" not highlighting buildings that can't be built about unciv HOT 7 CLOSED

Just to clarify, the Medical Network requires a Hospital in at least four cities. I think that requirement should be displayed underneath it, explaining why it can't be built.

from unciv.

Analyis... Compare RequiresBuildingInAllCities with RequiresBuildingInSomeCities - as UniqueType and almost-identically-named RejectionReasonType. I see no dissymmetry anywhere... Except in orderedImportantRejectionTypes. That will be 'it' most likely.

from unciv.

But - IConstruction.kt somehow really calls for a reorg. It's got six top-level declarations that should be each their own file (or nested), and those four RejectionReasonType collections in RejectionReason are - on the third or fourth look - a catastrophe. They're instantiated anew for each new RejectionReason, despite being static in nature. Oversight wouldn't have happened if the usecases were implemented as enum properties on the type - makes the enum instances a little longer, but all uses much shorter and more efficient...

from unciv.

Also, there's no Medical Network in the public Deciv Redux.

from unciv.

Faking the unique on the next best victim and patching orderedImportantRejectionTypes as stated above I can get:

from unciv.

Also, there's no Medical Network in the public Deciv Redux.

https://github.com/SpacedOutChicken/DeCiv-Redux/blob/0bf8d55beae75c1912a2095ebca02e7e2bb210bb/jsons/Buildings.json#L1721

This is likely to change soon, once Sullien updates the mod, but it's there right now.

from unciv.

there right now

I admit I just used the built-in mod manager, looking for date and update, then pedia... Strange as github blame says that line is a year old. But as you can see further down I just created the situation elsewhere.

from unciv.