Add debug message

AOSC-Dev · Feb 13, 2025 · efd168d · efd168d
1 parent aac2b31
commit efd168d
Showing 1 changed file with 23 additions and 12 deletions.
diff --git a/oma-history/src/migrations.rs b/oma-history/src/migrations.rs
@@ -1,7 +1,7 @@
 use oma_pm_operation_type::{InstallOperation, RemoveTag};
 use rusqlite::Connection;
 use serde::Deserialize;
-use tracing::info;
+use tracing::{info, warn};
 
 use crate::{
     HistoryEntryInner, HistoryError, HistoryResult, InstallHistoryEntry, RemoveHistoryEntry,
@@ -215,19 +215,30 @@ fn get_old_table(conn: &Connection) -> Result<Vec<OldTableEntry>, HistoryError>
             summary_type,
         ) = i.map_err(HistoryError::ExecuteError)?;
 
-        let Ok(install_packages) =
-            serde_json::from_str::<Vec<InstallHistoryEntry>>(&install_packages)
-        else {
-            continue;
-        };
+        let install_packages =
+            match serde_json::from_str::<Vec<InstallHistoryEntry>>(&install_packages) {
+                Ok(i) => i,
+                Err(e) => {
+                    warn!("Failed to parse item: {e}");
+                    continue;
+                }
+            };
 
-        let Ok(summary_type) = serde_json::from_str::<OldSummaryType>(&summary_type) else {
-            continue;
-        };
+        let remove_packages =
+            match serde_json::from_str::<Vec<RemoveHistoryEntry>>(&remove_packages) {
+                Ok(i) => i,
+                Err(e) => {
+                    warn!("Failed to parse item: {e}");
+                    continue;
+                }
+            };
 
-        let Ok(remove_packages) = serde_json::from_str::<Vec<RemoveHistoryEntry>>(&remove_packages)
-        else {
-            continue;
+        let summary_type = match serde_json::from_str::<OldSummaryType>(&summary_type) {
+            Ok(s) => s,
+            Err(e) => {
+                warn!("Failed to parse item: {e}");
+                continue;
+            }
         };
 
         res.push(OldTableEntry {