You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@jackrabbit.apache.org by "Giacomo Lamonaco (JIRA)" <ji...@apache.org> on 2018/07/22 18:19:00 UTC

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

Giacomo Lamonaco created JCR-4339:
-------------------------------------

             Summary: 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
    Affects Versions: 2.16.0
            Reporter: Giacomo Lamonaco


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)