fix: set per-user instead of globally

This commit is contained in:
transcaffeine 2024-08-08 23:16:22 +02:00
parent ea9a60558f
commit 7cb5b87311
Signed by: transcaffeine
GPG Key ID: 03624C433676E465
2 changed files with 17 additions and 14 deletions

View File

@ -16,20 +16,23 @@ use OCP\AppFramework\Http\Attribute\NoAdminRequired;
use OCP\AppFramework\Http\Attribute\NoCSRFRequired;
class SettingsController extends Controller {
public function __construct($appName, IRequest $request, private AppSettings $appSettings) {
private $userId;
public function __construct($appName, IRequest $request, private AppSettings $appSettings, $userId) {
parent::__construct($appName, $request);
}
#[NoAdminRequired]
#[NoCSRFRequired]
public function get(): JSONResponse {
return $this->makeJSONResponse(fn () => $this->appSettings->getAll());
return $this->makeJSONResponse(fn () => $this->appSettings->getAll($this->userId));
}
#[NoAdminRequired]
public function set(string $folderVariant, string $buttonVariant): JSONResponse {
$this->appSettings->set($folderVariant, $buttonVariant);
return $this->makeJSONResponse(fn () => $this->appSettings->getAll());
$this->appSettings->set($this->userId, $folderVariant, $buttonVariant);
return $this->makeJSONResponse(fn () => $this->appSettings->getAll($this->userId));
}
protected function makeJSONResponse(Closure $closure): JSONResponse {

View File

@ -12,23 +12,23 @@ class AppSettings {
public function __construct(private IConfig $config) {
}
public function getStringSetting(string $key, string $default = ''): string {
return $this->config->getAppValue(AppConstants::APP_ID, $key) ?: $default;
public function getStringSetting($userId, string $key, string $default = ''): string {
return $this->config->getUserValue(AppConstants::APP_ID, $userId, $key) ?: $default;
}
public function setStringSetting(string $key, string $value): void {
$this->config->setAppValue(AppConstants::APP_ID, $key, $value);
public function setStringSetting($userId, string $key, string $value): void {
$this->config->setUserValue(AppConstants::APP_ID, $userId, $key, $value);
}
public function getAll(): array {
public function getAll($userId): array {
return [
AppSettings::FOLDER_VARIANT => $this->getStringSetting(AppSettings::FOLDER_VARIANT, 'pride'),
AppSettings::BUTTON_VARIANT => $this->getStringSetting(AppSettings::BUTTON_VARIANT, 'trans'),
AppSettings::FOLDER_VARIANT => $this->getStringSetting($userId, AppSettings::FOLDER_VARIANT, 'pride'),
AppSettings::BUTTON_VARIANT => $this->getStringSetting($userId, AppSettings::BUTTON_VARIANT, 'trans'),
];
}
public function set($folder, $button): void {
$this->setStringSetting(AppSettings::FOLDER_VARIANT, $folder);
$this->setStringSetting(AppSettings::BUTTON_VARIANT, $button);
public function set($userId, $folder, $button): void {
$this->setStringSetting($userId, AppSettings::FOLDER_VARIANT, $folder);
$this->setStringSetting($userId, AppSettings::BUTTON_VARIANT, $button);
}
}