extract clearDebugUpdates to seperate function

Author: geof90

ios/CodePush/CodePush.m

@@ -76,32 +76,32 @@ + (NSURL *)bundleURLForResource:(NSString *)resourceName
 {
     bundleResourceName = resourceName;
     bundleResourceExtension = resourceExtension;
-    
+
     [self ensureBinaryBundleExists];
-    
+
     NSString *logMessageFormat = @"Loading JS bundle from %@";
-    
+
     NSError *error;
     NSString *packageFile = [CodePushPackage getCurrentPackageBundlePath:&error];
     NSURL *binaryBundleURL = [self binaryBundleURL];
-    
+
     if (error || !packageFile) {
         NSLog(logMessageFormat, binaryBundleURL);
         isRunningBinaryVersion = YES;
         return binaryBundleURL;
     }
-    
+
     NSString *binaryAppVersion = [[CodePushConfig current] appVersion];
     NSDictionary *currentPackageMetadata = [CodePushPackage getCurrentPackage:&error];
     if (error || !currentPackageMetadata) {
         NSLog(logMessageFormat, binaryBundleURL);
         isRunningBinaryVersion = YES;
         return binaryBundleURL;
     }
-    
+
     NSString *packageDate = [currentPackageMetadata objectForKey:BinaryBundleDateKey];
     NSString *packageAppVersion = [currentPackageMetadata objectForKey:AppVersionKey];
-    
+
     if ([[CodePushUpdateUtils modifiedDateStringOfFileAtURL:binaryBundleURL] isEqualToString:packageDate] && ([CodePush isUsingTestConfiguration] ||[binaryAppVersion isEqualToString:packageAppVersion])) {
         // Return package file because it is newer than the app store binary's JS bundle
         NSURL *packageUrl = [[NSURL alloc] initFileURLWithPath:packageFile];
@@ -113,11 +113,11 @@ + (NSURL *)bundleURLForResource:(NSString *)resourceName
 #ifndef DEBUG
         isRelease = YES;
 #endif
-        
+
         if (isRelease || ![binaryAppVersion isEqualToString:packageAppVersion]) {
             [CodePush clearUpdates];
         }
-        
+
         NSLog(logMessageFormat, binaryBundleURL);
         isRunningBinaryVersion = YES;
         return binaryBundleURL;
@@ -186,7 +186,7 @@ - (NSDictionary *)constantsToExport
              @"codePushInstallModeOnNextRestart":@(CodePushInstallModeOnNextRestart),
              @"codePushInstallModeImmediate": @(CodePushInstallModeImmediate),
              @"codePushInstallModeOnNextResume": @(CodePushInstallModeOnNextResume),
-             
+
              @"codePushUpdateStateRunning": @(CodePushUpdateStateRunning),
              @"codePushUpdateStatePending": @(CodePushUpdateStatePending),
              @"codePushUpdateStateLatest": @(CodePushUpdateStateLatest)
@@ -208,44 +208,39 @@ + (void)ensureBinaryBundleExists
 {
     if (![self binaryBundleURL]) {
         NSString *errorMessage;
-        
+
 #if TARGET_IPHONE_SIMULATOR
         errorMessage = @"React Native doesn't generate your app's JS bundle by default when deploying to the simulator. "
         "If you'd like to test CodePush using the simulator, you can do one of three things depending on your React "
         "Native version and/or preferred workflow:\n\n"
-        
+
         "1. Update your AppDelegate.m file to load the JS bundle from the packager instead of from CodePush. "
         "You can still test your CodePush update experience using this workflow (debug builds only).\n\n"
-        
+
         "2. Force the JS bundle to be generated in simulator builds by removing the if block that echoes "
         "\"Skipping bundling for Simulator platform\" in the \"node_modules/react-native/packager/react-native-xcode.sh\" file.\n\n"
-        
+
         "3. Deploy a release build to the simulator, which unlike debug builds, will generate the JS bundle (React Native >=0.22.0 only).";
 #else
         errorMessage = [NSString stringWithFormat:@"The specified JS bundle file wasn't found within the app's binary. Is \"%@\" the correct file name?", [bundleResourceName stringByAppendingPathExtension:bundleResourceExtension]];
 #endif
-        
+
         RCTFatal([CodePushErrorUtils errorWithMessage:errorMessage]);
     }
 }
 
 - (instancetype)init
 {
     self = [super init];
-    
+
     if (self) {
         [self initializeUpdateAfterRestart];
     }
-    
+
     return self;
 }
 
-/*
- * This method is used when the app is started to either
- * initialize a pending update or rollback a faulty update
- * to the previous version.
- */
-- (void)initializeUpdateAfterRestart
+- (void)clearDebugUpdates
 {
     dispatch_async(dispatch_get_main_queue(), ^{
         if ([_bridge.bundleURL.scheme hasPrefix:@"http"]) {
@@ -260,6 +255,16 @@ - (void)initializeUpdateAfterRestart
             }
         }
     });
+}
+
+/*
+ * This method is used when the app is started to either
+ * initialize a pending update or rollback a faulty update
+ * to the previous version.
+ */
+- (void)initializeUpdateAfterRestart
+{
+    [self clearDebugUpdates];
 
     NSUserDefaults *preferences = [NSUserDefaults standardUserDefaults];
     NSDictionary *pendingUpdate = [preferences objectForKey:PendingUpdateKey];
@@ -305,7 +310,7 @@ - (BOOL)isFailedHash:(NSString*)packageHash
                 }
             }
         }
