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

[Andrea Iacovitti]

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

(Updated Oct. 6, 2013, 1:25 p.m.)


Status
------

This change has been marked as submitted.


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/68b2d710/attachment.htm>