Review Request: Get rid of excessive Knm::Connection::typeAsString usage and remove hardcoded scope strings to make it i18n friendly.

[Lamarque Vieira Souza]

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



settings/config/manageconnectionwidget.cpp
<http://git.reviewboard.kde.org/r/101354/#comment2771>

    You must not save a translatable type in configuration file. If the user changes translation between two Plasma NM starts that could accidently changes the scope of the connections. That is the reason why scope was treated as English string.
    
    Or you always save the English string or the enum type converted to int. I prefer to save the enum type converted to int, that is less error prone.


- Lamarque Vieira


On May 13, 2011, 1:31 p.m., Gökcen Eraslan wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/101354/
> -----------------------------------------------------------
> 
> (Updated May 13, 2011, 1:31 p.m.)
> 
> 
> Review request for Network Management and Lamarque Vieira Souza.
> 
> 
> Summary
> -------
> 
> 2 things are done with this patch:
> 
> * "System" and "User" keywords are not translatable and hardcoded. There is a Knm::Connection::scopeAsString method but it's not used in anywhere. Patch makes use of it and also change this method to use i18ned words. Now I can see translated uses of System and User words in UI.
> 
> *  A little refactoring done in settings/config/manageconnectionwidget.cpp to remove excessive usage of Knm::Connection::typeAsString. I've used the enum value of Type to use a switch-case structure instead of string comparison with if-elses.
> 
> 
> Diffs
> -----
> 
>   libs/internals/connection.h bb7e21b 
>   libs/internals/connection.cpp 7085b1b 
>   settings/config/manageconnectionwidget.cpp 6d3dd44 
> 
> Diff: http://git.reviewboard.kde.org/r/101354/diff
> 
> 
> Testing
> -------
> 
> I did not encountered with an error.
> 
> 
> Thanks,
> 
> Gökcen
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.kde.org/pipermail/kde-networkmanager/attachments/20110513/c02adc26/attachment.htm