samtet1434 / support-tools

Automatically exported from code.google.com/p/support-tools
Apache License 2.0
0 stars 0 forks source link

Archived issues have broken formatting #176

Open GoogleCodeExporter opened 8 years ago

GoogleCodeExporter commented 8 years ago
Example: https://code.google.com/archive/p/sympy/issues/4105

You can see the correct formatting in the issue that was migrated to GitHub 
https://github.com/sympy/sympy/issues/7204.

The issue is that Google Code issues never had markup, so people would just 
paste code and tracebacks and whatever and it looked fine. But now it is trying 
to add markup to issues that never had it, making them hard if not impossible 
to read. 

Original issue reported on code.google.com by asmeurer@gmail.com on 26 Jan 2016 at 9:15

GoogleCodeExporter commented 8 years ago
Thank you for the bug report. And your assessment is spot on.

This is the content that exists in the Google Code Archive as part of the JSON 
dump. So this won't require reexporting anything, just a frontend JavaScript 
change. It looks like the conversion to Markdown isn't properly replacing \r\n 
with a <br>.

"\u0026gt;\u0026gt;\u0026gt; eye(1).free_symbols\r\nTraceback (most recent call 
last):\r\n  File \u0026quot;\u0026lt;stdin\u0026gt;\u0026quot;, line 1, in 
\u0026lt;module\u0026gt;\r\n  File 
\u0026quot;sympy\\matrices\\matrices.py\u0026quot;, line 3066, in 
__getattr__\r\n    \u0026quot;%s has no attribute %s.\u0026quot; % 
(self.__class__.__name__, attr))\r\nAttributeError: MutableDenseMatrix has no 
attribute free_symbols."

Original comment by chrsm...@google.com on 27 Jan 2016 at 3:01

GoogleCodeExporter commented 8 years ago
Also the __getattr__ shouldn't be bold. Really all markdown formatting 
shouldn't be enabled, because people wrote code in issue comments assuming it 
wasn't there. 

Original comment by asmeurer@gmail.com on 27 Jan 2016 at 4:33