https://bugs.kde.org/show_bug.cgi?id=384741

--- Comment #3 from David E. Narvaez <david.narv...@computer.org> ---
(In reply to Thomas Fischer from comment #2)
> I saw only after writing this patch and an answer to this bug report that
> you had proposed a patch on your own in phabricator:
> https://phabricator.kde.org/D7838
> 
> The patch in attachment 107942 [details] is more complex than your patch in
> D7838, but attachment 107942 [details] shines by having comments, showing
> the user both the original and the new id/key, and covering the problem of
> empty ids/keys.
> Therefore, I would go with attachment 107942 [details] as a solution to this
> bug, but please feel free to comment or suggest changes/improvements.

Works for me.

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to