BasicsPage: fix "show language selector" default

Since some boolean settings might be stored as string "0" or "1", the previous system no longer works, and it always sets the switch to true. The "no setting" check has been changed to reference `undefined`, so now the switch will only be defaulted to `true` if the setting truly hasn't been set.

Fixes https://github.com/flarum/core/issues/2574
This commit is contained in:
Alexander Skvortsov 2021-01-27 00:03:30 -05:00
parent 8fdddf91a2
commit c749abab21

View File

@ -61,7 +61,7 @@ export default class BasicsPage extends Page {
}
});
if (typeof this.values.show_language_selector() !== 'number') this.values.show_language_selector(1);
if (this.values.show_language_selector() === undefined) this.values.show_language_selector(1);
}
view() {