Edgewall Software
Modify

Opened 12 years ago

Last modified 4 years ago

#6630 new defect

single || in a line mistaken for table formatting

Reported by: Markus.Staab Owned by:
Priority: normal Milestone: topic-wikiengine
Component: wiki system Version:
Severity: normal Keywords: table pipe syntax
Cc: Branch:
Release Notes:
API Changes:

Description

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

Attachments (0)

Change History (7)

comment:1 by Emmanuel Blot, 12 years ago

Component: generalbrowser
Resolution: worksforme
Status: newclosed

SVN log messages are interpreted as wiki syntax.

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

comment:2 by Noah Kantrowitz, 12 years ago

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

comment:3 by Christian Boos, 12 years ago

Component: browserwiki
Keywords: table added
Milestone: 0.12
Resolution: worksforme
Status: closedreopened
Summary: using || in svn commentssingle || in a line mistaken for table formatting

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).

comment:4 by Christian Boos, 10 years ago

Milestone: next-major-0.1X0.12

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 ;-)

comment:5 by Christian Boos, 10 years ago

Keywords: pipe syntax added
Milestone: 0.120.13

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

comment:6 by Christian Boos, 9 years ago

Milestone: 0.130.14-wikiengine
Owner: changed from Jonas Borgström to Christian Boos
Status: reopenednew

comment:7 by Ryan J Ollos, 4 years ago

Owner: Christian Boos removed

Modify Ticket

Change Properties
Set your email in Preferences
Action
as new The ticket will remain with no owner.
The ticket will be disowned. Next status will be 'new'.
as The resolution will be set. Next status will be 'closed'.
The owner will be changed from (none) to anonymous. Next status will be 'assigned'.

Add Comment


E-mail address and name can be saved in the Preferences .
 
Note: See TracTickets for help on using tickets.