Details
-
Improvement
-
Status: Closed
-
Normal
-
Resolution: Outdated
-
None
-
None
-
None
Description
The current implementation of org.onehippo.taxonomy.impl.CategoryImpl.getInfos() returns a lazy map, which does not make sense at all now. Because Category already has translations map. It doesn't have to use lazy map at all now when it already has initialized a new map named 'translations'.
I think #getInfos() can return 'translations' member simply because it is initialized in the constructor already, or Collections.unmodifiableMap(translations).