Parameters not working when using complex filters

[sebastian-jansson]

Description of the Issue

When adding parameters to certain filter expressions (Whether that's through templates or direct ones), the parameter gets blank space characters added to it, rendering it useless as jailer won't recognize it as a parameter anymore , with a failed export as the result.

I noticed that this bug is only visible when opening the expression editor (Apart from the error log output), viewing it "inline" or directly displays the parameter correctly:

I attempted many different combinations, (My real filter expressions has lots of concatenation operators), but even simple ones as the example above fails. The only expressions I could get consistent working results with are ones where the full expression is solely the parameter name itself.

Other things that did not help:

• Adding the parameter from the sidebar instead of typing it out.
• Only using the parameter once instead of twice as in my full expression.
• Different naming standards did not resolve the issue. In fact, it happens for the default "old-value" parameter as well.

Expected Behavior

At save, runtime & edit, one expects the parameter to stay the way it was written out when closing the editor.

Actual Behavior

Parameters are changed into: $ { to_plc_id } from: ${to_plc_id}.

Debug Information

Windows 10 1909
Jailer 10.3.5

[sebastian-jansson]

Forgot to add, when replacing the parameter in the filter expression with the actual value I'm assigning the parameter, I can export the data successfully.

[Wisser]

Thanks for the report. I will have a look at it as soon as possible.

[sebastian-jansson]

Thanks for the report. I will have a look at it as soon as possible.

Many thanks Wisser!

[Wisser]

The reason for this was that the SQL formatter unfortunately inserted spaces in awkward places.
I have corrected this and published the fix in release 10.4.

[sebastian-jansson]

That was quick! Thanks a bunch., I'll download 10.4 and give it a try :)

I'm already loving this tool, has saved me a bunch of time for moving data around.