Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 0.9
    • Component/s: core, query
    • Labels:
      None

      Description

      The SQL2 parser doesn't correctly decode the path constraints, which can result in queries with no result.

      The fix is to apply a ISO9075#decode similar to the way Jackrabbit did it.

        Activity

        Hide
        Alex Parvulescu added a comment -

        Bulk close for the 0.9 release

        Show
        Alex Parvulescu added a comment - Bulk close for the 0.9 release
        Hide
        Thomas Mueller added a comment -

        Revision 1519928: Also use decode for //x and //element(x, *)
        Revision 1519930: Test case

        Show
        Thomas Mueller added a comment - Revision 1519928: Also use decode for //x and //element(x, *) Revision 1519930: Test case
        Hide
        Thomas Mueller added a comment -

        I was trying to write a test case. It works for queries of the form: /jcr:root/hel_x006c_o/*, which is converted to the SQL-2 query

        select [jcr:path], [jcr:score], * 
        from [nt:base] as a 
        where ischildnode(a, '/hel_x006c_o')
        

        however it doesn't work for queries of the form: //hel_x006c_o, which is converted to

        select [jcr:path], [jcr:score], * 
        from [nt:base] as a 
        where name(a) = 'hel_x006c_o'
        

        I will try to come up with a solution that covers those cases as well.

        Show
        Thomas Mueller added a comment - I was trying to write a test case. It works for queries of the form: /jcr:root/hel_x006c_o/* , which is converted to the SQL-2 query select [jcr:path], [jcr:score], * from [nt:base] as a where ischildnode(a, '/hel_x006c_o') however it doesn't work for queries of the form: //hel_x006c_o , which is converted to select [jcr:path], [jcr:score], * from [nt:base] as a where name(a) = 'hel_x006c_o' I will try to come up with a solution that covers those cases as well.
        Hide
        Alex Parvulescu added a comment -

        fixed with rev 1519474.

        Show
        Alex Parvulescu added a comment - fixed with rev 1519474.

          People

          • Assignee:
            Alex Parvulescu
            Reporter:
            Alex Parvulescu
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development