5.1.0 (#32)

v5.1.0

Author: dorin-ga

CMakeLists.txt

@@ -190,6 +190,8 @@ elseif(APPLE)
         "-framework Foundation"
         "-framework CoreServices"
         "-framework SystemConfiguration"
+        "-framework Metal"
+        "-framework MetalKit"
     )
 
     create_source_groups(MACOS_SOURCES)

include/GameAnalytics/GameAnalytics.h

@@ -120,6 +120,7 @@ namespace gameanalytics
 
          static int64_t getElapsedSessionTime();
          static int64_t getElapsedTimeFromAllSessions();
+         static int64_t getElapsedTimeForPreviousSession();
 
          // game state changes
          // will affect how session is started / ended

source/gameanalytics/GALogger.h

@@ -26,6 +26,11 @@ namespace gameanalytics
                     return;
                 }
 
+                if(logType == LogInfo && !getInstance().infoLogEnabled)
+                {
+                    return;
+                }
+
                 std::string tag = getInstance().tag;
                 tag += " :";
 

source/gameanalytics/GAState.cpp

@@ -426,6 +426,10 @@ namespace gameanalytics
                 out["session_num"] = getInstance()._sessionNum;
                 out["connection_type"] = device::GADevice::getConnectionType();
 
+                // playtime metrics
+                out["current_session_length"] = getInstance().calculateSessionLength();
+                out["lifetime_session_length"] = getInstance().getTotalSessionLength();
+
                 // ---- OPTIONAL ---- //
 
                 // A/B testing
@@ -535,6 +539,11 @@ namespace gameanalytics
             return "";
         }
 
+        int64_t GAState::getLastSessionLength() const
+        {
+            return _lastSessionTime;
+        }
+
         int64_t GAState::getTotalSessionLength() const
         {
             return _totalElapsedSessionTime + calculateSessionLength<std::chrono::seconds>();
@@ -586,10 +595,11 @@ namespace gameanalytics
 
                 try
                 {
-                    std::string cachedSessionTime = utilities::getOptionalValue<std::string>(state_dict, "total_session_time", "0");
+                    std::string cachedLastSessionTime = utilities::getOptionalValue<std::string>(state_dict, "last_session_time", "0");
+                    std::string cachedTotalSessionTime = utilities::getOptionalValue<std::string>(state_dict, "total_session_time", "0");
 
-                    _totalElapsedSessionTime = std::stoull(cachedSessionTime);
-
+                    _lastSessionTime = std::stoull(cachedLastSessionTime);
+                    _totalElapsedSessionTime = std::stoull(cachedTotalSessionTime);
                 }
                 catch(const std::exception& e)
                 {
@@ -1090,8 +1100,11 @@ namespace gameanalytics
 
         void GAState::updateTotalSessionTime()
         {
-            int64_t totalSessionTime = getTotalSessionLength();
-            _gaStore.setState("total_session_time", std::to_string(totalSessionTime));
+            _lastSessionTime = calculateSessionLength();
+            _totalElapsedSessionTime += _lastSessionTime;
+            
+            _gaStore.setState("last_session_time",  std::to_string(_lastSessionTime));
+            _gaStore.setState("total_session_time", std::to_string(_totalElapsedSessionTime));
         }
 
         std::string GAState::getBuild()

source/gameanalytics/GAState.h

@@ -169,6 +169,8 @@ namespace gameanalytics
 
                 int64_t getTotalSessionLength() const;
 
+                int64_t getLastSessionLength() const;
+
                 void populateConfigurations(json& sdkConfig);
 
                 json getRemoteConfigAnnotations();
@@ -229,6 +231,7 @@ namespace gameanalytics
             int64_t _sessionNum = 0;
             int64_t _transactionNum = 0;
 
+            int64_t _lastSessionTime = 0;
             int64_t _totalElapsedSessionTime = 0;
             std::chrono::high_resolution_clock::time_point _startTimepoint;
 

source/gameanalytics/GAStore.cpp

@@ -206,30 +206,6 @@ namespace gameanalytics
         {
             return sqlDatabase;
         }
-
-        bool GAStore::fixOldDatabase()
-        {
-            std::filesystem::path oldPath  = dbPath;
-            std::filesystem::path filename = oldPath.filename();
-            
-            oldPath = oldPath.parent_path() / ".." / filename;
-            
-            if(std::filesystem::exists(oldPath) && !std::filesystem::exists(dbPath))
-            {
-                try
-                {
-                    std::filesystem::rename(oldPath, dbPath);
-                }
-                catch(...)
-                {
-                    return false;
-                }
-                
-                return true;
-            }
-
-            return false;
-        }
 
         bool GAStore::initDatabaseLocation()
         {
@@ -244,8 +220,6 @@ namespace gameanalytics
             {
                 if(!std::filesystem::create_directory(p))
                     return false;
-                
-                fixOldDatabase();
             }
 
             return true;

source/gameanalytics/GAValidator.cpp

@@ -275,6 +275,9 @@ namespace gameanalytics
 
         bool validateProgressionString(std::string const& progression, ValidationResult& out, int progressionLvl)
         {
+            if(progressionLvl > 0 && progression.empty())
+                return true;
+
             if (!GAValidator::validateEventPartLength(progression, true))
             {
                 logging::GALogger::w("Validation fail - progression event - - progression0%d: Cannot be empty or above 64 characters. String: %s", progressionLvl + 1, progression.c_str());
@@ -458,17 +461,12 @@ namespace gameanalytics
         {
             constexpr uint32_t MAX_SIZE = 64u;
 
-            size_t size = eventPart.length();
-            if (allowNull == true && size == 0)
+            if (eventPart.empty())
             {
-                return true;
-            }
-
-            if (size == 0)
-            {
-                return false;
+                return allowNull;
             }
 
+            size_t size = eventPart.length();
             if (size > MAX_SIZE)
             {
                 return false;

source/gameanalytics/GameAnalytics.cpp

@@ -998,4 +998,9 @@ namespace gameanalytics
         return state::GAState::getInstance().calculateSessionLength<std::chrono::seconds>();
     }
 
+    int64_t GameAnalytics::getElapsedTimeForPreviousSession()
+    {
+        return state::GAState::getInstance().getLastSessionLength();
+    }
+
 } // namespace gameanalytics