After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 367949 - incorrect progress reported and incorrect total percent of copy wrong.
incorrect progress reported and incorrect total percent of copy wrong.
Status: RESOLVED FIXED
Product: gnome-commander
Classification: Other
Component: application
1.2.x
Other Linux
: Normal normal
: 1.2.2
Assigned To: GNOME Commander maintainer(s)
epiotr
: 466432 624281 626853 641220 644790 680860 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2006-10-30 21:55 UTC by Steve mitchell
Modified: 2014-06-16 08:12 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
short video of copy progress dialog box (796.22 KB, video/mpeg)
2006-10-30 22:47 UTC, Steve mitchell
  Details
screen shot while copying - look at the small info panel (159.01 KB, image/png)
2010-10-24 20:49 UTC, LAZA
  Details
Correct feedback when moving files (4.80 KB, patch)
2012-05-04 11:01 UTC, Jan Vleeshouwers
none Details | Review

Description Steve mitchell 2006-10-30 21:55:54 UTC
select a directory in one pane, to be copied or moved to the other pane.
use the function key F5  or F6.
click Ok to copy or move and then the progress dialog box comes up.

it displays 1 of 1 then 2 of 1...127 of 1, etc.

You would expect it to copy 1 of 127, 2 of 127...127 of 127.  
But you have only selected 1 item, a directory--but the number of directories does not represent the number of items (of files) that you are copying/moving.

A second anomaly in the same dialog box is that the percent done takes the size of the directory entry(1k?) as 100% and reports how much of that is copied.  So if you have a directory with 1 meg of files in it, it reports 1000% completed, which is sort of silly if you think about it.

I took a 10 second video (.mpg) of it so you could see what I mean.
Comment 1 Steve mitchell 2006-10-30 22:47:06 UTC
Created attachment 75698 [details]
short video of copy progress dialog box
Comment 2 Peter van der Sar 2009-12-07 11:54:10 UTC
No progress on this bug since 2006?
It's quite an annoying one.
Comment 3 Lisandro Pin 2010-01-26 05:12:40 UTC
Confirmed on v1.2.8.2. The bug is totaly harmless, but makes the progress indicator basically useless.
Comment 4 epiotr 2010-08-15 17:02:36 UTC
*** Bug 626853 has been marked as a duplicate of this bug. ***
Comment 5 LAZA 2010-10-24 20:49:48 UTC
Created attachment 173137 [details]
screen shot while copying - look at the small info panel
Comment 6 LAZA 2010-10-24 20:50:54 UTC
I got here an directory moved and ALL is completly wrong:

First: [Datei 33 von 1]
Second: 4.4 MB von 1024 k
Third: At the beginning the panel said about 446900 % copied. Now it's struck at 0 %... - and totally worthless
Comment 7 Jan Vleeshouwers 2012-05-04 11:01:52 UTC
Created attachment 213437 [details] [review]
Correct feedback when moving files
Comment 8 Jan Vleeshouwers 2012-05-04 11:03:22 UTC
As far as I can see, the problem is related to the GnomeVFS library (bugzilla product gnome-vfs): if gnome_vfs_async_xfer() is called with GnomeVFSXferOptions equal to GNOME_VFS_XFER_REMOVESOURCE, the GnomeVFSXferProgressInfo struct contains invalid data in fields files_total and bytes_total. Looks related to Bug 315336 (a bug filed in 2005, not solved yet apparently).

I think the feedback when moving files should be correct. That means a couple of changes in gnome-cmd-xfer.cc:
a) When moving files (xferOptions == GNOME_VFS_XFER_REMOVESOURCE), determine files_total and bytes_total in create_xfer_data();
b) Update these totals in async_xfer_callback() only if they increase

A slightly altered calc_tree_size() (utils.cc) can be used to determine the totals. The function needs to:
- return number of files as well as the byte count
- also return valid data when the argument is not a directory
Comment 9 Uwe Scholz 2014-04-08 15:13:46 UTC
Jan, I checked the patch you committed nearly two years ago and everything works fine. I just had to insert two missing braces in utils.cc, line 792 and 796.

Actually, the original bug occurs only when moving files in v1.4.1. Nevertheless, now GCMD displays the correct number of files and folders when moving them. Perfect and many thanks!

The fix will go into the next bugfix release of GCMD.
Comment 10 Uwe Scholz 2014-04-14 06:04:18 UTC
*** Bug 466432 has been marked as a duplicate of this bug. ***
Comment 11 Uwe Scholz 2014-04-29 09:26:02 UTC
*** Bug 641220 has been marked as a duplicate of this bug. ***
Comment 12 Uwe Scholz 2014-05-04 16:40:45 UTC
*** Bug 680860 has been marked as a duplicate of this bug. ***
Comment 13 Uwe Scholz 2014-05-07 17:38:02 UTC
*** Bug 624281 has been marked as a duplicate of this bug. ***
Comment 14 Uwe Scholz 2014-06-16 08:12:52 UTC
*** Bug 644790 has been marked as a duplicate of this bug. ***