mirror of
https://github.com/element-hq/element-web.git
synced 2024-12-15 20:51:44 +03:00
cb6f415a05
Fixes https://github.com/vector-im/riot-web/issues/6435 This is done through an on-the-fly inverter for the settings. All the settings changed are boolean values, so this should be more than safe to just let happen throughout the SettingsStore. Typically a change like this would be done in the individual handlers (similar to how setting names are remapped to different properties or even different storage locations on the fly), however doing that for this many settings would be a huge nightmare and involve changing *all* the layers. By putting a global "invert this" flag on the setting, we can get away with doing the inversion as the last possible step during a read (or write). To speed up calculations of the default values, we cache all the inverted values into a lookup table similar to how we represent the defaults already. Without this, the DefaultHandler would need to iterate the setting list and invert the values, slowing things down over time. We invert the value up front so we can keep the generic inversion logic without checking the level ahead of time. It is fully intended that a default value represents the new setting name, not the legacy name. This commit also includes a debugger for settings because it was hard to visualize what the SettingsStore was doing during development. Some added information is included as it may be helpful for when someone has a problem with their settings and we need to debug it. Typically the debugger would be run in conjunction with `mxSendRageshake`: `mxSettingsStore.debugSetting('showJoinLeaves') && mxSendRageshake('Debugging showJoinLeaves setting')`.
54 lines
1.7 KiB
JavaScript
54 lines
1.7 KiB
JavaScript
/*
|
|
Copyright 2017 Travis Ralston
|
|
|
|
Licensed under the Apache License, Version 2.0 (the "License");
|
|
you may not use this file except in compliance with the License.
|
|
You may obtain a copy of the License at
|
|
|
|
http://www.apache.org/licenses/LICENSE-2.0
|
|
|
|
Unless required by applicable law or agreed to in writing, software
|
|
distributed under the License is distributed on an "AS IS" BASIS,
|
|
WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
|
|
See the License for the specific language governing permissions and
|
|
limitations under the License.
|
|
*/
|
|
|
|
import SettingsHandler from "./SettingsHandler";
|
|
|
|
/**
|
|
* Gets settings at the "default" level. This handler does not support setting values.
|
|
* This handler does not make use of the roomId parameter.
|
|
*/
|
|
export default class DefaultSettingsHandler extends SettingsHandler {
|
|
/**
|
|
* Creates a new default settings handler with the given defaults
|
|
* @param {object} defaults The default setting values, keyed by setting name.
|
|
* @param {object} invertedDefaults The default inverted setting values, keyed by setting name.
|
|
*/
|
|
constructor(defaults, invertedDefaults) {
|
|
super();
|
|
this._defaults = defaults;
|
|
this._invertedDefaults = invertedDefaults;
|
|
}
|
|
|
|
getValue(settingName, roomId) {
|
|
let value = this._defaults[settingName];
|
|
if (value === undefined) {
|
|
value = this._invertedDefaults[settingName];
|
|
}
|
|
return value;
|
|
}
|
|
|
|
setValue(settingName, roomId, newValue) {
|
|
throw new Error("Cannot set values on the default level handler");
|
|
}
|
|
|
|
canSetValue(settingName, roomId) {
|
|
return false;
|
|
}
|
|
|
|
isSupported() {
|
|
return true;
|
|
}
|
|
}
|