diff --git a/src/plugins/qmldesigner/designercore/filemanager/addpropertyvisitor.cpp b/src/plugins/qmldesigner/designercore/filemanager/addpropertyvisitor.cpp index 52687f34f9c2cf8f778c19d6d1c912aef6ffdea6..d2b1fc81fbee1f79380070adecf1f42630127c7a 100644 --- a/src/plugins/qmldesigner/designercore/filemanager/addpropertyvisitor.cpp +++ b/src/plugins/qmldesigner/designercore/filemanager/addpropertyvisitor.cpp @@ -112,12 +112,12 @@ void AddPropertyVisitor::addInMembers(QmlJS::AST::UiObjectInitializer *initializ if (isOneLiner) { if (insertAfter == 0) { // we're inserting after an lbrace if (initializer->members) { // we're inserting before a member (and not the rbrace) - needsTrailingSemicolon = true; + needsTrailingSemicolon = m_propertyType == QmlRefactoring::ScriptBinding; } } else { // we're inserting after a member, not after the lbrace if (endOfPreviousMember.isValid()) { // there already is a semicolon after the previous member if (insertAfter->next && insertAfter->next->member) { // and the after us there is a member, not an rbrace, so: - needsTrailingSemicolon = true; + needsTrailingSemicolon = m_propertyType == QmlRefactoring::ScriptBinding; } } else { // there is no semicolon after the previous member (probably because there is an rbrace after us/it, so: needsPreceedingSemicolon = true; diff --git a/src/plugins/qmldesigner/designercore/filemanager/addpropertyvisitor.h b/src/plugins/qmldesigner/designercore/filemanager/addpropertyvisitor.h index d82bbcd60a48e16447698a130bf22cea077b5f34..5cb454da1d9bd53d35f9bbeaf42d40ef5eb380d6 100644 --- a/src/plugins/qmldesigner/designercore/filemanager/addpropertyvisitor.h +++ b/src/plugins/qmldesigner/designercore/filemanager/addpropertyvisitor.h @@ -58,7 +58,7 @@ private: quint32 m_parentLocation; QString m_name; QString m_value; - QmlDesigner::QmlRefactoring::PropertyType m_propertyType; + QmlRefactoring::PropertyType m_propertyType; QStringList m_propertyOrder; }; diff --git a/src/plugins/qmldesigner/designercore/model/rewriteactioncompressor.cpp b/src/plugins/qmldesigner/designercore/model/rewriteactioncompressor.cpp index b2af209ae5ad5f57fffef5766e5ce769c774e1fd..d3e52f12731f0c6d6745c625a35efaa2f014f372 100644 --- a/src/plugins/qmldesigner/designercore/model/rewriteactioncompressor.cpp +++ b/src/plugins/qmldesigner/designercore/model/rewriteactioncompressor.cpp @@ -108,7 +108,7 @@ void RewriteActionCompressor::compressImports(QList<RewriteAction *> &actions) c void RewriteActionCompressor::compressRereparentActions(QList<RewriteAction *> &actions) const { - QMap<ModelNode, ReparentNodeRewriteAction *> reparentedNodes; + QHash<ModelNode, ReparentNodeRewriteAction *> reparentedNodes; QMutableListIterator<RewriteAction*> iter(actions); iter.toBack(); diff --git a/tests/auto/qml/qmldesigner/coretests/testcore.cpp b/tests/auto/qml/qmldesigner/coretests/testcore.cpp index eaeeb0d2e2f21ccebfe19fcc051e1dc44c494dce..f980c55f1f65f916e345d27038d6aa088ae7e16b 100644 --- a/tests/auto/qml/qmldesigner/coretests/testcore.cpp +++ b/tests/auto/qml/qmldesigner/coretests/testcore.cpp @@ -804,8 +804,6 @@ void TestCore::testRewriterForGradientMagic() " }\n" "}"); - QSKIP("See BAUHAUS-674", SkipAll); - QPlainTextEdit textEdit; textEdit.setPlainText(qmlString); NotIndentingTextEditModifier modifier(&textEdit);