[jira] [Comment Edited] (JCR-4339) Indexing configuration condition property seems to be broken

["Julian Reschke (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/JCR-4339?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16638129#comment-16638129 ] 

Julian Reschke edited comment on JCR-4339 at 5/7/19 9:09 AM:
-------------------------------------------------------------

trunk: (2.17.5) [r1836620|http://svn.apache.org/r1836620]
2.16: (2.16.4) [r1842802|http://svn.apache.org/r1842802]
2.14: [r1858835|http://svn.apache.org/r1858835]



was (Author: reschke):
trunk: [r1836620|http://svn.apache.org/r1836620]
2.16: [r1842802|http://svn.apache.org/r1842802]


> Indexing configuration condition property seems to be broken 
> -------------------------------------------------------------
>
>                 Key: JCR-4339
>                 URL: https://issues.apache.org/jira/browse/JCR-4339
>             Project: Jackrabbit Content Repository
>          Issue Type: Bug
>          Components: core
>    Affects Versions: 2.16.0
>            Reporter: Giacomo Lamonaco
>            Assignee: Claus Köll
>            Priority: Major
>             Fix For: 2.18, 2.17.5, 2.16.4, 2.14.7
>
>         Attachments: JCR-4339.patch
>
>
> Hi, while upgrading jackrabbit library from 2.14 to 2.16 we noticed couple of issues with the indexing configuration [1]. More specifically one of this issue is related to the condition property. 
> While using the latest library the value of the condition property is totally ignored, this means that the index-rule is applied to all nodes with the specified nodetype.
> We did some investigations and maybe we found the root cause in this commit [2]. More specifically, if you have a look at line 805 of the IndexingConfigurationImpl, the appliesTo method is always returning true: it looks not correct to me. 
> As a quick workaround we replace that line with:
> {code:java}
> return condition == null || condition.evaluate(state);{code}
> After this small fix the condition property in the index rules is working again as expected. Please let us if this is a bug or a misunderstanding from our side. 
> [1][https://wiki.apache.org/jackrabbit/IndexingConfiguration]
> [2][https://github.com/apache/jackrabbit/commit/a7804ede88623ceaa3b62f638f5384ebb97b39d4#diff-a79478132fc8ad1fe2bce5fa64f45310R805] 



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)