OnlyofficeAppsettings::getSetting() - Code Metrics - Inspection of "Plugin: Onlyoffice: Import plugin from 1.11.x (nee..." - chamilo/chamilo-lms - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( 262b3f...86674d )

by Yannick

created 2025-07-09 15:47 UTC

OnlyofficeAppsettings::getSetting() F

↳ Parent: OnlyofficeAppsettings

Complexity

Conditions	20
Paths	650

Size

Total Lines	58
Code Lines	40

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
cc	20
eloc	40
c	1
b	0
f	0
nc	650
nop	1
dl	0
loc	58
rs	0.4861

How to fix Long Method Complexity

<?php
/**
 * (c) Copyright Ascensio System SIA 2025.
 *
 * Licensed under the Apache License, Version 2.0 (the "License");
 * you may not use this file except in compliance with the License.
 * You may obtain a copy of the License at
 *
 *     http://www.apache.org/licenses/LICENSE-2.0
 *
 * Unless required by applicable law or agreed to in writing, software
 * distributed under the License is distributed on an "AS IS" BASIS,
 * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
 * See the License for the specific language governing permissions and
 * limitations under the License.
 */
use Onlyoffice\DocsIntegrationSdk\Manager\Settings\SettingsManager;

class OnlyofficeAppsettings extends SettingsManager
{
    /**
     * Link to Docs Cloud.
     *
     * @var string
     */
    public const LINK_TO_DOCS = 'https://www.onlyoffice.com/docs-registration.aspx?referer=chamilo';
    /**
     * The settings key for the document server address.
     *
     * @var string
     */
    public $documentServerUrl = 'document_server_url';

    /**
     * The config key for the jwt header.
     *
     * @var string
     */
    public $jwtHeader = 'onlyoffice_jwt_header';

    /**
     * The config key for the internal url.
     *
     * @var string
     */
    public $documentServerInternalUrl = 'onlyoffice_internal_url';

    /**
     * The config key for the storage url.
     *
     * @var string
     */
    public $storageUrl = 'onlyoffice_storage_url';

    /**
     * The config key for the demo data.
     *
     * @var string
     */
    public $useDemoName = 'onlyoffice_connect_demo_data';

    /**
     * Chamilo plugin.
     */
    public $plugin;

    public $newSettings;

    /**
     * The config key for JWT secret key.
     *
     * @var string
     */
    protected $jwtKey = 'jwt_secret';

    public function __construct(Plugin $plugin, ?array $newSettings = null)
    {
        parent::__construct();
        $this->plugin = $plugin;
        $this->newSettings = $newSettings;
    }

    public function getSetting($settingName)
    {
        $value = null;
        if (null !== $this->newSettings) {
            if (isset($this->newSettings[$settingName])) {
                $value = $this->newSettings[$settingName];
            }

            if (empty($value)) {
                $prefix = $this->plugin->getPluginName();

                if (substr($settingName, 0, strlen($prefix)) == $prefix) {
                    $settingNameWithoutPrefix = substr($settingName, strlen($prefix) + 1);
                }

                if (isset($this->newSettings[$settingNameWithoutPrefix])) {

                    $value = $this->newSettings[$settingNameWithoutPrefix];
                }
            }
            if ($this->isSettingUrl($value)) {
                $value = $this->processUrl($value);
            }
            if (!empty($value)) {
                return $value;
            }
        }
        switch ($settingName) {
            case $this->jwtHeader:
                $settings = api_get_setting($settingName);
                $value = is_array($settings) && array_key_exists($this->plugin->getPluginName(), $settings)
                    ? $settings[$this->plugin->getPluginName()]
                    : null;

                if (empty($value)) {
                    $value = 'Authorization';
                }
                break;
            case $this->documentServerInternalUrl:
                $settings = api_get_setting($settingName);
                $value = is_array($settings) ? ($settings[$this->plugin->getPluginName()] ?? null) : null;
                break;
            case $this->useDemoName:
                $settings = api_get_setting($settingName);
                $value = is_array($settings) ? ($settings[0] ?? null) : null;
                break;
            case $this->jwtPrefix:
                $value = 'Bearer ';
                break;
            default:
                if (!empty($this->plugin) && method_exists($this->plugin, 'get')) {
                    $value = $this->plugin->get($settingName);
                }
        }
        if (empty($value)) {
            $value = api_get_configuration_value($settingName);
        }

        return $value;
    }

    public function setSetting($settingName, $value, $createSetting = false)
    {
        if (($settingName === $this->useDemoName) && $createSetting) {
            api_add_setting($value, $settingName, null, 'setting', 'Plugins');

            return;
        }

        $prefix = $this->plugin->getPluginName();
        if (!(substr($settingName, 0, strlen($prefix)) == $prefix)) {
            $settingName = $prefix.'_'.$settingName;
        }
        api_set_setting($settingName, $value);
    }

    public function getServerUrl()
    {
        return api_get_path(WEB_PATH);
    }

    /**
     * Get link to Docs Cloud.
     *
     * @return string
     */
    public function getLinkToDocs()
    {
        return self::LINK_TO_DOCS;
    }

    public function isSettingUrl($settingName)
    {
        return in_array($settingName, [$this->documentServerUrl, $this->documentServerInternalUrl, $this->storageUrl]);
    }
}