-        
+
         return NO;
     }
 }
@@ -319,13 +324,13 @@ - (BOOL)isPendingUpdate:(NSString*)packageHash
 {
     NSUserDefaults *preferences = [NSUserDefaults standardUserDefaults];
     NSDictionary *pendingUpdate = [preferences objectForKey:PendingUpdateKey];
-    
+
     // If there is a pending update whose "state" isn't loading, then we consider it "pending".
     // Additionally, if a specific hash was provided, we ensure it matches that of the pending update.
     BOOL updateIsPending = pendingUpdate &&
                            [pendingUpdate[PendingUpdateIsLoadingKey] boolValue] == NO &&
                            (!packageHash || [pendingUpdate[PendingUpdateHashKey] isEqualToString:packageHash]);
-    
+
     return updateIsPending;
 }
 
@@ -346,7 +351,7 @@ - (void)loadBundle
         if ([CodePush isUsingTestConfiguration] || ![_bridge.bundleURL.scheme hasPrefix:@"http"]) {
             [_bridge setValue:[CodePush bundleURL] forKey:@"bundleURL"];
         }
-        
+
         [_bridge reload];
     });
 }
@@ -362,10 +367,10 @@ - (void)rollbackPackage
 {
     NSError *error;
     NSDictionary *failedPackage = [CodePushPackage getCurrentPackage:&error];
-    
+
     // Write the current package's metadata to the "failed list"
     [self saveFailedUpdate:failedPackage];
-    
+
     // Rollback to the previous version and de-register the new update
     [CodePushPackage rollbackPackage];
     [CodePush removePendingUpdate];
@@ -388,7 +393,7 @@ - (void)saveFailedUpdate:(NSDictionary *)failedPackage
         // objects, regardless if you stored something mutable.
         failedUpdates = [failedUpdates mutableCopy];
     }
-    
+
     [failedUpdates addObject:failedPackage];
     [preferences setObject:failedUpdates forKey:FailedUpdatesKey];
     [preferences synchronize];
@@ -430,7 +435,7 @@ - (void)savePendingUpdate:(NSString *)packageHash
     NSDictionary *pendingUpdate = [[NSDictionary alloc] initWithObjectsAndKeys:
                                    packageHash,PendingUpdateHashKey,
                                    [NSNumber numberWithBool:isLoading],PendingUpdateIsLoadingKey, nil];
-    
+
     [preferences setObject:pendingUpdate forKey:PendingUpdateKey];
     [preferences synchronize];
 }
@@ -471,7 +476,7 @@ - (void)applicationWillResignActive
         [mutableUpdatePackage setValue:[CodePushUpdateUtils modifiedDateStringOfFileAtURL:binaryBundleURL]
                                 forKey:BinaryBundleDateKey];
     }
-    
+
     [CodePushPackage
         downloadPackage:mutableUpdatePackage
         expectedBundleFileName:[bundleResourceName stringByAppendingPathExtension:bundleResourceExtension]
@@ -492,11 +497,11 @@ - (void)applicationWillResignActive
             dispatch_async(_methodQueue, ^{
                 NSError *err;
                 NSDictionary *newPackage = [CodePushPackage getPackage:mutableUpdatePackage[PackageHashKey] error:&err];
-                    
+
                 if (err) {
                     return reject([NSString stringWithFormat: @"%lu", (long)err.code], err.localizedDescription, err);
                 }
-                    
+
                 resolve(newPackage);
             });
         }
