[Sdk 921] LogLevel removed from Çountly (#82)

* namespace added on countly classes

* Loglevel removed from countly
change log updated

* get logger method logic updated
logger unit tets updated
define added on 'getlogger' function
logger unit tests moved out of 'Custom' check

* COUNTLY_USE_CUSTOM_SHA256 added on getlogger function

* COUNTLY_BUILD_TESTS define added on getLogger function

* macros removed on 'getLogger'

* Update CHANGELOG.md

* Update CHANGELOG.md

Co-authored-by: ArtursKadikis <[email protected]>

Author: ZahidZafar

CHANGELOG.md

@@ -1,3 +1,6 @@
+22.06.1
+* !! Major breaking change !! We are removing the 'LogLevel' enum out of the 'Countly' class which will change how that enum can be referenced. 'Countly::LogLevel' will not work, you will have to use 'cly::LogLevel' instead.
+
 22.06.0
 * !! Major breaking change !! We are adding the 'cly' namespace on 'Countly' class which will change how that class can be referenced. 'Countly::' will not work, you will have to use 'cly::Countly::' instead.
 * !! Major breaking change !! We are extracting the 'Event' class out of the 'Countly' class which will change how that class can be referenced. 'Countly::Event' will not work, you will have to use 'cly::Event' instead.

examples/example_integration.cpp

@@ -7,19 +7,19 @@
 using namespace std;
 using namespace cly;
 
-void printLog(Countly::LogLevel level, const string &msg) {
+void printLog(LogLevel level, const string &msg) {
   string lvl = "[DEBUG]";
   switch (level) {
-  case Countly::LogLevel::DEBUG:
+  case LogLevel::DEBUG:
     lvl = "[Debug]";
     break;
-  case Countly::LogLevel::INFO:
+  case LogLevel::INFO:
     lvl = "[INFO]";
     break;
-  case Countly::LogLevel::WARNING:
+  case LogLevel::WARNING:
     lvl = "[WARNING]";
     break;
-  case Countly::LogLevel::FATAL:
+  case LogLevel::FATAL:
     lvl = "[FATAL]";
     break;
   default:

include/countly.hpp

@@ -41,15 +41,16 @@ class Countly : public cly::CountlyDelegates {
 
   void setSalt(const std::string &value);
 
-  enum LogLevel { DEBUG = 1, INFO = 2, WARNING = 3, ERROR = 4, FATAL = 5 };
-
   void setLogger(void (*fun)(LogLevel level, const std::string &message));
 
+
+  #ifdef COUNTLY_BUILD_TESTS
   /*
   This function should not be used as it will be removed in a future release. It is
   currently added as a temporary workaround.
   */
-  inline std::function<void(LogLevel, const std::string &)> getLogger() { return logger_function; }
+  inline std::function<void(LogLevel, const std::string &)> getLogger() { return logger->getLogger(); }
+  #endif
 
   struct HTTPResponse {
     bool success;
@@ -237,7 +238,6 @@ class Countly : public cly::CountlyDelegates {
 
   cly::SHA256Function sha256_function;
   HTTPClientFunction http_client_function;
-  void (*logger_function)(LogLevel level, const std::string &message) = nullptr;
 
   std::string host;
 

include/countly/logger_module.hpp

@@ -20,6 +20,12 @@ class LoggerModule {
    */
   void setLogger(LoggerFunction logger);
 
+
+  /**
+   * Get custom logger function.
+   */
+  const LoggerFunction getLogger();
+
   /**
    * Print important information.
    *