"PlayerState" requirement
Longturn people who take cool Pirates flag regulary suffer from unwanted effects.
I think that is intentional local modification by LT to make Pirates playable while they still are barbarians.
Reply To ihnatus
Player that has Statue of Liberty or just have finished revolen this turn can change governments for free and exploit it a lot, so let some bonuses be available only when the dust settles
Reply To cazfi
Reply To ihnatus
Player that has Statue of Liberty or just have finished revolen this turn can change governments for free and exploit it a lot, so let some bonuses be available only when the dust settles
I think that for this the engine should enforce some rules, and not leave it to ruleset to plug every possible exploit. This could work similar to multiplier changes that can be chosen during the turn freely, but are activated only at turn change (it has two variables; one that is the really active value, and second that tells what value user wants to activate in the next turn change)
-> #43604
Somewhat overlapping (but not replacement) idea would be "PlayerFlag" requirement type. The benefit of PlayerFlag over PlayerState would be that requirement system would automatically gain access to each new flag added (no further implementation needed once we have the generic flag checking in place). Each PlayerState type would need to be implemented separately.
Attached patch that supports just "Barbarian" state. Other states to be added in separate tickets.
New version, with README.effects changes added.
Reply To ihnatus
* "ScienceNonNegative" - for combining with bulbs cost of tech stealth, disable tech stealth when you have yet to sort out what you have stolen before.
Maybe separate "MinAccumulatedBulbs" requirement type would be more generally usable?
Would be good to have a new req type for player-wide special states. Possible state names:
edit: Also, considering civil war effect for civ1/civ2 rulesets compatibility, we should make local ranged requirement "HasMore(Less)Cities" and "HasMore(Less)Score" (that tests if the capital conqueror passed as other_player is in range), and similar local range variant for "IsBarbarian" (maybe other values also should test other_player when applied as "Local")