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

[Urs Joss]

> On 2009-05-18 14:56:00, Thomas McGuire wrote:
> > Committed to KAddressbook and KContactManager in r969783, see http://websvn.kde.org/?view=rev&revision=969783.
> > Thanks for the patch.
> > Please close the review request as submitted.

Thanks, Thomas


- Urs


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


On 2009-05-15 12:37:09, Urs Joss wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviewboard.kde.org/r/700/
> -----------------------------------------------------------
> 
> (Updated 2009-05-15 12:37:09)
> 
> 
> Review request for KDE PIM.
> 
> 
> 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
> -----
> 
>   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/