Bug 468728 - ERD lost foreign keys when loaded again
Summary: ERD lost foreign keys when loaded again
Status: RESOLVED FIXED
Alias: None
Product: umbrello
Classification: Applications
Component: general (show other bugs)
Version: unspecified
Platform: Other Linux
: NOR major
Target Milestone: ---
Assignee: Umbrello Development Group
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2023-04-20 15:55 UTC by rolf Schmidt
Modified: 2023-09-20 01:06 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In: 2.38.2 (KDE releases 23.08.2)


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description rolf Schmidt 2023-04-20 15:55:55 UTC
SUMMARY
Lost of foreign keys in ERdiagrams when safed and reloaded.

STEPS TO REPRODUCE
1. Build a ERdiagram with 2 entities (A and B) each with an autoincrment id-field
2. Add a second record in in Entity for example as "link" as long, unsigned integer
3. Safe the diagram f.e. as "withoutFk.xmi"
4. Now link entity A as a foreign constaint into B and save as f.e. "withFk.xmi"
5. Build a diff of both xmi files to see, that the foreigen key was written in "withFK.xmi"

OBSERVED RESULT
When reload the "withFK.xmi" file all foreign key information is gone.
When safe the file again no info about the foreign key is written - look again at a diff. 

EXPECTED RESULT
No lost of foreign key information 

SOFTWARE/OS VERSIONS
Linux (Debian testing) with Trinity : 

ADDITIONAL INFORMATION
BTW. export to SQL (only) didn't work so it should not shown.
Comment 1 Ralf Habacker 2023-09-20 01:06:37 UTC
Git commit 17fed409debff57cf5f25864a141ac74f87d6a0e by Ralf Habacker.
Committed on 20/09/2023 at 03:05.
Pushed by habacker into branch 'release/23.08'.

Fix 'ERD lost foreign keys when loaded again'

Constraints are wrapped by a 'packagedElement', which has not yet been
processed when loading.
FIXED-IN: 2.38.2 (KDE releases 23.08.2)

M  +1    -1    umbrello/umlmodel/entity.cpp

https://invent.kde.org/sdk/umbrello/-/commit/17fed409debff57cf5f25864a141ac74f87d6a0e