Hi,
the AbstractEntityCollectionAdapter<T> has a couple of abstract methods to get
informations from the entity. I find it more clear for the implementer, if
those functions would be separated into an interface and all other methods
would work on the wrapped entry object instead:
interface WrappedEntry<T> {
String getId();
String getName();
String getTitle();
DateTime getUpdated();
boolean isMediaEntry();
List<Person> getAuthors(RequestContext request);
T getContent(RequestContext request);
}
The getEntry method of AbstractEntityCollectionAdapter would return
WrappedEntry<?>.
The WrappedEntry interface would have a Single Responsibility and the
AbstractEntityCollectionAdapter would have one responsibility less.
Best regards,
Thomas Koch, http://www.koch.ro