1			<?php
2			/**
3			* (c) Copyright Ascensio System SIA 2025.
4			*
5			* Licensed under the Apache License, Version 2.0 (the "License");
6			* you may not use this file except in compliance with the License.
7			* You may obtain a copy of the License at
8			*
9			* http://www.apache.org/licenses/LICENSE-2.0
10			*
11			* Unless required by applicable law or agreed to in writing, software
12			* distributed under the License is distributed on an "AS IS" BASIS,
13			* WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
14			* See the License for the specific language governing permissions and
15			* limitations under the License.
16			*/
17			use Onlyoffice\DocsIntegrationSdk\Manager\Settings\SettingsManager;
18
19			class OnlyofficeAppsettings extends SettingsManager
20			{
21			/**
22			* Link to Docs Cloud.
23			*
24			* @var string
25			*/
26			public const LINK_TO_DOCS = 'https://www.onlyoffice.com/docs-registration.aspx?referer=chamilo';
27			/**
28			* The settings key for the document server address.
29			*
30			* @var string
31			*/
32			public $documentServerUrl = 'document_server_url';
33
34			/**
35			* The config key for the jwt header.
36			*
37			* @var string
38			*/
39			public $jwtHeader = 'onlyoffice_jwt_header';
40
41			/**
42			* The config key for the internal url.
43			*
44			* @var string
45			*/
46			public $documentServerInternalUrl = 'onlyoffice_internal_url';
47
48			/**
49			* The config key for the storage url.
50			*
51			* @var string
52			*/
53			public $storageUrl = 'onlyoffice_storage_url';
54
55			/**
56			* The config key for the demo data.
57			*
58			* @var string
59			*/
60			public $useDemoName = 'onlyoffice_connect_demo_data';
61
62			/**
63			* Chamilo plugin.
64			*/
65			public $plugin;
66
67			public $newSettings;
68
69			/**
70			* The config key for JWT secret key.
71			*
72			* @var string
73			*/
74			protected $jwtKey = 'jwt_secret';
75
76			public function __construct(Plugin $plugin, ?array $newSettings = null)
77			{
78			parent::__construct();
79			$this->plugin = $plugin;
80			$this->newSettings = $newSettings;
81			}
82
83			public function getSetting($settingName)
84			{
85			$value = null;
86			if (null !== $this->newSettings) {
87			if (isset($this->newSettings[$settingName])) {
88			$value = $this->newSettings[$settingName];
89			}
90
91			if (empty($value)) {
92			$prefix = $this->plugin->getPluginName();
93
94			if (substr($settingName, 0, strlen($prefix)) == $prefix) {
95			$settingNameWithoutPrefix = substr($settingName, strlen($prefix) + 1);
96			}
97
98			if (isset($this->newSettings[$settingNameWithoutPrefix])) {
			0 ignored issues – show Comprehensibility Best Practice introduced 2024-10-18 13:08 UTC by Report Bug Copy Issue Report The variable `$settingNameWithoutPrefix` does not seem to be defined for all execution paths leading up to this point. Loading history...
99			$value = $this->newSettings[$settingNameWithoutPrefix];
100			}
101			}
102			if ($this->isSettingUrl($value)) {
103			$value = $this->processUrl($value);
104			}
105			if (!empty($value)) {
106			return $value;
107			}
108			}
109			switch ($settingName) {
110			case $this->jwtHeader:
111			$settings = api_get_setting($settingName);
112			$value = is_array($settings) && array_key_exists($this->plugin->getPluginName(), $settings)
113			? $settings[$this->plugin->getPluginName()]
114			: null;
115
116			if (empty($value)) {
117			$value = 'Authorization';
118			}
119			break;
120			case $this->documentServerInternalUrl:
121			$settings = api_get_setting($settingName);
122			$value = is_array($settings) ? ($settings[$this->plugin->getPluginName()] ?? null) : null;
123			break;
124			case $this->useDemoName:
125			$settings = api_get_setting($settingName);
126			$value = is_array($settings) ? ($settings[0] ?? null) : null;
127			break;
128			case $this->jwtPrefix:
129			$value = 'Bearer ';
130			break;
131			default:
132			if (!empty($this->plugin) && method_exists($this->plugin, 'get')) {
133			$value = $this->plugin->get($settingName);
134			}
135			}
136			if (empty($value)) {
137			$value = api_get_configuration_value($settingName);
138			}
139
140			return $value;
141			}
142
143			public function setSetting($settingName, $value, $createSetting = false)
144			{
145			if (($settingName === $this->useDemoName) && $createSetting) {
146			api_add_setting($value, $settingName, null, 'setting', 'Plugins');
147
148			return;
149			}
150
151			$prefix = $this->plugin->getPluginName();
152			if (!(substr($settingName, 0, strlen($prefix)) == $prefix)) {
153			$settingName = $prefix.'_'.$settingName;
154			}
155			api_set_setting($settingName, $value);
156			}
157
158			public function getServerUrl()
159			{
160			return api_get_path(WEB_PATH);
161			}
162
163			/**
164			* Get link to Docs Cloud.
165			*
166			* @return string
167			*/
168			public function getLinkToDocs()
169			{
170			return self::LINK_TO_DOCS;
171			}
172
173			public function isSettingUrl($settingName)
174			{
175			return in_array($settingName, [$this->documentServerUrl, $this->documentServerInternalUrl, $this->storageUrl]);
176			}
177			}
178

chamilo / chamilo-lms

Push — master ( 262b3f...86674d )

OnlyofficeAppsettings::getSetting() F

Complexity

Size

Duplication

Importance

How to fix Long Method Complexity

Long Method

Duplication Side-by-Side

Filter issues like