I know that @stefankoegl is very keen on his `dedent` ;) but these two things are not the same. The new suggestion adds a lot of spaces into the xml string because dedent does not remove all initial spaces,but only those which are common (ie a total of 4 in this case). This means that not only are more bytes are sent over the network, but also it is a lot harder to get unit tests right, because we will need to make sure that test strings are built with exactly the same number of spaces.  

If we don't use `join` to join the strings together, we could wrap in () and rely on implicit string joining