Both the Patient model and the Entries model are being held responsible for knowing about the different types; this might be better if you had an array on the Entries class defining the list of types, and then referenced it here via `for type in Thorax.Collections.Entries::types`. That way, only one class would be responsible for knowing the different types.