diff --git a/src/plugins/qmldesigner/components/curveeditor/curveeditorview.cpp b/src/plugins/qmldesigner/components/curveeditor/curveeditorview.cpp index 0decde20297..0646e8de73d 100644 --- a/src/plugins/qmldesigner/components/curveeditor/curveeditorview.cpp +++ b/src/plugins/qmldesigner/components/curveeditor/curveeditorview.cpp @@ -144,7 +144,6 @@ void CurveEditorView::variantPropertiesChanged([[maybe_unused]] const QList &propertyList) { for (const auto &property : propertyList) { - if (property.name() == "keyframes" && property.parentModelNode().isValid()) { + if (property.name() == "keyframes") { ModelNode parent = property.parentModelNode(); if (dirtyfiesView(parent)) updateKeyframes(); @@ -260,10 +259,7 @@ ModelNode getTargetNode(PropertyTreeItem *item, const QmlTimeline &timeline) QmlTimelineKeyframeGroup timelineKeyframeGroup(QmlTimeline &timeline, PropertyTreeItem *item) { ModelNode node = getTargetNode(item, timeline); - if (node.isValid()) - return timeline.keyframeGroup(node, item->name().toLatin1()); - - return QmlTimelineKeyframeGroup(); + return timeline.keyframeGroup(node, item->name().toLatin1()); } void attachEasingCurve(const QmlTimelineKeyframeGroup &group, double frame, const QEasingCurve &curve) @@ -277,23 +273,21 @@ void attachEasingCurve(const QmlTimelineKeyframeGroup &group, double frame, cons void commitAuxiliaryData(ModelNode &node, TreeItem *item) { - if (node.isValid()) { - if (item->locked()) - node.setLocked(true); - else - node.setLocked(false); + if (item->locked()) + node.setLocked(true); + else + node.setLocked(false); - if (item->pinned()) - node.setAuxiliaryData(pinnedProperty, true); - else - node.removeAuxiliaryData(pinnedProperty); + if (item->pinned()) + node.setAuxiliaryData(pinnedProperty, true); + else + node.removeAuxiliaryData(pinnedProperty); - if (auto *pitem = item->asPropertyItem()) { - if (pitem->hasUnified()) - node.setAuxiliaryData(unifiedProperty, pitem->unifyString()); - else - node.removeAuxiliaryData(unifiedProperty); - } + if (auto *pitem = item->asPropertyItem()) { + if (pitem->hasUnified()) + node.setAuxiliaryData(unifiedProperty, pitem->unifyString()); + else + node.removeAuxiliaryData(unifiedProperty); } }