<table><tr><td style="">rkflx requested changes to this revision.<br />rkflx added a comment.<br />This revision now requires changes to proceed.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D12328">View Revision</a></tr></table><br /><div><div><blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>This also enables it for users who previously disabled it.</p></blockquote>

<p>Hm, normally changing a default is allowed, while overriding custom user settings is discouraged. However, for a <tt style="background: #ebebeb; font-size: 13px;">bool</tt> it does not make a difference, really ;)</p>

<p>For KConfigXT settings, changing a default is easy, because defaults are not repeated in the config file, so it's enough to change the default in the code. Here, it's more complicated, as you had to change the name to get that effect.</p>

<p>However, the old key should not stay around in the config file, it should be removed. We have <a href="https://techbase.kde.org/Development/Tools/Using_kconf_update" class="remarkup-link" target="_blank" rel="noreferrer"><tt style="background: #ebebeb; font-size: 13px;">kconf_update</tt></a> for that. Could you provide or update such a script for <tt style="background: #ebebeb; font-size: 13px;">~/.config/kdeglobals</tt>?</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R241 KIO</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D12328">https://phabricator.kde.org/D12328</a></div></div><br /><div><strong>To: </strong>anemeth, Frameworks, VDG, rkflx<br /><strong>Cc: </strong>rkflx, ngraham, Frameworks, michaelh, bruns<br /></div>