We shouldn't be hitting the database each time here. Why do we need to iterate over every field type, then every module_unit? the module_units have the content type information on them already.