Index: service/datastore/finalized_query.go |
diff --git a/service/datastore/finalized_query.go b/service/datastore/finalized_query.go |
index d2762881e04008c69e6b2674d3c17c45ff20d4ed..0f9d755ba66c3037ad3a2dc7dde8b40e5905aa09 100644 |
--- a/service/datastore/finalized_query.go |
+++ b/service/datastore/finalized_query.go |
@@ -312,8 +312,8 @@ func (q *FinalizedQuery) String() string { |
return q.GQL() |
} |
-// Valid returns true iff this FinalizedQuery is valid in the provided appID and |
-// namespace. |
+// Valid returns true iff this FinalizedQuery is valid in the provided |
+// KeyContext's App ID and Namespace. |
// |
// This checks the ancestor filter (if any), as well as the inequality filters |
// if they filter on '__key__'. |
@@ -321,17 +321,17 @@ func (q *FinalizedQuery) String() string { |
// In particular, it does NOT validate equality filters which happen to have |
// values of type PTKey, nor does it validate inequality filters that happen to |
// have values of type PTKey (but don't filter on the magic '__key__' field). |
-func (q *FinalizedQuery) Valid(aid, ns string) error { |
+func (q *FinalizedQuery) Valid(kc KeyContext) error { |
anc := q.Ancestor() |
- if anc != nil && (!anc.Valid(false, aid, ns) || anc.IsIncomplete()) { |
+ if anc != nil && (!anc.Valid(false, kc) || anc.IsIncomplete()) { |
return ErrInvalidKey |
} |
if q.ineqFiltProp == "__key__" { |
- if q.ineqFiltLowSet && !q.ineqFiltLow.Value().(*Key).Valid(false, aid, ns) { |
+ if q.ineqFiltLowSet && !q.ineqFiltLow.Value().(*Key).Valid(false, kc) { |
return ErrInvalidKey |
} |
- if q.ineqFiltHighSet && !q.ineqFiltHigh.Value().(*Key).Valid(false, aid, ns) { |
+ if q.ineqFiltHighSet && !q.ineqFiltHigh.Value().(*Key).Valid(false, kc) { |
return ErrInvalidKey |
} |
} |