Skip to content
This repository was archived by the owner on Jun 23, 2023. It is now read-only.

Commit d78d765

Browse files
committed
Apply review suggestions
1 parent e9648fc commit d78d765

File tree

1 file changed

+12
-9
lines changed

1 file changed

+12
-9
lines changed

logic/auth.js

Lines changed: 12 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -213,20 +213,23 @@ async function getSettings() {
213213

214214
async function updateSetting(setting, value) {
215215
try {
216-
const user = await diskLogic.readUserFile();
217-
const { settings = {} } = user;
218-
219-
if(setting) {
220-
if(value) settings[setting] = value;
221-
else delete settings[setting];
216+
const settings = await getSettings();
217+
218+
if(!setting) return settings;
219+
220+
if(typeof value !== undefined) {
221+
settings[setting] = value;
222+
} else {
223+
delete settings[setting];
222224
}
223-
225+
226+
const user = await diskLogic.readUserFile();
224227
user.settings = settings;
225228
await diskLogic.writeUserFile(user);
226229

227-
return user.settings;
230+
return settings;
228231
} catch (error) {
229-
throw new NodeError(`Unable to update ${setting || 'setting'}`);
232+
throw new NodeError(`Unable to update setting`);
230233
}
231234
};
232235

0 commit comments

Comments
 (0)