Merge pull request #1327 from OneSignal/v5/fix/direct_id_crash

nan-li · nan-li · commit 8f6c4a542b9a · 2023-10-30T15:00:33.000-07:00
[v5] Fix crash with direct influence but nil direct id
diff --git a/iOS_SDK/OneSignalSDK/OneSignalOutcomes/Source/Influence/OSChannelTracker.m b/iOS_SDK/OneSignalSDK/OneSignalOutcomes/Source/Influence/OSChannelTracker.m
@@ -146,7 +146,6 @@ - (OSInfluence *)currentSessionInfluence {
             } else {
                 [OneSignalLog onesignalLog:ONE_S_LL_ERROR message:@"OSChannelTracker:currentSessionInfluence found a direct influence without a direct id."];
             }
-            
         }
     } else if (_influenceType == INDIRECT) {
         if ([self isIndirectSessionEnabled]) {

Original file line number	Diff line number	Diff line change
`@@ -146,7 +146,6 @@ - (OSInfluence *)currentSessionInfluence {`
`146`	`146`	`} else {`
`147`	`147`	`[OneSignalLog onesignalLog:ONE_S_LL_ERROR message:@"OSChannelTracker:currentSessionInfluence found a direct influence without a direct id."];`
`148`	`148`	`}`
`149`		`-`
`150`	`149`	`}`
`151`	`150`	`} else if (_influenceType == INDIRECT) {`
`152`	`151`	`if ([self isIndirectSessionEnabled]) {`