From de85051c81a74f82d86cc82ec25ead5374a83844 Mon Sep 17 00:00:00 2001 From: Ingo Weinhold Date: Sat, 29 Jun 2013 13:58:10 +0200 Subject: [PATCH] BPoseView::AttributeChanged(): additional check in icon case Model::AttrChanged() requires the model to be open and so does IconCache::IconChanged(), so add the check. Not sure if the block is ever executed now (or ever worked before). This does at least prevent asserts being triggered when compiled with debugging enabled. --- src/kits/tracker/PoseView.cpp | 1 + 1 file changed, 1 insertion(+) diff --git a/src/kits/tracker/PoseView.cpp b/src/kits/tracker/PoseView.cpp index 1b03aafb4d..55d1154b1f 100644 --- a/src/kits/tracker/PoseView.cpp +++ b/src/kits/tracker/PoseView.cpp @@ -5596,6 +5596,7 @@ BPoseView::AttributeChanged(const BMessage* message) message->FindString("attr", &attrName); if (TargetModel() != NULL && *TargetModel()->NodeRef() == itemNode + && TargetModel()->IsNodeOpen() && TargetModel()->AttrChanged(attrName)) { // the icon of our target has changed, update drag icon // TODO: make this simpler (ie. store the icon with the window)