Uploaded image for project: 'ui-inventory'
  1. ui-inventory
  2. UIIN-589

Mask double quotes and backslash (searching by quote or backslash generates error)

    XMLWordPrintable

    Details

    • Template:
    • Sprint:
      Core: F - Sprint 81, Core: F - Sprint 82, Core: F - Sprint 83
    • Story Points:
      5
    • Development Team:
      Prokopovych

      Description

      Select "Subject" for the search type (or any search type - they all seem to generate an error when quotes or backslash are included)
      When entering
      \"
      or
      "
      into the search box there should be no error message (the backend will ignore these characters when the = operator for a full-text search is used, for an exact search using == correct masking is needed).

      Actual result for the second search term: Error popup: ERROR: in module @folio/inventory, operation GET on resource 'records' failed, saying: org.z3950.zing.cql.cql2pgjson.QueryValidationException: org.z3950.zing.cql.CQLParseException: expected ')', got EOF

      These are the query strings (after urldecoding) sent from front-end to back-end:

      (subjects=""") sortby title
      (subjects="\"") sortby title
      

      They should be

      (subjects="\"") sortby title
      (subjects="\\\"") sortby title
      

      For CQL quoted string masking see https://www.loc.gov/standards/sru/cql/spec.html#bnf

        TestRail: Results

          Attachments

            Issue Links

              Activity

                People

                Assignee:
                zburke Zak_Burke
                Reporter:
                julianladisch Julian Ladisch
                Tester Assignee:
                Charlotte Whitt Charlotte Whitt
                Votes:
                0 Vote for this issue
                Watchers:
                3 Start watching this issue

                  Dates

                  Created:
                  Updated:
                  Resolved:

                    TestRail: Runs

                      TestRail: Cases