Wiki syntax should be enforced (for ticket comments)

[eblot]

When the wiki syntax is not valid (missing closing tag, for example: ''italic'), Trac does not fix it up.

Although this is not a strong issue with Wiki pages that can be edited and fixed by human beings, it starts to become an issue with ticket comments: the missing wiki tag - which is translated to an XHTML missing tag - propagates across the whole ticket and may corrupt the whole page.

As ticket comments cannot be edited, the only current way to recover from such a simple user mistake is to tweak the SQL DB ;-(

IMHO, Trac should ensure that such an error does not propagate outside a comment, i.e. should always close any open tag along with the end of a comment.

It is worth noting that when post-commit hooks are used -such as the one from the /contrib dir- the user may not even notice than "his" wiki syntax is not correct, as it is never rendered from his subversion client (commit message).

[eblot]

Example of ticket page rendering, when one of the comments is short of a closing italic/emphasis tag

[anonymous]

Can you post the exact comment which was causing this? I can't reproduce the issue.

[eblot]

Sure. From sqlite3:

sqlite> select * from ticket_change where ticket=670 and time=1149159330;
670|1149159330|souquiej|comment||(In [575]) Refs #670
 * dyntest: ''sga:mdraw'' renamed to ''sga:drawm''
 * dyntest: new ''sga:adrawm' command, exercised in drawbat.dtf

There are remaining comments after this one, but the display misbehaves starting after the missing "'" char.
If it can help, the server runs [3320].

[cboos]

Ok, thanks, I could reproduce the problem. This is apparently because of the unfinished italic being in a list.

I wanted to implement anyway a more robust block-level elements and inline-level elements stacking algorithm, but this will have to wait 0.11.

I will nevertheless check if there's a simple fix for this specific problem, for 0.10.

[anonymous]

What if I want to have apostrophes in my italic text? How do we avoid false positives trying to "fix" the user's formatting?

[cboos]

The fix will be at the HTML level, not touching the wiki text. In the above example, the last "'" character will stay as is, simply the <i> tag will be closed.

[Manuzhai]

There seems to be consensus that this can wait for 0.11.

[cboos]

#4702 was closed as duplicate of this one.

[cboos]

WikiEngine refactoring and related fixes postponed.

[anonymous]

It would be nice if comments could be edited somehow; then one could at least work around the kind of problem discussed here.

[eblot]

Replying to anonymous:

It would be nice if comments could be edited somehow; then one could at least work around the kind of problem discussed here.

There is a dedicated ticket for this feature (#454), please do not deviate the thread from the original topic.

[eblot]

#5771 marked as a dup.

[cboos]

#6344 was closed as duplicate of this ticket.