Dark-Alex-17/managarr
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

refactor: Refactored a handful of Option calls to use take instead

Browse Source
This commit is contained in:
Alex Clarke
2025-01-18 13:00:21 -07:00
parent fd35106df8
commit 652bbcd5d4
2 changed files with 6 additions and 11 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/app/radarr/mod.rs
+2 -3
View File
@@ -157,12 +157,11 @@ impl<'a> App<'a> {
async fn check_for_radarr_prompt_action(&mut self) {
if self.data.radarr_data.prompt_confirm {
self.data.radarr_data.prompt_confirm = false;
if let Some(radarr_event) = &self.data.radarr_data.prompt_confirm_action {
if let Some(radarr_event) = self.data.radarr_data.prompt_confirm_action.take() {
self
.dispatch_network_event(radarr_event.clone().into())
.dispatch_network_event(radarr_event.into())
.await;
self.should_refresh = true;
self.data.radarr_data.prompt_confirm_action = None;
}
}
}
src/handlers/radarr_handlers/mod.rs
+4 -8
View File
@@ -126,10 +126,8 @@ pub fn handle_change_tab_left_right_keys(app: &mut App<'_>, key: Key) {
#[macro_export]
macro_rules! search_table {
($app:expr, $data_ref:ident, $error_block:expr) => {
let search_index = if let Some(search_str) = $app.data.radarr_data.search.as_ref() {
let search_string = search_str.text.clone().to_lowercase();
$app.data.radarr_data.search = None;
let search_index = if let Some(search_str) = $app.data.radarr_data.search.take() {
let search_string = search_str.text.to_lowercase();
$app
.data
@@ -153,10 +151,8 @@ macro_rules! search_table {
}
};
($app:expr, $data_ref:ident, $error_block:expr, $option:ident) => {
let search_index = if let Some(search_str) = $app.data.radarr_data.search.as_ref() {
let search_string = search_str.text.clone().to_lowercase();
$app.data.radarr_data.search = None;
let search_index = if let Some(search_str) = $app.data.radarr_data.search.take() {
let search_string = search_str.text.to_lowercase();
$app
.data