[Uml-devel] [Bug 192288] Umbrello crashes when closing class property window [UMLListView::contentsMouseDoubleClickEvent, Q3ScrollView::viewportMouseDoubleClickEvent]

[Oliver Kellogg]

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

Oliver Kellogg <okellogg at users.sourceforge.net> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |RESOLVED
                 CC|                            |okellogg at users.sourceforge.
                   |                            |net
         Resolution|---                         |UPSTREAM

--- Comment #24 from Oliver Kellogg <okellogg at users.sourceforge.net> ---
Hard to analyze because this PR is an amassment of different symptoms and
backtraces.
Going through the list one by one - using umbrello-2.8.1 (part of KDE 4.8.1)

(In reply to comment #0)
> 
> Error occured after checking abstract class and closing properties dialog.

Cannot confirm using umbrello-2.8.1

(In reply to comment #1)
> Another stack trace for this bug:
> [...]

Cannot confirm using umbrello-2.8.1

(In reply to comment #4)
> The error occurs, after changing a property and close the window.

Cannot confirm using umbrello-2.8.1.
Many other PRs were marked as duplicate of this (comments #5 to #11.)
I hope the triager knew what he was doing!

(In reply to comment #12)
> I get an Umbrello crash whenever I perform the following:
> 
> From the "Tree View" select the properties dialog for a class that is not yet part of a package. 
> Select a package for this class. Apply. The "Tree View" will reflect this change but when the
> dialog is closed (with either OK or Cancel) Umbrello will crash (seg fault).
> 
> If I do the same from the "Work Area" it works as expected.
> 
> Umbrello version 2.4.5

Cannot confirm using umbrello-2.8.1.

(In reply to comment #13)
> From bug 254799: (KDE SC 4.5.1)
> -- Information about the crash:
> Two packages in logical view. Added attribute of type package1.classA to
> package2.classB. Umbrello crashed after hitting 'ok'.
> 
Cannot confirm using umbrello-2.8.1.

> ...
> From bug 255176:
> -- Information about the crash:
> I had a foreign key defined. I wanted to edit it and double-clicked on it in
> the list of window named "Foreign key setup"

No problem observed using umbrello-2.8.1.

(In reply to comment #18)
> Created attachment 55363 [details]
> New crash information added by DrKonqi
> 
> umbrello (2.5.1) on KDE Platform 4.5.1 (KDE 4.5.1) using Qt 4.7.0
> 
> Setting class properties using the Properties dialog.  Crashes when I click
> OK every time.
> 
> -- Backtrace (Reduced):
> #7  0x085dc31c in UMLListView::contentsMouseDoubleClickEvent
> [...]

Cannot confirm using umbrello-2.8.1.

(In reply to comment #19)
> Created attachment 55770 [details]
> New crash information added by DrKonqi
> 
> umbrello (2.5.1) on KDE Platform 4.5.1 (KDE 4.5.1) using Qt 4.7.0
> 
> - What I was doing when the application crashed:
> 
> I changed the (Java) package of the class I was working on.  Clicking
> "Apply" applies the settings, but clicking OK or cancel crashes the
> application without fail.

This is probably a duplicate of bug 268469.

> 
> - Custom settings of the application:
> 
> Modelling in Java-specific UML
> Using Class diagram at the time
> Had a package (org) with a subpackage (called ris), thus org.ris package,
> but I'm not really sure how umbrello handles this.  In other words, I'm not
> sure if I should name a package org, and then make a package called ris
> within org, or if I should just make a package called org.ris.

Definitely the first way: First a package org, then a package ris nested inside
org.
In fact, it's a weakness of Umbrello that the second way is possible at all.
When given a name "org.ris", Umbrello should automatically split it at the "."
and create two packages (as in the first way.)

(In reply to comment #23)
> Created attachment 55832 [details]
> For sure final
> 
> Sorry to post once more, but the last diff file was backwards.

The renaming related code was overhauled by Ralf and me prior to 2.8.1.
Please try again with 2.8.1.

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