Inconsistent translation

[wdherndon]

In an attempt to fix a problem with SpecifiedTradePaymentTerms, I tried to implement my own version and found I could not get an HTML-linefeed ("
") in my output because of inconsistent translation:

If my code is:

result.AddTradePaymentTerms("line1\nline2", dueDate);

The result is:

  <ram:SpecifiedTradePaymentTerms>
    <ram:Description>line1

line2</ram:Description>

If my code is:

result.AddTradePaymentTerms("line1 line2", dueDate);

The result is:

  <ram:SpecifiedTradePaymentTerms>
    <ram:Description>line1&amp;#10;line2</ram:Description>

In neither case could I get the desired:

  <ram:SpecifiedTradePaymentTerms>
    <ram:Description>line1&#10;line2</ram:Description>

[stephanstapel]

happy if you could dive deeper into this and suggest how it should work.

[wdherndon]

I think it should do one of the following:

(1) translate the linefeed to html like it does the ampersand: "\n" -> "&#10;"
or
(2) not translate either the linefeed or the ampersand: "&" -> "&" rather than "&" -> "&amp;"
or
(3) both: add a bool to the function specifying whether the input is already html-translated or not. If the caller specifies it is not already translated, you then translate it (with "\n" -> "&#10;").

I think option (3) is the best with it defaulting to (1), offering maximum flexibility, default to the safe option, and maximal backwards compatibility.