strange FTP behaviour

David Faure faure at kde.org
Fri Jul 23 15:13:40 BST 2004


On Friday 23 July 2004 15:31, Jens Herden wrote:
> Hi!
> 
> > > And I wonder if this is the right place to fix it? Can't there be a
> > > solution inside of KIO?
> >
> > KIO::file_copy is too lowlevel:
> > When copying a big directory with Konqueror you would end up with a
> > "filesadded" notification after each and every file being copied - we
> > certainly do NOT want that, it triggers a re-listing in
> > KonqIconView/KonqListView! The notification has to be done at the high
> > level where we know for sure when the copy operation is finished, hence the
> > idea to do it in that KParts slot. I guess a solution would be a new
> > bool/enum argument to file_copy to indicate whether KDirNotify notification
> > is wanted. This would still require fixing this in all apps that use
> > KIO::file_copy (or KIO::put!) for saving, but the fix would be a one-liner
> > instead of a 4-liner.
> 
> OK, I see. But what about knetaccess? Shall we add it there too?

NetAccess is probably a good place to add it, indeed. Please make a patch,
otherwise I'll try to remember doing it when I'm back from holidays (see sig.)

> > > BTW. why is allDirNotify always created even if the command is not
> > > CMD_COPY or CMD_RENAME? I guess it would be a bit faster to test first
> > > m_command before creating allDirNotify.
> >
> > Yes, feel free to make that change.
> 
> I will make it. Shall I post the patch to the list first or is it simple 
> enough to commit directly?

Simple enough, go ahead.

-- 
David Faure, faure at kde.org, sponsored by Trolltech to work on KDE,
Konqueror (http://www.konqueror.org), and KOffice (http://www.koffice.org).
On holidays between 24/07 and 07/08.




More information about the kde-core-devel mailing list