mirror of
https://github.com/nushell/nushell.git
synced 2025-04-16 17:28:19 +02:00
# Description This PR allows the `into string` command to pass the `--group-digits` flag which already existed in this code but was hard coded to `false`. Now you can do things like ```nushell ❯ 1234567890 | into string --group-digits 1,234,567,890 ❯ ls | into string size --group-digits | last 5 ╭─#─┬────────name─────────┬─type─┬──size──┬───modified───╮ │ 0 │ README.md │ file │ 12,606 │ 4 weeks ago │ │ 1 │ rust-toolchain.toml │ file │ 1,125 │ 2 weeks ago │ │ 2 │ SECURITY.md │ file │ 2,712 │ 7 months ago │ │ 3 │ toolkit.nu │ file │ 21,929 │ 2 months ago │ │ 4 │ typos.toml │ file │ 542 │ 7 months ago │ ╰─#─┴────────name─────────┴─type─┴──size──┴───modified───╯ ❯ "12345" | into string --group-digits 12,345 ``` # User-Facing Changes <!-- List of all changes that impact the user experience here. This helps us keep track of breaking changes. --> # Tests + Formatting <!-- Don't forget to add tests that cover your changes. Make sure you've run and fixed any issues with these commands: - `cargo fmt --all -- --check` to check standard code formatting (`cargo fmt --all` applies these changes) - `cargo clippy --workspace -- -D warnings -D clippy::unwrap_used` to check that you're using the standard code style - `cargo test --workspace` to check that all tests pass (on Windows make sure to [enable developer mode](https://learn.microsoft.com/en-us/windows/apps/get-started/developer-mode-features-and-debugging)) - `cargo run -- -c "use toolkit.nu; toolkit test stdlib"` to run the tests for the standard library > **Note** > from `nushell` you can also use the `toolkit` as follows > ```bash > use toolkit.nu # or use an `env_change` hook to activate it automatically > toolkit check pr > ``` --> # After Submitting <!-- If your PR had any user-facing changes, update [the documentation](https://github.com/nushell/nushell.github.io) after the PR is merged, if necessary. This will help us keep the docs up to date. -->
322 lines
11 KiB
Rust
322 lines
11 KiB
Rust
use nu_cmd_base::input_handler::{operate, CmdArgument};
|
|
use nu_engine::command_prelude::*;
|
|
use nu_protocol::{shell_error::into_code, Config};
|
|
use nu_utils::get_system_locale;
|
|
use num_format::ToFormattedString;
|
|
use std::sync::Arc;
|
|
|
|
struct Arguments {
|
|
decimals_value: Option<i64>,
|
|
cell_paths: Option<Vec<CellPath>>,
|
|
config: Arc<Config>,
|
|
group_digits: bool,
|
|
}
|
|
|
|
impl CmdArgument for Arguments {
|
|
fn take_cell_paths(&mut self) -> Option<Vec<CellPath>> {
|
|
self.cell_paths.take()
|
|
}
|
|
}
|
|
|
|
#[derive(Clone)]
|
|
pub struct SubCommand;
|
|
|
|
impl Command for SubCommand {
|
|
fn name(&self) -> &str {
|
|
"into string"
|
|
}
|
|
|
|
fn signature(&self) -> Signature {
|
|
Signature::build("into string")
|
|
.input_output_types(vec![
|
|
(Type::Binary, Type::String),
|
|
(Type::Int, Type::String),
|
|
(Type::Number, Type::String),
|
|
(Type::String, Type::String),
|
|
(Type::Glob, Type::String),
|
|
(Type::Bool, Type::String),
|
|
(Type::Filesize, Type::String),
|
|
(Type::Date, Type::String),
|
|
(Type::Duration, Type::String),
|
|
(Type::Range, Type::String),
|
|
(
|
|
Type::List(Box::new(Type::Any)),
|
|
Type::List(Box::new(Type::String)),
|
|
),
|
|
(Type::table(), Type::table()),
|
|
(Type::record(), Type::record()),
|
|
])
|
|
.allow_variants_without_examples(true) // https://github.com/nushell/nushell/issues/7032
|
|
.rest(
|
|
"rest",
|
|
SyntaxShape::CellPath,
|
|
"For a data structure input, convert data at the given cell paths.",
|
|
)
|
|
.switch(
|
|
"group-digits",
|
|
"group digits together by the locale specific thousands separator",
|
|
Some('g'),
|
|
)
|
|
.named(
|
|
"decimals",
|
|
SyntaxShape::Int,
|
|
"decimal digits to which to round",
|
|
Some('d'),
|
|
)
|
|
.category(Category::Conversions)
|
|
}
|
|
|
|
fn description(&self) -> &str {
|
|
"Convert value to string."
|
|
}
|
|
|
|
fn search_terms(&self) -> Vec<&str> {
|
|
vec!["convert", "text"]
|
|
}
|
|
|
|
fn run(
|
|
&self,
|
|
engine_state: &EngineState,
|
|
stack: &mut Stack,
|
|
call: &Call,
|
|
input: PipelineData,
|
|
) -> Result<PipelineData, ShellError> {
|
|
string_helper(engine_state, stack, call, input)
|
|
}
|
|
|
|
fn examples(&self) -> Vec<Example> {
|
|
vec![
|
|
Example {
|
|
description: "convert int to string and append three decimal places",
|
|
example: "5 | into string --decimals 3",
|
|
result: Some(Value::test_string("5.000")),
|
|
},
|
|
Example {
|
|
description: "convert float to string and round to nearest integer",
|
|
example: "1.7 | into string --decimals 0",
|
|
result: Some(Value::test_string("2")),
|
|
},
|
|
Example {
|
|
description: "convert float to string",
|
|
example: "1.7 | into string --decimals 1",
|
|
result: Some(Value::test_string("1.7")),
|
|
},
|
|
Example {
|
|
description: "convert float to string and limit to 2 decimals",
|
|
example: "1.734 | into string --decimals 2",
|
|
result: Some(Value::test_string("1.73")),
|
|
},
|
|
Example {
|
|
description: "convert float to string",
|
|
example: "4.3 | into string",
|
|
result: Some(Value::test_string("4.3")),
|
|
},
|
|
Example {
|
|
description: "convert string to string",
|
|
example: "'1234' | into string",
|
|
result: Some(Value::test_string("1234")),
|
|
},
|
|
Example {
|
|
description: "convert boolean to string",
|
|
example: "true | into string",
|
|
result: Some(Value::test_string("true")),
|
|
},
|
|
Example {
|
|
description: "convert date to string",
|
|
example: "'2020-10-10 10:00:00 +02:00' | into datetime | into string",
|
|
result: Some(Value::test_string("Sat Oct 10 10:00:00 2020")),
|
|
},
|
|
Example {
|
|
description: "convert filepath to string",
|
|
example: "ls Cargo.toml | get name | into string",
|
|
result: None,
|
|
},
|
|
Example {
|
|
description: "convert filesize to string",
|
|
example: "1kB | into string",
|
|
result: Some(Value::test_string("1.0 kB")),
|
|
},
|
|
Example {
|
|
description: "convert duration to string",
|
|
example: "9day | into string",
|
|
result: Some(Value::test_string("1wk 2day")),
|
|
},
|
|
]
|
|
}
|
|
}
|
|
|
|
fn string_helper(
|
|
engine_state: &EngineState,
|
|
stack: &mut Stack,
|
|
call: &Call,
|
|
input: PipelineData,
|
|
) -> Result<PipelineData, ShellError> {
|
|
let head = call.head;
|
|
let decimals_value: Option<i64> = call.get_flag(engine_state, stack, "decimals")?;
|
|
let group_digits = call.has_flag(engine_state, stack, "group-digits")?;
|
|
if let Some(decimal_val) = decimals_value {
|
|
if decimal_val.is_negative() {
|
|
return Err(ShellError::TypeMismatch {
|
|
err_message: "Cannot accept negative integers for decimals arguments".to_string(),
|
|
span: head,
|
|
});
|
|
}
|
|
}
|
|
let cell_paths = call.rest(engine_state, stack, 0)?;
|
|
let cell_paths = (!cell_paths.is_empty()).then_some(cell_paths);
|
|
|
|
if let PipelineData::ByteStream(stream, metadata) = input {
|
|
// Just set the type - that should be good enough. There is no guarantee that the data
|
|
// within a string stream is actually valid UTF-8. But refuse to do it if it was already set
|
|
// to binary
|
|
if stream.type_().is_string_coercible() {
|
|
Ok(PipelineData::ByteStream(
|
|
stream.with_type(ByteStreamType::String),
|
|
metadata,
|
|
))
|
|
} else {
|
|
Err(ShellError::CantConvert {
|
|
to_type: "string".into(),
|
|
from_type: "binary".into(),
|
|
span: stream.span(),
|
|
help: Some("try using the `decode` command".into()),
|
|
})
|
|
}
|
|
} else {
|
|
let config = stack.get_config(engine_state);
|
|
let args = Arguments {
|
|
decimals_value,
|
|
cell_paths,
|
|
config,
|
|
group_digits,
|
|
};
|
|
operate(action, args, input, head, engine_state.signals())
|
|
}
|
|
}
|
|
|
|
fn action(input: &Value, args: &Arguments, span: Span) -> Value {
|
|
let digits = args.decimals_value;
|
|
let config = &args.config;
|
|
let group_digits = args.group_digits;
|
|
|
|
match input {
|
|
Value::Int { val, .. } => {
|
|
let decimal_value = digits.unwrap_or(0) as usize;
|
|
let res = format_int(*val, group_digits, decimal_value);
|
|
Value::string(res, span)
|
|
}
|
|
Value::Float { val, .. } => {
|
|
if let Some(decimal_value) = digits {
|
|
let decimal_value = decimal_value as usize;
|
|
Value::string(format!("{val:.decimal_value$}"), span)
|
|
} else {
|
|
Value::string(val.to_string(), span)
|
|
}
|
|
}
|
|
Value::Bool { val, .. } => Value::string(val.to_string(), span),
|
|
Value::Date { val, .. } => Value::string(val.format("%c").to_string(), span),
|
|
Value::String { val, .. } => {
|
|
if group_digits {
|
|
let number = val.parse::<i64>().unwrap_or_default();
|
|
let decimal_value = digits.unwrap_or(0) as usize;
|
|
Value::string(format_int(number, group_digits, decimal_value), span)
|
|
} else {
|
|
Value::string(val.to_string(), span)
|
|
}
|
|
}
|
|
Value::Glob { val, .. } => Value::string(val.to_string(), span),
|
|
|
|
Value::Filesize { val, .. } => {
|
|
if group_digits {
|
|
let decimal_value = digits.unwrap_or(0) as usize;
|
|
Value::string(format_int(val.get(), group_digits, decimal_value), span)
|
|
} else {
|
|
Value::string(input.to_expanded_string(", ", config), span)
|
|
}
|
|
}
|
|
Value::Duration { val: _, .. } => Value::string(input.to_expanded_string("", config), span),
|
|
|
|
Value::Error { error, .. } => Value::string(into_code(error).unwrap_or_default(), span),
|
|
Value::Nothing { .. } => Value::string("".to_string(), span),
|
|
Value::Record { .. } => Value::error(
|
|
// Watch out for CantConvert's argument order
|
|
ShellError::CantConvert {
|
|
to_type: "string".into(),
|
|
from_type: "record".into(),
|
|
span,
|
|
help: Some("try using the `to nuon` command".into()),
|
|
},
|
|
span,
|
|
),
|
|
Value::Binary { .. } => Value::error(
|
|
ShellError::CantConvert {
|
|
to_type: "string".into(),
|
|
from_type: "binary".into(),
|
|
span,
|
|
help: Some("try using the `decode` command".into()),
|
|
},
|
|
span,
|
|
),
|
|
Value::Custom { val, .. } => {
|
|
// Only custom values that have a base value that can be converted to string are
|
|
// accepted.
|
|
val.to_base_value(input.span())
|
|
.and_then(|base_value| match action(&base_value, args, span) {
|
|
Value::Error { .. } => Err(ShellError::CantConvert {
|
|
to_type: String::from("string"),
|
|
from_type: val.type_name(),
|
|
span,
|
|
help: Some("this custom value can't be represented as a string".into()),
|
|
}),
|
|
success => Ok(success),
|
|
})
|
|
.unwrap_or_else(|err| Value::error(err, span))
|
|
}
|
|
x => Value::error(
|
|
ShellError::CantConvert {
|
|
to_type: String::from("string"),
|
|
from_type: x.get_type().to_string(),
|
|
span,
|
|
help: None,
|
|
},
|
|
span,
|
|
),
|
|
}
|
|
}
|
|
|
|
fn format_int(int: i64, group_digits: bool, decimals: usize) -> String {
|
|
let locale = get_system_locale();
|
|
|
|
let str = if group_digits {
|
|
int.to_formatted_string(&locale)
|
|
} else {
|
|
int.to_string()
|
|
};
|
|
|
|
if decimals > 0 {
|
|
let decimal_point = locale.decimal();
|
|
|
|
format!(
|
|
"{}{decimal_point}{dummy:0<decimals$}",
|
|
str,
|
|
decimal_point = decimal_point,
|
|
dummy = "",
|
|
decimals = decimals
|
|
)
|
|
} else {
|
|
str
|
|
}
|
|
}
|
|
|
|
#[cfg(test)]
|
|
mod test {
|
|
use super::*;
|
|
|
|
#[test]
|
|
fn test_examples() {
|
|
use crate::test_examples;
|
|
|
|
test_examples(SubCommand {})
|
|
}
|
|
}
|