Fix module build issues

Author: paulb777

FirebaseAppDistribution/Sources/FIRFADLogger.m

@@ -14,7 +14,6 @@
 
 #import "FirebaseAppDistribution/Sources/FIRFADLogger.h"
 #import "FirebaseCore/Extension/FIRLogger.h"
-#import "FirebaseCore/Sources/Public/FirebaseCore/FIRLoggerLevel.h"
 
 FIRLoggerService kFIRLoggerAppDistribution = @"[FirebaseAppDistribution]";
 
@@ -23,31 +22,27 @@
 void FIRFADDebugLog(NSString *message, ...) {
   va_list args_ptr;
   va_start(args_ptr, message);
-  FIRLogBasic(FIRLoggerLevelDebug, kFIRLoggerAppDistribution, AppDistributionMessageCode, message,
-              args_ptr);
+  FIRLogDebug(kFIRLoggerAppDistribution, AppDistributionMessageCode, message, args_ptr);
   va_end(args_ptr);
 }
 
 void FIRFADInfoLog(NSString *message, ...) {
   va_list args_ptr;
   va_start(args_ptr, message);
-  FIRLogBasic(FIRLoggerLevelInfo, kFIRLoggerAppDistribution, AppDistributionMessageCode, message,
-              args_ptr);
+  FIRLogInfo(kFIRLoggerAppDistribution, AppDistributionMessageCode, message, args_ptr);
   va_end(args_ptr);
 }
 
 void FIRFADWarningLog(NSString *message, ...) {
   va_list args_ptr;
   va_start(args_ptr, message);
-  FIRLogBasic(FIRLoggerLevelWarning, kFIRLoggerAppDistribution, AppDistributionMessageCode, message,
-              args_ptr);
+  FIRLogWarning(kFIRLoggerAppDistribution, AppDistributionMessageCode, message, args_ptr);
   va_end(args_ptr);
 }
 
 void FIRFADErrorLog(NSString *message, ...) {
   va_list args_ptr;
   va_start(args_ptr, message);
-  FIRLogBasic(FIRLoggerLevelError, kFIRLoggerAppDistribution, AppDistributionMessageCode, message,
-              args_ptr);
+  FIRLogError(kFIRLoggerAppDistribution, AppDistributionMessageCode, message, args_ptr);
   va_end(args_ptr);
 }

FirebaseCore/Extension/FIRLogger.h

@@ -64,13 +64,23 @@ FIRLoggerLevel FIRGetLoggerLevel(void);
  */
 void FIRSetLoggerLevel(FIRLoggerLevel loggerLevel);
 
+void FIRSetLoggerLevelNotice(void);
+void FIRSetLoggerLevelWarning(void);
+void FIRSetLoggerLevelError(void);
+void FIRSetLoggerLevelDebug(void);
+
 /**
  * Checks if the specified logger level is loggable given the current settings.
  * (required) log level (one of the FirebaseLoggerLevel enum values).
  * (required) whether or not this function is called from the Analytics component.
  */
 BOOL FIRIsLoggableLevel(FIRLoggerLevel loggerLevel, BOOL analyticsComponent);
 
