diff --git a/src-tauri/src/db/mod.rs b/src-tauri/src/db/mod.rs index 5183c8c2..f01ad454 100644 --- a/src-tauri/src/db/mod.rs +++ b/src-tauri/src/db/mod.rs @@ -430,7 +430,7 @@ pub async fn convert_pgn( let uncompressed: Box = if extension == "bz2" { Box::new(bzip2::read::MultiBzDecoder::new(file)) } else if extension == "zst" { - Box::new(zstd::Decoder::new(file).expect("zstd decoder")) + Box::new(zstd::Decoder::new(file)?) } else { Box::new(file) }; @@ -551,7 +551,7 @@ pub async fn get_db_info( _ => "".to_string(), }; - let storage_size = path.metadata().expect("get metadata").len() as usize; + let storage_size = path.metadata()?.len() as usize; let filename = path.file_name().expect("get filename").to_string_lossy(); let is_indexed = check_index_exists(db)?; @@ -845,8 +845,7 @@ pub async fn get_games( count = Some( count_query .select(diesel::dsl::count(games::id)) - .first(db) - .expect("count games"), + .first(db)?, ); } @@ -855,7 +854,7 @@ pub async fn get_games( // diesel::debug_query::(&sql_query) // ); - let games: Vec<(Game, Player, Player, Event, Site)> = sql_query.load(db).expect("load games"); + let games: Vec<(Game, Player, Player, Event, Site)> = sql_query.load(db)?; let normalized_games = normalize_games(games); Ok(QueryResponse { @@ -934,7 +933,7 @@ pub async fn get_players( } if !query.options.skip_count { - count = Some(count_query.count().get_result(db).expect("count players")); + count = Some(count_query.count().get_result(db)?); } if let Some(limit) = query.options.page_size { @@ -960,7 +959,7 @@ pub async fn get_players( }, }; - let players = sql_query.load::(db).expect("load players"); + let players = sql_query.load::(db)?; Ok(QueryResponse { data: players, @@ -1000,7 +999,7 @@ pub async fn get_tournaments( } if !query.options.skip_count { - count = Some(count_query.count().get_result(db).expect("count players")); + count = Some(count_query.count().get_result(db)?); } if let Some(limit) = query.options.page_size { @@ -1022,7 +1021,7 @@ pub async fn get_tournaments( }, }; - let events = sql_query.load::(db).expect("load events"); + let events = sql_query.load::(db)?; Ok(QueryResponse { data: events, @@ -1059,7 +1058,7 @@ pub async fn get_players_game_info( .filter(games::white_id.eq(id).or(games::black_id.eq(id))); let info: Vec<(i32, i32, Option, Option, Option)> = - sql_query.load(db).expect("load games"); + sql_query.load(db)?; let mut game_info = PlayerGameInfo::default(); diff --git a/src-tauri/src/db/search.rs b/src-tauri/src/db/search.rs index 85461e3b..ba561dec 100644 --- a/src-tauri/src/db/search.rs +++ b/src-tauri/src/db/search.rs @@ -212,8 +212,7 @@ pub async fn search_position( games::white_material, games::black_material, )) - .load(db) - .expect("load games"); + .load(db)?; info!("got {} games: {:?}", games.len(), start.elapsed()); } @@ -281,8 +280,7 @@ pub async fn search_position( .inner_join(events::table.on(games::event_id.eq(events::id))) .inner_join(sites::table.on(games::site_id.eq(sites::id))) .filter(games::id.eq_any(ids)) - .load(db) - .expect("load games"); + .load(db)?; let normalized_games = normalize_games(games); let openings: Vec = openings.into_iter().map(|(_, v)| v).collect(); @@ -322,8 +320,7 @@ pub async fn is_position_in_db( games::white_material, games::black_material, )) - .load(db) - .expect("load games"); + .load(db)?; info!("got {} games: {:?}", games.len(), start.elapsed()); } diff --git a/src-tauri/src/puzzle.rs b/src-tauri/src/puzzle.rs index 140b6301..a622a3a7 100644 --- a/src-tauri/src/puzzle.rs +++ b/src-tauri/src/puzzle.rs @@ -93,7 +93,7 @@ pub struct PuzzleDatabaseInfo { pub async fn get_puzzle_db_info( file: PathBuf, app: tauri::AppHandle, -) -> Result { +) -> Result { let db_path = PathBuf::from("puzzles").join(file); let path = resolve_path( @@ -102,18 +102,14 @@ pub async fn get_puzzle_db_info( &app.env(), db_path, Some(BaseDirectory::AppData), - ) - .or(Err("resolve path"))?; + )?; let mut db = diesel::SqliteConnection::establish(&path.to_string_lossy()).expect("open database"); - let puzzle_count = puzzles::table - .count() - .get_result::(&mut db) - .expect("get puzzle count") as usize; + let puzzle_count = puzzles::table.count().get_result::(&mut db)? as usize; - let storage_size = path.metadata().expect("get metadata").len() as usize; + let storage_size = path.metadata()?.len() as usize; let filename = path.file_name().expect("get filename").to_string_lossy(); Ok(PuzzleDatabaseInfo {