Bug 124587

Summary: Keypress "Delete" removes association name in diagram but not in model
Product: [Applications] umbrello Reporter: Karl Pietrzak <kap4020>
Component: generalAssignee: Umbrello Development Group <umbrello-devel>
Status: RESOLVED FIXED    
Severity: normal CC: finex, ralf.habacker
Priority: NOR    
Version First Reported In: unspecified   
Target Milestone: ---   
Platform: unspecified   
OS: Linux   
Latest Commit: https://invent.kde.org/sdk/umbrello/-/commit/07e0d98a0c676641e5efdcb6d1012b51db2e5458 Version Fixed/Implemented In: 25.11.80
Sentry Crash Report:

Description Karl Pietrzak 2006-03-30 19:46:33 UTC 
Version:           1.5.1 (using KDE 3.4.2 Level "b" , SUSE 10.0)
Compiler:          Target: x86_64-suse-linux
OS:                Linux (x86_64) release 2.6.13-15.8-default

STACKTRACE

Using host libthread_db library "/lib64/tls/libthread_db.so.1".
[Thread debugging using libthread_db enabled]
[New Thread 46912561297920 (LWP 25026)]
[KCrash handler]
#3  0x0000000001be0b50 in ?? ()
#4  0x00000000005137c3 in DocWindow::updateDocumentation ()
#5  0x00000000005140c2 in DocWindow::showDocumentation ()
#6  0x00000000005c6841 in UMLWidget::setSelected ()
#7  0x00000000005c3021 in UMLWidget::mousePressEvent ()
#8  0x000000000056841d in ToolBarState::setSelectedWidget ()
#9  0x000000000056804f in ToolBarState::mousePress ()
#10 0x00000000005686ea in ToolBarStateArrow::mousePress ()
#11 0x00002aaaac999fd9 in QScrollView::viewportMousePressEvent ()
   from /usr/lib/qt3/lib64/libqt-mt.so.3
#12 0x00002aaaac99776a in QScrollView::eventFilter ()
   from /usr/lib/qt3/lib64/libqt-mt.so.3
#13 0x00002aaaac8a039a in QObject::activate_filters ()
   from /usr/lib/qt3/lib64/libqt-mt.so.3
#14 0x00002aaaac8a0407 in QObject::event ()
   from /usr/lib/qt3/lib64/libqt-mt.so.3
#15 0x00002aaaac8d3bf8 in QWidget::event ()
   from /usr/lib/qt3/lib64/libqt-mt.so.3
#16 0x00002aaaac84a603 in QApplication::internalNotify ()
   from /usr/lib/qt3/lib64/libqt-mt.so.3
#17 0x00002aaaac84b274 in QApplication::notify ()
   from /usr/lib/qt3/lib64/libqt-mt.so.3
#18 0x00002aaaabcaf98f in KApplication::notify ()
   from /opt/kde3/lib64/libkdecore.so.4
#19 0x00002aaaac7f057c in QETWidget::translateMouseEvent ()
   from /usr/lib/qt3/lib64/libqt-mt.so.3
#20 0x00002aaaac7ef428 in QApplication::x11ProcessEvent ()
   from /usr/lib/qt3/lib64/libqt-mt.so.3
#21 0x00002aaaac801447 in QEventLoop::processEvents ()
   from /usr/lib/qt3/lib64/libqt-mt.so.3
#22 0x00002aaaac85f228 in QEventLoop::enterLoop ()
   from /usr/lib/qt3/lib64/libqt-mt.so.3
#23 0x00002aaaac85f142 in QEventLoop::exec ()
   from /usr/lib/qt3/lib64/libqt-mt.so.3
#24 0x000000000053e2e2 in main ()


HOW TO REPRODUCE
1. create a new state diagram
2. create an initial state and a regular state
3. create state transition from initial state to the regular state
4. right click on the arrow, select "Change Association name"
5. enter 'a', select OK
6. click on the 'a' next to the arrow in the diagram, and hit DELETE
7. right click on the arrow, select "Change Association name"
8. it will still say 'a' (not sure if this is the correct behavior)
9. hit BACKSPACE to delete the 'a' and hit OK
10. right click on the arrow, select "Change Association name"
11. entire 'a'
12. try selecting the 'a' next to the arrow

CRASH!
Comment 1 Oliver Kellogg 2006-03-30 21:03:32 UTC 
I don't get the crash (tried it on current branches/KDE/3.5)
but I do confirm this:
> 6. click on the 'a' next to the arrow in the diagram, and hit DELETE 
> 7. right click on the arrow, select "Change Association name" 
> 8. it will still say 'a' (not sure if this is the correct behavior) 

Certainly not the correct behavior :)

Please try again using current branches/KDE/3.5.
If nobody confirms the crash then I suggest we rename this PR
to something like "association name not deleted in model".
Comment 2 Karl Pietrzak 2007-02-19 18:48:41 UTC 
Unfortunately, I don't have KDE 3.5, so I will not be able to try it out. :(
Comment 3 Oliver Kellogg 2007-02-19 21:34:21 UTC 
> I don't have KDE 3.5, so I will not be able to try it out.

Umbrello compiles with KDE >= 3.3, maybe even with 3.2.
Comment 4 Karl Pietrzak 2007-02-20 02:04:04 UTC 
I just tried 1.5.61, and the behavior is even stranger now

I followed steps 1-8 above, and yes, it does still say 'a'.  But try this:

9. Click 'Cancel' in the "Change association name" dialog 
10. Right-click again on the arrow and select "Change association name"
11. Now the association name is blank, like it's supposed to be.

That's kinda weird, heh.
Comment 5 FiNeX 2009-08-03 22:03:46 UTC 
I've tried to reproduce the crash on KDE4.
The state diagram doesn't allow to edit association names, so I've tried using the ER diagram. It has not crashed.
Comment 6 Ralf Habacker 2015-02-28 09:15:20 UTC 
see also Bug 141809
Comment 7 Justin Zobel 2021-03-09 07:00:38 UTC 
Thank you for the bug report.

As this report hasn't seen any changes in 5 years or more, we ask if you can please confirm that the issue still persists.

If this bug is no longer persisting or relevant please change the status to resolved.
Comment 8 Ralf Habacker 2025-11-17 12:47:31 UTC 
Git commit 07e0d98a0c676641e5efdcb6d1012b51db2e5458 by Ralf Habacker.
Committed on 01/11/2025 at 16:04.
Pushed by habacker into branch 'master'.

Fix bug not deleting uml association

When an association widget is deleted, the associated UML
association is not deleted. The remaining associations
are no longer visible because they do not appear in the
property dialog of a class. Since they are no longer
visible, cyclic inheritances can easily occur, which
can lead to a crash, e.g., of the Java export.

After this fix, it is important to restore the missing
widgets for such associations and to point out cyclic
inheritances when loading files.
FIXED-IN:25.11.80

M  +1    -1    umbrello/umlwidgets/associationwidget.cpp

https://invent.kde.org/sdk/umbrello/-/commit/07e0d98a0c676641e5efdcb6d1012b51db2e5458
Comment 9 Ralf Habacker 2025-11-17 12:48:24 UTC 
Git commit f7845f1127f6297da68fc000f2eef76cd0e39d10 by Ralf Habacker.
Committed on 01/11/2025 at 16:04.
Pushed by habacker into branch 'master'.

Readd lost association widgets in all views

M  +87   -0    umbrello/umldoc.cpp

https://invent.kde.org/sdk/umbrello/-/commit/f7845f1127f6297da68fc000f2eef76cd0e39d10