fix(lexicon): renaming and minor fixes

Signed-off-by: Maxence Lange <[email protected]>
nextcloud · Dec 12, 2024 · bf0b199 · bf0b199
1 parent cafa3eb
commit bf0b199
Show file tree

Hide file tree

Showing 4 changed files with 15 additions and 15 deletions.
diff --git a/lib/private/AppConfig.php b/lib/private/AppConfig.php
@@ -436,7 +436,7 @@ private function getTypedValue(
 		int $type,
 	): string {
 		$this->assertParams($app, $key, valueType: $type);
-		if (!$this->compareRegisteredConfigValues($app, $key, $lazy, $type, $default)) {
+		if (!$this->matchAndApplyLexiconDefinition($app, $key, $lazy, $type, $default)) {
 			return $default; // returns default if strictness of lexicon is set to WARNING (block and report)
 		}
 		$this->loadConfig($app, $lazy);
@@ -730,7 +730,7 @@ private function setTypedValue(
 		int $type,
 	): bool {
 		$this->assertParams($app, $key);
-		if (!$this->compareRegisteredConfigValues($app, $key, $lazy, $type)) {
+		if (!$this->matchAndApplyLexiconDefinition($app, $key, $lazy, $type)) {
 			return false; // returns false as database is not updated
 		}
 		$this->loadConfig(null, $lazy);
@@ -1573,13 +1573,13 @@ public function clearCachedConfig(): void {
 	}
 
 	/**
-	 * verify and compare current use of config values with defined lexicon
+	 * match and apply current use of config values with defined lexicon
 	 *
 	 * @throws AppConfigUnknownKeyException
 	 * @throws AppConfigTypeConflictException
 	 * @return bool TRUE if everything is fine compared to lexicon or lexicon does not exist
 	 */
-	private function compareRegisteredConfigValues(
+	private function matchAndApplyLexiconDefinition(
 		string $app,
 		string $key,
 		bool &$lazy,

diff --git a/lib/private/Config/UserConfig.php b/lib/private/Config/UserConfig.php
@@ -711,7 +711,7 @@ private function getTypedValue(
 		ValueType $type,
 	): string {
 		$this->assertParams($userId, $app, $key);
-		if (!$this->compareRegisteredConfigValues($app, $key, $lazy, $type, default: $default)) {
+		if (!$this->matchAndApplyLexiconDefinition($app, $key, $lazy, $type, default: $default)) {
 			return $default; // returns default if strictness of lexicon is set to WARNING (block and report)
 		}
 		$this->loadConfig($userId, $lazy);
@@ -1046,7 +1046,7 @@ private function setTypedValue(
 		ValueType $type,
 	): bool {
 		$this->assertParams($userId, $app, $key);
-		if (!$this->compareRegisteredConfigValues($app, $key, $lazy, $type, $flags)) {
+		if (!$this->matchAndApplyLexiconDefinition($app, $key, $lazy, $type, $flags)) {
 			return false; // returns false as database is not updated
 		}
 		$this->loadConfig($userId, $lazy);
@@ -1816,12 +1816,12 @@ private function decryptSensitiveValue(string $userId, string $app, string $key,
 	}
 
 	/**
-	 * verify and compare current use of config values with defined lexicon
+	 * match and apply current use of config values with defined lexicon
 	 *
 	 * @throws UnknownKeyException
 	 * @throws TypeConflictException
 	 */
-	private function compareRegisteredConfigValues(
+	private function matchAndApplyLexiconDefinition(
 		string $app,
 		string $key,
 		bool &$lazy,
@@ -1837,7 +1837,7 @@ private function compareRegisteredConfigValues(
 		/** @var ConfigLexiconEntry $configValue */
 		$configValue = $configDetails['entries'][$key];
 		if ($type === ValueType::MIXED) {
-			$type = $configValue->getValueType()->value; // we overwrite if value was requested as mixed
+			$type = $configValue->getValueType(); // we overwrite if value was requested as mixed
 		} elseif ($configValue->getValueType() !== $type) {
 			throw new TypeConflictException('The user config key ' . $app . '/' . $key . ' is typed incorrectly in relation to the config lexicon');
 		}

diff --git a/lib/unstable/Config/Lexicon/ConfigLexiconEntry.php b/lib/unstable/Config/Lexicon/ConfigLexiconEntry.php
@@ -174,7 +174,7 @@ public function getFlags(): int {
 	 * @experimental 31.0.0
 	 */
 	public function isFlagged(int $flag): bool {
-		return (bool)($flag & $this->getFlags());
+		return (($flag & $this->getFlags()) === $flag);
 	}
 
 	/**

diff --git a/lib/unstable/Config/Lexicon/ConfigLexiconStrictness.php b/lib/unstable/Config/Lexicon/ConfigLexiconStrictness.php
@@ -18,13 +18,13 @@
  *
  * @experimental 31.0.0
  */
-enum ConfigLexiconStrictness: int {
+enum ConfigLexiconStrictness {
 	/** @experimental 31.0.0 */
-	case IGNORE = 0; // fully ignore
+	case IGNORE; // fully ignore
 	/** @experimental 31.0.0 */
-	case NOTICE = 2; // ignore and report
+	case NOTICE; // ignore and report
 	/** @experimental 31.0.0 */
-	case WARNING = 3; // silently block (returns $default) and report
+	case WARNING; // silently block (returns $default) and report
 	/** @experimental 31.0.0 */
-	case EXCEPTION = 5; // block (throws exception) and report
+	case EXCEPTION; // block (throws exception) and report
 }