Bug 368081

Summary:	Importing abstract c++ class do not set "abstract class" property
Product:	[Applications] umbrello	Reporter:	Ralf Habacker <ralf.habacker>
Component:	general	Assignee:	Umbrello Development Group <umbrello-devel>
Status:	RESOLVED FIXED
Severity:	wishlist
Priority:	NOR
Version:	2.19.2 (KDE Applications 16.04.2)
Target Milestone:	---
Platform:	Other
OS:	Linux
Latest Commit:	http://commits.kde.org/umbrello/a0181860b9baaa5c5b1912ef18aa2b0a868f69bf	Version Fixed In:	2.20.1 (KDE Applications 16.08.1)
Sentry Crash Report:
Attachments:	test case

Description Ralf Habacker 2016-09-01 06:18:37 UTC

Classes handled by umbrello have an attribute "abstract class" to indicate that a class is abstract. Abstract classes are class with at least one pure virtual method. Importing such class do not set this property.




Reproducible: Always

Steps to Reproduce:
1. start umbrello
2. import appended test case


Actual Results:  
The imported class do not have the property "abstract class" set. 

Expected Results:  
The imported class should have set the property "abstract class".

Comment 1 Ralf Habacker 2016-09-01 06:20:59 UTC

Created attachment 100874 [details]
test case

Comment 2 Ralf Habacker 2016-09-01 06:54:56 UTC

Git commit a0181860b9baaa5c5b1912ef18aa2b0a868f69bf by Ralf Habacker.
Committed on 01/09/2016 at 06:53.
Pushed by habacker into branch 'Applications/16.08'.

Make imported c++ class abstract in case there are any pure virtual methods.
FIXED-IN:2.20.1 (KDE Applications 16.08.1)

A  +12   -0    test/import/cxx/abstract-class.h     [License: UNKNOWN]  *
M  +2    -0    umbrello/codeimport/kdevcppparser/cpptree2uml.cpp

The files marked with a * at the end have a non valid license. Please read: http://techbase.kde.org/Policies/Licensing_Policy and use the headers which are listed at that page.


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