Angela Schreiber created SLING-10769:
----------------------------------------
Summary: VaultPackageAssembler should not implement EntryHandler
Key: SLING-10769
URL: https://issues.apache.org/jira/browse/SLING-10769
Project: Sling
Issue Type: Improvement
Components: Content-Package to Feature Model Converter
Reporter: Angela Schreiber
[~kpauls], as stated with the PR for SLING-10754 i find it quite confusing the
{{VaultPackageAssembler}} implements {{EntryHandler}} and is used as default in
{{ContentPackage2FeatureModelConverter}}. even more so as the assembler is
accessed by other {{EntryHandler}} implementations to add package entries that
don't end up in repo-init.
while the effect is likely the same if the {{EntryHandler}} methods get moved
to a separate {{DefaultHandler}}, it would IMHO make the code easier to read.
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
