[plasmalnf] Signal more changes to the model

- also individual changes need to be signalled
- use QSignalBlocker to avoid spamming changes when calling
  aggregate change methods
- refactor findById() so that also a row number can be
  obtained, which is needed for the change signals.

src/modules/plasmalnf/ThemeInfo.cpp

@@ -65,10 +65,11 @@ ThemesModel::roleNames() const
 void
 ThemesModel::setThemeImage( const QString& id, const QString& imagePath )
 {
-    auto* theme = m_themes.findById( id );
+    auto [ i, theme ] = m_themes.indexById( id );
     if ( theme )
     {
         theme->imagePath = imagePath;
+        emit dataChanged( index( i, 0 ), index( i, 0 ), { ImageRole } );
     }
 }
 
@@ -80,20 +81,25 @@ ThemesModel::setThemeImage( const QMap< QString, QString >& images )
         return;
     }
 
+    // Don't emit signals from each call, aggregate to one call (below this block)
+    {
+        QSignalBlocker b( this );
         for ( const auto& k : images )
         {
             setThemeImage( k, images[ k ] );
         }
+    }
     emit dataChanged( index( 0, 0 ), index( m_themes.count() - 1 ), { ImageRole } );
 }
 
 void
 ThemesModel::showTheme( const QString& id, bool show )
 {
-    auto* theme = m_themes.findById( id );
+    auto [ i, theme ] = m_themes.indexById( id );
     if ( theme )
     {
         theme->show = show;
+        emit dataChanged( index( i, 0 ), index( i, 0 ), { ShownRole } );
     }
 }
 
@@ -105,6 +111,8 @@ ThemesModel::showOnlyThemes( const QMap< QString, QString >& onlyThese )
         return;
     }
 
+    // No signal blocker block needed here because we're not calling showTheme()
+    // QSignalBlocker b( this );
     for ( auto& t : m_themes )
     {
         t.show = onlyThese.contains( t.id );

src/modules/plasmalnf/ThemeInfo.h

@@ -52,10 +52,10 @@ struct ThemeInfo
 class ThemeInfoList : public QList< ThemeInfo >
 {
 public:
-    std::pair< int, ThemeInfo* > indexById( const QString& id )
+    std::pair< int, const ThemeInfo* > indexById( const QString& id ) const
     {
         int index = 0;
-        for ( ThemeInfo& i : *this )
+        for ( const ThemeInfo& i : *this )
         {
             if ( i.id == id )
             {
@@ -65,31 +65,26 @@ public:
         return { -1, nullptr };
     }
 
+    std::pair< int, ThemeInfo* > indexById( const QString& id )
+    {
+        // Call the const version and then munge the types
+        auto [ i, p ] = const_cast< const ThemeInfoList* >( this )->indexById( id );
+        return { i, const_cast< ThemeInfo* >( p ) };
+    }
+
 
     /** @brief Looks for a given @p id in the list of themes, returns nullptr if not found. */
     ThemeInfo* findById( const QString& id )
     {
-        for ( ThemeInfo& i : *this )
+        auto [ i, p ] = indexById( id );
-        {
+        return p;
-            if ( i.id == id )
-            {
-                return &i;
-            }
-        }
-        return nullptr;
     }
 
     /** @brief Looks for a given @p id in the list of themes, returns nullptr if not found. */
     const ThemeInfo* findById( const QString& id ) const
     {
-        for ( const ThemeInfo& i : *this )
+        auto [ i, p ] = indexById( id );
-        {
+        return p;
-            if ( i.id == id )
-            {
-                return &i;
-            }
-        }
-        return nullptr;
     }
 
     /** @brief Checks if a given @p id is in the list of themes. */