Fix clippy warnings and typos (#434)

* Fix clippy warnings.

* Fixe more clippy warnings.

* Fix exided lifetime warnings in clippy.

* Fix errors.

* Fix errors.

---------

Co-authored-by: Guy Chronister <[email protected]>

Author: guylamar2006

.github/workflows/main.yml

@@ -14,7 +14,7 @@ jobs:
   test_Ubuntu:
     runs-on: ubuntu-latest
     steps:
-    - uses: actions/checkout@v3
+    - uses: actions/checkout@v4
     - name: Rust Cache
       uses: Swatinem/rust-cache@v2    
     - name: dependencies
@@ -28,7 +28,7 @@ jobs:
   test_Windows:
     runs-on: windows-latest    
     steps:
-    - uses: actions/checkout@v3
+    - uses: actions/checkout@v4
     - name: Install latest stable
       uses: actions-rs/toolchain@v1
       with:
@@ -41,7 +41,7 @@ jobs:
  # test_Macos:
  #   runs-on: macos-latest    
  #   steps:
- #   - uses: actions/checkout@v3
+ #   - uses: actions/checkout@v4
  #   - name: Install latest stable
  #     uses: actions-rs/toolchain@v1
  #     with:

.github/workflows/release_on_v_tag.yml

@@ -38,7 +38,7 @@ jobs:
             asset_name: windows_BoilR.exe
             strip: false      
     steps:
-      - uses: actions/checkout@v3
+      - uses: actions/checkout@v4
       - name: Install dependencies
         if: ${{ matrix.os == 'ubuntu-latest'}}
         run: |

.github/workflows/rust-clippy.yml

@@ -28,7 +28,7 @@ jobs:
       actions: read # only required for a private repository by github/codeql-action/upload-sarif to get the Action run status
     steps:
       - name: Checkout code
-        uses: actions/checkout@v2
+        uses: actions/checkout@v4
 
       - name: Install Rust toolchain
         uses: actions-rs/toolchain@16499b5e05bf2e26879000db0c1d13f7e13fa3af #@v1
@@ -49,7 +49,7 @@ jobs:
         continue-on-error: true
 
       - name: Upload analysis results to GitHub
-        uses: github/codeql-action/upload-sarif@v1
+        uses: github/codeql-action/upload-sarif@v2
         with:
           sarif_file: rust-clippy-results.sarif
           wait-for-processing: true

src/platforms/egs/epic_ui.rs

@@ -13,7 +13,7 @@ impl EpicPlatform {
             };
 
             egui::CollapsingHeader::new(safe_mode_header)
-            .id_source("Epic_Launcher_safe_launch")
+            .id_salt("Epic_Launcher_safe_launch")
             .show(ui, |ui| {
                 ui.label("Some games must be started from the Epic Launcher, select those games below and BoilR will create shortcuts that opens the games through the Epic Launcher.");
                 let manifests =self.epic_manifests.get_or_insert_with(||{

src/platforms/heroic/heroic_platform.rs

@@ -244,7 +244,7 @@ impl GamesPlatform for HeroicPlatform {
             (true, x) => format!("{x} games forced to launch directly"),
         };
 
-        egui::CollapsingHeader::new(safe_mode_header).id_source("Heroic_Launcher_safe_launch").show(ui, |ui| {
+        egui::CollapsingHeader::new(safe_mode_header).id_salt("Heroic_Launcher_safe_launch").show(ui, |ui| {
             if self.settings.default_launch_through_heroic{
                 ui.label("Some games work best when launched directly, select those games below and BoilR will create shortcuts that launch the games directly.");
             } else {

src/platforms/itch/butler_db_parser.rs

@@ -38,15 +38,13 @@ fn parse_path(i: &[u8]) -> nom::IResult<&[u8], DbPaths> {
 
     let candidates = serde_json::from_str::<Vec<Candidate>>(&candidates_json);
     match candidates {
-        Ok(candidates) => {
-            return IResult::Ok((
-                i,
-                DbPaths {
-                    base_path,
-                    paths: candidates.iter().map(|c| c.path.clone()).collect(),
-                },
-            ))
-        }
+        Ok(candidates) => IResult::Ok((
+            i,
+            DbPaths {
+                base_path,
+                paths: candidates.iter().map(|c| c.path.clone()).collect(),
+            },
+        )),
         Err(_err) => {
             //we found a basepath, but no executables
             IResult::Ok((

src/platforms/lutris/game_list_parser.rs

@@ -1,11 +1,7 @@
 use super::lutris_game::LutrisGame;
 
 pub fn parse_lutris_games(input: &str) -> Vec<LutrisGame> {
-    let games = serde_json::from_str::<Vec<LutrisGame>>(input);
-    match games {
-        Ok(games) => games,
-        Err(_err) => Vec::new(),
-    }
+    serde_json::from_str::<Vec<LutrisGame>>(input).unwrap_or_default()
 }
 
 #[cfg(test)]

src/platforms/uplay/mod.rs

@@ -3,4 +3,3 @@ mod platform;
 mod settings;
 
 pub use platform::UplayPlatform;
-pub use settings::UplaySettings;

src/steam/utils.rs

@@ -90,11 +90,11 @@ pub fn get_shortcuts_paths(settings: &SteamSettings) -> eyre::Result<Vec<SteamUs
             let shortcuts_path = Path::new(path.as_str());
             let folder_string = folder_str.to_string();
             if shortcuts_path.exists() {
-                return SteamUsersInfo {
+                SteamUsersInfo {
                     steam_user_data_folder: folder_string,
                     shortcut_path: Some(shortcuts_path.to_string_lossy().to_string()),
                     user_id,
-                };
+                }
             } else {
                 SteamUsersInfo {
                     steam_user_data_folder: folder_string,

src/steamgriddb/cached_search.rs

@@ -11,7 +11,7 @@ pub struct CachedSearch<'a> {
 }
 
 impl<'a> CachedSearch<'a> {
-    pub fn new(client: &steamgriddb_api::Client) -> CachedSearch {
+    pub fn new(client: &'a steamgriddb_api::Client) -> CachedSearch<'a> {
         CachedSearch {
             search_map: get_search_map(),
             client,

src/steamgriddb/downloader.rs

@@ -45,7 +45,7 @@ impl SearchSettings for Settings {
     }
 }
 
-pub async fn download_images_for_users<'b>(
+pub async fn download_images_for_users(
     settings: &Settings,
     users: &[SteamUsersInfo],
     sender: &mut Option<Sender<SyncProgress>>,
@@ -388,10 +388,10 @@ async fn get_steam_image_url(game_id: usize, image_type: &ImageType) -> Option<S
                 d.platforms
                     .map(|p| p.steam.map(|s| s.metadata.map(|m| m.store_asset_mtime)))
             });
-            if let (Some(Some(Some(steam_app_id))), Some(Some(Some(Some(Some(mtime)))))) =
+            if let (Some(Some(Some(steam_app_id))), Some(Some(Some(Some(mtime))))) =
                 (game_id, mtime)
             {
-                return Some(image_type.steam_url(steam_app_id, mtime));
+                return Some(image_type.steam_url(steam_app_id.to_string(), mtime?));
             }
         }
     }
@@ -412,10 +412,10 @@ async fn get_steam_icon_url(game_id: usize) -> Option<String> {
                 d.platforms
                     .map(|p| p.steam.map(|s| s.metadata.map(|m| m.clienticon)))
             });
-            if let (Some(Some(Some(steam_app_id))), Some(Some(Some(Some(Some(mtime)))))) =
+            if let (Some(Some(Some(steam_app_id))), Some(Some(Some(Some(mtime))))) =
                 (game_id, mtime)
             {
-                return Some(icon_url(&steam_app_id, &mtime));
+                return Some(icon_url(&steam_app_id.to_string(), &mtime?.to_string()));
             }
         }
     }

src/ui/components/steam_user_select.rs

@@ -26,6 +26,6 @@ pub fn render_user_select<'a>(
             None
         }
     } else {
-        return steam_users.first();
+        steam_users.first()
     }
 }

src/ui/images/image_select_state.rs

@@ -1,7 +1,6 @@
-
 use steam_shortcuts_util::shortcut::ShortcutOwned;
 
-use crate::{steam::SteamUsersInfo, steamgriddb::ImageType, ui::FetcStatus};
+use crate::{steam::SteamUsersInfo, steamgriddb::ImageType, ui::FetchStatus};
 
 use super::{ gamemode::GameMode, possible_image::PossibleImage,  gametype::GameType};
 
@@ -18,7 +17,7 @@ pub struct ImageSelectState {
     pub user_shortcuts: Option<Vec<ShortcutOwned>>,
     pub game_mode: GameMode,
     pub image_type_selected: Option<ImageType>,
-    pub image_options: Receiver<FetcStatus<Vec<PossibleImage>>>,
+    pub image_options: Receiver<FetchStatus<Vec<PossibleImage>>>,
     pub steam_games: Option<Vec<crate::steam::SteamGameInfo>>,
 
     pub possible_names: Option<Vec<steamgriddb_api::search::SearchResult>>,
@@ -38,9 +37,8 @@ impl Default for ImageSelectState {
             game_mode: GameMode::Shortcuts,
             image_type_selected: Default::default(),
             possible_names: None,
-            image_options: watch::channel(FetcStatus::NeedsFetched).1,
+            image_options: watch::channel(FetchStatus::NeedsFetched).1,
             steam_games: None,
         }
     }
 }
-

src/ui/images/pages/pick_new_image.rs

@@ -11,7 +11,7 @@ use crate::{
             constants::MAX_WIDTH, hasimagekey::HasImageKey, image_select_state::ImageSelectState,
             possible_image::PossibleImage, useraction::UserAction,
         },
-        FetcStatus, MyEguiApp,
+        FetchStatus, MyEguiApp,
     },
 };
 
@@ -51,7 +51,7 @@ pub fn render_page_pick_image(
     let columns = (width / (column_width + column_padding)).floor() as u32;
     let mut column = 0;
     match &*state.image_options.borrow() {
-        FetcStatus::Fetched(images) => {
+        FetchStatus::Fetched(images) => {
             let x = Grid::new("ImageThumbnailSelectGrid")
                 .spacing([column_padding, column_padding])
                 .show(ui, |ui| {
@@ -128,7 +128,7 @@ pub fn handle_image_selected(app: &mut MyEguiApp, image: PossibleImage) {
 
         {
             app.image_selected_state.image_type_selected = None;
-            app.image_selected_state.image_options = watch::channel(FetcStatus::NeedsFetched).1;
+            app.image_selected_state.image_options = watch::channel(FetchStatus::NeedsFetched).1;
         }
     }
 }

src/ui/images/ui_image_download.rs

@@ -19,7 +19,7 @@ use crate::{
     steam::{get_installed_games, SteamUsersInfo},
     steamgriddb::{get_image_extension, get_query_type, ImageType},
     sync::{download_images, SyncProgress},
-    ui::{components::render_user_select, FetcStatus, MyEguiApp},
+    ui::{components::render_user_select, FetchStatus, MyEguiApp},
 };
 use egui::ScrollArea;
 use futures::executor::block_on;
@@ -274,7 +274,7 @@ impl MyEguiApp {
     fn handle_image_type_selected(&mut self, image_type: ImageType) {
         let state = &mut self.image_selected_state;
         state.image_type_selected = Some(image_type);
-        let (tx, rx) = watch::channel(FetcStatus::Fetching);
+        let (tx, rx) = watch::channel(FetchStatus::Fetching);
         self.image_selected_state.image_options = rx;
         let settings = self.settings.clone();
         if let Some(auth_key) = settings.steamgrid_db.auth_key {
@@ -298,7 +298,7 @@ impl MyEguiApp {
                                 full_url: possible_image.url.clone(),
                             });
                         }
-                        let _ = tx.send(FetcStatus::Fetched(result));
+                        let _ = tx.send(FetchStatus::Fetched(result));
                     }
                 });
             }

src/ui/ui_disconnect.rs

@@ -9,15 +9,15 @@ use crate::sync::disconnect_shortcut;
 use crate::sync::IsBoilRShortcut;
 
 #[derive(Default)]
-pub struct DiconnectState {
+pub struct DisconnectState {
     pub connected_shortcuts: Option<Result<Vec<ShortcutInfo>, String>>,
 }
 
 impl MyEguiApp {
     pub fn render_disconnect(&mut self, ui: &mut egui::Ui) {
         let steam_settings = self.settings.steam.clone();
         let users_info = self
-            .disconect_state
+            .disconnect_state
             .connected_shortcuts
             .get_or_insert_with(|| {
                 let users = get_shortcuts_paths(&steam_settings)
@@ -56,7 +56,7 @@ impl MyEguiApp {
 
                         for user in users.iter_mut() {
                             if has_multiple_users {
-                                ui.heading(&user.path.to_string_lossy().to_string());
+                                ui.heading(user.path.to_string_lossy().to_string());
                             }
                             for shortcut in user.shortcuts.iter() {
                                 if shortcut.is_boilr_shortcut()
@@ -69,7 +69,7 @@ impl MyEguiApp {
                         }
                     });
                 if redraw != 0 {
-                    self.disconect_state.connected_shortcuts = None;
+                    self.disconnect_state.connected_shortcuts = None;
                     self.settings.blacklisted_games.push(redraw);
                 }
             }

src/ui/ui_import_games.rs

@@ -22,18 +22,18 @@ use super::{
 
 const SECTION_SPACING: f32 = 25.0;
 
-pub enum FetcStatus<T> {
+pub enum FetchStatus<T> {
     NeedsFetched,
     Fetching,
     Fetched(T),
 }
 
-impl<T> FetcStatus<T> {
+impl<T> FetchStatus<T> {
     pub fn is_some(&self) -> bool {
         match self {
-            FetcStatus::NeedsFetched => false,
-            FetcStatus::Fetching => false,
-            FetcStatus::Fetched(_) => true,
+            FetchStatus::NeedsFetched => false,
+            FetchStatus::Fetching => false,
+            FetchStatus::Fetched(_) => true,
         }
     }
 }
@@ -58,14 +58,14 @@ impl MyEguiApp {
             for (name,status) in &self.games_to_sync{
                 ui.heading(name);
                 match &*status.borrow(){
-                    FetcStatus::NeedsFetched => {ui.label("Need to find games");},
-                    FetcStatus::Fetching => {
+                    FetchStatus::NeedsFetched => {ui.label("Need to find games");},
+                    FetchStatus::Fetching => {
                         ui.horizontal(|ui|{
                             ui.spinner();
                             ui.label("Finding installed games");
                         });
                     },
-                    FetcStatus::Fetched(shortcuts) => {
+                    FetchStatus::Fetched(shortcuts) => {
                         match shortcuts{
                             Ok(shortcuts) => {
                                 if shortcuts.is_empty(){