Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: current
    • Fix Version/s: milestone 1
    • Component/s: spec
    • Labels:
      None
    • Environment:

      Operating System: All
      Platform: All

    • Issuezilla Id:
      349

      Description

      Section 4.6.9 'Query Object'. Clarify what Query.getLanguage() returns if the
      query was created using JQOM. Query.getStatement() will return a string
      representation in JCR-SQL2. Should getLanguage() return JCR-SQL2 or JCR-JQOM?

      I think it should return JCR-SQL2 because that's what getStatement() will
      return. If we specified it that way then the constant Query.JCR-JQOM is probably
      useless and can be removed.

        Issue Links

          Activity

          Hide
          dpitfiel added a comment -

          For a JQOM query, the jcr:language property of nt:query should return JCR-JQOM
          to indicate that QueryManager.getQuery(Node) will return a QueryObjectModel not
          a Query or PreparedQuery.

          For consistency, Query.getLanguage should also return JCR-JQOM.

          Show
          dpitfiel added a comment - For a JQOM query, the jcr:language property of nt:query should return JCR-JQOM to indicate that QueryManager.getQuery(Node) will return a QueryObjectModel not a Query or PreparedQuery. For consistency, Query.getLanguage should also return JCR-JQOM.
          Hide
          mreutegg added a comment -

          You are right, I forgot about QueryManager.getQuery(Node).

          Show
          mreutegg added a comment - You are right, I forgot about QueryManager.getQuery(Node).
          Hide
          Peeter Piegaze added a comment -

          Fixed as per David Pitfield's suggestion.

          Show
          Peeter Piegaze added a comment - Fixed as per David Pitfield's suggestion.

            People

            • Assignee:
              jsr-283-issues
              Reporter:
              mreutegg
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: