From 50cbd16ec7cd1387504d75a2a6e5c70352ff3201 Mon Sep 17 00:00:00 2001 From: onthebridgetonowhere <71919805+onthebridgetonowhere@users.noreply.github.com> Date: Mon, 15 Nov 2021 02:43:40 +0100 Subject: [PATCH] Port str reverse (#337) Co-authored-by: Stefan Stanciulescu --- crates/nu-command/src/default_context.rs | 1 + crates/nu-command/src/strings/str_/mod.rs | 2 + crates/nu-command/src/strings/str_/reverse.rs | 106 ++++++++++++++++++ 3 files changed, 109 insertions(+) create mode 100644 crates/nu-command/src/strings/str_/reverse.rs diff --git a/crates/nu-command/src/default_context.rs b/crates/nu-command/src/default_context.rs index e3ab2bb98..a305c60f0 100644 --- a/crates/nu-command/src/default_context.rs +++ b/crates/nu-command/src/default_context.rs @@ -115,6 +115,7 @@ pub fn create_default_context() -> EngineState { StrSnakeCase, StrLpad, StrRpad, + StrReverse, Sys, Table, To, diff --git a/crates/nu-command/src/strings/str_/mod.rs b/crates/nu-command/src/strings/str_/mod.rs index e968f22f0..ba435300e 100644 --- a/crates/nu-command/src/strings/str_/mod.rs +++ b/crates/nu-command/src/strings/str_/mod.rs @@ -8,6 +8,7 @@ mod find_replace; mod index_of; mod length; mod lpad; +mod reverse; mod rpad; pub use capitalize::SubCommand as StrCapitalize; @@ -20,4 +21,5 @@ pub use find_replace::SubCommand as StrFindReplace; pub use index_of::SubCommand as StrIndexOf; pub use length::SubCommand as StrLength; pub use lpad::SubCommand as StrLpad; +pub use reverse::SubCommand as StrReverse; pub use rpad::SubCommand as StrRpad; diff --git a/crates/nu-command/src/strings/str_/reverse.rs b/crates/nu-command/src/strings/str_/reverse.rs new file mode 100644 index 000000000..b2ab3340c --- /dev/null +++ b/crates/nu-command/src/strings/str_/reverse.rs @@ -0,0 +1,106 @@ +use nu_engine::CallExt; +use nu_protocol::ast::Call; +use nu_protocol::ast::CellPath; +use nu_protocol::engine::{Command, EngineState, Stack}; +use nu_protocol::{Example, PipelineData, ShellError, Signature, Span, SyntaxShape, Value}; + +#[derive(Clone)] +pub struct SubCommand; + +impl Command for SubCommand { + fn name(&self) -> &str { + "str reverse" + } + + fn signature(&self) -> Signature { + Signature::build("str reverse").rest( + "rest", + SyntaxShape::CellPath, + "optionally reverse text by column paths", + ) + } + + fn usage(&self) -> &str { + "outputs the reversals of the strings in the pipeline" + } + + fn run( + &self, + engine_state: &EngineState, + stack: &mut Stack, + call: &Call, + input: PipelineData, + ) -> Result { + operate(engine_state, stack, call, input) + } + + fn examples(&self) -> Vec { + vec![Example { + description: "Return the reversals of multiple strings", + example: "'Nushell' | str reverse", + result: Some(Value::String { + val: "llehsuN".to_string(), + span: Span::unknown(), + }), + }] + } +} + +fn operate( + engine_state: &EngineState, + stack: &mut Stack, + call: &Call, + input: PipelineData, +) -> Result { + let head = call.head; + let column_paths: Vec = call.rest(engine_state, stack, 0)?; + input.map( + move |v| { + if column_paths.is_empty() { + action(&v, head) + } else { + let mut ret = v; + for path in &column_paths { + let r = + ret.update_cell_path(&path.members, Box::new(move |old| action(old, head))); + if let Err(error) = r { + return Value::Error { error }; + } + } + ret + } + }, + engine_state.ctrlc.clone(), + ) +} + +fn action(input: &Value, head: Span) -> Value { + match input { + Value::String { val, .. } => Value::String { + val: val.chars().rev().collect::(), + span: head, + }, + + other => Value::Error { + error: ShellError::UnsupportedInput( + format!( + "Input's type is {}. This command only works with strings.", + other.get_type() + ), + Span::unknown(), + ), + }, + } +} + +#[cfg(test)] +mod tests { + use super::*; + + #[test] + fn test_examples() { + use crate::test_examples; + + test_examples(SubCommand {}) + } +}