Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1048)

Unified Diff: service/datastore/properties.go

Issue 1289323002: Fix miscellaneous prod bugs. (Closed) Base URL: https://github.com/luci/gae@master
Patch Set: Moar fixes. Created 5 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« service/datastore/multiarg.go ('K') | « service/datastore/pls.go ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: service/datastore/properties.go
diff --git a/service/datastore/properties.go b/service/datastore/properties.go
index 8ce5970def31e1939b3f69638c85bfca264e00de..aa8dcdf0f149f15d6cfd2348b9ad4c6400f94c7a 100644
--- a/service/datastore/properties.go
+++ b/service/datastore/properties.go
@@ -206,7 +206,7 @@ func PropertyTypeOf(v interface{}, checkValid bool) (PropertyType, error) {
if checkValid && (x.Before(minTime) || x.After(maxTime)) {
err = errors.New("time value out of range")
}
- if checkValid && x.Location() != time.UTC {
+ if checkValid && !x.Equal(x.In(time.UTC)) {
err = fmt.Errorf("time value has wrong Location: %s", x.Location())
}
return PTTime, err
« service/datastore/multiarg.go ('K') | « service/datastore/pls.go ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698