Documentation schemas issue

[vtedesco]

The 3 schemas generated for this documentation page http://jmsyst.com/bundles/JMSPaymentCoreBundle/master/model look like wrong to me.

Sample :

[psrpinto]

I know it's been a while but could you please describe why you find the schemas to be wrong? What exactly is the problem? Thanks.

[vtedesco]

It's hard to read, arrow are pointing to nowhere because boxes are not properly aligned. I don't know about the data, I'm juste speaking about the visual rendering.

[psrpinto]

I see what you mean, thanks for reporting it. I'm pretty sure the images were correct in the past, not sure what happened here. Will look into it.

[greg0ire]

I discovered this the other day. Maybe consider switching to it instead of versioning images? It would make contributing graphs easier.

[psrpinto]

Thanks @greg0ire, nomnoml looks interesting, I'll consider it when I address this issue.

[greg0ire]

Great! Here is the code if you decide to use that :

<script src="https://cdn.jsdelivr.net/lodash/3.7.0/lodash.min.js"></script>
<script src="http://cpettitt.github.io/project/dagre/v0.4.3/dagre.min.js"></script>
<script src="https://cdnjs.cloudflare.com/ajax/libs/nomnoml/0.0.4/nomnoml.js"></script>

<canvas id="target-canvas"></canvas>
<script type="text/vnd.nomnoml.class" id="graph">
[<start>st]->[<state>new]
[new]->[<state>expired]
[expired]->[<end>end]
[new]->[<state>approving]
[approving]->[<state>canceled]
[canceled]->[end]
[approving]->[<state>approved]
[approved]->[<state>depositing]
[depositing]->[<state>deposited]
[depositing]->[<state>failed]
[failed]->[end]
[deposited]->[end]
</script>
<script>
    nomnoml.draw(
      document.getElementById('target-canvas'),
      document.getElementById('graph').innerHTML
    );
</script>

[psrpinto]

FYI, I decided to go with graphviz, which was the easiest way to do it. You can take look at the fixed schemas at http://jmspaymentcorebundle.readthedocs.io/en/latest/model.html

[greg0ire]

Congrats, it looks nice! And you also get a textual representation, which is really good too!

[psrpinto]

Fixed by #181