Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Context Diagram misses some of the links. Show different diagram when switching between data models #401

Closed
jelenadreskai opened this issue Nov 10, 2021 · 5 comments · Fixed by #411
Assignees
Projects
Milestone

Comments

@jelenadreskai
Copy link

Describe the bug
Context Diagram misses some of the links. It shows different diagram each time user switches between data models.

To Reproduce
Steps to reproduce the behavior:

  1. Import attached data model
  2. Click on Data Model Schema / Diagram
  3. Observe the data model
  4. Some links are missing in the diagram
  5. Click on some other data model in the tree (you may have to click away multiple times)
  6. Come back to the first data model, and observe the context diagram
  7. Some links are missing in the diagram, and diagram is different from what was seen in step 4.

Expected behavior
Context diagram should have all links visible, and diagram should be the same at all times.

Screenshots
Note: I have a feeling that self-references are causing this issue. See attached data model.

I would always expect to see something like this:

image

However in this instance, there is a missing reference from Table 1 to Table 2.
image

An in this instance, there is a missing reference from Table 1 to Table 1.
image

Desktop (please complete the following information):

  • Chrome
  • Windows 10

Additional context
CORE: 4.10.0
UI: 6.6.0

@jelenadreskai
Copy link
Author

testreferences_20211110T184050.xlsx

Here is the data model, exported using Excel.

@jelenadreskai
Copy link
Author

@jamesrwelch @olliefreeman @gammonpeter
Hi guys, Gail had a look at this and noticed that there is a newer library for this available. Is it worth trying just using the new lib and see if this helps with this bug?

@olliefreeman olliefreeman transferred this issue from MauroDataMapper/mdm-core Nov 30, 2021
@olliefreeman olliefreeman added this to Backlog in State Dec 7, 2021
@jelenadreskai
Copy link
Author

jelenadreskai commented Dec 8, 2021

@olliefreeman Gail found the issue with the code, and has a fix. Can you assign it to her?

Just to add to this, it was not caused by self-reference, the issue was discovered even in data models with no self references. As soon as there were 3 data models referencing each other, the links were messed up.

@jamesrwelch
Copy link
Contributor

Thanks @GBishop-PHSA !

@project-bot project-bot bot moved this from Backlog to In Progress in State Dec 8, 2021
GBishop-PHSA added a commit to GBishop-PHSA/mdm-ui that referenced this issue Jan 4, 2022
@GBishop-PHSA
Copy link
Contributor

@jamesrwelch @gammonpeter Ready for review

@pjmonks pjmonks linked a pull request Jan 7, 2022 that will close this issue
GBishop-PHSA added a commit to GBishop-PHSA/mdm-ui that referenced this issue Jan 8, 2022
State automation moved this from In Progress to Merged to Develop Jan 10, 2022
pjmonks added a commit that referenced this issue Jan 10, 2022
gh-401 Updated schema diagrams to have unique reference ids
@pjmonks pjmonks added this to the 7.0.0 milestone Jan 10, 2022
GBishop-PHSA added a commit to GBishop-PHSA/mdm-ui that referenced this issue Jan 12, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
State
Merged to Develop
Development

Successfully merging a pull request may close this issue.

4 participants