#4621 closed Bug (fixed)
Seeding progress bars do not show in Web UI
Reported by: | boywithumbrella | Owned by: | jordan |
---|---|---|---|
Priority: | Normal | Milestone: | 2.51 |
Component: | Web Client | Version: | 2.42 |
Severity: | Normal | Keywords: | backport-2.4x web client, seeding progress |
Cc: |
Description
After upgrading Transmission on my ReadyNAS Duo from 2.33 to 2.42, the dark-green bar that showed seeding progress is no longer present in the Web UI. I've seen on the forums that others seem to have the same problem, but haven't found an appropriate ticket here in the tracker.
The issue is present both in compact view
and in expanded view
I tested it in Safari 5.1.1, Firefox 6.0 and Google Chrome 15 on Mac OS X Lion, with same results.
Attachments (4)
Change History (22)
comment:1 Changed 9 years ago by rb07
comment:2 Changed 9 years ago by jordan
- Keywords backport-2.4x added
- Milestone changed from None Set to 2.50
- Owner set to jordan
- Status changed from new to assigned
comment:3 Changed 9 years ago by jordan
- Resolution set to fixed
- Status changed from assigned to closed
Fixed in r13082. Thanks for reporting this!
comment:4 Changed 9 years ago by rb07
- Resolution fixed deleted
- Status changed from closed to reopened
With version 2.50 a new variation of this problem appeared, best described in this forum post:
https://forum.transmissionbt.com/viewtopic.php?f=2&t=12933#p59094
Changed 9 years ago by cfpp2p
comment:5 Changed 9 years ago by cfpp2p
Here is patch to fix the variation. I'm sure the devs will clean it up if needed. Also there are refresh problems but the issues with refresh, by my testing, were pre-existing and are not related to the patch. Patch works good for all cases of seeding progress bars.
comment:6 Changed 9 years ago by jordan
cfpp2p: trac's moderation system won't let me accept two attachments that share the same name. If you want to submit a second attachment, it needs to be under a different name... :/
comment:7 Changed 9 years ago by cfpp2p
jordon: verified that the correct patch is attached. Thanks for letting me know, I think I somehow attached twice by mistake.
comment:8 Changed 9 years ago by cfpp2p
Patch updated to correct issues with inactive torrents not being refreshed when the seed ratio preferences are changed in the web client's preferences dialog.
comment:9 Changed 9 years ago by jordan
This patch confuses me:
- Why is PrefsDialog invoking Transmission.initializeTorrents(), which is a function previously only called from Transmission's constructor, and which asks the server to re-download all the torrents' metadata + stats?
- Why is PrefsDialog invoking any Transmission methods directly; those calls could be moved to Transmission.onPrefsDialogClosed()?
- Why was Torrent.seedRatioLimit() changed to ignore torrents that are set to seed forever regardless of the global setting?
- Why is PrefsDialog being referenced as the data store at all? Are the values in Transmission._prefs incorrect?
I guess I would be happiest if you would describe what, in the javascript code, the actual bug is.
comment:10 Changed 9 years ago by cfpp2p
comment:11 Changed 9 years ago by jordan
I appreciate the patch, so don't get me wrong -- but linking to a post of yours that just says "Thorough testing, no bugs" in boldface and italics doesn't really answer my questions...
comment:12 Changed 9 years ago by cfpp2p
jordan,
I hope this helps you with your confusion understanding the code. I spent quite a bit of time with the patch so if anyone has free time to tweak it (it already works quite nicely), please feel free.
1.) & 2.) Don't need:
Torrent.Fields.Metadata = [ 'addedDate', 'name', 'totalSize' ];
for simplicity just using existing functions transmission.initializeTorrents(); and transmission.refilter(true);
both could be moved to Transmission.onPrefsDialogClosed()
3.) & 4.) Both broken
Torrent._RatioUseGlobal: return controller.seedRatioLimit(); this._prefs;
fixed and logical flow corrected in patch
comment:13 Changed 9 years ago by jordan
- Milestone changed from 2.50 to 2.51
comment:14 Changed 9 years ago by jordan
- Resolution set to fixed
- Status changed from reopened to closed
Fixed in r13271
comment:15 Changed 9 years ago by cfpp2p
- Resolution fixed deleted
- Status changed from closed to reopened
What am I not seeing/doing here?. With r13271 I no longer see ANY global seed ratio bars, ONLY those with torrent specific ratios defined. With my submitted ratio-rfsh.patch I am able to see bars defined by BOTH the global and torrent specific ratios set. Sorry but I don't understand how r13271 fixes the problem by eliminating showing bars for any torrents using the global seed ratio. If I am in error and not doing something properly please correct me and disregard.
comment:16 Changed 9 years ago by jordan
- Resolution set to fixed
- Status changed from reopened to closed
I've attached a screenshot of the global seed ratio bar behavior I'm seeing now. It looks correct to me, so my first guess is that there's a configuration issue on your end.
If you find this to be in error, please reopen this ticket with more information on how to reproduce the seed ratio bar bug from a fresh checkout of svn trunk. Thanks!
comment:17 Changed 9 years ago by cfpp2p
OK, working correctly in r13283, I just needed to get TRANSMISSION_HOME and TRANSMISSION_WEB_HOME both configured correctly for my setup. Thanks Jordan!
comment:18 Changed 9 years ago by cfpp2p
for anyone interested, with v2.52 using the web-ui for global seed bar ratio changes, changes are not updated immediately when the torrent has no leechers.
my old patch updated updated (to fix an issue with seed forever regardless of the global setting) is attached. Patch tested and in use for over 1 month.
Additionally, as we commented on the forum, the seeding bar sometimes appears if you select the torrent, other times not even selecting makes it appear. From version 2.40 to 2.42 it got worse, it takes more time and/or opening/closing the detail.
Tested with Firefox 7.0.1 and all recent preceding versions. Also IE 8, same weird behavior.
Firefox 8.0, same.
And you definitely need to open the inspector or the seed portion of the bar doesn't appear when you select a torrent. Also the "remaining time" text only appears when selecting the torrent (and the inspector is showing).