<table><tr><td style="">sredman created this revision.<br />sredman added a reviewer: KDE Connect.<br />Herald added a project: KDE Connect.<br />Herald added a subscriber: kdeconnect.<br />sredman requested review of this revision.
</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/D17634">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>This patch fixes <a href="https://phabricator.kde.org/T10183" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">T10183</a> and stops the SMS app from crashing when a conversation is selected but no devices are connected. It also allows the SMS app to access the cached messages in the ConversationsDbusInterface so the app is still slightly useful even when the device is disconnected.</p></div></div><br /><div><strong>TEST PLAN</strong><div><ul class="remarkup-list">
<li class="remarkup-list-item">Open sms app</li>
<li class="remarkup-list-item">Open a few conversations</li>
<li class="remarkup-list-item">Disconnect phone (Force close app?)</li>
<li class="remarkup-list-item">Re-open a conversation which was previously opened<ul class="remarkup-list">
<li class="remarkup-list-item">Verify that the messages appear. It is possible to scroll up to view any older cached messages too!</li>
</ul></li>
<li class="remarkup-list-item">Open a conversation which was not opened previously<ul class="remarkup-list">
<li class="remarkup-list-item">Verify that a single messages is shown (since this was the only one in cache, from populating the list of all conversations)</li>
<li class="remarkup-list-item">Verify that attempting to scroll this conversation does nothing, but also does not crash the app</li>
</ul></li>
</ul>

<p>Note: Opening the app with no phone connected will cause it to lose its handle on the deviceId, so it can't spawn a new Dbus interface, so it will remain blank and empty. Solving that is a project for another day.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R224 KDE Connect</div></div></div><br /><div><strong>BRANCH</strong><div><div>smsapp-cachable</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D17634">https://phabricator.kde.org/D17634</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>plugins/sms/conversationsdbusinterface.cpp<br />
plugins/sms/requestconversationworker.cpp<br />
plugins/sms/requestconversationworker.h<br />
smsapp/conversationlistmodel.cpp<br />
smsapp/qml/ConversationDisplay.qml<br />
smsapp/qml/ConversationList.qml</div></div></div><br /><div><strong>To: </strong>sredman, KDE Connect<br /><strong>Cc: </strong>kdeconnect, mschroeder, varunp, shivanshukantprasad, skymoore, brute4s99, wistak, anoopv, dvalencia, rmenezes, julioc, Leptopoda, timothyc, Danial0_0, johnq, Pitel, adeen-s, SemperPeritus, daniel.z.tg, jeanv, seebauer, bugzy, MayeulC, menasshock, mikesomov, tctara, apol<br /></div>