diff options
| author | nsfisis <nsfisis@gmail.com> | 2026-06-20 01:16:50 +0900 |
|---|---|---|
| committer | nsfisis <nsfisis@gmail.com> | 2026-06-20 02:22:41 +0900 |
| commit | efec43b3b8827820cf35fe1b73d8e33f5fe84eb4 (patch) | |
| tree | a62bbba72324de48be5f8e689559f8d9e288fc61 /crates/shirabe/src/command/repository_command.rs | |
| parent | cac18ef73a39b4ac41fa4d6ccb753804d4c42cb7 (diff) | |
| download | php-shirabe-efec43b3b8827820cf35fe1b73d8e33f5fe84eb4.tar.gz php-shirabe-efec43b3b8827820cf35fe1b73d8e33f5fe84eb4.tar.zst php-shirabe-efec43b3b8827820cf35fe1b73d8e33f5fe84eb4.zip | |
refactor: auto-fix clippy warnings
Diffstat (limited to 'crates/shirabe/src/command/repository_command.rs')
| -rw-r--r-- | crates/shirabe/src/command/repository_command.rs | 66 |
1 files changed, 33 insertions, 33 deletions
diff --git a/crates/shirabe/src/command/repository_command.rs b/crates/shirabe/src/command/repository_command.rs index 284b523..6ea26d2 100644 --- a/crates/shirabe/src/command/repository_command.rs +++ b/crates/shirabe/src/command/repository_command.rs @@ -36,6 +36,12 @@ pub struct RepositoryCommand { config_source: Option<JsonConfigSource>, } +impl Default for RepositoryCommand { + fn default() -> Self { + Self::new() + } +} + impl RepositoryCommand { pub fn new() -> Self { let mut command = RepositoryCommand { @@ -94,14 +100,13 @@ impl RepositoryCommand { } if let PhpMixed::Array(ref repo_map) = *repo { - if repo_map.len() == 1 { - if let Some(first_val) = repo_map.values().next() { - if matches!(**first_val, PhpMixed::Bool(false)) { - let first_key = repo_map.keys().next().unwrap(); - io.write(&format!("[{}] <info>disabled</info>", first_key)); - continue; - } - } + if repo_map.len() == 1 + && let Some(first_val) = repo_map.values().next() + && matches!(**first_val, PhpMixed::Bool(false)) + { + let first_key = repo_map.keys().next().unwrap(); + io.write(&format!("[{}] <info>disabled</info>", first_key)); + continue; } let name = repo_map @@ -237,12 +242,11 @@ impl Command for RepositoryCommand { _output: std::rc::Rc<std::cell::RefCell<dyn OutputInterface>>, ) -> anyhow::Result<i64> { let action = strtolower( - &input + input .borrow() .get_argument("action")? .as_string() - .unwrap_or("") - .to_string(), + .unwrap_or(""), ); let name = input .borrow() @@ -406,8 +410,24 @@ impl Command for RepositoryCommand { })); } let name_str = name.as_deref().unwrap(); - if let Some(repo) = repos.get(name_str) { - if let PhpMixed::Array(ref repo_map) = *repo { + if let Some(repo) = repos.get(name_str) + && let PhpMixed::Array(ref repo_map) = *repo + { + let url = repo_map.get("url").and_then(|v| v.as_string()); + if let Some(url) = url { + self.get_io().write(url); + return Ok(0); + } + return Err(anyhow::anyhow!(InvalidArgumentException { + message: format!("The {} repository does not have a URL", name_str), + code: 0, + })); + } + for (_key, val) in &repos { + if let PhpMixed::Array(ref repo_map) = *val + && let Some(n) = repo_map.get("name").and_then(|v| v.as_string()) + && n == name_str + { let url = repo_map.get("url").and_then(|v| v.as_string()); if let Some(url) = url { self.get_io().write(url); @@ -419,26 +439,6 @@ impl Command for RepositoryCommand { })); } } - for (_key, val) in &repos { - if let PhpMixed::Array(ref repo_map) = *val { - if let Some(n) = repo_map.get("name").and_then(|v| v.as_string()) { - if n == name_str { - let url = repo_map.get("url").and_then(|v| v.as_string()); - if let Some(url) = url { - self.get_io().write(url); - return Ok(0); - } - return Err(anyhow::anyhow!(InvalidArgumentException { - message: format!( - "The {} repository does not have a URL", - name_str - ), - code: 0, - })); - } - } - } - } Err(anyhow::anyhow!(InvalidArgumentException { message: format!("There is no {} repository defined", name_str), code: 0, |
