Re: Syncope ignores braces "( )"

2016-09-02 Thread Francesco Chicchiriccò
- Il 2-set-16, alle 15:58, Colm O hEigeartaigh ha scritto: > Hehe ok will revisit next week! Cool! > On Fri, Sep 2, 2016 at 2:51 PM, Francesco Chicchiriccò < ilgro...@apache.org > > wrote: >> - Il 2-set-16, alle 15:29, Colm O hEigeartaigh < cohei...@apache.org

Re: Syncope ignores braces "( )"

2016-09-02 Thread Colm O hEigeartaigh
Hehe ok will revisit next week! Colm. On Fri, Sep 2, 2016 at 2:51 PM, Francesco Chicchiriccò wrote: > - Il 2-set-16, alle 15:29, Colm O hEigeartaigh > ha scritto: > > I updated the patch to also handle "AND". Yes it solves the issue. > > > Now

Re: Syncope ignores braces "( )"

2016-09-02 Thread Francesco Chicchiriccò
- Il 2-set-16, alle 15:29, Colm O hEigeartaigh ha scritto: > I updated the patch to also handle "AND". Yes it solves the issue. Now only a unit test in AnySearchTest and a correspondent integration test in SearchITCase are missing, then... ;-) > On Fri, Sep 2, 2016

Re: Syncope ignores braces "( )"

2016-09-02 Thread Colm O hEigeartaigh
I updated the patch to also handle "AND". Yes it solves the issue. Colm. On Fri, Sep 2, 2016 at 1:38 PM, Francesco Chicchiriccò wrote: > - Il 2-set-16, alle 14:34, Colm O hEigeartaigh > ha scritto: > > Yep. I've attached a proposed patch to

Re: Syncope ignores braces "( )"

2016-09-02 Thread Francesco Chicchiriccò
- Il 2-set-16, alle 14:34, Colm O hEigeartaigh ha scritto: > Yep. I've attached a proposed patch to SYNCOPE-929. It simply makes sure that > OR > expressions are encapsulated in parentheses. It feels like a bit of a hack > though :-) I've taken a look at your patch

Re: Syncope ignores braces "( )"

2016-09-02 Thread Colm O hEigeartaigh
Yep. I've attached a proposed patch to SYNCOPE-929. It simply makes sure that OR expressions are encapsulated in parentheses. It feels like a bit of a hack though :-) Colm. On Fri, Sep 2, 2016 at 12:59 PM, Francesco Chicchiriccò wrote: > - Il 2-set-16, alle 13:33,

Re: Syncope ignores braces "( )"

2016-09-02 Thread Francesco Chicchiriccò
- Il 2-set-16, alle 13:33, Colm O hEigeartaigh ha scritto: > Hi Francesco, > I've dug into this a bit deeper. The logic in SearchCondVisitor is fine I > think. > The problem is in JPAAnySearchDao when the SearchCond is converted into a > String. Using the example I

Re: Syncope ignores braces "( )"

2016-09-02 Thread Colm O hEigeartaigh
Hi Francesco, I've dug into this a bit deeper. The logic in SearchCondVisitor is fine I think. The problem is in JPAAnySearchDao when the SearchCond is converted into a String. Using the example I gave, "(city==dublin,state==abc);country==ireland", yields the following String: SELECT DISTINCT

AW: Role Synchronization with DB Backend possible?

2016-09-02 Thread Jan Bernhardt
Many thanks for your feedback on this mailing list as well as on my blog!! Looks like that I have to learn some more about the Scripted SQL connectors. Best regards Jan Von: Marco Di Sabatino Di Diodoro [mailto:marco.disabat...@tirasa.net] Gesendet: Donnerstag, 1. September 2016 17:17 An:

Re: Syncope ignores braces "( )"

2016-09-02 Thread Francesco Chicchiriccò
So you are not able to access https://issues.apache.org/jira/secure/CreateIssue!default.jspa once authenticated? - Il 2-set-16, alle 10:56, Iurii Smyrnov ha scritto: > Hi all, > actually I just can create Service Desk Request in the Jira >

Re: Syncope ignores braces "( )"

2016-09-02 Thread Iurii Smyrnov
Hi all, actually I just can create Service Desk Request in the Jira https://issues.apache.org/jira/secure/Dashboard.jspa , seems I can't create Issues. Could you please help and create the issue for this case ? Thanks [cid:f0c5b3b4-5db7-4bcb-b78a-8fb390ab73cf] Thanks, Iurii Smyrnov