Track files in batches

Author: rubencm

migrations/v30x/m1_storage.php

@@ -15,6 +15,8 @@
 
 class m1_storage extends \phpbb\db\migration\container_aware_migration
 {
+	private const BATCH_SIZE = 100;
+
 	/**
 	 * {@inheritdoc}
 	 */
@@ -33,19 +35,19 @@ public function effectively_installed()
 	 */
 	public static function depends_on()
 	{
-		return array(
+		return [
 			'\phpbb\db\migration\data\v400\dev',
 			'\phpbb\ads\migrations\v20x\m1_hide_ad_for_group',
-		);
+		];
 	}
 
 	public function update_data()
 	{
-		return array(
+		return [
 			['config.add', ['storage\\phpbb_ads\\provider', local::class]],
 			['config.add', ['storage\\phpbb_ads\\config\\path', 'images/phpbb_ads']],
 			['custom', [[$this, 'migrate_ads_storage']]],
-		);
+		];
 	}
 
 	public function migrate_ads_storage()
@@ -67,14 +69,29 @@ public function migrate_ads_storage()
 
 		if ($handle)
 		{
+			$files = [];
 			while (($file = readdir($handle)) !== false)
 			{
 				if ($file === '.' || $file === '..')
 				{
 					continue;
 				}
 
-				$file_tracker->track_file('phpbb_ads', $file, filesize($dir . '/' . $file));
+				$files[] = [
+					'file_path'		=> $file,
+					'filesize'		=> filesize($dir . '/' . $file),
+				];
+
+				if (count($files) >= self::BATCH_SIZE)
+				{
+					$file_tracker->track_files('phpbb_ads', $files);
+					$files = [];
+				}
+			}
+
+			if (!empty($files))
+			{
+				$file_tracker->track_files('phpbb_ads', $files);
 			}
 
 			closedir($handle);