I don't understand the difference between this and `public List<ModuleMetadata> findAll(String moduleIdToUse)` (apart from the (cheated) pagination stuff)

This name should be kept, and the ambiguous `findAll` should be dropped IMHO.