- Issue created by @joachim
- First commit to issue fork.
- 🇮🇳India ajinkya45 Pune
ajinkya45 → changed the visibility of the branch 3414703-improve-the-exception to hidden.
- Merge request !12676Issue #3414703: Improve exception message when saving an array into a config string → (Open) created by Unnamed author
- 🇮🇳India ajinkya45 Pune
Proposed a fix by adding a type check in TypedConfigManager::buildDataDefinition(). Now throws a clear error if an array is saved into a string-typed config property.
- 🇺🇸United States smustgrave
Seems to be doing more then updating a message, seems like a new check that should have test coverage.
- 🇧🇪Belgium borisson_ Mechelen, 🇧🇪
I don't agree with the route taken in the current MR as well. It seems like this is very small scope, instead of hard checking against the string type can we check that it's not a type that's iterable, such as map or sequence?
- 🇮🇳India ajinkya45 Pune
Thanks for the suggestion! I will update the check to validate against iterables like map or sequence instead of strictly arrays and include a test to cover the behavior.