Remove NSLog call in FIRDocumentSnapshot (#3213)

Author: wilhuff

Firestore/Source/API/FIRDocumentSnapshot.mm

@@ -40,6 +40,7 @@
 #include "Firestore/core/src/firebase/firestore/model/field_value_options.h"
 #include "Firestore/core/src/firebase/firestore/nanopb/nanopb_util.h"
 #include "Firestore/core/src/firebase/firestore/util/hard_assert.h"
+#include "Firestore/core/src/firebase/firestore/util/log.h"
 #include "Firestore/core/src/firebase/firestore/util/string_apple.h"
 
 namespace util = firebase::firestore::util;
@@ -261,12 +262,11 @@ - (id)convertedReference:(const FieldValue &)value {
   const DatabaseId &refDatabase = ref.database_id();
   const DatabaseId &database = _snapshot.firestore()->database_id();
   if (refDatabase != database) {
-    // TODO(b/32073923): Log this as a proper warning.
-    NSLog(@"WARNING: Document %@ contains a document reference within a different database "
-           "(%s/%s) which is not supported. It will be treated as a reference within the "
-           "current database (%s/%s) instead.",
-          self.reference.path, refDatabase.project_id().c_str(), refDatabase.database_id().c_str(),
-          database.project_id().c_str(), database.database_id().c_str());
+    LOG_WARN("Document %s contains a document reference within a different database (%s/%s) which "
+             "is not supported. It will be treated as a reference within the current database "
+             "(%s/%s) instead.",
+             _snapshot.CreateReference().Path(), refDatabase.project_id(),
+             refDatabase.database_id(), database.project_id(), database.database_id());
   }
   const DocumentKey &key = ref.key();
   return [[FIRDocumentReference alloc] initWithKey:key firestore:_snapshot.firestore()];