While working on a mod I found this in the WorkGiver_GrowerHarvest core class:
This results in a useless check for room == room, so I guess the 'c' in the if should be 'c2'.
(As it is like that "correct" way for example in the WorkGiver_CleanFilth.)
This results in the harvesting colonist maybe walking a long way to another room, when there is a crop "nearer" to harvest.
See screenshot: Jenni will harvest 1 - 2 - 3 when undrafted and not 1 - 3 - 2.
[attachment deleted due to age]
Code Select
Room room = c.GetRoom(map, RegionType.Set_Passable);
float num = 0f;
for (int i = 0; i < 40; i++)
{
IntVec3 c2 = c + GenRadial.RadialPattern[i];
if (c.GetRoom(map, RegionType.Set_Passable) == room)
{
This results in a useless check for room == room, so I guess the 'c' in the if should be 'c2'.
(As it is like that "correct" way for example in the WorkGiver_CleanFilth.)
This results in the harvesting colonist maybe walking a long way to another room, when there is a crop "nearer" to harvest.
See screenshot: Jenni will harvest 1 - 2 - 3 when undrafted and not 1 - 3 - 2.
[attachment deleted due to age]


