Problem: rice as a food subtype

[Langbart]

Gurken Khan pointed out a rice issue

Yeah, no. I hereby petition rice to count as vegetarian! ;)

• ​Alpha 26 Pre-release/Release Candidate Build Testing (9/Jun/22)

​food.json lists rice as a subtype

Line
6	"subtypes": {
7	"fish": "Fish",
8	"fruit": "Fruit",
9	"grain": "Grain",
10	"meat": "Meat",
11	"rice": "Rice"
12	},
13	"properties": [
14	"barterable",

The issue was pointed out by Nescio0 with commit ​​498061f (16/Jul/20).

Don't introduce rice resource subtype:

• This is not compatible with the public mod.
• Making it only available to Han China means their fields are useless when captured by other civs, whereas the reverse is not true.
• food.grain does not represent only wheat, but also other grains:cereals (e.g. barley, oats, rye, spelt, sorghum, millet, maize, and yes, rice) ...

In a later commit (​​de1bd77 (25/Sep/21)) it was reintroduced and later folded into the main game.

related discussion

IRC 0ad-dev ​16/May/22

12:09 < marder> do we really need a food.rice resource for the han?
[...]
13:09 <@Stan> @tell marder it was justified since rice fields are smaller iirc

forum threads

• ​Addition of Han Chinese to 0AD (20/Apr/21)
• ​Fixing Chinese Han (20/Feb/22)