summaryrefslogtreecommitdiffstats
path: root/mobile-widgets/qmlmanager.cpp
diff options
context:
space:
mode:
authorGravatar Berthold Stoeger <bstoeger@mail.tuwien.ac.at>2017-12-11 23:20:18 +0100
committerGravatar Jan Mulder <jlmulder@xs4all.nl>2018-01-08 12:01:04 +0100
commit099662023c89f5bb0a3e2e1b803de22d56b20f6b (patch)
tree745fc10122feef73b2d89ffacc75af6eb9560186 /mobile-widgets/qmlmanager.cpp
parent35b5b9282f5d2068651dee7eb3993978fc258ac7 (diff)
downloadsubsurface-099662023c89f5bb0a3e2e1b803de22d56b20f6b.tar.gz
Move resetting of current file out of clear_dive_file_data()
This is the only case where C-code sets the current file. Remove this call for a better separation of C-backend and C++-frontend parts. There were four callers of clear_dive_file_data(). Two of them would call set_filename() anyway. For the remaining two add an explicit call to set_filename(). This commit fixes a bug introduced in commit b3901aa8f90499ee2a34efdddc2463105afc53f1: The cloud-online menu entry was still enabled after "closing" the cloud storage. Signed-off-by: Berthold Stoeger <bstoeger@mail.tuwien.ac.at>
Diffstat (limited to 'mobile-widgets/qmlmanager.cpp')
-rw-r--r--mobile-widgets/qmlmanager.cpp1
1 files changed, 1 insertions, 0 deletions
diff --git a/mobile-widgets/qmlmanager.cpp b/mobile-widgets/qmlmanager.cpp
index 697d71dc8..296243f7e 100644
--- a/mobile-widgets/qmlmanager.cpp
+++ b/mobile-widgets/qmlmanager.cpp
@@ -602,6 +602,7 @@ void QMLManager::loadDivesWithValidCredentials()
appendTextToLog(errorString);
setNotificationText(errorString);
revertToNoCloudIfNeeded();
+ set_filename(NULL);
return;
}
consumeFinishedLoad(currentDiveTimestamp);