search

We-the-People-civ4col-mod / Mod

This is the repository where the mod resides.
89 stars 37 forks source link

the building offered in the event has already been built #980

Open MrZorG33 opened 8 months ago

MrZorG33 commented 8 months ago

in my test game the EVENTTRIGGER_BURNING_CORN_CHAMBER trigger worked. however, despite the presence of gold in the treasury, I cannot make the choice of the third option (subtract 1500 gold from the treasury and get Granary). Civ4ScreenShot0005

and this happened due to the fact that the building proposed for construction (Granary) had already been built. Civ4ScreenShot0006

Is it possible to add to the trigger a check for the presence of the building proposed in the third option?

Schmiddie1979 commented 6 months ago

I think in this case it is ok not to have the option - you already have the building. Ok, would be nice in this case the third option would not be shown...but I assume that would require a further Python function/code. Hence, I think we can live with that?

MrZorG33 commented 6 months ago

yes we can live with it, it's not something that needs an instant fix. but it's better to fix it.

Razonatair commented 3 months ago

What's wrong with allowing players to view other potential options?

MrZorG33 commented 3 months ago

@Razonatair There is nothing wrong. but this confuses the player. as an option, show a hint why this choice cannot be made.