<table><tr><td style="">martinkostolny updated this revision to Diff 12058.<br />martinkostolny added a comment.
</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/D4847" rel="noreferrer">View Revision</a></tr></table><br /><div><div><p>Next iteration, still without autotests, I need to study them more.</p>

<p>This diff includes preserving owner and group when file is saved with elevated privileges. Actually loosing owner was (and is) already happening before my patch when user has the permission to write the file and is not his owner. In this case, I believe we have 3 options:</p>

<ol class="remarkup-list">
<li class="remarkup-list-item">Leave it as it is</li>
<li class="remarkup-list-item">Detect different owner and in such situation use direct writing (without QSaveFile)</li>
<li class="remarkup-list-item">Use QSaveFile but in case of different owner ask for elevated privilege to be able to reset the owner</li>
</ol>

<p>Is there another solution I'm missing?</p>

<p>PS: Regarding windows and other non-linux platforms I unfortunately cannot test the code there right now.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R39 KTextEditor</div></div></div><br /><div><strong>CHANGES SINCE LAST UPDATE</strong><div><a href="https://phabricator.kde.org/D4847?vs=12054&id=12058" rel="noreferrer">https://phabricator.kde.org/D4847?vs=12054&id=12058</a></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D4847" rel="noreferrer">https://phabricator.kde.org/D4847</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>src/CMakeLists.txt<br />
src/buffer/katesecuretextbuffer.cpp<br />
src/buffer/katesecuretextbuffer.h<br />
src/buffer/katetextbuffer.cpp<br />
src/buffer/org.kde.ktexteditor.katetextbuffer.actions</div></div></div><br /><div><strong>EMAIL PREFERENCES</strong><div><a href="https://phabricator.kde.org/settings/panel/emailpreferences/" rel="noreferrer">https://phabricator.kde.org/settings/panel/emailpreferences/</a></div></div><br /><div><strong>To: </strong>martinkostolny, dhaumann, KTextEditor<br /><strong>Cc: </strong>dhaumann, graesslin, davidedmundson, palant, kwrite-devel, Frameworks, head7, cullmann, kfunk, sars<br /></div>