Skip to content

E3-1652: Fix db029

Simon Rose requested to merge fix_db029 into main

Some people, when confronted with a problem, think "I know, I'll use regular expressions." Now they have two problems.

Anyhow, we can probably squash these commits in the end, but there are a few outstanding points:

  • The solution here checks for brackets more stringently i.e. only match field( ..., ... ) instead of arbitrary brackets. Is that correct?
  • We do not have any tests or checks on the record name! This could fit in here, but it might be better as a separate test case/requirement.
  • I did my best to see what other fields could legit have parentheses in the values, which you can see in the commit history.

Merge request reports