Fixes

Author: tom-andersen

firebase-firestore/src/main/java/com/google/firebase/firestore/model/Document.java

@@ -35,6 +35,8 @@ public interface Document {
    */
   SnapshotVersion getVersion();
 
+  SnapshotVersion getCreateTime();
+
   /**
    * Returns the timestamp at which this document was read from the remote server. Returns
    * `SnapshotVersion.NONE` for documents created by the user.

firebase-firestore/src/main/java/com/google/firebase/firestore/model/FieldPath.java

@@ -22,7 +22,12 @@
 /** A dot separated path for navigating sub-objects with in a document */
 public final class FieldPath extends BasePath<FieldPath> {
 
+  public static final String UPDATE_TIME_NAME = "__update_time__";
+  public static final String CREATE_TIME_NAME = "__create_time__";
+
   public static final FieldPath KEY_PATH = fromSingleSegment(DocumentKey.KEY_FIELD_NAME);
+  public static final FieldPath UPDATE_TIME_PATH = fromSingleSegment(UPDATE_TIME_NAME);
+  public static final FieldPath CREATE_TIME_PATH = fromSingleSegment(CREATE_TIME_NAME);
   public static final FieldPath EMPTY_PATH = new FieldPath(Collections.emptyList());
 
   private FieldPath(List<String> segments) {

firebase-firestore/src/main/java/com/google/firebase/firestore/model/MutableDocument.java

@@ -63,6 +63,7 @@ private enum DocumentState {
   private DocumentType documentType;
   private SnapshotVersion version;
   private SnapshotVersion readTime;
+  private SnapshotVersion createTime;
   private ObjectValue value;
   private DocumentState documentState;
 
@@ -173,6 +174,11 @@ public DocumentKey getKey() {
     return key;
   }
 
+  @Override
+  public SnapshotVersion getCreateTime() {
+    return createTime;
+  }
+
   @Override
   public SnapshotVersion getVersion() {
     return version;

firebase-firestore/src/main/java/com/google/firebase/firestore/pipeline/evaluation.kt

@@ -43,10 +43,7 @@ import kotlin.math.log10
 import kotlin.math.pow
 import kotlin.math.sqrt
 
-internal class EvaluationContext(
-  val db: FirebaseFirestore,
-  val userDataReader: UserDataReader
-)
+internal class EvaluationContext(val db: FirebaseFirestore, val userDataReader: UserDataReader)
 
 internal typealias EvaluateDocument = (input: MutableDocument) -> EvaluateResult
 

firebase-firestore/src/main/java/com/google/firebase/firestore/pipeline/expressions.kt

@@ -23,10 +23,10 @@ import com.google.firebase.firestore.Pipeline
 import com.google.firebase.firestore.UserDataReader
 import com.google.firebase.firestore.VectorValue
 import com.google.firebase.firestore.model.DocumentKey
+import com.google.firebase.firestore.model.FieldPath as ModelFieldPath
 import com.google.firebase.firestore.model.FieldPath.CREATE_TIME_PATH
 import com.google.firebase.firestore.model.FieldPath.KEY_PATH
 import com.google.firebase.firestore.model.FieldPath.UPDATE_TIME_PATH
-import com.google.firebase.firestore.model.FieldPath as ModelFieldPath
 import com.google.firebase.firestore.model.MutableDocument
 import com.google.firebase.firestore.model.Values
 import com.google.firebase.firestore.model.Values.encodeValue
@@ -1022,8 +1022,7 @@ abstract class Expr internal constructor() {
      * @return A new [BooleanExpr] representing the 'IN' comparison.
      */
     @JvmStatic
-    fun eqAny(expression: Expr, values: List<Any>): BooleanExpr =
-      eqAny(expression, array(values))
+    fun eqAny(expression: Expr, values: List<Any>): BooleanExpr = eqAny(expression, array(values))
 
     /**
      * Creates an expression that checks if an [expression], when evaluated, is equal to any of the
@@ -1047,8 +1046,7 @@ abstract class Expr internal constructor() {
      * @return A new [BooleanExpr] representing the 'IN' comparison.
      */
     @JvmStatic
-    fun eqAny(fieldName: String, values: List<Any>): BooleanExpr =
-      eqAny(fieldName, array(values))
+    fun eqAny(fieldName: String, values: List<Any>): BooleanExpr = eqAny(fieldName, array(values))
 
     /**
      * Creates an expression that checks if a field's value is equal to any of the elements of
@@ -2780,8 +2778,7 @@ abstract class Expr internal constructor() {
      * @return A new [BooleanExpr] representing the arrayContainsAll operation.
      */
     @JvmStatic
-    fun arrayContainsAll(array: Expr, values: List<Any>) =
-      arrayContainsAll(array, array(values))
+    fun arrayContainsAll(array: Expr, values: List<Any>) = arrayContainsAll(array, array(values))
 
     /**
      * Creates an expression that checks if [array] contains all elements of [arrayExpression].
@@ -2803,12 +2800,7 @@ abstract class Expr internal constructor() {
      */
     @JvmStatic
     fun arrayContainsAll(arrayFieldName: String, values: List<Any>) =
-      BooleanExpr(
-        "array_contains_all",
-        evaluateArrayContainsAll,
-        arrayFieldName,
-        array(values)
-      )
+      BooleanExpr("array_contains_all", evaluateArrayContainsAll, arrayFieldName, array(values))
 
     /**
      * Creates an expression that checks if array field contains all elements of [arrayExpression].
@@ -2830,12 +2822,7 @@ abstract class Expr internal constructor() {
      */
     @JvmStatic
     fun arrayContainsAny(array: Expr, values: List<Any>) =
-      BooleanExpr(
-        "array_contains_any",
-        evaluateArrayContainsAny,
-        array,
-        array(values)
-      )
+      BooleanExpr("array_contains_any", evaluateArrayContainsAny, array, array(values))
 
     /**
      * Creates an expression that checks if [array] contains any elements of [arrayExpression].
@@ -2857,12 +2844,7 @@ abstract class Expr internal constructor() {
      */
     @JvmStatic
     fun arrayContainsAny(arrayFieldName: String, values: List<Any>) =
-      BooleanExpr(
-        "array_contains_any",
-        evaluateArrayContainsAny,
-        arrayFieldName,
-        array(values)
-      )
+      BooleanExpr("array_contains_any", evaluateArrayContainsAny, arrayFieldName, array(values))
 
     /**
      * Creates an expression that checks if array field contains any elements of [arrayExpression].
@@ -4194,14 +4176,17 @@ class Field internal constructor(private val fieldPath: ModelFieldPath) : Select
   internal fun toProto(): Value =
     Value.newBuilder().setFieldReferenceValue(fieldPath.canonicalString()).build()
 
-  override fun evaluateContext(context: EvaluationContext) = block@{ input: MutableDocument ->
-    EvaluateResultValue(when (fieldPath) {
-      KEY_PATH -> encodeValue(DocumentReference.forPath(input.key.path, context.db))
-      CREATE_TIME_PATH -> encodeValue(input.createTime.timestamp)
-      UPDATE_TIME_PATH -> encodeValue(input.version.timestamp)
-      else -> input.getField(fieldPath) ?: return@block EvaluateResultUnset
-    })
-  }
+  override fun evaluateContext(context: EvaluationContext) =
+    block@{ input: MutableDocument ->
+      EvaluateResultValue(
+        when (fieldPath) {
+          KEY_PATH -> encodeValue(DocumentReference.forPath(input.key.path, context.db))
+          CREATE_TIME_PATH -> encodeValue(input.createTime.timestamp)
+          UPDATE_TIME_PATH -> encodeValue(input.version.timestamp)
+          else -> input.getField(fieldPath) ?: return@block EvaluateResultUnset
+        }
+      )
+    }
 }
 
 /**