Indenting lost from comments in the Moodle Tracker

Indenting lost from comments in the Moodle Tracker

de Tim Hunt -
Número de respuestas: 5
Imagen de Core developers Imagen de Documentation writers Imagen de Particularly helpful Moodlers Imagen de Peer reviewers Imagen de Plugin developers
So I don't forget, the bug I was trying to file was:


Summary: Code gets mangled when pasted into a bug report

Description: For example: MDL-6752. All the new code that I am being asked to consider has lost its indent, which is a pain in the neck. Is there any way round this?

(It's not that difficult to copy and paste the code into an editor, and reindent it, but it would be nice if I did not have to.)
Promedio de valoraciones: -
En respuesta a Tim Hunt

Re: Catch-22 at tracker

de Thomas Robb -
People who submit code should be asked to upload it as a text file if it involves indentation (as it should).  This is what I did for my yet unresolved submission MDL-6249.
En respuesta a Thomas Robb

Re: Catch-22 at tracker

de Martín Langhoff -
And yet, it's invaluable to be able to post code, and post back comments on the code. Code as attachments means it's harder to review the actual code.

+1 for making it usable to post diffs straight there, without attachments.
En respuesta a Martín Langhoff

Indenting lost from comments in the Moodle Tracker

de Martin Dougiamas -
Imagen de Core developers Imagen de Documentation writers Imagen de Moodle HQ Imagen de Particularly helpful Moodlers Imagen de Plugin developers Imagen de Testers
It sort of works ... it seems to only collapse non-leading whitespace

See MDL-6732 (feel free to test in there)
En respuesta a Martin Dougiamas

Re: Indenting lost from comments in the Moodle Tracker

de Martín Langhoff -
I can see it. It messes up diffs! triste

Looking at the html source, it's replacing spaces with   but only leading/trailing spaces, not within the line -- maybe it could respect PRE tags in the comments to support pasting code?

{comment also in bug}