Alex19pov31\BitrixHelper\Iblock\IblockPropertyTable

Complexity

Total Complexity

12

Size/Duplication

Total Lines	81
Duplicated Lines	0 %

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	getIblockId()	0	3	1
A	getTableName()	0	3	1
B	getMap()	0	55	9
A	__construct()	0	2	1

<?php

namespace Alex19pov31\BitrixHelper\Iblock;

use Bitrix\Iblock\PropertyTable;

The type Bitrix\Iblock\PropertyTable was not found. Maybe you did not declare it correctly or list all dependencies?

use Bitrix\Main\ORM\Data\DataManager;

The type Bitrix\Main\ORM\Data\DataManager was not found. Maybe you did not declare it correctly or list all dependencies?

use Bitrix\Main\ORM\Fields\IntegerField;

The type Bitrix\Main\ORM\Fields\IntegerField was not found. Maybe you did not declare it correctly or list all dependencies?

use Bitrix\Main\ORM\Fields\StringField;

The type Bitrix\Main\ORM\Fields\StringField was not found. Maybe you did not declare it correctly or list all dependencies?

use Bitrix\Sender\Connector\Filter\NumberField;

The type Bitrix\Sender\Connector\Filter\NumberField was not found. Maybe you did not declare it correctly or list all dependencies?

abstract class IblockPropertyTable extends DataManager
{
    const TTL = 180;

    private static $map;
    protected static $iblockCode;

    public function __construct(string $iblockCode) {
        $this->iblockCode = $iblockCode;
    }

    public static function getIblockId()
    {
        return getIblockId(static::$iblockCode, 180);
    }

    public static function getTableName()
    {
        return 'b_iblock_element_prop_s' . static::getIblockId();
    }

    /**
     * Карта свойств
     *
     * @return void
     */
    public static function getMap()
    {
        if (!is_null(static::$map['IBLOCK_CODE'])) {

Since $map is declared private, accessing it with static will lead to errors in possible sub-classes; you can either use self, or increase the visibility of $map to at least protected.

            return static::$map['IBLOCK_CODE'];
        }

        $propertyList = PropertyTable::getList([
            'filter' => [
                'IBLOCK_ID' => static::getIblockId(),
            ],
            'cache' => [
                'ttl' => static::TTL * 60,
            ],
        ])->fetchAll();

        $fields = [
            'IBLOCK_ELEMENT_ID' => new IntegerField(
                'IBLOCK_ELEMENT_ID',
                [
                    'title' => 'Идентификатор элемента инфоблока',
                ]
            ),
        ];
        foreach ($propertyList as $property) {
            $class = StringField::class;
            switch ($property['PROPERTY_TYPE']) {
                case PropertyTable::TYPE_STRING:
                    $class = StringField::class;
                    break;
                case PropertyTable::TYPE_NUMBER:
                    $class = NumberField::class;
                    break;
                case PropertyTable::TYPE_FILE:
                    $class = NumberField::class;
                    break;
                case PropertyTable::TYPE_ELEMENT:
                    $class = StringField::class;
                    break;
                case PropertyTable::TYPE_SECTION:
                    $class = StringField::class;
                    break;
                case PropertyTable::TYPE_LIST:
                    $class = StringField::class;
                    break;
            }

            $fields[$property['CODE']] = new $class(
                'PROPERTY_' . $property['ID'],
                [
                    'title' => $property['NAME'],
                ]
            );
        }

        return static::$map['IBLOCK_CODE'] = $fields;

The expression return static::map['IBLOCK_CODE'] = $fields returns the type array<string,Bitrix\Main\ORM\Fields\IntegerField> which is incompatible with the documented return type void.

    }
}