Removed validation and auto-logging code

Author: nsingh-branch

Branch-TestBed/Branch-TestBed/AppDelegate.m

@@ -61,9 +61,7 @@ - (BOOL)application:(UIApplication *)application
 
     [self setLogFile:@"OpenNInstall"];
     // [branch setIdentity:@"Bobby Branch"];
-    
-    [Branch setLogInAppPurchasesAsEventsEnabled:true];
-    
+        
     [branch initSessionWithLaunchOptions:launchOptions
         andRegisterDeepLinkHandlerUsingBranchUniversalObject:
         ^ (BranchUniversalObject * _Nullable universalObject, BranchLinkProperties * _Nullable linkProperties, NSError * _Nullable error) {

BranchSDK/BNCAppleReceipt.h

@@ -21,7 +21,6 @@ NS_ASSUME_NONNULL_BEGIN
 // this is only available on builds from Apple
 - (nullable NSString *)installReceipt;
 - (BOOL)isTestFlight;
-+ (BOOL)isReceiptValid;
 
 @end
 

BranchSDK/BNCAppleReceipt.m

@@ -63,15 +63,4 @@ - (BOOL)isTestFlight {
     return self.isSandboxReceipt;
 }
 
-+ (BOOL)isReceiptValid {
-    NSURL *receiptURL = [[NSBundle mainBundle] appStoreReceiptURL];
-    NSData *receiptData = [NSData dataWithContentsOfURL:receiptURL];
-    
-    if (receiptData == nil) {
-        return false;
-    } else {
-        return true;
-    }
-}
-
 @end

BranchSDK/BNCPreferenceHelper.h

@@ -80,7 +80,6 @@ NSURL* /* _Nonnull */ BNCURLForBranchDirectory(void);
 @property (assign, nonatomic) NSInteger highestConversionValueSent;
 @property (strong, nonatomic) NSDate   *firstAppLaunchTime;
 @property (assign, nonatomic) BOOL invokeRegisterApp;
-@property (assign, nonatomic) BOOL logInAppPurchasesAsBranchEvents;
 
 - (void) clearTrackingInformation;
 

BranchSDK/BNCPreferenceHelper.m

@@ -110,7 +110,6 @@ @implementation BNCPreferenceHelper
     skanCurrentWindow = _skanCurrentWindow,
     firstAppLaunchTime = _firstAppLaunchTime,
     highestConversionValueSent = _highestConversionValueSent,
-    logInAppPurchasesAsBranchEvents = _logInAppPurchasesAsBranchEvents,
     referringURLQueryParameters = _referringURLQueryParameters,
     anonID = _anonID;
 
@@ -824,16 +823,6 @@ - (void) setInvokeRegisterApp:(BOOL)invoke {
     }
 }
 
-- (BOOL) logInAppPurchasesAsBranchEvents {
-    _logInAppPurchasesAsBranchEvents = [self readBoolFromDefaults:BRANCH_PREFS_KEY_LOG_IAP_AS_EVENTS];
-    return _logInAppPurchasesAsBranchEvents;
-}
-
-- (void) setLogInAppPurchasesAsBranchEvents:(BOOL)logInAppPurchasesAsBranchEvents {
-    _logInAppPurchasesAsBranchEvents = logInAppPurchasesAsBranchEvents;
-    [self writeBoolToDefaults:BRANCH_PREFS_KEY_LOG_IAP_AS_EVENTS value:logInAppPurchasesAsBranchEvents];
-}
-
 - (void) clearTrackingInformation {
     @synchronized(self) {
         /*

BranchSDK/Branch.h

@@ -1878,9 +1878,6 @@ typedef NS_ENUM(NSUInteger, BranchCreditHistoryOrder) {
 - (void)passPasteItemProviders:(NSArray<NSItemProvider *> *)itemProviders API_AVAILABLE(ios(16));
 #endif
 
-+ (void)setLogInAppPurchasesAsEventsEnabled:(BOOL)enabled;
-+ (BOOL)logInAppPurchasesBranchEventsEnabled;
-
 @end
 
 NS_ASSUME_NONNULL_END

BranchSDK/Branch.m

@@ -46,7 +46,6 @@
 #import "NSError+Branch.h"
 #import "BNCLog.h"
 #import "UIViewController+Branch.h"
-#import <StoreKit/StoreKit.h>
 #import "BNCReferringURLUtility.h"
 
 #if !TARGET_OS_TV
@@ -126,7 +125,7 @@ typedef NS_ENUM(NSInteger, BNCInitStatus) {
     BNCInitStatusInitialized
 };
 
-@interface Branch() <BranchDeepLinkingControllerCompletionDelegate, SKPaymentTransactionObserver> {
+@interface Branch() <BranchDeepLinkingControllerCompletionDelegate> {
     NSInteger _networkCount;
     BNCURLFilter *_userURLFilter;
 }
@@ -1677,10 +1676,6 @@ + (Branch *)getInstanceInternal:(NSString *)key {
                     cache:[[BNCLinkCache alloc] init]
                     preferenceHelper:preferenceHelper
                     key:key];
-            
-            if ([BNCPreferenceHelper sharedInstance].logInAppPurchasesAsBranchEvents == YES) {
-                [[SKPaymentQueue defaultQueue] addTransactionObserver:branch];
-            }
         });
         return branch;
     }
@@ -2602,51 +2597,4 @@ + (void) clearAll {
     [BNCPreferenceHelper clearAll];
 }
 
-#pragma mark - SKPaymentTransactionObserver Methods
-+ (void)setLogInAppPurchasesAsEventsEnabled:(BOOL)enabled {
-    @synchronized(self) {
-        if (enabled) {
-            [BNCPreferenceHelper sharedInstance].logInAppPurchasesAsBranchEvents = YES;
-        } else {
-            [BNCPreferenceHelper sharedInstance].logInAppPurchasesAsBranchEvents = NO;
-        }
-    }
-}
-
-+ (BOOL)logInAppPurchasesBranchEventsEnabled {
-    @synchronized(self) {
-        return [BNCPreferenceHelper sharedInstance].logInAppPurchasesAsBranchEvents;
-    }
-}
-
-//Logs incoming in-app purchases as events
-- (void)paymentQueue:(SKPaymentQueue *)queue updatedTransactions:(NSArray<SKPaymentTransaction *> *)transactions {
-    for (SKPaymentTransaction *transaction in transactions) {
-        switch (transaction.transactionState) {
-            case SKPaymentTransactionStatePurchased: {
-                
-                [[SKPaymentQueue defaultQueue] finishTransaction:(SKPaymentTransaction *)transaction];
-                
-                if ([BNCPreferenceHelper sharedInstance].logInAppPurchasesAsBranchEvents == YES) {
-                    BNCLogDebug([NSString stringWithFormat:@"Automatically logging transaction as Branch event."]);
-                    
-                    BranchEvent *event = [BranchEvent standardEvent:BranchStandardEventPurchase];
-                    [event logEventWithTransaction:transaction];
-                }
-                break;
-            }
-            case SKPaymentTransactionStateFailed: {
-                [[SKPaymentQueue defaultQueue] finishTransaction:(SKPaymentTransaction *)transaction];
-                break;
-            }
-            case SKPaymentTransactionStateRestored: {
-                [[SKPaymentQueue defaultQueue] finishTransaction:(SKPaymentTransaction *)transaction];
-                break;
-            }
-            default:
-                break;
-        }
-    }
-}
-
 @end

BranchSDK/BranchEvent.m

@@ -373,19 +373,15 @@ - (NSString*_Nonnull) description {
 #pragma mark - IAP Methods
 
 - (void) logEventWithTransaction:(SKPaymentTransaction *)transaction {
-    if ([BNCAppleReceipt isReceiptValid]) {
-        self.transactionID = transaction.transactionIdentifier;
-        [[BNCEventUtils shared] storeEvent:self];
-        
-        NSString *productId = transaction.payment.productIdentifier;
-        SKProductsRequest *productsRequest = [[SKProductsRequest alloc] initWithProductIdentifiers:[NSSet setWithObject:productId]];
-        
-        _request = productsRequest;
-        productsRequest.delegate = self;
-        [productsRequest start];
-    } else {
-        BNCLogDebug(@"Invalid receipt. Branch Event was not logged.");
-    }
+    self.transactionID = transaction.transactionIdentifier;
+    [[BNCEventUtils shared] storeEvent:self];
+    
+    NSString *productId = transaction.payment.productIdentifier;
+    SKProductsRequest *productsRequest = [[SKProductsRequest alloc] initWithProductIdentifiers:[NSSet setWithObject:productId]];
+    
+    _request = productsRequest;
+    productsRequest.delegate = self;
+    [productsRequest start];
 }
 
 - (void)productsRequest:(SKProductsRequest *)request didReceiveResponse:(SKProductsResponse *)response {