D10691: [kactivities-stats] Fix plasmashell crash when database is broken
Ivan Čukić
noreply at phabricator.kde.org
Wed Feb 21 00:02:55 UTC 2018
ivan requested changes to this revision.
ivan added a comment.
This revision now requires changes to proceed.
Well, in a library that 99% relies on a database, having no database is not something that can be gracefully handled (the returned nullptr is asserted on as well in the function that calls ::instance in ResultSet :) - because of this, I'm changing to 'Request Changes').
Graceful handling instead of qFatal/Q_ASSERT could be implemented so that all models and queries return empty sets. Which would be useless, but as I said, I don't have anything against that.
When we get the bug reports about empty favourites, there will be an additional test step "Hi, do you have this message in the plasmashell output 'Database can not be opened...'" :)
REPOSITORY
R159 KActivities Statistics
REVISION DETAIL
https://phabricator.kde.org/D10691
To: kpiwowarski, #plasma, #frameworks, hein, ivan
Cc: ivan, plasma-devel, #frameworks, michaelh, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20180221/3d63d72c/attachment.html>
More information about the Kde-frameworks-devel
mailing list