Nested generics on ClassDiagram

[nicanor-busto]

Description

Hi, I've not found an issue about it, but if I put nested generic types on method return type they're represented in a wrong way, for example:

This:

classDiagram
    direction TB

    class Foo {
        + doSomething() Either~List~Foo~,Pojo~
    }

Returns something like:

Which is incorrect.

Steps to reproduce

Create a data type with two generics, one of them must have a generic type as well.

Code Sample

[erichiller]

I think I'm having the same issue, but if not let me know I can create a different issue.

Defining a property in classDiagram that has two Generic Type parameters, such as:

+ Dictionary~KeyType,List~ElType~~ Dict

• Broken on v10.5.0 (using the LiveEditor), renders as: + Dictionary~KeyType,List~ElType~> Dict

• Errors on v10.4.0 with Syntax error in text

• Broken on whatever version GitHub uses with Maximum call stack size exceeded

• Works great on v9.2.2 + Dictionary<KeyType,List<ElType>> Dict

It appears it is the comma breaking things, as this works:

                    ↓ no comma works
+ Dictionary~KeyType->List~ElType~~ Dict

Here is a full sample:

classDiagram

class ChartViewModel {
    + Dictionary~KeyType->List~ElType~~ Dict
}

vs.

classDiagram

class ChartViewModel {
    + Dictionary~KeyType,List~ElType~~ Dict
}

[immortal-boy]

It could write like this doSomething() Either<List<Foo>,Pojo> now

[jgreywolf]

@erichiller is correct (missed the comment somehow) - this is due to the commas, and something I am, slowly, in the process of working on.

[sidrus]

I would like to offer my workaround for this. I just used the #44; HTML entity for the comma:

class OrderBook {
    -Dictionary ~decimal#44; Queue~Order~~ BuyQueue
    -Dictionary ~decimal#44; Queue~Order~~ SellQueue
    +AddOrder() void       
}

That resulted in this output:

[jgreywolf]

I am sorry for the LONG delay - but I am now actively working on an update to the class diagram syntax that will address this, and other, issues.

Should have something within a couple of days :)