[shrinkwrap-issues] [JBoss JIRA] Updated: (SHRINKWRAP-268) MavenDependencyResolver.loadReposFromPom and loadDependenciesFromPom methods are missleading

MavenDependencyResolver.loadReposFromPom and loadDependenciesFromPom methods are missleading -------------------------------------------------------------------------------------------- Key: SHRINKWRAP-268 URL: https://issues.jboss.org/browse/SHRINKWRAP-268 Project: ShrinkWrap Issue Type: Enhancement Components: ext-resolver Affects Versions: 1.0.0-alpha-12 Reporter: Karel Piwko Assignee: Samuel Santos Current state: {code} loadReposFromPom(String file) {code} Loads a POM file and activates all repositories present there, moreover it loads all <dependencies> into cache so user can omit version in artifact specification. => should be renamed to loadMetadataFromPom(String file) or split to loadReposFromPom(String file), loadDependencyVersionsFromPom(String file) and aggregator loadMetadataFromPom(String file) {code} loadDepedenciesFromPom(String file) {code} Loads a POM file and adds all dependencies present there to the dependency chain. This means once the resolveXYZ() is called dependencies from POM are resolved. => should be renamed to includeDependenciesFromPom(String file) {code} loadDependenciesFromPom(String file, MavenResolutionFilter filter) {code} Does the same as previous, filter is not used there but during resolveXYZ() => should be removed