Opened 6 years ago

Closed 6 years ago

'comma' is not processed correctly

Reported by: Owned by: tom.ghyselinck@… Ryan J Ollos normal PlantUmlMacro normal 0.12

Description

The following code shows no @startuml found (see plantuml.png)

{{{
#!PlantUML
@startuml
note right of test : a, b
@enduml
}}}


while

{{{
#!PlantUML
@startuml
note right of test : a\, b
@enduml
}}}


shows the correct image (see plantuml2.png)

invalid parsing

Changed 6 years ago by anonymous

correctly parsed when escaping ,

comment:1 Changed 6 years ago by Ryan J Ollos

Owner: changed from Álvaro Iradier to Ryan J Ollos new → assigned

Thank you for the report. I'm glad to know someone is using the plugin! I'll see if I can get this fixed today.

comment:2 Changed 6 years ago by Ryan J Ollos

Also found that

{{{
#!PlantUML
}}}


Would result in a list index out of range error, rather than the proper No UML text defined! message.

comment:3 Changed 6 years ago by Ryan J Ollos

[11476] by rjollos on 04/11/12 23:54:51:

Refs #9955: FIX: If no PlantUML markup was defined, a list index out of range error would result, rather than the No UML text defined! message that was intended.

comment:4 Changed 6 years ago by Ryan J Ollos

Resolution: → fixed assigned → closed

(In [11478]) Fixes #9955: Commas in content were being escaped with backslashes by parse_args in the Trac API.

comment:5 Changed 6 years ago by Ryan J Ollos

[11478] seems to fix the issue. Please reopen if that is not the case for you.

comment:6 Changed 6 years ago by anonymous

Thanks!

Working fine now!

Modify Ticket

Change Properties