I'm afraid I don't understand the benefit of this change

it doesn't make sense to me to have this logic on a Mapping class

this class represents one Mapping instance, the logic referenced here works on a whole collection of these instances

am i missing something?