- 
                Notifications
    You must be signed in to change notification settings 
- Fork 7
Closed
Labels
code qualityCode readability or structure is improvedCode readability or structure is improved
Description
After refinements related to #969, the DataSource that is required for instanciating EntitySource is effectively only used in EntitySource#buildEntityData. Since all other methods are static or build upon this method, one can get confused easily here: When invoking one of these methods, does the output result from the input parameters, the field EntitySource.dataSource, or both?
Thus, I'm suggesting to make EntitySource completely static. Where data is needed from DataSource, either this object or a list of field-to-attribute sets can be passed as a parameter.
One can also discuss whether to also apply this transformation to subclasses of EntitySource, although this might be a bit more complex.
Metadata
Metadata
Assignees
Labels
code qualityCode readability or structure is improvedCode readability or structure is improved