Make sure to only return hasOriginID if there are only origin.id fields.

lostlevels · lostlevels · commit 90975e354656 · 2024-07-11T12:27:47.000-07:00
diff --git a/data/store/mongo/mongo_datum.go b/data/store/mongo/mongo_datum.go
@@ -556,7 +556,6 @@ func validateAndTranslateSelectors(selectors *data.Selectors) (filter bson.M, ha
 			if selector.ID != nil {
 				selectorIDs = append(selectorIDs, *selector.ID)
 			} else if selector.Origin != nil && selector.Origin.ID != nil {
-				hasOriginID = true
 				selectorOriginIDs = append(selectorOriginIDs, *selector.Origin.ID)
 			}
 		}
@@ -578,7 +577,7 @@ func validateAndTranslateSelectors(selectors *data.Selectors) (filter bson.M, ha
 		return nil, false, errors.New("selectors is invalid")
 	}
 
-	return selector, hasOriginID, nil
+	return selector, len(selectorOriginIDs) > 0 && len(selectorIDs) == 0, nil
 }
 
 func (d *DatumRepository) GetDataRange(ctx context.Context, userId string, typ []string, status *data.UserDataStatus) (*mongo.Cursor, error) {

Original file line number	Diff line number	Diff line change
`@@ -556,7 +556,6 @@ func validateAndTranslateSelectors(selectors *data.Selectors) (filter bson.M, ha`
`556`	`556`	`if selector.ID != nil {`
`557`	`557`	`selectorIDs = append(selectorIDs, *selector.ID)`
`558`	`558`	`} else if selector.Origin != nil && selector.Origin.ID != nil {`
`559`		`- hasOriginID = true`
`560`	`559`	`selectorOriginIDs = append(selectorOriginIDs, *selector.Origin.ID)`
`561`	`560`	`}`
`562`	`561`	`}`
`@@ -578,7 +577,7 @@ func validateAndTranslateSelectors(selectors *data.Selectors) (filter bson.M, ha`
`578`	`577`	`return nil, false, errors.New("selectors is invalid")`
`579`	`578`	`}`
`580`	`579`
`581`		`- return selector, hasOriginID, nil`
	`580`	`+ return selector, len(selectorOriginIDs) > 0 && len(selectorIDs) == 0, nil`
`582`	`581`	`}`
`583`	`582`
`584`	`583`	`func (d DatumRepository) GetDataRange(ctx context.Context, userId string, typ []string, status data.UserDataStatus) (*mongo.Cursor, error) {`