BREAKING: return Ids when saving snapshots

Author: aawsome

crates/core/src/backend/decrypt.rs

@@ -276,15 +276,18 @@ pub trait DecryptWriteBackend: WriteBackend + Clone + 'static {
         &self,
         list: I,
         p: impl Progress,
-    ) -> RusticResult<()> {
+    ) -> RusticResult<Vec<Id>> {
         p.set_length(list.len() as u64);
-        list.par_bridge().try_for_each(|file| -> RusticResult<_> {
-            _ = self.save_file(file)?;
-            p.inc(1);
-            Ok(())
-        })?;
+        let ids = list
+            .par_bridge()
+            .map(|file| -> RusticResult<Id> {
+                let id = self.save_file(file)?;
+                p.inc(1);
+                Ok(id)
+            })
+            .collect::<RusticResult<_>>()?;
         p.finish();
-        Ok(())
+        Ok(ids)
     }
 
     /// Deletes the given list of files.

crates/core/src/commands/copy.rs

@@ -4,7 +4,7 @@ use log::trace;
 use rayon::prelude::{IntoParallelRefIterator, ParallelBridge, ParallelIterator};
 
 use crate::{
-    backend::{decrypt::DecryptWriteBackend, node::NodeType},
+    backend::node::NodeType,
     blob::{packer::Packer, tree::TreeStreamerOnce, BlobType},
     error::RusticResult,
     index::{indexer::Indexer, ReadIndex},
@@ -130,8 +130,7 @@ pub(crate) fn copy<'a, Q, R: IndexedFull, P: ProgressBars, S: IndexedIds>(
     _ = tree_packer.finalize()?;
     indexer.write().unwrap().finalize()?;
 
-    let p = pb.progress_counter("saving snapshots...");
-    be_dest.save_list(snaps.iter(), p)?;
+    let _ = repo_dest.save_snapshots(snaps)?;
     Ok(())
 }
 

crates/core/src/repository.rs

@@ -1020,13 +1020,12 @@ impl<P: ProgressBars, S: Open> Repository<P, S> {
     /// * [`CryptBackendErrorKind::SerializingToJsonByteVectorFailed`] - If the file could not be serialized to json.
     ///
     /// [`CryptBackendErrorKind::SerializingToJsonByteVectorFailed`]: crate::error::CryptBackendErrorKind::SerializingToJsonByteVectorFailed
-    pub fn save_snapshots(&self, mut snaps: Vec<SnapshotFile>) -> RusticResult<()> {
+    pub fn save_snapshots(&self, mut snaps: Vec<SnapshotFile>) -> RusticResult<Vec<Id>> {
         for snap in &mut snaps {
             snap.id = Id::default();
         }
         let p = self.pb.progress_counter("saving snapshots...");
-        self.dbe().save_list(snaps.iter(), p)?;
-        Ok(())
+        self.dbe().save_list(snaps.iter(), p)
     }
 
     /// Check the repository for errors or inconsistencies