mab122
/
radicle-heartwood-lfs
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

cli: Handle message editor corner cases 

1. Split on double newline fails when description is empty
2. Title must be a single line
This commit is contained in:
Michael Raitza 2024-02-13 11:50:37 +01:00 committed by cloudhead
parent 3ad2b4431f
commit f0b8446515
No known key found for this signature in database
1 changed files with 4 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
radicle-cli/src/terminal/patch.rs
View File

@ -87,12 +87,13 @@ impl Message {
placeholder.push_str(help); placeholder.push_str(help);
let output = Self::Edit.get(&placeholder)?; let output = Self::Edit.get(&placeholder)?;
let Some((title, description)) = output.split_once("\n\n") else { let (title, description) = match output.split_once("\n\n") {
return Ok(None); Some((x, y)) => (x, y),
None => (output.as_str(), ""),
}; };
let (title, description) = (title.trim(), description.trim()); let (title, description) = (title.trim(), description.trim());
if title.is_empty() { if title.is_empty() | title.contains('\n') {
return Ok(None); return Ok(None);
} }
Ok(Some((title.to_owned(), description.to_owned()))) Ok(Some((title.to_owned(), description.to_owned())))