[
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
[
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
[
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
[
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
[
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
[
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
[
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.
[
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
[
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
[
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.
[
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
[
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
[
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
[
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
[
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
[
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
16 matches
Mail list logo