Skip to content

Commit 301338e

Browse files
committed
Fixed issue where kOSSettingsKeyInFocusDisplayOption was ignored
1 parent 9d7907e commit 301338e

File tree

2 files changed

+17
-1
lines changed

2 files changed

+17
-1
lines changed

iOS_SDK/OneSignalSDK/Source/OneSignal.m

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -393,7 +393,7 @@ + (id)initWithLaunchOptions:(NSDictionary*)launchOptions appId:(NSString*)appId
393393
NSNumber *IFDSetting = settings[kOSSettingsKeyInFocusDisplayOption];
394394
BOOL inFocusDisplayPassed = IFDSetting && IFDSetting.integerValue > -1 && IFDSetting.integerValue < 3;
395395

396-
if (inAppAlertsPassed || inAppAlertsPassed) {
396+
if (inAppAlertsPassed || inFocusDisplayPassed) {
397397
if (!inFocusDisplayPassed)
398398
self.inFocusDisplayType = (OSNotificationDisplayType)IAASetting.integerValue;
399399
else

iOS_SDK/OneSignalSDK/UnitTests/UnitTests.m

Lines changed: 16 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1040,6 +1040,22 @@ - (void)testInitOnSimulator {
10401040
}
10411041

10421042

1043+
- (void)testFocusSettingsOnInit {
1044+
// Test old kOSSettingsKeyInFocusDisplayOption
1045+
[OneSignal initWithLaunchOptions:nil appId:@"b2f7f966-d8cc-11e4-bed1-df8f05be55ba"
1046+
handleNotificationAction:nil
1047+
settings:@{kOSSettingsKeyInFocusDisplayOption: @(OSNotificationDisplayTypeNone)}];
1048+
1049+
XCTAssertEqual(OneSignal.inFocusDisplayType, OSNotificationDisplayTypeNone);
1050+
1051+
[self clearStateForAppRestart];
1052+
1053+
// Test old very old kOSSettingsKeyInAppAlerts
1054+
[OneSignal initWithLaunchOptions:nil appId:@"b2f7f966-d8cc-11e4-bed1-df8f05be55ba"
1055+
handleNotificationAction:nil
1056+
settings:@{kOSSettingsKeyInAppAlerts: @(false)}];
1057+
XCTAssertEqual(OneSignal.inFocusDisplayType, OSNotificationDisplayTypeNone);
1058+
}
10431059

10441060
- (void)testCallingMethodsBeforeInit {
10451061
[self setCurrentNotificationPermission:true];

0 commit comments

Comments
 (0)