Pass schema in as a param to GetFieldByID

Note: The only place that calls this only wants the field name, which long term will be available on the CollectionFieldDescription anyway.  I hope this param can be removed again soon. Additionally all properties of Schema used in that calling location will soon be on Collection itself, so the reference to Schema there is hopefully a very temporary thing too.

client/descriptions.go

@@ -41,8 +41,8 @@ func (col CollectionDescription) IDString() string {
 
 // GetFieldByID searches for a field with the given ID. If such a field is found it
 // will return it and true, if it is not found it will return false.
-func (col CollectionDescription) GetFieldByID(id FieldID) (FieldDescription, bool) {
-	for _, field := range col.Schema.Fields {
+func (col CollectionDescription) GetFieldByID(id FieldID, schema *SchemaDescription) (FieldDescription, bool) {
+	for _, field := range schema.Fields {
 		if field.ID == id {
 			return field, true
 		}

db/collection.go

@@ -1324,14 +1324,16 @@ func (c *collection) saveValueToMerkleCRDT(
 			return nil, 0, err
 		}
 
-		field, ok := c.Description().GetFieldByID(client.FieldID(fieldID))
+		schema := c.Schema()
+
+		field, ok := c.Description().GetFieldByID(client.FieldID(fieldID), &schema)
 		if !ok {
 			return nil, 0, client.NewErrFieldIndexNotExist(fieldID)
 		}
 
 		merkleCRDT, err := c.db.crdtFactory.InstanceWithStores(
 			txn,
-			core.NewCollectionSchemaVersionKey(c.Schema().VersionID),
+			core.NewCollectionSchemaVersionKey(schema.VersionID),
 			c.db.events.Updates,
 			ctype,
 			key,