<table><tr><td style="">jtamate updated this revision to Diff 37119.<br />jtamate retitled this revision from "make this test work again with new uds implementation" to "Make this test work again with new uds implementation".<br />jtamate edited the summary of this revision. <a href="https://phabricator.kde.org/transactions/detail/PHID-XACT-DREV-c2yeaskjz6vba55/">(Show Details)</a><br />jtamate added a comment.
</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/D13813">View Revision</a></tr></table><br /><div><div><p>Call items.append inside the loop.</p></div></div><br /><div><strong>CHANGES TO REVISION SUMMARY</strong><div><div style="white-space: pre-wrap; color: #74777D;">With the new uds implementation, when some data is meant to replace an old one, it has to use replace() instead of insert() to avoid an assert of data already exists.<br />
<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">Instead of replacing data in one entry, use</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">Use</span> 3 different entries and only insert().</div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R318 Dolphin</div></div></div><br /><div><strong>CHANGES SINCE LAST UPDATE</strong><div><a href="https://phabricator.kde.org/D13813?vs=36976&id=37119">https://phabricator.kde.org/D13813?vs=36976&id=37119</a></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D13813">https://phabricator.kde.org/D13813</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>src/tests/kfileitemmodeltest.cpp</div></div></div><br /><div><strong>To: </strong>jtamate, Dolphin, Frameworks, elvisangelaccio<br /><strong>Cc: </strong>aacid, ngraham, bruns, elvisangelaccio, kfm-devel, spoorun, navarromorales, isidorov, firef, andrebarros, emmanuelp<br /></div>