I want to keep the FactionRelation class because we will define change of the reputation on certain event there. also there might be ways to solve this loop better, but there are 10 factions at max, so for the moment I'll leave it like this, I think. This PR iss meant to be a skeleton with minimal implementation.