Compare commits

..

3 Commits

Author SHA1 Message Date
spinline
743596d701 fix(ci): remove tracing calls that caused compile error in shared crate
All checks were successful
Build MIPS Binary / build (push) Successful in 1m58s
2026-02-21 01:31:25 +03:00
spinline
598f038ea6 fix(files): use refetch callback instead of Action to avoid reactive disposal panic
Some checks failed
Build MIPS Binary / build (push) Failing after 1m10s
2026-02-21 01:29:26 +03:00
spinline
7f8c721115 debug: add tracing logs to set_file_priority for diagnosis
Some checks failed
Build MIPS Binary / build (push) Failing after 1m10s
2026-02-21 01:27:22 +03:00
2 changed files with 17 additions and 20 deletions

View File

@@ -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
let refresh_files = Action::new(|h: &String| {
let h = h.clone();
async move { shared::server_fns::torrent::get_files(h).await.unwrap_or_default() }
// Callback to trigger a refetch — safe, doesn't destroy existing components
let on_refresh = Callback::new(move |_: ()| {
files_resource.refetch();
});
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() {
Some(f) => f,
None => files_resource.get().unwrap_or_default(),
};
let files = 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,7 +81,7 @@ 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();
@@ -107,7 +102,7 @@ fn FileRow(file: TorrentFile, hash: String, refresh_action: Action<String, Vec<T
<FileContextMenu
torrent_hash=hash
file_index=f_idx
refresh_action=refresh_action
on_refresh=on_refresh
set_priority=set_priority
>
<TableRow class="hover:bg-muted/50 transition-colors group">
@@ -141,7 +136,7 @@ fn FileContextMenu(
children: Children,
torrent_hash: String,
file_index: u32,
refresh_action: Action<String, Vec<TorrentFile>>,
on_refresh: Callback<()>,
set_priority: Action<(String, u32, u8), Result<(), ServerFnError>>,
) -> impl IntoView {
let hash_c1 = torrent_hash.clone();
@@ -159,11 +154,11 @@ fn FileContextMenu(
<ContextMenuGroup>
<ContextMenuItem on:click={
let h = hash_c1;
let ra = refresh_action.clone();
let sp = set_priority.clone();
let ra = on_refresh.clone();
move |_| {
sp.dispatch((h.clone(), file_index, 2));
ra.dispatch(h.clone());
ra.run(());
crate::components::ui::context_menu::close_context_menu();
}
}>
@@ -173,11 +168,11 @@ fn FileContextMenu(
<ContextMenuItem on:click={
let h = hash_c2;
let ra = refresh_action.clone();
let sp = set_priority.clone();
let ra = on_refresh.clone();
move |_| {
sp.dispatch((h.clone(), file_index, 1));
ra.dispatch(h.clone());
ra.run(());
crate::components::ui::context_menu::close_context_menu();
}
}>
@@ -187,11 +182,11 @@ 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();
let ra = on_refresh.clone();
move |_| {
sp.dispatch((h.clone(), file_index, 0));
ra.dispatch(h.clone());
ra.run(());
crate::components::ui::context_menu::close_context_menu();
}
}>

View File

@@ -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;