refactor(policy): resolve collect_metadata via PolicyService in signature text

Signed-off-by: Vitor Mattos <[email protected]>

Author: vitormattos

lib/Service/SignatureTextService.php

@@ -15,6 +15,8 @@
 use ImagickPixel;
 use OCA\Libresign\AppInfo\Application;
 use OCA\Libresign\Exception\LibresignException;
+use OCA\Libresign\Service\Policy\PolicyService;
+use OCA\Libresign\Service\Policy\Provider\CollectMetadata\CollectMetadataPolicy;
 use OCA\Libresign\Vendor\Twig\Environment;
 use OCA\Libresign\Vendor\Twig\Error\SyntaxError;
 use OCA\Libresign\Vendor\Twig\Loader\FilesystemLoader;
@@ -41,6 +43,7 @@ public function __construct(
 		private IRequest $request,
 		private IUserSession $userSession,
 		protected LoggerInterface $logger,
+		private ?PolicyService $policyService = null,
 	) {
 	}
 
@@ -190,7 +193,7 @@ public function getAvailableVariables(): array {
 			'{{SignerEmail}}' => $this->l10n->t('The signer\'s email is optional and can be left blank.'),
 			'{{SignerIdentifier}}' => $this->l10n->t('Unique information used to identify the signer (such as email, phone number, or username).'),
 		];
-		$collectMetadata = $this->appConfig->getValueBool(Application::APP_ID, 'collect_metadata', false);
+		$collectMetadata = $this->isCollectMetadataEnabled();
 		if ($collectMetadata) {
 			$list['{{SignerIP}}'] = $this->l10n->t('IP address of the person who signed the document.');
 			$list['{{SignerUserAgent}}'] = $this->l10n->t('Browser and device information of the person who signed the document.');
@@ -400,7 +403,7 @@ private function mbWordwrap(string $text, int $width, string $break = "\n", bool
 	}
 
 	public function getDefaultTemplate(): string {
-		$collectMetadata = $this->appConfig->getValueBool(Application::APP_ID, 'collect_metadata', false);
+		$collectMetadata = $this->isCollectMetadataEnabled();
 		if ($collectMetadata) {
 			// TRANSLATORS Variables enclosed in double curly braces {{variableName}} are template placeholders.
 			//
@@ -480,21 +483,33 @@ private function getSanitizedDimension(string $key, float $default): float {
 	}
 
 	public function getTemplateFontSize(): float {
-		$collectMetadata = $this->appConfig->getValueBool(Application::APP_ID, 'collect_metadata', false);
+		$collectMetadata = $this->isCollectMetadataEnabled();
 		if ($collectMetadata) {
 			return $this->appConfig->getValueFloat(Application::APP_ID, 'template_font_size', self::TEMPLATE_DEFAULT_FONT_SIZE - 1);
 		}
 		return $this->appConfig->getValueFloat(Application::APP_ID, 'template_font_size', self::TEMPLATE_DEFAULT_FONT_SIZE);
 	}
 
 	public function getDefaultTemplateFontSize(): float {
-		$collectMetadata = $this->appConfig->getValueBool(Application::APP_ID, 'collect_metadata', false);
+		$collectMetadata = $this->isCollectMetadataEnabled();
 		if ($collectMetadata) {
 			return self::TEMPLATE_DEFAULT_FONT_SIZE - 0.2;
 		}
 		return self::TEMPLATE_DEFAULT_FONT_SIZE;
 	}
 
+	private function isCollectMetadataEnabled(): bool {
+		if ($this->policyService !== null) {
+			try {
+				return (bool)$this->policyService->resolve(CollectMetadataPolicy::KEY)->effectiveValue();
+			} catch (\Throwable) {
+				// Fallback keeps legacy behavior during migration rollout.
+			}
+		}
+
+		return $this->appConfig->getValueBool(Application::APP_ID, CollectMetadataPolicy::SYSTEM_APP_CONFIG_KEY, false);
+	}
+
 	public function getSignatureFontSize(): float {
 		return $this->appConfig->getValueFloat(Application::APP_ID, 'signature_font_size', self::SIGNATURE_DEFAULT_FONT_SIZE);
 	}