Wheat farms stuff trouble.

[dalerank]

Even though they are fully staffed, the popup says that they are not fully staffed.

[Gustavo Alves]

Farms productivity depends on the number of meadows it covers.

math::Percent Farm::productivity() const
{
  return Factory::productivity() * _d->meadowsCoverage;
}

Staff description is being generated upon this value.

std::string WorkingBuildingHelper::productivity2desc( WorkingBuildingPtr w, const std::string& prefix )
{
  std::string factoryType = w->info().typeName();
  unsigned int workKoeff = w->productivity() * productivityDescriptionCount / 100;

  workKoeff = math::clamp( workKoeff, 0u, productivityDescriptionCount-1 );

  if( prefix.empty() )
  {
    return fmt::format( "##{0}_{1}##",
                        factoryType, productivityDescription[ workKoeff ] );
  }
  else
  {
    return fmt::format( "##{0}_{1}_{2}##",
                        factoryType, prefix, productivityDescription[ workKoeff ] );
  }
}

I can fix this getting the number of workers directly onto the method above. But I don't know if this is the right approach.

unsigned int workKoeff = math::percentage(w->numberWorkers(), w->maximumWorkers()) * productivityDescriptionCount / 100;

Perhaps, info box should display additional information about this meadow related productivity.

[Gustavo Alves]

I submitted a pull request #132 fixing this issue. But, as I mentioned above I don't know if I'm doing it the right way.

[dalerank]

Let me check it, pls... @GSTVAC May be disable this rule?

[Gustavo Alves]

I don't know if Caesar III has this rule. I don't have it installed here to compare.

[dalerank]

fixed #1161. farm now correct check productivity on build

→ <<cset 5ff57405f1e3>>