ETM-Schurig/powersync-app
Archived
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Fix AWS config casing

Browse Source 
Now that we rely on the ui to set a config value, we need to deal with
an old bug we had, which might had left users with a lowercase
config in the settings.
This commit is contained in:
Michael Zanetti 2020-01-14 12:51:30 +01:00
parent fa99ea9d63
commit 2a2f47ef5d
1 changed files with 12 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

17
libnymea-app-core/connection/awsclient.cpp
View File

@ -730,13 +730,20 @@ QString AWSClient::config() const
void AWSClient::setConfig(const QString &config)
{
if (m_usedConfig != config) {
if (!m_configs.contains(config)) {
qWarning() << "AWS: Config" << config << "not known. Not switching AWS config";
// We had a bug in some version where the UI would set "community" instead of "Community".
// Let's correct that here in case the user still has the wrong value in its config.
QString fixedConfig = config;
if (fixedConfig.length() > 0) {
fixedConfig = fixedConfig.at(0).toUpper() + fixedConfig.right(fixedConfig.length() - 1);
}
if (m_usedConfig != fixedConfig) {
if (!m_configs.contains(fixedConfig)) {
qWarning() << "AWS: Config" << fixedConfig << "not known. Not switching AWS config";
return;
}
qDebug() << "Setting AWS configuration to" << config;
m_usedConfig = config;
qDebug() << "Setting AWS configuration to" << fixedConfig;
m_usedConfig = fixedConfig;
emit configChanged();
}
}