Skip to content

Commit 04b4c74

Browse files
authored
Merge pull request #2879 from haslinghuis/fix_expert_mode
2 parents 7744058 + 2fb9308 commit 04b4c74

File tree

1 file changed

+14
-15
lines changed

1 file changed

+14
-15
lines changed

src/js/serial_backend.js

Lines changed: 14 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -193,12 +193,6 @@ function finishClose(finishedCallback) {
193193
// reset active sensor indicators
194194
sensor_status(0);
195195

196-
// reset expert mode
197-
ConfigStorage.get('permanentExpertMode', function (result) {
198-
const checked = result.permanentExpertMode;
199-
$('input[name="expertModeCheckbox"]').prop('checked', checked).trigger('change');
200-
});
201-
202196
if (wasConnected) {
203197
// detach listeners and remove element data
204198
$('#content').empty();
@@ -232,17 +226,22 @@ function onOpen(openInfo) {
232226
GUI.log(i18n.getMessage('serialPortOpened', serial.connectionType === 'serial' ? [serial.connectionId] : [openInfo.socketId]));
233227

234228
// save selected port with chrome.storage if the port differs
235-
ConfigStorage.get('last_used_port', function (result) {
236-
if (result.last_used_port) {
237-
if (result.last_used_port !== GUI.connected_to) {
238-
// last used port doesn't match the one found in local db, we will store the new one
239-
ConfigStorage.set({'last_used_port': GUI.connected_to});
240-
}
241-
} else {
242-
// variable isn't stored yet, saving
229+
let result = ConfigStorage.get('last_used_port');
230+
if (result.last_used_port) {
231+
if (result.last_used_port !== GUI.connected_to) {
232+
// last used port doesn't match the one found in local db, we will store the new one
243233
ConfigStorage.set({'last_used_port': GUI.connected_to});
244234
}
245-
});
235+
} else {
236+
// variable isn't stored yet, saving
237+
ConfigStorage.set({'last_used_port': GUI.connected_to});
238+
}
239+
240+
// reset expert mode
241+
result = ConfigStorage.get('permanentExpertMode');
242+
if (result.permanentExpertMode) {
243+
$('input[name="expertModeCheckbox"]').prop('checked', result.permanentExpertMode).trigger('change');
244+
}
246245

247246
serial.onReceive.addListener(read_serial);
248247
setConnectionTimeout();

0 commit comments

Comments
 (0)