[umbrello-devel] [umbrello] [Bug 339278] Crash when resizing class diagram display

[Ralf Habacker]

https://bugs.kde.org/show_bug.cgi?id=339278

--- Comment #4 from Ralf Habacker <ralf.habacker at freenet.de> ---
Git commit cd9b676b941a42cf12f60b325b82284e51e2a354 by Ralf Habacker, on behalf
of Oliver Kellogg.
Committed on 22/09/2014 at 22:36.
Pushed by habacker into branch 'master'.

Address http://bugs.kde.org/attachment.cgi?id=88780

(In reply to Mark Stanton from comment #0)
> [...] The display doesn't resize and the app crashes.
> I may have tried this before the project had fully loaded. [...]

The backtrace confirms this, apparently you double clicked on a class widget
on a diagram while Umbrello was still busy in UMLDoc::resolveTypes().
By the way, Umbrello is finished loading when you see "Ready." in the lower
left corner of Umbrello's main window.

The following changes fend off double clicks on diagram objects while the
program is still busy in loadFromXMI() or resolveTypes():

umbrello/umldoc.cpp
- In default constructor, initialize m_bTypesAreResolved to true.
- In function closeDocument() remove assignment to m_bTypesAreResolved.
- In function openDocument(),
  - Add a few missing KIO::NetAccess::removeTempFile() calls in return-on-
    error situations;
  - Set m_bTypesAreResolved false before loadFromXMI()/loadFromMDL() calls;
  - Set m_bLoading false and m_bTypesAreResolved true before returning.
- In function loading() return true also when m_bTypesAreResolved is false.
- In function resolveTypes() postpone setting of m_bTypesAreResolved to true
  until type resolution has actually completed.

umbrello/umlscene.cpp
- In function mousePressEvent() call m_pToolBarState->mouseDoubleClick()
  only if m_doc->loading() returns false.

M  +18   -18   umbrello/umldoc.cpp
M  +2    -1    umbrello/umlscene.cpp

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

-- 
You are receiving this mail because:
You are the assignee for the bug.