Skip to content

Commit 98d232e

Browse files
jamesnroktclaude
andauthored
fix: map mobileSha256 to MPIdentityOther2 instead of MPIdentityOther (#761)
Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>
1 parent 7ee2b02 commit 98d232e

3 files changed

Lines changed: 11 additions & 12 deletions

File tree

UnitTests/ObjCTests/MPIdentityApiRequestTests.m

Lines changed: 8 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -100,28 +100,27 @@ - (void)testSetMobileSha256 {
100100
XCTAssertNil(request.mobileSha256);
101101
request.mobileSha256 = @"mobilehash456";
102102
XCTAssertEqualObjects(@"mobilehash456", request.mobileSha256);
103-
XCTAssertEqualObjects(@"mobilehash456", request.mutableIdentities[@(MPIdentityOther)]);
103+
XCTAssertEqualObjects(@"mobilehash456", request.mutableIdentities[@(MPIdentityOther2)]);
104104

105105
request.mobileSha256 = nil;
106106
XCTAssertNil(request.mobileSha256);
107-
XCTAssertEqualObjects(request.mutableIdentities[@(MPIdentityOther)], [NSNull null]);
107+
XCTAssertEqualObjects(request.mutableIdentities[@(MPIdentityOther2)], [NSNull null]);
108108
}
109109

110-
- (void)testMobileSha256DelegatesToSetIdentityOther {
110+
- (void)testMobileSha256DelegatesToSetIdentityOther2 {
111111
MPIdentityApiRequest *request = [[MPIdentityApiRequest alloc] init];
112112
request.mobileSha256 = @"mobilehash";
113-
XCTAssertEqualObjects(@"mobilehash", [request.identities objectForKey:@(MPIdentityOther)]);
113+
XCTAssertEqualObjects(@"mobilehash", [request.identities objectForKey:@(MPIdentityOther2)]);
114114
}
115115

116-
- (void)testEmailSha256AndMobileSha256ShareOtherSlot {
116+
- (void)testEmailSha256AndMobileSha256UseDistinctSlots {
117117
MPIdentityApiRequest *request = [[MPIdentityApiRequest alloc] init];
118118
request.emailSha256 = @"emailhash";
119-
XCTAssertEqualObjects(@"emailhash", request.emailSha256);
120-
XCTAssertEqualObjects(@"emailhash", request.mobileSha256);
121-
122119
request.mobileSha256 = @"mobilehash";
120+
XCTAssertEqualObjects(@"emailhash", request.emailSha256);
123121
XCTAssertEqualObjects(@"mobilehash", request.mobileSha256);
124-
XCTAssertEqualObjects(@"mobilehash", request.emailSha256);
122+
XCTAssertEqualObjects(@"emailhash", request.mutableIdentities[@(MPIdentityOther)]);
123+
XCTAssertEqualObjects(@"mobilehash", request.mutableIdentities[@(MPIdentityOther2)]);
125124
}
126125

127126
@end

mParticle-Apple-SDK/Identity/MPIdentityApiRequest.m

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -84,15 +84,15 @@ - (void)setEmailSha256:(NSString *)emailSha256 {
8484
}
8585

8686
- (NSString *)mobileSha256 {
87-
NSObject *result = _mutableIdentities[@(MPIdentityOther)];
87+
NSObject *result = _mutableIdentities[@(MPIdentityOther2)];
8888
if ([result isKindOfClass:[NSString class]]) {
8989
return (NSString *)result;
9090
}
9191
return nil;
9292
}
9393

9494
- (void)setMobileSha256:(NSString *)mobileSha256 {
95-
[self setIdentity:mobileSha256 identityType:MPIdentityOther];
95+
[self setIdentity:mobileSha256 identityType:MPIdentityOther2];
9696
}
9797

9898
- (NSDictionary<NSNumber*, NSObject*> *)identities {

mParticle-Apple-SDK/Include/MPIdentityApiRequest.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,7 @@ NS_ASSUME_NONNULL_BEGIN
3030

3131
/**
3232
SHA-256 hashed mobile number for privacy-safe identity resolution.
33-
Maps to the `other` identity type (@c MPIdentityOther).
33+
Maps to the `other2` identity type (@c MPIdentityOther2).
3434
*/
3535
@property (nonatomic, strong, nullable) NSString *mobileSha256;
3636

0 commit comments

Comments
 (0)