D12558: Don't filter out duplicated entries from places panel

Elvis Angelaccio noreply at phabricator.kde.org
Sun Apr 29 12:03:09 BST 2018


elvisangelaccio accepted this revision as: elvisangelaccio.
elvisangelaccio added a comment.
This revision is now accepted and ready to land.


  This seems indeed an unrelated change from D9985 <https://phabricator.kde.org/D9985>. Since that commit doesn't explain why this code was needed, I think we can drop it.
  
  In D12558#254797 <https://phabricator.kde.org/D12558#254797>, @ngraham wrote:
  
  > However is this what we want, behaviorally? The code you're removing was designed to prevent showing duplicates.
  
  
  Nope, that is handled by D9333 <https://phabricator.kde.org/D9333> and it should still work after this patch.

REPOSITORY
  R318 Dolphin

REVISION DETAIL
  https://phabricator.kde.org/D12558

To: arojas, #dolphin, ngraham, elvisangelaccio, renatoo
Cc: xyquadrat
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.kde.org/mailman/private/kfm-devel/attachments/20180429/51600558/attachment.htm>


More information about the kfm-devel mailing list