fix: secondary for rw transaction

vincent-herlemont · Sep 1, 2024 · 7e0d719 · 7e0d719
1 parent 7810f2f
commit 7e0d719
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 7 deletions.
diff --git a/src/transaction/query/len.rs b/src/transaction/query/len.rs
@@ -105,10 +105,9 @@ impl RwLen<'_, '_> {
     /// Get the number of values by secondary key.
     ///
     /// Same as [`RLen::secondary()`](struct.RLen.html#method.secondary).
-    pub fn secondary<T: ToInput>(
-        &self,
-        _key_def: impl ToKeyDefinition<KeyOptions>,
-    ) -> Result<Option<T>> {
-        todo!()
+    pub fn secondary<T: ToInput>(&self, key_def: impl ToKeyDefinition<KeyOptions>) -> Result<u64> {
+        let model = T::native_db_model();
+        let result = self.internal.secondary_len(model, key_def)?;
+        Ok(result)
     }
 }
diff --git a/tests/upgrade/from_0_7_x_to_0_8_x.rs b/tests/upgrade/from_0_7_x_to_0_8_x.rs
@@ -1,9 +1,9 @@
+use itertools::Itertools;
 use native_db::*;
 use native_model::{native_model, Model};
+use redb::ReadableTable;
 use serde::{Deserialize, Serialize};
-use itertools::Itertools;
 use std::path::PathBuf;
-use redb::ReadableTable;
 
 const OLD_TABLE: redb::TableDefinition<Key, Key> = redb::TableDefinition::new("2_1_name");
 const NEW_TABLE: redb::MultimapTableDefinition<Key, Key> =