[moin-user] ||<tableclass="=?utf-8?q?=E2=80=A6?="> processing broken in Moin 1.9.9

[Paul Boddie]

On Saturday 25. November 2017 14.37.32 lists at hcf.yourweb.de wrote:
> I found a bug in Moin 1.9.9, but my account for the moinmo.in wiki has been
> deleted and I cannot register a new account as I don't know the magic
> password. So I am posting this here and hope some developer sees this. :)
> 
> MoinMoin-Wiki Markup Source:
> ||<tableclass="navigation" tablestyle="background:#dfdfc0;"> '''Table
> ||Header''' || <style="background:#fffff0;vertical-align: top"> Table Data
> ||||
> 
> Expected output (as seen in MoinMoin 1.9.7 and below):
> 
>   <table class="navigation" style="background:#dfdfc0;"> …
>   <td style="background:#fffff0; vertical-align: top"> …
> 
> 
> Actual output from MoinMoin 1.9.9:
> 
>   <table class="&quot;navigation&quot;" style="&quot;
> background:#dfdfc0; &quot;"> … <td style="&quot;
> background:#fffff0; vertical-align: top&quot;"> …

Quoting issues are almost always frustrating. Looking at the differences 
between 1.9.7 and 1.9.9, there are few changes to the parser and formatter 
code, but I did spot something interesting in one of the tests:

https://bitbucket.org/thomaswaldmann/moin-1.9/commits/465cb6f5c6d7a88e5c3685d86ad20ff0e41318d5#chg-
MoinMoin/parser/_tests/test_text_moin_wiki.py

The commit message is useful because it indicates that Werkzeug got upgraded 
and had changed its behaviour. I haven't chased down the sequence of 
operations that might involve the escape function, however.

What is also odd is that the above looks like double-quoting.

Paul