[SLING-8271] Improve AbstractResourceVisitor to allow to stop visiting child resources

• Type: Improvement
• Status: Resolved
• Priority: Major
• Resolution: Won't Fix
• Affects Version/s: API 2.20.0
• Component/s: API
• Assignee: Konrad Windszus
• Reporter: Konrad Windszus
• Created: 11/Feb/19 15:14
• Updated: 15/Jul/19 09:03
• Resolved: 15/Jul/19 09:03

Description

Currently the AbstractResourceVisitor (https://github.com/apache/sling-org-apache-sling-api/blob/master/src/main/java/org/apache/sling/api/resource/AbstractResourceVisitor.java) completely decouples the visit method from the actual resource traversal. That means it is currently rather hard to programmatically add a break condition which should lead to no further descending into child resources.

I propose to extend AbstractResourceVisitor by an additional visit method which is supposed to return a boolean value. Only if true is returned it will further descend into the children of the current resource. The default implementation should just return true to make this change backwards compatible.

Issue Links

• is related to: SLING-8581 Include Resource Filter in Sling Starter
• links to: GitHub Pull Request #12
• links to: GitHub Pull Request #13