[jira] [Commented] (SLING-8271) Improve AbstractResourceVisitor to allow to stop visiting child resources

2019-07-15 Thread Konrad Windszus (JIRA)
[ https://issues.apache.org/jira/browse/SLING-8271?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16884961#comment-16884961 ] Konrad Windszus commented on SLING-8271: I updated the javadoc accordingly in

[jira] [Commented] (SLING-8271) Improve AbstractResourceVisitor to allow to stop visiting child resources

2019-07-12 Thread Konrad Windszus (JIRA)
[ https://issues.apache.org/jira/browse/SLING-8271?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16883570#comment-16883570 ] Konrad Windszus commented on SLING-8271: Actually after having a second thought I am not sure if

[jira] [Commented] (SLING-8271) Improve AbstractResourceVisitor to allow to stop visiting child resources

2019-07-11 Thread Konrad Windszus (JIRA)
[ https://issues.apache.org/jira/browse/SLING-8271?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16883180#comment-16883180 ] Konrad Windszus commented on SLING-8271: [~jebailey] Thanks, makes sense. I will even add a link

[jira] [Commented] (SLING-8271) Improve AbstractResourceVisitor to allow to stop visiting child resources

2019-07-11 Thread Jason E Bailey (JIRA)
[ https://issues.apache.org/jira/browse/SLING-8271?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16883178#comment-16883178 ] Jason E Bailey commented on SLING-8271: --- Before pulling this, I think this is a bit limited in

[jira] [Commented] (SLING-8271) Improve AbstractResourceVisitor to allow to stop visiting child resources

2019-07-11 Thread Konrad Windszus (JIRA)
[ https://issues.apache.org/jira/browse/SLING-8271?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16883090#comment-16883090 ] Konrad Windszus commented on SLING-8271: I provided a new PR in

[jira] [Commented] (SLING-8271) Improve AbstractResourceVisitor to allow to stop visiting child resources

2019-02-20 Thread Robert Munteanu (JIRA)
[ https://issues.apache.org/jira/browse/SLING-8271?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16772858#comment-16772858 ] Robert Munteanu commented on SLING-8271: {quote}I don't think we need to deprecate the existing

[jira] [Commented] (SLING-8271) Improve AbstractResourceVisitor to allow to stop visiting child resources

2019-02-20 Thread Carsten Ziegeler (JIRA)
[ https://issues.apache.org/jira/browse/SLING-8271?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16772857#comment-16772857 ] Carsten Ziegeler commented on SLING-8271: - I don't think we need to deprecate the existing one.

[jira] [Commented] (SLING-8271) Improve AbstractResourceVisitor to allow to stop visiting child resources

2019-02-20 Thread Konrad Windszus (JIRA)
[ https://issues.apache.org/jira/browse/SLING-8271?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16772764#comment-16772764 ] Konrad Windszus commented on SLING-8271: After having a second thought I think it would be good

[jira] [Commented] (SLING-8271) Improve AbstractResourceVisitor to allow to stop visiting child resources

2019-02-12 Thread Konrad Windszus (JIRA)
[ https://issues.apache.org/jira/browse/SLING-8271?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16766138#comment-16766138 ] Konrad Windszus commented on SLING-8271: In most use cases I have in mind the same logic is

[jira] [Commented] (SLING-8271) Improve AbstractResourceVisitor to allow to stop visiting child resources

2019-02-12 Thread Julian Sedding (JIRA)
[ https://issues.apache.org/jira/browse/SLING-8271?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16766103#comment-16766103 ] Julian Sedding commented on SLING-8271: --- I see this is already committed. But not yet released.

[jira] [Commented] (SLING-8271) Improve AbstractResourceVisitor to allow to stop visiting child resources

2019-02-12 Thread Robert Munteanu (JIRA)
[ https://issues.apache.org/jira/browse/SLING-8271?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16765825#comment-16765825 ] Robert Munteanu commented on SLING-8271: Sure, works for me. > Improve AbstractResourceVisitor

[jira] [Commented] (SLING-8271) Improve AbstractResourceVisitor to allow to stop visiting child resources

2019-02-12 Thread Konrad Windszus (JIRA)
[ https://issues.apache.org/jira/browse/SLING-8271?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16765824#comment-16765824 ] Konrad Windszus commented on SLING-8271: [~rombert] {{maybeVisit}} sounds like you can prevent

[jira] [Commented] (SLING-8271) Improve AbstractResourceVisitor to allow to stop visiting child resources

2019-02-11 Thread Robert Munteanu (JIRA)
[ https://issues.apache.org/jira/browse/SLING-8271?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16765453#comment-16765453 ] Robert Munteanu commented on SLING-8271: How about {{maybeVisit}} or {{checkedVisit}}? > Improve

[jira] [Commented] (SLING-8271) Improve AbstractResourceVisitor to allow to stop visiting child resources

2019-02-11 Thread Carsten Ziegeler (JIRA)
[ https://issues.apache.org/jira/browse/SLING-8271?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16765145#comment-16765145 ] Carsten Ziegeler commented on SLING-8271: - [~kwin] Proposal looks good to me, the method name is

[jira] [Commented] (SLING-8271) Improve AbstractResourceVisitor to allow to stop visiting child resources

2019-02-11 Thread Konrad Windszus (JIRA)
[ https://issues.apache.org/jira/browse/SLING-8271?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16765114#comment-16765114 ] Konrad Windszus commented on SLING-8271: My proposal is in

[jira] [Commented] (SLING-8271) Improve AbstractResourceVisitor to allow to stop visiting child resources

2019-02-11 Thread Konrad Windszus (JIRA)
[ https://issues.apache.org/jira/browse/SLING-8271?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16765061#comment-16765061 ] Konrad Windszus commented on SLING-8271: [~cziegeler] Since you added the code originally WDYT