github-project icon indicating copy to clipboard operation
github-project copied to clipboard

Throw error in case multiple fields / field options are matched

Open gr2m opened this issue 3 years ago • 0 comments

Please avoid duplicates

  • [X] I checked all open bugs and none of them matched my problem.

Reproducible test case

No response

Please select the environment(s) that are relevant to your bug report

  • [X] Browsers
  • [X] Node
  • [X] Deno

Versions

1.3.1

What happened?

In theory, a field configured in the fields constructor option can be matched to multiple project fields when using the matchFieldName constructor option. The same is true for matching field options and the matchFieldOptionValue option.

I think right now the first match wins. But what we should do instead in both cases is to match all fields / field options respectively and verify that there is exactly one match. We throw an error if there is no match. We should also throw an error if there is more than one match

Would you be interested in contributing a fix?

  • [ ] yes

gr2m avatar Mar 15 '22 21:03 gr2m