gabrielkheisa/ryzen-controller
1
0
Fork 0
mirror of https://gitlab.com/ryzen-controller-team/ryzen-controller.git synced 2024-12-22 10:03:28 +07:00
Code Issues Projects Releases Wiki Activity

fix: Ensure preset exist when applying.

Browse Source 
This fix an issue where you could apply a preset on ac status change after deleting it.
This commit is contained in:
Quentin Decaunes 2019-08-17 17:48:58 +02:00
parent 1a7fd0098a
commit 2152f32b8f
1 changed files with 20 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

23
js/preset.js
View File

@ -130,15 +130,32 @@ function preset_updateList() {
presetTab.innerHTML = content; presetTab.innerHTML = content;
} }
/**
* Will check if the preset exists.
*
* @param {String} name The preset name to look for.
*/
function preset_isExist(name) {
const preset = require('electron-settings').get(`presets`)[name];
return !!preset;
}
/** /**
* This will apply the preset you asked for. * This will apply the preset you asked for.
* @param {string} presetName The preset name to be applied. * @param {string} presetName The preset name to be applied.
*/ */
function preset_apply(presetName) { function preset_apply(presetName) {
const presets = require('electron-settings').get(`presets`)[presetName]; if (!preset_isExist(presetName)) {
appendLog(`preset_apply(): preset ${presetName}: ${JSON.stringify(presets)}`); notification('danger', `Unable to apply unexisting preset "${presetName}".`);
var ret = require('electron-settings').set("latest_controller_tabs_settings", presets); return;
}
const preset = require('electron-settings').get(`presets`)[presetName];
appendLog(`preset_apply(): preset ${presetName}: ${JSON.stringify(preset)}`);
var ret = require('electron-settings').set("latest_controller_tabs_settings", preset);
appendLog(`preset_apply(): saved preset: ${JSON.stringify(ret)}`); appendLog(`preset_apply(): saved preset: ${JSON.stringify(ret)}`);
loadLatestUsedSettings(); loadLatestUsedSettings();
applyRyzenSettings(); applyRyzenSettings();
toggleOptionDisplayBasedOnApplyCheckbox(); toggleOptionDisplayBasedOnApplyCheckbox();