Remove activity logs after logging out

RELEASE-NOTES.txt

@@ -2,6 +2,7 @@
 -----
 * [*] Fix a layout issue impacting the "No media matching your search" empty state message of the Media Picker screen.  [#19820]
 * [***] [Jetpack-only] Stats Insights Update. Helps you understand how your content is performing and what’s resonating with your audience. [#19909]
+* [***] [internal] Delete all the activity logs after logging out. [#19930]
 
 21.5
 -----

WordPress/Classes/Utility/AccountHelper.swift

@@ -80,6 +80,10 @@ import Foundation
 
         service.removeDefaultWordPressComAccount()
 
+        deleteAccountData()
+    }
+
+    @objc static func deleteAccountData() {
         // Delete saved dashboard states
         BlogDashboardState.resetAllStates()
 
@@ -97,5 +101,8 @@ import Foundation
 
         // Refresh Remote Feature Flags
         WordPressAppDelegate.shared?.updateFeatureFlags()
+
+        // Delete all the logs after logging out
+        WPLogger.shared().deleteAllLogs()
     }
 }

WordPress/Classes/Utility/Logging/WPLogger.h

@@ -28,4 +28,14 @@
  */
 + (WPLogger * _Nonnull)shared;
 
+/**
+ * @brief Deletes all the logs from the device
+ */
+- (void)deleteAllLogs;
+
+/**
+ * @brief Deletes all the old archived logs from the device
+ */
+- (void)deleteArchivedLogs;
+
 @end

WordPress/Classes/Utility/Logging/WPLogger.m

@@ -108,6 +108,30 @@ - (NSString *)getLogFilesContentWithMaxSize:(NSInteger)maxSize
     return description;
 }
 
+#pragma mark - Deleting
+
+- (void)deleteAllLogs
+{
+    NSArray *logFiles = self.fileLogger.logFileManager.sortedLogFileInfos;
+    for (DDLogFileInfo *logFileInfo in logFiles) {
+        [[NSFileManager defaultManager] removeItemAtPath:logFileInfo.filePath error:nil];
+    }
+
+    DDLogWarn(@"All log files erased.");
+}
+
+- (void)deleteArchivedLogs
+{
+    NSArray *logFiles = self.fileLogger.logFileManager.sortedLogFileInfos;
+    for (DDLogFileInfo *logFileInfo in logFiles) {
+        if (logFileInfo.isArchived) {
+            [[NSFileManager defaultManager] removeItemAtPath:logFileInfo.filePath error:nil];
+        }
+    }
+
+    DDLogWarn(@"All archived log files erased.");
+}
+
 #pragma mark - Public static methods
 
 + (void)configureLoggerLevelWithExtraDebug {

WordPress/Classes/ViewRelated/Blog/Blog Details/BlogDetailsViewController.m

@@ -1720,6 +1720,11 @@ - (void)confirmRemoveSite
     if ([Feature enabled:FeatureFlagContentMigration] && [AppConfiguration isWordPress]) {
         [ContentMigrationCoordinator.shared cleanupExportedDataIfNeeded];
     }
+
+    // Delete local data after removing the last site
+    if (!AccountHelper.isLoggedIn) {
+        [AccountHelper deleteAccountData];
+    }
 
     [self.navigationController popToRootViewControllerAnimated:YES];
 }

WordPress/Classes/ViewRelated/Blog/Blog List/BlogListViewController.m

@@ -713,6 +713,11 @@ - (void)confirmRemoveSiteForIndexPath:(NSIndexPath *)indexPath
     if ([Feature enabled:FeatureFlagContentMigration] && [AppConfiguration isWordPress]) {
         [ContentMigrationCoordinator.shared cleanupExportedDataIfNeeded];
     }
+
+    // Delete local data after removing the last site
+    if (!AccountHelper.isLoggedIn) {
+        [AccountHelper deleteAccountData];
+    }
 
     [self.tableView reloadData];
 }

WordPress/Classes/ViewRelated/Me/Help & Support/Activity Logs/ActivityLogViewController.m

@@ -155,12 +155,7 @@ - (void)tableView:(UITableView *)tableView didSelectRowAtIndexPath:(NSIndexPath
                                     actionWithTitle:trashButtonTitle
                                     style:UIAlertActionStyleDestructive
                                     handler:^(UIAlertAction *action) {
-                                      for (DDLogFileInfo *logFileInfo in self.logFiles) {
-                                          if (logFileInfo.isArchived) {
-                                              [[NSFileManager defaultManager] removeItemAtPath:logFileInfo.filePath error:nil];
-                                          }
-                                      }
-                                      DDLogWarn(@"All archived log files erased.");
+                                      [WPLogger.shared deleteArchivedLogs];
                                       [self loadLogFiles];
                                       [self.tableView reloadData];
                                     }];