Skip to:

  1. Skip to Jira Navigation
  2. Skip to Side Navigation
  3. Skip to Main Content

Searching by Close Parens Generates Error

Description

Steps to repro:

  1. Go to Users

  2. Entering ")" into the search criteria and hitting Search

Expected: Return results containing this character or show "No results found for ")". Please check your spelling and filters."

Actual: Generates an error: ERROR: in module @folio/users, operation GET on resource 'records' failed, saying: Internal Server Error, Please contact System Administrator or try again

Additional info: Entering "^" retrieves all records. Not sure why this is, but I guess it's okay.

Notes

The query for ")" result in the following CQL query: (title=")" or contributors adj "\"name\": \")\"" or identifiers adj "\"value\": \")\"") sortby title which seems OK. The error returned by the server seems to be coming from Postgres.

ErrorMessage(fields=Map(Line -> 208, File -> regexp.c, SQLSTATE -> 2201B, Routine -> RE_compile_and_cache, V -> ERROR, Message -> invalid regular expression: parentheses () not balanced, Severity -> ERROR))

we should verify if the problem is fixes in RMB 25.

CSP Request Details

None

CSP Rejection Details

None

Potential Workaround

None

Attachments

1
  • 07 Jan 2019, 08:01 PM

Checklist

hide

TestRail: Results

Activity

Show:

Cate Boerema July 8, 2019 at 3:12 PM

Yay - searching Users by close parens no longer generates an error!

I did notice this bug, though:

  1. Edit a user

  2. Add a ) to the end of their first name

  3. Save user

  4. Search for )

Expected: System would return the user with the close parens in the name

Actual: No users returned

I'll file a separate bug for this because it seems different.

Adam Dickmeiss June 5, 2019 at 1:53 PM
Edited

I added a unit test to /cql2pgjson-java v4.0.0 with this and it failed with the error as shown above.

I then added a unit test to cql2pgjson of RMB 25. And that works fine (0 hits).

It illustrates that it is important to update to RMB 25 when out.

Cate Boerema April 8, 2019 at 11:12 AM

Linking to https://folio-org.atlassian.net/browse/CQLPG-96#icft=CQLPG-96, as Jakub thinks that might address this one.

Cate Boerema January 24, 2019 at 1:31 PM

Interesting. I guess I'm not opposed to having "^" retrieve all records (or am I missing something?) I think I'll turn this into a bug about the error

Done

Details

Assignee

Reporter

Labels

Priority

Sprint

Development Team

Core: Platform

TestRail: Cases

Open TestRail: Cases

TestRail: Runs

Open TestRail: Runs
Created January 7, 2019 at 8:01 PM
Updated August 20, 2019 at 8:21 AM
Resolved July 8, 2019 at 3:12 PM
TestRail: Cases
TestRail: Runs

Flag notifications