Inefficient streams in GoalSelector

The goal selector since 1.13 switched to expensive stream and lambda constructs which waste a lot of AI performance every tick.

We reverted our mod goal selector back to the original goal selector with for loops ( + some cleanup) to preserve this performance. (Link: https://github.com/ldtteam/minecolonies/blob/d1ebb34de4124bbba9f535c9e3b81b79639b89fd/src/api/java/com/minecolonies/api/entity/CustomGoalSelector.java)
A huge chunk of the vanilla entity performance sinks here atm.

Streams should not be used in performance critical places where the collection has only very few values (most entities have a very limited amount of goals).