Review Request 118564: Fix locale-aware reading in KDesktopFile

[Martin Gräßlin]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/118564/
-----------------------------------------------------------

(Updated June 12, 2014, 5:11 a.m.)


Status
------

This change has been marked as submitted.


Review request for KDE Frameworks and John Layt.


Repository: kconfig


Description
-------

Fix locale-aware reading in KDesktopFile

The underlying KConfig used QLocale::name() for getting the locale
aware part. But this returns "de_DE" while the desktop files store
"de".

In addition it constructs a QLocale object instead of using the
system locale. This has the advantage that the usage of
QLocale::setDafault() gets honored by KConfig.


Diffs
-----

  autotests/helper.h PRE-CREATION 
  autotests/kconfigtest.cpp 2b6de0d7d63df6aee69210aa09418628f0b8110a 
  autotests/kdesktopfiletest.h d57351fd6edcefd6a0efd9126e454546cfc63b55 
  autotests/kdesktopfiletest.cpp 494a43c0fb5808a261b9beb56cdc4afd8580ab21 
  src/core/kconfig.cpp ea9746c001e235529a1cdd5865b9e1b5c129b56a 

Diff: https://git.reviewboard.kde.org/r/118564/diff/


Testing
-------

added unit test failed before. I'm not 100 % sure whether using bcp47Name is correct.


Thanks,

Martin Gräßlin

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20140612/1e85a2fb/attachment.html>