joerghoh opened a new pull request, #31: URL: https://github.com/apache/sling-org-apache-sling-servlets-resolver/pull/31
This is the 1st part of SLING-11558 (Remove resource -> path -> resource conversion). It replaces the existing ```LocationIterator``` (which is a iterator-based approach to resolve all potential resources for resource types) by a list-based approach; this avoids the complex logic using a lot of global variables, and makes the subsequently planned change easier to implement. All the tests from the ```LocationIteratorTest``` have been ported 1:1 to the new ```LocationCollectorTest``` and streamlined. Also the more complex logic in the ```getResourceSuperType``` methods in the ```LocationIterator``` has been mostly reused 'as-is' in the ```LocationCollector```. That means, the actual amount of re-implemented code is rather small. Though the logic has been changed a bit: ```getResourceSuperType``` will never return the baseResourceTypes (the default resource types), but these are always added outside of the recursion in ```collectPaths```. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
