shubham added a comment.
In D14610#303706 <https://phabricator.kde.org/D14610#303706>, @rkflx wrote: > In D14610#303537 <https://phabricator.kde.org/D14610#303537>, @rkflx wrote: > > > `KPropertiesDialog::setFileNameReadOnly` > > `m_bFromTemplate` > > > @shubham Any comments on that (see my questions to @dfaure above)? sorry, don't know about it INLINE COMMENTS > rkflx wrote in kpropertiesdialog.cpp:988 > Without your patch, we only tested for `!itemList.supportsMoving()`. Now you > also test for `(itemList.isDirectory() & !itemList.supportsWriting()`. Could > you explain in what cases we need that new test? As far as I can see this > blocks renaming the folder you removed the write permission from (while only > its children should be prevented from being renamed). > > (`&` instead of `&&` also caught my eye.) 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 REPOSITORY R241 KIO REVISION DETAIL https://phabricator.kde.org/D14610 To: shubham, rkflx, dfaure, ngraham Cc: kde-frameworks-devel, michaelh, ngraham, bruns