refactor(account-service): replace deprecated config APIs

Signed-off-by: Luis Amorim <luisfelipeamorim@hotmail.com>

Author: lfals

lib/Service/AccountService.php

@@ -25,6 +25,7 @@
 use OCA\Libresign\Helper\FileUploadHelper;
 use OCA\Libresign\Helper\ValidateHelper;
 use OCA\Settings\Mailer\NewUserMailHelper;
+use OCP\Config\IUserConfig;
 use OCP\Accounts\IAccountManager;
 use OCP\AppFramework\Db\DoesNotExistException;
 use OCP\AppFramework\Utility\ITimeFactory;
@@ -35,7 +36,6 @@
 use OCP\Files\NotFoundException;
 use OCP\Http\Client\IClientService;
 use OCP\IAppConfig;
-use OCP\IConfig;
 use OCP\IGroupManager;
 use OCP\IL10N;
 use OCP\IURLGenerator;
@@ -61,8 +61,8 @@ public function __construct(
 		private SignFileService $signFileService,
 		private RequestSignatureService $requestSignatureService,
 		private CertificateEngineFactory $certificateEngineFactory,
-		private IConfig $config,
 		private IAppConfig $appConfig,
+		private IUserConfig $userConfig,
 		private IMountProviderCollection $mountProviderCollection,
 		private NewUserMailHelper $newUserMail,
 		private IdentifyMethodService $identifyMethodService,
@@ -160,7 +160,7 @@ public function createToSign(string $uuid, string $email, string $password, ?str
 
 		$this->updateIdentifyMethodToAccount($signRequest->getId(), $email, $newUser->getUID());
 
-		if ($this->config->getAppValue('core', 'newUser.sendEmail', 'yes') === 'yes') {
+		if ($this->appConfig->getValueString('core', 'newUser.sendEmail', 'yes') === 'yes') {
 			try {
 				$emailTemplate = $this->newUserMail->generateTemplate($newUser, false);
 				$this->newUserMail->sendMail($newUser, $emailTemplate);
@@ -262,16 +262,15 @@ private function getUserConfigByKey(string $key, ?IUser $user = null): string {
 		if (!$user) {
 			return '';
 		}
-
-		return $this->config->getUserValue($user->getUID(), Application::APP_ID, $key);
+		return $this->userConfig->getValueString($user->getUID(), Application::APP_ID, $key);
 	}
 
 	private function getUserConfigIdDocsFilters(?IUser $user = null): array {
 		if (!$user) {
 			return [];
 		}
 
-		$value = $this->config->getUserValue($user->getUID(), Application::APP_ID, 'id_docs_filters', '');
+		$value = $this->userConfig->getValueString($user->getUID(), Application::APP_ID, 'id_docs_filters', '');
 		if (empty($value)) {
 			return [];
 		}
@@ -285,7 +284,7 @@ private function getUserConfigCrlFilters(?IUser $user = null): array {
 			return [];
 		}
 
-		$value = $this->config->getUserValue($user->getUID(), Application::APP_ID, 'crl_filters', '');
+		$value = $this->userConfig->getValueString($user->getUID(), Application::APP_ID, 'crl_filters', '');
 		if (empty($value)) {
 			return [];
 		}
@@ -299,7 +298,7 @@ private function getUserConfigCrlSort(?IUser $user): array {
 			return ['sortBy' => 'revoked_at', 'sortOrder' => 'DESC'];
 		}
 
-		$value = $this->config->getUserValue($user->getUID(), Application::APP_ID, 'crl_sort', '');
+		$value = $this->userConfig->getValueString($user->getUID(), Application::APP_ID, 'crl_sort', '');
 		if (empty($value)) {
 			return ['sortBy' => 'revoked_at', 'sortOrder' => 'DESC'];
 		}
@@ -313,7 +312,7 @@ private function getUserConfigIdDocsSort(?IUser $user): array {
 			return ['sortBy' => null, 'sortOrder' => null];
 		}
 
-		$value = $this->config->getUserValue($user->getUID(), Application::APP_ID, 'id_docs_sort', '');
+		$value = $this->userConfig->getValueString($user->getUID(), Application::APP_ID, 'id_docs_sort', '');
 		if (empty($value)) {
 			return ['sortBy' => null, 'sortOrder' => null];
 		}