single || in a line mistaken for table formatting

[Markus.Staab]

if you write sth like

DQL still uses the CONCAT() and not the || syntax

in a svn comment, the changeset view looks like

  DQL still uses the CONCAT() and not the syntax

so trac adds <td> </td> for

. thats not correct in this case because the user wants to tell sth about the

operator.

see ​http://trac.phpdoctrine.org/changeset/3497

[Emmanuel Blot]

SVN log messages are interpreted as wiki syntax.

Either use verbatim blocks ({{{ }}}) or quoting (``) or disable wiki syntax for log messages: TracIni#changeset-section

[Noah Kantrowitz]

The one-line formatter used for those messages probably shouldn't be processing table directives though.

[Christian Boos]

Well, probably yes, but that would only solve help while displaying it in the timeline (short form), not when visualizing the changeset. We can improve the regexp for a simple table row so that it only handles stuff like || (... ||)+ (that could still fail in some situations, but less systematically than now).

[Christian Boos]

Now if we make the regexp treat the line as a whole, we could also probably allow the single pipe syntax:

^\s*\|(?:(?:[^|]*)\|)+\s*\\?$

(the split in cells and handling of !| being done in a second step)

One step closer to full WikiCreole compatibility ;-)

[Christian Boos]

Forgot about pipes within backticks or within {{{...}}}, this is a bit harder than I thought.