@@ -506,7 +511,7 @@ - (void)applicationWillResignActive
                 if ([CodePushErrorUtils isCodePushError:err]) {
                     [self saveFailedUpdate:mutableUpdatePackage];
                 }
-                
+
                 reject([NSString stringWithFormat: @"%lu", (long)err.code], err.localizedDescription, err);
             });
         }];
@@ -531,21 +536,21 @@ - (void)applicationWillResignActive
             resolve(configuration);
             return;
         }
-        
+
         if (binaryHash == nil) {
             // The hash was not generated either due to a previous unknown error or the fact that
             // the React Native assets were not bundled in the binary (e.g. during dev/simulator)
             // builds.
             resolve(configuration);
             return;
         }
-        
+
         NSMutableDictionary *mutableConfiguration = [configuration mutableCopy];
         [mutableConfiguration setObject:binaryHash forKey:PackageHashKey];
         resolve(mutableConfiguration);
         return;
     }
-    
+
     resolve(configuration);
 }
 
@@ -567,10 +572,10 @@ - (void)applicationWillResignActive
         // wanted to retrieve the pending or running update.
         return resolve(nil);
     }
-    
+
     // We have a CodePush update, so let's see if it's currently in a pending state.
     BOOL currentUpdateIsPending = [self isPendingUpdate:[package objectForKey:PackageHashKey]];
-   
+
     if (updateState == CodePushUpdateStatePending && !currentUpdateIsPending) {
         // The caller wanted a pending update
         // but there isn't currently one.
@@ -590,7 +595,7 @@ - (void)applicationWillResignActive
             // disk that is not actually running.
             [package setObject:@(YES) forKey:@"_isDebugOnly"];
         }
-    
+
         // Enable differentiating pending vs. non-pending updates
         [package setObject:@(currentUpdateIsPending) forKey:PackageIsPendingKey];
         resolve(package);
@@ -610,16 +615,16 @@ - (void)applicationWillResignActive
     [CodePushPackage installPackage:updatePackage
                 removePendingUpdate:[self isPendingUpdate:nil]
                               error:&error];
-    
+
     if (error) {
         reject([NSString stringWithFormat: @"%lu", (long)error.code], error.localizedDescription, error);
     } else {
         [self savePendingUpdate:updatePackage[PackageHashKey]
                       isLoading:NO];
-        
+
         if (installMode == CodePushInstallModeOnNextResume) {
             _minimumBackgroundDuration = minimumBackgroundDuration;
-            
+
             if (!_hasResumeListener) {
                 // Ensure we do not add the listener twice.
                 // Register for app resume notifications so that we
@@ -628,16 +633,16 @@ - (void)applicationWillResignActive
                                                          selector:@selector(applicationWillEnterForeground)
                                                              name:UIApplicationWillEnterForegroundNotification
                                                            object:[UIApplication sharedApplication]];
-                
+
                 [[NSNotificationCenter defaultCenter] addObserver:self
                                                          selector:@selector(applicationWillResignActive)
                                                              name:UIApplicationWillResignActiveNotification
                                                            object:[UIApplication sharedApplication]];
-                
+
                 _hasResumeListener = YES;
             }
         }
-        
+
         // Signal to JS that the update has been applied.
         resolve(nil);
     }
@@ -668,7 +673,7 @@ - (void)applicationWillResignActive
                         && nil != packageHash
                         && [packageHash length] > 0
                         && [packageHash isEqualToString:[CodePushPackage getCurrentPackageHash:&error]];
-    
+
     resolve(@(isFirstRun));
 }
 
@@ -715,7 +720,7 @@ - (void)applicationWillResignActive
  */
 RCT_EXPORT_METHOD(getNewStatusReport:(RCTPromiseResolveBlock)resolve
                             rejecter:(RCTPromiseRejectBlock)reject)
-{    
+{
     if (needToReportRollback) {
         needToReportRollback = NO;
         NSUserDefaults *preferences = [NSUserDefaults standardUserDefaults];
@@ -739,7 +744,7 @@ - (void)applicationWillResignActive
         resolve([CodePushTelemetryManager getBinaryUpdateReport:appVersion]);
         return;
     }
-    
+
     resolve(nil);
 }