fix: statistics as loadable in 1 of 2 cases:

1. it's the first time we are loading the statistics request 2. we have a different request Signed-off-by: Lachezar Lechev <[email protected]>
Stremio · Dec 5, 2023 · e1108e7 · e1108e7
1 parent c74d1c1
commit e1108e7
Showing 1 changed file with 9 additions and 1 deletion.
diff --git a/src/models/streaming_server.rs b/src/models/streaming_server.rs
@@ -63,6 +63,7 @@ pub struct StreamingServer {
     pub base_url: Loadable<Url, EnvError>,
     pub playback_devices: Loadable<Vec<PlaybackDevice>, EnvError>,
     pub torrent: Option<(String, Loadable<ResourcePath, EnvError>)>,
+    /// [`Loadable::Loading`] is used only on the first statistics request.
     pub statistics: Option<Loadable<Statistics, EnvError>>,
 }
 
@@ -179,7 +180,14 @@ impl<E: Env + 'static> UpdateWithCtx<E> for StreamingServer {
             Msg::Action(Action::StreamingServer(ActionStreamingServer::GetStatistics(request))) => {
                 let selected_effects =
                     eq_update(&mut self.selected.statistics, Some(request.to_owned()));
-                let statistics_effects = eq_update(&mut self.statistics, Some(Loadable::Loading));
+                let is_different_request = self.selected.statistics.as_ref() != Some(request);
+                //set the Loading state only on the first fetch of the statistics
+                let statistics_effects = match (&self.statistics, is_different_request) {
+                    (None, _) | (_, true) => {
+                        eq_update(&mut self.statistics, Some(Loadable::Loading))
+                    }
+                    _ => Effects::none().unchanged(),
+                };
                 Effects::one(get_torrent_statistics::<E>(
                     &self.selected.transport_url,
                     request,