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

angela commented on OAK-6290:
-----------------------------

[~tmueller], argh.... thanks for spotting... in fact it needs to be {{if 
(selectorNames.length >= 1) {}} because it seems that there are cases where 
there are >1 selectorNames but {{Result.getColumnSelectorNames}} just returns 
1, in which case i still ran into the {{IllegalArgumentException}}... i don't 
know the very details on why that happens but the test commited with 1797008 
illustrates it.

> UserQueryManager.findAuthorizables fails with IllegalArgumentException when 
> there are multiple selectors
> --------------------------------------------------------------------------------------------------------
>
>                 Key: OAK-6290
>                 URL: https://issues.apache.org/jira/browse/OAK-6290
>             Project: Jackrabbit Oak
>          Issue Type: Bug
>          Components: core
>            Reporter: angela
>            Assignee: angela
>         Attachments: OAK-6290.patch
>
>
> while writing test cases i managed to create a query that results in 
> {{IllegalArgumentException}} because there were multiple selectors but 
> {{ResultRowToAuthorizable}} would call {{ResultRow.getPath()}}, without 
> specifying the selector.
> [~tmueller] instructed me to make use of {{Result.getColumnSelectorNames}} to 
> spot the cases where there are multiple selector names.
> [~alex.parvulescu], i will attach a patch and would appreciate if you could 
> take a look.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

Reply via email to