----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/114105/ -----------------------------------------------------------
Review request for KDE Base Apps and Dawit Alemayehu. Repository: kde-baseapps Description ------- In case of 'system proxy' proxyType, NoProxyFor config key holds the name of the env variable (e.g. no_proxy) and not its value. Because of this KProtocolManager::noProxyFor() can not be used to get NoProxyFor config setting in KProxyDialog::load(), as it returns the resolved value of the environment variable and not its name: i added helper method KSaveIOConfig::noProxyFor() to read that value directly from config file. Also make sure to uncheck showEnvValueCheckBox before filling proxy edit fields with environment variable names in KProxyDialog::on_autoDetectButton_clicked(). Diffs ----- konqueror/settings/kio/kproxydlg.cpp e80afeb konqueror/settings/kio/ksaveioconfig.h 2318198 konqueror/settings/kio/ksaveioconfig.cpp c822f7b Diff: http://git.reviewboard.kde.org/r/114105/diff/ Testing ------- various... Thanks, Andrea Iacovitti
