hein added a comment.
This makes sense. In theory this is done by this code:
` connect(m_selectedTranslationsModel,
&SelectedTranslationsModel::selectedLanguagesChanged,
m_availableTranslationsModel,
&AvailableTranslationsModel::setSelectedLanguages);`
However `SelectedTranslationsModel` only emits the signal when the list has
changed, and an initial call to `setSelectedLanguages` with both the member and
the parameter being empty doesn't cause an emit.
This means your patch can be simplified though. Only the
`Translations::load()` needs to call
`AvailableTranslationsModel::setSelectedLanguages` to seed things, `save` and
`defaults` are covered by the signal-slot connection keeping the models in sync.
REPOSITORY
R119 Plasma Desktop
REVISION DETAIL
https://phabricator.kde.org/D15115
To: sitter, hein
Cc: ngraham, plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed,
jensreuterberg, abetts, sebas, apol, mart
