graphql: Refactor From<Entity> for BtreeMap

leoyvens · leoyvens · commit ad9fbc977bbd · 2020-06-18T13:12:36.000-03:00
diff --git a/graph/src/data/store/mod.rs b/graph/src/data/store/mod.rs
@@ -557,11 +557,7 @@ impl DerefMut for Entity {
 
 impl From<Entity> for BTreeMap<String, query::Value> {
     fn from(entity: Entity) -> BTreeMap<String, query::Value> {
-        let mut fields = BTreeMap::new();
-        for (attr, value) in entity.0.into_iter() {
-            fields.insert(attr, value.into());
-        }
-        fields
+        entity.0.into_iter().map(|(k, v)| (k, v.into())).collect()
     }
 }
 

Original file line number	Diff line number	Diff line change
`@@ -557,11 +557,7 @@ impl DerefMut for Entity {`
`557`	`557`
`558`	`558`	`impl From<Entity> for BTreeMap<String, query::Value> {`
`559`	`559`	`fn from(entity: Entity) -> BTreeMap<String, query::Value> {`
`560`		`- let mut fields = BTreeMap::new();`
`561`		`- for (attr, value) in entity.0.into_iter() {`
`562`		`- fields.insert(attr, value.into());`
`563`		`- }`
`564`		`- fields`
	`560`	`+ entity.0.into_iter().map(\|(k, v)\| (k, v.into())).collect()`
`565`	`561`	`}`
`566`	`562`	`}`
`567`	`563`