Bug 420073

Summary: Umbrello does not show aggregation association
Product: [Applications] umbrello Reporter: wcchou <keiko.zhou>
Component: generalAssignee: Umbrello Development Group <umbrello-devel>
Status: RESOLVED FIXED    
Severity: normal CC: dvd2gold, ing.and.cattaneo, okellogg, subdiff
Priority: NOR    
Version: 2.30.3 (KDE releases 19.12.3)   
Target Milestone: ---   
Platform: Neon   
OS: Linux   
Latest Commit: Version Fixed In: 2.33.80 (KDE releases 21.03.80)
Attachments: example-view

Description wcchou 2020-04-14 09:14:41 UTC
SUMMARY


STEPS TO REPRODUCE
1. Launch Umbrello
2. Create 2 classes with default names
3. Click the "aggregation" association in the toolbar
4. Click the 1st class and click the 2nd class to create "aggregation" association

OBSERVED RESULT
1. The new attribute "new_attribute: new_class_1*" was created correctly
2. But, there is no "aggregation" line between these 2 classes

EXPECTED RESULT
The aggregation line was displayed.

SOFTWARE/OS VERSIONS
Operating System: KDE neon 5.18
KDE Plasma Version: 5.18.4
KDE Frameworks Version: 5.69.0
Qt Version: 5.14.1
Kernel Version: 5.3.0-46-generic
OS Type: 64-bit
Processors: 8 × Intel® Core™ i7-4710MQ CPU @ 2.50GHz
Memory: 31.0 GiB of RAM


ADDITIONAL INFORMATION
https://stackoverflow.com/questions/60614281/umbrello-doesnt-show-aggregation-association
Comment 1 Roman Gilg 2020-10-27 11:09:51 UTC
Created attachment 132789 [details]
example-view

I have the same issue on master. Could this be looked into? An example is attached. See the Toplevel class aggregating Wrapland::Server::Surface but not showing this connection in the view.
Comment 2 Oliver Kellogg 2020-10-30 21:12:19 UTC
Git commit 2bb95adcdac2b4211466fd1d7bc5425f89e12752 by Oliver Kellogg.
Committed on 30/10/2020 at 21:10.
Pushed by okellogg into branch 'master'.

umbrello/toolbarstateassociation.cpp partial reverting of commit 62ff388
- In function setSecondWidget(), if validation by AssocRules is passed
  and both widgetA and widgetB are class widgets then do not create an
  attribute for case (type == Uml::AssociationType::Aggregation).
  Reason: Creating the attribute prevents creation of the aggregation to
  the role B type.  For more info see comments at the source location.

M  +9    -2    umbrello/toolbarstateassociation.cpp

https://invent.kde.org/sdk/umbrello/commit/2bb95adcdac2b4211466fd1d7bc5425f89e12752
Comment 3 Oliver Kellogg 2022-05-24 22:18:07 UTC
*** Bug 452521 has been marked as a duplicate of this bug. ***
Comment 4 Oliver Kellogg 2022-05-25 19:46:15 UTC
*** Bug 437661 has been marked as a duplicate of this bug. ***