davidre updated this revision to Diff 61318. davidre added a comment.
Don't start if we are in config mode I had another crash here via reload -> processPaths -> endInsertRows. Triggered by repeatetly checking and unchecking a checkbox. My guess is that because of multiple calls to reload and beginInsertRows, endInsertRows there were some inconsistencies. With this I can't crash it anymore by checking and unchecking a checkbox. REPOSITORY R120 Plasma Workspace CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D22121?vs=61314&id=61318 BRANCH slideshow (branched from master) REVISION DETAIL https://phabricator.kde.org/D22121 AFFECTED FILES wallpapers/image/CMakeLists.txt wallpapers/image/backgroundlistmodel.h wallpapers/image/image.cpp wallpapers/image/image.h wallpapers/image/imagepackage/contents/ui/WallpaperDelegate.qml wallpapers/image/imagepackage/contents/ui/config.qml wallpapers/image/imagepackage/contents/ui/main.qml wallpapers/image/slidefiltermodel.cpp wallpapers/image/slidefiltermodel.h wallpapers/image/slidemodel.cpp wallpapers/image/slidemodel.h wallpapers/image/slideshowpackage/contents/config/main.xml To: davidre, #plasma, davidedmundson Cc: davidedmundson, msdobrescu, ngraham, filipf, plasma-devel, LeGast00n, jraleigh, fbampaloukas, GB_2, ragreen, Pitel, ZrenBot, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart