[Kde-pim] Review Request: Replace hard coded country list of kaddressbook by the l10n resources maintained in systemsettings

[Urs Joss]

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

(Updated 2009-05-15 12:37:09.548946)


Review request for KDE PIM.


Changes
-------

Using KLocale::allDigitSetsList and KLocale::countryToName to access the countries


Summary
-------

Inspired by bug 168410 I adapted the source for the country list used in countryselectordialog.cpp of kcontrol (roughly lines 271 to 290). Thus kaddressbook reads the system wide country names from as found in the entry.desktop files of the l10n subdirectories.

I did not try to follow up the rationale of implementing a hard-coded list in addresseditwidget.cpp. While it certainly gives kaddressbook developers more freedom in adding countries, it also led to outdated countries. The patch removes the burden of maintaining this list.


This addresses bug 168410.
    https://bugs.kde.org/show_bug.cgi?id=168410


Diffs (updated)
-----

  trunk/KDE/kdepim/kaddressbook/addresseditwidget.cpp 968241 

Diff: http://reviewboard.kde.org/r/700/diff


Testing
-------

I have applied the patch to the kaddressbook-4.2.3 in gentoo. I have successfully modified existing addresses (switching countries). I have also added new addresses to existing contacts, and last but not least created new contacts with new addresses.


Thanks,

Urs

_______________________________________________
KDE PIM mailing list kde-pim at kde.org
https://mail.kde.org/mailman/listinfo/kde-pim
KDE PIM home page at http://pim.kde.org/