fix: add a config lexicon entry

Signed-off-by: Anupam Kumar <kyteinsky@gmail.com>

Author: kyteinsky

core/AppInfo/ConfigLexicon.php

@@ -29,6 +29,7 @@ class ConfigLexicon implements ILexicon {
 	public const USER_LANGUAGE = 'lang';
 	public const OCM_DISCOVERY_ENABLED = 'ocm_discovery_enabled';
 	public const OCM_INVITE_ACCEPT_DIALOG = 'ocm_invite_accept_dialog';
+	public const DAV_REPAIR_REMOVED_BROKEN_PROPERTIES = 'dav_repair_removed_broken_properties';
 
 	public const USER_LOCALE = 'locale';
 	public const USER_TIMEZONE = 'timezone';
@@ -103,6 +104,13 @@ public function getAppConfigs(): array {
 				defaultRaw: true,
 				definition: 'Whether on demand preview migration is enabled.'
 			),
+			new Entry(
+				key: self::DAV_REPAIR_REMOVED_BROKEN_PROPERTIES,
+				type: ValueType::BOOL,
+				defaultRaw: false,
+				definition: 'Whether the RemoveBrokenProperties repair step has already been run.',
+				lazy: true,
+			),
 		];
 	}
 

lib/private/Repair/RemoveBrokenProperties.php

@@ -17,6 +17,8 @@
 use Override;
 
 class RemoveBrokenProperties implements IRepairStep {
+	public const GUARD_CONFIG_KEY = 'dav_repair_removed_broken_properties';
+
 	public function __construct(
 		private readonly IDBConnection $db,
 		private readonly IAppConfig $appConfig,
@@ -30,7 +32,7 @@ public function getName(): string {
 
 	#[Override]
 	public function run(IOutput $output): void {
-		if ($this->appConfig->getValueBool('core', 'repair_removed_broken_properties', lazy: true)) {
+		if ($this->appConfig->getValueBool('core', self::GUARD_CONFIG_KEY, lazy: true)) {
 			return;
 		}
 
@@ -63,7 +65,7 @@ public function run(IOutput $output): void {
 		}
 		$total = count($brokenIds);
 
-		$this->appConfig->setValueBool('core', 'repair_removed_broken_properties', true, lazy: true);
+		$this->appConfig->setValueBool('core', self::GUARD_CONFIG_KEY, true, lazy: true);
 		$output->info("$total broken object properties removed");
 	}
 }