Output_Penalty_Tile description in README.effects is incomplete
ddeanbrown Also, in future versions (maybe already in 3.2) this certainly should go to the ruleset, listed just as a requirement for the effect.
ddeanbrown ddeanbrown ddeanbrown ddeanbrown This patch works for S3_0 & S3_1. Maybe want it in S2_6 and/or S2_6_2? If so will need another patch because this one fails there.
Agree 3.2/master is a different situation with the new "CityStatus" requirement value "Celebration". Looks to me like we need another ticket for that.
ddeanbrown created new ticket #45236 "Output_Penalty_Tile changes for 3.2"
Please don't claim that celebratesize is (always) 3, like it was hardcoded.
Reply To cazfi
Please don't claim that celebratesize is (always) 3, like it was hardcoded.
Will you make an updated version?
Thought I'd update this myself, to still get it in 3.0.4, but I'm not sure if one should change it to refer to 'ruleset defined celebrate_size_limit' instead of '3', should we just leave the entire sentence out, or should we add similar sentence also to all other effects which applies only to celebrating cities or only to cities not celebrating.
Reply To cazfi
I'm not sure if one should change it to refer to 'ruleset defined celebrate_size_limit' instead of '3', should we just leave the entire sentence out, or should we add similar sentence also to all other effects which applies only to celebrating cities or only to cities not celebrating.
Attached patches just leave the sentence out. At least this is an improvement already, without it. With the delay of 3.0.4 this should still make it to it.
3.0.1, most rulesets.
In README.effects, the description text for Output_Penalty_Tile says -
"When a tile yields more output than amount, it gets a penalty of -1."
In the auto-generated help in the governments section for Anarchy and Despotism, it says -
"Each worked tile that gives more than 2 Food, Shield, or Trade will suffer a -1 penalty, unless the city working it is celebrating. (Cities below size 3 will not celebrate.)"
Output_Penalty_Tile description in README.effects should say something about the celebrating exception. I updated the wiki page "Editing Effects".