search

beyondcode / invoker-er-diagram-plugin

Generate ER Diagrams in Invoker
16 stars 2 forks source link

ER Diagramm displays every relationship twice #3

Open Enaah opened 3 years ago

Enaah commented 3 years ago

Hi there,

The ER Diagram-Plugin from Invoker does visualize all my Relationships twice. The Loadingscreen even displays Loading model definitions (18/9).

Here are two Screenshots.

This one is from Invoker ER Diagramm Invoker

And this one if from your Package (laravel-er-diagram-generator): ER Diagramm - package

Have a nice weekend! 🌈

jwktje commented 3 years ago

Just wanted to report this too. Having the same issue

sschlein commented 3 years ago

Hey, I have moved this issue to the correct repository. The ER Plugin is open source and we accept Pull-Requests (but will also have a look what causes this issue)

Enaah commented 3 years ago

Thanks @sschlein (I diddn't looked for this repo, sorry). Even in the Example Picture in the README every Relationship is displayed twice. Just to be sure, this it not intentional right?

Edit: Could take some time until I can do a PR...

AndreSchwarzer commented 2 years ago

Hey @Enaah and @jwktje

I've created a PR which fixes the issue for me. Can you verify it works on your end too?

Feedback is appreciated; either way.

AndreSchwarzer commented 2 years ago

I attached the built plugin.zip to the PR so it is easier for you to test it.

jwktje commented 2 years ago

@AndreSchwarzer Just tested this and for me this also solves the problem! Good job on the patch!

Enaah commented 1 year ago

Sorry, @AndreSchwarzer I must have overlooked your comment and work.
Thank you very much.
FYI: for the composer package there is a recently released new version: https://github.com/beyondcode/laravel-er-diagram-generator/releases/tag/2.0.0