Bug 331215 - Documentation not saved with object widgets
Summary: Documentation not saved with object widgets
Status: RESOLVED FIXED
Alias: None
Product: umbrello
Classification: Applications
Component: general (show other bugs)
Version: 2.12.2
Platform: unspecified Linux
: NOR normal
Target Milestone: ---
Assignee: Umbrello Development Group
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2014-02-16 20:17 UTC by Ralf Habacker
Modified: 2014-10-27 09:04 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In: 4.14.2
Sentry Crash Report:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Ralf Habacker 2014-02-16 20:17:19 UTC
Adding widget documentation to an object widget of a sequence diagram through the widget property dialog do not add documentation. Adding documentation through the doc windows works. 


Reproducible: Always

Steps to Reproduce:
1. open umbrello
2. create a sequence diagram
3. add two objects
4. Double click on an object and edit documentation
5. Save propertes
Actual Results:  
Reopen the widget property dialog do not show the entered documentation. 

Expected Results:  
The documentation should be added to the widget
Comment 1 Ralf Habacker 2014-02-16 20:31:13 UTC
looks similar to bug 255424
Comment 2 Ralf Habacker 2014-02-16 20:37:31 UTC
There seems to be a logic failure:
The doc is saved into the widget in void ClassGenPage::updateObject().
On the next mouse click the widgets doc is overwritten from DocWindow::updateDocumentation

0	UMLObject::setDoc	   umlobject.cpp	422	0x63fa60	
1	DocWindow::updateDocumentation	docwindow.cpp	222	0x5ee397	
2	DocWindow::showDocumentation	docwindow.cpp	156	0x5ee6bf	
3	UMLScene::mousePressEvent	umlscene.cpp	851	0x6485bd
Comment 3 Felix Rohrbach 2014-09-28 21:25:43 UTC
Git commit 864832c0d310197c8b0cc8c941fdefec27d923e7 by Felix Rohrbach.
Committed on 28/09/2014 at 21:17.
Pushed by felixr into branch 'master'.

Fix creation of documentation for UML-Components

This fixes two issues connected to the DocWindow.

1. When DocWindow::showDocumentation is called, it checks whether it
is called with the same UMLObject/Scene/... But it missed that
UMLObjects and UMLScenes might be passed to the function embedded in a
widget.

2. The overwrite parameter of DocWindow::showDocumentation did the
opposite of what the documentation said. I changed it to match the
documentation and adjusted a few functions using it wrong.
Related: bug 255424
FIXED-IN: 4.14.2
REVIEW: 120369

M  +34   -15   umbrello/docwindow.cpp
M  +2    -2    umbrello/umlscene.cpp
M  +1    -0    umbrello/widgets/associationwidget.cpp
M  +1    -1    umbrello/widgets/notewidget.cpp

http://commits.kde.org/umbrello/864832c0d310197c8b0cc8c941fdefec27d923e7
Comment 4 Ralf Habacker 2014-10-27 09:04:45 UTC
Git commit a802474df0a660e4cf6bfdd360b6bb8a692b89ba by Ralf Habacker, on behalf of Felix Rohrbach.
Committed on 28/09/2014 at 21:17.
Pushed by habacker into branch 'master'.

Fix creation of documentation for UML-Components

This fixes two issues connected to the DocWindow.

1. When DocWindow::showDocumentation is called, it checks whether it
is called with the same UMLObject/Scene/... But it missed that
UMLObjects and UMLScenes might be passed to the function embedded in a
widget.

2. The overwrite parameter of DocWindow::showDocumentation did the
opposite of what the documentation said. I changed it to match the
documentation and adjusted a few functions using it wrong.
Related: bug 255424
FIXED-IN: 4.14.2
REVIEW: 120369

M  +34   -15   umbrello/docwindow.cpp
M  +2    -2    umbrello/umlscene.cpp
M  +1    -0    umbrello/widgets/associationwidget.cpp
M  +1    -1    umbrello/widgets/notewidget.cpp

http://commits.kde.org/umbrello/a802474df0a660e4cf6bfdd360b6bb8a692b89ba