ConfigMe icon indicating copy to clipboard operation
ConfigMe copied to clipboard

Do not split dot paths by default / remove config option

Open ljacqu opened this issue 4 years ago • 0 comments

For #214 it was necessary to introduce a configuration to not split dots in paths. I think the feature should be removed altogether for ConfigMe 2.0 because I can't really think of useful cases where . in a YAML property should be split. It would keep the code base easier to remove the handling that has had to be introduced for that new YAML file configuration option. At the same time I didn't want to change it in 1.3.0 in case someone does depend on it...

Within this issue we should also see whether it makes sense to support . in paths more fully: methods that define paths (like on Property) or that retrieve paths (like on YamlFileReader) automatically treat dots as to mean path elements. I think that's nice handling but maybe there should be a way to retrieve or define paths with dots in them, although I still think outside of dynamic content like the example in #214 there's not really a good reason to do so.

One idea is to escape dots by \. (and likewise, \\ would escape a backslash so we can still represent any string we like) but not sure implementation-wise how to approach that at the moment.

ljacqu avatar Oct 03 '21 14:10 ljacqu