<table><tr><td style="">ivan requested changes to this revision.<br />ivan added a comment.<br />This revision now requires changes to proceed.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D10691" rel="noreferrer">View Revision</a></tr></table><br /><div><div><p>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').</p>

<p>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.</p>

<p>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...'" :)</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R159 KActivities Statistics</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D10691" rel="noreferrer">https://phabricator.kde.org/D10691</a></div></div><br /><div><strong>To: </strong>kpiwowarski, Plasma, Frameworks, hein, ivan<br /><strong>Cc: </strong>ivan, plasma-devel, Frameworks, michaelh, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart<br /></div>