Merge PR #638: Fix current index in main menu after a setup is aborted

This commit is contained in:
Jenkins nymea 2021-08-11 21:44:26 +02:00
commit c2235df414

View File

@ -17,8 +17,8 @@ ConfiguredHostsModel::ConfiguredHostsModel(QObject *parent) : QAbstractListModel
host->setName(cachedName);
addHost(host);
settings.endGroup();
m_currentIndex = settings.value("currentIndex", 0).toInt();
}
m_currentIndex = settings.value("currentIndex", 0).toInt();
settings.endGroup();
// If there aren't any in the config, try migrating settings from old tab model
@ -42,6 +42,11 @@ ConfiguredHostsModel::ConfiguredHostsModel(QObject *parent) : QAbstractListModel
if (m_list.isEmpty()) {
createHost();
}
// Make sure the currentIndex from the config isn't out of place
if (m_currentIndex >= m_list.count()) {
m_currentIndex = m_list.count()-1;
}
}
int ConfiguredHostsModel::rowCount(const QModelIndex &parent) const