[umbrello] [Bug 368282] Make type casting more robust
https://bugs.kde.org/show_bug.cgi?id=368282 --- Comment #12 from Ralf Habacker--- Git commit c036a0c04d9c967ba1ad2566eb731a663cc066d3 by Ralf Habacker. Committed on 11/09/2016 at 11:13. Pushed by habacker into branch 'master'. Check return value from dynamic_cast against zero in ClassGeneralPage constructor. M +1-1umbrello/dialogs/pages/classgeneralpage.cpp http://commits.kde.org/umbrello/c036a0c04d9c967ba1ad2566eb731a663cc066d3 -- You are receiving this mail because: You are watching all bug changes.
[umbrello] [Bug 368282] Make type casting more robust
https://bugs.kde.org/show_bug.cgi?id=368282 Ralf Habackerchanged: What|Removed |Added Version Fixed In|http://commits.kde.org/umbr |2.20.80 (KDE Applications |ello/3e9f089d6f90085d3e312f |16.11.80) |3a06d780d9d1c026ff | Latest Commit|2.20.80 (KDE Applications |http://commits.kde.org/umbr |16.11.80) |ello/3e9f089d6f90085d3e312f ||3a06d780d9d1c026ff -- You are receiving this mail because: You are watching all bug changes.
[umbrello] [Bug 368282] Make type casting more robust
https://bugs.kde.org/show_bug.cgi?id=368282 Ralf Habackerchanged: What|Removed |Added Version Fixed In||http://commits.kde.org/umbr ||ello/3e9f089d6f90085d3e312f ||3a06d780d9d1c026ff Latest Commit||2.20.80 (KDE Applications ||16.11.80) Resolution|--- |FIXED Status|UNCONFIRMED |RESOLVED -- You are receiving this mail because: You are watching all bug changes.
[umbrello] [Bug 368282] Make type casting more robust
https://bugs.kde.org/show_bug.cgi?id=368282 --- Comment #11 from Ralf Habacker--- Git commit 3e9f089d6f90085d3e312f3a06d780d9d1c026ff by Ralf Habacker. Committed on 09/09/2016 at 09:18. Pushed by habacker into branch 'master'. Complete migration of dynamic_cast to UMLObject::asUML...() wrapper. M +1-1umbrello/clipboard/umlclipboard.cpp M +1-1umbrello/codegenerators/ada/adawriter.cpp M +2-2umbrello/codegenerators/classifiercodedocument.cpp M +1-1umbrello/codegenerators/codeclassfield.cpp M +2-2umbrello/codegenerators/cpp/cppwriter.cpp M +1-1umbrello/codegenerators/d/dcodeclassfielddeclarationblock.cpp M +1-1 umbrello/codegenerators/java/javacodeclassfielddeclarationblock.cpp M +1-1umbrello/codegenerators/pascal/pascalwriter.cpp M +2-2 umbrello/codegenerators/ruby/rubycodeclassfielddeclarationblock.cpp M +2-2umbrello/codegenerators/xml/xmlschemawriter.cpp M +2-2umbrello/dialogs/umlattributedialog.cpp M +2-2umbrello/dialogs/umlentityattributedialog.cpp M +1-1umbrello/dialogs/umlenumliteraldialog.cpp M +1-1umbrello/dialogs/umltemplatedialog.cpp M +3-3umbrello/umllistview.cpp M +3-3umbrello/umlmodel/classifier.cpp M +2-2umbrello/umlmodel/umlcanvasobject.cpp M +2-2umbrello/umlmodel/umlobject.cpp M +1-1umbrello/umlscene.cpp M +2-2umbrello/umlwidgets/messagewidget.cpp http://commits.kde.org/umbrello/3e9f089d6f90085d3e312f3a06d780d9d1c026ff --- Comment #10 from Ralf Habacker --- Git commit 120af5d34c2424c2eb4dd631aea925edb227fded by Ralf Habacker. Committed on 09/09/2016 at 09:18. Pushed by habacker into branch 'master'. Complete migration of UMLObject related static_casts to UMLObject::asUML...() wrapper. M +1-1umbrello/clipboard/umldragdata.cpp http://commits.kde.org/umbrello/120af5d34c2424c2eb4dd631aea925edb227fded -- You are receiving this mail because: You are watching all bug changes.
[umbrello] [Bug 368282] Make type casting more robust
https://bugs.kde.org/show_bug.cgi?id=368282 --- Comment #11 from Ralf Habacker--- Git commit 3e9f089d6f90085d3e312f3a06d780d9d1c026ff by Ralf Habacker. Committed on 09/09/2016 at 09:18. Pushed by habacker into branch 'master'. Complete migration of dynamic_cast to UMLObject::asUML...() wrapper. M +1-1umbrello/clipboard/umlclipboard.cpp M +1-1umbrello/codegenerators/ada/adawriter.cpp M +2-2umbrello/codegenerators/classifiercodedocument.cpp M +1-1umbrello/codegenerators/codeclassfield.cpp M +2-2umbrello/codegenerators/cpp/cppwriter.cpp M +1-1umbrello/codegenerators/d/dcodeclassfielddeclarationblock.cpp M +1-1 umbrello/codegenerators/java/javacodeclassfielddeclarationblock.cpp M +1-1umbrello/codegenerators/pascal/pascalwriter.cpp M +2-2 umbrello/codegenerators/ruby/rubycodeclassfielddeclarationblock.cpp M +2-2umbrello/codegenerators/xml/xmlschemawriter.cpp M +2-2umbrello/dialogs/umlattributedialog.cpp M +2-2umbrello/dialogs/umlentityattributedialog.cpp M +1-1umbrello/dialogs/umlenumliteraldialog.cpp M +1-1umbrello/dialogs/umltemplatedialog.cpp M +3-3umbrello/umllistview.cpp M +3-3umbrello/umlmodel/classifier.cpp M +2-2umbrello/umlmodel/umlcanvasobject.cpp M +2-2umbrello/umlmodel/umlobject.cpp M +1-1umbrello/umlscene.cpp M +2-2umbrello/umlwidgets/messagewidget.cpp http://commits.kde.org/umbrello/3e9f089d6f90085d3e312f3a06d780d9d1c026ff --- Comment #10 from Ralf Habacker --- Git commit 120af5d34c2424c2eb4dd631aea925edb227fded by Ralf Habacker. Committed on 09/09/2016 at 09:18. Pushed by habacker into branch 'master'. Complete migration of UMLObject related static_casts to UMLObject::asUML...() wrapper. M +1-1umbrello/clipboard/umldragdata.cpp http://commits.kde.org/umbrello/120af5d34c2424c2eb4dd631aea925edb227fded -- You are receiving this mail because: You are watching all bug changes.
[umbrello] [Bug 368282] Make type casting more robust
https://bugs.kde.org/show_bug.cgi?id=368282 --- Comment #9 from Ralf Habacker--- Git commit 972a4253237b93dae76b18b6980eb8e8c1464ebf by Ralf Habacker. Committed on 08/09/2016 at 19:05. Pushed by habacker into branch 'master'. Fix regression in type cast migration. With commit a5f92b67494666e4e6779b1f2443b1a1df2b031d a regression has been introduced which removes UMLClassifier instances containing associations from their parent folder while loading from xmi file. On saving the uml model into a xmi file those instances are excluded because they are not in the parents object list (class member UMLPackage::m_objects). After (re)loading such files some LOST_ items in the tree view indicates this issue. M +1-1umbrello/umlmodel/classifier.cpp http://commits.kde.org/umbrello/972a4253237b93dae76b18b6980eb8e8c1464ebf -- You are receiving this mail because: You are watching all bug changes.
[umbrello] [Bug 368282] Make type casting more robust
https://bugs.kde.org/show_bug.cgi?id=368282 --- Comment #8 from Ralf Habacker--- Git commit d9b4afea8444320ca51f62dc5fa49d4ee2bfa8b3 by Ralf Habacker. Committed on 07/09/2016 at 07:06. Pushed by habacker into branch 'master'. Do not exclude parents of UMLClassifierListItem instances from copying and comparing. After adding a related testcase it raises the issue that the parent of UMLClassifierListItem based objects are not been copied. Because we use the same parent store for all UMLObject based objects now it is possible to unify this behavior. M +3-3umbrello/umlmodel/umlobject.cpp M +10 -0unittests/TEST_umlobject.cpp http://commits.kde.org/umbrello/d9b4afea8444320ca51f62dc5fa49d4ee2bfa8b3 -- You are receiving this mail because: You are watching all bug changes.
[umbrello] [Bug 368282] Make type casting more robust
https://bugs.kde.org/show_bug.cgi?id=368282 --- Comment #6 from Ralf Habacker--- Git commit 8c6bad8a76df59a11bf67ce62e76c3bda0f12007 by Ralf Habacker. Committed on 06/09/2016 at 17:08. Pushed by habacker into branch 'master'. Fix warning about missing brackets and logic. M +2-2umbrello/umlmodel/umlobject.cpp http://commits.kde.org/umbrello/8c6bad8a76df59a11bf67ce62e76c3bda0f12007 --- Comment #5 from Ralf Habacker --- Git commit 6d5686f264ddfda0e6bf030e16902569e7dc02df by Ralf Habacker. Committed on 06/09/2016 at 16:59. Pushed by habacker into branch 'master'. Fix crash in TEST_UMLObject::test_fullyQualifiedName() caused by creating uml model related objects on stack. Because QCOMPARE() returns the function immediatly on error we need to delete related objects on ourself on exit. This is performed by adding objects to delete with cleanupOnExit(). M +15 -12 unittests/TEST_umlobject.cpp M +8-0unittests/testbase.cpp M +4-0unittests/testbase.h http://commits.kde.org/umbrello/6d5686f264ddfda0e6bf030e16902569e7dc02df -- You are receiving this mail because: You are watching all bug changes.
[umbrello] [Bug 368282] Make type casting more robust
https://bugs.kde.org/show_bug.cgi?id=368282 --- Comment #7 from Ralf Habacker--- Git commit 191e52ef5dc2e9a03b3a6c42a6566c4a82630d66 by Ralf Habacker. Committed on 06/09/2016 at 17:11. Pushed by habacker into branch 'master'. Reduce the number of calls to umlPackage() in UMLObject::fullyQualifiedName() by making a local copy. M +5-4umbrello/umlmodel/umlobject.cpp http://commits.kde.org/umbrello/191e52ef5dc2e9a03b3a6c42a6566c4a82630d66 -- You are receiving this mail because: You are watching all bug changes.
[umbrello] [Bug 368282] Make type casting more robust
https://bugs.kde.org/show_bug.cgi?id=368282 --- Comment #6 from Ralf Habacker--- Git commit 8c6bad8a76df59a11bf67ce62e76c3bda0f12007 by Ralf Habacker. Committed on 06/09/2016 at 17:08. Pushed by habacker into branch 'master'. Fix warning about missing brackets and logic. M +2-2umbrello/umlmodel/umlobject.cpp http://commits.kde.org/umbrello/8c6bad8a76df59a11bf67ce62e76c3bda0f12007 --- Comment #5 from Ralf Habacker --- Git commit 6d5686f264ddfda0e6bf030e16902569e7dc02df by Ralf Habacker. Committed on 06/09/2016 at 16:59. Pushed by habacker into branch 'master'. Fix crash in TEST_UMLObject::test_fullyQualifiedName() caused by creating uml model related objects on stack. Because QCOMPARE() returns the function immediatly on error we need to delete related objects on ourself on exit. This is performed by adding objects to delete with cleanupOnExit(). M +15 -12 unittests/TEST_umlobject.cpp M +8-0unittests/testbase.cpp M +4-0unittests/testbase.h http://commits.kde.org/umbrello/6d5686f264ddfda0e6bf030e16902569e7dc02df -- You are receiving this mail because: You are watching all bug changes.
[umbrello] [Bug 368282] Make type casting more robust
https://bugs.kde.org/show_bug.cgi?id=368282 --- Comment #4 from Ralf Habacker--- Git commit badc01d753ca0cb46b51a3fd2ed07d265cf9bfed by Ralf Habacker. Committed on 06/09/2016 at 15:54. Pushed by habacker into branch 'master'. Removed obsolate UMLObject::m_pUMLPackage from uml model. M +288 -2195 models/UmbrelloArchitecture/umlmodel.xmi http://commits.kde.org/umbrello/badc01d753ca0cb46b51a3fd2ed07d265cf9bfed -- You are receiving this mail because: You are watching all bug changes.
[umbrello] [Bug 368282] Make type casting more robust
https://bugs.kde.org/show_bug.cgi?id=368282 --- Comment #3 from Ralf Habacker--- Git commit dd847f1576b37f9da31df51084fa3dda243ccc3c by Ralf Habacker. Committed on 06/09/2016 at 15:00. Pushed by habacker into branch 'master'. Replace usage of UMLObject::m_pUMLPackage by QObject member 'parent' through new method umlParent(). In the past UMLObject::m_pUMLPackage has been used in some classes, while other uses the QObject class member 'parent'. This commit unify both approaches by using QObject class member 'parent' in all classes. class UMLObject now has two new functions setUMLParent() and umlParent() to provides a UMLObject based interface to the parent uml object; method umlPackage() is now a shortcut of umlParent()->asUMLPackage(). Reviewed-by: aaron.nottbeck at sag.eu M +1-1umbrello/codegenerators/ada/adawriter.cpp M +1-1umbrello/codeimport/import_utils.cpp M +1-1umbrello/dialogs/umlforeignkeyconstraintdialog.cpp M +1-1umbrello/dialogs/umloperationdialog.cpp M +1-1umbrello/dialogs/umluniqueconstraintdialog.cpp M +3-3umbrello/dialogs/widgets/umldatatypewidget.cpp M +2-2umbrello/model_utils.cpp M +4-4umbrello/refactoring/refactoringassistant.cpp M +3-3umbrello/umldoc.cpp M +2-2umbrello/umllistview.cpp M +6-6umbrello/umllistviewitem.cpp M +9-9umbrello/umlmodel/association.cpp M +6-6umbrello/umlmodel/attribute.cpp M +1-1umbrello/umlmodel/checkconstraint.cpp M +3-5umbrello/umlmodel/classifierlistitem.cpp M +2-2umbrello/umlmodel/entity.cpp M +1-1umbrello/umlmodel/entityattribute.cpp M +1-1umbrello/umlmodel/enumliteral.cpp M +2-2umbrello/umlmodel/folder.cpp M +4-4umbrello/umlmodel/foreignkeyconstraint.cpp M +4-4umbrello/umlmodel/operation.cpp M +1-1umbrello/umlmodel/template.cpp M +50 -24 umbrello/umlmodel/umlobject.cpp M +4-2umbrello/umlmodel/umlobject.h M +8-7umbrello/umlmodel/uniqueconstraint.cpp M +3-3umbrello/umlwidgets/associationwidget.cpp M +1-1umbrello/umlwidgets/linkwidget.cpp http://commits.kde.org/umbrello/dd847f1576b37f9da31df51084fa3dda243ccc3c --- Comment #2 from Ralf Habacker --- Git commit fa3f51e764c4296c81454f25b340d00241f172d3 by Ralf Habacker. Committed on 06/09/2016 at 14:59. Pushed by habacker into branch 'master'. Fix QPointer related dynamic_casts to use asUML... wrapper(). Reviewed-by: aaron.nottbeck at sag.eu M +5-5umbrello/umlmodel/classifier.cpp M +1-1umbrello/umlmodel/classifierlistitem.cpp M +2-2umbrello/umlwidgets/artifactwidget.cpp M +8-8umbrello/umlwidgets/associationwidget.cpp M +1-1umbrello/umlwidgets/categorywidget.cpp M +1-1umbrello/umlwidgets/classifierwidget.cpp M +3-3umbrello/umlwidgets/componentwidget.cpp M +8-8umbrello/umlwidgets/entitywidget.cpp M +4-4umbrello/umlwidgets/enumwidget.cpp M +1-1umbrello/umlwidgets/messagewidget.cpp http://commits.kde.org/umbrello/fa3f51e764c4296c81454f25b340d00241f172d3 --- Comment #1 from Ralf Habacker --- Git commit a5f92b67494666e4e6779b1f2443b1a1df2b031d by Ralf Habacker. Committed on 06/09/2016 at 14:59. Pushed by habacker into branch 'master'. Convert casts in UML model classes to use related casting methods. Reviewed-by: aaron.nottbeck at sag.eu M +7-7umbrello/clipboard/umlclipboard.cpp M +1-1umbrello/clipboard/umldragdata.cpp M +2-2umbrello/codegenerators/ada/adawriter.cpp M +2-2umbrello/codegenerators/advancedcodegenerator.cpp M +1-1umbrello/codegenerators/classifiercodedocument.cpp M +8-6umbrello/codegenerators/codeclassfield.cpp M +1-1umbrello/codegenerators/codeclassfielddeclarationblock.cpp M +2-2umbrello/codegenerators/codedocument.cpp M +3-3umbrello/codegenerators/codegenerator.cpp M +1-1umbrello/codegenerators/codegenobjectwithtextblocks.cpp M +2-2umbrello/codegenerators/codeoperation.cpp M +3-3umbrello/codegenerators/codeparameter.cpp M +1-1umbrello/codegenerators/cpp/cppcodeclassfield.cpp M +2-2umbrello/codegenerators/cpp/cppcodegenerator.cpp M +2-2umbrello/codegenerators/cpp/cppheadercodedocument.cpp M +1-1umbrello/codegenerators/csharp/csharpwriter.cpp M +1-1umbrello/codegenerators/d/dclassifiercodedocument.cpp M +1-1umbrello/codegenerators/d/dcodeclassfield.cpp M +1-1umbrello/codegenerators/java/javaantcodedocument.cpp M +1-1umbrello/codegenerators/java/javaclassifiercodedocument.cpp M +1-1umbrello/codegenerators/java/javacodeclassfield.cpp M +2-2umbrello/codegenerators/ownedcodeblock.cpp M +1-1umbrello/codegenerators/ownedhierarchicalcodeblock.cpp M +2-2
[umbrello] [Bug 368282] Make type casting more robust
https://bugs.kde.org/show_bug.cgi?id=368282 --- Comment #3 from Ralf Habacker--- Git commit dd847f1576b37f9da31df51084fa3dda243ccc3c by Ralf Habacker. Committed on 06/09/2016 at 15:00. Pushed by habacker into branch 'master'. Replace usage of UMLObject::m_pUMLPackage by QObject member 'parent' through new method umlParent(). In the past UMLObject::m_pUMLPackage has been used in some classes, while other uses the QObject class member 'parent'. This commit unify both approaches by using QObject class member 'parent' in all classes. class UMLObject now has two new functions setUMLParent() and umlParent() to provides a UMLObject based interface to the parent uml object; method umlPackage() is now a shortcut of umlParent()->asUMLPackage(). Reviewed-by: aaron.nottbeck at sag.eu M +1-1umbrello/codegenerators/ada/adawriter.cpp M +1-1umbrello/codeimport/import_utils.cpp M +1-1umbrello/dialogs/umlforeignkeyconstraintdialog.cpp M +1-1umbrello/dialogs/umloperationdialog.cpp M +1-1umbrello/dialogs/umluniqueconstraintdialog.cpp M +3-3umbrello/dialogs/widgets/umldatatypewidget.cpp M +2-2umbrello/model_utils.cpp M +4-4umbrello/refactoring/refactoringassistant.cpp M +3-3umbrello/umldoc.cpp M +2-2umbrello/umllistview.cpp M +6-6umbrello/umllistviewitem.cpp M +9-9umbrello/umlmodel/association.cpp M +6-6umbrello/umlmodel/attribute.cpp M +1-1umbrello/umlmodel/checkconstraint.cpp M +3-5umbrello/umlmodel/classifierlistitem.cpp M +2-2umbrello/umlmodel/entity.cpp M +1-1umbrello/umlmodel/entityattribute.cpp M +1-1umbrello/umlmodel/enumliteral.cpp M +2-2umbrello/umlmodel/folder.cpp M +4-4umbrello/umlmodel/foreignkeyconstraint.cpp M +4-4umbrello/umlmodel/operation.cpp M +1-1umbrello/umlmodel/template.cpp M +50 -24 umbrello/umlmodel/umlobject.cpp M +4-2umbrello/umlmodel/umlobject.h M +8-7umbrello/umlmodel/uniqueconstraint.cpp M +3-3umbrello/umlwidgets/associationwidget.cpp M +1-1umbrello/umlwidgets/linkwidget.cpp http://commits.kde.org/umbrello/dd847f1576b37f9da31df51084fa3dda243ccc3c --- Comment #2 from Ralf Habacker --- Git commit fa3f51e764c4296c81454f25b340d00241f172d3 by Ralf Habacker. Committed on 06/09/2016 at 14:59. Pushed by habacker into branch 'master'. Fix QPointer related dynamic_casts to use asUML... wrapper(). Reviewed-by: aaron.nottbeck at sag.eu M +5-5umbrello/umlmodel/classifier.cpp M +1-1umbrello/umlmodel/classifierlistitem.cpp M +2-2umbrello/umlwidgets/artifactwidget.cpp M +8-8umbrello/umlwidgets/associationwidget.cpp M +1-1umbrello/umlwidgets/categorywidget.cpp M +1-1umbrello/umlwidgets/classifierwidget.cpp M +3-3umbrello/umlwidgets/componentwidget.cpp M +8-8umbrello/umlwidgets/entitywidget.cpp M +4-4umbrello/umlwidgets/enumwidget.cpp M +1-1umbrello/umlwidgets/messagewidget.cpp http://commits.kde.org/umbrello/fa3f51e764c4296c81454f25b340d00241f172d3 --- Comment #1 from Ralf Habacker --- Git commit a5f92b67494666e4e6779b1f2443b1a1df2b031d by Ralf Habacker. Committed on 06/09/2016 at 14:59. Pushed by habacker into branch 'master'. Convert casts in UML model classes to use related casting methods. Reviewed-by: aaron.nottbeck at sag.eu M +7-7umbrello/clipboard/umlclipboard.cpp M +1-1umbrello/clipboard/umldragdata.cpp M +2-2umbrello/codegenerators/ada/adawriter.cpp M +2-2umbrello/codegenerators/advancedcodegenerator.cpp M +1-1umbrello/codegenerators/classifiercodedocument.cpp M +8-6umbrello/codegenerators/codeclassfield.cpp M +1-1umbrello/codegenerators/codeclassfielddeclarationblock.cpp M +2-2umbrello/codegenerators/codedocument.cpp M +3-3umbrello/codegenerators/codegenerator.cpp M +1-1umbrello/codegenerators/codegenobjectwithtextblocks.cpp M +2-2umbrello/codegenerators/codeoperation.cpp M +3-3umbrello/codegenerators/codeparameter.cpp M +1-1umbrello/codegenerators/cpp/cppcodeclassfield.cpp M +2-2umbrello/codegenerators/cpp/cppcodegenerator.cpp M +2-2umbrello/codegenerators/cpp/cppheadercodedocument.cpp M +1-1umbrello/codegenerators/csharp/csharpwriter.cpp M +1-1umbrello/codegenerators/d/dclassifiercodedocument.cpp M +1-1umbrello/codegenerators/d/dcodeclassfield.cpp M +1-1umbrello/codegenerators/java/javaantcodedocument.cpp M +1-1umbrello/codegenerators/java/javaclassifiercodedocument.cpp M +1-1umbrello/codegenerators/java/javacodeclassfield.cpp M +2-2umbrello/codegenerators/ownedcodeblock.cpp M +1-1umbrello/codegenerators/ownedhierarchicalcodeblock.cpp M +2-2