- ๐ฌ๐งUnited Kingdom steven jones
Thanks everyone for your patience, I'll note that I'm committing a partial version of what was in MR!75, so if you're using all the features of this MR/patch then you might want to stick with that until all the sub issues of โจ [meta]Make XML output more configurable Active are in, and then upgrade to the version of VDE that has those in.
-
steven jones โ
committed dabf0613 on 8.x-1.x
Issue #2886357 by xperd, steven jones, joelpittet, upchuk, nikolay...
-
steven jones โ
committed dabf0613 on 8.x-1.x
- ๐ฌ๐งUnited Kingdom steven jones
@joelpittet yeah, agreed, I think it should almost always be in there, but I don't want to break something random in someone's export, so I think I'll punt that change to โจ Allow specifying the XML encoding property Active and maybe we can make it the default for new installs or something like that.
You happy with the code otherwise?
- ๐จ๐ฆCanada joelpittet Vancouver
I recommend keeping encoding="UTF-8" in there โ it ensures correct character decoding, avoids cross-platform inconsistencies, and even if XML parsers are required to assume UTF-8 by default, itโs still best practice to include it explicitly.