diff options
| author | Paul Lemire <paul.lemire@kdab.com> | 2018-01-22 11:47:57 +0100 |
|---|---|---|
| committer | Sean Harmer <sean.harmer@kdab.com> | 2018-01-23 18:23:54 +0000 |
| commit | e2e162b9ca122e68c444b536888f30c6f1c10de4 (patch) | |
| tree | 494c99fa5b2a86f4ef905415db2313a07a13f518 /src/render/io/scenemanager.cpp | |
| parent | f6ae313f0628adbd222bebcf41c9fa42059dc060 (diff) | |
SceneLoader: fix broken scene tree unloading
To unload the scene, we need to process the empty path in the backend to reset
the status and send a change to the frontend QSceneLoader that will take care
of deleting the previously created scene tree.
With the introduction of remote urls support, the case of the empty path was
not properly handled. It would end up being recognized as a remote path, a
download request would be made. Eventually it would silently fail. A warning
was added when a download request fails.
Change-Id: I60222453ccde63d69f6a107d33d7c842313d98ed
Reviewed-by: Sean Harmer <sean.harmer@kdab.com>
Diffstat (limited to 'src/render/io/scenemanager.cpp')
| -rw-r--r-- | src/render/io/scenemanager.cpp | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/src/render/io/scenemanager.cpp b/src/render/io/scenemanager.cpp index fddb814c4..464871b48 100644 --- a/src/render/io/scenemanager.cpp +++ b/src/render/io/scenemanager.cpp @@ -118,6 +118,8 @@ void SceneDownloader::onCompleted() return; if (succeeded()) m_manager->addSceneData(url(), m_sceneComponent, m_data); + else + qWarning() << "Failed to download scene at" << url(); m_manager->clearSceneDownload(this); } |