+BOOL FIRIsLoggableLevelNotice(void);
+BOOL FIRIsLoggableLevelWarning(void);
+BOOL FIRIsLoggableLevelError(void);
+BOOL FIRIsLoggableLevelDebug(void);
+
 /**
  * Logs a message to the Xcode console and the device log. If running from AppStore, will
  * not log any messages with a level higher than FirebaseLoggerLevelNotice to avoid log spamming.

FirebaseCore/Sources/FIRLogger.m

@@ -95,6 +95,26 @@ void FIRSetLoggerLevel(FIRLoggerLevel loggerLevel) {
   GULSetLoggerLevel((GULLoggerLevel)loggerLevel);
 }
 
+void FIRSetLoggerLevelNotice(void) {
+  FIRLoggerInitialize();
+  GULSetLoggerLevel(GULLoggerLevelNotice);
+}
+
+void FIRSetLoggerLevelWarning(void) {
+  FIRLoggerInitialize();
+  GULSetLoggerLevel(GULLoggerLevelWarning);
+}
+
+void FIRSetLoggerLevelError(void) {
+  FIRLoggerInitialize();
+  GULSetLoggerLevel(GULLoggerLevelError);
+}
+
+void FIRSetLoggerLevelDebug(void) {
+  FIRLoggerInitialize();
+  GULSetLoggerLevel(GULLoggerLevelDebug);
+}
+
 #ifdef DEBUG
 void FIRResetLogger(void) {
   extern void GULResetLogger(void);
@@ -124,6 +144,22 @@ void FIRSetLoggerUserDefaults(NSUserDefaults *defaults) {
   return GULIsLoggableLevel((GULLoggerLevel)loggerLevel);
 }
 
+BOOL FIRIsLoggableLevelNotice(void) {
+  return FIRIsLoggableLevel(FIRLoggerLevelNotice, NO);
+}
+
+BOOL FIRIsLoggableLevelWarning(void) {
+  return FIRIsLoggableLevel(FIRLoggerLevelWarning, NO);
+}
+
+BOOL FIRIsLoggableLevelError(void) {
+  return FIRIsLoggableLevel(FIRLoggerLevelError, NO);
+}
+
+BOOL FIRIsLoggableLevelDebug(void) {
+  return FIRIsLoggableLevel(FIRLoggerLevelDebug, NO);
+}
+
 void FIRLogBasic(FIRLoggerLevel level,
                  NSString *category,
                  NSString *messageCode,

Firestore/Source/API/FSTFirestoreComponent.mm

@@ -28,7 +28,7 @@
 #import "FirebaseCore/Extension/FIRComponentContainer.h"
 #import "FirebaseCore/Extension/FIRComponentType.h"
 #import "FirebaseCore/Extension/FIRLibrary.h"
-#import "FirebaseCore/Sources/Public/FirebaseCore/FIROptions.h"
+#import "FirebaseCore/Sources/FIROptionsInternal.h"
 #import "Firestore/Source/API/FIRFirestore+Internal.h"
 
 #include "Firestore/core/include/firebase/firestore/firestore_version.h"

Firestore/core/src/remote/firebase_metadata_provider_apple.mm

@@ -18,7 +18,7 @@
 
 #import "FirebaseCore/Extension/FIRAppInternal.h"
 #import "FirebaseCore/Extension/FIRHeartbeatLogger.h"
-#import "FirebaseCore/Sources/Public/FirebaseCore/FIROptions.h"
+#import "FirebaseCore/Sources/FIROptionsInternal.h"
 
 #include "Firestore/core/src/util/string_apple.h"
 

Firestore/core/src/util/log_apple.mm

@@ -24,7 +24,6 @@
 #include <string>
 
 #import "FirebaseCore/Extension/FIRLogger.h"
-#import "FirebaseCore/Sources/Public/FirebaseCore/FIRLoggerLevel.h"
 
 #include "Firestore/core/src/util/string_apple.h"
 
@@ -36,39 +35,38 @@
 
 const FIRLoggerService kFIRLoggerFirestore = @"[FirebaseFirestore]";
 
-// Translates a C++ LogLevel to the equivalent Objective-C FIRLoggerLevel
-FIRLoggerLevel ToFIRLoggerLevel(LogLevel level) {
-  switch (level) {
-    case kLogLevelDebug:
-      return FIRLoggerLevelDebug;
-    case kLogLevelNotice:
-      return FIRLoggerLevelNotice;
-    case kLogLevelWarning:
-      return FIRLoggerLevelWarning;
-    case kLogLevelError:
-      return FIRLoggerLevelError;
-    default:
-      // Unsupported log level. FIRSetLoggerLevel will deal with it.
-      return static_cast<FIRLoggerLevel>(-1);
-  }
-}
-
 // Actually logs a message via FIRLogger. This must be a C varargs function
 // so that we can call FIRLogBasic which takes a `va_list`.
 void LogMessageV(LogLevel level, NSString* format, ...) {
   va_list list;
   va_start(list, format);
 
-  FIRLogBasic(ToFIRLoggerLevel(level), kFIRLoggerFirestore, @"I-FST000001",
-              format, list);
-
+  switch (level) {
+    case kLogLevelDebug:
+      FIRLogDebug(kFIRLoggerFirestore, @"I-FST000001", format, list);
+    case kLogLevelNotice:
+      FIRLogNotice(kFIRLoggerFirestore, @"I-FST000001", format, list);
+    case kLogLevelWarning:
+      FIRLogWarning(kFIRLoggerFirestore, @"I-FST000001", format, list);
+    case kLogLevelError:
+      FIRLogError(kFIRLoggerFirestore, @"I-FST000001", format, list);
+  }
   va_end(list);
 }
 
 }  // namespace
 
 void LogSetLevel(LogLevel level) {
-  FIRSetLoggerLevel(ToFIRLoggerLevel(level));
+  switch (level) {
+    case kLogLevelDebug:
+      FIRSetLoggerLevelDebug();
+    case kLogLevelNotice:
+      FIRSetLoggerLevelNotice();
+    case kLogLevelWarning:
+      FIRSetLoggerLevelWarning();
+    case kLogLevelError:
+      FIRSetLoggerLevelError();
+  }
 }
 
 // Note that FIRLogger's default level can be changed by persisting a
@@ -82,7 +80,17 @@ void LogSetLevel(LogLevel level) {
 //   defaults write firestore_util_test /google/firebase/debug_mode NO
 
 bool LogIsLoggable(LogLevel level) {
-  return FIRIsLoggableLevel(ToFIRLoggerLevel(level), false);
+  switch (level) {
+    case kLogLevelDebug:
+      return FIRIsLoggableLevelDebug();
+    case kLogLevelNotice:
+      return FIRIsLoggableLevelNotice();
+    case kLogLevelWarning:
+      return FIRIsLoggableLevelWarning();
+    case kLogLevelError:
+      return FIRIsLoggableLevelError();
+  }
+  return false;
 }
 
 void LogMessage(LogLevel level, const std::string& message) {