Review Request 113850: Update the Places Panel entries when switching the language

[Commit Hook]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/113850/#review43701
-----------------------------------------------------------


This review has been submitted with commit 6dd2ae4e15b52ac09859637786dddb4a028e0d4f by Frank Reininghaus to branch KDE/4.12.

- Commit Hook


On Nov. 14, 2013, 8:43 a.m., Frank Reininghaus wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/113850/
> -----------------------------------------------------------
> 
> (Updated Nov. 14, 2013, 8:43 a.m.)
> 
> 
> Review request for Dolphin and Burkhard Lück.
> 
> 
> Repository: kde-baseapps
> 
> 
> Description
> -------
> 
> Currently, we store the translated "Places" in .kde4/share/apps/kfileplaces/bookmarks.xml. This is not intentional - it only happens because PlacesItem::updateBookmarkForRole(const QByteArray& role) always stores the translated text (returned by PlacesItem::text()) in the KBookmark.
> 
> This can be fixed by first checking if the text() is equal to the translation of the text that is already stored in the KBookmark, and not updating it in that case. Note that we have to make sure that all "Places"-related use the same context "KFile System Bookmarks" to make that work.
> 
> The first version of the patch (see https://bugs.kde.org/show_bug.cgi?id=319282#c11) had a small problem: the "Places" texts would be reset to the en_US values on first use after less than a second. This was caused by PlacesItem::setBookmark(const KBookmark& bookmark), which reset the PlacesItem's values to the ones from the untranslated bookmark, even if the bookmark had not changed at all. This is fixed by the new check in that function.
> 
> 
> Diffs
> -----
> 
>   dolphin/src/panels/places/placesitem.cpp 76b51e3 
> 
> Diff: http://git.reviewboard.kde.org/r/113850/diff/
> 
> 
> Testing
> -------
> 
> Fixes the bug for me - now it always shows the correct translation for me even after switching the language, at least if I delete the file .kde4/share/apps/kfileplaces/bookmarks.xml initially (it still had the translated value from the buggy old version written to it).
> 
> 
> Thanks,
> 
> Frank Reininghaus
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.kde.org/mailman/private/kfm-devel/attachments/20131115/b2fe4ce7/attachment.htm>