Skip to content

Commit 5b4cde4

Browse files
committed
fix(api): use correct PolicyService method names
- Use saveSystem() instead of saveSystemPolicy() - Use getEffectiveValue() instead of effectiveValue() - Psalm no longer reports errors for SignatureTextService Signed-off-by: Vitor Mattos <[email protected]>
1 parent ae4f7ca commit 5b4cde4

1 file changed

Lines changed: 13 additions & 13 deletions

File tree

lib/Service/SignatureTextService.php

Lines changed: 13 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -108,12 +108,12 @@ public function save(
108108
$template = html_entity_decode($template);
109109
if ($this->policyService !== null) {
110110
try {
111-
$this->policyService->saveSystemPolicy(SignatureTextPolicyProvider::KEY_TEMPLATE, $template);
112-
$this->policyService->saveSystemPolicy(SignatureTextPolicyProvider::KEY_SIGNATURE_WIDTH, $signatureWidth);
113-
$this->policyService->saveSystemPolicy(SignatureTextPolicyProvider::KEY_SIGNATURE_HEIGHT, $signatureHeight);
114-
$this->policyService->saveSystemPolicy(SignatureTextPolicyProvider::KEY_TEMPLATE_FONT_SIZE, $templateFontSize);
115-
$this->policyService->saveSystemPolicy(SignatureTextPolicyProvider::KEY_SIGNATURE_FONT_SIZE, $signatureFontSize);
116-
$this->policyService->saveSystemPolicy(SignatureTextPolicyProvider::KEY_RENDER_MODE, $renderMode);
111+
$this->policyService->saveSystem(SignatureTextPolicyProvider::KEY_TEMPLATE, $template);
112+
$this->policyService->saveSystem(SignatureTextPolicyProvider::KEY_SIGNATURE_WIDTH, $signatureWidth);
113+
$this->policyService->saveSystem(SignatureTextPolicyProvider::KEY_SIGNATURE_HEIGHT, $signatureHeight);
114+
$this->policyService->saveSystem(SignatureTextPolicyProvider::KEY_TEMPLATE_FONT_SIZE, $templateFontSize);
115+
$this->policyService->saveSystem(SignatureTextPolicyProvider::KEY_SIGNATURE_FONT_SIZE, $signatureFontSize);
116+
$this->policyService->saveSystem(SignatureTextPolicyProvider::KEY_RENDER_MODE, $renderMode);
117117
} catch (\Throwable) {
118118
$this->appConfig->setValueString(Application::APP_ID, 'signature_text_template', $template);
119119
$this->appConfig->setValueFloat(Application::APP_ID, 'signature_width', $signatureWidth);
@@ -196,7 +196,7 @@ public function parse(string $template = '', array $context = []): array {
196196
public function getTemplate(): string {
197197
if ($this->policyService !== null) {
198198
try {
199-
return (string)$this->policyService->resolve(SignatureTextPolicyProvider::KEY_TEMPLATE)->effectiveValue();
199+
return (string)$this->policyService->resolve(SignatureTextPolicyProvider::KEY_TEMPLATE)->getEffectiveValue();
200200
} catch (\Throwable) {
201201
// Fallback keeps legacy behavior during migration rollout.
202202
}
@@ -485,7 +485,7 @@ public function getFullSignatureHeight(): float {
485485
public function getSignatureWidth(): float {
486486
if ($this->policyService !== null) {
487487
try {
488-
$current = (float)$this->policyService->resolve(SignatureTextPolicyProvider::KEY_SIGNATURE_WIDTH)->effectiveValue();
488+
$current = (float)$this->policyService->resolve(SignatureTextPolicyProvider::KEY_SIGNATURE_WIDTH)->getEffectiveValue();
489489
} catch (\Throwable) {
490490
$current = $this->appConfig->getValueFloat(Application::APP_ID, 'signature_width', self::DEFAULT_SIGNATURE_WIDTH);
491491
}
@@ -505,7 +505,7 @@ public function getSignatureHeight(): float {
505505
private function getSanitizedDimension(string $key, float $default): float {
506506
if ($this->policyService !== null) {
507507
try {
508-
$value = (float)$this->policyService->resolve($key)->effectiveValue();
508+
$value = (float)$this->policyService->resolve($key)->getEffectiveValue();
509509
} catch (\Throwable) {
510510
$value = $this->appConfig->getValueFloat(Application::APP_ID, $key, $default);
511511
}
@@ -529,7 +529,7 @@ public function getTemplateFontSize(): float {
529529
$default = $collectMetadata ? self::TEMPLATE_DEFAULT_FONT_SIZE - 1 : self::TEMPLATE_DEFAULT_FONT_SIZE;
530530
if ($this->policyService !== null) {
531531
try {
532-
return (float)$this->policyService->resolve(SignatureTextPolicyProvider::KEY_TEMPLATE_FONT_SIZE)->effectiveValue();
532+
return (float)$this->policyService->resolve(SignatureTextPolicyProvider::KEY_TEMPLATE_FONT_SIZE)->getEffectiveValue();
533533
} catch (\Throwable) {
534534
// Fallback keeps legacy behavior during migration rollout.
535535
}
@@ -548,7 +548,7 @@ public function getDefaultTemplateFontSize(): float {
548548
private function isCollectMetadataEnabled(): bool {
549549
if ($this->policyService !== null) {
550550
try {
551-
return (bool)$this->policyService->resolve(CollectMetadataPolicy::KEY)->effectiveValue();
551+
return (bool)$this->policyService->resolve(CollectMetadataPolicy::KEY)->getEffectiveValue();
552552
} catch (\Throwable) {
553553
// Fallback keeps legacy behavior during migration rollout.
554554
}
@@ -560,7 +560,7 @@ private function isCollectMetadataEnabled(): bool {
560560
public function getSignatureFontSize(): float {
561561
if ($this->policyService !== null) {
562562
try {
563-
return (float)$this->policyService->resolve(SignatureTextPolicyProvider::KEY_SIGNATURE_FONT_SIZE)->effectiveValue();
563+
return (float)$this->policyService->resolve(SignatureTextPolicyProvider::KEY_SIGNATURE_FONT_SIZE)->getEffectiveValue();
564564
} catch (\Throwable) {
565565
// Fallback keeps legacy behavior during migration rollout.
566566
}
@@ -571,7 +571,7 @@ public function getSignatureFontSize(): float {
571571
public function getRenderMode(): string {
572572
if ($this->policyService !== null) {
573573
try {
574-
return (string)$this->policyService->resolve(SignatureTextPolicyProvider::KEY_RENDER_MODE)->effectiveValue();
574+
return (string)$this->policyService->resolve(SignatureTextPolicyProvider::KEY_RENDER_MODE)->getEffectiveValue();
575575
} catch (\Throwable) {
576576
// Fallback keeps legacy behavior during migration rollout.
577577
}

0 commit comments

Comments
 (0)