biipadmin/Burn-My-Windows
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

♻️ Use glob import for utils

Browse Source
This commit is contained in:
Simon Schneegans
2023-09-02 10:21:40 +02:00
parent e9ef77617f
commit 9ae7f9c042
5 changed files with 37 additions and 36 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/migrate.js
+25 -24
View File
@@ -15,14 +15,15 @@
'use strict';
import {ProfileManager} from './ProfileManager.js';
import {debug, executeCommand} from './utils.js';
import * as utils from './utils.js';
// Migrating from pre-27 versions is pretty involved. Before, all settings were stored in
// the standard Gio.Settings backend. Now, there are effect profiles. Depending on the
// settings one or two profiles may be required.
export async function fromVersion26() {
return executeCommand(['dconf', 'dump', '/org/gnome/shell/extensions/burn-my-windows/'])
.catch(r => debug('Failed to get old settings for effect migration: ' + r))
return utils
.executeCommand(['dconf', 'dump', '/org/gnome/shell/extensions/burn-my-windows/'])
.catch(r => utils.debug('Failed to get old settings for effect migration: ' + r))
.then(r => {
// If there were no settings before, we do not have to migrate anything. Just use
// the new defaults.
@@ -30,7 +31,7 @@ export async function fromVersion26() {
return;
}
debug('Starting Burn-My-Windows profile migration (old version <= 26)!');
utils.debug('Starting Burn-My-Windows profile migration (old version <= 26)!');
// We use this to write the new profiles.
const profileManager = new ProfileManager();
@@ -38,19 +39,19 @@ export async function fromVersion26() {
// The default value of this was false, so not-present is equal to false.
const destroyDialogs = r.includes('destroy-dialogs=true');
if (!destroyDialogs) {
debug('Only normal windows will be burned by the new profile(s).');
utils.debug('Only normal windows will be burned by the new profile(s).');
}
// The default value of this was false, so not-present is equal to false.
const disableOnBattery = r.includes('disable-on-battery=true');
if (disableOnBattery) {
debug('The new profile(s) will not be active on battery.');
utils.debug('The new profile(s) will not be active on battery.');
}
// The default value of this was false, so not-present is equal to false.
const disableOnPowerSave = r.includes('disable-on-power-save=true');
if (disableOnPowerSave) {
debug('The new profile(s) will not be active in power-save mode.');
utils.debug('The new profile(s) will not be active in power-save mode.');
}
// Remove some unnecessary lines.
@@ -85,21 +86,21 @@ export async function fromVersion26() {
}
// Print all effects we've found.
debug('Enabled open effects: ' + openEffects);
debug('Enabled close effects: ' + closeEffects);
utils.debug('Enabled open effects: ' + openEffects);
utils.debug('Enabled close effects: ' + closeEffects);
// If the same effects were used for opening and closing windows or there is either
// no opening or no closing animation configured, only one profile is required. Else
// we have to create two new profiles.
const numProfiles = openEffects.join() == closeEffects.join() ? 1 : 2;
if (numProfiles == 1) {
debug('Only one profile is required.');
utils.debug('Only one profile is required.');
} else if (closeEffects.length == 0) {
debug('Only a window-open profile is required.');
utils.debug('Only a window-open profile is required.');
} else if (openEffects.length == 0) {
debug('Only a window-close profile is required.');
utils.debug('Only a window-close profile is required.');
} else {
debug('Two profiles are required.');
utils.debug('Two profiles are required.');
}
// Remove all open / close lines.
@@ -140,8 +141,8 @@ export async function fromVersion26() {
// Print and save the new profile.
const profile = profileLines.join('\n');
debug('The new profile:');
debug(profile);
utils.debug('The new profile:');
utils.debug(profile);
profileManager.createProfile(profile);
@@ -162,8 +163,8 @@ export async function fromVersion26() {
// Print and save the new profile.
const profile = profileLines.join('\n');
debug('The new window-open profile:');
debug(profile);
utils.debug('The new window-open profile:');
utils.debug(profile);
profileManager.createProfile(profile);
@@ -184,8 +185,8 @@ export async function fromVersion26() {
// Print and save the new profile.
const profile = profileLines.join('\n');
debug('The new window-close profile:');
debug(profile);
utils.debug('The new window-close profile:');
utils.debug(profile);
profileManager.createProfile(profile);
@@ -213,14 +214,14 @@ export async function fromVersion26() {
const openProfile = openProfileLines.join('\n');
const closeProfile = closeProfileLines.join('\n');
debug('The new open-window profile:');
debug(openProfile);
debug('The new close-window profile:');
debug(closeProfile);
utils.debug('The new open-window profile:');
utils.debug(openProfile);
utils.debug('The new close-window profile:');
utils.debug(closeProfile);
profileManager.createProfile(openProfile);
profileManager.createProfile(closeProfile);
}
})
.catch(r => debug('Failed to migrate settings: ' + r));
.catch(r => utils.debug('Failed to migrate settings: ' + r));
}