Skip to content

KFilePlacesModel: Only create places item for known devices

Kai Uwe Broulik requested to merge work/kbroulik/kfileplaces-broken-devices into master

Otherwise we'll create entries for devices that aren't there, leading to broken entries. It would also bluntly overwrite an existing device entry with a broken one further down.

While at it, mutate the container (erase) only after the value has been used.

BUG: 470698


@meven

My Places panel doesn’t show a bunch of empty entries anymore. I can still hide the entry and it stays hidden when I restart the app.

Before:

Screenshot_20230704_130452

After:

Screenshot_20230704_130517

Edited by Kai Uwe Broulik

Merge request reports