You are viewing a plain text version of this content. The canonical link for it is here.
Posted to oak-issues@jackrabbit.apache.org by "Vikas Saurabh (JIRA)" <ji...@apache.org> on 2017/12/17 09:59:01 UTC

[jira] [Updated] (OAK-7070) rep:excerpt selector broken as regression of OAK-6750

     [ https://issues.apache.org/jira/browse/OAK-7070?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Vikas Saurabh updated OAK-7070:
-------------------------------
    Description: 
The change made here:
https://github.com/apache/jackrabbit-oak/commit/00c94b71293abcae6d76bb162c3f55c7d09b702e#diff-d4bdf443c61f24b634f33aab607e2114

breaks the logic in line 676:

{{else if (oakPropertyName.equals(QueryConstants.REP_EXCERPT + "("))}}

This statement doesn't make much sense considering a query like {{select \[rep:excerpt] from \[test:Page] as page where contains(\*, 'term\*')}} or even {{select \[rep:excerpt(text)] from \[test:Page] as page where contains(page.\[text], 'term\*')}}

  was:
The change made here:
https://github.com/apache/jackrabbit-oak/commit/00c94b71293abcae6d76bb162c3f55c7d09b702e#diff-d4bdf443c61f24b634f33aab607e2114

breaks the logic in line 676:

{{else if (oakPropertyName.equals(QueryConstants.REP_EXCERPT + "("))}}

This statement doesn't make much sense considering a query like {{select \[rep:excerpt] from \[test:Page] as page where contains(*, 'term*')}} or even {{select \[rep:excerpt(text)] from \[test:Page] as page where contains(page.\[text], 'term*')}}


> rep:excerpt selector broken as regression of OAK-6750
> -----------------------------------------------------
>
>                 Key: OAK-7070
>                 URL: https://issues.apache.org/jira/browse/OAK-7070
>             Project: Jackrabbit Oak
>          Issue Type: Bug
>    Affects Versions: 1.8
>            Reporter: Dirk Rudolph
>
> The change made here:
> https://github.com/apache/jackrabbit-oak/commit/00c94b71293abcae6d76bb162c3f55c7d09b702e#diff-d4bdf443c61f24b634f33aab607e2114
> breaks the logic in line 676:
> {{else if (oakPropertyName.equals(QueryConstants.REP_EXCERPT + "("))}}
> This statement doesn't make much sense considering a query like {{select \[rep:excerpt] from \[test:Page] as page where contains(\*, 'term\*')}} or even {{select \[rep:excerpt(text)] from \[test:Page] as page where contains(page.\[text], 'term\*')}}



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)