We should not add a new entry LIMIT_BW_KEY in the dictionary since that will require a FFe (since the API changes), and also because is not necessary.
The bug is clearly where you detected it (the missing call to limits_enabled = yield self.sd_client.bandwidth_throttling_enabled()), so we should indeed add that call, and then make both result[DOWNLOAD_KEY] = limits['download'] and result[UPLOAD_KEY] be -1, since the agreement between the UI and the backend is:
- if limits are -1, then it means that throttling is disabled.
- if any limit is not -1, then throttling is enabled for that limit.
Thanks for working on this!
We should not add a new entry LIMIT_BW_KEY in the dictionary since that will require a FFe (since the API changes), and also because is not necessary.
The bug is clearly where you detected it (the missing call to limits_enabled = yield self.sd_ client. bandwidth_ throttling_ enabled( )), so we should indeed add that call, and then make both result[ DOWNLOAD_ KEY] = limits['download'] and result[UPLOAD_KEY] be -1, since the agreement between the UI and the backend is:
- if limits are -1, then it means that throttling is disabled.
- if any limit is not -1, then throttling is enabled for that limit.
Does the above make sense?