Skip to:

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

REFind/NoCase(scope="ALL") returns an empty array instead an empty match set when no match is not found

Description

Set Application's regex engine to Java

this.regex.type = 'java'
this.regex.type = 'java'

Run this code
writedump(reFind("(f)(oo)", "bar", 1, false, "ALL"));
writedump(reFind("(f)(oo)", "bar", 1, false));

And you get an empty array, followed by an empty match set (len, match, pos)

Change this.regex.type to 'perl' and the same code returns an empty match set for each try.

Environment

Windows 10, IIS

Activity

Show:

Pothys - MitrahSoft 6 August 2021 at 15:00

Issue solved, Works file in fixed version

Michael Offner 6 August 2021 at 13:20

changed the return value in case we have no match and all records are expected

https://github.com/lucee/Lucee/commit/cbe5e018dab9ca095018a8d39b5244e4ec08a206

Zac Spitzer 3 August 2021 at 13:23

the above test case throws the following error (against 5.3.8.199)

[java] [script] Failed: test.tickets.LDEV3532.reFind() no matches with regex type java and returnsubexpressions = true ---> Expected [[{"match":[""],"len":[0],"pos":[0]}]] but received [[]] [java] [script] at [java] [script] C:\work\lucee\test\tickets\LDEV3532.cfc:11

https://github.com/lucee/Lucee/pull/1347/files#diff-83fbf94dd5594c4a87efa1274950e477834aaee80c4a5304d587720d73bac52eR11

Pothys - MitrahSoft 3 August 2021 at 06:54

I checked this ticket with the fixed version. Now refind works fine without errors. But still, one incompatibility is there with ACF.

In regex type java, refind with returnsubexpressions=true and scope all found no matches means returns empty array instead of an array of struct match set.

dump(reFind("(f)(oo)", "bar", 1, true,"all")); // With regex type java returns empty array instead of array of struct match set.

https://github.com/lucee/Lucee/blob/5.3/core/src/main/java/lucee/runtime/regex/JavaRegex.java#L90

Michael Offner 2 August 2021 at 15:07

Fixed

Details

Assignee

Reporter

Priority

Labels

Fix versions

New Issue warning screen

Before you create a new Issue, please post to the mailing list first https://dev.lucee.org

Once the issue has been verified, one of the Lucee team will ask you to file an issue

Affects versions

Created 21 May 2021 at 19:13
Updated 6 August 2021 at 15:00
Resolved 6 August 2021 at 15:00

Flag notifications