From 97a1d7e03c676ae59e05d3f3eeee756868fd9f35 Mon Sep 17 00:00:00 2001 From: Sijmen Date: Tue, 25 Apr 2023 03:44:03 +0200 Subject: [PATCH] Show filename for playlists too --- src/mpd.rs | 19 +++++++++++-------- 1 file changed, 11 insertions(+), 8 deletions(-) diff --git a/src/mpd.rs b/src/mpd.rs index 4219034..3f25df6 100644 --- a/src/mpd.rs +++ b/src/mpd.rs @@ -1,12 +1,20 @@ +use std::borrow::Cow; + use mpdrs::lsinfo::LsInfoResponse; pub(crate) const HOST: &str = "192.168.1.203:6600"; pub(crate) async fn ls(path: &str) -> anyhow::Result> { - // TODO mpdrs seems to be the only one to implement lsinfo let mut mpd = mpdrs::Client::connect(HOST)?; let info = mpd.lsinfo(path)?; + fn filename(path: &str) -> Cow { + std::path::Path::new(path) + .file_name() + .map(|x| x.to_string_lossy()) + .unwrap_or(Cow::Borrowed("n/a")) + } + Ok(info .iter() .map(|e| match e { @@ -17,19 +25,14 @@ pub(crate) async fn ls(path: &str) -> anyhow::Result> { }, LsInfoResponse::Directory { path, .. } => { - let filename = std::path::Path::new(&path) - .file_name() - .map(|x| x.to_string_lossy().to_string()) - .unwrap_or("n/a".to_string()); - Entry::Directory { - name: filename, + name: filename(path).to_string(), path: path.to_string(), } } LsInfoResponse::Playlist { path, .. } => Entry::Playlist { - name: path.to_string(), + name: filename(path).to_string(), path: path.to_string(), }, })