fix(files): trigger refetch only after set_priority completes on server

frontend/src/components/torrent/files.rs

@@ -15,10 +15,9 @@ pub fn TorrentFilesTab(hash: String) -> impl IntoView {
         |h| async move { shared::server_fns::torrent::get_files(h).await.unwrap_or_default() }
     );
 
-    // Refresh action
+    // Callback to trigger a refetch — safe, doesn't destroy existing components
-    let refresh_files = Action::new(|h: &String| {
+    let on_refresh = Callback::new(move |_: ()| {
-        let h = h.clone();
+        files_resource.refetch();
-        async move { shared::server_fns::torrent::get_files(h).await.unwrap_or_default() }
     });
 
     let stored_hash = StoredValue::new(hash);
@@ -26,10 +25,7 @@ pub fn TorrentFilesTab(hash: String) -> impl IntoView {
     view! {
         <Suspense fallback=move || view! { <FilesFallback /> }>
             {move || {
-                let files = match refresh_files.value().get() {
+                let files = files_resource.get().unwrap_or_default();
-                    Some(f) => f,
-                    None => files_resource.get().unwrap_or_default(),
-                };
 
                 if files.is_empty() {
                     return view! {
@@ -61,12 +57,11 @@ pub fn TorrentFilesTab(hash: String) -> impl IntoView {
                                         key=|f| f.index
                                         children={move |f| {
                                             let p_hash = stored_hash.get_value();
-                                            let r_files = refresh_files.clone();
                                             view! {
                                                 <FileRow 
                                                     file=f 
                                                     hash=p_hash
-                                                    refresh_action=r_files
+                                                    on_refresh=on_refresh.clone()
                                                 />
                                             }
                                         }}
@@ -86,11 +81,14 @@ pub fn TorrentFilesTab(hash: String) -> impl IntoView {
 }
 
 #[component]
-fn FileRow(file: TorrentFile, hash: String, refresh_action: Action<String, Vec<TorrentFile>>) -> impl IntoView {
+fn FileRow(file: TorrentFile, hash: String, on_refresh: Callback<()>) -> impl IntoView {
     let f_idx = file.index;
     let path_clone = file.path.clone();
 
-    let set_priority = Action::new(|req: &(String, u32, u8)| {
+    // on_refresh is called AFTER the server responds, not before
+    let on_refresh_stored = StoredValue::new(on_refresh);
+
+    let set_priority = Action::new(move |req: &(String, u32, u8)| {
         let (h, idx, p) = req.clone();
         async move {
             let res = shared::server_fns::torrent::set_file_priority(h, idx, p).await;
@@ -98,6 +96,8 @@ fn FileRow(file: TorrentFile, hash: String, refresh_action: Action<String, Vec<T
                 crate::store::show_toast(shared::NotificationLevel::Error, format!("Öncelik değiştirilemedi: {:?}", e));
             } else {
                 crate::store::show_toast(shared::NotificationLevel::Success, "Dosya önceliği güncellendi.".to_string());
+                // Refetch AFTER the server has saved the priority
+                on_refresh_stored.get_value().run(());
             }
             res
         }
@@ -107,7 +107,6 @@ fn FileRow(file: TorrentFile, hash: String, refresh_action: Action<String, Vec<T
         <FileContextMenu 
             torrent_hash=hash 
             file_index=f_idx 
-            refresh_action=refresh_action 
             set_priority=set_priority
         >
             <TableRow class="hover:bg-muted/50 transition-colors group">
@@ -141,7 +140,6 @@ fn FileContextMenu(
     children: Children,
     torrent_hash: String,
     file_index: u32,
-    refresh_action: Action<String, Vec<TorrentFile>>,
     set_priority: Action<(String, u32, u8), Result<(), ServerFnError>>,
 ) -> impl IntoView {
     let hash_c1 = torrent_hash.clone();
@@ -159,11 +157,9 @@ fn FileContextMenu(
                 <ContextMenuGroup>
                     <ContextMenuItem on:click={
                         let h = hash_c1;
-                        let ra = refresh_action.clone();
                         let sp = set_priority.clone();
                         move |_| {
                             sp.dispatch((h.clone(), file_index, 2));
-                            ra.dispatch(h.clone());
                             crate::components::ui::context_menu::close_context_menu();
                         }
                     }>
@@ -173,11 +169,9 @@ fn FileContextMenu(
                     
                     <ContextMenuItem on:click={
                         let h = hash_c2;
-                        let ra = refresh_action.clone();
                         let sp = set_priority.clone();
                         move |_| {
                             sp.dispatch((h.clone(), file_index, 1));
-                            ra.dispatch(h.clone());
                             crate::components::ui::context_menu::close_context_menu();
                         }
                     }>
@@ -187,11 +181,9 @@ fn FileContextMenu(
                     
                     <ContextMenuItem class="text-destructive focus:bg-destructive/10" on:click={
                         let h = hash_c3;
-                        let ra = refresh_action.clone();
                         let sp = set_priority.clone();
                         move |_| {
                             sp.dispatch((h.clone(), file_index, 0));
-                            ra.dispatch(h.clone());
                             crate::components::ui::context_menu::close_context_menu();
                         }
                     }>

shared/src/server_fns/torrent.rs

@@ -225,6 +225,7 @@ pub async fn set_file_priority(
     let ctx = expect_context::<crate::ServerContext>();
     let client = RtorrentClient::new(&ctx.scgi_socket_path);
 
+    // rTorrent f.set_priority takes: target = "HASH:fINDEX", value = priority
     let target = format!("{}:f{}", hash, file_index);
     let params = vec![
         RpcParam::from(target.as_str()),
@@ -234,8 +235,9 @@ pub async fn set_file_priority(
     client
         .call("f.set_priority", &params)
         .await
-        .map_err(|e| ServerFnError::new(format!("RPC error: {}", e)))?;
+        .map_err(|e| ServerFnError::new(format!("RPC error setting priority: {}", e)))?;
 
+    // Notify rTorrent to update its internal priority state
     let _ = client
         .call("d.update_priorities", &[RpcParam::from(hash.as_str())])
         .await;