dfaure requested changes to this revision. dfaure added a comment. This revision now requires changes to proceed.
Please re-read your diff and, indeed, revert any unnecessary changes like indentation or no-op moving of code. INLINE COMMENTS > shubham wrote in kpropertiesdialog.cpp:988 > this test was intended for blocking directories without write access to > rename , but as you say it should be done to it's child > > I missed that one & by mistake You can remove the last condition completely. `!itemList.supportsMoving()` is enough to detect the case of files in a non-writable directory. See the unittest I just pushed about that, commit 0799ca8f32 <https://phabricator.kde.org/R241:0799ca8f32226cfe5ad97ce6041f46c4b6795798>. REPOSITORY R241 KIO REVISION DETAIL https://phabricator.kde.org/D14610 To: shubham, rkflx, dfaure, ngraham Cc: kde-frameworks-devel, michaelh, ngraham, bruns