Review Request 113129: proxy configuration: fix setting the use of proxy environment variable

[Commit Hook]

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


This review has been submitted with commit 623adc32ca7f4c151a00b7de078bff2aa527201a by Andrea Iacovitti to branch KDE/4.11.

- Commit Hook


On Oct. 6, 2013, 12:36 p.m., Andrea Iacovitti wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/113129/
> -----------------------------------------------------------
> 
> (Updated Oct. 6, 2013, 12:36 p.m.)
> 
> 
> Review request for KDE Base Apps and Dawit Alemayehu.
> 
> 
> Bugs: 304707
>     http://bugs.kde.org/show_bug.cgi?id=304707
> 
> 
> Repository: kde-baseapps
> 
> 
> Description
> -------
> 
> - Use the correct variable, instead of the entire QStringList, to fill in lineedit's text in autoDetectSystemProxy.
> - When saving for "System proxy configuration", in KProxyDialog::save(), be sure to write the name of environment variables and not their values in the configuration file.
> - Added missing signal connection for mUi.systemProxyRadioButton.
> - Connect slotChanged() to textEdited() signal (instead of textChanged()) for systemProxy's lineedits, this avoid to emit changed() when "Show the value of the envirnment variables" checkbox is clicked.
> 
> NOTE:
> Currently (and from kde3 time) the proxy configuration keys in kioslaverc: httpProxy,httpsProxy,ftpProxy,socksProxy,NoProxyFor, are used with a double semantic. They hold the proxy address in case of "Manual Proxy Configuration" or the name of environment variables to use in case of "System Proxy Configuration". (See getSystemProxyFor() in kprotocolmanager.cpp)
> 
> 
> Diffs
> -----
> 
>   konqueror/settings/kio/kproxydlg.cpp 7c777d3 
> 
> Diff: http://git.reviewboard.kde.org/r/113129/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Andrea Iacovitti
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20131006/d95ab72e/attachment.htm>