Remove hard-coding for resource types from the engine

[GunChleoc]

In order to define message timeouts, we have an individual enum class member for each map resource in enum class::Message:

    kGeologistsCoal,
    kGeologistsGold,
    kGeologistsStones,
    kGeologistsIron,
    kGeologistsWater,

These are also hard-coded into Worker::run_geologist_find.

The reason for this hard-coding is to define separate message timeouts for the individual map resource types.

We could fix this by adding a new string parameter to Player::add_message_with_timeout add_message_with_timeout that will define the resources types to control the timeout.

[GunChleoc (gunchleoc)]

Fixed in build20-rc1