✨ add disable option for permission

src/app.rs

@@ -69,7 +69,7 @@ pub struct Cli {
  pub directory_only: bool,
 
  /// How to display permissions [default: rwx]
- #[arg(long, value_name = "MODE", value_parser = ["rwx", "octal"])]
+ #[arg(long, value_name = "MODE", value_parser = ["rwx", "octal", "disable"])]
  pub permission: Option<String>,
 
  /// How to display size [default: default]

src/config_file.rs

@@ -1,3 +1,5 @@
+//! This module provides methods to handle the program's config files and
+//! operations related to this.
 use crate::flags::display::Display;
 use crate::flags::icons::{IconOption, IconTheme};
 use crate::flags::layout::Layout;
@@ -6,8 +8,6 @@ use crate::flags::size::SizeFlag;
 use crate::flags::sorting::{DirGrouping, SortColumn};
 use crate::flags::HyperlinkOption;
 use crate::flags::{ColorOption, ThemeOption};
-///! This module provides methods to handle the program's config files and operations related to
-///! this.
 use crate::print_error;
 
 use std::path::{Path, PathBuf};

src/core.rs

@@ -1,7 +1,8 @@
 use crate::color::Colors;
 use crate::display;
 use crate::flags::{
- ColorOption, Display, Flags, HyperlinkOption, Layout, Literal, SortOrder, ThemeOption,
+ ColorOption, Display, Flags, HyperlinkOption, Layout, Literal, PermissionFlag, SortOrder,
+ ThemeOption,
 };
 use crate::git::GitCache;
 use crate::icon::Icons;
@@ -105,7 +106,11 @@ impl Core {
  };
 
  for path in paths {
- let mut meta = match Meta::from_path(&path, self.flags.dereference.0) {
+ let mut meta = match Meta::from_path(
+ &path,
+ self.flags.dereference.0,
+ self.flags.permission == PermissionFlag::Disable,
+ ) {
  Ok(meta) => meta,
  Err(err) => {
  print_error!("{}: {}.", path.display(), err);

src/display.rs

@@ -683,7 +683,7 @@ mod tests {
  dir.child("one.d").create_dir_all().unwrap();
  dir.child("one.d/two").touch().unwrap();
  dir.child("one.d/.hidden").touch().unwrap();
- let mut metas = Meta::from_path(Path::new(dir.path()), false)
+ let mut metas = Meta::from_path(Path::new(dir.path()), false, false)
  .unwrap()
  .recurse_into(42, &flags, None)
  .unwrap()
@@ -716,7 +716,7 @@ mod tests {
  let dir = assert_fs::TempDir::new().unwrap();
  dir.child("dir").create_dir_all().unwrap();
  dir.child("dir/file").touch().unwrap();
- let metas = Meta::from_path(Path::new(dir.path()), false)
+ let metas = Meta::from_path(Path::new(dir.path()), false, false)
  .unwrap()
  .recurse_into(42, &flags, None)
  .unwrap()
@@ -757,7 +757,7 @@ mod tests {
  let dir = assert_fs::TempDir::new().unwrap();
  dir.child("dir").create_dir_all().unwrap();
  dir.child("dir/file").touch().unwrap();
- let metas = Meta::from_path(Path::new(dir.path()), false)
+ let metas = Meta::from_path(Path::new(dir.path()), false, false)
  .unwrap()
  .recurse_into(42, &flags, None)
  .unwrap()
@@ -797,7 +797,7 @@ mod tests {
  let dir = assert_fs::TempDir::new().unwrap();
  dir.child("one.d").create_dir_all().unwrap();
  dir.child("one.d/two").touch().unwrap();
- let metas = Meta::from_path(Path::new(dir.path()), false)
+ let metas = Meta::from_path(Path::new(dir.path()), false, false)
  .unwrap()
  .recurse_into(42, &flags, None)
  .unwrap()
@@ -828,7 +828,7 @@ mod tests {
  let dir = assert_fs::TempDir::new().unwrap();
  dir.child("testdir").create_dir_all().unwrap();
  dir.child("test").touch().unwrap();
- let metas = Meta::from_path(Path::new(dir.path()), false)
+ let metas = Meta::from_path(Path::new(dir.path()), false, false)
  .unwrap()
  .recurse_into(1, &flags, None)
  .unwrap()
@@ -862,7 +862,7 @@ mod tests {
 
  let dir = assert_fs::TempDir::new().unwrap();
  dir.child("testdir").create_dir_all().unwrap();
- let metas = Meta::from_path(Path::new(dir.path()), false)
+ let metas = Meta::from_path(Path::new(dir.path()), false, false)
  .unwrap()
  .recurse_into(1, &flags, None)
  .unwrap()
@@ -892,11 +892,11 @@ mod tests {
 
  let file_path = tmp_dir.path().join("file");
  std::fs::File::create(&file_path).expect("failed to create the file");
- let file = Meta::from_path(&file_path, false).unwrap();
+ let file = Meta::from_path(&file_path, false, false).unwrap();
 
  let dir_path = tmp_dir.path().join("dir");
  std::fs::create_dir(&dir_path).expect("failed to create the dir");
- let dir = Meta::from_path(&dir_path, false).unwrap();
+ let dir = Meta::from_path(&dir_path, false, false).unwrap();
 
  assert_eq!(
  display_folder_path(&dir),
@@ -942,15 +942,15 @@ mod tests {
 
  let file_path = tmp_dir.path().join("file");
  std::fs::File::create(&file_path).expect("failed to create the file");
- let file = Meta::from_path(&file_path, false).unwrap();
+ let file = Meta::from_path(&file_path, false, false).unwrap();
 
  let dir_path = tmp_dir.path().join("dir");
  std::fs::create_dir(&dir_path).expect("failed to create the dir");
- let dir = Meta::from_path(&dir_path, false).unwrap();
+ let dir = Meta::from_path(&dir_path, false, false).unwrap();
 
  let link_path = tmp_dir.path().join("link");
  std::os::unix::fs::symlink("dir", &link_path).unwrap();
- let link = Meta::from_path(&link_path, false).unwrap();
+ let link = Meta::from_path(&link_path, false, false).unwrap();
 
  let grid_flags = Flags {
  layout: Layout::Grid,

src/flags/blocks.rs

@@ -350,7 +350,7 @@ mod test_blocks {
  fn test_from_cli_none() {
  let argv = ["lsd"];
  let cli = Cli::try_parse_from(argv).unwrap();
- assert!(matches!(Blocks::from_cli(&cli), None));
+ assert!(Blocks::from_cli(&cli).is_none());
  }
 
  #[test]

src/flags/ignore_globs.rs

@@ -153,14 +153,11 @@ mod test {
  fn test_from_cli_none() {
  let argv = ["lsd"];
  let cli = Cli::try_parse_from(argv).unwrap();
- assert!(matches!(IgnoreGlobs::from_cli(&cli), None));
+ assert!(IgnoreGlobs::from_cli(&cli).is_none());
  }
 
  #[test]
  fn test_from_config_none() {
- assert!(matches!(
- IgnoreGlobs::from_config(&Config::with_none()),
- None
- ));
+ assert!(IgnoreGlobs::from_config(&Config::with_none()).is_none());
  }
 }

src/flags/permission.rs

@@ -17,13 +17,16 @@ pub enum PermissionFlag {
  Rwx,
  /// The variant to show file permissions in octal format
  Octal,
+ /// Disable the display of owner and permissions, may be used to speed up in Windows
+ Disable,
 }
 
 impl PermissionFlag {
  fn from_arg_str(value: &str) -> Self {
  match value {
  "rwx" => Self::Rwx,
  "octal" => Self::Octal,
+ "disable" => Self::Disable,
  // Invalid value should be handled by `clap` when building an `Cli`
  other => unreachable!("Invalid value '{other}' for 'permission'"),
  }
@@ -96,6 +99,16 @@ mod test {
  assert_eq!(Some(PermissionFlag::Octal), PermissionFlag::from_cli(&cli));
  }
 
+ #[test]
+ fn test_from_cli_permissions_disable() {
+ let argv = ["lsd", "--permission", "disable"];
+ let cli = Cli::try_parse_from(argv).unwrap();
+ assert_eq!(
+ Some(PermissionFlag::Disable),
+ PermissionFlag::from_cli(&cli)
+ );
+ }
+
  #[test]
  #[should_panic]
  fn test_from_cli_unknown() {

src/flags/recursion.rs

@@ -162,7 +162,7 @@ mod test {
  fn test_depth_from_cli_empty() {
  let argv = ["lsd"];
  let cli = Cli::try_parse_from(argv).unwrap();
- assert!(matches!(cli.depth, None));
+ assert!(cli.depth.is_none());
  }
 
  #[test]

src/icon.rs

@@ -85,7 +85,7 @@ mod test {
  let tmp_dir = tempdir().expect("failed to create temp dir");
  let file_path = tmp_dir.path().join("file.txt");
  File::create(&file_path).expect("failed to create file");
- let meta = Meta::from_path(&file_path, false).unwrap();
+ let meta = Meta::from_path(&file_path, false, false).unwrap();
 
  let icons = Icons::new(true, IconOption::Never, FlagTheme::Fancy, " ".to_string());
  let icon = icons.get(&meta.name);
@@ -97,7 +97,7 @@ mod test {
  let tmp_dir = tempdir().expect("failed to create temp dir");
  let file_path = tmp_dir.path().join("file.txt");
  File::create(&file_path).expect("failed to create file");
- let meta = Meta::from_path(&file_path, false).unwrap();
+ let meta = Meta::from_path(&file_path, false, false).unwrap();
 
  let icons = Icons::new(false, IconOption::Never, FlagTheme::Fancy, " ".to_string());
  let icon = icons.get(&meta.name);
@@ -110,7 +110,7 @@ mod test {
  let tmp_dir = tempdir().expect("failed to create temp dir");
  let file_path = tmp_dir.path().join("file.txt");
  File::create(&file_path).expect("failed to create file");
- let meta = Meta::from_path(&file_path, false).unwrap();
+ let meta = Meta::from_path(&file_path, false, false).unwrap();
 
  let icons = Icons::new(false, IconOption::Auto, FlagTheme::Fancy, " ".to_string());
  let icon = icons.get(&meta.name);
@@ -122,7 +122,7 @@ mod test {
  let tmp_dir = tempdir().expect("failed to create temp dir");
  let file_path = tmp_dir.path().join("file.txt");
  File::create(&file_path).expect("failed to create file");
- let meta = Meta::from_path(&file_path, false).unwrap();
+ let meta = Meta::from_path(&file_path, false, false).unwrap();
 
  let icons = Icons::new(true, IconOption::Auto, FlagTheme::Fancy, " ".to_string());
  let icon = icons.get(&meta.name);
@@ -135,7 +135,7 @@ mod test {
  let tmp_dir = tempdir().expect("failed to create temp dir");
  let file_path = tmp_dir.path().join("file");
  File::create(&file_path).expect("failed to create file");
- let meta = Meta::from_path(&file_path, false).unwrap();
+ let meta = Meta::from_path(&file_path, false, false).unwrap();
 
  let icon = Icons::new(true, IconOption::Always, FlagTheme::Fancy, " ".to_string());
  let icon_str = icon.get(&meta.name);
@@ -148,7 +148,7 @@ mod test {
  let tmp_dir = tempdir().expect("failed to create temp dir");
  let file_path = tmp_dir.path().join("file");
  File::create(&file_path).expect("failed to create file");
- let meta = Meta::from_path(&file_path, false).unwrap();
+ let meta = Meta::from_path(&file_path, false, false).unwrap();
 
  let icon = Icons::new(false, IconOption::Always, FlagTheme::Fancy, " ".to_string());
  let icon_str = icon.get(&meta.name);
@@ -161,7 +161,7 @@ mod test {
  let tmp_dir = tempdir().expect("failed to create temp dir");
  let file_path = tmp_dir.path().join("file");
  File::create(&file_path).expect("failed to create file");
- let meta = Meta::from_path(&file_path, false).unwrap();
+ let meta = Meta::from_path(&file_path, false, false).unwrap();
 
  let icon = Icons::new(
  false,
@@ -178,7 +178,7 @@ mod test {
  fn get_icon_default_directory() {
  let tmp_dir = tempdir().expect("failed to create temp dir");
  let file_path = tmp_dir.path();
- let meta = Meta::from_path(file_path, false).unwrap();
+ let meta = Meta::from_path(file_path, false, false).unwrap();
 
  let icon = Icons::new(false, IconOption::Always, FlagTheme::Fancy, " ".to_string());
  let icon_str = icon.get(&meta.name);
@@ -190,7 +190,7 @@ mod test {
  fn get_icon_default_directory_unicode() {
  let tmp_dir = tempdir().expect("failed to create temp dir");
  let file_path = tmp_dir.path();
- let meta = Meta::from_path(file_path, false).unwrap();
+ let meta = Meta::from_path(file_path, false, false).unwrap();
 
  let icon = Icons::new(
  false,
@@ -210,7 +210,7 @@ mod test {
  for (file_name, file_icon) in &IconTheme::get_default_icons_by_name() {
  let file_path = tmp_dir.path().join(file_name);
  File::create(&file_path).expect("failed to create file");
- let meta = Meta::from_path(&file_path, false).unwrap();
+ let meta = Meta::from_path(&file_path, false, false).unwrap();
 
  let icon = Icons::new(false, IconOption::Always, FlagTheme::Fancy, " ".to_string());
  let icon_str = icon.get(&meta.name);
@@ -226,7 +226,7 @@ mod test {
  for (ext, file_icon) in &IconTheme::get_default_icons_by_extension() {
  let file_path = tmp_dir.path().join(format!("file.{ext}"));
  File::create(&file_path).expect("failed to create file");
- let meta = Meta::from_path(&file_path, false).unwrap();
+ let meta = Meta::from_path(&file_path, false, false).unwrap();
 
  let icon = Icons::new(false, IconOption::Always, FlagTheme::Fancy, " ".to_string());
  let icon_str = icon.get(&meta.name);

src/meta/filetype.rs

@@ -144,7 +144,7 @@ mod test {
  fn test_dir_type() {
  let tmp_dir = tempdir().expect("failed to create temp dir");
  #[cfg(not(windows))]
- let meta = crate::meta::Meta::from_path(tmp_dir.path(), false)
+ let meta = crate::meta::Meta::from_path(tmp_dir.path(), false, false)
  .expect("failed to get tempdir path");
  let metadata = tmp_dir.path().metadata().expect("failed to get metas");