SiteUtility::getSolrConnectionConfiguration() - Code Metrics - Inspection of "[ACTIONS:2022.12.22:P:main] Use fixed typo3/coding..." - TYPO3-Solr/ext-solr - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Pull Request — main (#3432)

by Rafael

created 2022-12-22 19:46 UTC

SiteUtility::getSolrConnectionConfiguration() A

↳ Parent: SiteUtility

Complexity

Conditions	4
Paths	2

Size

Total Lines	37
Code Lines	28

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	31
CRAP Score	4

Importance

Changes

Metric	Value
eloc	28
dl	0
loc	37
ccs	31
cts	31
cp	1
rs	9.472
c	0
b	0
f	0
cc	4
nc	2
nop	2
crap	4

<?php

/*
 * This file is part of the TYPO3 CMS project.
 *
 * It is free software; you can redistribute it and/or modify it under
 * the terms of the GNU General Public License, either version 2
 * of the License, or any later version.
 *
 * For the full copyright and license information, please read the
 * LICENSE.txt file that was distributed with this source code.
 *
 * The TYPO3 project - inspiring people to share!
 */

namespace ApacheSolrForTypo3\Solr\System\Util;

use TYPO3\CMS\Core\Exception\SiteNotFoundException;
use TYPO3\CMS\Core\Site\Entity\Site;
use TYPO3\CMS\Core\Site\SiteFinder;
use TYPO3\CMS\Core\Utility\GeneralUtility;

/**
 * This class contains related functions for the new site management that was introduced with TYPO3 9.
 */
class SiteUtility
{
    /**
     * @var array
     */
    public static array $languages = [];

    /**
     * Determines if the site where the page belongs to is managed with the TYPO3 site management.
     *
     * @param int $pageId
     * @return bool
     */
    public static function getIsSiteManagedSite(int $pageId): bool
    {
        $siteFinder = GeneralUtility::makeInstance(SiteFinder::class);
        try {
            /* @var SiteFinder $siteFinder */
            return $siteFinder->getSiteByPageId($pageId) instanceof Site;
        } catch (SiteNotFoundException $e) {

        }
        return false;
    }

    /**
     * This method is used to retrieve the connection configuration from the TYPO3 site configuration.
     *
     * Note: Language context properties have precedence over global settings.
     *
     * The configuration is done in the globals configuration of a site, and be extended in the language specific configuration
     * of a site.
     *
     * Typically, everything except the core name is configured on the global level and the core name differs for each language.
     *
     * In addition, every property can be defined for the ```read``` and ```write``` scope.
     *
     * The convention for property keys is "solr_{propertyName}_{scope}". With the configuration "solr_host_read" you define the host
     * for the solr read connection.
     *
     * @param Site $typo3Site
     * @param string $property
     * @param int $languageId
     * @param string $scope
     * @param mixed $defaultValue
     * @return mixed
     */
    public static function getConnectionProperty(
        Site $typo3Site,
        string $property,
        int $languageId,
        string $scope,
        $defaultValue = null
    ) {
        $value = self::getConnectionPropertyOrFallback($typo3Site, $property, $languageId, $scope);
        if ($value === null) {
            return $defaultValue;
        }
        return $value;
    }

    /**
     * Builds the Solr connection configuration
     *
     * @param Site $typo3Site
     * @param int $languageUid
     * @return array|null
     */
    public static function getSolrConnectionConfiguration(Site $typo3Site, int $languageUid): ?array
    {
        $solrEnabled = self::getConnectionProperty($typo3Site, 'enabled', $languageUid, 'read', true);
        $solrReadCore = self::getConnectionProperty($typo3Site, 'core', $languageUid, 'read');
        $solrWriteCore = self::getConnectionProperty($typo3Site, 'core', $languageUid, 'write');
        if (!$solrEnabled || empty($solrReadCore) || empty($solrWriteCore)) {
            return null;
        }

        $rootPageUid = $typo3Site->getRootPageId();
        return [
            'connectionKey' => $rootPageUid . '|' . $languageUid,
            'rootPageUid' => $rootPageUid,
            'read' => [
                'scheme' => self::getConnectionProperty($typo3Site, 'scheme', $languageUid, 'read', 'http'),
                'host' => self::getConnectionProperty($typo3Site, 'host', $languageUid, 'read', 'localhost'),
                'port' => (int)self::getConnectionProperty($typo3Site, 'port', $languageUid, 'read', 8983),
                // @todo: transform core to path
                'path' =>
                self::getConnectionProperty($typo3Site, 'path', $languageUid, 'read', '/solr/') .
                $solrReadCore . '/' ,
                'username' => self::getConnectionProperty($typo3Site, 'username', $languageUid, 'read', ''),
                'password' => self::getConnectionProperty($typo3Site, 'password', $languageUid, 'read', ''),
            ],
            'write' => [
                'scheme' => self::getConnectionProperty($typo3Site, 'scheme', $languageUid, 'write', 'http'),
                'host' => self::getConnectionProperty($typo3Site, 'host', $languageUid, 'write', 'localhost'),
                'port' => (int)self::getConnectionProperty($typo3Site, 'port', $languageUid, 'write', 8983),
                // @todo: transform core to path
                'path' =>
                self::getConnectionProperty($typo3Site, 'path', $languageUid, 'write', '/solr/') .
                $solrWriteCore . '/' ,
                'username' => self::getConnectionProperty($typo3Site, 'username', $languageUid, 'write', ''),
                'password' => self::getConnectionProperty($typo3Site, 'password', $languageUid, 'write', ''),
            ],

            'language' => $languageUid,
        ];
    }

    /**
     * Builds the Solr connection configuration for all languages of given TYPO3 site
     *
     * @param Site $typo3Site
     * @return array
     */
    public static function getAllSolrConnectionConfigurations(Site $typo3Site): array
    {
        $connections = [];
        foreach ($typo3Site->getLanguages() as $language) {
            $connection = self::getSolrConnectionConfiguration($typo3Site, $language->getLanguageId());
            if ($connection !== null) {
                $connections[$language->getLanguageId()] = $connection;
            }
        }

        return $connections;
    }

    /**
     * Resolves site configuration properties.
     * Language context properties have precedence over global settings.
     *
     * @param Site $typo3Site
     * @param string $property
     * @param int $languageId
     * @param string $scope
     * @return string|bool|null
     */
    protected static function getConnectionPropertyOrFallback(
        Site $typo3Site,
        string $property,
        int $languageId,
        string $scope
    ) {
        if ($scope === 'write' && !self::writeConnectionIsEnabled($typo3Site, $languageId)) {
            $scope = 'read';
        }

        // convention key solr_$property_$scope
        $keyToCheck = 'solr_' . $property . '_' . $scope;

        // convention fallback key solr_$property_read
        $fallbackKey = 'solr_' . $property . '_read';

        // try to find language specific setting if found return it
        $rootPageUid = $typo3Site->getRootPageId();
        if (isset(self::$languages[$rootPageUid][$languageId]) === false) {
            self::$languages[$rootPageUid][$languageId] = $typo3Site->getLanguageById($languageId)->toArray();
        }
        $value = self::getValueOrFallback(self::$languages[$rootPageUid][$languageId], $keyToCheck, $fallbackKey);
        if ($value !== null) {
            return $value;
        }

        // if not found check global configuration
        $siteBaseConfiguration = $typo3Site->getConfiguration();
        return self::getValueOrFallback($siteBaseConfiguration, $keyToCheck, $fallbackKey);
    }

    /**
     * Checks whether write connection is enabled.
     * Language context properties have precedence over global settings.
     *
     * @param Site $typo3Site
     * @param int $languageId
     * @return bool
     */
    protected static function writeConnectionIsEnabled(Site $typo3Site, int $languageId): bool
    {
        $rootPageUid = $typo3Site->getRootPageId();
        if (isset(self::$languages[$rootPageUid][$languageId]) === false) {
            self::$languages[$rootPageUid][$languageId] = $typo3Site->getLanguageById($languageId)->toArray();
        }
        $value = self::getValueOrFallback(self::$languages[$rootPageUid][$languageId], 'solr_use_write_connection', 'solr_use_write_connection');
        if ($value !== null) {
            return $value;

        }

        $siteBaseConfiguration = $typo3Site->getConfiguration();
        $value = self::getValueOrFallback($siteBaseConfiguration, 'solr_use_write_connection', 'solr_use_write_connection');
        if ($value !== null) {
            return $value;

        }
        return false;
    }

    /**
     * @param array $data
     * @param string $keyToCheck
     * @param string $fallbackKey
     * @return string|bool|null
     */
    protected static function getValueOrFallback(array $data, string $keyToCheck, string $fallbackKey)
    {
        $value = $data[$keyToCheck] ?? null;
        if ($value === '0' || $value === 0 || !empty($value)) {
            return self::evaluateConfigurationData($value);
        }

        return self::evaluateConfigurationData($data[$fallbackKey] ?? null);
    }

    /**
     * Evaluate configuration data
     *
     * Setting boolean values via environment variables
     * results in strings like 'false' that may be misinterpreted
     * thus we check for boolean values in strings.
     *
     * @param string|bool|null $value
     * @return string|bool|null
     */
    protected static function evaluateConfigurationData($value)
    {
        if ($value === 'true') {
            return true;
        }
        if ($value === 'false') {
            return false;
        }

        return $value;
    }
}


1		<?php
2
3		/*
4		* This file is part of the TYPO3 CMS project.
5		*
6		* It is free software; you can redistribute it and/or modify it under
7		* the terms of the GNU General Public License, either version 2
8		* of the License, or any later version.
9		*
10		* For the full copyright and license information, please read the
11		* LICENSE.txt file that was distributed with this source code.
12		*
13		* The TYPO3 project - inspiring people to share!
14		*/
15
16		namespace ApacheSolrForTypo3\Solr\System\Util;
17
18		use TYPO3\CMS\Core\Exception\SiteNotFoundException;
19		use TYPO3\CMS\Core\Site\Entity\Site;
20		use TYPO3\CMS\Core\Site\SiteFinder;
21		use TYPO3\CMS\Core\Utility\GeneralUtility;
22
23		/**
24		* This class contains related functions for the new site management that was introduced with TYPO3 9.
25		*/
26		class SiteUtility
27		{
28		/**
29		* @var array
30		*/
31		public static array $languages = [];
32
33		/**
34		* Determines if the site where the page belongs to is managed with the TYPO3 site management.
35		*
36		* @param int $pageId
37		* @return bool
38		*/
39	29	public static function getIsSiteManagedSite(int $pageId): bool
40		{
41	29	$siteFinder = GeneralUtility::makeInstance(SiteFinder::class);
42		try {
43		/* @var SiteFinder $siteFinder */
44	29	return $siteFinder->getSiteByPageId($pageId) instanceof Site;
45		} catch (SiteNotFoundException $e) {
		0 ignored issues – show Coding Style Comprehensibility introduced 2020-02-26 13:27 UTC by Report Bug Copy Issue Report Consider adding a comment why this CATCH block is empty. Loading history...
46		}
47		return false;
48		}
49
50		/**
51		* This method is used to retrieve the connection configuration from the TYPO3 site configuration.
52		*
53		* Note: Language context properties have precedence over global settings.
54		*
55		* The configuration is done in the globals configuration of a site, and be extended in the language specific configuration
56		* of a site.
57		*
58		* Typically, everything except the core name is configured on the global level and the core name differs for each language.
59		*
60		* In addition, every property can be defined for the ```read``` and ```write``` scope.
61		*
62		* The convention for property keys is "solr_{propertyName}_{scope}". With the configuration "solr_host_read" you define the host
63		* for the solr read connection.
64		*
65		* @param Site $typo3Site
66		* @param string $property
67		* @param int $languageId
68		* @param string $scope
69		* @param mixed $defaultValue
70		* @return mixed
71		*/
72	225	public static function getConnectionProperty(
73		Site $typo3Site,
74		string $property,
75		int $languageId,
76		string $scope,
77		$defaultValue = null
78		) {
79	225	$value = self::getConnectionPropertyOrFallback($typo3Site, $property, $languageId, $scope);
80	225	if ($value === null) {
81	207	return $defaultValue;
82		}
83	224	return $value;
84		}
85
86		/**
87		* Builds the Solr connection configuration
88		*
89		* @param Site $typo3Site
90		* @param int $languageUid
91		* @return array\|null
92		*/
93	206	public static function getSolrConnectionConfiguration(Site $typo3Site, int $languageUid): ?array
94		{
95	206	$solrEnabled = self::getConnectionProperty($typo3Site, 'enabled', $languageUid, 'read', true);
96	206	$solrReadCore = self::getConnectionProperty($typo3Site, 'core', $languageUid, 'read');
97	206	$solrWriteCore = self::getConnectionProperty($typo3Site, 'core', $languageUid, 'write');
98	206	if (!$solrEnabled \|\| empty($solrReadCore) \|\| empty($solrWriteCore)) {
99	81	return null;
100		}
101
102	206	$rootPageUid = $typo3Site->getRootPageId();
103	206	return [
104	206	'connectionKey' => $rootPageUid . '\|' . $languageUid,
105	206	'rootPageUid' => $rootPageUid,
106	206	'read' => [
107	206	'scheme' => self::getConnectionProperty($typo3Site, 'scheme', $languageUid, 'read', 'http'),
108	206	'host' => self::getConnectionProperty($typo3Site, 'host', $languageUid, 'read', 'localhost'),
109	206	'port' => (int)self::getConnectionProperty($typo3Site, 'port', $languageUid, 'read', 8983),
110		// @todo: transform core to path
111	206	'path' =>
112	206	self::getConnectionProperty($typo3Site, 'path', $languageUid, 'read', '/solr/') .
113	206	$solrReadCore . '/' ,
114	206	'username' => self::getConnectionProperty($typo3Site, 'username', $languageUid, 'read', ''),
115	206	'password' => self::getConnectionProperty($typo3Site, 'password', $languageUid, 'read', ''),
116	206	],
117	206	'write' => [
118	206	'scheme' => self::getConnectionProperty($typo3Site, 'scheme', $languageUid, 'write', 'http'),
119	206	'host' => self::getConnectionProperty($typo3Site, 'host', $languageUid, 'write', 'localhost'),
120	206	'port' => (int)self::getConnectionProperty($typo3Site, 'port', $languageUid, 'write', 8983),
121		// @todo: transform core to path
122	206	'path' =>
123	206	self::getConnectionProperty($typo3Site, 'path', $languageUid, 'write', '/solr/') .
124	206	$solrWriteCore . '/' ,
125	206	'username' => self::getConnectionProperty($typo3Site, 'username', $languageUid, 'write', ''),
126	206	'password' => self::getConnectionProperty($typo3Site, 'password', $languageUid, 'write', ''),
127	206	],
128
129	206	'language' => $languageUid,
130	206	];
131		}
132
133		/**
134		* Builds the Solr connection configuration for all languages of given TYPO3 site
135		*
136		* @param Site $typo3Site
137		* @return array
138		*/
139	2	public static function getAllSolrConnectionConfigurations(Site $typo3Site): array
140		{
141	2	$connections = [];
142	2	foreach ($typo3Site->getLanguages() as $language) {
143	2	$connection = self::getSolrConnectionConfiguration($typo3Site, $language->getLanguageId());
144	2	if ($connection !== null) {
145	2	$connections[$language->getLanguageId()] = $connection;
146		}
147		}
148
149	2	return $connections;
150		}
151
152		/**
153		* Resolves site configuration properties.
154		* Language context properties have precedence over global settings.
155		*
156		* @param Site $typo3Site
157		* @param string $property
158		* @param int $languageId
159		* @param string $scope
160		* @return string\|bool\|null
161		*/
162	225	protected static function getConnectionPropertyOrFallback(
163		Site $typo3Site,
164		string $property,
165		int $languageId,
166		string $scope
167		) {
168	225	if ($scope === 'write' && !self::writeConnectionIsEnabled($typo3Site, $languageId)) {
169	208	$scope = 'read';
170		}
171
172		// convention key solr_$property_$scope
173	225	$keyToCheck = 'solr_' . $property . '_' . $scope;
174
175		// convention fallback key solr_$property_read
176	225	$fallbackKey = 'solr_' . $property . '_read';
177
178		// try to find language specific setting if found return it
179	225	$rootPageUid = $typo3Site->getRootPageId();
180	225	if (isset(self::$languages[$rootPageUid][$languageId]) === false) {
181	218	self::$languages[$rootPageUid][$languageId] = $typo3Site->getLanguageById($languageId)->toArray();
182		}
183	225	$value = self::getValueOrFallback(self::$languages[$rootPageUid][$languageId], $keyToCheck, $fallbackKey);
184	225	if ($value !== null) {
185	207	return $value;
186		}
187
188		// if not found check global configuration
189	223	$siteBaseConfiguration = $typo3Site->getConfiguration();
190	223	return self::getValueOrFallback($siteBaseConfiguration, $keyToCheck, $fallbackKey);
191		}
192
193		/**
194		* Checks whether write connection is enabled.
195		* Language context properties have precedence over global settings.
196		*
197		* @param Site $typo3Site
198		* @param int $languageId
199		* @return bool
200		*/
201	213	protected static function writeConnectionIsEnabled(Site $typo3Site, int $languageId): bool
202		{
203	213	$rootPageUid = $typo3Site->getRootPageId();
204	213	if (isset(self::$languages[$rootPageUid][$languageId]) === false) {
205	7	self::$languages[$rootPageUid][$languageId] = $typo3Site->getLanguageById($languageId)->toArray();
206		}
207	213	$value = self::getValueOrFallback(self::$languages[$rootPageUid][$languageId], 'solr_use_write_connection', 'solr_use_write_connection');
208	213	if ($value !== null) {
209		return $value;
		0 ignored issues – show Bug Best Practice introduced 2021-03-12 11:55 UTC by Report Bug Copy Issue Report The expression `return $value` could return the type `string` which is incompatible with the type-hinted return `boolean`. Consider adding an additional type-check to rule them out. Loading history...
210		}
211
212	213	$siteBaseConfiguration = $typo3Site->getConfiguration();
213	213	$value = self::getValueOrFallback($siteBaseConfiguration, 'solr_use_write_connection', 'solr_use_write_connection');
214	213	if ($value !== null) {
215	211	return $value;
		0 ignored issues – show Bug Best Practice introduced 2021-03-12 11:55 UTC by Report Bug Copy Issue Report The expression `return $value` could return the type `string` which is incompatible with the type-hinted return `boolean`. Consider adding an additional type-check to rule them out. Loading history...
216		}
217	83	return false;
218		}
219
220		/**
221		* @param array $data
222		* @param string $keyToCheck
223		* @param string $fallbackKey
224		* @return string\|bool\|null
225		*/
226	225	protected static function getValueOrFallback(array $data, string $keyToCheck, string $fallbackKey)
227		{
228	225	$value = $data[$keyToCheck] ?? null;
229	225	if ($value === '0' \|\| $value === 0 \|\| !empty($value)) {
230	223	return self::evaluateConfigurationData($value);
231		}
232
233	224	return self::evaluateConfigurationData($data[$fallbackKey] ?? null);
234		}
235
236		/**
237		* Evaluate configuration data
238		*
239		* Setting boolean values via environment variables
240		* results in strings like 'false' that may be misinterpreted
241		* thus we check for boolean values in strings.
242		*
243		* @param string\|bool\|null $value
244		* @return string\|bool\|null
245		*/
246	225	protected static function evaluateConfigurationData($value)
247		{
248	225	if ($value === 'true') {
249	1	return true;
250		}
251	225	if ($value === 'false') {
252	1	return false;
253		}
254
255	225	return $value;
256		}
257		}
258

TYPO3-Solr / ext-solr

Pull Request — main (#3432)

SiteUtility::getSolrConnectionConfiguration() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like