From 1fcd62f21342702e6aedfce090c952e48600305f Mon Sep 17 00:00:00 2001 From: wmayer Date: Tue, 10 Jan 2023 17:46:05 +0100 Subject: [PATCH] Gui: [skip ci] print warning in getViewProviderByPath, getViewProviderByPathFromTail, getViewProvidersOfType if for View3DInventorViewer no document is set This gives the client programmer a clue if one of the mentioned methods fails --- src/Gui/View3DInventorViewer.cpp | 12 +++++++++--- 1 file changed, 9 insertions(+), 3 deletions(-) diff --git a/src/Gui/View3DInventorViewer.cpp b/src/Gui/View3DInventorViewer.cpp index 3e87861603..ebbd583cfd 100644 --- a/src/Gui/View3DInventorViewer.cpp +++ b/src/Gui/View3DInventorViewer.cpp @@ -3365,22 +3365,28 @@ void View3DInventorViewer::removeEventCallback(SoType eventtype, SoEventCallback ViewProvider* View3DInventorViewer::getViewProviderByPath(SoPath* path) const { - if (!guiDocument) + if (!guiDocument) { + Base::Console().Warning("View3DInventorViewer::getViewProviderByPath: No document set\n"); return nullptr; + } return guiDocument->getViewProviderByPathFromHead(path); } ViewProvider* View3DInventorViewer::getViewProviderByPathFromTail(SoPath* path) const { - if (!guiDocument) + if (!guiDocument) { + Base::Console().Warning("View3DInventorViewer::getViewProviderByPathFromTail: No document set\n"); return nullptr; + } return guiDocument->getViewProviderByPathFromTail(path); } std::vector View3DInventorViewer::getViewProvidersOfType(const Base::Type& typeId) const { - if (!guiDocument) + if (!guiDocument) { + Base::Console().Warning("View3DInventorViewer::getViewProvidersOfType: No document set\n"); return {}; + } return guiDocument->getViewProvidersOfType(typeId); }