[PATCH] : KIO::chmod that uses KURL::List
kzk
mover at cool.ne.jp
Fri Jul 18 18:17:50 BST 2003
On Thursday 17 July 2003 21:49, David Faure wrote:
> Your idea sounds good, but I'm afraid the "mask" should be removed, since
> there's no way to know the current permissions of the urls - so you can
> only use chmod(list of urls) to apply a complete set of permissions to all
> files, you can't selectively change only one bit on them (like one can do
> with the mask).
>
> To be more precise, the current line 79, which uses item->permissions(),
> will always get -1 in your case, which will result in wrong stuff being
> done, unless m_mask is 0777 (or so).
Thank you for your suggestion.
On my test use, item->permissions() returns currened permission, didn't return
-1.
But I finally found that this code works properly only LOCAL file.
If local file, current permission is reset even if permission ==
KFileItem::Unknown.
Please see at kfileitem.cpp (at line 156, void KFileItem::init( bool
_determineMimeTypeOnDemand ) )
So, I think improving KFileItem::init may solve this problem.
( and I'm sorry that I don't have enough knowledge about KDE/QT to implement
this.... )
bye
kzk
More information about the kde-core-devel
mailing list