OCA\Ocr\Service\AppConfigService

Complexity

Total Complexity

25

Size/Duplication

Total Lines	148
Duplicated Lines	9.46 %

Coupling/Cohesion

Components	1
Dependencies	2

Test Coverage

Coverage

100%

Importance

Changes

0

8 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	6	1
A	getAppValue()	0	3	1
A	evaluateRedisSettings()	0	11	4
B	setAppValue()	0	20	6
A	checkLanguages()	3	6	3
A	checkRedisHost()	5	7	3
A	checkRedisPort()	3	5	4
A	checkRedisDb()	3	5	3

<?php

/**
 * Nextcloud - OCR
 * This file is licensed under the Affero General Public License version 3 or
 * later.
 * See the COPYING file.
 * 
 * @author Janis Koehr <janiskoehr@icloud.com>
 * @copyright Janis Koehr 2017
 */
namespace OCA\Ocr\Service;

use \OCP\IConfig;
use OCP\IL10N;
use OCA\Ocr\Constants\OcrConstants;
use OCA\Ocr\Util\RedisUtil;
use OCP\ILogger;


class AppConfigService {

    /**
     *
     * @var IConfig
     */
    private $config;

    /**
     *
     * @var IL10N
     */
    private $l10n;

    /**
     *
     * @var RedisUtil
     */
    private $redisUtil;

    /**
     *
     * @var ILogger
     */
    private $logger;

    /**
     * Constructor
     * 
     * @param IConfig $config            
     * @param IL10N $l10n            
     * @param RedisUtil $redisUtil            
     * @param ILogger $logger            
     */
    public function __construct(IConfig $config, IL10N $l10n, RedisUtil $redisUtil, ILogger $logger) {
        $this->config = $config;
        $this->l10n = $l10n;
        $this->redisUtil = $redisUtil;
        $this->logger = $logger;
    }

    /**
     * Get a value by key
     * 
     * @param string $key            
     * @return string
     */
    public function getAppValue($key) {
        return $this->config->getAppValue(OcrConstants::APP_NAME, $key);
    }

    /**
     * Evaluate if all needed redis related settings are set.
     * 
     * @return boolean
     */
    public function evaluateRedisSettings() {
        if ($this->config->getAppValue(OcrConstants::APP_NAME, OcrConstants::REDIS_CONFIG_KEY_HOST) !== '' &&
                $this->config->getAppValue(OcrConstants::APP_NAME, OcrConstants::REDIS_CONFIG_KEY_PORT) !== '' &&
                        $this->config->getAppValue(OcrConstants::APP_NAME, OcrConstants::REDIS_CONFIG_KEY_DB) !== '') {
            // test the connection and authenticate
            $this->redisUtil->setupRedisInstance();
            return true;
        } else {
            throw new NotFoundException($this->l10n->t('Please setup Redis in the administration settings first.'));
        }
    }

    /**
     * Set a value for the given key.
     * 
     * @param string $key            
     * @param string $value            
     * @return string
     * @throws NotFoundException
     */
    public function setAppValue($key, $value) {
        if (empty($key)) {
            throw new NotFoundException($this->l10n->t('The given settings key is empty.'));
        }
        switch ($key) {
            case OcrConstants::LANGUAGES_CONFIG_KEY :
                $this->checkLanguages($value);
                break;
            case OcrConstants::REDIS_CONFIG_KEY_HOST :
                $this->checkRedisHost($value);
                break;
            case OcrConstants::REDIS_CONFIG_KEY_PORT :
                $this->checkRedisPort($value);
                break;
            case OcrConstants::REDIS_CONFIG_KEY_DB :
                $this->checkRedisDb($value);
                break;
        }
        return $this->config->setAppValue(OcrConstants::APP_NAME, $key, $value);
    }

    /**
     * Checks for the right languages format.
     * 
     * @param string $value            
     * @throws NotFoundException
     */
    private function checkLanguages($value) {
        if (empty($value) ||

This code seems to be duplicated across your project.

                 !preg_match('/^[a-zA-Z\_\-\/]+(\;[a-zA-Z\_\-\/]+)*$/', $value)) {
            throw new NotFoundException($this->l10n->t('The languages are not specified in the correct format.'));
        }
    }

    /**
     * Checks for the right Redis host format.
     * 
     * @param string $value            
     * @throws NotFoundException
     */
    private function checkRedisHost($value) {
        if (empty($value) || !preg_match(
                '/(^(([0-9]|[1-9][0-9]|1[0-9]{2}|2[0-4][0-9]|25[0-5])\.){3}([0-9]|[1-9][0-9]|1[0-9]{2}|2[0-4][0-9]|25[0-5])$|^(([a-zA-Z0-9]|[a-zA-Z0-9][a-zA-Z0-9\-]*[a-zA-Z0-9])\.)*([A-Za-z0-9]|[A-Za-z0-9][A-Za-z0-9\-]*[A-Za-z0-9])$)/', 
                $value)) {
            throw new NotFoundException($this->l10n->t('The Redis host is not specified in the correct format.'));
        }
    }

    /**
     * Checks for the right Redis port format.
     * 
     * @param string $value            
     * @throws NotFoundException
     */
    private function checkRedisPort($value) {
        if (empty($value) || !($value > 0 && $value < 65535)) {
            throw new NotFoundException($this->l10n->t('The Redis port number is not specified in the correct format.'));
        }
    }

    /**
     * Checks for the right Redis DB format.
     * 
     * @param string $value            
     * @throws NotFoundException
     */
    private function checkRedisDb($value) {
        if ($value === '' || !preg_match('/^\d*$/', $value)) {
            throw new NotFoundException($this->l10n->t('The Redis DB is not specified in the correct format.'));
        }
    }
}