Illegal characters in parameter/container names

[Yuri05]

At the moment some illegal characters can be included into parameter names, which might lead to problems, e.g.

• Double quotes
• Pipes
• ... (anything else?)

Naming check should be extended, so that those characters will be forbidden.

[msevestre]

in MoBi there is absolutly no Name check

WHAT?!?

[PavelBal]

in MoBi there is absolutly no Name check

But I think that a formula using such an "alias" would not work?

I can create a parameter 123, but in the formula the alias must be _123, otherwise an error in the equation field is shown.

[Yuri05]

@PavelBal For a formula it's true, but it will crash elsewhere.
Try to create a simulation in this project: P1P2.zip 😉

[msevestre]

Should not we simply disallow usage of all weird stuff like in PKSim and do if for ALL entities created from now on.
?

[Yuri05]

Should not we simply disallow usage of all weird stuff like in PKSim and do if for ALL entities created from now on?

Would make sense :)

[msevestre]

The base class in core already is like this
https://github.com/Open-Systems-Pharmacology/OSPSuite.Core/blob/develop/src/OSPSuite.Presentation/DTO/NonEmptyNameDTO.cs#L26

Cleary Mobi does not use this